From 52db503897de94deae3121e1ce194c88b517f300 Mon Sep 17 00:00:00 2001 From: Joren De Cuyper Date: Thu, 28 Feb 2013 00:45:05 +0100 Subject: [PATCH] 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 Tested-by: Muthu Subramanian K --- sw/source/ui/docvw/PostItMgr.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sw/source/ui/docvw/PostItMgr.cxx b/sw/source/ui/docvw/PostItMgr.cxx index 6e6f80e79789..cd94ba066259 100644 --- a/sw/source/ui/docvw/PostItMgr.cxx +++ b/sw/source/ui/docvw/PostItMgr.cxx @@ -968,7 +968,7 @@ bool SwPostItMgr::LayoutByPage(std::list &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 &aVisiblePostItList,cons } } } - if (aNextPostIt !=aVisiblePostItList.end() && (*aNextPostIt)->IsFollow()) + if ( (*aNextPostIt)->IsFollow() ) lSpaceUsed += (*i)->VirtualSize().Height() + ANCHORLINE_WIDTH; else lSpaceUsed += (*i)->VirtualSize().Height() + GetSpaceBetween();