<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small;color:#000000">I'm gonna assume you guys now the code base better then I do :p<br>By just indenting the braces, its gonna most likely be much cleaner code,<br>though then all code in UnwrappedLineParser will be the exact same for Allman and Whitesmiths.<br>Any ideas how to make sure this will stay in sync in the future?</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small;color:#000000"><br>Anyhow, for now I'm gonna play a bit more with the code trying to do it the 'easy way' and get familiar with the code some more.</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small;color:#000000">It won't hurt since I've noticed that I also need another feature to be compliant with our current style sheet.<br>If it wouldn't work out I'll send a cry and otherwise a patch.</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small;color:#000000"><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><font face="verdana, sans-serif">JVApen<font size="1"><br><br></font></font></div></div></div></div>
<br><div class="gmail_quote">On 5 January 2015 at 14:11, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I agree that that should lead to an way simpler implementation. It has the downside of spreading the brace handling stuff more outside of the UnwrappedLineParser, but that is probably ok in this case.<br>
<br>
So, we should remove almost all the changes to the UnwrappedLineParser and instead modify UnwrappedLineFormatter::formatFirstToken(). JVApen, can you make those changes? Alternatively, I can also pick up and finish the patch.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D6833" target="_blank">http://reviews.llvm.org/D6833</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>