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

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Oct 22 08:04:17 UTC 2018


 sw/source/uibase/uiview/viewcoll.cxx |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 8acf598338f2255f13eb1182f9ecffc730584530
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Sun Oct 21 22:58:42 2018 +0200
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Mon Oct 22 10:03:55 2018 +0200

    tdf#120703 (PVS): Recurring check.
    
    V571 The 'pArgs' condition was already verified in line 47.
    
    Change-Id: Id63eb556610ec457864e44b7d3f49e46f133e5a9
    Reviewed-on: https://gerrit.libreoffice.org/62162
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/uibase/uiview/viewcoll.cxx b/sw/source/uibase/uiview/viewcoll.cxx
index fa322098a6fa..4008af513055 100644
--- a/sw/source/uibase/uiview/viewcoll.cxx
+++ b/sw/source/uibase/uiview/viewcoll.cxx
@@ -46,8 +46,7 @@ void SwView::ExecColl(SfxRequest const &rReq)
         {
             if( pArgs )
             {
-                if (pArgs &&
-                    SfxItemState::SET == pArgs->GetItemState( nWhich , true, &pItem ))
+                if (SfxItemState::SET == pArgs->GetItemState( nWhich , true, &pItem ))
                 {
                     if( static_cast<const SfxStringItem*>(pItem)->GetValue() !=
                                             GetWrtShell().GetCurPageStyle() )


More information about the Libreoffice-commits mailing list