changeset 1417:855728acd1ad draft

Cosmetic tweak, slightly more conventional for loop.
author Rob Landley <rob@landley.net>
date Sat, 02 Aug 2014 19:52:44 -0500
parents 0019500c86cc
children b4700f9f10ca
files toys/posix/id.c
diffstat 1 files changed, 3 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/toys/posix/id.c	Sat Aug 02 19:50:03 2014 -0500
+++ b/toys/posix/id.c	Sat Aug 02 19:52:44 2014 -0500
@@ -113,14 +113,13 @@
   i = sizeof(toybuf)/sizeof(gid_t);
   ngroups = username ? getgrouplist(username, gid, groups, &i)
     : getgroups(i, groups);
-  if (0 >= ngroups) perror_exit(0);
+  if (ngroups<0) perror_exit(0);
 
-  for (i = 0;;) {
+  for (i = 0; i<ngroups; i++) {
+    if (i) xputc(' ');
     if (!(grp = getgrgid(groups[i]))) perror_msg(0);
     else if (flags & FLAG_G) s_or_u(grp->gr_name, grp->gr_gid, 0);
     else if (grp->gr_gid != egid) showid("", grp->gr_gid, grp->gr_name);
-    if (++i >= ngroups) break;
-    xputc(' ');
   }
   xputc('\n');
 }