OpenStruct with #frozen_string_literal: true
The current implementation of OpenStruct needs minimal changes in order to support frozen strings.
The mutable strings are only on inspect method.
IMO, we have two options:
a) leave the code as it is and just adding .dup where is needed
b) Change to a functional style (map + join) instead of flags + concat.
lib/ostruct.rb: Use frozen literals.
Patch adapted from Espartaco Palma. [GH-1714] [Bug #14000]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60406 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Updated by duerst (Martin Dürst) over 1 year ago
Thanks for looking at where we can use
I think it's good to try this, but
#frozen_string_literal: true should not be used as a cargo cult. So it would be good to know whether your patch leads to some/any speedups or not, or some other improvements.