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

Add support for options in CRuby, JRuby and FFI #14594

Closed
wants to merge 22 commits into from
Closed

Add support for options in CRuby, JRuby and FFI #14594

wants to merge 22 commits into from

Commits on May 16, 2023

  1. Add support for descriptor options in ruby interface

    @jsteinberg
    jsteinberg committed May 16, 2023
    Configuration menu
    Copy the full SHA
    d5ed3dd View commit details
    Browse the repository at this point in the history

Commits on May 17, 2023

  1. DRY converting serialized options into ruby object

    @jsteinberg
    jsteinberg committed May 17, 2023
    Configuration menu
    Copy the full SHA
    71f7226 View commit details
    Browse the repository at this point in the history

  2. make serialized_options methods private

    @jsteinberg
    jsteinberg committed May 17, 2023
    Configuration menu
    Copy the full SHA
    04758ed View commit details
    Browse the repository at this point in the history

Commits on May 18, 2023

  1. Options messages should recursively freeze (deep_freeze)

    @jsteinberg
    jsteinberg committed May 18, 2023
    Configuration menu
    Copy the full SHA
    188f456 View commit details
    Browse the repository at this point in the history

  2. Message internal_deep_freeze now recurses over repeated/map value fields

    @jsteinberg
    jsteinberg committed May 18, 2023
    Configuration menu
    Copy the full SHA
    730fb06 View commit details
    Browse the repository at this point in the history

Commits on May 22, 2023

  1. Fix spelling issue (Deeep -> Deep)

    @jsteinberg
    jsteinberg committed May 22, 2023
    Configuration menu
    Copy the full SHA
    10b3e2b View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2023

  1. Merge remote-tracking branch 'jsteinberg/add-support-for-options-in-r… 

    …uby' into add-support-for-options-in-ruby
    @JasonLunn
    JasonLunn committed Nov 1, 2023
    Configuration menu
    Copy the full SHA
    018f8a7 View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2023

  1. Implement serialized_options and for JRuby Descriptor classes and `… 

    …internal_deep_freeze` for JRuby message classes.
    @JasonLunn
    JasonLunn committed Nov 2, 2023
    Configuration menu
    Copy the full SHA
    eb67c03 View commit details
    Browse the repository at this point in the history

  2. Implement serialized_options and for FFI Descriptor classes and `in… 

    …ternal_deep_freeze` for FFI message classes.
    @JasonLunn
    JasonLunn committed Nov 2, 2023
    Configuration menu
    Copy the full SHA
    8295112 View commit details
    Browse the repository at this point in the history

Commits on Nov 7, 2023

  1. Move implementation of options from Ruby to C. 

    Don't expose `internal_deep_freeze`, even as a private method.
    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    887ab6b View commit details
    Browse the repository at this point in the history

  2. Fix FFI implementation of options

    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    7830f55 View commit details
    Browse the repository at this point in the history

  3. Fix test assertions that swapped expected and actual by using ass… 

    …ertion specific to class instance.
    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    33f6236 View commit details
    Browse the repository at this point in the history

  4. Refactor JRuby implementation of serialized_options into options. 

    Don't expose `internal_deep_freeze` under JRuby, even as private method.
    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    5b52c3a View commit details
    Browse the repository at this point in the history

  5. Don't expose internal_deep_freeze under CRuby, even as private method.

    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    7ce6d7f View commit details
    Browse the repository at this point in the history

  6. Remove extraneous import.

    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    73dfef7 View commit details
    Browse the repository at this point in the history

  7. Fix calls to `*_internal_deep_freeze

    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    957654f View commit details
    Browse the repository at this point in the history

  8. Remove extraneous import.

    @JasonLunn
    JasonLunn committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    1c86a90 View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2023

  1. Implement feedback from PR review: 

    * Pass arena to glue functions.
* Memoize `options` accessor.
* Remove use of VLA.
    @JasonLunn
    JasonLunn committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    6a230b1 View commit details
    Browse the repository at this point in the history

  2. Improve tests by: 

    * Adding test coverage for `FieldOptions`
* Handle case where fields are not materialized until after `freeze`
    @JasonLunn
    JasonLunn committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    1f2c8ba View commit details
    Browse the repository at this point in the history

  3. Add options to FieldDescriptors under JRuby.

    @JasonLunn
    JasonLunn committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    3beac3d View commit details
    Browse the repository at this point in the history

  4. Fix imports.

    @JasonLunn
    JasonLunn committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    34b0dea View commit details
    Browse the repository at this point in the history

  5. Simplify deep freeze logic.

    @JasonLunn
    JasonLunn committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    16cc9e3 View commit details
    Browse the repository at this point in the history