Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge DesugaredLambda and Lambda to one active pattern. #1633

Merged
merged 1 commit into from
Apr 11, 2021

Conversation

nojaf
Copy link
Contributor

@nojaf nojaf commented Apr 11, 2021

Fixes #1631.

    | Lambda of
        fromMethod: bool *
        inLambdaSeq: bool *
        args: SynSimplePats *
        body: SynExpr *
        parsedData: (SynPat list * SynExpr) option *
        range: range

parsedData contains all the information necessary to print out the parameters of a lambda (destructed or not).
This simplifies Fantomas a bit like the whole ComplexPats dance can be skipped.

@nojaf nojaf merged commit 03d7a32 into fsprojects:master Apr 11, 2021
@nojaf nojaf deleted the fix-1631 branch April 11, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fantomas add extra parenthesis in desugared lambda.
1 participant