[issue24010] Add error checks to PyInit__locale()

2021-12-11 Thread Irit Katriel
Irit Katriel added the comment: This function now looks like this: PyMODINIT_FUNC PyInit__locale(void) { return PyModuleDef_Init(&_localemodule); } -- nosy: +iritkatriel resolution: -> out of date stage: patch review -> resolved status: open -> closed ___

[issue24010] Add error checks to PyInit__locale()

2018-08-17 Thread Berker Peksag
Berker Peksag added the comment: A variant of localemodule.patch without error checking for PyModule_AddIntMacro() has been applied in https://github.com/python/cpython/commit/ff4fddde57d5579dff3a83d99e20cd06366b10d6. Christian, can this be closed or do we still need to add error checking to

[issue24010] Add error checks to PyInit__locale()

2015-04-19 Thread Christian Heimes
New submission from Christian Heimes: The init function of the locale module fails to check for errors in a couple of places. The patch replaces PyDict_SetItemString() calls with PyModule_AddIntMacro() and error checks. An exception is unlikely so I'm OK when the patch just lands in 3.4 and 3.