OCLint ASTMatcher Regel. Übereinstimmende NS_ENUM

Question

Ich versuche, eine OCLint-Regel zu erstellen, die beide typedef enum und typedef NS_ENUM Deklarationen mit wenig Erfolg zu erstellen. Ich habe eine Objective-C-Datei (TestClass.m) mit folgenden Enum Erklärungen darin:

typedef NS_ENUM(NSInteger, TestEnum) { 
    TestEnumNone, 
    TestEnumSome, 
    TestEnumAll 
}; 

typedef enum { 
    othertestvalue = 0, 
    othertestvalue1, 
    othertestvalue2 
} OtherTestEnum; 

die AST mit diesem Befehl Dumping:

clang -Xclang -ast-dump -fsyntax-only Classes/TestClass.m -- | grep Enum 

Gibt mir diesen Ausgang dieses enthält:

|-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' 

ich habe eine ASTMatcherRule (ObjCNsEnumRule), wo ich sowohl typedef enum passe auch versuchen, wie typedef NS_ENUM Hier ist der Code für das heißt:

#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()); 

Allerdings, wenn ich diese Regel laufen, bekomme ich nur die Ausgabe für die typedef enum Erklärung.

Classes/TestClass.m:73:9: obj c ns enum rule P3 Found enum 

Was mache ich hier falsch? Beide Enums werden im AST-Dump angezeigt, in der OCLint-Regel jedoch nur ein Treffer.

bearbeiten

ich denke, das mit dem AST-Dump zu tun haben, kann die EnumDecl für die NS_ENUM wie definiert in einer anderen Quelldatei (wahrscheinlich wegen der NS_ENUM Makro) zeigt, wie ich die typedef bieten kann , aber nicht das enumdecl.

Answer 1

Es sieht nicht so aus, als ob es momentan einen Weg gibt, dies in Oklint zu tun. Makros sind keinen Oblint-Regeln für ASTVisitor oder ASTMatcher Regeln ausgesetzt. Siehe hier: https://github.com/oclint/oclint/issues/148

ich dies als eine einfache SourceCodeReaderRule endete Umsetzung wie folgt aus:

#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());