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

Make FileLister::acceptFile() a static method. 

We don't need different acceptFile() implementations per platform
so it is better to have one static method.
This commit is contained in:
Kimmo Varis 2011-02-03 11:48:16 +02:00
parent 9b9b9c99f5
commit 96d62553bc
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cli/filelister.h
View File

@ -55,7 +55,7 @@ public:
* @param filename filename to check * @param filename filename to check
* @return returns true if the file extension indicates it should be checked * @return returns true if the file extension indicates it should be checked
*/ */
virtual bool acceptFile(const std::string &filename); static bool acceptFile(const std::string &filename);
/** /**
* @brief Is given path a directory? * @brief Is given path a directory?

2
cli/filelister_unix.cpp
View File

@ -72,7 +72,7 @@ void FileListerUnix::recursiveAddFiles2(std::vector<std::string> &relative,
continue; continue;
} }
if (Path::sameFileName(path,filename) || FileListerUnix::acceptFile(filename)) if (Path::sameFileName(path,filename) || FileLister::acceptFile(filename))
{ {
relative.push_back(filename); relative.push_back(filename);
absolute.push_back(fname); absolute.push_back(fname);