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

Reply via email to