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: simplify MPZ_CTX_INIT with positional initializer

Browse Source 
Per gemini-code-assist review on #6791: replace member-by-member
assignment with C89/C++98-style positional aggregate initialization,
keeping the macro a single declaration and matching the existing
pool_storage initializer style in the same macro.

Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
Yukihiro "Matz" Matsumoto
2026-04-15 09:40:06 +09:00
parent bbf7c43c37
commit 81bcd4e931
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-bigint/core/bigint.c
+5 -6
View File
@@ -51,15 +51,14 @@ typedef struct mpz_context {
} mpz_ctx_t;
/* Convenience macros for context creation.
* Uses per-member assignment instead of a C99 compound literal with
* designated initializers so the file compiles as C++ on legacy
* toolchains (pre-C++20). */
* Uses positional aggregate initialization instead of a C99 compound
* literal with designated initializers, so the file compiles as C++
* on legacy toolchains (pre-C++20). Member order must match the
* mpz_context struct declaration above. */
#define MPZ_CTX_INIT(mrb_ptr, ctx, pool_ptr) \
mpz_pool_t pool ## _storage = {{0}};\
mpz_pool_t *pool_ptr = &pool ## _storage;\
mpz_ctx_t ctx ## _struct; \
ctx ## _struct.mrb = (mrb_ptr); \
ctx ## _struct.pool = (pool_ptr); \
mpz_ctx_t ctx ## _struct = { (mrb_ptr), (pool_ptr) }; \
mpz_ctx_t *ctx = &(ctx ## _struct);
/* Access macros for readability */