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:34 +0100 |
| commit | e4c16d917d7cb4c7c220e9e51cd8c4a386317305 (patch) | |
| tree | 20d4cfb128436ba64d0718cb06c6efe7334dcff8 | |
| parent | 2845f350064f9999a52fa08b752490c3ef7e98c4 (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/51148
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 0dd9819023a5..fba4d61de3a3 100644 --- a/sw/source/core/crsr/crstrvl.cxx +++ b/sw/source/core/crsr/crstrvl.cxx @@ -981,8 +981,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 |