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

[PRISM] Locations #9945

Merged
merged 4 commits into from
Feb 13, 2024
Merged

[PRISM] Locations #9945

merged 4 commits into from
Feb 13, 2024

Conversation

kddnewton
Copy link
Contributor

This set of commits does a lot of stuff.

  • Fixes up the brace style in prism_compile.c to be consistent.
  • Stops passing around both scope_node and parser, because parser is on scope_node.
  • Introduces a bunch of PUSH_* macros that are equivalent to their ADD_* counterparts but operators on pm_line_column_t structs instead of NODE * pointers. This is to allow us to pass the locations directly instead of going through dummy_line_node.
  • Starts converting things over to using the new macros. This is far from done, but gets the process started.

@kddnewton kddnewton merged commit 4e481c7 into ruby:master Feb 13, 2024
98 checks passed
@kddnewton kddnewton deleted the line-numbers branch February 13, 2024 15:26
@kddnewton kddnewton restored the line-numbers branch February 21, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant