Commit 1df355e3 authored by Masaki Muranaka's avatar Masaki Muranaka

Reduce implicit type casts.

parent 2f8dbd8e
......@@ -79,9 +79,9 @@ mrb_ary_new(mrb_state *mrb)
*
*/
static inline void
array_copy(mrb_value *dst, const mrb_value *src, size_t size)
array_copy(mrb_value *dst, const mrb_value *src, mrb_int size)
{
size_t i;
mrb_int i;
for (i = 0; i < size; i++) {
dst[i] = src[i];
......@@ -238,7 +238,9 @@ mrb_ary_s_create(mrb_state *mrb, mrb_value self)
int len;
mrb_get_args(mrb, "*", &vals, &len);
return mrb_ary_new_from_values(mrb, len, vals);
mrb_assert(len <= MRB_INT_MAX); /* A rare case. So choosed assert() not raise(). */
return mrb_ary_new_from_values(mrb, (mrb_int)len, vals);
}
static void
......
......@@ -962,8 +962,9 @@ fix_xor(mrb_state *mrb, mrb_value x)
#define NUMERIC_SHIFT_WIDTH_MAX (MRB_INT_BIT-1)
static mrb_value
lshift(mrb_state *mrb, mrb_int val, size_t width)
lshift(mrb_state *mrb, mrb_int val, mrb_int width)
{
mrb_assert(width >= 0);
if (width > NUMERIC_SHIFT_WIDTH_MAX) {
mrb_raisef(mrb, E_RANGE_ERROR, "width(%S) > (%S:MRB_INT_BIT-1)",
mrb_fixnum_value(width),
......@@ -974,8 +975,9 @@ lshift(mrb_state *mrb, mrb_int val, size_t width)
}
static mrb_value
rshift(mrb_int val, size_t width)
rshift(mrb_int val, mrb_int width)
{
mrb_assert(width >= 0);
if (width >= NUMERIC_SHIFT_WIDTH_MAX) {
if (val < 0) {
val = -1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment