search

eclipse / gemoc-studio-execution-ale

Gemoc studio execution ale
0 stars 7 forks source link

Use AleInterpreter close #24

Closed dvojtise closed 4 years ago

dvojtise commented 4 years ago

This PR bumps to the latest version of ALE which now offer a close() method in its interpreter.

Calling this method ensures dispose of unused resources that are retained by Sirius JavaExtension (via registration in the eclipse WorkkspaceListener)

ebousse commented 4 years ago

Looks great! If you are confident that the new ALE version is stable then let's merge.

BTW a bit out of topic: is there a changelog and/or a roadmap available for ALE? As I start using it a bit I would be interested :)

dvojtise commented 4 years ago

This latest version fixes several bugs (including a memory leak in the editor and += operator fix) and several improvements in the editor validator (it provides better feedback for several error)

However, integration with GEMOC is still a little bit ugly due to https://github.com/gemoc/ale-lang/issues/26 (the workaround is to duplicate the key in the dsl file...)

You're right there no clear roadmap except fixing issues listed in the tracker when they become too annoying for an upcoming demo ... (everyone is welcome to add a +1 on an issue or feature request to help sort the issues :wink: )

About changelog, ALE follows the same strategy as in GEMOC: every important change is done and reviewed via pull requests.

echebbi commented 4 years ago

@ebousse ALE now provides a CHANGELOG: ale-lang/CHANGELOG.md