[PATCH] D24894: [clang-tidy] Prefer transparent functors to non-transparent one.

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 15 02:45:13 PST 2016


xazax.hun marked 2 inline comments as done.
xazax.hun added inline comments.


================
Comment at: clang-tidy/modernize/UseTransparentFunctorsCheck.cpp:89
+    diag(FuncInst->getLocStart(), Message)
+        << (FuncClass->getName() + "<>").str();
+    return;
----------------
aaron.ballman wrote:
> Is the `.str()` required? (Same question applies below.)
It does not compile without it. It looks like the << is not overloaded for Twine.


https://reviews.llvm.org/D24894





More information about the cfe-commits mailing list