klimek added inline comments.

================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:444-448
@@ +443,7 @@
+  };
+  // Keep track of the different coverage situations that have been spotted
+  // during the process: Coverage[Covered][Empty] will tell if there exists any
+  // range that is covered by the first set but not by the second one,
+  // Coverage[Covered][Covered] will tell if there is a range covered by both
+  // sets, etc.
+  bool Coverage[2][2] = {{false, false}, {false, false}};
----------------
I'd just call out the 3 cases, like I suggested in my previous comment ("etc" 
sounds for me like there's more than 1 case left ;)

================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:461
@@ +460,3 @@
+
+  // If they never appeared at the same position, there is no overlapping.
+  if (!Coverage[Covered][Covered])
----------------
I'd say:
// If both sets never cover the same range, there is no overlap.

================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:472-475
@@ +471,6 @@
+  // first one, but not the other way around.
+  if (Coverage[Empty][Covered])
+    return OK_FirstInsideSecond;
+
+  return OK_SecondInsideFirst;
+}
----------------
I'd make that:
return Coverage[Empty][Covered] ? OK_FirstInsideSecond : OK_SecondInsideFirst;
(and adapt the comment)


http://reviews.llvm.org/D13516



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to