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

mruby-bigint: convert simple add/sub operations to use simplified api

Browse Source 
Simplified several Ruby bigint operations by removing redundant mpz_init calls:
- mrb_bint_add_n: mpz_add now handles initialization internally
- mrb_bint_sub_n: mpz_sub now handles initialization internally
- mrb_bint_add_ii: mpz_add now handles initialization internally
- mrb_bint_sub_ii: mpz_sub now handles initialization internally

These changes demonstrate the benefit of the *_auto API - operations that
previously required separate init + operation calls now work with just
the operation call, as the simplified functions handle memory allocation
automatically.

Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
Yukihiro "Matz" Matsumoto
2025-07-26 22:31:16 +09:00
parent 9dda6c0579
commit e9c896408a
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-bigint/core/bigint.c
-4
View File
@@ -3005,7 +3005,6 @@ mrb_bint_add_n(mrb_state *mrb, mrb_value x, mrb_value y)
}
y = mrb_as_bint(mrb, y);
bint_as_mpz(RBIGINT(y), &b);
mpz_init(&ctx, &z);
mpz_add(&ctx, &z, &a, &b);
struct RBigint *v = bint_new(mrb, &z);
return mrb_obj_value(v);
@@ -3049,7 +3048,6 @@ mrb_bint_sub_n(mrb_state *mrb, mrb_value x, mrb_value y)
}
y = mrb_as_bint(mrb, y);
bint_as_mpz(RBIGINT(y), &b);
mpz_init(&ctx, &z);
mpz_sub(&ctx, &z, &a, &b);
struct RBigint *v = bint_new(mrb, &z);
return mrb_obj_value(v);
@@ -3145,7 +3143,6 @@ mrb_bint_add_ii(mrb_state *mrb, mrb_int x, mrb_int y)
mpz_init_set_int(&ctx, &a, x);
mpz_init_set_int(&ctx, &b, y);
mpz_init(&ctx, &z);
mpz_add(&ctx, &z, &a, &b);
mpz_clear(&ctx, &a);
mpz_clear(&ctx, &b);
@@ -3160,7 +3157,6 @@ mrb_bint_sub_ii(mrb_state *mrb, mrb_int x, mrb_int y)
mpz_init_set_int(&ctx, &a, x);
mpz_init_set_int(&ctx, &b, y);
mpz_init(&ctx, &z);
mpz_sub(&ctx, &z, &a, &b);
mpz_clear(&ctx, &a);
mpz_clear(&ctx, &b);