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

Comment after arrow is being duplicated #1082

Closed
3 tasks
nojaf opened this issue Sep 4, 2020 · 0 comments · Fixed by #1085
Closed
3 tasks

Comment after arrow is being duplicated #1082

nojaf opened this issue Sep 4, 2020 · 0 comments · Fixed by #1085

Comments

@nojaf
Copy link
Contributor

nojaf commented Sep 4, 2020

Issue created from fantomas-online

Code

List.tryFind(fun { Type = t; Range = r }  ->
                    match t with
                    | MainNode SynMemberDefn_Member
                    | MainNode SynMemberSig_Member -> // trying to get AST trivia
                        RangeHelpers.``range contains`` r rangeOfBindingAndRhs

                    | Token(MEMBER, _) -> // trying to get token trivia
                        r.StartLine = rangeOfBindingAndRhs.StartLine

                    | _ -> false
                )

Result

List.tryFind (fun { Type = t; Range = r } -> // trying to get AST trivia
    match t with
    | MainNode SynMemberDefn_Member
    | MainNode SynMemberSig_Member -> // trying to get AST trivia
        RangeHelpers.``range contains`` r rangeOfBindingAndRhs

    | Token (MEMBER, _) -> // trying to get token trivia
        r.StartLine = rangeOfBindingAndRhs.StartLine

    | _ -> false)

Problem description

Trivia bug: the range of the first arrow should be limited instead of the entire lambda body.

Extra information

  • The formatted result breaks by code.
  • The formatted result gives compiler warnings.
  • I or my company would be willing to help fix this.

Options

Fantomas Master at 09/03/2020 19:59:28 - 573b009

Default Fantomas configuration

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 a pull request may close this issue.

1 participant