[clang][modules] Don't prevent translation of FW_Private includes when explicitly...
[llvm-project.git] / clang-tools-extra / docs / clang-tidy / checks / cppcoreguidelines / avoid-const-or-ref-data-members.rst
blob5783280478dc1a47d93499aac0877e52dd61ecd8
1 .. title:: clang-tidy - cppcoreguidelines-avoid-const-or-ref-data-members
3 cppcoreguidelines-avoid-const-or-ref-data-members
4 =================================================
6 This check warns when structs or classes that are copyable or movable, and have
7 const-qualified or reference (lvalue or rvalue) data members. Having such
8 members is rarely useful, and makes the class only copy-constructible but not
9 copy-assignable.
11 Examples:
13 .. code-block:: c++
15   // Bad, const-qualified member
16   struct Const {
17     const int x;
18   }
20   // Good:
21   class Foo {
22    public:
23     int get() const { return x; }
24    private:
25     int x;
26   };
28   // Bad, lvalue reference member
29   struct Ref {
30     int& x;
31   };
33   // Good:
34   struct Foo {
35     int* x;
36     std::unique_ptr<int> x;
37     std::shared_ptr<int> x;
38     gsl::not_null<int> x;
39   };
41   // Bad, rvalue reference member
42   struct RefRef {
43     int&& x;
44   };
46 This check implements `C.12
47 <https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Rc-constref>`_
48 from the C++ Core Guidelines.
50 Further reading:
51 `Data members: Never const <https://quuxplusone.github.io/blog/2022/01/23/dont-const-all-the-things/#data-members-never-const>`_.