admin/mruby-mruby
1
0
Fork 0
mirror of https://github.com/mruby/mruby synced 2026-06-08 16:11:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

bigint.c: inline usub_inplace() at call site

Browse Source 
Function was only called once and contained just 2 lines of code.
Inlining directly reduces code size and improves clarity.

Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
Yukihiro "Matz" Matsumoto
2026-01-16 17:11:43 +09:00
parent ad1254e3cc
commit 8089fdadf0
1 changed files with 2 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-bigint/core/bigint.c
+2 -12
View File
@@ -400,17 +400,6 @@ usub(mpz_t *z, mpz_t *y, mpz_t *x)
z->sz = digits(z);
}
/* In-place unsigned subtraction: a -= b */
/* Precondition: abs(a) >= abs(b), no allocation needed */
static void
usub_inplace(mpz_t *a, mpz_t *b)
{
/* a->sz >= b->sz is guaranteed by precondition */
mpn_sub(a->p, a->p, a->sz, b->p, b->sz);
/* Normalize: find actual size after subtraction */
a->sz = digits(a);
}
/* compare abs(x) and abs(y) */
static int
ucmp(mpz_t *y, mpz_t *x)
@@ -605,7 +594,8 @@ mpz_sub(mpz_ctx_t *ctx, mpz_t *z, mpz_t *x, mpz_t *y)
{
/* In-place optimization: z == x, both positive, x >= y */
if (z == x && x->sn > 0 && y->sn > 0 && ucmp(x, y) >= 0) {
usub_inplace(x, y);
mpn_sub(x->p, x->p, x->sz, y->p, y->sz);
x->sz = digits(x);
if (x->sz == 0) x->sn = 0;
return;
}