Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: ReferenceNodes usage with mask set #683

Merged
merged 5 commits into from
Aug 29, 2023
Merged

fix: ReferenceNodes usage with mask set #683

merged 5 commits into from
Aug 29, 2023

Commits on Aug 29, 2023

  1. fix: fix ReferenceNodes usage with mask set

    @sruehl
    sruehl committed Aug 29, 2023
    Configuration menu
    Copy the full SHA
    27ba3b2 View commit details
    Browse the repository at this point in the history

  2. refactor: more efficient production of nodes from expanded node id

    @sruehl
    sruehl committed Aug 29, 2023
    Configuration menu
    Copy the full SHA
    5f8da7d View commit details
    Browse the repository at this point in the history

  3. fix: do a proper deep copy when using NewNodeIDFromExpandedNodeID

    @sruehl
    sruehl committed Aug 29, 2023
    Configuration menu
    Copy the full SHA
    2db34e0 View commit details
    Browse the repository at this point in the history

  4. test: add tests for NewNodeIDFromExpandedNodeID

    @sruehl
    sruehl committed Aug 29, 2023
    Configuration menu
    Copy the full SHA
    64de80d View commit details
    Browse the repository at this point in the history

  5. test: add tests for NewNodeIDFromExpandedNodeID for NewExpandedNodeID 

    this proves that the mask will be correct
    @sruehl
    sruehl committed Aug 29, 2023
    Configuration menu
    Copy the full SHA
    731d545 View commit details
    Browse the repository at this point in the history