<div dir="ltr"><div dir="ltr">On Mon, Mar 13, 2023 at 9:40 PM Todd C. Miller <<a href="mailto:Todd.Miller@sudo.ws">Todd.Miller@sudo.ws</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sat, 11 Mar 2023 11:18:43 +0100, Martin Jansa via tz wrote:<br>
<br>
> I was building tz with gcc-13 and noticed that without explicit -std=c2x it<br>
> fails to build 2022g version with:<br>
> ```<br>
> zic.c:462:1: warning: ‘noreturn’ attribute ignored [-Wattributes]<br>
>   462 | static ATTRIBUTE_NORETURN void<br>
>       | ^~~~~~<br>
> zic.c:462:27: error: expected identifier or ‘(’ before ‘void’<br>
>   462 | static ATTRIBUTE_NORETURN void<br>
>       |                           ^~~~<br>
> ```<br>
<br>
Try swapping the order of ATTRIBUTE_NORETURN and "static".  C23<br>
attributes are supposed to come first (unlike gcc-style attributes)<br>
similar to C++.<br></blockquote><div><br></div><div>Thanks, you're right.</div><div><br></div><div>And when I've started to do that instead of my work around ("if __has_c_attribute(noreturn) && 202000 <= __STDC_VERSION__") I've forked tz on github to send it as PR and discovered that it's already resolved there in version newer than 2022g I was using locally, don't know how I've overlooked that before (probably because I was looking more at private.h and less on zic.c/zdump.c).</div><div><br></div><div>It's resolved in:</div><div><a href="https://github.com/eggert/tz/commit/9cfe9507fcc22cd4a0c4da486ea1c7f0de6b075f">https://github.com/eggert/tz/commit/9cfe9507fcc22cd4a0c4da486ea1c7f0de6b075f</a><br></div><div><br></div><div>I'll at least backport it to OpenEmbedded now :).<br></div><div><br></div><div>Regards, </div></div></div>