[GR-17457] Update StringIO position correctly after reading multi-byte characters.

PullRequest: truffleruby/3421

Author: aardvark179

CHANGELOG.md

@@ -6,6 +6,7 @@ New features:
 
 Bug fixes:
 
+* Fix `StringIO` to set position correctly after reading multi-byte characters (#2207, @aardvark179).
 
 Compatibility:
 

lib/truffle/stringio.rb

@@ -211,7 +211,7 @@ def each_byte
     d = @__data__
     string = d.string
 
-    while d.pos < string.length
+    while d.pos < string.bytesize
       check_readable
       byte = string.getbyte d.pos
       d.pos += 1
@@ -468,7 +468,7 @@ def read(length = nil, buffer = nil)
       buffer.replace str if buffer
     end
 
-    d.pos += str.length
+    d.pos += str.bytesize
     str
   end
 

spec/ruby/library/stringio/shared/read.rb

@@ -89,6 +89,12 @@
     @io.send(@method)
     @io.pos.should eql(7)
   end
+
+  it "correctly update the current position in bytes when multi-byte characters are used" do
+    @io.print("example\u03A3") # Overwrite the original string with 8 characters containing 9 bytes.
+    @io.send(@method)
+    @io.pos.should eql(9)
+  end
 end
 
 describe :stringio_read_nil, shared: true do