Another one of those places I'd say should be done with Logger instead of
puts. It's already part of ruby, so it doesn't cost anything to load and
use it (afaik).


On Mon, Feb 18, 2013 at 4:08 PM, Ryan Waldron <notifications@github.com>wrote:

> In lib/asciidoctor/table.rb:
>
> > +    advance
> > +    nil
> > +  end
> > +
> > +  # Public: Close the current cell, instantiate a new Table::Cell, add it to
> > +  # the current row and, if the number of expected columns for the current
> > +  # row has been met, close the row and begin a new one.
> > +  #
> > +  # returns nothing
> > +  def close_cell(eol = false)
> > +    cell_text = @buffer.strip
> > +    @buffer = ''
> > +    if format == 'psv'
> > +      cell_spec = take_cell_spec
> > +      if cell_spec.nil?
> > +        puts 'asciidoctor: ERROR: table missing leading separator, recovering automatically'
>
> Mean to leave the puts in here?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/asciidoctor/asciidoctor/pull/181/files#r3055127>.
>
>



-- 
Jason Porter
http://en.gravatar.com/lightguardjp