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

JRuby fixes for 3.20.x #9714

Merged
merged 2 commits into from
Apr 2, 2022
Merged

JRuby fixes for 3.20.x #9714

merged 2 commits into from
Apr 2, 2022

Conversation

JasonLunn
Copy link
Contributor

@JasonLunn JasonLunn commented Mar 31, 2022

Cherry pick JRuby changes that fix NPE during encoding (#9507) and implements respond_to? (#9202) into 3.20.x.

All synthetic methods implemented by `method_missing` are now supported by `respond_to?`.

Fixes issue protocolbuffers#9202.

* Fix null pointer exceptions exposed by new regression tests.
* Fix clear_ on oneofs so that it is safe to call repeatedly and so that respond_to? does not depend on whether the oneof is currently cleared.
* Code cleanup: reenable more tests on JRuby.
* Align JRuby behavior with CRuby by throwing a RuntimeError when attempting to assign to a oneof.

(cherry picked from commit 8e7f936)
@JasonLunn JasonLunn added ruby release notes: yes jruby Issues unique to the JRuby interpreter labels Mar 31, 2022
@JasonLunn JasonLunn changed the title JRuby fixes for 3.20.1 JRuby fixes for 3.20.x Mar 31, 2022
@jorgbrown
Copy link
Contributor

Can you change the description to be more specific, i.e. something like

Merge JRuby fixes, "Fix NPE during encoding" and "Implement respond_to" into 3.20.x

@JasonLunn
Copy link
Contributor Author

Can you change the description to be more specific, i.e. something like

Merge JRuby fixes, "Fix NPE during encoding" and "Implement respond_to" into 3.20.x

Done.

@jorgbrown
Copy link
Contributor

One nit:

9507 is an issue number #9507 while 9677 is a pull request number #9677

Can you change "9507" to "9637"?

ps Sorry I didn't see this earlier but I thought the title at the top would reflect the change, as opposed to just the first line of the "Conversation" section.

@jorgbrown
Copy link
Contributor

LGTM

@jorgbrown jorgbrown merged commit e097b36 into protocolbuffers:3.20.x Apr 2, 2022
@JasonLunn JasonLunn mentioned this pull request Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jruby Issues unique to the JRuby interpreter ruby
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants