From 890d1f91ff23dc63c9f3513826ac431bcc8a375a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?miKa=E3=81=B4=E3=82=87=E3=82=93?= Date: Sun, 27 Sep 2015 06:12:12 +0200 Subject: [PATCH] Try to follow coding conventions Some tiny changes as requested: - Changed loop bracket style - Cleaned up the loop for more readability - Using a const_interator - Removed NULL check --- indra/newview/llfloaterblacklist.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/indra/newview/llfloaterblacklist.cpp b/indra/newview/llfloaterblacklist.cpp index d2f6b46e6..dde1e07c1 100644 --- a/indra/newview/llfloaterblacklist.cpp +++ b/indra/newview/llfloaterblacklist.cpp @@ -208,11 +208,12 @@ void LLFloaterBlacklist::onClickRemove(void* user_data) if(list->getFirstSelected()) { uuid_vec_t selectedIDs = list->getSelectedIDs(); - for(typename uuid_vec_t::iterator iterator = selectedIDs.begin(); - iterator != selectedIDs.end(); - ++iterator) { + typename uuid_vec_t::const_iterator iterator; + for(iterator = selectedIDs.begin(); + iterator != selectedIDs.end(); + ++iterator) + { LLUUID selectedID = *iterator; - if(selectedID.isNull()) continue; blacklist_entries.erase(selectedID); } list->deleteSelectedItems();