On Wed, Oct 13, 2010 at 04:31:00PM +0530, Dipankar Patro wrote: > Thanks for reviewing it, James.
Sorry, it was late, and I misunderstood the code you had written ... I withdraw my comment "You are effectively repeating the previous if statement but using the output ... seems a bit obscure." Apart from the lack of text wrap in the commit message, I've no issues with the patch. Reviewed-by: James Cameron <qu...@laptop.org> -- James Cameron http://quozl.linux.org.au/ _______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel