[#81999] [Ruby trunk Bug#13737] "can't modify frozen String" when installing bundled gems — ko1@...
Issue #13737 has been updated by ko1 (Koichi Sasada).
4 messages
2017/07/11
[#82005] [Ruby trunk Bug#13737] "can't modify frozen String" when installing bundled gems — nobu@...
Issue #13737 has been updated by nobu (Nobuyoshi Nakada).
3 messages
2017/07/12
[#82102] Re: register_fstring_tainted:FL_TEST_RAW(str, RSTRING_FSTR) — Eric Wong <normalperson@...>
Koichi Sasada <ko1@atdot.net> wrote:
4 messages
2017/07/18
[#82151] [Ruby trunk Feature#13637] [PATCH] tool/runruby.rb: test with smallest possible machine stack — Rei.Odaira@...
Issue #13637 has been updated by ReiOdaira (Rei Odaira).
3 messages
2017/07/24
[ruby-core:82012] Re: [Ruby trunk Bug#13737] "can't modify frozen String" when installing bundled gems
From:
Eric Wong <normalperson@...>
Date:
2017-07-12 22:40:35 UTC
List:
ruby-core #82012
Eric Wong <normalperson@yhbt.net> wrote: > nobu@ruby-lang.org wrote: > > Or should `fstring_cmp` consider also tainted flags? > > I considered doing that, but it may not be optimal since I want > to share heap allocations with the non-tainted version via > rb_str_dup. Slightly better version with commit message + stronger test for sharing, and also using rb_str_resurrect instead of rb_str_dup: https://80x24.org/spew/20170712223226.20939-1-e@80x24.org/raw I will commit in a few days if no response. Also, rb_hash_bulk_insert should do the same and avoid blind calls to rb_str_new_frozen. This will be a separate, trivial (I hope) change to try rb_fstring_existing before rb_str_new_frozen. Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe> <http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>