aaronpuchert added inline comments.
================
Comment at: clang/lib/AST/CommentSema.cpp:135-137
+      if (const auto *VD = dyn_cast<VarDecl>(ThisDeclInfo->CurrentDecl))
+        if (VD->getType()->isFunctionPointerType())
+          return;
----------------
Something might be wrong here anyway: the 
[documentation](https://developer.apple.com/library/archive/documentation/DeveloperTools/Conceptual/HeaderDoc/tags/tags.html)
 says that `@callback`
> Specifies the name and description of a callback field in a structure.
But we're looking for a `VarDecl` instead of a `FieldDecl`. Is somebody still 
around who knows HeaderDoc? Seems like Apple doesn't care any longer.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113795/new/

https://reviews.llvm.org/D113795

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to