Правило OCLint ASTMatcher. Соответствует NS_ENUM

Я пытаюсь создать правило 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.

9

Решение

Похоже, что в настоящее время есть способ сделать это в 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());
0

Другие решения


По вопросам рекламы [email protected]