Glad I could clarify my modifications. :)<br>I will send the corrected e-mail as soon as possible.<br><br><br><div class="gmail_quote">On Thu, Oct 14, 2010 at 4:29 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">On Wed, Oct 13, 2010 at 04:31:00PM +0530, Dipankar Patro wrote:<br>
> Thanks for reviewing it, James.<br>
<br>
</div>Sorry, it was late, and I misunderstood the code you had written ... I<br>
withdraw my comment "You are effectively repeating the previous if<br>
<div>statement but using the output ... seems a bit obscure."<br><br> </div></blockquote><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Apart from the lack of text wrap in the commit message, I've no issues<br>
with the patch.<br>
<br>
Reviewed-by: James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>><br>
<div><div></div><div class="h5"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</div></div></blockquote></div><br>