From e06d213365d407c65f35acada77adce538009166 Mon Sep 17 00:00:00 2001 From: Uwe Rathmann Date: Thu, 24 Mar 2022 08:20:05 +0100 Subject: [PATCH] using const getter --- src/graphic/QskGraphicProviderMap.cpp | 12 ++++++------ src/layouts/QskGridBox.cpp | 2 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/graphic/QskGraphicProviderMap.cpp b/src/graphic/QskGraphicProviderMap.cpp index 25b76ba9..a7a49043 100644 --- a/src/graphic/QskGraphicProviderMap.cpp +++ b/src/graphic/QskGraphicProviderMap.cpp @@ -47,8 +47,8 @@ void QskGraphicProviderMap::insert( void QskGraphicProviderMap::remove( const QString& providerId ) { - const auto it = m_data->hashTab.find( qskKey( providerId ) ); - if ( it == m_data->hashTab.end() ) + const auto it = m_data->hashTab.constFind( qskKey( providerId ) ); + if ( it == m_data->hashTab.constEnd() ) delete it.value(); m_data->hashTab.erase( it ); @@ -58,8 +58,8 @@ QskGraphicProvider* QskGraphicProviderMap::take( const QString& providerId ) { QskGraphicProvider* provider = nullptr; - const auto it = m_data->hashTab.find( qskKey( providerId ) ); - if ( it == m_data->hashTab.end() ) + const auto it = m_data->hashTab.constFind( qskKey( providerId ) ); + if ( it == m_data->hashTab.constEnd() ) provider = it.value(); m_data->hashTab.erase( it ); @@ -69,8 +69,8 @@ QskGraphicProvider* QskGraphicProviderMap::take( const QString& providerId ) QskGraphicProvider* QskGraphicProviderMap::provider( const QString& providerId ) const { - const auto it = m_data->hashTab.find( qskKey( providerId ) ); - if ( it == m_data->hashTab.end() ) + const auto it = m_data->hashTab.constFind( qskKey( providerId ) ); + if ( it == m_data->hashTab.constEnd() ) return nullptr; if ( it.value().isNull() ) diff --git a/src/layouts/QskGridBox.cpp b/src/layouts/QskGridBox.cpp index f5d49516..40548887 100644 --- a/src/layouts/QskGridBox.cpp +++ b/src/layouts/QskGridBox.cpp @@ -79,7 +79,7 @@ static void qskUpdateFocusChain( } else { - const auto itemLast = box->childItems().last(); + const auto itemLast = box->childItems().constLast(); if ( itemLast != item ) item->stackAfter( itemLast ); }