The function guarantees it will write everything, so I will update the comment.

> On 7 Dec 2013, at 08:37, Richard Mortier <notifications@github.com> wrote:
> 
> In lib/v1.mli:
> 
> > @@ -156,16 +156,16 @@ module type CONSOLE = sig
> >    (** [write_all t buf off len] is a thread that writes [String.sub buf
> >        off len] to the console [t] and returns [len] when done. Raises
> >        {!Invalid_argument} if [len > buf - off]. *)
> > -  val write_all : t -> string -> int -> int -> int io
> > +  val write_all : t -> string -> int -> int -> unit io
> is the [len] return value wrapped in the unit io then? or does the comment need updating?
> 
> —
> Reply to this email directly or view it on GitHub.