From 0c6d962f0b46f78939d07f5f6f24d267cfecfde4 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Mon, 4 Jan 2016 15:48:47 +0300 Subject: Add checks for // TRANSLATORS: comment for gettext strings. --- src/Makefile.am | 1 + src/rules/translation.cpp | 70 +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 71 insertions(+) create mode 100644 src/rules/translation.cpp diff --git a/src/Makefile.am b/src/Makefile.am index 58030c9..e516d03 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -26,6 +26,7 @@ mplint_SOURCES = \ rules/license.cpp \ rules/packet.cpp \ rules/po.cpp \ + rules/translation.cpp \ rules/virtual.cpp \ rules/xml.cpp diff --git a/src/rules/translation.cpp b/src/rules/translation.cpp new file mode 100644 index 0000000..e8a2fdd --- /dev/null +++ b/src/rules/translation.cpp @@ -0,0 +1,70 @@ +/* + * 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 . + */ + +#include "template.hpp" + +registerRuleExt(Translation, "014", "(.+)[.](cpp|h|hpp|cc|inc)") + +startRule(Translation) +{ + if (strEndWith(file, "/catch.hpp") || + strEndWith(file, "/localconsts.h") || + strEndWith(file, "/socialtabbase.h") || + strEndWith(file, "/utils/gettext.h")) + { + terminateRule(); + } +} + +endRule(Translation) +{ +} + +bool gotComment(false); + +parseLineRule(Translation) +{ + trim(data); + if (data.find("__attribute__") != std::string::npos || + data.find(",J?4P'.P\"_(\\?d'.,") != std::string::npos) + { + // nothing + } + else if (data.find("_(") != std::string::npos || + data.find("N_(") != std::string::npos) + { + if (!gotComment) + print("Missing \"// TRANSLATORS: \" comment before this line."); + else + gotComment = false; + } + else if (data.find("// TRANSLATORS: ") == 0) + { + gotComment = true; + } + else if (strStartWith(data, "//") && gotComment) + { + // nothing + } + else + { + gotComment = false; + } +} -- cgit v1.2.3-70-g09d2