================
Comment at: clang-tidy/readability/ShrinkToFitCheck.cpp:51
@@ +50,3 @@
+               unaryOperator(
+                   has((memberExpr(member(valueDecl().bind("CtorParam")))))),
+               unaryOperator(has((declRefExpr(
----------------
nit: It looks like there's one pair of parentheses more than needed. Also below 
on lines 57-59.

================
Comment at: test/clang-tidy/readability-shrink-to-fit.cpp:46
@@ +45,3 @@
+  // CHECK-FIXES: {{^  }}v.shrink_to_fit();{{$}}
+}
+
----------------
Can you add one more test?

in g():
    ...
    T v2;
    T(v2).swap(v2);


in h():
    g<std::vector<int>>();

http://reviews.llvm.org/D7087

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



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

Reply via email to