GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||
GeezerVison is showing github.com — gh-112026: Restore removed _PyDict_GetItemStringWithError() by vstinner · Pull Request #112119 · python/cpython Conversation
|
|
@serhiy-storchaka: This private function is no longer used by Python, but I restore it to unblock numpy on Python 3.13 (keep dependency in the Python ecosystem). Later, I plan to deprecate this function and write documentation on how to update existing code using it. Apparently, just removing private functions without providing any solution didn't "please" users. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Is not PyObject * _PyDict_GetItemStringWithError(PyObject *obj, const char *key) { PyObject *value; (void)PyDict_GetItemStringRef(obj, key, &value); if (Py_REFCNT(value) == 1) { Py_DECREF(value); return NULL; } Py_DECREF(value); return value; } |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Solution to which problem? numpy doesn't built if Python C API doesn't contain _PyDict_GetItemStringWithError(). I restored the function to unblock numpy. I plan to update numpy, deprecate _PyDict_GetItemStringWithError(), and later remove it. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…thon#112119) Restore the removed _PyDict_GetItemStringWithError() function. It is used by numpy.
See numpy/numpy#26282, would also appreciate code review if you have time. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I wrote #119855 to deprecate _PyDict_GetItemStringWithError(). |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…thon#112119) Restore the removed _PyDict_GetItemStringWithError() function. It is used by numpy.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.