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

[Mike Kaganski]

 sw/source/uibase/docvw/PostItMgr.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit b4aa631867638b993fda98b1c4fcfc47e5c6a1aa
Author: Mike Kaganski <mike.kaganski at collabora.com>
Date:   Wed Nov 15 12:53:01 2017 +0200

    3813dd1315 follow-up: revert incorrect loop changes
    
    Change-Id: Ifdccf64630e76f6189668aa7f7412c41a6af04c2
    Reviewed-on: https://gerrit.libreoffice.org/44756
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
    Tested-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sw/source/uibase/docvw/PostItMgr.cxx b/sw/source/uibase/docvw/PostItMgr.cxx
index 06c0bc6d75ce..f88c6a19a881 100644
--- a/sw/source/uibase/docvw/PostItMgr.cxx
+++ b/sw/source/uibase/docvw/PostItMgr.cxx
@@ -246,10 +246,10 @@ SwPostItMgr::~SwPostItMgr()
 void SwPostItMgr::CheckForRemovedPostIts()
 {
     bool bRemoved = false;
-    for(auto i = mvPostItFields.begin(); i != mvPostItFields.end(); ++i)
+    for(auto i = mvPostItFields.begin(); i != mvPostItFields.end(); )
     {
-        auto it = ++i;
-        if ( it != mvPostItFields.end() && !(*it)->UseElement() )
+        auto it = i++;
+        if ( !(*it)->UseElement() )
         {
             SwSidebarItem* p = (*it);
             mvPostItFields.remove(*it);