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

AST design guide lines #1

Closed
mar9000 opened this Issue Jan 7, 2019 · 3 comments

Comments

Projects
None yet
2 participants
@mar9000
Copy link

commented Jan 7, 2019

In extending my implementation of ECMAScript I had to take decisions on how to structuring language concepts hierarchy. In parsed languages like javascript one can try to adhere to:

  1. parse tree
  2. AST proposed by the library used for import.
  3. none of the above and just follow minimum effort in MPS.

For instance the export statements:

export {a, b};
export {a, b} from 'file.js';

are implemented with different node in the AST of shift-java (ExportLocals and ExportFrom respectively) but at parser, and probably at MPS level, would be better to implement with the same production/concept with an optional FromClause.

@coolya coolya transferred this issue from coolya/basic-lang Jan 23, 2019

@coolya

This comment has been minimized.

Copy link
Owner

commented Feb 5, 2019

@mar9000 sorry for the delay. I would like to answer this one this thursday in the stream if that would fit for you. Otherwise I can also do it next week. I'm going to return to the usual Tuesday / Thursday schedule now.

@mar9000

This comment has been minimized.

Copy link
Author

commented Feb 5, 2019

Hi, I saw the schedule. On thursday I have always an appointment around 20.30/21 so I can follow the first part of the streaming but not the whole one. On Tuesday I'm free the whole evening.

@coolya

This comment has been minimized.

Copy link
Owner

commented Feb 5, 2019

The lets schedule it for next Tuesday. :)

@mar9000 mar9000 closed this Mar 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.