gh-107211: No longer export internal _PyLong_FromUid() (#109037)

vstinner · web-flow · commit 19eddb515a8c · 2023-09-07T02:09:06.000+02:00
No longer export _PyLong_FromUid() and _Py_Sigset_Converter()
internal C API function.
diff --git a/Modules/posixmodule.h b/Modules/posixmodule.h
@@ -2,31 +2,37 @@
 
 #ifndef Py_POSIXMODULE_H
 #define Py_POSIXMODULE_H
+#ifndef Py_LIMITED_API
 #ifdef __cplusplus
 extern "C" {
 #endif
 
 #ifdef HAVE_SYS_TYPES_H
-#include <sys/types.h>
+#  include <sys/types.h>          // uid_t
 #endif
 
-#ifndef Py_LIMITED_API
 #ifndef MS_WINDOWS
-PyAPI_FUNC(PyObject *) _PyLong_FromUid(uid_t);
-PyAPI_FUNC(PyObject *) _PyLong_FromGid(gid_t);
+extern PyObject* _PyLong_FromUid(uid_t);
+
+// Export for 'grp' shared extension
+PyAPI_FUNC(PyObject*) _PyLong_FromGid(gid_t);
+
+// Export for '_posixsubprocess' shared extension
 PyAPI_FUNC(int) _Py_Uid_Converter(PyObject *, uid_t *);
+
+// Export for 'grp' shared extension
 PyAPI_FUNC(int) _Py_Gid_Converter(PyObject *, gid_t *);
-#endif /* MS_WINDOWS */
+#endif   // !MS_WINDOWS
 
-#if defined(PYPTHREAD_SIGMASK) || defined(HAVE_SIGWAIT) || \
-        defined(HAVE_SIGWAITINFO) || defined(HAVE_SIGTIMEDWAIT)
-# define HAVE_SIGSET_T
+#if (defined(PYPTHREAD_SIGMASK) || defined(HAVE_SIGWAIT) \
+     || defined(HAVE_SIGWAITINFO) || defined(HAVE_SIGTIMEDWAIT))
+#  define HAVE_SIGSET_T
 #endif
 
-PyAPI_FUNC(int) _Py_Sigset_Converter(PyObject *, void *);
-#endif /* Py_LIMITED_API */
+extern int _Py_Sigset_Converter(PyObject *, void *);
 
 #ifdef __cplusplus
 }
 #endif
-#endif /* !Py_POSIXMODULE_H */
+#endif   // !Py_LIMITED_API
+#endif   // !Py_POSIXMODULE_H
