Skip to content

Commit e541b9d

Browse files
committed
PyEval_GetFrame should return a borrowed reference
1 parent 13d9266 commit e541b9d

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/cext/PythonCextCEvalBuiltins.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
import static com.oracle.graal.python.builtins.modules.cext.PythonCextBuiltins.CApiCallPath.Direct;
4545
import static com.oracle.graal.python.builtins.modules.cext.PythonCextBuiltins.CApiCallPath.Ignored;
4646
import static com.oracle.graal.python.builtins.objects.cext.capi.transitions.ArgDescriptor.Int;
47-
import static com.oracle.graal.python.builtins.objects.cext.capi.transitions.ArgDescriptor.PyFrameObjectTransfer;
47+
import static com.oracle.graal.python.builtins.objects.cext.capi.transitions.ArgDescriptor.PyFrameObjectBorrowed;
4848
import static com.oracle.graal.python.builtins.objects.cext.capi.transitions.ArgDescriptor.PyObject;
4949
import static com.oracle.graal.python.builtins.objects.cext.capi.transitions.ArgDescriptor.PyObjectBorrowed;
5050
import static com.oracle.graal.python.builtins.objects.cext.capi.transitions.ArgDescriptor.PyObjectConstPtr;
@@ -136,7 +136,7 @@ Object release(
136136
}
137137
}
138138

139-
@CApiBuiltin(ret = PyFrameObjectTransfer, args = {}, call = Direct)
139+
@CApiBuiltin(ret = PyFrameObjectBorrowed, args = {}, call = Direct)
140140
abstract static class PyEval_GetFrame extends CApiNullaryBuiltinNode {
141141
@Specialization
142142
Object getFrame(

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/capi/transitions/ArgDescriptor.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -220,6 +220,7 @@ public enum ArgDescriptor {
220220
PyFunctionObject(ArgBehavior.PyObject, "PyFunctionObject*"),
221221
PyFrameConstructor("PyFrameConstructor*"),
222222
PyFrameObject(ArgBehavior.PyObject, "PyFrameObject*"),
223+
PyFrameObjectBorrowed(ArgBehavior.PyObjectBorrowed, "PyFrameObject*"),
223224
PyFrameObjectTransfer(ArgBehavior.PyObject, "PyFrameObject*", true),
224225
_PyFrameEvalFunction("_PyFrameEvalFunction"),
225226
_PyInterpreterFrame("struct _PyInterpreterFrame*"),

0 commit comments

Comments
 (0)