Check for 'static inline'

Build a C extension with two C files to check that all
pythoncapi_compat.h functions are declared with "static inline".

Author: vstinner

tests/extension.c

@@ -0,0 +1,34 @@
+// C extension made of two C files (extension.c and static_inline.c) to detect
+// when a pythoncapi_compat.h function is implemented without "static inline".
+
+#ifndef MODULE_NAME
+#   error "MODULE_NAME macro is not set"
+#endif
+
+#include "pythoncapi_compat.h"
+
+#define STR(NAME) #NAME
+#define CONCAT(a, b) a ## b
+
+// No method and no module exec method needed, most basic C extension
+
+static struct PyModuleDef module_def = {
+    PyModuleDef_HEAD_INIT,
+    STR(MODULE_NAME),    // m_name
+    _Py_NULL,            // m_doc
+    0,                   // m_size
+    _Py_NULL,            // m_methods
+    _Py_NULL,            // m_slots
+    _Py_NULL,            // m_traverse
+    _Py_NULL,            // m_clear
+    _Py_NULL,            // m_free
+};
+
+
+#define INIT_FUNC CONCAT(PyInit_, MODULE_NAME)
+
+PyMODINIT_FUNC
+INIT_FUNC(void)
+{
+    return PyModule_Create(&module_def);
+}

tests/setup.py

@@ -202,6 +202,26 @@ def add_common_flags(flags, limited):
             language='c++')
         extensions.append(ext)
 
+    # Build a C extension with two C files to detect when a pythoncapi_compat.h
+    # function is implemented without "static inline".
+    sources = [
+        'extension.c',
+        'static_inline.c',
+    ]
+    for limited in (False, True):
+        name = 'test_pythoncapi_compat_ext'
+        assert name != C_EXTENSION_PREFIX
+        flags = list(COMMON_FLAGS)
+        if limited:
+            name += LIMITED_SUFFIX
+            flags.append(limited_flag)
+        flags.append(f'-DMODULE_NAME={name}')
+        ext = Extension(
+            name,
+            sources=sources,
+            extra_compile_args=flags)
+        extensions.append(ext)
+
     setup(name="test_pythoncapi_compat", ext_modules=extensions)
 
 

tests/static_inline.c

@@ -0,0 +1,6 @@
+// C extension made of two C files (extension.c and static_inline.c) to detect
+// when a pythoncapi_compat.h function is implemented without "static inline".
+
+#include "pythoncapi_compat.h"
+
+// No function defined, only #include "pythoncapi_compat.h"

tests/test_pythoncapi_compat_cext.c

@@ -1,17 +1,17 @@
 // Always enable assertions
 #undef NDEBUG
 
+#ifndef MODULE_NAME
+#   error "MODULE_NAME macro is not set"
+#endif
+
 #include "pythoncapi_compat.h"
 #include <structmember.h>   // T_SHORT, READONLY
 
 #ifdef NDEBUG
 #   error "assertions must be enabled"
 #endif
 
-#ifndef MODULE_NAME
-#   error "MODULE_NAME macro is not set"
-#endif
-
 #define _STR(NAME) #NAME
 #define STR(NAME) _STR(NAME)
 #define _CONCAT(a, b) a ## b