]> Savannah Git Hosting - gnulib.git/commitdiff
userspec: no need for static vars
authorPaul Eggert <eggert@cs.ucla.edu>
Thu, 24 Feb 2022 16:45:55 +0000 (08:45 -0800)
committerPaul Eggert <eggert@cs.ucla.edu>
Fri, 25 Feb 2022 01:11:40 +0000 (17:11 -0800)
* lib/userspec.c (parse_with_separator): Simplify.

ChangeLog
lib/userspec.c

index 9748e5ab5039532b109c06e678022ab29a0fb9ee..dda27c7eb9139500021125cc5c2ca9ba194168fe 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2022-02-24  Paul Eggert  <eggert@cs.ucla.edu>
+
+       userspec: no need for static vars
+       * lib/userspec.c (parse_with_separator): Simplify.
+
 2022-02-22  Benno Schulenberg  <bensberg@telfort.nl>  (tiny change)
 
        doc: add two missing closing parentheses
index f05ccbe63545e8c367151111739439ec84386156..e75feb255df2400763deb85ff37d4e0401ab9545 100644 (file)
@@ -103,10 +103,6 @@ parse_with_separator (char const *spec, char const *separator,
                       uid_t *uid, gid_t *gid,
                       char **username, char **groupname)
 {
-  static const char *E_invalid_user = N_("invalid user");
-  static const char *E_invalid_group = N_("invalid group");
-  static const char *E_bad_spec = N_("invalid spec");
-
   const char *error_msg;
   struct passwd *pwd;
   struct group *grp;
@@ -167,7 +163,7 @@ parse_with_separator (char const *spec, char const *separator,
             {
               /* If there is no group,
                  then there may not be a trailing ":", either.  */
-              error_msg = E_bad_spec;
+              error_msg = N_("invalid spec");
             }
           else
             {
@@ -176,7 +172,7 @@ parse_with_separator (char const *spec, char const *separator,
                   && tmp <= MAXUID && (uid_t) tmp != (uid_t) -1)
                 unum = tmp;
               else
-                error_msg = E_invalid_user;
+                error_msg = N_("invalid user");
             }
         }
       else
@@ -209,7 +205,7 @@ parse_with_separator (char const *spec, char const *separator,
               && tmp <= MAXGID && (gid_t) tmp != (gid_t) -1)
             gnum = tmp;
           else
-            error_msg = E_invalid_group;
+            error_msg = N_("invalid group");
         }
       else
         gnum = grp->gr_gid;