[PATCH] D18264: [clang-tidy] misc-assign-operator-signature checker checks return value of all assign operators
Gábor Horváth via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 21 07:18:34 PDT 2016
xazax.hun added a comment.
In http://reviews.llvm.org/D18264#379092, @aaron.ballman wrote:
> Also, it's quite common for older code (pre C++11) to return void and make these functions private (with no definition) as an early form of deleting the operator.
This checker will not warn on private methods.
About EDSLs, I think they are not the common case. In case it is important, it might make sense to make this configurable, but I think the minority of the C++ projects have EDSLs so it make more sense to warn on those cases by default. What do you think?
More information about the cfe-commits