If it looks good, can you commit for me? I do not have commit right.

  Thanks.


================
Comment at: unittests/Tooling/RefactoringTest.cpp:336
@@ +335,3 @@
+TEST(Range, overlaps) {
+  EXPECT_TRUE(Range(10, 10).overlaps(Range(0, 11)));
+  EXPECT_TRUE(Range(0, 11).overlaps(Range(10, 10)));
----------------
Manuel Klimek wrote:
> Now that I see that test, can we call it overlapsWith (or is that already 
> correct English? - non-native-speaker here ;)
non-native-speaker as well, but overlapsWith() sounds better, I renamed it.


http://llvm-reviews.chandlerc.com/D1184

BRANCH
  range-predicates

ARCANIST PROJECT
  clang
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to