Fix #13524 FP passedByValue for array of std::array (#7175)

Author: chrchr-github

lib/checkother.cpp

@@ -1416,7 +1416,7 @@ void CheckOther::checkPassByReference()
         if (inconclusive && !mSettings->certainty.isEnabled(Certainty::inconclusive))
             continue;
 
-        if (var->isArray() && var->getTypeName() != "std::array")
+        if (var->isArray() && (!var->isStlType() || Token::simpleMatch(var->nameToken()->next(), "[")))
             continue;
 
         const bool isConst = var->isConst();

test/testother.cpp

@@ -2440,6 +2440,9 @@ class TestOther : public TestFixture {
               "int h(const std::array<std::vector<int>, 2> a) { return a[0][0]; }\n");
         ASSERT_EQUALS("[test.cpp:4]: (performance) Function parameter 'a' should be passed by const reference.\n", errout_str());
 
+        check("void f(const std::array<int, 10> a[]) {}\n"); // #13524
+        ASSERT_EQUALS("", errout_str());
+
         /*const*/ Settings settings1 = settingsBuilder().platform(Platform::Type::Win64).build();
         check("using ui64 = unsigned __int64;\n"
               "ui64 Test(ui64 one, ui64 two) { return one + two; }\n",