core.git: chart2/source

[Bogdan Buzea (via logerrit)]

 chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx |    4 
+---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 6ce9e887158c20c37dc0f5c06f1e236c007229af
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Fri Oct 18 17:11:11 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Sat Nov 2 00:58:15 2024 +0100

    tdf#163486: PVS: Identical branches
    
    
    Change-Id: I2f34381ccf0e9649874652b382e43fbd1737577a
    V1037: Two or more case-branches perform the same actions. Check lines: 
121, 127
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175147
    Tested-by: Jenkins
    Reviewed-by: David Gilbert <freedesk...@treblig.org>

diff --git 
a/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx 
b/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx
index 209300f2f682..6043c5ff298a 100644
--- a/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx
+++ b/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx
@@ -118,14 +118,12 @@ sal_Int32 lcl_getSymbolType( const css::chart2::Symbol& 
rSymbol )
         case chart2::SymbolStyle_NONE:
             break;
         case chart2::SymbolStyle_AUTO:
+        case chart2::SymbolStyle_POLYGON://new feature
             nSymbol = css::chart::ChartSymbolType::AUTO;
             break;
         case chart2::SymbolStyle_STANDARD:
             nSymbol = rSymbol.StandardSymbol%15;
             break;
-        case chart2::SymbolStyle_POLYGON://new feature
-            nSymbol = css::chart::ChartSymbolType::AUTO;
-            break;
         case chart2::SymbolStyle_GRAPHIC:
             nSymbol = css::chart::ChartSymbolType::BITMAPURL;
             break;