malcolm.parsons added inline comments.
================ Comment at: clang-tidy/readability/OneNamePerDeclarationCheck.cpp:86 + + Diag << FixItHint::CreateReplacement(CommaRange, ";") + << FixItHint::CreateReplacement(AfterCommaToVarNameRange, ---------------- A `SourceLocation` can be implicitly converted to a `SourceRange`. ================ Comment at: clang-tidy/readability/OneNamePerDeclarationCheck.cpp:87 + Diag << FixItHint::CreateReplacement(CommaRange, ";") + << FixItHint::CreateReplacement(AfterCommaToVarNameRange, + "\n" + CurrentIndent + ---------------- Is an insert simpler? ``` FixItHint::CreateInsertion(CommaLocation.getLocWithOffset(1), "\n" + CurrentIndent + UserWrittenType + " "); ``` https://reviews.llvm.org/D27621 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits