Skip to content

Commit

Permalink
Workaround clazy warnings in qtestcase.h
Browse files Browse the repository at this point in the history
This is fixed upstream in Qt 6.3+ so for now we just disable
the check in the test file where we run into it for Qt versions
lower than 6.3.
  • Loading branch information
milianw committed Dec 10, 2023
1 parent a35fdd0 commit 233c027
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/integrationtests/tst_perfparser.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,12 @@
#include "../testutils.h"
#include <hotspot-config.h>

#if QT_VERSION < QT_VERSION_CHECK(6, 3, 0)
// workaround issues with string literals in QTest that we cannot workaround locally
// this was fixed upstream, see: https://codereview.qt-project.org/c/qt/qtbase/+/354227
// clazy:excludeall=qstring-allocations
#endif

namespace {
template<typename T>
bool searchForChildSymbol(const T& root, const QString& searchString, bool exact = true)
Expand Down

0 comments on commit 233c027

Please sign in to comment.