diff options
| author | Denys Vlasenko | 2018-12-14 16:24:01 +0100 |
|---|---|---|
| committer | Denys Vlasenko | 2018-12-14 16:24:01 +0100 |
| commit | 87b49beeda51ef708e5b83c908edeab1ca88339b (patch) | |
| tree | 507ce6fa38633ae0a9c357bf2cf7970c4a2837d8 /miscutils | |
| parent | 240d7ee3fcff86b557cfac12cb74073e119957a1 (diff) | |
| download | busybox-87b49beeda51ef708e5b83c908edeab1ca88339b.zip busybox-87b49beeda51ef708e5b83c908edeab1ca88339b.tar.gz | |
bc: tweak bc_num_parseDecimal() for readability, logic is not changed
function old new delta
zbc_program_num 836 835 -1
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'miscutils')
| -rw-r--r-- | miscutils/bc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/miscutils/bc.c b/miscutils/bc.c index 547af8e..2e8d044 100644 --- a/miscutils/bc.c +++ b/miscutils/bc.c @@ -2334,13 +2334,13 @@ static void bc_num_parseDecimal(BcNum *n, const char *val) n->num[n->len] = val[i] - '0'; ++n->len; skip_dot: - if ((ssize_t)--i == (ssize_t)-1) break; - if (val[i] == '.') goto skip_dot; + if (i == 0) break; + if (val[--i] == '.') goto skip_dot; } break; } } - // if this is reached, the value is entirely zero + // if for() exits without hitting if(), the value is entirely zero } // Note: n is already "bc_num_zero()"ed, |