Skip to content

Commit 8b862ba

Browse files
committed
Fix: invalid 'PythonNativeWrapper.isInstance' implementation.
1 parent 39116af commit 8b862ba

File tree

1 file changed

+2
-6
lines changed
  • graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext

1 file changed

+2
-6
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/NativeWrappers.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ public boolean isNative() {
7777
}
7878

7979
static boolean isInstance(TruffleObject o) {
80-
return o instanceof PythonObjectNativeWrapper || o instanceof TruffleObjectNativeWrapper;
80+
return o instanceof DynamicObjectNativeWrapper || o instanceof TruffleObjectNativeWrapper;
8181
}
8282

8383
@Override
@@ -120,10 +120,6 @@ public PythonAbstractObject getPythonObject() {
120120
return pythonObject;
121121
}
122122

123-
public static boolean isInstance(TruffleObject o) {
124-
return o instanceof PythonObjectNativeWrapper;
125-
}
126-
127123
public static PythonObjectNativeWrapper wrap(PythonAbstractObject obj, ConditionProfile noWrapperProfile) {
128124
// important: native wrappers are cached
129125
PythonObjectNativeWrapper nativeWrapper = obj.getNativeWrapper();
@@ -146,7 +142,7 @@ public String toString() {
146142
}
147143
}
148144

149-
abstract static class PrimitiveNativeWrapper extends DynamicObjectNativeWrapper {
145+
public abstract static class PrimitiveNativeWrapper extends DynamicObjectNativeWrapper {
150146

151147
private PythonObject materializedObject;
152148

0 commit comments

Comments
 (0)