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, 3 insertions, 4 deletions

diff --git a/protocols/MSN/src/msn_avatar.cpp b/protocols/MSN/src/msn_avatar.cpp
index cf520027a7..b2f1f36cb9 100644
--- a/protocols/MSN/src/msn_avatar.cpp
+++ b/protocols/MSN/src/msn_avatar.cpp
@@ -121,8 +121,7 @@ void __cdecl CMsnProto::MSN_AvatarsThread(void*)
 	}

 

 	mir_cslock lck(csAvatarQueue);

-	while (lsAvatarQueue.getCount() > 0) {

-		delete lsAvatarQueue[0];

-		lsAvatarQueue.remove(0);

-	}

+	for (auto &it : lsAvatarQueue)

+		delete it;

+	lsAvatarQueue.destroy();

 }