[PATCH] D72233: Add a new AST matcher 'optionally'.

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 10 12:33:25 PST 2020

Previous message: [PATCH] D72233: Add a new AST matcher 'optionally'.
Next message: [PATCH] D70689: [analyzer] Fix SARIF column locations
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

steveire added a comment.

LGTM too. I described a more ad-hoc implementation previously: https://steveire.wordpress.com/2018/11/20/composing-ast-matchers-in-clang-tidy/ but I prefer this implementation.


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

https://reviews.llvm.org/D72233

Previous message: [PATCH] D72233: Add a new AST matcher 'optionally'.
Next message: [PATCH] D70689: [analyzer] Fix SARIF column locations
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the cfe-commits mailing list