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

Improve the parser error messages #4

Open
nikita-volkov opened this issue Jan 21, 2015 · 3 comments
Open

Improve the parser error messages #4

nikita-volkov opened this issue Jan 21, 2015 · 3 comments

Comments

@nikita-volkov
Copy link
Owner

Attoparsec's messages seem to be anything but useful. I mean I get

    Parser failure: Failed reading: satisfy. Contexts: ["exp","'('"].

for almost any syntactical mistake. I guess unless there'll be major changes involving integration with "haskell-src-exts", we'll have to switch to Parsec.

Suggestions?

@nikita-volkov
Copy link
Owner Author

Just a heads up. The preprocessor reimplementation addresses this.

@rabipelais
Copy link
Contributor

Also, the current parser (I don't know about the new implementation) chokes up with infix operators, like <$> or !*! (from Linear) when used in record expressions, for example:
x = [r| {matrix = mat1 !*! mat2, wrappedNumber = fromIntegral <$> stuff} |]

(I does work if I let-bind the sub expressions and then assign them, i.e. [r| {matrix = mat',....}|] where mat' = mat1 !*! mat2...)

@nikita-volkov
Copy link
Owner Author

@rabipelais It's just the lack of the support for this part of syntax in the current parser implementation. The preprocessor version will support all of the Haskell syntax.

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

No branches or pull requests

2 participants