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

False syntax error on a.b.c = "x" #18

Open
agronholm opened this issue Sep 13, 2013 · 5 comments
Open

False syntax error on a.b.c = "x" #18

agronholm opened this issue Sep 13, 2013 · 5 comments

Comments

@agronholm
Copy link

It complains about "no viable input" on the "=".

@paulvi
Copy link
Member

paulvi commented Sep 13, 2013

Thank you.

Check README, the project is in DIY mode.

I know that syntax check makes more questions that solves.

@agronholm
Copy link
Author

Ah, okay...I just hadn't run into this with 0.2.2, so I failed to realize no real work had been done on 0.3.

@paulvi
Copy link
Member

paulvi commented Sep 14, 2013

Well, it is refreshing up project, that stood still for 1 year.
At least I hope you enjoyed new icon.

Internally we switched to maven, the build is easier now.
However as this project requires XTEXT knowledge, and non of active
contributor have it now,
the project will be in free state.

If I find time to look into, I would like just to disable all warnings
and syntax errors,
and leave only syntax highlight.

@agronholm
Copy link
Author

I've located the problem in the grammar file. The "assignable" construct does not follow the original grammar definition. I just might be able to fix this.
I also found the source of another annoyance: the editor inserting tabs as indent regardless of my preferences. There's a TODO in the code there, but I have no clue how to get the preference from the general editor settings. I've never done Eclipse plugin development so I'm a little clueless here. A few pointers would go a long way. Do you hang out on IRC?

@paulvi
Copy link
Member

paulvi commented Sep 15, 2013

As with any project, it takes some time to look into the project and find related place.
Than it is possible to ask around for help.

Or try just to change a bit and install locally
The good news is that now build is simple mvn package

paulvi added a commit that referenced this issue Sep 15, 2013
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