Re: master ad2d50d762: Fix last change in xterm.c

[Po Lu]

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.  Do you get them
without nativecomp?

Thanks.