fly/flightgear
1
0
Fork 0
Code Activity

Performance optimization: empty() instead of size()>0

Browse source 
empty() is guaranteed to be constant complexity for both vectors and lists, while size() has linear complexity for lists.
This commit is contained in:
Tom Paoletti 2013-03-21 19:43:14 -07:00 committed by James Turner
parent 1f88d1d11b
commit b27eb2571c
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/Scripting/NasalSys.cxx
View file

@ -893,7 +893,7 @@ void FGNasalSys::loadScriptDirectory(simgear::Dir nasalDir)
// Create module with list of scripts // Create module with list of scripts
void FGNasalSys::addModule(string moduleName, simgear::PathList scripts) void FGNasalSys::addModule(string moduleName, simgear::PathList scripts)
{ {
if (scripts.size()>0) if (! scripts.empty())
{ {
SGPropertyNode* nasal = globals->get_props()->getNode("nasal"); SGPropertyNode* nasal = globals->get_props()->getNode("nasal");
SGPropertyNode* module_node = nasal->getChild(moduleName,0,true); SGPropertyNode* module_node = nasal->getChild(moduleName,0,true);