From: Sam James <sam@gentoo.org>
Date: Fri, 30 May 2025 04:42:19 +0000 (+0100)
Subject: gh-134768: Fix definition of `mt_continue_should_break()` (#134769)
X-Git-Tag: v3.15.0a1~1453
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=2f2bee21118adce653ee5bc4eb31d30327465966;p=thirdparty%2FPython%2Fcpython.git

gh-134768: Fix definition of `mt_continue_should_break()` (#134769)

In 121ed71f4e395948d313249b2ad33e1e21581f8a, mt_continue_should_break
was changed to be guarded by `Py_DEBUG`, but it's used in `compress_mt_continue_lock_held`
with just `assert`, so it needs to be available when `NDEBUG` is undefined
too.

`Py_DEBUG` implies `NDEBUG` is undefined, so we can check just that.

Fixes: 121ed71f4e395948d313249b2ad33e1e21581f8a
---

diff --git a/Modules/_zstd/compressor.c b/Modules/_zstd/compressor.c
index 0fc3d7d36c68..8ff2a3aadc1c 100644
--- a/Modules/_zstd/compressor.c
+++ b/Modules/_zstd/compressor.c
@@ -512,7 +512,7 @@ error:
     return NULL;
 }
 
-#ifdef Py_DEBUG
+#ifndef NDEBUG
 static inline int
 mt_continue_should_break(ZSTD_inBuffer *in, ZSTD_outBuffer *out)
 {