fix __repr__ message for ReferenceType instances

Author: cosminbasca

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/WeakRefModuleBuiltins.java

@@ -48,6 +48,7 @@
 import com.oracle.graal.python.builtins.objects.function.PFunction;
 import com.oracle.graal.python.builtins.objects.object.PythonObject;
 import com.oracle.graal.python.builtins.objects.referencetype.PReferenceType;
+import com.oracle.graal.python.builtins.objects.type.PythonClass;
 import com.oracle.graal.python.nodes.function.PythonBuiltinNode;
 import com.oracle.graal.python.runtime.exception.PythonErrorType;
 import com.oracle.truffle.api.dsl.Fallback;
@@ -67,19 +68,18 @@ protected List<? extends NodeFactory<? extends PythonBuiltinNode>> getNodeFactor
     @Builtin(name = "ReferenceType", minNumOfArguments = 2, maxNumOfArguments = 3, constructsClass = PReferenceType.class)
     @GenerateNodeFactory
     public abstract static class ReferenceTypeNode extends PythonBuiltinNode {
-
         @Specialization
-        public PReferenceType refType(Object cls, PythonObject pythonObject, PNone none) {
-            return factory().createReferenceType(pythonObject, null);
+        public PReferenceType refType(PythonClass cls, PythonObject pythonObject, PNone none) {
+            return factory().createReferenceType(cls, pythonObject, null);
         }
 
         @Specialization
-        public PReferenceType refType(Object cls, PythonObject pythonObject, PFunction callback) {
-            return factory().createReferenceType(pythonObject, callback);
+        public PReferenceType refType(PythonClass cls, PythonObject pythonObject, PFunction callback) {
+            return factory().createReferenceType(cls, pythonObject, callback);
         }
 
         @Fallback
-        public PReferenceType refType(@SuppressWarnings("unused") Object cls, @SuppressWarnings("unused") Object object, @SuppressWarnings("unused") Object callback) {
+        public PReferenceType refType(Object cls, Object object, Object callback) {
             throw raise(PythonErrorType.TypeError, "cannot create weak reference to '%p' object", object);
         }
     }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/referencetype/ReferenceTypeBuiltins.java

@@ -122,9 +122,9 @@ public String repr(PReferenceType self,
             Object object = self.getObject();
             Object name = getNameNode.execute(object, __NAME__);
             if (name == PNone.NO_VALUE) {
-                return String.format("<weakref at %s; to '%s' at %s>", self.hashCode(), object, object.hashCode());
+                return String.format("<weakref at %s; to '%s' at %s>", self.hashCode(), object.hashCode(), object.hashCode());
             } else {
-                return String.format("<weakref at %s; to '%s' at %s (%s)>", self.hashCode(), object, object.hashCode(), name);
+                return String.format("<weakref at %s; to '%s' at %s (%s)>", self.hashCode(), name, object.hashCode(), object);
             }
         }
     }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/object/PythonObjectFactory.java

@@ -489,8 +489,12 @@ public PMappingproxy createMappingproxy(PythonClass cls, PythonObject object) {
         return trace(new PMappingproxy(cls, object));
     }
 
+    public PReferenceType createReferenceType(PythonClass cls, PythonObject object, PFunction callback) {
+        return trace(new PReferenceType(cls, object, callback));
+    }
+
     public PReferenceType createReferenceType(PythonObject object, PFunction callback) {
-        return trace(new PReferenceType(lookupClass(PythonBuiltinClassType.PReferenceType), object, callback));
+        return createReferenceType(lookupClass(PythonBuiltinClassType.PReferenceType), object, callback);
     }
 
     /*