diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2018-12-18 00:39:24 +0100 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2018-12-18 03:30:23 +0100 |
commit | 4113e1f2cd2f45a95bcb0920bf2e3ee75b906281 (patch) | |
tree | af0914f73ee6af0d10fe6f4b8fb509905b1678d1 /miscutils/bc.c | |
parent | 57734c926bbfe69d1c3bb6096fbdf428628ed838 (diff) |
bc: rewrite bc_num_compare() to be readable
function old new delta
bc_num_compare 59 51 -8
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'miscutils/bc.c')
-rw-r--r-- | miscutils/bc.c | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/miscutils/bc.c b/miscutils/bc.c index e5ad0ed86..eaab6cee6 100644 --- a/miscutils/bc.c +++ b/miscutils/bc.c @@ -1475,10 +1475,20 @@ static void bc_num_subArrays(BcDig *restrict a, BcDig *restrict b, static ssize_t bc_num_compare(BcDig *restrict a, BcDig *restrict b, size_t len) { - size_t i; - int c = 0; - for (i = len - 1; i < len && !(c = a[i] - b[i]); --i); - return BC_NUM_NEG(i + 1, c < 0); + size_t i = len; + for (;;) { + int c; + if (i == 0) + return 0; + i--; + c = a[i] - b[i]; + if (c != 0) { + i++; + if (c < 0) + return -i; + return i; + } + } } static ssize_t bc_num_cmp(BcNum *a, BcNum *b) |