summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2014-06-03 18:57:23 +0300
committerAndrei Karas <akaras@inbox.ru>2014-06-03 18:57:23 +0300
commit11cc29197700eed64f9a82f81b503dd75dfa1811 (patch)
tree3784a4e4e6541f22c0dfd9d7c39131640188925c /src
parentd0cc5158fbd3168a6ece834ca46b1635cb150012 (diff)
downloadmplint-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.am1
-rw-r--r--src/rules/include.cpp64
-rw-r--r--src/stringutils.cpp10
-rw-r--r--src/stringutils.h2
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