[Libreoffice-commits] .: sw/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Sun Oct 21 07:39:25 PDT 2012


 sw/source/core/layout/findfrm.cxx |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 4d837ff3d0a4121216800eb88e7f48c336a5fb03
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Sun Oct 21 16:36:51 2012 +0200

    Fix  Variable "X" is reassigned a value before the old one has been used
    
    Change-Id: I293b42d075d9c680a781a3bb993f05f4d7d62208

diff --git a/sw/source/core/layout/findfrm.cxx b/sw/source/core/layout/findfrm.cxx
index 555ac8a..fa07f8f 100644
--- a/sw/source/core/layout/findfrm.cxx
+++ b/sw/source/core/layout/findfrm.cxx
@@ -334,9 +334,9 @@ const SwLayoutFrm *SwFrm::ImplGetNextLayoutLeaf( bool bFwd ) const
     bool bGoingUp = !bFwd;          // false for forward, true for backward
     do {
 
-         bool bGoingFwdOrBwd = false, bGoingDown = false;
+         bool bGoingFwdOrBwd = false;
 
-         bGoingDown = ( !bGoingUp && ( 0 != (p = lcl_GetLower( pFrm, bFwd ) ) ) );
+         bool bGoingDown = ( !bGoingUp && ( 0 != (p = lcl_GetLower( pFrm, bFwd ) ) ) );
          if ( !bGoingDown )
          {
              // I cannot go down, because either I'm currently going up or
@@ -394,9 +394,9 @@ const SwCntntFrm* SwCntntFrm::ImplGetNextCntntFrm( bool bFwd ) const
     sal_Bool bGoingUp = sal_False;
     do {
         const SwFrm *p = 0;
-        sal_Bool bGoingFwdOrBwd = sal_False, bGoingDown = sal_False;
+        sal_Bool bGoingFwdOrBwd = sal_False;
 
-        bGoingDown = ( !bGoingUp && ( 0 != ( p = lcl_GetLower( pFrm, true ) ) ) );
+        sal_Bool bGoingDown = ( !bGoingUp && ( 0 != ( p = lcl_GetLower( pFrm, true ) ) ) );
         if ( !bGoingDown )
         {
             bGoingFwdOrBwd = ( 0 != ( p = lcl_FindLayoutFrame( pFrm, bFwd ) ) );


More information about the Libreoffice-commits mailing list