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

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;