Hide CAPI_WARN etc. behind NDEBUG

fangerer · fangerer · commit 4678d484507b · 2023-03-14T13:09:21.000+01:00
diff --git a/numpy/core/include/numpy/ndarraytypes.h b/numpy/core/include/numpy/ndarraytypes.h
@@ -48,10 +48,12 @@ void _print_stack() {
 static NPY_INLINE HPyContext *
 npy_get_context(void)
 {
+#ifndef NDEBUG
     if (capi_warn_level() > 1) {
         printf("DEBUG WARNING: HPy context lookup (set CAPI_WARN_STACKTRACE=<limit> to show stack trace)\n");
         _print_stack();
     }
+#endif
     assert(numpy_global_ctx != NULL);
     return numpy_global_ctx;
 }
@@ -80,9 +82,14 @@ void perf_warn(const char *msg, const char *file, int lineno) {
     }
 }
 
+#ifndef NDEBUG
 #define CAPI_WARN(_x) capi_warn0(_x, __FILE__, __LINE__)
-
 #define HPY_PERFORMANCE_WARNING(_x) perf_warn(_x, __FILE__, __LINE__)
+#else
+#define CAPI_WARN(_x)
+#define HPY_PERFORMANCE_WARNING(_x)
+#endif
+
 
 static __attribute__ ((noreturn)) NPY_INLINE
 void hpy_abort_not_implemented(const char *where) {

Original file line number	Diff line number	Diff line change
`@@ -48,10 +48,12 @@ void _print_stack() {`
`48`	`48`	`static NPY_INLINE HPyContext *`
`49`	`49`	`npy_get_context(void)`
`50`	`50`	`{`
	`51`	`+#ifndef NDEBUG`
`51`	`52`	`if (capi_warn_level() > 1) {`
`52`	`53`	`printf("DEBUG WARNING: HPy context lookup (set CAPI_WARN_STACKTRACE=<limit> to show stack trace)\n");`
`53`	`54`	`_print_stack();`
`54`	`55`	`}`
	`56`	`+#endif`
`55`	`57`	`assert(numpy_global_ctx != NULL);`
`56`	`58`	`return numpy_global_ctx;`
`57`	`59`	`}`
`@@ -80,9 +82,14 @@ void perf_warn(const char msg, const char file, int lineno) {`
`80`	`82`	`}`
`81`	`83`	`}`
`82`	`84`
	`85`	`+#ifndef NDEBUG`
`83`	`86`	`#define CAPI_WARN(_x) capi_warn0(_x, __FILE__, __LINE__)`
`84`		`-`
`85`	`87`	`#define HPY_PERFORMANCE_WARNING(_x) perf_warn(_x, __FILE__, __LINE__)`
	`88`	`+#else`
	`89`	`+#define CAPI_WARN(_x)`
	`90`	`+#define HPY_PERFORMANCE_WARNING(_x)`
	`91`	`+#endif`
	`92`	`+`
`86`	`93`
`87`	`94`	`static __attribute__ ((noreturn)) NPY_INLINE`
`88`	`95`	`void hpy_abort_not_implemented(const char *where) {`