commit e1ce781f266394a6e73150d08466274db5db584d
author Rob Landley <rob@landley.net> Thu Jul 02 12:17:59 2015 -0500
committer Rob Landley <rob@landley.net> Thu Jul 02 12:17:59 2015 -0500
tree a70f1965e3de8db959c13c4c255f5767c89e5be1
parent 4ab24f2228b5d51491a5ef2a59ebcf424dced6f7

Minor cleanups on xxd.

toys/pending/xxd.c

diff --git a/toys/pending/xxd.c b/toys/pending/xxd.c
index 3a84c98..03ead7c 100644
--- a/toys/pending/xxd.c
+++ b/toys/pending/xxd.c
@@ -6,7 +6,7 @@
  * TODO: support > 4GiB files?
  * TODO: -s seek
 
-USE_XXD(NEWTOY(xxd, ">1c#l#g#", TOYFLAG_USR|TOYFLAG_BIN))
+USE_XXD(NEWTOY(xxd, ">1c#<0=16l#g#<0=2", TOYFLAG_USR|TOYFLAG_BIN))
 
 config XXD
   bool "xxd"
@@ -29,8 +29,8 @@
 
 GLOBALS(
   long bytes_per_group; // -g
-  long limit; // -l
-  long bytes_per_line; // -c
+  long limit;           // -l
+  long bytes_per_line;  // -c
 )
 
 static void xxd_file(FILE *fp)
@@ -40,20 +40,17 @@
       2*TT.bytes_per_line + TT.bytes_per_line/TT.bytes_per_group + 1;
   size_t line_size = hex_size + TT.bytes_per_line + 1;
   char *line = xmalloc(line_size);
-  int offset = 0;
-  int bytes_this_line = 0;
-  int line_index = 0;
-  int ch;
+  int offset = 0, bytes_this_line = 0, index = 0, ch;
 
   memset(line, ' ', line_size);
   line[line_size - 1] = 0;
 
   while ((ch = getc(fp)) != EOF) {
-    if (bytes_this_line == 0) line_index = sprintf(line, "%08x: ", offset);
+    if (!bytes_this_line) index = sprintf(line, "%08x: ", offset);
     ++offset;
 
-    line[line_index++] = hex_digits[(ch >> 4) & 0xf];
-    line[line_index++] = hex_digits[ch & 0xf];
+    line[index++] = hex_digits[(ch >> 4) & 0xf];
+    line[index++] = hex_digits[ch & 0xf];
     line[hex_size + bytes_this_line] = (ch >= ' ' && ch <= '~') ? ch : '.';
 
     ++bytes_this_line;
@@ -61,14 +58,10 @@
       puts(line);
       memset(line, ' ', line_size - 1);
       bytes_this_line = 0;
-    } else if ((bytes_this_line % TT.bytes_per_group) == 0) {
-      line[line_index++] = ' ';
-    }
-    if ((toys.optflags & FLAG_l) && offset == TT.limit) {
-      break;
-    }
+    } else if (!(bytes_this_line % TT.bytes_per_group)) line[index++] = ' ';
+    if ((toys.optflags & FLAG_l) && offset == TT.limit) break;
   }
-  if (bytes_this_line != 0) {
+  if (bytes_this_line) {
     line[hex_size + bytes_this_line] = 0;
     puts(line);
   }
@@ -80,14 +73,6 @@
 {
   FILE *fp;
 
-  if (!TT.bytes_per_line) TT.bytes_per_line = 16;
-  else if (TT.bytes_per_line < 0)
-    error_exit("invalid -c value: %d", TT.bytes_per_line);
-
-  if (!TT.bytes_per_group) TT.bytes_per_group = 2;
-  else if (TT.bytes_per_group < 0)
-    error_exit("invalid -g value: %d", TT.bytes_per_group);
-
   if (!*toys.optargs || !strcmp(*toys.optargs, "-")) fp = stdin;
   else fp = xfopen(*toys.optargs, "r");