core.git: sc/source

[Bogdan Buzea (via logerrit)]

 sc/source/ui/cctrl/checklistmenu.cxx |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit 972b33b14c3106d360ee2ba28bb780379e150b18
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sun Oct 20 11:39:26 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Sun Oct 20 19:56:17 2024 +0200

    tdf#163486: PVS: Identical branches
    
    V1037 Two or more case-branches perform the same actions. Check lines: 913, 
922
    
    Change-Id: Ie9ef10d15c51b9b0702dbda41768afdee3655eab
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175236
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/sc/source/ui/cctrl/checklistmenu.cxx 
b/sc/source/ui/cctrl/checklistmenu.cxx
index 792bd3aebd99..cd66538719b1 100644
--- a/sc/source/ui/cctrl/checklistmenu.cxx
+++ b/sc/source/ui/cctrl/checklistmenu.cxx
@@ -909,14 +909,11 @@ IMPL_LINK_NOARG(ScCheckListMenuControl, TriStateHdl, 
weld::Toggleable&, void)
 {
     switch (mePrevToggleAllState)
     {
-        case TRISTATE_FALSE:
-            mxChkToggleAll->set_state(TRISTATE_TRUE);
-            setAllMemberState(true);
-        break;
         case TRISTATE_TRUE:
             mxChkToggleAll->set_state(TRISTATE_FALSE);
             setAllMemberState(false);
         break;
+        case TRISTATE_FALSE:
         case TRISTATE_INDET:
         default:
             mxChkToggleAll->set_state(TRISTATE_TRUE);