Index: modularize/ModuleAssistant.cpp =================================================================== --- modularize/ModuleAssistant.cpp +++ modularize/ModuleAssistant.cpp @@ -68,7 +68,7 @@ // Destructor. Module::~Module() { // Free submodules. - while (SubModules.size()) { + while (!SubModules.empty()) { Module *last = SubModules.back(); SubModules.pop_back(); delete last; Index: module-map-checker/ModuleMapChecker.cpp =================================================================== --- module-map-checker/ModuleMapChecker.cpp +++ module-map-checker/ModuleMapChecker.cpp @@ -261,7 +261,7 @@ findUnaccountedForHeaders(); // Check for warnings. - if (UnaccountedForHeaders.size()) + if (!UnaccountedForHeaders.empty()) returnValue = std::error_code(1, std::generic_category()); // Dump module map if requested.