Move logging last in PythonObjectReference ctor

fangerer · fangerer · commit 298f28679386 · 2024-03-06T07:39:05.000+01:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/capi/transitions/CApiTransitions.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/capi/transitions/CApiTransitions.java
@@ -216,11 +216,11 @@ private PythonObjectReference(HandleContext handleContext, PythonNativeWrapper r
             super(handleContext, referent);
             this.pointer = pointer;
             this.strongReference = strong ? referent : null;
+            referent.ref = this;
+            this.handleTableIndex = handleTableIndex;
             if (LOGGER.isLoggable(Level.FINE)) {
                 LOGGER.fine(PythonUtils.formatJString("new %s", toString()));
             }
-            referent.ref = this;
-            this.handleTableIndex = handleTableIndex;
         }
 
         static PythonObjectReference create(HandleContext handleContext, PythonAbstractObjectNativeWrapper referent, boolean strong, long pointer, int idx) {

Original file line number	Diff line number	Diff line change
`@@ -216,11 +216,11 @@ private PythonObjectReference(HandleContext handleContext, PythonNativeWrapper r`
`216`	`216`	`super(handleContext, referent);`
`217`	`217`	`this.pointer = pointer;`
`218`	`218`	`this.strongReference = strong ? referent : null;`
	`219`	`+ referent.ref = this;`
	`220`	`+ this.handleTableIndex = handleTableIndex;`
`219`	`221`	`if (LOGGER.isLoggable(Level.FINE)) {`
`220`	`222`	`LOGGER.fine(PythonUtils.formatJString("new %s", toString()));`
`221`	`223`	`}`
`222`		`- referent.ref = this;`
`223`		`- this.handleTableIndex = handleTableIndex;`
`224`	`224`	`}`
`225`	`225`
`226`	`226`	`static PythonObjectReference create(HandleContext handleContext, PythonAbstractObjectNativeWrapper referent, boolean strong, long pointer, int idx) {`