Author: Ronan Lamy <[email protected]>
Branch: multiphase
Changeset: r92289:8de4e0b09ce1
Date: 2017-08-31 17:23 +0100
http://bitbucket.org/pypy/pypy/changeset/8de4e0b09ce1/

Log:    disable _testmultiphase.call_state_registration_func()

diff --git a/lib-python/3/test/test_importlib/extension/test_loader.py 
b/lib-python/3/test/test_importlib/extension/test_loader.py
--- a/lib-python/3/test/test_importlib/extension/test_loader.py
+++ b/lib-python/3/test/test_importlib/extension/test_loader.py
@@ -146,7 +146,8 @@
             importlib.reload(module)
             self.assertIs(ex_class, module.Example)
 
-    def test_try_registration(self):
+    # XXX: PyPy doesn't support the PyState_* functions yet
+    def XXXtest_try_registration(self):
         '''Assert that the PyState_{Find,Add,Remove}Module C API doesn't 
work'''
         module = self.load_module()
         with self.subTest('PyState_FindModule'):
diff --git a/lib_pypy/_testmultiphase.c b/lib_pypy/_testmultiphase.c
--- a/lib_pypy/_testmultiphase.c
+++ b/lib_pypy/_testmultiphase.c
@@ -119,43 +119,43 @@
 
 /* Test that PyState registration fails  */
 
-PyDoc_STRVAR(call_state_registration_func_doc,
-"register_state(0): call PyState_FindModule()\n\
-register_state(1): call PyState_AddModule()\n\
-register_state(2): call PyState_RemoveModule()");
-
-static PyObject *
-call_state_registration_func(PyObject *mod, PyObject *args)
-{
-    int i, ret;
-    PyModuleDef *def = PyModule_GetDef(mod);
-    if (def == NULL) {
-        return NULL;
-    }
-    if (!PyArg_ParseTuple(args, "i:call_state_registration_func", &i))
-        return NULL;
-    switch (i) {
-        case 0:
-            mod = PyState_FindModule(def);
-            if (mod == NULL) {
-                Py_RETURN_NONE;
-            }
-            return mod;
-        case 1:
-            ret = PyState_AddModule(mod, def);
-            if (ret != 0) {
-                return NULL;
-            }
-            break;
-        case 2:
-            ret = PyState_RemoveModule(def);
-            if (ret != 0) {
-                return NULL;
-            }
-            break;
-    }
-    Py_RETURN_NONE;
-}
+//PyDoc_STRVAR(call_state_registration_func_doc,
+//"register_state(0): call PyState_FindModule()\n\
+//register_state(1): call PyState_AddModule()\n\
+//register_state(2): call PyState_RemoveModule()");
+//
+//static PyObject *
+//call_state_registration_func(PyObject *mod, PyObject *args)
+//{
+//    int i, ret;
+//    PyModuleDef *def = PyModule_GetDef(mod);
+//    if (def == NULL) {
+//        return NULL;
+//    }
+//    if (!PyArg_ParseTuple(args, "i:call_state_registration_func", &i))
+//        return NULL;
+//    switch (i) {
+//        case 0:
+//            mod = PyState_FindModule(def);
+//            if (mod == NULL) {
+//                Py_RETURN_NONE;
+//            }
+//            return mod;
+//        case 1:
+//            ret = PyState_AddModule(mod, def);
+//            if (ret != 0) {
+//                return NULL;
+//            }
+//            break;
+//        case 2:
+//            ret = PyState_RemoveModule(def);
+//            if (ret != 0) {
+//                return NULL;
+//            }
+//            break;
+//    }
+//    Py_RETURN_NONE;
+//}
 
 
 static PyType_Slot Str_Type_slots[] = {
@@ -174,8 +174,8 @@
 static PyMethodDef testexport_methods[] = {
     {"foo",             testexport_foo,         METH_VARARGS,
         testexport_foo_doc},
-    {"call_state_registration_func",  call_state_registration_func,
-        METH_VARARGS, call_state_registration_func_doc},
+//    {"call_state_registration_func",  call_state_registration_func,
+//        METH_VARARGS, call_state_registration_func_doc},
     {NULL,              NULL}           /* sentinel */
 };
 
_______________________________________________
pypy-commit mailing list
[email protected]
https://mail.python.org/mailman/listinfo/pypy-commit

Reply via email to