Fix repr of bytearray with native storage

Author: msimacek

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/bytes/BytesBuiltins.java

@@ -444,33 +444,51 @@ public abstract static class RMulNode extends MulNode {
     @GenerateNodeFactory
     public abstract static class ReprNode extends PythonUnaryBuiltinNode {
         @TruffleBoundary
-        private static final StringBuilder newStringBuilder() {
-            return new StringBuilder("b'");
+        private static StringBuilder newStringBuilder() {
+            return new StringBuilder();
         }
 
         @TruffleBoundary
-        private static final String sbFinishAndToString(StringBuilder sb) {
-            sb.append("'");
+        private static String sbToString(StringBuilder sb) {
             return sb.toString();
         }
 
+        @TruffleBoundary
+        private static void sbAppend(StringBuilder sb, String s) {
+            sb.append(s);
+        }
+
         @Specialization
         public Object repr(VirtualFrame frame, PBytes self,
                         @Cached SequenceStorageNodes.LenNode lenNode,
                         @Cached SequenceStorageNodes.GetItemNode getItemNode) {
             SequenceStorage store = self.getSequenceStorage();
             int len = lenNode.execute(store);
             StringBuilder sb = newStringBuilder();
+            sbAppend(sb, "b'");
             for (int i = 0; i < len; i++) {
                 BytesUtils.byteRepr(sb, (byte) getItemNode.executeInt(frame, store, i));
             }
-            return sbFinishAndToString(sb);
+            sbAppend(sb, "'");
+            return sbToString(sb);
         }
 
         @Specialization
-        public Object repr(PByteArray self, @Cached("create()") TypeNodes.GetNameNode getNameNode) {
+        public Object repr(VirtualFrame frame, PByteArray self,
+                        @Cached("create()") TypeNodes.GetNameNode getNameNode,
+                        @Cached SequenceStorageNodes.LenNode lenNode,
+                        @Cached SequenceStorageNodes.GetItemNode getItemNode) {
             String typeName = getNameNode.execute(self.getLazyPythonClass());
-            return self.formatByteArray(typeName);
+            SequenceStorage store = self.getSequenceStorage();
+            int len = lenNode.execute(store);
+            StringBuilder sb = newStringBuilder();
+            sb.append(typeName);
+            sb.append("(b'");
+            for (int i = 0; i < len; i++) {
+                BytesUtils.byteRepr(sb, (byte) getItemNode.executeInt(frame, store, i));
+            }
+            sb.append("')");
+            return sbToString(sb);
         }
     }