MetzNet
/
dfhack
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fixed (fix as in hack) crash when std::string was used to store a command line argument

develop
mizipzor 2010-02-19 21:24:20 +01:00
parent adbf4b81b4
commit 7f404483f1
1 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
library/argstream/argstream.h
Unescape Escape View File

@ -635,15 +635,17 @@ namespace
// The option and its associated value are removed, the subtle thing // The option and its associated value are removed, the subtle thing
// is that someother options might have this associated value too, // is that someother options might have this associated value too,
// which we must invalidate. // which we must invalidate.
s.values_.erase(iter->second); s.values_.erase(iter->second);
for (std::map<std::string,argstream::value_iterator>::iterator
jter = s.options_.begin();jter != s.options_.end();++jter) // FIXME this loop seems to crash if a std::string is used as the value
{ //for (std::map<std::string,argstream::value_iterator>::iterator
if (jter->second == iter->second) // jter = s.options_.begin();jter != s.options_.end();++jter)
{ //{
jter->second = s.values_.end(); // if (jter->second == iter->second)
} // {
} // jter->second = s.values_.end();
// }
//}
s.options_.erase(iter); s.options_.erase(iter);
} }
else else