BRNSystems/minetest
1
0
Fork 0
forked from Mirrorlandia_minetest/minetest
Code Pull Requests 1 Activity

Replace for loop with call to standard library function (#8194)

Browse Source 
This loop makes multiple passes over m_stack (type std::list) in order to remove all elements with a specified value. Replacing the loop with a call to std::list::remove does the same job, but in only one pass.
This commit is contained in:
Benjamin Lindley 2019-03-05 01:13:15 -06:00 committed by Loïc Blot
parent 5d2624ab82
commit e19565c170
1 changed files with 1 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/gui/mainmenumanager.h
View File

@ -62,21 +62,7 @@ public:
virtual void deletingMenu(gui::IGUIElement *menu) virtual void deletingMenu(gui::IGUIElement *menu)
{ {
// Remove all entries if there are duplicates // Remove all entries if there are duplicates
bool removed_entry; m_stack.remove(menu);
do{
removed_entry = false;
for(std::list<gui::IGUIElement*>::iterator
i = m_stack.begin();
i != m_stack.end(); ++i)
{
if(*i == menu)
{
m_stack.erase(i);
removed_entry = true;
break;
}
}
}while(removed_entry);
/*core::list<GUIModalMenu*>::Iterator i = m_stack.getLast(); /*core::list<GUIModalMenu*>::Iterator i = m_stack.getLast();
assert(*i == menu); assert(*i == menu);