[truffle] Avoid copying the backing byte[] from ByteList in String#by…

…tes.
jruby · Sep 17, 2015 · f3d3fb8 · f3d3fb8
1 parent 65416fb
commit f3d3fb8
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/core/StringNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/core/StringNodes.java
@@ -623,16 +623,17 @@ public BytesNode(RubyContext context, SourceSection sourceSection) {
         }
 
         @Specialization
-        public DynamicObject bytes(DynamicObject string) {
-            final byte[] bytes = Layouts.STRING.getByteList(string).bytes();
+        public DynamicObject bytes(VirtualFrame frame, DynamicObject string) {
+            final ByteList byteList = Layouts.STRING.getByteList(string);
+            final byte[] bytes = byteList.unsafeBytes();
 
-            final int[] store = new int[bytes.length];
+            final int[] store = new int[byteList.realSize()];
 
             for (int n = 0; n < store.length; n++) {
-                store[n] = ((int) bytes[n]) & 0xFF;
+                store[n] = ((int) bytes[n + byteList.begin()]) & 0xFF;
             }
 
-            return Layouts.ARRAY.createArray(getContext().getCoreLibrary().getArrayFactory(), store, bytes.length);
+            return Layouts.ARRAY.createArray(getContext().getCoreLibrary().getArrayFactory(), store, store.length);
         }
 
     }