[PATCH] D29393: [clang-tidy] Don't warn about call to unresolved operator*

[Malcolm Parsons via Phabricator via cfe-commits]

malcolm.parsons added inline comments.


================
Comment at: clang-tidy/misc/UnconventionalAssignOperatorCheck.cpp:63
+            cxxOperatorCallExpr(argumentCountIs(1),
+                                callee(unresolvedLookupExpr()),
+                                hasArgument(0, cxxThisExpr())))))));
----------------
idlecode wrote:
> Seems that it will catch all unary operators with ##this## as first argument.
> e.g. in case `operator-` is defined somewhere, check will not warn about returning `-this`.
> Do you think adding `hasOverloadedOperatorName("*")` for such abstract case is worth it?
I'm just trying to suppress them warning from the template. The operator can be checked properly in an instantiation.


https://reviews.llvm.org/D29393