Re: master 4d03f70b7f0 1/4: Correct doc strings for x-*-keysym

[Eli Zaretskii <eliz@gnu.org>]

> Auto-Submitted: auto-generated
> Date: Sat, 16 Mar 2024 03:26:54 -0400 (EDT)
> From: Po Lu <luangruo@yahoo.com>
> 
> branch: master
> commit 4d03f70b7f01477a8d72f827ba8b0dabba8c0a61
> Author: Po Lu <luangruo@yahoo.com>
> Commit: Po Lu <luangruo@yahoo.com>
> 
>     Correct doc strings for x-*-keysym
>     
>     * src/xterm.c (syms_of_xterm): Clarify whether x-*-keysym
>     affects the modifier key reported for a keysym or the other way
>     around.
> ---
>  src/xterm.c | 42 ++++++++++++++++++++++--------------------
>  1 file changed, 22 insertions(+), 20 deletions(-)
> 
> diff --git a/src/xterm.c b/src/xterm.c
> index c8a43785564..bebc30c9103 100644
> --- a/src/xterm.c
> +++ b/src/xterm.c
> @@ -32536,38 +32536,40 @@ Android does not support scroll bars at all.  */);
>    DEFSYM (Qreally_fast, "really-fast");
>  
>    DEFVAR_LISP ("x-ctrl-keysym", Vx_ctrl_keysym,
> -    doc: /* Which keys Emacs uses for the ctrl modifier.
> -This should be one of the symbols `ctrl', `alt', `hyper', `meta',
> -`super'.  For example, `ctrl' means use the Ctrl_L and Ctrl_R keysyms.
> -The default is nil, which is the same as `ctrl'.  */);
> +    doc: /* Which modifer value Emacs reports when Ctrl is depressed.
> +This should be one of the symbols `ctrl', `alt', `hyper', `meta', or
> +`super', representing a modifier to be reported in key events with the
> +Ctrl modifier (i.e. the keysym Ctrl_L or Ctrl_R) depressed.  */);
>    Vx_ctrl_keysym = Qnil;

This loses the information about the meaning of the default value of
nil, doesn't it?  This leaves the user who looks at the default value
confused as to what that means.

Also, the "depressed" part at the end of the last sentence is not
right: a modifier reported in a key event cannot be "depressed", it
can only be "set".  Right?