Handle macro expansion args in placeholder expansion #3092
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update
ExpandEditorPlaceholdersToLiteralClosures
andCallToTrailingClosures
such that they can handle macro expansion exprs and decls. This unfortunately requires changing them such that they take aSyntax
input and output to satisfy the conformance toSyntaxRefactoringProvider
, but that seems preferable to making the refactoring types themselves generic.If in the future we decide to expose
CallLikeSyntax
as a public protocol, we could decide to expose additional genericrefactor
overloads here. However it's not clear there's enough clients of these APIs to motivate doing that in this PR.