Skip to content

Commit 2fc7ea9

Browse files
committed
fix formatting
1 parent 75da86c commit 2fc7ea9

File tree

4 files changed

+11
-10
lines changed

4 files changed

+11
-10
lines changed

clang/lib/Format/ContinuationIndenter.cpp

+5-6
Original file line numberDiff line numberDiff line change
@@ -834,16 +834,15 @@ void ContinuationIndenter::addTokenOnCurrentLine(LineState &State, bool DryRun,
834834
/* For backward compatibility, use AlignAfterOpenBracket
835835
* in case AlignAfterControlStatement is not initialized */
836836
return Style.AlignAfterControlStatement == FormatStyle::BACSS_MultiLine ||
837-
(Style.AlignAfterControlStatement == FormatStyle::BACSS_Default &&
838-
Style.AlignAfterOpenBracket == FormatStyle::BAS_AlwaysBreak);
837+
(Style.AlignAfterControlStatement == FormatStyle::BACSS_Default &&
838+
Style.AlignAfterOpenBracket == FormatStyle::BAS_AlwaysBreak);
839839
}
840-
if (IsOtherConditional(*Tok.Previous)) {
840+
if (IsOtherConditional(*Tok.Previous))
841841
return Style.AlignAfterControlStatement == FormatStyle::BACSS_MultiLine;
842-
}
843842
if (Style.AlignAfterOpenBracket == FormatStyle::BAS_AlwaysBreak ||
844-
Style.AlignAfterOpenBracket == FormatStyle::BAS_BlockIndent) {
843+
Style.AlignAfterOpenBracket == FormatStyle::BAS_BlockIndent) {
845844
return !Tok.Previous->is(TT_CastRParen) &&
846-
!(Style.isJavaScript() && Tok.is(Keywords.kw_await));
845+
!(Style.isJavaScript() && Tok.is(Keywords.kw_await));
847846
}
848847
return false;
849848
};

clang/lib/Format/Format.cpp

+4-2
Original file line numberDiff line numberDiff line change
@@ -202,8 +202,10 @@ template <> struct MappingTraits<FormatStyle::BraceWrappingFlags> {
202202
}
203203
};
204204

205-
template <> struct ScalarEnumerationTraits<FormatStyle::BreakAfterControlStatementStyle> {
206-
static void enumeration(IO &IO, FormatStyle::BreakAfterControlStatementStyle &Value) {
205+
template <>
206+
struct ScalarEnumerationTraits<FormatStyle::BreakAfterControlStatementStyle> {
207+
static void enumeration(IO &IO,
208+
FormatStyle::BreakAfterControlStatementStyle &Value) {
207209
IO.enumCase(Value, "Default", FormatStyle::BACSS_Default);
208210
IO.enumCase(Value, "MultiLine", FormatStyle::BACSS_MultiLine);
209211
IO.enumCase(Value, "No", FormatStyle::BACSS_No);

clang/lib/Format/TokenAnnotator.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -6221,7 +6221,8 @@ bool TokenAnnotator::canBreakBefore(const AnnotatedLine &Line,
62216221
const FormatToken *Previous = Right.MatchingParen->Previous;
62226222
if (!Previous)
62236223
return true;
6224-
if (Previous->isIf() || Previous->isOneOf(tok::kw_for, tok::kw_while, tok::kw_switch)) {
6224+
if (Previous->isIf() ||
6225+
Previous->isOneOf(tok::kw_for, tok::kw_while, tok::kw_switch)) {
62256226
return Style.AlignAfterControlStatement == FormatStyle::BACSS_MultiLine;
62266227
}
62276228
return true;

clang/unittests/Format/FormatTest.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -9797,7 +9797,6 @@ TEST_F(FormatTest, AlignAfterConditionalStatements) {
97979797
"}",
97989798
Style);
97999799

9800-
98019800
Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak;
98029801
Style.AlignAfterControlStatement = FormatStyle::BACSS_MultiLine;
98039802

0 commit comments

Comments
 (0)