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

Compiled operators names are replaced with source names #409

Closed
auduchinok opened this issue Feb 1, 2019 · 1 comment
Closed

Compiled operators names are replaced with source names #409

auduchinok opened this issue Feb 1, 2019 · 1 comment

Comments

@auduchinok
Copy link
Contributor

Sometimes using compiled name is preferred as source name produces a warning in this particular case. The problem is FCS replaces all source names with compiled names in AST idents and we don't know whether compiled or source name was used in the actual code.

Issue created from fantomas-ui.

Code

type T() =
    static member op_LessThan(a, b) = a < b

Result

type T() =
    static member (<) (a, b) = a < b

Options

Fantomas 2.9.1

Name Value
IndentSpaceNum 4
PageWidth 80
PreserveEndOfLine true
SemicolonAtEndOfLine false
SpaceBeforeArgument true
SpaceBeforeColon true
SpaceAfterComma true
SpaceAfterSemicolon true
IndentOnTryWith false
ReorderOpenDeclaration false
SpaceAroundDelimiter true
StrictMode false
@nojaf
Copy link
Contributor

nojaf commented Jul 9, 2019

Hi, I have a fix for this in 112452e.
However, I'm not 100% sure I covered all the possible cases here.
Let me know if you find scenarios where it is still a problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants