Skip to content

Commit db0e895

Browse files
committed
make singleton test for Py_True/Py_False a little simpler
1 parent 5737203 commit db0e895

File tree

1 file changed

+8
-8
lines changed
  • graalpython/com.oracle.graal.python.test/src/tests/cpyext

1 file changed

+8
-8
lines changed

graalpython/com.oracle.graal.python.test/src/tests/cpyext/test_bool.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -64,22 +64,22 @@ def compile_module(self, name):
6464
test_PyBools_areSingleton = CPyExtFunction(
6565
lambda args: 1,
6666
lambda: (
67-
("True",),
68-
("False",),
67+
(1,),
68+
(0,),
6969
),
7070
callfunction="CheckPyTrue",
7171
code="""
72-
static int CheckPyTrue(const char* str) {
73-
if (!strcmp(str, "True")) {
74-
return PyRun_StringFlags("True", Py_eval_input, PyDict_New(), PyDict_New(), NULL) == Py_True;
72+
static int CheckPyTrue(int flag) {
73+
if (flag) {
74+
return PyBool_FromLong(flag) == Py_True;
7575
} else {
76-
return PyRun_StringFlags("False", Py_eval_input, PyDict_New(), PyDict_New(), NULL) == Py_False;
76+
return PyBool_FromLong(flag) == Py_False;
7777
}
7878
}
7979
""",
8080
resultspec="i",
81-
argspec="s",
82-
arguments=["const char* source"],
81+
argspec="i",
82+
arguments=["int flag"],
8383
)
8484

8585
test_PyBool_Check = CPyExtFunction(

0 commit comments

Comments
 (0)