Я пытаюсь создать правило OCLint, которое соответствует обоим typedef enum
а также typedef NS_ENUM
декларации, с небольшим успехом.
У меня есть файл Objective-C (TestClass.m) со следующими объявлениями перечисления в нем:
typedef NS_ENUM(NSInteger, TestEnum) {
TestEnumNone,
TestEnumSome,
TestEnumAll
};
typedef enum {
othertestvalue = 0,
othertestvalue1,
othertestvalue2
} OtherTestEnum;
Дамп AST с помощью этой команды:
clang -Xclang -ast-dump -fsyntax-only Classes/TestClass.m -- | grep Enum
Дает мне этот вывод, содержащий это:
|-TypedefDecl 0x7f9d3accd630 <col:1, col:28> col:28 TestEnum 'enum TestEnum':'enum TestEnum'
|-EnumDecl 0x7f9d3accd6a8 prev 0x7f9d3accd530 </System/Library/Frameworks/CoreFoundation.framework/Headers/CFAvailability.h:171:57, Classes/TestClass.m:71:1> line:67:28 TestEnum 'NSInteger':'long'
| |-EnumConstantDecl 0x7f9d3accd738 <line:68:5> col:5 TestEnumNone 'NSInteger':'long'
| |-EnumConstantDecl 0x7f9d3accd788 <line:69:5> col:5 TestEnumSome 'NSInteger':'long'
| `-EnumConstantDecl 0x7f9d3accd7d8 <line:70:5> col:5 TestEnumAll 'NSInteger':'long'
|-EnumDecl 0x7f9d3accd828 <line:73:9, line:77:1> line:73:9
| |-EnumConstantDecl 0x7f9d3accd900 <line:74:5, col:22> col:5 othertestvalue 'int'
| |-EnumConstantDecl 0x7f9d3accd950 <line:75:5> col:5 othertestvalue1 'int'
| `-EnumConstantDecl 0x7f9d3accd9a0 <line:76:5> col:5 othertestvalue2 'int'
|-TypedefDecl 0x7f9d3accda40 <line:73:1, line:77:3> col:3 OtherTestEnum 'enum OtherTestEnum':'OtherTestEnum'
У меня есть ASTMatcherRule (ObjCNsEnumRule), где я пытаюсь сопоставить оба typedef enum
так же как typedef NS_ENUM
Вот код для этого:
#include "oclint/AbstractASTMatcherRule.h"#include "oclint/RuleSet.h"
using namespace std;
using namespace clang;
using namespace clang::ast_matchers;
using namespace oclint;
class ObjCNsEnumRuleRule : public AbstractASTMatcherRule
{
public:
virtual const string name() const override
{
return "obj c ns enum rule";
}
virtual int priority() const override
{
return 3;
}
virtual void callback(const MatchFinder::MatchResult &result) override
{
const EnumDecl *enumDecl = result.Nodes.getNodeAs<EnumDecl>("enum");
if (enumDecl) {
addViolation(enumDecl, this, "Found enum");
}
}
virtual void setUpMatcher() override
{
addMatcher(enumDecl().bind("enum"));
}
};
static RuleSet rules(new ObjCNsEnumRuleRule());
Однако, когда я запускаю это правило, я получаю только вывод для typedef enum
декларация.
Classes/TestClass.m:73:9: obj c ns enum rule P3 Found enum
Что я здесь не так делаю? Оба перечисления отображаются в дампе AST, но только одно совпадение в правиле OCLint.
редактировать
Я думаю, что это может быть связано с дампом AST, показывающим EnumDecl
для NS_ENUM
как определено в другом исходном файле (возможно, из-за макроса NS_ENUM), поскольку я могу соответствовать typedef, но не enumdecl.
Похоже, что в настоящее время есть способ сделать это в Oclint. На макросы не распространяются правила oclint для ASTVisitor
или же ASTMatcher
правила. Посмотреть здесь: https://github.com/oclint/oclint/issues/148
Я реализовал это как простой SourceCodeReaderRule
, как это:
#include "oclint/AbstractSourceCodeReaderRule.h"#include "oclint/RuleSet.h"#include <iostream>
#include <string>
#include <regex>
using namespace std;
using namespace oclint;
class TypedefEnumStatementRule : public AbstractSourceCodeReaderRule
{
public:
virtual const string name() const override {
return "typedef enum statement";
}
virtual int priority() const override {
return 1;
}
virtual void eachLine(int lineNumber, string line) override {
regex rgx("typedef\\senum");
smatch match;
if (regex_search(line, rgx, regex_constants::match_continuous)) {
string description = "Enums should not be declared with 'typedef enum' use 'typedef NS_ENUM' instead";
addViolation(lineNumber, 1, lineNumber, 1, this, description);
}
}
};
static RuleSet rules(new TypedefEnumStatementRule());