1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

LONG2NUM() should be used for rmatch_offset::{beg,end}

Browse Source 
https://github.com/ruby/ruby/pull/5518#discussion_r809645406
This commit is contained in:
Shugo Maeda 2022-02-18 19:47:08 +09:00
parent e9a2b30744
commit cda5aee74e
No known key found for this signature in database
GPG Key ID: 2DFE34085E97CE47
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
re.c
View File

@ -1230,8 +1230,8 @@ match_offset(VALUE match, VALUE n)
return rb_assoc_new(Qnil, Qnil); return rb_assoc_new(Qnil, Qnil);
update_char_offset(match); update_char_offset(match);
return rb_assoc_new(INT2FIX(RMATCH(match)->rmatch->char_offset[i].beg), return rb_assoc_new(LONG2NUM(RMATCH(match)->rmatch->char_offset[i].beg),
INT2FIX(RMATCH(match)->rmatch->char_offset[i].end)); LONG2NUM(RMATCH(match)->rmatch->char_offset[i].end));
} }
@ -1265,7 +1265,7 @@ match_begin(VALUE match, VALUE n)
return Qnil; return Qnil;
update_char_offset(match); update_char_offset(match);
return INT2FIX(RMATCH(match)->rmatch->char_offset[i].beg); return LONG2NUM(RMATCH(match)->rmatch->char_offset[i].beg);
} }
@ -1299,7 +1299,7 @@ match_end(VALUE match, VALUE n)
return Qnil; return Qnil;
update_char_offset(match); update_char_offset(match);
return INT2FIX(RMATCH(match)->rmatch->char_offset[i].end); return LONG2NUM(RMATCH(match)->rmatch->char_offset[i].end);
} }
/* /*