diff options
author | Thorbjørn Lindeijer <thorbjorn@lindeijer.nl> | 2011-03-17 00:42:38 +0100 |
---|---|---|
committer | Thorbjørn Lindeijer <thorbjorn@lindeijer.nl> | 2011-03-17 00:57:16 +0100 |
commit | 9344a79233882ac278b3812b91b6edf874ef5d16 (patch) | |
tree | e5bd324104a9d8dcf1839de8a880845f8739a37a /src/utils/stringfilter.cpp | |
parent | 0e24b15a386d45f43cea76c1b8ad744728a3190e (diff) | |
download | manaserv-9344a79233882ac278b3812b91b6edf874ef5d16.tar.gz manaserv-9344a79233882ac278b3812b91b6edf874ef5d16.tar.bz2 manaserv-9344a79233882ac278b3812b91b6edf874ef5d16.tar.xz manaserv-9344a79233882ac278b3812b91b6edf874ef5d16.zip |
Micro-optimizations related to std::string
* Rely on the fact that a std::string is empty by default
* Use std::string::empty() rather than comparing to ""
* Construct with std::string() rather than from ""
Reviewed-by: Bertram
Diffstat (limited to 'src/utils/stringfilter.cpp')
-rw-r--r-- | src/utils/stringfilter.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/utils/stringfilter.cpp b/src/utils/stringfilter.cpp index 617da90d..b48d267f 100644 --- a/src/utils/stringfilter.cpp +++ b/src/utils/stringfilter.cpp @@ -43,13 +43,13 @@ bool StringFilter::loadSlangFilterList() { mInitialized = false; - std::string slangsList = Configuration::getValue("SlangsList", ""); - if (slangsList != "") { + const std::string slangsList = Configuration::getValue("SlangsList", + std::string()); + if (!slangsList.empty()) { std::istringstream iss(slangsList); std::string tmp; - while (getline(iss, tmp, ',')) { + while (getline(iss, tmp, ',')) mSlangs.push_back(tmp); - } mInitialized = true; } @@ -59,7 +59,7 @@ bool StringFilter::loadSlangFilterList() void StringFilter::writeSlangFilterList() { // Write the list to config - std::string slangsList = ""; + std::string slangsList; for (SlangIterator i = mSlangs.begin(); i != mSlangs.end(); ) { slangsList += *i; |