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

[Joren De Cuyper]

 sw/source/ui/docvw/PostItMgr.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 52db503897de94deae3121e1ce194c88b517f300
Author: Joren De Cuyper <joren.libreoffice at telenet.be>
Date:   Thu Feb 28 00:45:05 2013 +0100

    Delete double condition check
    
    We don't need to check if aNextPostIt != aVisiblePostItList.end()
    again because we only can enter the enclosing if-statement when
    aNextPostIt != aVisiblePostItList.end()
    
    Change-Id: I853ed465e957bd26dae8312d2ce717ea1ca0b4e8
    Reviewed-on: https://gerrit.libreoffice.org/2453
    Reviewed-by: Muthu Subramanian K <muthusuba at gmail.com>
    Tested-by: Muthu Subramanian K <muthusuba at gmail.com>

diff --git a/sw/source/ui/docvw/PostItMgr.cxx b/sw/source/ui/docvw/PostItMgr.cxx
index 6e6f80e..cd94ba0 100644
--- a/sw/source/ui/docvw/PostItMgr.cxx
+++ b/sw/source/ui/docvw/PostItMgr.cxx
@@ -968,7 +968,7 @@ bool SwPostItMgr::LayoutByPage(std::list<SwSidebarWin*> &aVisiblePostItList,cons
                 SwSidebarWin_iterator aNextPostIt = i;
                 ++aNextPostIt;
 
-                if (aNextPostIt !=aVisiblePostItList.end())
+                if (aNextPostIt != aVisiblePostItList.end())
                 {
                     lTranslatePos = ( (*i)->VirtualPos().Y() + (*i)->VirtualSize().Height()) - (*aNextPostIt)->VirtualPos().Y();
                     if (lTranslatePos > 0) // note windows overlaps the next one
@@ -1020,7 +1020,7 @@ bool SwPostItMgr::LayoutByPage(std::list<SwSidebarWin*> &aVisiblePostItList,cons
                             }
                         }
                     }
-                    if (aNextPostIt !=aVisiblePostItList.end() && (*aNextPostIt)->IsFollow())
+                    if ( (*aNextPostIt)->IsFollow() )
                         lSpaceUsed += (*i)->VirtualSize().Height() + ANCHORLINE_WIDTH;
                     else
                         lSpaceUsed += (*i)->VirtualSize().Height() + GetSpaceBetween();