Page MenuHomePhabricator

Add getDeserializationListener to ASTReader
ClosedPublic

Authored by yamaguchi on Apr 21 2018, 6:02 AM.

Details

Summary

We need to know if ASTReader already has a DeserializationListner or
not, and this also helps to create a multiplexing deserialization
listener if there is one already attached.

Diff Detail

Repository
rL LLVM

Event Timeline

yamaguchi created this revision.Apr 21 2018, 6:02 AM
rsmith accepted this revision.Apr 24 2018, 10:26 AM

This seems fine, but I wonder if we should expose a mechanism that will create a multiplex listener automatically as needed?

This revision is now accepted and ready to land.Apr 24 2018, 10:26 AM

Do you mean something like addDeserializationListener which create a multiplex listener with an existing listener? I think that'll be good as well!

This revision was automatically updated to reflect the committed changes.