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-set: convert kset_copy_* macros to functions

Browse Source 
Convert kset_copy_merge and kset_copy_replace from macros to static
functions for better maintainability and debugging.

Benefits:
- Better debugging: can set breakpoints and step through code
- Improved type safety: proper function parameter checking
- Cleaner code: no macro expansion bloat at call sites
- Better error messages: meaningful function names in stack traces
- Easier maintenance: functions are simpler to modify than complex macros

The operations are substantial enough (memory allocation, loops with GC
management) that function call overhead is negligible compared to the
actual work performed.

Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
Yukihiro "Matz" Matsumoto
2025-08-01 16:52:25 +09:00
parent 76188b46ef
commit 8f7bfa4f68
1 changed files with 25 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-set/src/set.c
+25 -21
View File
@@ -50,27 +50,6 @@ typedef khint_t kset_iter_t;
#define kset_put2(mrb, s, k, r) kh_put2(set_val, mrb, s, k, r)
#define kset_get(mrb, s, k) kh_get(set_val, mrb, s, k)
#define kset_del(mrb, s, k) kh_del(set_val, mrb, s, k)
/* Copy all elements from src to dst (merge operation) */
#define kset_copy_merge(mrb, dst, src) do { \
if (!kset_is_empty(src)) { \
int __ai = mrb_gc_arena_save(mrb); \
KSET_FOREACH(src, __k) { \
kset_put(mrb, dst, kset_key(src, __k)); \
mrb_gc_arena_restore(mrb, __ai); \
} \
} \
} while(0)
/* Replace dst with a copy of src */
#define kset_copy_replace(mrb, dst, src) do { \
kset_t *__tmp = kh_copy(set_val, mrb, src); \
if (__tmp) { \
kset_destroy_data(mrb, dst); \
*(dst) = *__tmp; \
mrb_free(mrb, __tmp); \
} \
} while(0)
#define kset_exist(s, k) kh_exist(set_val, s, k)
#define kset_key(s, k) kh_key(set_val, s, k)
#define kset_size(s) kh_size(s)
@@ -82,6 +61,31 @@ typedef khint_t kset_iter_t;
#define kset_is_uninitialized(s) ((s)->data == NULL)
#define kset_is_empty(s) (kset_is_uninitialized(s) || kset_size(s) == 0)
/* Copy all elements from src to dst (merge operation) */
static void
kset_copy_merge(mrb_state *mrb, kset_t *dst, kset_t *src)
{
if (!kset_is_empty(src)) {
int ai = mrb_gc_arena_save(mrb);
KSET_FOREACH(src, k) {
kset_put(mrb, dst, kset_key(src, k));
mrb_gc_arena_restore(mrb, ai);
}
}
}
/* Replace dst with a copy of src */
static void
kset_copy_replace(mrb_state *mrb, kset_t *dst, kset_t *src)
{
kset_t *tmp = kh_copy(set_val, mrb, src);
if (tmp) {
kset_destroy_data(mrb, dst);
*dst = *tmp;
mrb_free(mrb, tmp);
}
}
/* Embedded set structure in RSet - exactly 3 pointers */
struct RSet {
MRB_OBJECT_HEADER;