Do not pass source bytes to Loader.java at all

* That way we are sure it's not retained by it.
* Source#bytes seems to have no usages.
* Arrays.binarySearch() works just fine if the value is > the last array element.

Author: eregon

java-wasm/src/main/java/org/jruby/parser/prism/wasm/Prism.java

@@ -83,7 +83,7 @@ public byte[] serialize(byte[] packedOptions, byte[] sourceBytes, int sourceLeng
     public ParseResult serializeParse(byte[] packedOptions, String source) {
         var sourceBytes = source.getBytes(StandardCharsets.ISO_8859_1);
         byte[] result = serialize(packedOptions, sourceBytes, sourceBytes.length);
-        return Loader.load(result, sourceBytes);
+        return Loader.load(result);
     }
 
     @Override

rakelib/serialization.rake

@@ -25,8 +25,7 @@ task "test:java_loader:internal" => :compile do
     puts
     puts path
     serialized = Prism.dump_file(path)
-    source_bytes = File.binread(path).unpack('c*')
-    parse_result = org.ruby_lang.prism.Loader.load(serialized.unpack('c*'), source_bytes)
+    parse_result = org.ruby_lang.prism.Loader.load(serialized.unpack('c*'))
     puts parse_result.value
   end
 end

templates/java/org/ruby_lang/prism/Loader.java.erb

@@ -15,8 +15,8 @@ import java.util.Locale;
 // @formatter:off
 public class Loader {
 
-    public static ParseResult load(byte[] serialized, byte[] sourceBytes) {
-        return new Loader(serialized).load(sourceBytes);
+    public static ParseResult load(byte[] serialized) {
+        return new Loader(serialized).load();
     }
 
     // Overridable methods
@@ -81,10 +81,8 @@ public class Loader {
         this.buffer = ByteBuffer.wrap(serialized).order(ByteOrder.nativeOrder());
     }
 
-    // We pass sourceBytes here and not in the constructor to avoid keeping
-    // the sourceBytes in memory unnecessarily with lazy DefNode's which hold on the Loader.
-    protected ParseResult load(byte[] sourceBytes) {
-        Nodes.Source source = new Nodes.Source(sourceBytes);
+    protected ParseResult load() {
+        Nodes.Source source = new Nodes.Source();
 
         expect((byte) 'P', "incorrect prism header");
         expect((byte) 'R', "incorrect prism header");

templates/java/org/ruby_lang/prism/Nodes.java.erb

@@ -47,12 +47,10 @@ public abstract class Nodes {
     }
 
     public static final class Source {
-        public final byte[] bytes;
         private int startLine = 1;
         private int[] lineOffsets = null;
 
-        Source(byte[] bytes) {
-          this.bytes = bytes;
+        Source() {
         }
 
         void setStartLine(int startLine) {
@@ -70,7 +68,6 @@ public abstract class Nodes {
 
         // 0-based
         public int findLine(int byteOffset) {
-            if (byteOffset >= bytes.length) byteOffset = bytes.length - 1;
             assert byteOffset >= 0 : byteOffset;
             int index = Arrays.binarySearch(lineOffsets, byteOffset);
             int line;