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.

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

fix: XML::Reader XML::Attr garbage collection (backport to v1.13.x) #2602

Merged
merged 6 commits into from
Jul 22, 2022

Conversation

flavorjones
Copy link
Member

What problem is this PR intended to solve?

Closes #2598.

Backport of #2599 to v1.13.x for a patch release. See that issue for details.

which is now being used to implement Reader#attributes

fix: Reader#namespaces on JRuby now returns an empty hash when no
attributes are present (previously returned nil)
which will use rb_category_warning if it's available, and rb_warning
if it's not
This method will be removed in a future version
@flavorjones flavorjones changed the title Flavorjones fix reader node gc backport v1.13.x fix: XML::Reader XML::Attr garbage collection (backport to v1.13.x) Jul 22, 2022
@flavorjones flavorjones merged commit d6c4a2b into v1.13.x Jul 22, 2022
@flavorjones flavorjones deleted the flavorjones-fix-reader-node-gc_backport-v1.13.x branch July 22, 2022 19:57
@flavorjones flavorjones added topic/memory Segfaults, memory leaks, valgrind testing, etc. backport Backport of a PR to the current release branch labels Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backport of a PR to the current release branch topic/memory Segfaults, memory leaks, valgrind testing, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant