Skip to content

/ html5ever

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

Track line numbers in the tokenizer #232

Closed
wants to merge 5 commits into from
Closed

Track line numbers in the tokenizer #232

wants to merge 5 commits into from
+151 −9

Conversation

@karenher
Copy link
Contributor

karenher commented Nov 3, 2016 •
edited by larsbergstrom

This change is Reviewable
karenher and others added 5 commits Oct 3, 2016
@karenher
added argument line_number to process_token
623098f
@karenher
line_number argument added to process_token in trait
ba5b6fa
@karenher1
added test functions to check that tokens are on expected lines
abaca40
@karenher1
added variables line_number parameter into process_token functions
6fb84df
@karenher1
added comment for TokenSink implementation
Loading status checks…
7a8a7bf
@karenher karenher changed the title Added new variable current_line for tracking line numbers for tokens. Created impl of TokenSink called TokenMatch to test behavior of Tokenizer's process_token function. Track line numbers in the tokenizer Nov 7, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2016

☔ The latest upstream changes (presumably #231) made this pull request unmergeable. Please resolve the merge conflicts.
@karenher karenher closed this Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@karenher @bors-servo @karenher1
You can’t perform that action at this time.