[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5862 Prefer a

From:

Auto mailings of changes to Lily Issues via Testlilyissues-auto

Subject:

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5862 Prefer astyle 3.1 and update clang-format options

Date:

Sat, 28 Mar 2020 17:46:35 -0000

commit f2b0913e93ce24fee9e15e071c42f5d1aee54240
Author: Dan Eble <address@hidden>
Date:   Mon Mar 23 20:35:48 2020 -0400

    Issue 5862/2: adjust clang-format options

    Add a few clang-format options to reduce (slightly) the extent of
    changes that a subsequent fixcc.py makes.

    ...

commit 0731c054e413f78e29eee89e44f4fb40ae5bb268
Author: Dan Eble <address@hidden>
Date:   Sun Mar 22 18:03:57 2020 -0400

    Issue 5862/1: Change preferred version of astyle from 2.04 to 3.1

[issues:#5862] Prefer astyle 3.1 and update clang-format options

Status: Started
Created: Tue Mar 24, 2020 01:05 AM UTC by Dan Eble
Last Updated: Sat Mar 28, 2020 09:17 AM UTC
Owner: Dan Eble

Change the preferred version of astyle from 2.04 to 3.1.

In a separate commit, add a few clang-format options to reduce
(slightly) the extent of changes that a subsequent fixcc.py makes.

Test procedure:
1. clang-format everything (version 9.0.0)
2. git commit
3. fixcc.py everything (astyle version 3.1)
4. git diff | wc -l

BASELINE                                          LINES
master@786c3669ff08804465a89013d349fc745fd783c9   19702

BENEFICIAL OPTIONS (TESTED INDIVIDUALLY)          LINES
ConstructorInitializerIndentWidth: 2              19062
ContinuationIndentWidth: 2                        15018
SpaceAfterCStyleCast: true                        19540
[Tested together]                                 14171

OTHER OPTIONS (TESTED INDIVIDUALLY)               LINES
AlignEscapedNewlines: Left                        19702
AlignOperands: false                              20760
AllowShortFunctionsOnASingleLine: InlineOnly      19708
AlwaysBreakTemplateDeclarations: MultiLine        19702
BinPackArguments: false                           21588
BinPackParameters: false                          20557
BreakBeforeTernaryOperators: true                 19702
BreakConstructorInitializers: BeforeColon         19702
BreakInheritanceList: BeforeColon                 19702
ColumnLimit: 80                                   19702
SortIncludes: false                               19702
SpaceAfterTemplateKeyword: false                  19702

https://codereview.appspot.com/551640043

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.