<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 6, 2014 at 10:31 AM, Lukas Larsson <span dir="ltr"><<a href="mailto:lukas@erlang.org" target="_blank">lukas@erlang.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="im">On Thu, Feb 6, 2014 at 9:38 AM, Loïc Hoguin <span dir="ltr"><<a href="mailto:essen@ninenines.eu" target="_blank">essen@ninenines.eu</a>></span> wrote:<br>


</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Plus this doesn't help your code get merged at all. If you submit code with broken whitespace, then tools will also display this whitespace change. You can of course hide whitespace changes, but the problem is that whitespace does matter in some places, so you're basically hiding potentially important information by doing that.<br>



<br></blockquote></div><div>This is in my opinion the reason why it would be a very bad thing to do a huge commit that changes the current whitespace format of a project to another. It is already very hard to figure out what change could have caused a bug without having to deal with whitespace changes in the middle of a diff. </div>

</div></div></div></blockquote><div><br></div><div>well then commit a whitespace change diff, then the change you really wanted to make. It of course requires to have a standard defined at first :)</div><div><br></div><div>

- benoit </div></div></div></div>