================
@@ -568,6 +568,34 @@ class ASTWalker : public RecursiveASTVisitor<ASTWalker> {
     }
     return true;
   }
+
+  void reportObjCLiteralMethod(SourceLocation Loc, ObjCMethodDecl *Method) {
+    if (!Method)
+      return;
+    report(Loc, Method->getClassInterface());
+  }
----------------
nico wrote:

I thought something like

```
void reportObjCLiteralMethod(SourceLocation Loc, ObjCMethodDecl *Method) {
  if (!Method)
    return;
  report(Loc, Method);
  report(Loc, Method->getClassInterface());
}
```

Is the first not needed? Maybe for the category test? Speaking of which, where 
did it go? It was a good test :) Bring it back? (with annotation moved to the 
method line)

https://github.com/llvm/llvm-project/pull/216201
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to