diff options
author | Andrei Karas <akaras@inbox.ru> | 2014-06-03 18:57:23 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2014-06-03 18:57:23 +0300 |
commit | 11cc29197700eed64f9a82f81b503dd75dfa1811 (patch) | |
tree | 3784a4e4e6541f22c0dfd9d7c39131640188925c /src | |
parent | d0cc5158fbd3168a6ece834ca46b1635cb150012 (diff) | |
download | mplint-11cc29197700eed64f9a82f81b503dd75dfa1811.tar.gz mplint-11cc29197700eed64f9a82f81b503dd75dfa1811.tar.bz2 mplint-11cc29197700eed64f9a82f81b503dd75dfa1811.tar.xz mplint-11cc29197700eed64f9a82f81b503dd75dfa1811.zip |
add rule for checking include file dirs.
Diffstat (limited to 'src')
-rw-r--r-- | src/Makefile.am | 1 | ||||
-rw-r--r-- | src/rules/include.cpp | 64 | ||||
-rw-r--r-- | src/stringutils.cpp | 10 | ||||
-rw-r--r-- | src/stringutils.h | 2 |
4 files changed, 77 insertions, 0 deletions
diff --git a/src/Makefile.am b/src/Makefile.am index 13592ca..4bcbe76 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -21,6 +21,7 @@ mplint_SOURCES = \ rules/debug.cpp \ rules/dump.cpp \ rules/final.cpp \ + rules/include.cpp \ rules/license.cpp # set the include path found by configure diff --git a/src/rules/include.cpp b/src/rules/include.cpp new file mode 100644 index 0000000..347bf58 --- /dev/null +++ b/src/rules/include.cpp @@ -0,0 +1,64 @@ +/* + * The ManaPlus Client + * Copyright (C) 2014 The ManaPlus Developers + * + * This file is part of The ManaPlus Client. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 3 of the License, or + * any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see <http://www.gnu.org/licenses/>. + */ + +#include "template.hpp" + +registerRuleExt(include, "008", "(.+)[.](cpp|h)") + +startRule(include) +{ +} + +endRule(include) +{ +} + +parseLineRule(include) +{ + std::string data0 = data; + trim(data); + if (data == "#include \"config.h\"") + return; + + if (findCutFirst(data, "#include \"")) + { + if (findCutLast(data, "\"")) + { + if (!fileExists(rootDir + data)) + { + print("Wrong include " + data0 + ". " + "Probably you should use path from src dir," + " or use #include <file.ext>"); + } + } + } + else if (findCutFirst(data, "#include <")) + { + if (findCutLast(data, ">")) + { + if (fileExists(rootDir + data)) + { + print("Wrong include " + data0 + ". " + "Found local file with same name."); + } + } + } +} + diff --git a/src/stringutils.cpp b/src/stringutils.cpp index d750165..1f5ff47 100644 --- a/src/stringutils.cpp +++ b/src/stringutils.cpp @@ -31,6 +31,8 @@ #include <list> #include <regex> +#include <sys/stat.h> + std::string &trim(std::string &str) { size_t pos = str.find_last_not_of(' '); @@ -711,3 +713,11 @@ bool isMatch(const std::string &str, const std::string &exp) std::regex regExp(exp); return std::regex_match(str, regExp); } + +bool fileExists(const std::string &name) +{ + struct stat statbuf; + if (stat(name.c_str(), &statbuf)) + return false; + return true; +} diff --git a/src/stringutils.h b/src/stringutils.h index 817114e..3cbcdb9 100644 --- a/src/stringutils.h +++ b/src/stringutils.h @@ -239,4 +239,6 @@ void secureChatCommand(std::string &str); bool isMatch(const std::string &str, const std::string &exp); +bool fileExists(const std::string &name); + #endif // STRINGUTILS_H |