how about using "auto" here for better readability, instead of "std::vector<std::string>::const_iterator"?