core.git: chart2/source

[Xisco Fauli (via logerrit)]

 chart2/source/model/template/XYDataInterpreter.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 7785b72d49051c052c829499d13cba4414df9540
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Mon Oct 21 10:47:54 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Mon Oct 21 13:20:17 2024 +0200

    tdf#163486: PVS: Two similar code fragments were found
    
    Since
    commit 4c2701d4cc8b27eed56172b4188b61efb8783c55
    Author: Vladimir Glazounov <v...@openoffice.org>
    Date:   Tue May 22 17:53:11 2007 +0000
    
        INTEGRATION: CWS chart2mst3 (1.1.4); FILE ADDED
    
    V778    Two similar code fragments were found. Perhaps, this is a typo and 
'xValuesX' variable should be used instead of 'xValuesY'.
    
    Change-Id: I24a904ee2e0fa81eb50756545d4b1127b93579fc
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175322
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/chart2/source/model/template/XYDataInterpreter.cxx 
b/chart2/source/model/template/XYDataInterpreter.cxx
index 2404f5e7f521..d6daa2a5b1d9 100644
--- a/chart2/source/model/template/XYDataInterpreter.cxx
+++ b/chart2/source/model/template/XYDataInterpreter.cxx
@@ -180,7 +180,7 @@ InterpretedData XYDataInterpreter::reinterpretDataSeries(
                 {
                     xValuesX = aValueSeqVec[nIndex++];
                     if( xValuesX.is())
-                        SetRole( xValuesY->getValues(), u"values-x"_ustr);
+                        SetRole( xValuesX->getValues(), u"values-x"_ustr);
                 }
             }
             if( xValuesY.is())