[Libreoffice-commits] core.git: sc/source

[Libreoffice Gerrit user]

 sc/source/ui/cctrl/checklistmenu.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 05b9bf2099460a370af4e9c13feff815cdd234d0
Author:     Arkadiy Illarionov <qarkai at gmail.com>
AuthorDate: Mon Jan 14 22:39:57 2019 +0300
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Tue Jan 15 08:05:06 2019 +0100

    Use proper iterator in loop body
    
    Loop iterates over children to set their state
    7967e5e51e5210b8c3d3dc63502bd7d875eb36b7 follow-up
    
    Change-Id: I425b3204fb3581a22e9ed2b00c7b8c24ed2060a8
    Reviewed-on: https://gerrit.libreoffice.org/66333
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sc/source/ui/cctrl/checklistmenu.cxx b/sc/source/ui/cctrl/checklistmenu.cxx
index 5dd7e9b9ced5..8153d13b581b 100644
--- a/sc/source/ui/cctrl/checklistmenu.cxx
+++ b/sc/source/ui/cctrl/checklistmenu.cxx
@@ -1153,7 +1153,7 @@ void ScCheckListMenuWindow::setAllMemberState(bool bSet)
             SvTreeListEntries& rEntries = (*itr)->GetChildEntries();
             for (auto it = rEntries.begin(), itEnd = rEntries.end(); it != itEnd; ++ it)
             {
-                maChecks->CheckEntry(*itr, bSet);
+                maChecks->CheckEntry(it->get(), bSet);
             }
         }
     }