diff options
author | Caolán McNamara <caolanm@redhat.com> | 2018-03-12 14:29:48 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2018-03-12 22:07:28 +0100 |
commit | 02cf066ee9b8a833a6f19c41489aadff3561a56b (patch) | |
tree | 8b30a21733e2f6c2edc5ede5bc727a9562bc4ca4 | |
parent | aeb925a031c69a2c6572a9d6de3c3ebd74b4f2e5 (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/51146
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@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 ad156de3c4b7..2fac420a2966 100644 --- a/sw/source/core/crsr/crstrvl.cxx +++ b/sw/source/core/crsr/crstrvl.cxx @@ -973,8 +973,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 |