Fixed #13320 (False positive: functionConst reported for method marked with &&) (danmar#7017)

Author: danmar

lib/checkclass.cpp

@@ -2119,6 +2119,8 @@ void CheckClass::checkConst()
                 continue;
             if (func.functionPointerUsage)
                 continue;
+            if (func.hasRvalRefQualifier())
+                continue;
 
             // don't suggest const when returning non-const pointer/reference, but still suggest static
             auto isPointerOrReference = [this](const Token* start, const Token* end) -> bool {

test/testclass.cpp

@@ -186,6 +186,7 @@ class TestClass : public TestFixture {
         TEST_CASE(const92);
         TEST_CASE(const93);
         TEST_CASE(const94);
+        TEST_CASE(const95); // #13320 - do not warn about r-value ref method
 
         TEST_CASE(const_handleDefaultParameters);
         TEST_CASE(const_passThisToMemberOfOtherClass);
@@ -6705,6 +6706,14 @@ class TestClass : public TestFixture {
         ASSERT_EQUALS("", errout_str());
     }
 
+    void const95() { // #13320
+        checkConst("class C {\n"
+                   "    std::string x;\n"
+                   "    std::string get() && { return x; }\n"
+                   "};\n");
+        ASSERT_EQUALS("", errout_str());
+    }
+
     void const_handleDefaultParameters() {
         checkConst("struct Foo {\n"
                    "    void foo1(int i, int j = 0) {\n"