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.

Sign up for GitHub

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

Jump to bottom

Transform multiple nodes without creating statements #175

Merged

thekid merged 1 commit into master from feature/transform-multiple-without-statements

Oct 1, 2023

Member

thekid commented Oct 1, 2023 •

edited

Loading

Otherwise the following would generate a ; following a method, which is a syntax error.

// Transform one method to two
$emitter->transform('method', function($codegen, $node) {
  yield null => new Method(...);
  yield null => new Method(...);
});

This can be considered a workaround, it would be better to have Statement nodes, but this would be a significant refactoring!


          Transform multiple nodes without creating statements

f600243

thekid merged commit e047051 into master

18 checks passed

thekid deleted the feature/transform-multiple-without-statements branch

October 1, 2023 18:38

Member Author

thekid commented Oct 1, 2023

Released in https://github.com/xp-framework/compiler/releases/tag/v8.16.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet