From d41d091c071da1e029b98b49ae01a91b4041fc6b Mon Sep 17 00:00:00 2001
From: uburuntu <bekbulatov.ramzan@ya.ru>
Date: Sat, 27 May 2017 04:15:54 +0200
Subject: [PATCH] ENH: perfomance: seeking of one char is more efficiently

---
 lib/cppcheck.cpp |  6 +++---
 tools/dmake.cpp  | 10 +++++-----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/lib/cppcheck.cpp b/lib/cppcheck.cpp
index 6e42e5d0d..6b7f0ab25 100644
--- a/lib/cppcheck.cpp
+++ b/lib/cppcheck.cpp
@@ -143,11 +143,11 @@ unsigned int CppCheck::processFile(const std::string& filename, const std::strin
 
         if (!_settings.plistOutput.empty()) {
             std::string filename2;
-            if (filename.find("/") != std::string::npos)
-                filename2 = filename.substr(filename.rfind("/") + 1);
+            if (filename.find('/') != std::string::npos)
+                filename2 = filename.substr(filename.rfind('/') + 1);
             else
                 filename2 = filename;
-            filename2 = _settings.plistOutput + filename2.substr(0, filename2.find(".")) + ".plist";
+            filename2 = _settings.plistOutput + filename2.substr(0, filename2.find('.')) + ".plist";
             plistFile.open(filename2);
             plistFile << ErrorLogger::plistHeader(version(), files);
         }
diff --git a/tools/dmake.cpp b/tools/dmake.cpp
index 84dbe9868..2fc053bd9 100644
--- a/tools/dmake.cpp
+++ b/tools/dmake.cpp
@@ -37,7 +37,7 @@ static std::string builddir(std::string filename)
 
 static std::string objfile(std::string cppfile)
 {
-    cppfile.erase(cppfile.rfind("."));
+    cppfile.erase(cppfile.rfind('.'));
     return builddir(cppfile + ".o");
 }
 
@@ -50,15 +50,15 @@ static void getDeps(const std::string &filename, std::vector<std::string> &depfi
     std::ifstream f(filename.c_str());
     if (! f.is_open()) {
         if (filename.compare(0, 4, "cli/") == 0 || filename.compare(0, 5, "test/") == 0)
-            getDeps("lib" + filename.substr(filename.find("/")), depfiles);
+            getDeps("lib" + filename.substr(filename.find('/')), depfiles);
         return;
     }
     if (filename.find(".c") == std::string::npos)
         depfiles.push_back(filename);
 
     std::string path(filename);
-    if (path.find("/") != std::string::npos)
-        path.erase(1 + path.rfind("/"));
+    if (path.find('/') != std::string::npos)
+        path.erase(1 + path.rfind('/'));
 
     std::string line;
     while (std::getline(f, line)) {
@@ -67,7 +67,7 @@ static void getDeps(const std::string &filename, std::vector<std::string> &depfi
             continue;
         pos1 += 10;
 
-        std::string::size_type pos2 = line.find("\"", pos1);
+        std::string::size_type pos2 = line.find('\"', pos1);
         std::string hfile(path + line.substr(pos1, pos2 - pos1));
         if (hfile.find("/../") != std::string::npos)    // TODO: Ugly fix
             hfile.erase(0, 4 + hfile.find("/../"));