mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
* ext/stringio/stringio.c (check_modifiable): performance
improvement. avoid calling rb_str_modify() twice. * ext/stringio/stringio.c (strio_ungetc): ditto. * ext/stringio/stringio.c (strio_putc): ditto. * ext/stringio/stringio.c (strio_write): ditto, and use rb_str_cat() as possible. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@2345 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
f240fc637c
commit
e223518948
3 changed files with 29 additions and 6 deletions
|
|
@ -125,7 +125,6 @@ check_modifiable(ptr)
|
|||
if (OBJ_FROZEN(ptr->string)) {
|
||||
rb_raise(rb_eIOError, "not modifiable string");
|
||||
}
|
||||
rb_str_modify(ptr->string);
|
||||
}
|
||||
|
||||
static VALUE strio_s_allocate _((VALUE));
|
||||
|
|
@ -548,6 +547,7 @@ strio_ungetc(self, ch)
|
|||
if ((unsigned char)RSTRING(ptr->string)->ptr[--ptr->pos] !=
|
||||
(unsigned char)cc) {
|
||||
check_modifiable(ptr);
|
||||
rb_str_modify(ptr->string);
|
||||
RSTRING(ptr->string)->ptr[ptr->pos] = cc;
|
||||
}
|
||||
}
|
||||
|
|
@ -740,10 +740,18 @@ strio_write(self, str)
|
|||
if (ptr->flags & STRIO_APPEND) {
|
||||
ptr->pos = RSTRING(ptr->string)->len;
|
||||
}
|
||||
if (ptr->pos + len > RSTRING(ptr->string)->len) {
|
||||
rb_str_resize(ptr->string, ptr->pos + len);
|
||||
if (ptr->pos == RSTRING(ptr->string)->len) {
|
||||
rb_str_cat(ptr->string, RSTRING(str)->ptr, len);
|
||||
}
|
||||
else {
|
||||
if (ptr->pos + len > RSTRING(ptr->string)->len) {
|
||||
rb_str_resize(ptr->string, ptr->pos + len);
|
||||
}
|
||||
else {
|
||||
rb_str_modify(ptr->string);
|
||||
}
|
||||
rb_str_update(ptr->string, ptr->pos, len, str);
|
||||
}
|
||||
rb_str_update(ptr->string, ptr->pos, len, str);
|
||||
ptr->pos += len;
|
||||
return LONG2NUM(len);
|
||||
}
|
||||
|
|
@ -768,6 +776,9 @@ strio_putc(self, ch)
|
|||
if (ptr->pos >= RSTRING(ptr->string)->len) {
|
||||
rb_str_resize(ptr->string, ptr->pos + 1);
|
||||
}
|
||||
else {
|
||||
rb_str_modify(ptr->string);
|
||||
}
|
||||
RSTRING(ptr->string)->ptr[ptr->pos++] = c;
|
||||
return ch;
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue