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

WIP attempting to track output locations #256

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

camertron
Copy link

Hey @mbj :) This is the approach I came up with for emitting output locations. I made only a small number of the necessary changes. I'd like to ask for your feedback before making any further progress. What do you think? Here's a quick usage example:

# NOTE: the buffer parameter here can be removed eventually, I added it for debugging purposes
Unparser.unparse(ast) do |buffer, old_pos, new_pos|
  ...
end

@mbj
Copy link
Owner

mbj commented Jun 24, 2021

@camertron I'm tooooo busys to reply verbosely. Your approach is right, but. and that but requires a longer elaboration I cannot find the time for this and likely next week.

@camertron
Copy link
Author

@mbj hahaha no problem, no rush!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants