Issue #19512: Use the new _PyId_builtins identifier

diff --git a/Modules/_lsprof.c b/Modules/_lsprof.c
index 894788916d7e2ca6d0aced3b290e22e8aa1c0355..3ef7d65e490a762153679783189cbfc926bb148d 100644 (file)
--- a/Modules/_lsprof.c
+++ b/Modules/_lsprof.c
@@ -185,7 +185,7 @@ normalizeUserObj(PyObject *obj)
             }
         }
         if (modname != NULL) {
-            if (PyUnicode_CompareWithASCIIString(modname, "builtins") != 0) {
+            if (_PyUnicode_CompareWithId(modname, &_PyId_builtins) != 0) {
                 PyObject *result;
                 result = PyUnicode_FromFormat("<%U.%s>", modname,
                                               fn->m_ml->ml_name);

diff --git a/Objects/object.c b/Objects/object.c
index 80786239fe0008fa54494bf8501dbe3b56c4cb4b..9d96e86c589827cd75f49a5c889b1ea0c049f663 100644 (file)
--- a/Objects/object.c
+++ b/Objects/object.c
@@ -1122,8 +1122,12 @@ PyObject_SelfIter(PyObject *obj)
 PyObject *
 _PyObject_GetBuiltin(const char *name)
 {
-    PyObject *mod, *attr;
-    mod = PyImport_ImportModule("builtins");
+    PyObject *mod_name, *mod, *attr;
+
+    mod_name = _PyUnicode_FromId(&_PyId_builtins);   /* borrowed */
+    if (mod_name == NULL)
+        return NULL;
+    mod = PyImport_Import(mod_name);
     if (mod == NULL)
         return NULL;
     attr = PyObject_GetAttrString(mod, name);

diff --git a/Python/import.c b/Python/import.c
index c96106f373265b44282c06e0dbc45ed3a6bd2e5b..aea29e28f307bdb1ce8a03ab3ab88ad74a10fedd 100644 (file)
--- a/Python/import.c
+++ b/Python/import.c
@@ -310,7 +310,7 @@ PyImport_Cleanup(void)
 
     /* XXX Perhaps these precautions are obsolete. Who knows? */
 
-    value = PyDict_GetItemString(modules, "builtins");
+    value = _PyDict_GetItemId(modules, &_PyId_builtins);
     if (value != NULL && PyModule_Check(value)) {
         dict = PyModule_GetDict(value);
         if (Py_VerboseFlag)