commit 89188837f60d83a901b2c43f73d30e1957920f50
author Caolán McNamara <caolanm@redhat.com> Wed Dec 14 11:10:33 2022 +0000
committer Caolán McNamara <caolanm@redhat.com> Wed Dec 14 15:36:52 2022 +0000
tree f87d117468cfe0f99c479c2ff5c7c6383925f6fc
parent 473006f247098c94f28b346621eb11859b1cc1fd

Resolves: tdf#151753 'revert coverity#1212509 Logically dead code'

revert 0d6f45377592a62ff4ee510d5fbd3bdd9e20160a:

    coverity#1212509 Logically dead code

Change-Id: I8eb7355a94ce2c93faa18c77be63de41bccd3a99
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144159
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
(cherry picked from commit 6880d32d851783a56c977872df5fbf11b507876a)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144128

sw/source/core/unocore/unochart.cxx

diff --git a/sw/source/core/unocore/unochart.cxx b/sw/source/core/unocore/unochart.cxx
index ec75013..03886d2 100644
--- a/sw/source/core/unocore/unochart.cxx
+++ b/sw/source/core/unocore/unochart.cxx
@@ -753,9 +753,9 @@ uno::Reference< chart2::data::XDataSource > SwChartDataProvider::Impl_createData
    {
        sal_Int32 nFirstSeqLen = 0;
        sal_Int32 nFirstSeqLabelIdx = -1;
        bool bFirstFound = false;
        for (oi = 0; oi < oiEnd; ++oi)
        {
            bool bFirstFound = false;
            // row/col used at all?
            if (aDataStartIdx[oi] != -1 &&
                (!bFirstIsLabel || aLabelIdx[oi] != -1))