Skip to content

Conversation

@copybara-service
Copy link

@copybara-service copybara-service bot commented Jul 12, 2024

Enable editions support in j2objc plugin.

Since this is implemented using the C++ runtime, no significant changes were required. This was validated in three ways:

  1. Manual audit of the plugin, runtime, and generated code
  2. Codegen tests that verify all known edge cases produce identical gencode before and after a migration to edition 2023
  3. Migration of all j2objc test protos to edition 2023 passes existing tests

Since j2objc mirrors java-lite, all of the relevant feature information is embedded into the gencode using a custom format. Therefore, the runtime doesn't need to worry about feature resolution for reflection.

@copybara-service copybara-service bot force-pushed the test_651572153 branch 2 times, most recently from c7dc2ed to d4360e2 Compare July 16, 2024 02:01
@copybara-service copybara-service bot changed the title Enable editions support in j2cl plugin. Enable editions support in j2objc plugin. Jul 16, 2024
@copybara-service copybara-service bot force-pushed the test_651572153 branch 3 times, most recently from 9beb81e to 008f4b7 Compare July 16, 2024 03:28
Since this is implemented using the C++ runtime, no significant changes were required.  This was validated in three ways:
1) Manual audit of the plugin, runtime, and generated code
2) Codegen tests that verify all known edge cases produce identical gencode before and after a migration to edition 2023
3) Migration of all j2objc test protos to edition 2023 passes existing tests

Since j2objc mirrors java-lite, all of the relevant feature information is embedded into the gencode using a custom format.  Therefore, the runtime doesn't need to worry about feature resolution for reflection.

PiperOrigin-RevId: 652680827
@copybara-service copybara-service bot merged commit 058e7b3 into master Jul 16, 2024
@copybara-service copybara-service bot deleted the test_651572153 branch July 16, 2024 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant