aboutsummaryrefslogtreecommitdiff
path: root/toys
diff options
context:
space:
mode:
authorRob Landley <rob@landley.net>2015-07-02 12:17:59 -0500
committerRob Landley <rob@landley.net>2015-07-02 12:17:59 -0500
commite1ce781f266394a6e73150d08466274db5db584d (patch)
treea70f1965e3de8db959c13c4c255f5767c89e5be1 /toys
parent4ab24f2228b5d51491a5ef2a59ebcf424dced6f7 (diff)
downloadtoybox-e1ce781f266394a6e73150d08466274db5db584d.tar.gz
Minor cleanups on xxd.
Diffstat (limited to 'toys')
-rw-r--r--toys/pending/xxd.c35
1 files changed, 10 insertions, 25 deletions
diff --git a/toys/pending/xxd.c b/toys/pending/xxd.c
index 3a84c988..03ead7c2 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 @@ static const char* hex_digits = "0123456789abcdef";
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 @@ static void xxd_file(FILE *fp)
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 @@ static void xxd_file(FILE *fp)
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 @@ void xxd_main(void)
{
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");