[pull] master from ruby:master#880
Merged
pull[bot] merged 14 commits intoturkdevops:masterfrom Mar 24, 2026
Merged
Conversation
Currently there is just one very basic one. ruby/prism@5a184b8085
`::X &&= 1` for example emitted `on_int` twice ruby/prism@44d064c2bc
They may not be part of `on_param` but ripper still calls `on_ident` for them ruby/prism@fb5303f2b8
* The Loader API lives under java/api. * The current native endpoint for the Prism shared library lives under java/native. * The WASM build and binding lives under java/wasm. The libraries will be released together but can be developed and snapshotted independently. Users that copy the source from the previous java/ will want to grab both java/api/src/main/java and java/native/src/main/java contents. ruby/prism@03358d5bcf
This uses the JRuby rake-maven-plugin to generate the templates as part of the Maven build. The generated output for the Java templates will be under java/api/target/generated-sources/java. ruby/prism@b257151391
Callers should use the safer `RTYPEDDATA_GET_DATA`.
This helps distinguish an OutOfMemory (intended to be for running out of
code space) from a native frame needing too much stack space. On
protoboeuf, for example:
Top-2 compile error reasons (100.0% of total 5,798,926):
out_of_memory: 3,074,342 (53.0%)
native_stack_too_large: 2,724,584 (47.0%)
We were accidentally marking OutOfMemory when actually the problem was
that the native stack was too big. Slightly re-think emit APIs to make
this mistake harder to make.
Now protoboeuf looks like this:
Top-1 compile error reasons (100.0% of total 5,798,926):
native_stack_too_large: 5,798,926 (100.0%)
Too-complex shapes use hash tables for ivar storage, and rb_shape_get_iv_index() doesn't work for them (it asserts in debug builds). Without this check, the polymorphic getinstancevariable optimization incorrectly returns nil for ivars on too-complex objects. Let the fallthrough GetIvar handle these shapes instead.
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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )