reportdesign/source/ui/dlg/GroupsSorting.cxx |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

New commits:
commit 00c2d0c6e5dd567c0b235ad95f6b3c4282725821
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Tue Oct 22 20:56:46 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Fri Oct 25 03:10:23 2024 +0200

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

diff --git a/reportdesign/source/ui/dlg/GroupsSorting.cxx 
b/reportdesign/source/ui/dlg/GroupsSorting.cxx
index 8080f5d1a33b..a067cf166795 100644
--- a/reportdesign/source/ui/dlg/GroupsSorting.cxx
+++ b/reportdesign/source/ui/dlg/GroupsSorting.cxx
@@ -1093,9 +1093,8 @@ void OGroupsSortingDialog::displayGroup(const 
uno::Reference<report::XGroup>& _x
         case report::GroupOn::DEFAULT:
             nPos = 0;
             break;
+        case report::GroupOn::INTERVAL:
         case report::GroupOn::PREFIX_CHARACTERS:
-            nPos = 1;
-            break;
         case report::GroupOn::YEAR:
             nPos = 1;
             break;
@@ -1117,9 +1116,6 @@ void OGroupsSortingDialog::displayGroup(const 
uno::Reference<report::XGroup>& _x
         case report::GroupOn::MINUTE:
             nPos = 7;
             break;
-        case report::GroupOn::INTERVAL:
-            nPos = 1;
-            break;
         default:
             nPos = 0;
     }

Reply via email to