Subject: Re: bug#65919: 29.1; build without xinput does not get focused when hovering over window

From: Amin Bandali <[email-masked]>
Date: Fri, 15 Sep 2023 01:02:33 -0400

Po Lu writes:

> Amin Bandali <[email-masked]> writes:
>
>> Hello,
>>
>> Po Lu writes:
>>
>>> This should be fixed on the master branch.  [...]
>>
>> FYI your e1a730017d6 patch introduces the following compiler warning:
>>
>> process.c: In function ‘child_signal_notify’:
>> process.c:7436:54: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body]
>>  7436 |     /* emacs_perror ("writing to child signal FD") */;
>>       |                                                      ^
>
> Thanks.  Does this silence the warning?
>
> diff --git a/configure.ac b/configure.ac
> index 7ca75be996d..3b5f3c1c37a 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1776,6 +1776,9 @@ AC_DEFUN
>    # Emacs doesn't need this paranoia.
>    nw="$nw -Wbidi-chars=any,ucn"
>  
> +  # Or this hysteria, which impedes commentary within if statements.
> +  nw="$nw -Wempty-body"
> +
>    if test "$emacs_cv_clang" = yes; then
>      nw="$nw -Wdouble-promotion"
>      nm="$nm -Wunknown-pragmas"
>

Thanks, but it doesn't seem to.  Also, IMHO this is a useful warning,
and the better fix would be to add the braces for that if statement.
That part of the code now looks quite fragile, and a problem waiting
to happen if that lone semicolon is accidentally mistakenly removed.