diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-11-12 20:33:06 +0000 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2014-11-18 12:57:40 +0000 |
| commit | 60d34e1c840d2c317bb7d0a5b14f4602c22b3fcc (patch) | |
| tree | 94e0f04f979b966fde20577f4bcee01bf74cb571 /sw | |
| parent | a81a24b6a33fca9a750077d9160ebb8316b2fb26 (diff) | |
coverity#735517 Logically dead code
its possible that this was the original intent, maybe
Change-Id: Ia7ab679b84f16ae7aab201b43eb69ce6a693c0f2
Reviewed-on: https://gerrit.libreoffice.org/12392
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/core/layout/layact.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sw/source/core/layout/layact.cxx b/sw/source/core/layout/layact.cxx index 122567b2ab30..355f5e55d911 100644 --- a/sw/source/core/layout/layact.cxx +++ b/sw/source/core/layout/layact.cxx @@ -473,8 +473,7 @@ void SwLayAction::InternalAction() sal_uInt16 nPercentPageNum = 0; while ( (pPage && !IsInterrupt()) || nCheckPageNum != USHRT_MAX ) { - if ( !pPage && nCheckPageNum != USHRT_MAX && - (!pPage || pPage->GetPhyPageNum() >= nCheckPageNum) ) + if (!pPage || (nCheckPageNum != USHRT_MAX && pPage->GetPhyPageNum() >= nCheckPageNum)) { if ( !pPage || pPage->GetPhyPageNum() > nCheckPageNum ) { |