procyberian/emacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Explain why we remove stuff from 'user-full-name'

Browse source 
* src/editfns.c (Fuser_full_name): Document why we chop everything
starting from the first comma.  (Bug#52785)
This commit is contained in:
Eli Zaretskii 2021-12-25 09:38:39 +02:00
parent c09ad0cabd
commit 5fccd1e371
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/editfns.c
View file

@ -1233,7 +1233,11 @@ return "unknown".
If optional argument UID is an integer, return the full name
of the user with that uid, or nil if there is no such user.
If UID is a string, return the full name of the user with that login
name, or nil if there is no such user. */)
name, or nil if there is no such user.
If the full name includes commas, remove everything starting with
the first comma, because the \\='gecos\\=' field of the \\='/etc/passwd\\=' file
is in general a comma-separated list. */)
(Lisp_Object uid)
{
struct passwd *pw;
@ -1263,7 +1267,8 @@ name, or nil if there is no such user. */)
return Qnil;
p = USER_FULL_NAME;
/* Chop off everything after the first comma. */
/* Chop off everything after the first comma, since 'pw_gecos' is a
comma-separated list. */
q = strchr (p, ',');
full = make_string (p, q ? q - p : strlen (p));