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: namespace nodes behave with compaction #2658

Merged
merged 6 commits into from
Oct 16, 2022
Merged

fix: namespace nodes behave with compaction #2658

merged 6 commits into from
Oct 16, 2022

Commits on Oct 16, 2022

  1. dev: remove the DEBUG macro 

    I haven't used it in years, TBH, and what it does is completely
insufficient to debug memory issues in modern Ruby anyway.
    @flavorjones
    flavorjones committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    afc6e94 View commit details
    Browse the repository at this point in the history

  2. cleanup: xml_node.c follows Ruby style for rb_data_type_t

    @flavorjones
    flavorjones committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    e68a17f View commit details
    Browse the repository at this point in the history

  3. test: small improvements to memory leak test suite 

    Related to #1603
    @flavorjones
    flavorjones committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    65a28e3 View commit details
    Browse the repository at this point in the history

  4. test: repro namespace_scopes compaction issue 

    Co-Authored-By: Matt Valentine-House <[email protected]>
Co-Authored-By: Peter Zhu <[email protected]>
    @flavorjones @eightbitraptor @peterzhu2118
    3 people committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    cf14265 View commit details
    Browse the repository at this point in the history

  5. fix: namespace nodes behave properly when compacted

    @flavorjones
    flavorjones committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    96452fd View commit details
    Browse the repository at this point in the history

  6. fix: Document#remove_namespaces! use-after-free bug

    @flavorjones
    flavorjones committed Oct 16, 2022
    Configuration menu
    Copy the full SHA
    13c8aaa View commit details
    Browse the repository at this point in the history