[ruby/stringio] Remove special handling of chilled strings

[Feature #20205]

Followup: https://github.com/ruby/stringio/pull/94

They no longer need to be special cases. If StringIO end up
mutating a chilled string, a warning will be emitted.

https://github.com/ruby/stringio/commit/dc62d65449
This commit is contained in:
Jean Boussier 2024-05-30 14:41:52 +02:00 коммит произвёл git
Родитель 78bfde5d9f
Коммит 15501e13d7
2 изменённых файлов: 3 добавлений и 12 удалений

Просмотреть файл

@ -51,18 +51,11 @@ static long strio_write(VALUE self, VALUE str);
#define IS_STRIO(obj) (rb_typeddata_is_kind_of((obj), &strio_data_type))
#define error_inval(msg) (rb_syserr_fail(EINVAL, msg))
#define get_enc(ptr) ((ptr)->enc ? (ptr)->enc : !NIL_P((ptr)->string) ? rb_enc_get((ptr)->string) : NULL)
#ifndef HAVE_RB_STR_CHILLED_P
static bool
rb_str_chilled_p(VALUE str)
{
return false;
}
#endif
static bool
readonly_string_p(VALUE string)
{
return OBJ_FROZEN_RAW(string) && !rb_str_chilled_p(string);
return OBJ_FROZEN_RAW(string);
}
static struct StringIO *
@ -184,9 +177,6 @@ check_modifiable(struct StringIO *ptr)
if (NIL_P(ptr->string)) {
/* Null device StringIO */
}
else if (rb_str_chilled_p(ptr->string)) {
rb_str_modify(ptr->string);
}
else if (OBJ_FROZEN_RAW(ptr->string)) {
rb_raise(rb_eIOError, "not modifiable string");
}

Просмотреть файл

@ -990,7 +990,8 @@ class TestStringIO < Test::Unit::TestCase
assert_predicate(s.string, :ascii_only?)
end
if eval(%{ "test".frozen? && !"test".equal?("test") }) # Ruby 3.4+ chilled strings
require "objspace"
if ObjectSpace.respond_to?(:dump) && ObjectSpace.dump(eval(%{"test"})).include?('"chilled":true') # Ruby 3.4+ chilled strings
def test_chilled_string
chilled_string = eval(%{""})
io = StringIO.new(chilled_string)