<div class="gmail_quote">On Mon, Jun 11, 2012 at 6:53 PM, Peter Kümmel <span dir="ltr"><<a href="mailto:syntheticpp@gmx.net" target="_blank">syntheticpp@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 11.06.2012 18:02, David Cole wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In 'next' we are seeing dashboard test failures (and I am seeing it on my local build too, of course) that are related<br>
to recent edits in Source/cmNinjaTargetGenerator.<u></u>cxx.<br>
<br>
   193: Processing /Users/davidcole/Dashboards/My Tests/CMake/Source/<u></u>cmNinjaTargetGenerator.cxx<br>
   193: Error #0 (341) Line length exceed 80 (max=79)<br>
   193: Error #0 (382) Line length exceed 100 (max=79)<br>
<br>
Please fix them in time for the dashboard run tonight if it's at all possible.<br>
<br>
Brad and I would like to do our final merge session before 2.8.9-rc1 tomorrow.<br>
<br>
</blockquote>
<br></div></div>
OK, I fix it. Maybe more than 'allowed'.<br>
<br>
Peter<br>
--<br>
<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/<u></u>opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the CMake FAQ at: <a href="http://www.cmake.org/Wiki/CMake_FAQ" target="_blank">http://www.cmake.org/Wiki/<u></u>CMake_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers" target="_blank">http://public.kitware.com/cgi-<u></u>bin/mailman/listinfo/cmake-<u></u>developers</a><br>
</blockquote></div><div><br></div><br><div>Thank you for fixing your source files to conform to our 79 character line length limit.</div><div><br></div><div>But ... this commit is not cool:</div><div><br></div><div>  <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7b91c3dfac88b2d01a8bac71e800032fcd03ec54">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7b91c3dfac88b2d01a8bac71e800032fcd03ec54</a></div>
<div><br></div><div>And your reply here leads me to believe that you suspected that we wouldn't think it's cool... Doubly not cool.</div><div><br></div><div>If we want to increase the line length limitation, then we should discuss it and come to a reasonable consensus.</div>
<div><br></div><div>We have reasons for wanting to keep the line length short, and we realize it's not always comfortable. But it really helps out viewing diffs in side-by-side diff viewers, (as one example). I'm sure there are other examples, too, and I'll let others chime in if they'd like to, but we will be putting this value back down to 79. We're not going to increase it unless it's discussed first, and everybody agrees that it would be better with a higher value.</div>
<div><br></div><div><br></div><div>Thanks for listening,</div><div>David</div><div><br></div>