Axel sayeth:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">>> <snip, whitespace-only changes for consistency to <br>
>> coding standard></blockquote><div><br></div><div>+1 YAY, Axel!</div><div><br></div><div>This is a personal wish by me -- consistency.  Including for whitespace.</div><div><br></div><div>I find it astounding the "legacy-code-bases" that essentially decide that they cannot be improved -- no such patches are welcome -- because it would require work, add merge conflicts, etc.  While those are real issues, the result suggests the project *must* go "stale", and become increasingly annoying/inconsistent.  </div>
<div><br></div><div>I work in such systems.  And, that makes me sad.</div><div><br></div><div>Oswald respondeth:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
> <snip, patch welcome, should be consistent, probably <br>> should be done periodically, and for incremental parts <br>> of the code base as they can be addressed></blockquote><div><br></div><div>Seems like a sensible response.</div>
<div><br></div><div>Axel:</div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><snip>,<br>
To summarize my suggestion:<br>
 * accept partly style fixes (in either type or file space)<br>
 * accept them on an a timely manner to make it feasible at all with<br>
regards to merge conflicts<br>
 * intentionally ignore stuff in src/3rdparty<br>
<br>
I might even be interested in looking into the automatic style check<br>
issue, but can't promise anything.<br></blockquote><div><br></div><div>Sounds good.  Long-term, the automatic-style-check thing (e.g., scripts to reformat upon check-in) would be ideal, but I realize that can be tricky.</div>
<div><br></div><div>It makes sense that we would eventually get the whitespace-formatting into the tool-chain workflow, though.  I like where the CI-system and review-process is going, and it seems like this is merely an extension of that.</div>
<div><br></div><div>--charley</div><div><br></div></div></div>