Implement respond_to? in RubyMessage by JasonLunn · Pull Request #9677 · protocolbuffers/protobuf (original) (raw)
…implemented by method_missing return true. Add regression tests.
Fix null pointer exceptions exposed by new unit tests.
Code cleanup: remove test code that was skipping several unit tests only on JRuby.
Fixes issue protocolbuffers#9202.
…t respond_to? does not depend on whether the oneof is currently cleared.
…t respond_to? does not depend on whether the oneof is currently cleared.
Code cleanup: reenable more tests on JRuby.
…pting to assign to a oneof.
Code cleanup: Eliminate noisy warnings when running tests by
- removing or using unused variables
- wrapping ambiguous parameters in parentheses
JasonLunn added a commit to JasonLunn/protobuf that referenced this pull request
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)
mkruskal-google added a commit that referenced this pull request
- Fix NPE during encoding and add regression test for issue 9507.
(cherry picked from commit 58e320a)
- Implement
respond_to?in RubyMessage (#9677)
All synthetic methods implemented by method_missing are now supported by respond_to?.
Fixes issue #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)
Update protobuf version
Merge pull request #9727 from mlocati/build-packaged-php-extension
Fix building packaged PHP extension
(cherry picked from commit 7f9901c)
Update protobuf version
Update changelogs for 3.20.1-rc1
Co-authored-by: Jason Lunn jason.lunn@gmail.com Co-authored-by: Jorg Brown jorg.brown@gmail.com
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})