diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-11-25 14:51:21 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-11-26 09:47:52 +0100 |
commit | c4499bef94845176b53bbe9678d242638d86040c (patch) | |
tree | b9ba5de308ce48a22f0505ade1385e6cd419f896 /framework/source/uiconfiguration | |
parent | 2132da5066143d7828984bc1aa42d10223bc5824 (diff) |
loplugin:stringliteraldefine in framework
Change-Id: I8bdda62c232b8f20aa6bb1e5e7ad303d6ffa47ff
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125809
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'framework/source/uiconfiguration')
-rw-r--r-- | framework/source/uiconfiguration/windowstateconfiguration.cxx | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/framework/source/uiconfiguration/windowstateconfiguration.cxx b/framework/source/uiconfiguration/windowstateconfiguration.cxx index 31b08076e953..cc6f72f36a11 100644 --- a/framework/source/uiconfiguration/windowstateconfiguration.cxx +++ b/framework/source/uiconfiguration/windowstateconfiguration.cxx @@ -79,7 +79,7 @@ const sal_Int16 PROPERTY_DOCKPOS = 14; const sal_Int16 PROPERTY_DOCKSIZE = 15; // Order must be the same as WindowStateMask!! -const char* CONFIGURATION_PROPERTIES[] = +rtl::OUStringConstExpr CONFIGURATION_PROPERTIES[] { WINDOWSTATE_PROPERTY_LOCKED, WINDOWSTATE_PROPERTY_DOCKED, @@ -96,8 +96,7 @@ const char* CONFIGURATION_PROPERTIES[] = WINDOWSTATE_PROPERTY_INTERNALSTATE, WINDOWSTATE_PROPERTY_STYLE, WINDOWSTATE_PROPERTY_DOCKPOS, - WINDOWSTATE_PROPERTY_DOCKSIZE, - nullptr + WINDOWSTATE_PROPERTY_DOCKSIZE }; // Configuration access class for WindowState supplier implementation @@ -223,12 +222,8 @@ ConfigurationAccess_WindowState::ConfigurationAccess_WindowState( std::u16string m_bModified( false ) { // Initialize access array with property names. - sal_Int32 n = 0; - while ( CONFIGURATION_PROPERTIES[n] ) - { - m_aPropArray.push_back( OUString::createFromAscii( CONFIGURATION_PROPERTIES[n] )); - ++n; - } + for (const rtl::OUStringConstExpr & s : CONFIGURATION_PROPERTIES ) + m_aPropArray.push_back(s); } ConfigurationAccess_WindowState::~ConfigurationAccess_WindowState() |