[Libreoffice-commits] core.git: cui/source

[Armin Le Grand (Allotropia) (via logerrit)]

 cui/source/options/optpath.cxx |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit bcc28948d0fd7b4f005a13db923c3e853d74817c
Author:     Armin Le Grand (Allotropia) <armin.le.grand at me.com>
AuthorDate: Wed Jun 9 13:33:26 2021 +0200
Commit:     Armin Le Grand <Armin.Le.Grand at me.com>
CommitDate: Wed Jun 9 17:12:59 2021 +0200

    tdf#130428 remove unnecessary usage of SfxItemState::UNKNOWN
    
    See task, slowly trying to reduce usages of that flag
    
    Change-Id: I50dc8e21e2f5e82e21bf335d63d07cee1ee18d01
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116913
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>
    Reviewed-by: Armin Le Grand <Armin.Le.Grand at me.com>
    Tested-by: Jenkins

diff --git a/cui/source/options/optpath.cxx b/cui/source/options/optpath.cxx
index e1f638faa673..e0f940219016 100644
--- a/cui/source/options/optpath.cxx
+++ b/cui/source/options/optpath.cxx
@@ -77,14 +77,14 @@ namespace {
 struct PathUserData_Impl
 {
     SvtPathOptions::Paths  nRealId;
-    SfxItemState    eState;
+    bool            bItemStateSet;
     OUString        sUserPath;
     OUString        sWritablePath;
     bool            bReadOnly;
 
     explicit PathUserData_Impl(SvtPathOptions::Paths nId)
         : nRealId(nId)
-        , eState(SfxItemState::UNKNOWN)
+        , bItemStateSet(false)
         , bReadOnly(false)
     {
     }
@@ -227,7 +227,7 @@ bool SvxPathTabPage::FillItemSet( SfxItemSet* )
     {
         PathUserData_Impl* pPathImpl = reinterpret_cast<PathUserData_Impl*>(m_xPathBox->get_id(i).toInt64());
         SvtPathOptions::Paths nRealId = pPathImpl->nRealId;
-        if (pPathImpl->eState == SfxItemState::SET)
+        if (pPathImpl->bItemStateSet )
             SetPathList( nRealId, pPathImpl->sUserPath, pPathImpl->sWritablePath );
     }
     return true;
@@ -403,7 +403,7 @@ IMPL_LINK_NOARG(SvxPathTabPage, StandardHdl_Impl, weld::Button&, void)
                 }
             }
             m_xPathBox->set_text(rEntry, Convert_Impl(sTemp), 1);
-            pPathImpl->eState = SfxItemState::SET;
+            pPathImpl->bItemStateSet = true;
             pPathImpl->sUserPath = sUserPath.makeStringAndClear();
             pPathImpl->sWritablePath = sWritablePath;
         }
@@ -448,7 +448,7 @@ void SvxPathTabPage::ChangeCurrentEntry( const OUString& _rFolder )
         return;
 
     m_xPathBox->set_text(nEntry, Convert_Impl(sNewPathStr), 1);
-    pPathImpl->eState = SfxItemState::SET;
+    pPathImpl->bItemStateSet = true;
     pPathImpl->sWritablePath = sNewPathStr;
     if ( SvtPathOptions::Paths::Work == pPathImpl->nRealId )
     {
@@ -530,7 +530,7 @@ IMPL_LINK_NOARG(SvxPathTabPage, PathHdl_Impl, weld::Button&, void)
 
             m_xPathBox->set_text(nEntry, Convert_Impl(sFullPath), 1);
             // save modified flag
-            pPathImpl->eState = SfxItemState::SET;
+            pPathImpl->bItemStateSet = true;
             pPathImpl->sUserPath = sUser;
             pPathImpl->sWritablePath = sWritable;
         }