[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master ad2d50d762: Fix last change in xterm.c
From: |
Andreas Schwab |
Subject: |
Re: master ad2d50d762: Fix last change in xterm.c |
Date: |
Mon, 21 Feb 2022 09:39:03 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.91 (gnu/linux) |
On Feb 21 2022, Po Lu wrote:
> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
>> I have no idea whether it's this change that caused it or not, but:
>>
>> xterm.c: In function ‘handle_one_xevent’:
>> xterm.c:8880:3: warning: macro expands to multiple statements
>> [-Wmultistatement-macros]
>> 8880 | if (temp_index == ARRAYELTS (temp_buffer))
>> | ^~
>> xterm.c:8880:3: note: in definition of macro ‘STORE_KEYSYM_FOR_DEBUG’
>> 8880 | if (temp_index == ARRAYELTS (temp_buffer))
>> | ^~
>> xterm.c:10125:13: note: some parts of macro expansion are not guarded by
>> this ‘for’ clause
>> 10125 | for (i = 0; i < nbytes; i++)
>>
>> I get these warnings when compiling now with nativecomp, and I don't
>> think they were there a couple of days ago? But I may be mistaken.
>
> That's odd, since that code has been there forever.
That's not true, see commit 0a9c8855b0.
--
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1
"And now for something completely different."