Re: master 7f9335f91d: Implement _NET_WM_USER_TIME protocol

[Po Lu]

Robert Pluim <rpluim@gmail.com> writes:

>>>>>> On Wed,  9 Feb 2022 03:14:53 -0500 (EST), Po Lu via Mailing list for 
>>>>>> Emacs changes <emacs-diffs@gnu.org> said:
>
>     Po> branch: master
>     Po> commit 7f9335f91d3e0ecdff9c3adb93aad08454002035
>     Po> Author: Po Lu <luangruo@yahoo.com>
>     Po> Commit: Po Lu <luangruo@yahoo.com>
>
>     Po>     Implement _NET_WM_USER_TIME protocol
>     
>     Po>     * src/xterm.c (x_display_set_last_user_time): Set user time on
>     Po>     the active window.
>     Po>     (handle_one_xevent): Only set user time when the user actually
>     Po>     interacted with Emacs.
>     Po>     (x_make_frame_visible): Set user time if non-zero.
>     Po>     (x_term_init): Intern `_NET_WM_USER_TIME'.
>     Po>     * src/xterm.h (struct x_display_info): New field
>     Po>     `Xatom_net_wm_user_time'.
>     Po>     (x_display_set_last_user_time): Implement in xterm.c instead.
>     Po> ---
>     Po>  src/xterm.c | 101 
> ++++++++++++++++++++++++++++++++++++++----------------------
>     Po>  src/xterm.h |  14 +++------
>     Po>  2 files changed, 69 insertions(+), 46 deletions(-)
>
>   CC       xterm.o
> xterm.c: In function ‘handle_one_xevent’:
> xterm.c:11762:6: error: a label can only be part of a statement and a 
> declaration is not a statement
> 11762 |      XKeyPressedEvent xkey;
>       |      ^~~~~~~~~~~~~~~~
> make[1]: *** [Makefile:414: xterm.o] Error 1
> make[1]: Leaving directory '/home/rpluim/repos/emacs/src'
> make: *** [Makefile:469: src] Error 2
>
> Are you compiling emacs with a c++ compiler?

No, and that can't work anyway.  (But I see the problem now, thanks.)