Page MenuHomePhabricator

Removed the 'id' AST matcher, which is superseded by '.bind()'
ClosedPublic

Authored by gribozavr on Aug 20 2019, 2:46 AM.

Details

Summary

The 'id' matcher is not even included in the AST Matchers Reference
document, so I don't expect there to be a significant number of users.

There's no reason to provide two ways to do the exact same thing that
only have a minor syntactic difference.

Diff Detail

Repository
rL LLVM

Event Timeline

gribozavr created this revision.Aug 20 2019, 2:46 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 20 2019, 2:46 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
gribozavr updated this revision to Diff 216082.Aug 20 2019, 2:48 AM

Fixed a typo.

klimek accepted this revision.Aug 20 2019, 2:51 AM

Yay, thanks!

This revision is now accepted and ready to land.Aug 20 2019, 2:51 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 20 2019, 6:01 AM