xmloff/source/chart/SchXMLExport.cxx |   10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

New commits:
commit 9ab81e7c44001ad86461673719cf1ac22f8932c5
Author:     Xisco Fauli <[email protected]>
AuthorDate: Fri Nov 22 11:59:18 2024 +0100
Commit:     Xisco Fauli <[email protected]>
CommitDate: Fri Nov 22 13:44:47 2024 +0100

    PVS: V547 Expression '!aPropertyStates.empty()' is always true.
    
    Change-Id: Id71d10060c79d36095bbd952b89aa6af68a29a6c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176999
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <[email protected]>

diff --git a/xmloff/source/chart/SchXMLExport.cxx 
b/xmloff/source/chart/SchXMLExport.cxx
index a1fac7424cf2..e2780ee0b684 100644
--- a/xmloff/source/chart/SchXMLExport.cxx
+++ b/xmloff/source/chart/SchXMLExport.cxx
@@ -3158,10 +3158,7 @@ void SchXMLExportHelper_Impl::exportRegressionCurve(
             if( bExportContent )
             {
                 // add style name attribute
-                if( !aPropertyStates.empty())
-                {
-                    AddAutoStyleAttribute( aPropertyStates );
-                }
+                AddAutoStyleAttribute( aPropertyStates );
 
                 SvXMLElementExport aRegressionExport( mrExport, 
XML_NAMESPACE_CHART, XML_REGRESSION_CURVE, true, true );
                 if( bExportEquation )
@@ -3191,10 +3188,7 @@ void SchXMLExportHelper_Impl::exportRegressionCurve(
             }
             else    // autostyles
             {
-                if( !aPropertyStates.empty())
-                {
-                    CollectAutoStyle( std::move(aPropertyStates) );
-                }
+                CollectAutoStyle( std::move(aPropertyStates) );
                 if( bExportEquation && !aEquationPropertyStates.empty())
                 {
                     CollectAutoStyle( std::move(aEquationPropertyStates) );

Reply via email to