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

Don't refetch ptr and len

Browse Source 
The call to RSTRING_GETMEM already fetched the pointer and length, so we
don't need to fetch it again.
This commit is contained in:
Peter Zhu 2023-05-31 09:57:33 -04:00
parent d5a5333168
commit 8a8618d4f3
Notes: git 2023-06-01 12:32:50 +00:00 
Merged: https://github.com/ruby/ruby/pull/7879
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
string.c
View File

@ -3057,13 +3057,9 @@ str_buf_cat4(VALUE str, const char *ptr, long len, bool keep_cr)
if (STR_EMBED_P(str)) { if (STR_EMBED_P(str)) {
capa = str_embed_capa(str) - termlen; capa = str_embed_capa(str) - termlen;
sptr = RSTRING(str)->as.embed.ary;
olen = RSTRING_EMBED_LEN(str);
} }
else { else {
capa = RSTRING(str)->as.heap.aux.capa; capa = RSTRING(str)->as.heap.aux.capa;
sptr = RSTRING(str)->as.heap.ptr;
olen = RSTRING(str)->as.heap.len;
} }
if (olen > LONG_MAX - len) { if (olen > LONG_MAX - len) {
rb_raise(rb_eArgError, "string sizes too big"); rb_raise(rb_eArgError, "string sizes too big");