diff -r 777256817c1b Lib/test/test_exceptions.py --- a/Lib/test/test_exceptions.py Thu Aug 04 12:14:51 2016 -0500 +++ b/Lib/test/test_exceptions.py Thu Aug 04 21:52:07 2016 +0300 @@ -1096,6 +1096,19 @@ class ImportErrorTests(unittest.TestCase self.assertEqual(exc.name, 'somename') self.assertEqual(exc.path, 'somepath') + msg = "'invalid' is an invalid keyword argument for this function" + with self.assertRaisesRegex(TypeError, msg): + ImportError('test', invalid='keyword') + + with self.assertRaisesRegex(TypeError, msg): + ImportError('test', name='name', invalid='keyword') + + with self.assertRaisesRegex(TypeError, msg): + ImportError('test', path='path', invalid='keyword') + + with self.assertRaisesRegex(TypeError, msg): + ImportError(invalid='keyword') + def test_non_str_argument(self): # Issue #15778 with check_warnings(('', BytesWarning), quiet=True): diff -r 777256817c1b Objects/exceptions.c --- a/Objects/exceptions.c Thu Aug 04 12:14:51 2016 -0500 +++ b/Objects/exceptions.c Thu Aug 04 21:52:07 2016 +0300 @@ -612,36 +612,36 @@ SimpleExtendsException(PyExc_BaseExcepti static int ImportError_init(PyImportErrorObject *self, PyObject *args, PyObject *kwds) { + static char *kwlist[] = {"name", "path", 0}; + PyObject *empty_tuple; PyObject *msg = NULL; PyObject *name = NULL; PyObject *path = NULL; -/* Macro replacement doesn't allow ## to start the first line of a macro, - so we move the assignment and NULL check into the if-statement. */ -#define GET_KWD(kwd) { \ - kwd = PyDict_GetItemString(kwds, #kwd); \ - if (kwd) { \ - Py_INCREF(kwd); \ - Py_XSETREF(self->kwd, kwd); \ - if (PyDict_DelItemString(kwds, #kwd)) \ - return -1; \ - } \ + if (BaseException_init((PyBaseExceptionObject *)self, args, NULL) == -1) + return -1; + + empty_tuple = PyTuple_New(0); + if (!empty_tuple) + return -1; + if (!PyArg_ParseTupleAndKeywords(empty_tuple, kwds, "|$OO:ImportError", kwlist, + &name, &path)) + return -1; + Py_DECREF(empty_tuple); + + if (name) { + Py_INCREF(name); + Py_XSETREF(self->name, name); } - - if (kwds) { - GET_KWD(name); - GET_KWD(path); + if (path) { + Py_INCREF(path); + Py_XSETREF(self->path, path); } - - if (BaseException_init((PyBaseExceptionObject *)self, args, kwds) == -1) - return -1; - if (PyTuple_GET_SIZE(args) != 1) - return 0; - if (!PyArg_UnpackTuple(args, "ImportError", 1, 1, &msg)) - return -1; - - Py_INCREF(msg); - Py_XSETREF(self->msg, msg); + if (PyTuple_GET_SIZE(args) == 1) { + msg = PyTuple_GET_ITEM(args, 0); + Py_INCREF(msg); + Py_XSETREF(self->msg, msg); + } return 0; }