Skip to content

Commit

Permalink
Remove unnecessary inline declaration; ref #3409
Browse files Browse the repository at this point in the history
  • Loading branch information
matz committed Jan 23, 2017
1 parent 9d0f07e commit fdec607
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/array.c
Original file line number Diff line number Diff line change
Expand Up @@ -631,7 +631,7 @@ mrb_ary_splice(mrb_state *mrb, mrb_value ary, mrb_int head, mrb_int len, mrb_val
}
ary_fill_with_nil(a->ptr + a->len, head - a->len);
if (argc > 0) {
for (int i = 0; i < argc; i++) {
for (i = 0; i < argc; i++) {

This comment has been minimized.

Copy link
@ksss

ksss Jan 23, 2017

Contributor

I think, array_copy(a->ptr + head, argv, argc) is better.

This comment has been minimized.

Copy link
@matz

matz Jan 23, 2017

Author Member

Send PR, please.

This comment has been minimized.

Copy link
@ksss

ksss Jan 23, 2017

Contributor

OK, I sent it #3412

a->ptr[head + i] = argv[i];
}
}
Expand Down

0 comments on commit fdec607

Please sign in to comment.