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

Daniel Jasper djasper at google.com
Thu Oct 24 11:30:15 PDT 2013


Yes, I imagin an integer-value. But finding the "right" value is hard. In
the patch's current form, it uses the same as for function definitions. And
in function definitions, it is definitely worse, as they more often look
like a "wall of text" then. I would start with half the penalty and see how
it goes.


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

> On Thu, Oct 24, 2013 at 11:22 AM, Daniel Jasper <djasper at google.com>wrote:
>
>> 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.
>>
>
> I'm also wondering whether we really want to be able to tune this more
> precisely. IE, it may have overcompensated in its current form....
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131024/4f1b0ac6/attachment.html>


More information about the cfe-commits mailing list