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, 1 insertions, 1 deletions

diff --git a/protocols/IcqOscarJ/src/utilities.cpp b/protocols/IcqOscarJ/src/utilities.cpp
index 97253e2cc1..e7af9c8a33 100644
--- a/protocols/IcqOscarJ/src/utilities.cpp
+++ b/protocols/IcqOscarJ/src/utilities.cpp
@@ -388,10 +388,10 @@ void CIcqProto::DeleteFromContactsCache(MCONTACT hContact)
 

 	for (auto &it : contactsCache) {

 		if (it->hContact == hContact) {

-			contactsCache.remove(it);

 			// Release memory

 			SAFE_FREE((void**)&it->szUid);

 			SAFE_FREE((void**)&it);

+			contactsCache.removeItem(&it);

 			break;

 		}

 	}