[m-rev.] for review: add string.format_table/2

Ian MacLarty maclarty at cs.mu.OZ.AU
Fri Feb 4 11:44:02 AEDT 2005


On Thu, Feb 03, 2005 at 10:08:37PM +1100, Julien Fischer wrote:
> 
> Some initial comments:
> 
> On Thu, 3 Feb 2005, Ian MacLarty wrote:
> 
> > +	% format_table(Columns, Seperator) = Table
> > +	% format_table/2 takes a list of columns and a column seperator
> s/seperator/separator/
> 

Fixed.

> > +	% and returns a formatted table, where each field in each column
> > +	% has been aligned and fields are seperated with Seperator.
> s/Seperator/Separator/

Fixed.

> > +	% A newline character is inserted between each row.
> > +	%
> What happens if the columns have different lengths?
> 

I've added the following to the comment describing format_string:

	% If the columns are not all the same length then an exception is
	% thrown.

Thanks,
Ian.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list