<HTML>
<HEAD>
<TITLE>Re: [vtk-developers] New git commit hooks</TITLE>
</HEAD>
<BODY>
<FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'>I could, go either way on the trailing whitespace. Trailing whitespace doesn’t bother me much, and I find the check pretty annoying now, but I could get used to it if the community thinks it’s important.<BR>
<BR>
-Ken<BR>
<BR>
<BR>
On 4/23/10 12:24 AM, "Mark Olesen" <<a href="Mark.Olesen@faurecia.com">Mark.Olesen@faurecia.com</a>> wrote:<BR>
<BR>
</SPAN></FONT><BLOCKQUOTE><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'>On Fri, 2010-04-23 at 10:34 +1000, Andrew Maclean wrote:<BR>
> I agree with you, I think that there also should be a commit check<BR>
> for tabs, otherwise layout gets really messy.<BR>
<BR>
What about checking/stripping trailing whitespace as well?<BR>
This is something else that tends to look messy and result in lots of<BR>
noisy commits.<BR>
<BR>
/mark<BR>
<BR>
<BR>
DISCLAIMER:<BR>
This electronic transmission (and any attachments thereto) is intended solely for the use of the addressee(s). It may contain confidential or legally privileged information. If you are not the intended recipient of this message, you must delete it immediately and notify the sender. Any unauthorized use or disclosure of this message is strictly prohibited. Faurecia does not guarantee the integrity of this transmission and shall therefore never be liable if the message is altered or falsified nor for any virus, interception or damage to your system.<BR>
<BR>
<BR>
<BR>
</SPAN></FONT></BLOCKQUOTE><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'><BR>
</SPAN></FONT><FONT SIZE="1"><FONT FACE="Monaco, Courier New"><SPAN STYLE='font-size:7.5pt'><BR>
**** Kenneth Moreland<BR>
*** Sandia National Laboratories<BR>
*********** <BR>
*** *** *** email: <a href="kmorel@sandia.gov">kmorel@sandia.gov</a><BR>
** *** ** phone: (505) 844-8919<BR>
*** web: <a href="http://www.cs.unm.edu/~kmorel">http://www.cs.unm.edu/~kmorel</a><BR>
</SPAN></FONT></FONT><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'><BR>
</SPAN></FONT>
</BODY>
</HTML>