Issue #15241 has been updated by jeremyevans0 (Jeremy Evans).

Status changed from Open to Rejected

I don't think that this should be considered a bug.  Going from immutable to mutable should not break things, while going from mutable to immutable can.  It is true that the using `String#+` can decrease performance slightly, but that should not be considered a bug.

----------------------------------------
Bug #15241: net/pop fix to use mutable strings was too eager
https://bugs.ruby-lang.org/issues/15241#change-81211

* Author: eventualbuddha (Brian Donovan)
* Status: Rejected
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 2.5.2
* Backport: 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: UNKNOWN
----------------------------------------
See [[https://github.com/ruby/ruby/commit/3993fbb5f6bfdae0bce040988d7e2dd632247cdc#commitcomment-30987842|this thread on Github]]. I believe the changes made in Backport #14416 for 2.5.2 (on both trunk and 2.5.x) over-eagerly applied mutability to literal strings. In particular, the strings returned by `inspect` have no real need to be mutable as far as I can tell.



-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>