Can't we add a hook that doesn't affect already checked in files? Or make a giant "fix style" commit?<br><br>David<br>
<br><br><div class="gmail_quote">On Tue, Apr 5, 2011 at 12:23 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I recall that VTK does not have that hook. It is because so much of vtk already has trailing spaces. VTYK has never (at least for many years) used a style checker and their are many violations.<br><font color="#888888"><br>
Bill</font><div><div></div><div class="h5"><br><br><br><div class="gmail_quote">
On Tue, Apr 5, 2011 at 12:19 PM, David Doria <span dir="ltr"><<a href="mailto:daviddoria@gmail.com" target="_blank">daviddoria@gmail.com</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">

I created a brand new fresh git clone this morning and ran it.<br><font color="#888888"><br>David</font><div><div></div><div><br>
<br><br><div class="gmail_quote">On Tue, Apr 5, 2011 at 12:18 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</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">


The SetupForDevelopment.sh script  should have taken care of it. When was the last time you ran it?<br><br><div class="gmail_quote">On Tue, Apr 5, 2011 at 12:07 PM, Code Review <span dir="ltr"><<a href="mailto:review@kitware.com" target="_blank">review@kitware.com</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>From David Doria <<a href="mailto:daviddoria@gmail.com" target="_blank">daviddoria@gmail.com</a>>:<br>




<br>
</div>David Doria has posted comments on this change.<div><br>
<div><br>
Change subject: ENH: Added vector-on-plane projection and related test<br>
......................................................................<br>
<br>
<br>
</div>Patch Set 1:<br>
<br></div>
I ran the SetupForDevelopment.sh script - I thought this was supposed to take care of everything.<div><br>
<div><br>
--<br>
To view, visit <a href="http://review.source.kitware.com/1317" target="_blank">http://review.source.kitware.com/1317</a><br>
To unsubscribe, visit <a href="http://review.source.kitware.com/settings" target="_blank">http://review.source.kitware.com/settings</a><br>
<br>
</div>Gerrit-MessageType: comment<br>
<div>Gerrit-Change-Id: I7220a5b0c2c24dffe99b354b7605c8a9f2d91865<br>
Gerrit-PatchSet: 1<br>
Gerrit-Project: VTK<br>
Gerrit-Branch: master<br>
Gerrit-Owner: David Doria <<a href="mailto:daviddoria@gmail.com" target="_blank">daviddoria@gmail.com</a>><br>
Gerrit-Reviewer: Arnaud Gelas <<a href="mailto:arnaud_gelas@hms.harvard.edu" target="_blank">arnaud_gelas@hms.harvard.edu</a>><br>
Gerrit-Reviewer: Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
</div>Gerrit-Reviewer: David Doria <<a href="mailto:daviddoria@gmail.com" target="_blank">daviddoria@gmail.com</a>><br>
</div></blockquote></div><br>
</blockquote></div><br>
</div></div></blockquote></div><br>
</div></div></blockquote></div><br>