================
@@ -0,0 +1,47 @@
+.. title:: clang-tidy - bugprone-misleading-setter-of-reference
+
+bugprone-misleading-setter-of-reference
+=======================================
+
+Finds setter-like member functions that take a pointer parameter and set a
+(non-const) reference member of the same class with the pointed value.
+
+The checker detects public member functions that have a single parameter (which
+is a pointer) and contain a single (maybe overloaded) assignment operator call.
+The assignment should set a member variable with the dereference of the
+parameter pointer. The member variable can have any visibility.
----------------
NagyDonat wrote:

```suggestion
The checker detects public member functions that take a single pointer
parameter, and contain a single expression statement that dereferences the
parameter and assigns the result to a data member with a reference type.
```
This paragraph was a bit unclear, because it tries to describe a very complex 
situation. I tried to reword it (with some help from Endre) to make it easier 
to understand. 

In my suggestion I don't explicitly mention "(maybe overloaded)" and "any 
visibility" because those are the defaults from the point of view from the 
user: in theory overloaded assignment operators are equivalent to built-in ones 
and if we don't speak about constraints on the visibility of a data member, 
then we imply that it can have any visibility.

https://github.com/llvm/llvm-project/pull/132242
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to