|
| From: | Sevan Janiyan |
| Subject: | Re: [PATCH] lib/stdio.in.h: define _POSIX_C_SOURCE on >=Sierra |
| Date: | Thu, 9 Nov 2023 20:17:00 +0000 |
| User-agent: | Mozilla Thunderbird |
On 09/11/2023 19:00, Sevan Janiyan wrote:
# if (defined MAC_OS_X_VERSION_MIN_REQUIRED \ && 101300 <= MAC_OS_X_VERSION_MIN_REQUIRED)This is wrong, you're applying the change to anything before High Sierra (10.13). You're after Ventura and above (13).# if (defined MAC_OS_X_VERSION_MIN_REQUIRED \ && 130000 >= MAC_OS_X_VERSION_MIN_REQUIRED)
It inadvertently fixes the issue on Tiger 10.4 & older since __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ is never defined to start with. Tested & there's no issue for Leopard either where __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ is defined, but it would have recreated the warnings for sprintf and vsprintf on macOS 13 and newer.
Sevan
| [Prev in Thread] | Current Thread | [Next in Thread] |