Refactor the mrb_get_args()
function
#5540
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ARGV
macro.The current path doesn't go into the mruby VM and there's also no need to separate variables.
mrb_ensure_string_type()
to check the string instead ofmrb_to_str()
.This is for consistency with array and hash.
mrb_ensure_array_type()
to check the array instead ofto_ary()
.mrb_ensure_hash_type()
to check the hash instead ofto_hash()
.ensure_class_type()
to check class and module.mrb_int
toint
.Even if it is
int16_t
, it is enough.mrb_int
is overkill, especially ifMRB_32BIT
andMRB_INT64
are defined.