diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | io.c | 23 |
2 files changed, 15 insertions, 14 deletions
@@ -1,3 +1,9 @@ +Wed Jan 14 12:38:02 2009 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * io.c (appendline): reformed. + + * io.c (rb_io_s_pipe): reduced nest of rb_ensure of main block. + Tue Jan 13 21:45:53 2009 Yuki Sonoda (Yugui) <yugui@yugui.jp> * lib/ostruct.rb (OpenStruct#inspect): fixed the recursion check. @@ -2002,7 +2002,7 @@ appendline(rb_io_t *fptr, int delim, VALUE *strp, long *lp) if (NEED_READCONV(fptr)) { make_readconv(fptr, 0); - while (1) { + do { const char *p, *e; int searchlen; if (fptr->cbuf_len) { @@ -2036,15 +2036,12 @@ appendline(rb_io_t *fptr, int delim, VALUE *strp, long *lp) return (unsigned char)RSTRING_PTR(str)[RSTRING_LEN(str)-1]; } } - - if (more_char(fptr) == -1) { - *lp = limit; - return EOF; - } - } + } while (more_char(fptr) != -1); + *lp = limit; + return EOF; } - while (1) { + do { long pending = READ_DATA_PENDING_COUNT(fptr); if (pending > 0) { const char *p = READ_DATA_PENDING_PTR(fptr); @@ -2068,15 +2065,13 @@ appendline(rb_io_t *fptr, int delim, VALUE *strp, long *lp) *lp = limit; if (e) return delim; if (limit == 0) - return (unsigned char)RSTRING_PTR(str)[RSTRING_LEN(str)-1]; + return (unsigned char)RSTRING_PTR(str)[RSTRING_LEN(str)-1]; } rb_thread_wait_fd(fptr->fd); rb_io_check_closed(fptr); - if (io_fillbuf(fptr) < 0) { - *lp = limit; - return EOF; - } - } + } while (io_fillbuf(fptr) >= 0); + *lp = limit; + return EOF; } static inline int |