sw/source/ui/docvw/PostItMgr.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
New commits: commit 52db503897de94deae3121e1ce194c88b517f300 Author: Joren De Cuyper <joren.libreoff...@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 <muthus...@gmail.com> Tested-by: Muthu Subramanian K <muthus...@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(); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits