HASUMI Hitoshi cc16afb7f3 mrb_str_aset_m() should return replace instead of str ...

`string[]=(idx, replace)` should return `replace`.

## Actual (wrong)

```
string.[]=(idx, replace) → string
string.[]=(idx, len, replace) → string
```

## Expected

```
string.[]=(idx, replace) → replace
string.[]=(idx, len, replace) → replace
```

## Sidenote

As of the current mruby-compiler, `(string[idx] = 'X')` creates not only "CALL_NODE" but also "ASGN_NODE" and "OP_MOVE", overriding the wrong return value.
On the other hand, `string.[]=(idx, 'X')` creates only "CALL_NODE", exposing the wrong return value.

If my new mruby-compiler2, leveraging Prism, took the place of official compiler, `(string[idx] = 'X')` and `string.[]=(idx, 'X')` would be going to generate the same VM code without "OP_MOVE".
So I paranoidly added tests.

FYI: You can find how the new mruby-compiler2's AST and VM code look like in mruby/c's issue (mruby/c had the same bug): https://github.com/mrubyc/mrubyc/pull/210

2024-07-16 18:55:26 +09:00

..

t

mrb_str_aset_m() should return replace instead of str

2024-07-16 18:55:26 +09:00

assert.rb

ruby: standardize whitespace

2022-10-31 16:25:56 +10:00

bintest.rb

Remove unneeded return keyword from last line in Ruby functions

2022-12-27 11:42:48 +10:00