diff options
| author | Tillmann Karras <tilkax@gmail.com> | 2014-02-23 23:03:39 +0100 |
|---|---|---|
| committer | Tillmann Karras <tilkax@gmail.com> | 2014-02-28 12:43:20 +0100 |
| commit | 315a8ba1c04bf8cffd9b04096be898ac462fa89e (patch) | |
| tree | 5655bdf4d940b3fdd372cb76181d70137982aefc /Source/Core/Common/IniFile.cpp | |
| parent | 5f0a8008f45f2c30a1aed044a428c5b4ce09ce88 (diff) | |
Various changes suggested by cppcheck
- remove unused variables
- reduce the scope where it makes sense
- correct limits (did you know that strcat()'s last parameter does not
include the \0 that is always added?)
- set some free()'d pointers to NULL
Diffstat (limited to 'Source/Core/Common/IniFile.cpp')
| -rw-r--r-- | Source/Core/Common/IniFile.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/Source/Core/Common/IniFile.cpp b/Source/Core/Common/IniFile.cpp index 0637342a3d..2cc1a97e60 100644 --- a/Source/Core/Common/IniFile.cpp +++ b/Source/Core/Common/IniFile.cpp @@ -124,13 +124,12 @@ bool IniFile::Section::Get(const std::string& key, std::vector<std::string>& out } // ignore starting , if any size_t subStart = temp.find_first_not_of(","); - size_t subEnd; // split by , while (subStart != std::string::npos) { // Find next , - subEnd = temp.find_first_of(",", subStart); + size_t subEnd = temp.find_first_of(",", subStart); if (subStart != subEnd) // take from first char until next , out.push_back(StripSpaces(temp.substr(subStart, subEnd - subStart))); |