basctl/source/basicide/bastype3.cxx |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

New commits:
commit c0975e6344bf93921655a8f362660b181f564ee2
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Fri Oct 18 17:36:10 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Sat Nov 2 01:01:29 2024 +0100

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

diff --git a/basctl/source/basicide/bastype3.cxx 
b/basctl/source/basicide/bastype3.cxx
index deb28b7cf41d..741409665d44 100644
--- a/basctl/source/basicide/bastype3.cxx
+++ b/basctl/source/basicide/bastype3.cxx
@@ -299,6 +299,7 @@ EntryDescriptor SbTreeListBox::GetEntryDescriptor(const 
weld::TreeIter* pEntry)
                 }
                 break;
                 case OBJ_TYPE_MODULE:
+                case OBJ_TYPE_DIALOG:
                 {
                     aName = pair.second;
                     eType = pBE->GetType();
@@ -310,12 +311,6 @@ EntryDescriptor SbTreeListBox::GetEntryDescriptor(const 
weld::TreeIter* pEntry)
                     eType = pBE->GetType();
                 }
                 break;
-                case OBJ_TYPE_DIALOG:
-                {
-                    aName = pair.second;
-                    eType = pBE->GetType();
-                }
-                break;
                 case OBJ_TYPE_DOCUMENT_OBJECTS:
                 case OBJ_TYPE_USERFORMS:
                 case OBJ_TYPE_NORMAL_MODULES:

Reply via email to