alexfh added a comment.
In http://reviews.llvm.org/D18264#381846, @baloghadamsoftware wrote:
> Oh, I was searching in the C++ Core Guidlines, but at the wrong place because
> I did not find it. So I will change this option to be enabled by default. DSL
> users who do not follow this rule for the non copy and non move assign
> operators can disable it.
I'd go without the option until we find a use case for it in real code.
================
Comment at: clang-tidy/misc/AssignOperatorSignatureCheck.cpp:48
@@ +47,3 @@
+ .bind("method");
+ const auto IsSelfAssign =
+ cxxMethodDecl(IsAssign, hasParameter(0, parmVarDecl(hasType(IsSelf))))
----------------
Just do whatever `clang-format -style=llvm` does.
http://reviews.llvm.org/D18264
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
