summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorYukihiro "Matz" Matsumoto <[email protected]>2019-08-20 22:50:21 +0900
committerYukihiro "Matz" Matsumoto <[email protected]>2019-08-20 22:50:21 +0900
commit4c001673b6dbc321cb0988f1c3e25f0d47d4b529 (patch)
tree6f15458aeb836575fd3a1d88a454964430f9d1b7
parent183fb961b313ab5dc6bcb15c952c34a1aa842fb7 (diff)
downloadmruby-4c001673b6dbc321cb0988f1c3e25f0d47d4b529.tar.gz
mruby-4c001673b6dbc321cb0988f1c3e25f0d47d4b529.zip
Avoid C++ strict aliasing warning.
Based on code proposed by @dearblue to avoid the warning: `dereferencing type-punned pointer will break strict-aliasing rules`.
-rw-r--r--include/mruby/array.h10
1 files changed, 2 insertions, 8 deletions
diff --git a/include/mruby/array.h b/include/mruby/array.h
index dde9f80cb..9664214d6 100644
--- a/include/mruby/array.h
+++ b/include/mruby/array.h
@@ -33,9 +33,7 @@ struct RArray {
} aux;
mrb_value *ptr;
} heap;
-#if defined(__cplusplus)
- mrb_value ary[MRB_ARY_EMBED_LEN_MAX];
-#endif
+ void *ary[3];
} as;
};
@@ -48,11 +46,7 @@ struct RArray {
#define ARY_UNSET_EMBED_FLAG(a) ((a)->flags &= ~(MRB_ARY_EMBED_MASK))
#define ARY_EMBED_LEN(a) ((mrb_int)(((a)->flags & MRB_ARY_EMBED_MASK) - 1))
#define ARY_SET_EMBED_LEN(a,len) ((a)->flags = ((a)->flags&~MRB_ARY_EMBED_MASK) | ((uint32_t)(len) + 1))
-#if defined(__cplusplus)
-#define ARY_EMBED_PTR(a) (a)->as.ary
-#else
-#define ARY_EMBED_PTR(a) ((mrb_value*)(&(a)->as))
-#endif
+#define ARY_EMBED_PTR(a) ((mrb_value*)(&(a)->as.ary))
#define ARY_LEN(a) (ARY_EMBED_P(a)?ARY_EMBED_LEN(a):(a)->as.heap.len)
#define ARY_PTR(a) (ARY_EMBED_P(a)?ARY_EMBED_PTR(a):(a)->as.heap.ptr)