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.

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

Fix #5357 #5360

Merged
merged 1 commit into from Oct 19, 2018

Conversation

Projects
None yet
3 participants
@ahorek
Copy link
Contributor

ahorek commented Oct 12, 2018

#5357 introduced a bug

proc {|a| a }["sometext"].frozen?
without # frozen_string_literal: true it should return false

on the other hand hashes are frozen by default
{"sometext": ''}["sometext"] = always frozen

spec
ruby/spec#627

@kares kares added this to the JRuby 9.2.1.0 milestone Oct 12, 2018

@ahorek ahorek force-pushed the ahorek:fix_frozen branch 10 times, most recently from 6e86846 to 1add902 Oct 12, 2018

pavel

@ahorek ahorek force-pushed the ahorek:fix_frozen branch from 1add902 to 8a8e24b Oct 14, 2018

@headius headius merged commit 2572016 into jruby:master Oct 19, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.