[Libreoffice-commits] .: Branch 'libreoffice-3-5' - sdext/source

Petr Mladek pmladek at kemper.freedesktop.org
Tue Mar 20 06:25:55 PDT 2012


 sdext/source/minimizer/optimizerdialog.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 8eb5b36284c3a6c6535508a3f0651be1b5ada2dc
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Tue Mar 20 14:21:22 2012 +0100

    cppcheck report: logical disjunction always evaluates to true
    
    Signed-off-by: Petr Mladek <pmladek at suse.cz>

diff --git a/sdext/source/minimizer/optimizerdialog.cxx b/sdext/source/minimizer/optimizerdialog.cxx
index 9aae49e..b0acbd4 100644
--- a/sdext/source/minimizer/optimizerdialog.cxx
+++ b/sdext/source/minimizer/optimizerdialog.cxx
@@ -273,7 +273,7 @@ sal_Bool OptimizerDialog::execute()
 
 void OptimizerDialog::SwitchPage( sal_Int16 nNewStep )
 {
-    if ( ( nNewStep != mnCurrentStep ) && ( ( nNewStep <= MAX_STEP ) || ( nNewStep >= 0 ) ) )
+    if ( ( nNewStep != mnCurrentStep ) && ( nNewStep <= MAX_STEP ) && ( nNewStep >= 0 ) )
     {
         sal_Int16 nOldStep = mnCurrentStep;
         if ( nNewStep == 0 )


More information about the Libreoffice-commits mailing list