On Fri, Jan 25, 2013 at 9:12 AM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On Jan 25, 2013, at 7:43 , Daniel Jasper <<a href="mailto:djasper@google.com">djasper@google.com</a>> wrote:<br>
<br>
> Author: djasper<br>
> Date: Fri Jan 25 09:43:32 2013<br>
> New Revision: 173457<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=173457&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=173457&view=rev</a><br>
> Log:<br>
> Fix some alignment and line break decisions.<br>
><br>
> This combines two small changes:<br>
> 1) Put a penalty on breaking after "<"<br>
> 2) Only produce a hanging indent when parameters are separated by<br>
> commas.<br>
><br>
> Before:<br>
> aaaaaaaaaaaaaaaaaaaaaaaa<<br>
>    aaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa>(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa);<br>
> aaaaaa(new Aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(<br>
>               aaaaaaaaaaaaaaaaaaaaaaaaaaaaa));<br>
><br>
> After:<br>
> aaaaaaaaaaaaaaaaaaaaaaaa<aaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa>(<br>
>    aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa);<br>
> aaaaaa(new Aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(<br>
>    aaaaaaaaaaaaaaaaaaaaaaaaaaaaa));<br>
><br>
> This changes one ObjC test, but AFAICT this is not according to any<br>
> style guide (neither before nor after). We probably should be aligning<br>
> on the ":" there according to:<br>
> <a href="http://google-styleguide.googlecode.com/svn/trunk/objcguide.xml?showone=Method_Invocations#Method_Invocations" target="_blank">http://google-styleguide.googlecode.com/svn/trunk/objcguide.xml?showone=Method_Invocations#Method_Invocations</a><br>

<br>
</div></div>Apple style is the same, except possibly for the long cases. (I think the "before" ObjC case is better than the "after" case here, but neither is what we want in the long run anyway.)<br></blockquote>
<div><br></div><div>I also think the "before" case is better, but since it too is unusably bad it doesn't matter much :-) PR14939 tracks formatting selector expressions correctly.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br>