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

Exp10 needs to be refactored #13

Open
rtoal opened this issue Apr 20, 2016 · 2 comments
Open

Exp10 needs to be refactored #13

rtoal opened this issue Apr 20, 2016 · 2 comments
Assignees

Comments

@rtoal
Copy link
Collaborator

rtoal commented Apr 20, 2016

Currently the following is syntactically legal

7 . 2

It shouldn't be.

@SirSeim
Copy link
Owner

SirSeim commented Apr 20, 2016

What I was discussing with Iko was that is it necessary for it to be a syntax error. As I had it in my mind, this is valid syntax, so it would pass the parser, but it would be in the analyzer where it is determined to be incorrect.

Is this thinking incorrect, should it really be in the grammar that 7 . 2 should not be allowed?

@rtoal
Copy link
Collaborator Author

rtoal commented Apr 20, 2016

You can do it either way.

But for things that can be restricted in the syntax, why not?

The syntax can certainly prohibit literals before dots, just as it can certainly ensure that the only thing coming after a dot is an id.

It's good practice to tighten things up in the syntax.

On Apr 20, 2016, at 2:29 PM, Edward Seim <notifications@github.commailto:notifications@github.com> wrote:

What I was discussing with Iko was that is it necessary for it to be a syntax error. As I had it in my mind, this is valid syntax, so it would pass the parser, but it would be in the analyzer where it is determined to be incorrect.

Is this thinking incorrect, should it really be in the grammar that 7 . 2 should not be allowed?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com//issues/13#issuecomment-212613813

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