Additional fixes for chomp with nil sep

See feaa2ec
ruby · Aug 14, 2023 · 64b2771 · 64b2771
1 parent 8a579bb
commit 64b2771
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/ext/java/org/jruby/ext/stringio/StringIO.java b/ext/java/org/jruby/ext/stringio/StringIO.java
@@ -599,6 +599,8 @@ public IRubyObject getline(ThreadContext context, StringIO self, IRubyObject rs,
                 return RubyString.newEmptyString(context.runtime, self.getEncoding());
             }
 
+            if (rs.isNil()) chomp = false;
+
             IRubyObject result = self.getline(context, rs, limit, chomp);
 
             context.setLastLine(result);
@@ -616,6 +618,8 @@ public StringIO getline(ThreadContext context, StringIO self, IRubyObject rs, in
                 throw context.runtime.newArgumentError("invalid limit: 0 for each_line");
             }
 
+            if (rs.isNil()) chomp = false;
+
             while (!(line = self.getline(context, rs, limit, chomp)).isNil()) {
                 block.yieldSpecific(context, line);
             }
@@ -634,6 +638,8 @@ public RubyArray getline(ThreadContext context, StringIO self, IRubyObject rs, i
                 throw context.runtime.newArgumentError("invalid limit: 0 for readlines");
             }
 
+            if (rs.isNil()) chomp = false;
+
             while (!(line = self.getline(context, rs, limit, chomp)).isNil()) {
                 ary.append(line);
             }