10

typedef enum宣言と宣言の両方に一致する OCLint ルールを作成しようとしていますが、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 enumtypedef 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 ルールに一致するのは 1 つだけです。

編集

EnumDeclこれは、別のソース ファイルで定義されているforを示す AST ダンプに関係している可能性があると思いNS_ENUMます (おそらく NS_ENUM マクロのため)。

4

1 に答える 1

0

現在、oclint でこれを行う方法はないようです。ASTVisitorマクロは、 orルールの oclint ルールには公開されません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());
于 2015-08-14T18:03:33.460 に答える