diff options
author | Marc Mutz <marc.mutz@qt.io> | 2022-10-05 08:17:22 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2022-10-06 18:20:17 +0200 |
commit | c338447261878111df7198fbd96051926464e865 (patch) | |
tree | d21b5af1efae225f75d5c05dfe370daefb1ea1d1 /src/assistant/help/qoptionswidget.cpp | |
parent | 6495329e6de803025e6e4e8291b648f94893551c (diff) | |
download | qttools-c338447261878111df7198fbd96051926464e865.tar.gz |
Port from container::count() and length() to size()
This is a semantic patch using ClangTidyTransformator as in
qtbase/df9d882d41b741fef7c5beeddb0abe9d904443d8:
auto QtContainerClass = anyOf(
expr(hasType(cxxRecordDecl(isSameOrDerivedFrom(hasAnyName(classes))))).bind(o),
expr(hasType(namedDecl(hasAnyName(<classes>)))).bind(o));
makeRule(cxxMemberCallExpr(on(QtContainerClass),
callee(cxxMethodDecl(hasAnyName({"count", "length"),
parameterCountIs(0))))),
changeTo(cat(access(o, cat("size"), "()"))),
cat("use 'size()' instead of 'count()/length()'"))
a.k.a qt-port-to-std-compatible-api with config Scope: 'Container',
with the extended set of container classes recognized.
Change-Id: I95f6410e57a6a92b1cf91bbedfbe3d517cab6b44
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Kai Koehne <kai.koehne@qt.io>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Diffstat (limited to 'src/assistant/help/qoptionswidget.cpp')
-rw-r--r-- | src/assistant/help/qoptionswidget.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/assistant/help/qoptionswidget.cpp b/src/assistant/help/qoptionswidget.cpp index 71e8cd435..01abbfdef 100644 --- a/src/assistant/help/qoptionswidget.cpp +++ b/src/assistant/help/qoptionswidget.cpp @@ -103,14 +103,14 @@ void QOptionsWidget::setOptions(const QStringList &validOptions, for (const QString &option : m_invalidOptions) appendItem(option, false, true); - if ((validSelectedOptions.count() + m_invalidOptions.count()) - && validUnselectedOptions.count()) { + if ((validSelectedOptions.size() + m_invalidOptions.size()) + && validUnselectedOptions.size()) { appendSeparator(); } for (const QString &option : validUnselectedOptions) { appendItem(option, true, false); - if (option.isEmpty() && validUnselectedOptions.count() > 1) // special No Option item + if (option.isEmpty() && validUnselectedOptions.size() > 1) // special No Option item appendSeparator(); } } |