[Python-checkins] rename _imp initialization function to follow conventions (#5432)

Benjamin Peterson webhook-mailer at python.org
Mon Jan 29 14:34:00 EST 2018


https://github.com/python/cpython/commit/c65ef772c335324deb03626cc447b89987781f27
commit: c65ef772c335324deb03626cc447b89987781f27
branch: master
author: Benjamin Peterson <benjamin at python.org>
committer: GitHub <noreply at github.com>
date: 2018-01-29T11:33:57-08:00
summary:

rename _imp initialization function to follow conventions (#5432)

When the C imp module became _imp in 6f44d66bc491bad5b8d897a68da68e009e27829d, the initialization function should have been renamed from PyInit_imp to PyInit__imp.

files:
M Include/import.h
M Modules/config.c.in
M PC/config.c
M Python/import.c
M Python/pylifecycle.c

diff --git a/Include/import.h b/Include/import.h
index 26c4b1f18ef2..ac3fc3bd28c0 100644
--- a/Include/import.h
+++ b/Include/import.h
@@ -10,7 +10,7 @@ extern "C" {
 #ifndef Py_LIMITED_API
 PyAPI_FUNC(_PyInitError) _PyImportZip_Init(void);
 
-PyMODINIT_FUNC PyInit_imp(void);
+PyMODINIT_FUNC PyInit__imp(void);
 #endif /* !Py_LIMITED_API */
 PyAPI_FUNC(long) PyImport_GetMagicNumber(void);
 PyAPI_FUNC(const char *) PyImport_GetMagicTag(void);
diff --git a/Modules/config.c.in b/Modules/config.c.in
index 7b77199c2e93..d69e8e88b0ca 100644
--- a/Modules/config.c.in
+++ b/Modules/config.c.in
@@ -25,7 +25,7 @@ extern "C" {
 /* -- ADDMODULE MARKER 1 -- */
 
 extern PyObject* PyMarshal_Init(void);
-extern PyObject* PyInit_imp(void);
+extern PyObject* PyInit__imp(void);
 extern PyObject* PyInit_gc(void);
 extern PyObject* PyInit__ast(void);
 extern PyObject* _PyWarnings_Init(void);
@@ -39,7 +39,7 @@ struct _inittab _PyImport_Inittab[] = {
     {"marshal", PyMarshal_Init},
 
     /* This lives in import.c */
-    {"_imp", PyInit_imp},
+    {"_imp", PyInit__imp},
 
     /* This lives in Python/Python-ast.c */
     {"_ast", PyInit__ast},
diff --git a/PC/config.c b/PC/config.c
index 91f15b5867d5..6209d7e284f3 100644
--- a/PC/config.c
+++ b/PC/config.c
@@ -74,7 +74,7 @@ extern PyObject* PyInit__opcode(void);
 /* -- ADDMODULE MARKER 1 -- */
 
 extern PyObject* PyMarshal_Init(void);
-extern PyObject* PyInit_imp(void);
+extern PyObject* PyInit__imp(void);
 
 struct _inittab _PyImport_Inittab[] = {
 
@@ -147,7 +147,7 @@ struct _inittab _PyImport_Inittab[] = {
     {"marshal", PyMarshal_Init},
 
     /* This lives it with import.c */
-    {"_imp", PyInit_imp},
+    {"_imp", PyInit__imp},
 
     /* These entries are here for sys.builtin_module_names */
     {"builtins", NULL},
diff --git a/Python/import.c b/Python/import.c
index 8d306403a29d..eb5aeac55461 100644
--- a/Python/import.c
+++ b/Python/import.c
@@ -1,4 +1,3 @@
-
 /* Module definition and import implementation */
 
 #include "Python.h"
@@ -2253,7 +2252,7 @@ static struct PyModuleDef impmodule = {
 const char *_Py_CheckHashBasedPycsMode = "default";
 
 PyMODINIT_FUNC
-PyInit_imp(void)
+PyInit__imp(void)
 {
     PyObject *m, *d;
 
diff --git a/Python/pylifecycle.c b/Python/pylifecycle.c
index 82ab9154256f..5db586e15dff 100644
--- a/Python/pylifecycle.c
+++ b/Python/pylifecycle.c
@@ -312,7 +312,7 @@ initimport(PyInterpreterState *interp, PyObject *sysmod)
     Py_INCREF(interp->import_func);
 
     /* Import the _imp module */
-    impmod = PyInit_imp();
+    impmod = PyInit__imp();
     if (impmod == NULL) {
         return _Py_INIT_ERR("can't import _imp");
     }



More information about the Python-checkins mailing list