[Libreoffice-commits] core.git: sw/source

[Caolán McNamara (via logerrit)]

 sw/source/core/unocore/unochart.cxx |   15 ---------------
 1 file changed, 15 deletions(-)

New commits:
commit 002d3a9474d92929df8089ee03f6277f45a5ef77
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Wed Dec 14 11:15:17 2022 +0000
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Wed Dec 14 14:07:43 2022 +0000

    cid#1212509 Logically dead code
    
    accept the original logic then and drop the dead path instead of
    the attempt to "fix it" of
    
    commit 0d6f45377592a62ff4ee510d5fbd3bdd9e20160a
    Author: Caolán McNamara <caol...@redhat.com>
    Date:   Wed Oct 15 09:52:01 2014 +0100
    
        coverity#1212509 Logically dead code
    
    Change-Id: I373a85f53ceb395e27f85cc44cd17f650088b5f2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144160
    Tested-by: Caolán McNamara <caol...@redhat.com>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sw/source/core/unocore/unochart.cxx 
b/sw/source/core/unocore/unochart.cxx
index 03886d20bc5d..34af9f068002 100644
--- a/sw/source/core/unocore/unochart.cxx
+++ b/sw/source/core/unocore/unochart.cxx
@@ -751,28 +751,13 @@ uno::Reference< chart2::data::XDataSource > 
SwChartDataProvider::Impl_createData
     sal_Int32 nNumLDS = 0;
     if (oiEnd > 0)
     {
-        sal_Int32 nFirstSeqLen = 0;
-        sal_Int32 nFirstSeqLabelIdx = -1;
         for (oi = 0; oi < oiEnd; ++oi)
         {
-            bool bFirstFound = false;
             // row/col used at all?
             if (aDataStartIdx[oi] != -1 &&
                 (!bFirstIsLabel || aLabelIdx[oi] != -1))
             {
                 ++nNumLDS;
-                if (!bFirstFound)
-                {
-                    nFirstSeqLen        = aDataLen[oi];
-                    nFirstSeqLabelIdx   = aLabelIdx[oi];
-                    bFirstFound = true;
-                }
-                else
-                {
-                    if (nFirstSeqLen != aDataLen[oi] ||
-                        nFirstSeqLabelIdx != aLabelIdx[oi])
-                        throw lang::IllegalArgumentException();
-                }
             }
         }
     }