[PATCH] D50389: [clang-tidy] Abseil: integral division of Duration check

Deanna Garcia via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 13 05:59:23 PDT 2018


deannagarcia added inline comments.


================
Comment at: test/clang-tidy/abseil-duration-division.cpp:58
+  // CHECK-MESSAGES: [[@LINE-4]]:45: warning: operator/ on absl::Duration objects
+  // CHECK-FIXES: double DoubleDivision(T t1, T t2) {return
+  // absl::FDivDuration(t1, t2);}
----------------
hokein wrote:
> I think we should ignore templates. The template function could be used by other types, fixing the template is not correct. 
I removed this template function, but left the template function TakesGeneric below to show that the automatic type does not require/give a warning. Is that alright?


https://reviews.llvm.org/D50389





More information about the cfe-commits mailing list