diff options
| author | Elliott Hughes <enh@google.com> | 2021-06-08 12:33:10 -0700 |
|---|---|---|
| committer | Rob Landley <rob@landley.net> | 2021-06-09 18:41:59 -0500 |
| commit | 412054cff027ee52cadc3684f6018cfb0e5eb8e2 (patch) | |
| tree | b37228e8b9f04dfc1a4363344a2a9403faf452b0 | |
| parent | 827cabf78905d4ccb24087df57f01adfeb30ad84 (diff) | |
| download | toybox-412054cff027ee52cadc3684f6018cfb0e5eb8e2.tar.gz | |
md5sum.c: simplify the table setup slightly.
I don't think it was wrong before, but the strchr() in particular left
me scratching my head. This formulation is slightly cheaper and seems
more obviously correct.
(Unrelated but nearby, I don't understand why anyone would ever want to
calculate these tables at runtime, so this seems like another
unnecessary CFG_TOYBOX_FLOAT to me, though an unusual one where we might
want to _remove_ the floating-point code.)
| -rw-r--r-- | toys/lsb/md5sum.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/toys/lsb/md5sum.c b/toys/lsb/md5sum.c index 4df53a5c..feb403f7 100644 --- a/toys/lsb/md5sum.c +++ b/toys/lsb/md5sum.c @@ -514,10 +514,10 @@ void md5sum_main(void) TT.rconsttable32 = xmalloc(64*4); for (i = 0; i<64; i++) TT.rconsttable32[i] = fabs(sin(i+1))*(1LL<<32); } else TT.rconsttable32 = md5nofloat; - } else if (strchr("25", toys.which->name[4])) { // 224, 256 + } else if (toys.which->name[3] == '2') { // sha224, sha256 TT.rconsttable32 = xmalloc(64*4); for (i=0; i<64; i++) TT.rconsttable32[i] = sha512nofloat[i] >> 32; - } else if (toys.which->name[3] != '1') TT.rconsttable64 = sha512nofloat; + } else TT.rconsttable64 = sha512nofloat; // sha384, sha512 } if (FLAG(c)) for (i = 0; toys.optargs[i]; i++) do_c_file(toys.optargs[i]); |