diff options
| author | Joren De Cuyper <joren.libreoffice@telenet.be> | 2013-02-28 00:45:05 +0100 |
|---|---|---|
| committer | Muthu Subramanian K <muthusuba@gmail.com> | 2013-03-04 12:34:39 +0000 |
| commit | 52db503897de94deae3121e1ce194c88b517f300 (patch) | |
| tree | 264d84260c4876dd1ce2c440032e5c2d06d2e337 | |
| parent | b2cc8263bbfc01a7b8f51fbe2c56b61bd4da3b5c (diff) | |
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@gmail.com>
Tested-by: Muthu Subramanian K <muthusuba@gmail.com>
| -rw-r--r-- | sw/source/ui/docvw/PostItMgr.cxx | 4 |
1 files 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<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(); |