walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Refactoring: Using string::compare instead of strncmp

This commit is contained in:
Daniel Marjamäki 2010-02-19 07:29:27 +01:00
parent 5e90945486
commit 8411f449f8
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/mathlib.cpp
View File

@ -32,15 +32,15 @@
long MathLib::toLongNumber(const std::string &str)
{
if (strncmp(str.c_str(), "0x" , 2) == 0
|| strncmp(str.c_str(), "+0x", 3) == 0
|| strncmp(str.c_str(), "-0x", 3) == 0)
if (str.compare(0, 2, "0x") == 0
|| str.compare(0, 3, "+0x") == 0
|| str.compare(0, 3, "-0x") == 0)
{
return std::strtoul(str.c_str(), '\0', 16);
}
if (strncmp(str.c_str(), "0" , 1) == 0
|| strncmp(str.c_str(), "+0", 2) == 0
|| strncmp(str.c_str(), "-0", 2) == 0)
if (str.compare(0, 1, "0") == 0
|| str.compare(0, 2, "+0") == 0
|| str.compare(0, 2, "-0") == 0)
{
return std::strtoul(str.c_str(), '\0', 8);
}
@ -51,7 +51,7 @@ long MathLib::toLongNumber(const std::string &str)
double MathLib::toDoubleNumber(const std::string &str)
{
if (strncmp(str.c_str(), "0x", 2) == 0)
if (str.compare(0, 2, "0x") == 0)
{
return std::strtoul(str.c_str(), '\0', 16);
}