search

thSoft / elysium

LilyPond IDE for Eclipse
http://elysium.thsoft.hu
14 stars 3 forks source link

add validation preferences #117

Closed nittka closed 8 years ago

nittka commented 8 years ago

This pull request adds validation preferences, addressing and fixing #99/#116 and preparing the resolution of #86.

nittka commented 8 years ago

I have added a first handling of the "\include \foo" case. I decided to use UnparsedCommand rather than Reference (so no linking to the definition of foo) for the following reason. Claiming not to handle variable includes contradicts resolving the variable - if you find the variable why not use the result for the include. Generally, this would pose the problem of having to link for obtaining the includes for linking..

nittka commented 8 years ago

I have incorporated your comments.

nittka commented 8 years ago

I added the explanation. With merging this pull request, I would consider #99, #116, #86 and #93 resolved. Supporting scheme constructs in general is a different matter of a completely different scale...

thSoft commented 8 years ago

Awesome, thanks!