0
@@ -61,9 +61,9 @@
0
   end
0
   
0
   def redirect_streams
0
-    @stdin.bind_input(STDIN)
0
-    @stdout.bind_output(STDOUT)
0
-    @stderr.bind_output(STDERR)
0
+    @stdin.bind_writer(STDIN)
0
+    @stdout.bind_reader(STDOUT)
0
+    @stderr.bind_reader(STDERR)
0
   end
0
   
0
   def process_streams(stdin, stdout, stderr, child_pid)

0
@@ -1,31 +1,41 @@
0
 require "ropen"
0
 
0
-# TODO: document me
0
-
0
+# A bidirectional IO pipe wrapper.
0
 class Ropen::Pipe
0
-  attr_reader :reader, :writer
0
+  # An +IO+ instance from which the output of the pipe can be read.
0
+  attr_reader :reader
0
+  # An +IO+ instance to which the input of the pipe can be written.
0
+  attr_reader :writer
0
   
0
+  # Creates a new Ropen::Pipe with a unconnected +reader+ and +writer+.
0
   def initialize
0
     @reader, @writer = IO.pipe
0
   end
0
   
0
-  def bind_output(stream)
0
+  # Binds the pipe's +reader+ to +stream+ and closes the +writer+ to make it a
0
+  # read-only pipe (e.g., +STDOUT+ or +STDERR+).
0
+  def bind_reader(stream)
0
     swap_streams(stream, @reader, @writer)
0
   end
0
   
0
-  def bind_input(stream)
0
+  # Binds the pipe's +writer+ to +stream+ and closes the +reader+ to make it a
0
+  # write-only pipe (e.g., +STDIN+).
0
+  def bind_writer(stream)
0
     swap_streams(stream, @writer, @reader)
0
   end
0
   
0
+  # Closes both the +reader+ and +writer+, if either are open.
0
   def close
0
     close_reader
0
     close_writer
0
   end
0
   
0
+  # Closes the +reader+ if it is open.
0
   def close_reader
0
     @reader.close unless @reader.closed?
0
   end
0
   
0
+  # Closes the +writer+ if it is open.
0
   def close_writer
0
     @writer.close unless @writer.closed?
0
   end

0
@@ -22,7 +22,7 @@
0
     @pipe.reader.should_receive(:close)
0
     STDOUT.should_receive(:reopen).with(@pipe.writer)
0
     @pipe.writer.should_receive(:close)
0
-    @pipe.bind_output(STDOUT)
0
+    @pipe.bind_reader(STDOUT)
0
   end
0
   
0
   it "should bind a reader to a stream" do
0
@@ -30,7 +30,7 @@
0
     STDIN.should_receive(:reopen).with(@pipe.reader)
0
     @pipe.reader.should_receive(:close)
0
     
0
-    @pipe.bind_input(STDIN)
0
+    @pipe.bind_writer(STDIN)
0
   end
0
   
0
   it "should close the reader" do