diff options
author | Rich Felker <dalias@aerifal.cx> | 2024-02-26 09:37:44 -0500 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2024-02-26 14:14:08 -0500 |
commit | a7239cbc1bc4144eb34e0b85f71769f0acda58dd (patch) | |
tree | bcf8d3e57e464d8de6cc25fc77f5a0b54e5ebece /src/stat/futimesat.c | |
parent | 2c887f24da36fa30eccd72e50d91222828fa526e (diff) | |
download | musl-a7239cbc1bc4144eb34e0b85f71769f0acda58dd.tar.gz musl-a7239cbc1bc4144eb34e0b85f71769f0acda58dd.tar.xz musl-a7239cbc1bc4144eb34e0b85f71769f0acda58dd.zip |
switch __STDC_UTF_{16,32}__ macro definitions from #undef to #ifndef
originally, compilers did not provide these macros and we had to provide them ourselves. this meant we were redefining them, which was technically invalid unless the token sequence of the original definition matched exactly. the original patch proposed by Jules Maselbas to fix this made the definitions conditional on them not already being defined; however I suggested using #undef to avoid any possibly-wrong definitions already in place and ensure that the definitions are 1. the version adopted as commit 8b7048680731707d135ea231f81eb3eaf52378ee made this change. unfortunately, gcc is loud about not liking #undef of any __STDC_* macro name, and while warnings are suppressed in the system include path, there is apparently no way to suppress this warning if the system include dir has also been provided via -I. while normally we don't go out of our way to satisfy warnings over style in the public headers, in this case, it seems to be a matter of disagreement over contract of which part of "the implementation" is entitled to define or undefine macros belonging to the implementation, and it's quite reasonable to conclude that the compiler may reject attempts to undefine them. this commit reverts to the originally-submitted version of the patch making the definitions conditional.
Diffstat (limited to 'src/stat/futimesat.c')
0 files changed, 0 insertions, 0 deletions