[clang-tidy] "modernize-pass-by-value" - should ignore move constructors from protected/private section

[denzor200]

class A {
    std::string m_val;
public:
    A() = default;
    A(const A& other) = default;
    A& operator= (const A& other) = default;
protected:
    A(A&& other) = default;
    A& operator= (A&& other) = default;
};

This protected move constructor must be ignored, redundant warning with invalid fixit must not be provided. Also we should pay attention for friend classes.

Full snippet

[llvmbot]

@llvm/issue-subscribers-clang-tidy

Author: Denis Mikhailov (denzor200)

``` class A { std::string m_val; public: A() = default; A(const A& other) = default; A& operator= (const A& other) = default; protected: A(A&& other) = default; A& operator= (A&& other) = default; }; ```

This protected move constructor must be ignored, redundant warning with invalid fixit must not be provided. Also we should pay attention for friend classes.

Full snippet