diff options
| author | Daniel Bovensiepen <[email protected]> | 2012-12-05 01:01:34 +0800 |
|---|---|---|
| committer | Daniel Bovensiepen <[email protected]> | 2012-12-05 01:01:34 +0800 |
| commit | 471c00a31dcaa04d4658de9b59fc23b24479d2fd (patch) | |
| tree | b53fb2f431f52002c0a2a908623beaa3c1a929e7 /src/array.c | |
| parent | 40bfc86a5e0a1a70f904a080569da9662747181f (diff) | |
| parent | 674c55eb7d0a4338d4500516b6a3e9aba56ee61e (diff) | |
| download | mruby-471c00a31dcaa04d4658de9b59fc23b24479d2fd.tar.gz mruby-471c00a31dcaa04d4658de9b59fc23b24479d2fd.zip | |
Merge remote-tracking branch 'mruby/master' into mrbgems
Diffstat (limited to 'src/array.c')
| -rw-r--r-- | src/array.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/src/array.c b/src/array.c index ea461e029..d4f5d1c5e 100644 --- a/src/array.c +++ b/src/array.c @@ -719,24 +719,21 @@ mrb_ary_aset(mrb_state *mrb, mrb_value self) mrb_get_args(mrb, "*", &argv, &argc); switch(argc) { case 2: - if (mrb_fixnum_p(argv[0])) { - mrb_ary_set(mrb, self, mrb_fixnum(argv[0]), argv[1]); - } - else { + if (!mrb_fixnum_p(argv[0])) { /* Should we support Range object for 1st arg ? */ mrb_raise(mrb, E_TYPE_ERROR, "expected Fixnum for 1st argument"); } - break; + mrb_ary_set(mrb, self, mrb_fixnum(argv[0]), argv[1]); + return argv[1]; case 3: mrb_ary_splice(mrb, self, mrb_fixnum(argv[0]), mrb_fixnum(argv[1]), argv[2]); - break; + return argv[2]; default: mrb_raise(mrb, E_ARGUMENT_ERROR, "wrong number of arguments"); + return mrb_nil_value(); } - - return self; } mrb_value |
