[PATCH] D54757: [clang-tidy] new check: bugprone-branch-clone

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 20 11:03:17 PST 2018


lebedev.ri added inline comments.


================
Comment at: clang-tidy/bugprone/BranchCloneCheck.cpp:43
+
+  for (unsigned i = 0; i < LHS.size(); i++) {
+    if (!areStatementsIdentical(LHS[i]->stripLabelLikeStatements(),
----------------
Eugene.Zelenko wrote:
> Please use size_t.
Also
```
for (unsigned i = 0, unsigned Size = LHS.size(); i < Size; i++) {
```


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54757





More information about the cfe-commits mailing list