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

Consolidate line label parsing #83

Open
rillian opened this issue Jun 21, 2019 · 0 comments
Open

Consolidate line label parsing #83

rillian opened this issue Jun 21, 2019 · 0 comments

Comments

@rillian
Copy link
Contributor

rillian commented Jun 21, 2019

Working on #78, I noticed some inconsistencies in the way line labels (numbers) are recognized by the lexer. For example, the list of characters accepted as a primer marker for relative line numbers is different in different contexts, and none of them accept labels like 109a. in Q000040 from the sample corpus.

Probably the pattern for the line label should be declared once and concatenated into all the lexer patterns which need to recognize them.

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

No branches or pull requests

1 participant