[git commit] bc: add and use bc_num_init_DEF_SIZE()
Denys Vlasenko
vda.linux at googlemail.com
Sun Dec 9 10:44:20 UTC 2018
commit: https://git.busybox.net/busybox/commit/?id=e20e00de341518f2a9d49bde77b9e4876b472a4e
branch: https://git.busybox.net/busybox/commit/?id=refs/heads/master
function old new delta
bc_num_init_DEF_SIZE - 10 +10
bc_num_binary 150 148 -2
bc_program_pushVar 208 203 -5
bc_program_modexp 741 736 -5
bc_program_copyToVar 316 311 -5
bc_program_call 352 347 -5
bc_array_expand 88 83 -5
bc_program_num 1125 1115 -10
bc_vm_init 821 786 -35
bc_program_exec 4152 4098 -54
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 0/9 up/down: 10/-126) Total: -116 bytes
text data bss dec hex filename
985988 477 7296 993761 f29e1 busybox_old
985872 477 7296 993645 f296d busybox_unstripped
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
miscutils/bc.c | 54 +++++++++++++++++++++++++++++-------------------------
1 file changed, 29 insertions(+), 25 deletions(-)
diff --git a/miscutils/bc.c b/miscutils/bc.c
index 0dc3f843c..2e3c4139e 100644
--- a/miscutils/bc.c
+++ b/miscutils/bc.c
@@ -1407,6 +1407,11 @@ static void bc_num_init(BcNum *n, size_t req)
n->cap = req;
}
+static void bc_num_init_DEF_SIZE(BcNum *n)
+{
+ bc_num_init(n, BC_NUM_DEF_SIZE);
+}
+
static void bc_num_expand(BcNum *n, size_t req)
{
req = req >= BC_NUM_DEF_SIZE ? req : BC_NUM_DEF_SIZE;
@@ -2260,8 +2265,8 @@ static void bc_num_parseBase(BcNum *n, const char *val, BcNum *base)
for (i = 0; zero && i < len; ++i) zero = (val[i] == '.' || val[i] == '0');
if (zero) return;
- bc_num_init(&temp, BC_NUM_DEF_SIZE);
- bc_num_init(&mult, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&temp);
+ bc_num_init_DEF_SIZE(&mult);
for (i = 0; i < len; ++i) {
@@ -2584,7 +2589,7 @@ static BcStatus bc_num_sqrt(BcNum *a, BcNum *restrict b, size_t scale)
bc_num_init(&num1, len);
bc_num_init(&num2, len);
- bc_num_init(&half, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&half);
bc_num_one(&half);
half.num[0] = 5;
@@ -2701,7 +2706,7 @@ static BcStatus bc_num_modexp(BcNum *a, BcNum *b, BcNum *c, BcNum *restrict d)
bc_num_expand(d, c->len);
bc_num_init(&base, c->len);
bc_num_init(&exp, b->len);
- bc_num_init(&two, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&two);
bc_num_init(&temp, b->len);
bc_num_one(&two);
@@ -2792,7 +2797,7 @@ static void bc_array_expand(BcVec *a, size_t len)
if (a->size == sizeof(BcNum) && a->dtor == bc_num_free) {
while (len > a->len) {
- bc_num_init(&data.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&data.n);
bc_vec_push(a, &data.n);
}
}
@@ -5530,7 +5535,7 @@ static BcStatus bc_program_op(char inst)
s = bc_program_binOpPrep(&opd1, &n1, &opd2, &n2, false);
if (s) return s;
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
s = bc_program_ops[inst - BC_INST_POWER](n1, n2, &res.d.n, G.prog.scale);
if (s) goto err;
@@ -5784,7 +5789,7 @@ static BcStatus bc_program_logical(char inst)
s = bc_program_binOpPrep(&opd1, &n1, &opd2, &n2, false);
if (s) return s;
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
if (inst == BC_INST_BOOL_AND)
cond = bc_num_cmp(n1, &G.prog.zero) && bc_num_cmp(n2, &G.prog.zero);
@@ -5896,7 +5901,7 @@ static BcStatus bc_program_copyToVar(char *name, bool var)
v = bc_program_search(name, var);
if (var) {
- bc_num_init(&r.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&r.d.n);
bc_num_copy(&r.d.n, n);
}
else {
@@ -6035,7 +6040,7 @@ static BcStatus bc_program_pushVar(char *code, size_t *bgn,
r.t = BC_RESULT_TEMP;
- bc_num_init(&r.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&r.d.n);
bc_num_copy(&r.d.n, num);
}
else {
@@ -6164,7 +6169,7 @@ static BcStatus bc_program_call(char *code, size_t *idx)
v = bc_program_search(a->name, a->idx);
if (a->idx) {
- bc_num_init(¶m.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(¶m.n);
bc_vec_push(v, ¶m.n);
}
else {
@@ -6203,7 +6208,7 @@ static BcStatus bc_program_return(char inst)
bc_num_copy(&res.d.n, num);
}
else {
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
bc_num_zero(&res.d.n);
}
@@ -6261,7 +6266,7 @@ static BcStatus bc_program_builtin(char inst)
return bc_error_variable_is_wrong_type();
#endif
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
if (inst == BC_INST_SQRT) s = bc_num_sqrt(num, &res.d.n, G.prog.scale);
#if ENABLE_BC
@@ -6299,7 +6304,7 @@ static BcStatus bc_program_divmod(void)
s = bc_program_binOpPrep(&opd1, &n1, &opd2, &n2, false);
if (s) return s;
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
bc_num_init(&res2.d.n, n2->len);
s = bc_num_divmod(n1, n2, &res2.d.n, &res.d.n, G.prog.scale);
@@ -6369,7 +6374,7 @@ static void bc_program_stackLen(void)
res.t = BC_RESULT_TEMP;
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
bc_num_ulong2num(&res.d.n, len);
bc_vec_push(&G.prog.results, &res);
}
@@ -6392,7 +6397,7 @@ static BcStatus bc_program_asciify(void)
if (BC_PROG_NUM(r, num)) {
- bc_num_init(&n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&n);
bc_num_copy(&n, num);
bc_num_truncate(&n, n.rdx);
@@ -6610,7 +6615,7 @@ static void bc_program_pushGlobal(char inst)
else
val = (unsigned long) G.prog.ob_t;
- bc_num_init(&res.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&res.d.n);
bc_num_ulong2num(&res.d.n, val);
bc_vec_push(&G.prog.results, &res);
}
@@ -6760,7 +6765,7 @@ static BcStatus bc_program_exec(void)
case BC_INST_BOOL_NOT:
s = bc_program_prep(&ptr, &num);
if (s) return s;
- bc_num_init(&r.d.n, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&r.d.n);
if (!bc_num_cmp(num, &G.prog.zero))
bc_num_one(&r.d.n);
else
@@ -7356,31 +7361,30 @@ static void bc_program_init(void)
memset(&ip, 0, sizeof(BcInstPtr));
/* G.prog.nchars = G.prog.scale = 0; - already is */
-
- bc_num_init(&G.prog.ib, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.ib);
bc_num_ten(&G.prog.ib);
G.prog.ib_t = 10;
- bc_num_init(&G.prog.ob, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.ob);
bc_num_ten(&G.prog.ob);
G.prog.ob_t = 10;
- bc_num_init(&G.prog.hexb, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.hexb);
bc_num_ten(&G.prog.hexb);
G.prog.hexb.num[0] = 6;
#if ENABLE_DC
- bc_num_init(&G.prog.strmb, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.strmb);
bc_num_ulong2num(&G.prog.strmb, UCHAR_MAX + 1);
#endif
- bc_num_init(&G.prog.last, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.last);
bc_num_zero(&G.prog.last);
- bc_num_init(&G.prog.zero, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.zero);
bc_num_zero(&G.prog.zero);
- bc_num_init(&G.prog.one, BC_NUM_DEF_SIZE);
+ bc_num_init_DEF_SIZE(&G.prog.one);
bc_num_one(&G.prog.one);
bc_vec_init(&G.prog.fns, sizeof(BcFunc), bc_func_free);
More information about the busybox-cvs
mailing list