diff options
| author | Jean Boussier <byroot@ruby-lang.org> | 2023-11-08 14:05:52 +0100 |
|---|---|---|
| committer | Jean Boussier <jean.boussier@gmail.com> | 2023-11-09 12:38:10 +0100 |
| commit | ea1b1ea1aa98bc9488564ef18aa4032aa1cb5536 (patch) | |
| tree | 8e0428d9ddf0cf765bc06538adda3b7a98ccb0be /test/ruby | |
| parent | 0a7e620a36a74c4fc604f9068fb839658678f96c (diff) | |
String#force_encoding don't clear coderange if encoding is unchanged
Some code out there blind calls `force_encoding` without checking
what the original encoding was, which clears the coderange uselessly.
If the String is big, it can be a rather costly mistake.
For instance the `rack-utf8_sanitizer` gem does this on request
bodies.
Diffstat (limited to 'test/ruby')
| -rw-r--r-- | test/ruby/test_process.rb | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/test/ruby/test_process.rb b/test/ruby/test_process.rb index 47228d35e6..0416b20176 100644 --- a/test/ruby/test_process.rb +++ b/test/ruby/test_process.rb @@ -2747,6 +2747,7 @@ EOS require 'objspace' begin; obj = "a" * 12 + obj.force_encoding(Encoding::UTF_16LE) obj.force_encoding(Encoding::BINARY) assert_include(ObjectSpace.dump(obj), '"coderange":"unknown"') Process.warmup |