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

Mike Kaganski (via logerrit) logerrit at kemper.freedesktop.org
Fri Jan 3 20:53:01 UTC 2020


 sfx2/source/appl/appuno.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 894efac8fd6f49d272db340d6f132acd262f93bc
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Fri Jan 3 17:56:58 2020 +0100
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Fri Jan 3 21:52:25 2020 +0100

    Make variable const, and drop a bogus assert after check
    
    Change-Id: I5160618a6af171e8ce048959bc4152f09d1754b6
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86195
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sfx2/source/appl/appuno.cxx b/sfx2/source/appl/appuno.cxx
index 466f31e87250..2989ee0ad2ac 100644
--- a/sfx2/source/appl/appuno.cxx
+++ b/sfx2/source/appl/appuno.cxx
@@ -176,7 +176,7 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert
     if ( nSlotId == SID_OPENURL )
         nSlotId = SID_OPENDOC;
 
-    sal_Int32 nCount = rArgs.getLength();
+    const sal_Int32 nCount = rArgs.getLength();
     if ( !nCount )
         return;
 
@@ -268,8 +268,6 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert
         return;
     }
 
-    OSL_ASSERT(nCount > 0);
-
 #ifdef DBG_UTIL
     // detect parameters that don't match to any formal argument or one of its members
     sal_Int32 nFoundArgs = 0;


More information about the Libreoffice-commits mailing list