diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2019-08-20 22:50:21 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2019-08-20 22:50:21 +0900 |
| commit | 4c001673b6dbc321cb0988f1c3e25f0d47d4b529 (patch) | |
| tree | 6f15458aeb836575fd3a1d88a454964430f9d1b7 | |
| parent | 183fb961b313ab5dc6bcb15c952c34a1aa842fb7 (diff) | |
| download | mruby-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.h | 10 |
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) |
