diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2021-08-30 07:35:58 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2021-09-01 07:00:54 +0900 |
| commit | e86c9cb57c0ee799ca66d18612e15c6c8983f872 (patch) | |
| tree | f60224602ddf0cfb7bcca1546648b5d42d6e2a95 /mrbgems/mruby-kernel-ext | |
| parent | 572a43de8406da89daeaa9b433761da762d7b1c4 (diff) | |
| download | mruby-e86c9cb57c0ee799ca66d18612e15c6c8983f872.tar.gz mruby-e86c9cb57c0ee799ca66d18612e15c6c8983f872.zip | |
Do no use return values from `mrb_ensure_` functions.
They return the checking argument without modification, so the values
are already there. Maybe we should change the return type to `void` but
keep them unchanged for compatibility.
Diffstat (limited to 'mrbgems/mruby-kernel-ext')
| -rw-r--r-- | mrbgems/mruby-kernel-ext/src/kernel.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mrbgems/mruby-kernel-ext/src/kernel.c b/mrbgems/mruby-kernel-ext/src/kernel.c index 15b4c5f72..17692a801 100644 --- a/mrbgems/mruby-kernel-ext/src/kernel.c +++ b/mrbgems/mruby-kernel-ext/src/kernel.c @@ -202,7 +202,8 @@ mrb_f_hash(mrb_state *mrb, mrb_value self) if (mrb_nil_p(arg) || (mrb_array_p(arg) && RARRAY_LEN(arg) == 0)) { return mrb_hash_new(mrb); } - return mrb_ensure_hash_type(mrb, arg); + mrb_ensure_hash_type(mrb, arg); + return arg; } void |