[PATCH] Prefer breaking in places other than '(' for default style

Daniel Jasper djasper at google.com
Thu Oct 24 11:22:41 PDT 2013


Yes, his change does that. All that I am asking is to not hard-code this
but instead introduce a style-dependent value. I can pick this up
tomorrow.. Should be a very straight-forward patch.


On Thu, Oct 24, 2013 at 8:15 PM, Chandler Carruth <chandlerc at google.com>wrote:

> I think what Eric is aiming at here, and something that I would prefer for
> the LLVM style at least, is a change to the relative badness assigned to
> two line breaks:
>
> variable = function( // this line break
>     argument, argument, argument);
>
> vs.
>
> variable = // this line break
>     function(argument, argument, argument);
>
>
> I think Eric would like the first one to be significantly "badder" than
> the second. Is his change actually accomplishing this? Could you point to
> the code that would accomplish this so we could evaluate different weights?
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131024/860f5523/attachment.html>


More information about the cfe-commits mailing list