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

Fixities #1

Open
feuerbach opened this Issue Dec 1, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@feuerbach
Copy link
Member

feuerbach commented Dec 1, 2012

Note that fixities for both local and global symbols must be supported.

@phischu

This comment has been minimized.

Copy link
Contributor

phischu commented Feb 10, 2015

Fixities are out of scope of this library

@phischu phischu closed this Feb 10, 2015

@feuerbach

This comment has been minimized.

Copy link
Member Author

feuerbach commented Feb 10, 2015

Why do you think so?

@phischu

This comment has been minimized.

Copy link
Contributor

phischu commented Feb 11, 2015

To be honest, just because I personally don't need them at the moment. We could indeed offer a function that creates a Map Symbol Fixity. We could then offer a second function that reassociates a given module AST with the given fixities. But these are orthogonal to name resolution. I'll reopen.

@phischu phischu reopened this Feb 11, 2015

@feuerbach

This comment has been minimized.

Copy link
Member Author

feuerbach commented Feb 11, 2015

We could indeed offer a function that creates a Map Symbol Fixity.

That wouldn't suffice; local names may have fixities, too. It could be Map (Either Symbol SrcLoc) Fixity

Strictly speaking, it doesn't have to belong here, but there's no other place to put it (except a separate package). Fixitiy resolution is a direct client of name resolution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.