]> granicus.if.org Git - python/commitdiff
Remove _PyArg_NoStackKeywords(). (#2641)
authorSerhiy Storchaka <storchaka@gmail.com>
Mon, 10 Jul 2017 08:25:34 +0000 (11:25 +0300)
committerVictor Stinner <victor.stinner@gmail.com>
Mon, 10 Jul 2017 08:25:34 +0000 (10:25 +0200)
Include/modsupport.h
Modules/_hashopenssl.c
Python/getargs.c

index 853860f4d27e8d7cdd771dbc7b7df57efb8d2d62..8c7cf39d9a3ea941019dff4481fb5feca6063f81 100644 (file)
@@ -60,12 +60,9 @@ PyAPI_FUNC(int) _PyArg_UnpackStack(
     ...);
 
 PyAPI_FUNC(int) _PyArg_NoKeywords(const char *funcname, PyObject *kwargs);
-PyAPI_FUNC(int) _PyArg_NoStackKeywords(const char *funcname, PyObject *kwnames);
 PyAPI_FUNC(int) _PyArg_NoPositional(const char *funcname, PyObject *args);
 #define _PyArg_NoKeywords(funcname, kwargs) \
     ((kwargs) == NULL || _PyArg_NoKeywords((funcname), (kwargs)))
-#define _PyArg_NoStackKeywords(funcname, kwnames) \
-    ((kwnames) == NULL || _PyArg_NoStackKeywords((funcname), (kwnames)))
 #define _PyArg_NoPositional(funcname, args) \
     ((args) == NULL || _PyArg_NoPositional((funcname), (args)))
 
index 7ae7ea5be54b099ba352daf1f3ad7fd914627b38..037fa4e2e99af9ec8693dcbc40235c0e0b191727 100644 (file)
@@ -925,15 +925,11 @@ generate_hash_name_list(void)
  */
 #define GEN_CONSTRUCTOR(NAME)  \
     static PyObject * \
-    EVP_new_ ## NAME (PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames) \
+    EVP_new_ ## NAME (PyObject *self, PyObject **args, Py_ssize_t nargs) \
     { \
         PyObject *data_obj = NULL; \
         Py_buffer view = { 0 }; \
         PyObject *ret_obj; \
-     \
-        if (!_PyArg_NoStackKeywords(#NAME, kwnames)) { \
-            return NULL; \
-        } \
      \
         if (!_PyArg_ParseStack(args, nargs, "|O:" #NAME , &data_obj)) { \
             return NULL; \
@@ -967,7 +963,7 @@ generate_hash_name_list(void)
 
 /* a PyMethodDef structure for the constructor */
 #define CONSTRUCTOR_METH_DEF(NAME)  \
-    {"openssl_" #NAME, (PyCFunction)EVP_new_ ## NAME, METH_FASTCALL | METH_KEYWORDS, \
+    {"openssl_" #NAME, (PyCFunction)EVP_new_ ## NAME, METH_FASTCALL, \
         PyDoc_STR("Returns a " #NAME \
                   " hash object; optionally initialized with a string") \
     }
index f555870f7e0a79683840f9ad9db011806a372939..c61d9451896aa7f4cbb990b61e971f384338ace7 100644 (file)
@@ -2483,7 +2483,6 @@ _PyArg_UnpackStack(PyObject **args, Py_ssize_t nargs, const char *name,
 
 
 #undef _PyArg_NoKeywords
-#undef _PyArg_NoStackKeywords
 #undef _PyArg_NoPositional
 
 /* For type constructors that don't take keyword args
@@ -2511,23 +2510,6 @@ _PyArg_NoKeywords(const char *funcname, PyObject *kwargs)
 }
 
 
-int
-_PyArg_NoStackKeywords(const char *funcname, PyObject *kwnames)
-{
-    if (kwnames == NULL) {
-        return 1;
-    }
-    assert(PyTuple_CheckExact(kwnames));
-    if (PyTuple_GET_SIZE(kwnames) == 0) {
-        return 1;
-    }
-
-    PyErr_Format(PyExc_TypeError, "%.200s() takes no keyword arguments",
-                    funcname);
-    return 0;
-}
-
-
 int
 _PyArg_NoPositional(const char *funcname, PyObject *args)
 {