starmath/source/mathtype.cxx |   26 +++-----------------------
 1 file changed, 3 insertions(+), 23 deletions(-)

New commits:
commit 9af69930f06ac8b5ff56ad04c42b90a3ef97c0b9
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sat Oct 19 19:07:15 2024 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Fri Nov 15 14:56:56 2024 +0100

    tdf#163486: PVS: Identical branches
    
    V1037 Two or more case-branches perform the same actions. Check lines: 830, 
840, 850
    V1037 Two or more case-branches perform the same actions. Check lines: 960, 
1148
    
    Change-Id: Ifff7612b44d2a3bc0ea573077104b460fed8cf76
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175222
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
    Tested-by: Jenkins

diff --git a/starmath/source/mathtype.cxx b/starmath/source/mathtype.cxx
index d78879ee7087..4289a9691975 100644
--- a/starmath/source/mathtype.cxx
+++ b/starmath/source/mathtype.cxx
@@ -820,32 +820,14 @@ bool MathType::HandleRecords(int nLevel, sal_uInt8 
nSelector,
                             rRet.append(" {");
                             break;
                         case tmLARROW:
-                            if (nPart == 0)
-                            {
-                                if (nVariation == 0)
-                                    rRet.append(" widevec ");//left arrow above
-                                else if (nVariation == 1)
-                                    rRet.append(" widevec ");//left arrow below
-                                rRet.append(" {");
-                            }
-                            break;
                         case tmRARROW:
-                            if (nPart == 0)
-                            {
-                                if (nVariation == 0)
-                                    rRet.append(" widevec ");//right arrow 
above
-                                else if (nVariation == 1)
-                                    rRet.append(" widevec ");//right arrow 
below
-                                rRet.append(" {");
-                            }
-                            break;
                         case tmBARROW:
                             if (nPart == 0)
                             {
                                 if (nVariation == 0)
-                                    rRet.append(" widevec ");//double arrow 
above
+                                    rRet.append(" widevec ");//arrow above
                                 else if (nVariation == 1)
-                                    rRet.append(" widevec ");//double arrow 
below
+                                    rRet.append(" widevec ");//arrow below
                                 rRet.append(" {");
                             }
                             break;
@@ -950,6 +932,7 @@ bool MathType::HandleRecords(int nLevel, sal_uInt8 
nSelector,
                             break;
                         case tmUHBRACE:
                         case tmLHBRACE:
+                        case tmSLFRACT:
                             rRet.append(" {");
                             break;
                         case tmSUM:
@@ -1137,9 +1120,6 @@ bool MathType::HandleRecords(int nLevel, sal_uInt8 
nSelector,
                             if (nVariation == 1)
                                 rRet.append("overline ");
                             break;
-                        case tmSLFRACT:
-                            rRet.append(" {");
-                            break;
                         case tmINTOP:
                             if (nPart == 0)
                             {

Reply via email to