LIST<> iterators:

- new method LIST::removeItem added to save a pointer to removed record;
- code cleaning related to the fact that LIST::remove() shall be the last operation inside an iterator, because otherwise the reference to it will point to a record next to deleted one;
- a few remaining cycles converted to iterators

1 files changed, 6 insertions, 3 deletions

diff --git a/plugins/Popup/src/popup_thread.cpp b/plugins/Popup/src/popup_thread.cpp
index 75e306a0b8..5caede1cf1 100644
--- a/plugins/Popup/src/popup_thread.cpp
+++ b/plugins/Popup/src/popup_thread.cpp
@@ -179,9 +179,12 @@ static LRESULT CALLBACK PopupThreadManagerWndProc(HWND hwnd, UINT message, WPARA
 		break;

 

 	case UTM_REMOVE_WINDOW:

-		for (int i = popupList.getCount() - 1; i >= 0; i--)

-			if (popupList[i] == wnd)

-				popupList.remove(i);

+		{

+			auto T = popupList.rev_iter();

+			for (auto &it : T)

+				if (it == wnd)

+					popupList.remove(T.indexOf(&it));

+		}

 

 		RepositionPopups();

 		--nPopups;