<div class="gmail_quote">On Wed, Mar 7, 2012 at 10:42 AM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, 7 Mar 2012 10:39:15 -0500, Aashish Chaudhary said:<br>
<br>
>Cool! but looks like it has introduced white space error (I have to verify<br>
>it using vim but gerrit shows it as ws error)<br>
><br>
><a href="http://review.source.kitware.com/#/c/4555/1/Common/vtkParametricDini.h" target="_blank">http://review.source.kitware.com/#/c/4555/1/Common/vtkParametricDini.h</a><br>
<br>
</div>It seems to me gerrit is showing that a trailing space was removed... a good thing, no?<br></blockquote><div><br></div><div>That's the part I am not sure, if the WS is gone, good! (though I would say that this is a side effect and not the original intent so not exactly great) but I am not sure why would gerrit show it as a WS error (and to see it listed as WS error you have to login to gerrit) </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>| Aashish Chaudhary <br>| R&D Engineer         <br>| Kitware Inc.            <br>| <a href="http://www.kitware.com">www.kitware.com</a>    <br>