diff --git a/clang/tools/clang-format/ClangFormat.cpp b/clang/tools/clang-format/ClangFormat.cpp --- a/clang/tools/clang-format/ClangFormat.cpp +++ b/clang/tools/clang-format/ClangFormat.cpp @@ -358,9 +358,10 @@ if (!Status.FormatComplete) outs() << " line='" << Status.Line << "'"; outs() << ">\n"; - if (Cursor.getNumOccurrences() != 0) + if (Cursor.getNumOccurrences() != 0) { outs() << "" << FormatChanges.getShiftedCodePosition(CursorPosition) << "\n"; + } outputReplacementsXML(Replaces); outs() << "\n"; @@ -436,11 +437,11 @@ .Case("left", FormatStyle::QAS_Left) .Default(FormatStyle->QualifierAlignment); - if (FormatStyle->QualifierAlignment == FormatStyle::QAS_Left) + if (FormatStyle->QualifierAlignment == FormatStyle::QAS_Left) { FormatStyle->QualifierOrder = {"const", "volatile", "type"}; - else if (FormatStyle->QualifierAlignment == FormatStyle::QAS_Right) + } else if (FormatStyle->QualifierAlignment == FormatStyle::QAS_Right) { FormatStyle->QualifierOrder = {"type", "const", "volatile"}; - else if (QualifierAlignmentOrder.contains("type")) { + } else if (QualifierAlignmentOrder.contains("type")) { FormatStyle->QualifierAlignment = FormatStyle::QAS_Custom; SmallVector Qualifiers; QualifierAlignmentOrder.split(Qualifiers, " ", /*MaxSplit=*/-1, @@ -463,9 +464,8 @@ if (FormatStyle->isJson() && !FormatStyle->DisableFormat) { auto Err = Replaces.add(tooling::Replacement( tooling::Replacement(AssumedFileName, 0, 0, "x = "))); - if (Err) { + if (Err) llvm::errs() << "Bad Json variable insertion\n"; - } } auto ChangedCode = tooling::applyAllReplacements(Code->getBuffer(), Replaces); @@ -480,11 +480,10 @@ reformat(*FormatStyle, *ChangedCode, Ranges, AssumedFileName, &Status); Replaces = Replaces.merge(FormatChanges); if (OutputXML || DryRun) { - if (DryRun) { + if (DryRun) return emitReplacementWarnings(Replaces, AssumedFileName, Code); - } else { + else outputXML(Replaces, FormatChanges, Status, Cursor, CursorPosition); - } } else { IntrusiveRefCntPtr InMemoryFileSystem( new llvm::vfs::InMemoryFileSystem); @@ -579,9 +578,8 @@ return 0; } - if (DumpConfig) { + if (DumpConfig) return dumpConfig(); - } if (!Files.empty()) { std::ifstream ExternalFileOfFiles{std::string(Files)}; @@ -608,9 +606,10 @@ unsigned FileNo = 1; for (const auto &FileName : FileNames) { - if (Verbose) + if (Verbose) { errs() << "Formatting [" << FileNo++ << "/" << FileNames.size() << "] " << FileName << "\n"; + } Error |= clang::format::format(FileName); } return Error ? 1 : 0;