From e3d5dc3fc7b697b232f09c5bde33a34d60cfd12f Mon Sep 17 00:00:00 2001 From: Volker Hilsheimer Date: Wed, 3 Sep 2025 08:02:44 +0300 Subject: [PATCH] QRM: make implementation of resetRoleNames out of line No reason to be inline, so avoid that we pay the compile-time and code generation cost at each call site. Address header review. Pick-to: 6.10 Change-Id: I2f21b26519f8b8b1ce390533031dee95db205a20 Reviewed-by: Marc Mutz --- src/corelib/itemmodels/qrangemodel.cpp | 5 +++++ src/corelib/itemmodels/qrangemodel.h | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/corelib/itemmodels/qrangemodel.cpp b/src/corelib/itemmodels/qrangemodel.cpp index 8ee4e9f1a18..cdedc85bf39 100644 --- a/src/corelib/itemmodels/qrangemodel.cpp +++ b/src/corelib/itemmodels/qrangemodel.cpp @@ -1084,6 +1084,11 @@ void QRangeModel::setRoleNames(const QHash &names) Q_EMIT roleNamesChanged(); } +void QRangeModel::resetRoleNames() +{ + setRoleNames({}); +} + /*! \reimp */ diff --git a/src/corelib/itemmodels/qrangemodel.h b/src/corelib/itemmodels/qrangemodel.h index 4ceeeeb0fa1..376aff9166d 100644 --- a/src/corelib/itemmodels/qrangemodel.h +++ b/src/corelib/itemmodels/qrangemodel.h @@ -76,7 +76,7 @@ public: QHash roleNames() const override; void setRoleNames(const QHash &names); - void resetRoleNames() { setRoleNames({}); } + void resetRoleNames(); bool canFetchMore(const QModelIndex &parent) const override; void fetchMore(const QModelIndex &parent) override;