sfx2/source/control/dispatch.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 32d81ed85c581134dbbfe04707863a59f0101a1a
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Mon Nov 25 12:03:33 2024 +0100
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Mon Nov 25 14:12:34 2024 +0100

    PVS: V560 A part of conditional expression is always true
    
    Change-Id: I1c84f84f5d656e2a11f5b3a594f7f19641ff51ae
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177255
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/sfx2/source/control/dispatch.cxx b/sfx2/source/control/dispatch.cxx
index e704d324a355..29fa907ae32e 100644
--- a/sfx2/source/control/dispatch.cxx
+++ b/sfx2/source/control/dispatch.cxx
@@ -1114,7 +1114,7 @@ void SfxDispatcher::Update_Impl( bool bForce )
 
     bool bIsActive = false;
     SfxDispatcher *pActDispat = pWorkWin->GetBindings().GetDispatcher_Impl();
-    if ( !bIsActive && this == pActDispat )
+    if ( this == pActDispat )
         bIsActive = true;
 
     Update_Impl_( bUIActive, !bIsIPActive, bIsIPActive, pWorkWin );
@@ -1140,7 +1140,7 @@ void SfxDispatcher::Update_Impl_( bool bUIActive, bool 
bIsMDIApp, bool bIsIPOwne
     SfxWorkWindow *pWorkWin = xImp->pFrame->GetFrame().GetWorkWindow_Impl();
     bool bIsActive = false;
     SfxDispatcher *pActDispat = pWorkWin->GetBindings().GetDispatcher_Impl();
-    if ( pActDispat && !bIsActive )
+    if ( pActDispat )
     {
         if ( this == pActDispat )
             bIsActive = true;
@@ -1276,7 +1276,7 @@ void SfxDispatcher::Update_Impl_( bool bUIActive, bool 
bIsMDIApp, bool bIsIPOwne
     bool bIsTaskActive = false;
 
     SfxDispatcher *pActDispatcher = 
pTaskWin->GetBindings().GetDispatcher_Impl();
-    if ( pActDispatcher && !bIsTaskActive )
+    if ( pActDispatcher )
     {
         if ( this == pActDispatcher )
             bIsTaskActive = true;

Reply via email to