Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

Get rid of seen_line_token #476

Open
jyn514 opened this issue Jun 12, 2020 · 0 comments
Open

Get rid of seen_line_token #476

jyn514 opened this issue Jun 12, 2020 · 0 comments
Labels
enhancement New feature or request lexer Issue dealing with parsing the lexical tokens of a program preprocessor Issue in the preprocessor (probably cycle detection) technical-debt

Comments

@jyn514
Copy link
Owner

jyn514 commented Jun 12, 2020

Now that the lexer returns whitespace tokens (#356), we don't need it anymore.

To change this, you'd move seen_line_token from the lexer to the preprocessor, and in the preprocessor check if any whitespace token contained a \n; if so, reset seen_line_token.

@jyn514 jyn514 added enhancement New feature or request lexer Issue dealing with parsing the lexical tokens of a program technical-debt preprocessor Issue in the preprocessor (probably cycle detection) labels Jun 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request lexer Issue dealing with parsing the lexical tokens of a program preprocessor Issue in the preprocessor (probably cycle detection) technical-debt
Projects
None yet
Development

No branches or pull requests

1 participant