commit 65e5486f6db884a5ef6e30abc41a6dc06d1cfb1a
author Caolán McNamara <caolanm@redhat.com> Wed Dec 14 11:10:33 2022 +0000
committer Xisco Fauli <xiscofauli@libreoffice.org> Wed Dec 14 15:41:31 2022 +0000
tree eadff03c55704dbe3fb17ef333c05fdd79111c9e
parent d5dcd17c1d29109dd75a61d2ca434b6a5b29a367

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/+/144129
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>

sw/source/core/unocore/unochart.cxx

diff --git a/sw/source/core/unocore/unochart.cxx b/sw/source/core/unocore/unochart.cxx
index e1b8e14..c8c282c 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))