Hi,

I have played a bit with clang-format and it turns out that
we need

AlwaysBreakBeforeMultilineStrings: false
ColumnLimit: 0

To fix most cluttering the issues.

But now we have a new one:
Double indentation fails in certain palaces.




Am 10.06.2015 um 17:01 schrieb Max Linke:
>
>
> On 06/10/2015 04:16 PM, Daniel Schürmann wrote:
>> Which version of clang-format do you use?
>> Can this be a bug like this:
>> https://github.com/travisjeffery/ClangFormat-Xcode/issues/81
>>
>> Can you give an example where ColumnLimit: 0 fails?
>
> I use clang-format-3.6
>
>
>>     QLayout* pLayout = NULL;
>> -    if (context.hasNode(node, "Layout")) {
>> -        QString layout = context.selectString(node, "Layout");
>> -        if (layout == "vertical") {
>> +    if (context.hasNode(node,
>> +                        "Layout")) {
>> +        QString layout = context.selectString(node,
>> +                                              "Layout");
>> +        if (layout ==
>> +            "vertical") {
>>              pLayout = new QVBoxLayout();
>> -        } else if (layout == "horizontal") {
>> +        } else if (layout ==
>> +                   "horizontal") {
>>              pLayout = new QHBoxLayout();
>> -        } else if (layout == "stacked") {
>> +        } else if (layout ==
>> +                   "stacked") {
>
>
> This doesn't happen if I set the column limit to 80


------------------------------------------------------------------------------
_______________________________________________
Get Mixxx, the #1 Free MP3 DJ Mixing software Today
http://mixxx.org


Mixxx-devel mailing list
Mixxx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mixxx-devel

Reply via email to