Project

General

Profile

Actions

Bug #6285

closed

Documents of IO#read, IO#readpartial, etc. should describe how they are executed when second argument is not empty String

Added by nobuoka (yu nobuoka) almost 12 years ago. Updated almost 12 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.0.0dev (2012-04-12 trunk 35312) [x86_64-linux]
Backport:
[ruby-core:44314]

Description

IO#read, IO#readpartial, IO#read_nonblock, and IO#sysread methods can receive a String object as second argument ((|outbuf|)). If it is present, it will receive the data. This is documented.

So, when I pass the empty String object to the method, it is expected that the object contains the data after the method is executed.

r,w = IO.pipe
w << 'abcd'
w.close
r.read( 4, buf = '' )
buf #=> 'abcd'

However, when I pass the not empty String object to the method as second argument, what happen?

r,w = IO.pipe
w << 'abcd'
w.close
r.read( 4, buf = '012345' )
buf #=> 'abcd'? or 'abcd45'? or '012345abcd'?

In current implementation, the ((|outbuf|)) contain only the received data after the method is executed even though it is not empty at the beginning. I think it should be documented.

The same applies to ARGF.read, ARGF.readpartial, StringIO#read, StringIO#readpartial, StringIO#sysread, StringIO#read_nonblock methods.

I wrote documents and tests. Please see attached file.


Files

__diff__.txt (8.03 KB) __diff__.txt nobuoka (yu nobuoka), 04/13/2012 02:47 AM

Updated by drbrain (Eric Hodel) almost 12 years ago

  • Category set to doc
  • Status changed from Open to Assigned
  • Assignee set to drbrain (Eric Hodel)
Actions #2

Updated by drbrain (Eric Hodel) almost 12 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r35391.
yu, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • io.c (io_readpartial): Document the output buffer parameter is
    overwritten with the read contents even when non-empty.
    Patch by yu nobuoka. [ruby-trunk - Bug #6285]
  • io.c (io_read_nonblock): ditto.
  • io.c (io_read): ditto.
  • io.c (rb_io_sysread): ditto.
  • io.c (argf_read): ditto.
  • io.c (argf_readpartial): ditto.
  • ext/stringio/stringio.c (strio_read): ditto.
  • test/ruby/test_argf.rb (class TestArgf): Add test for existing
    behavior of read outbuf.
  • test/ruby/test_io.rb (class TestIO): ditto.
  • test/stringio/test_stringio.rb (class TestStringIO): ditto.
Actions

Also available in: Atom PDF

Like0
Like0Like0