diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp --- a/clang/lib/Format/Format.cpp +++ b/clang/lib/Format/Format.cpp @@ -1907,9 +1907,7 @@ class BracesInserter : public TokenAnalyzer { public: BracesInserter(const Environment &Env, const FormatStyle &Style) - : TokenAnalyzer(Env, Style) { - this->Style.RemoveBracesLLVM = false; - } + : TokenAnalyzer(Env, Style) {} std::pair analyze(TokenAnnotator &Annotator, @@ -1962,9 +1960,7 @@ class BracesRemover : public TokenAnalyzer { public: BracesRemover(const Environment &Env, const FormatStyle &Style) - : TokenAnalyzer(Env, Style) { - this->Style.InsertBraces = false; - } + : TokenAnalyzer(Env, Style) {} std::pair analyze(TokenAnnotator &Annotator, @@ -3347,6 +3343,9 @@ FormatStyle Expanded = Style; expandPresetsBraceWrapping(Expanded); expandPresetsSpaceBeforeParens(Expanded); + Expanded.InsertBraces = false; + Expanded.RemoveBracesLLVM = false; + Expanded.RemoveSemicolon = false; switch (Expanded.RequiresClausePosition) { case FormatStyle::RCPS_SingleLine: case FormatStyle::RCPS_WithPreceding: @@ -3403,20 +3402,26 @@ } if (Style.InsertBraces) { + FormatStyle S = Expanded; + S.InsertBraces = true; Passes.emplace_back([&](const Environment &Env) { - return BracesInserter(Env, Expanded).process(/*SkipAnnotation=*/true); + return BracesInserter(Env, S).process(/*SkipAnnotation=*/true); }); } if (Style.RemoveBracesLLVM) { + FormatStyle S = Expanded; + S.RemoveBracesLLVM = true; Passes.emplace_back([&](const Environment &Env) { - return BracesRemover(Env, Expanded).process(/*SkipAnnotation=*/true); + return BracesRemover(Env, S).process(/*SkipAnnotation=*/true); }); } if (Style.RemoveSemicolon) { + FormatStyle S = Expanded; + S.RemoveSemicolon = true; Passes.emplace_back([&](const Environment &Env) { - return SemiRemover(Env, Expanded).process(/*SkipAnnotation=*/true); + return SemiRemover(Env, S).process(/*SkipAnnotation=*/true); }); }