diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2018-03-12 14:29:48 +0000 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2018-03-13 20:57:48 +0100 |
| commit | b8345af80078f9eee29a391fe84b2506b37d24ea (patch) | |
| tree | a023a056253744749bc921b4e01239824443899a | |
| parent | 97dd69362dc4924cf837843bc949ba8ffec0d063 (diff) | |
tdf#107094 backward heading search doesn't always work
regression from...
commit 84e44386d70e886f9bae633ebe915ea3a52f8dbd
Author: Caolán McNamara <caolanm@redhat.com>
Date: Wed Nov 12 10:35:18 2014 +0000
coverity#982189 Unchecked return value
so put it back the way it was and explicitly ignore the return
instead
Change-Id: I6f5ef42b38236ea5f70ddbccfa25a81f2f3e976c
Change-Id: I0c0084247e8570ace2198fa893db273dab00d2b0
Reviewed-on: https://gerrit.libreoffice.org/51147
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
| -rw-r--r-- | sw/source/core/crsr/crstrvl.cxx | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sw/source/core/crsr/crstrvl.cxx b/sw/source/core/crsr/crstrvl.cxx index 8c1a0627f1d0..b65ba7d8dcf3 100644 --- a/sw/source/core/crsr/crstrvl.cxx +++ b/sw/source/core/crsr/crstrvl.cxx @@ -976,8 +976,9 @@ bool SwCursorShell::GotoPrevOutline() SwNode* pNd = &(pCursor->GetNode()); SwOutlineNodes::size_type nPos; - bool bRet = rNds.GetOutLineNds().Seek_Entry(pNd, &nPos); - if (bRet && nPos) + bool bRet = false; + (void)rNds.GetOutLineNds().Seek_Entry(pNd, &nPos); + if (nPos) { --nPos; // before |