Skip to content

More frozen string tweaks#7928

Merged
headius merged 2 commits intojruby:masterfrom
headius:more_frozen_string_tweaks
Sep 8, 2023
Merged

More frozen string tweaks#7928
headius merged 2 commits intojruby:masterfrom
headius:more_frozen_string_tweaks

Conversation

@headius
Copy link
Member

@headius headius commented Sep 8, 2023

It wasn't reporting itself as frozen, leading to #7924.

Fixes #7924 and includes a spec.

No flag was set and nothing was overridden, so it did not appear
frozen via isFrozen checks. This caused jruby#7924.

The fix here both sets that flag and also overrides isFrozen to
always be true.

Fixes jruby#7924
@headius headius added this to the JRuby 9.4.4.0 milestone Sep 8, 2023
@headius headius merged commit e55f5a5 into jruby:master Sep 8, 2023
@headius headius deleted the more_frozen_string_tweaks branch September 8, 2023 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(master) frozen string errors with --debug

1 participant