GNU bug report logs - #65919
29.1; build without xinput does not get focused when hovering over window

Previous Next

Full log

View this message in rfc822 format

From: Po Lu <luangruo <at> yahoo.com>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 65919 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>, Ivan Popovych <ivan <at> ipvych.com>, Amin Bandali <bandali <at> gnu.org>, Bob Rogers <rogers <at> rgrjr.com>
Subject: bug#65919: 29.1; build without xinput does not get focused when hovering over window
Date: Fri, 15 Sep 2023 13:17:46 +0800

Stefan Kangas <stefankangas <at> gmail.com> writes:

> Po Lu via "Bug reports for GNU Emacs, the Swiss army knife of text
> editors" <bug-gnu-emacs <at> gnu.org> writes:
>
>>> 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?
>
> AFAIU, the point here is to fix a warning in
>
>     if (emacs_write (fd, &dummy, 1) != 1)
>       /* emacs_perror ("writing to child signal FD") */;
>
> for which the easiest solution should be
>
>     emacs_write (fd, &dummy, 1);
>
> This avoids having to disable -Wempty-body everywhere, which IMO doesn't
> seems justified just to silence this one warning.
>
> What am I missing?

It's not to fix a warning, but to communicate that we would like to call
`emacs_perror' there, yet cannot, given that it is not a reentrant
function.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.