chart2/source/model/main/ChartModel_Persistence.cxx |   10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

New commits:
commit 3a2e5c6d93aac9cee35c861cf259b46071832432
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Thu Oct 17 14:08:29 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Thu Oct 17 21:00:30 2024 +0200

    tdf#163486: PVS: identical conditional expressions
    
    V581: The conditional expressions of the 'if' statements situated alongside 
each other are identical. Check lines: 406, 410.
    V581: The conditional expressions of the 'if' statements situated alongside 
each other are identical. Check lines: 410, 418.
    
    Change-Id: I971f243870a96f7195d2819f2a0a44bb63840fac
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175083
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
    Tested-by: Jenkins

diff --git a/chart2/source/model/main/ChartModel_Persistence.cxx 
b/chart2/source/model/main/ChartModel_Persistence.cxx
index 3c3481420069..d508c1de527b 100644
--- a/chart2/source/model/main/ChartModel_Persistence.cxx
+++ b/chart2/source/model/main/ChartModel_Persistence.cxx
@@ -404,19 +404,15 @@ void ChartModel::insertDefaultChart()
                 if( bIsRTL )
                     xLegend->setPropertyValue( u"AnchorPosition"_ustr, 
uno::Any( chart2::LegendPosition_LINE_START ));
                 if(xDiagram.is())
+                {
                     xDiagram->setLegend( xLegend );
 
-                // set simple 3D look
-                if( xDiagram.is() )
-                {
+                    // set simple 3D look
                     xDiagram->setPropertyValue( u"RightAngledAxes"_ustr, 
uno::Any( true ));
                     xDiagram->setPropertyValue( u"D3DScenePerspective"_ustr, 
uno::Any( drawing::ProjectionMode_PARALLEL ));
                     xDiagram->setScheme( 
ThreeDLookScheme::ThreeDLookScheme_Realistic );
-                }
 
-                //set some new 'defaults' for wall and floor
-                if( xDiagram.is() )
-                {
+                    //set some new 'defaults' for wall and floor
                     Reference< beans::XPropertySet > xWall( 
xDiagram->getWall() );
                     if( xWall.is() )
                     {

Reply via email to