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.

Sign up for GitHub

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

Jump to bottom

The t_ignore_newline method is unnecessarily used in this code file src/cql/lexer.py #1

Closed
vignesh1507 opened this issue Oct 4, 2024 · 1 comment · May be fixed by #2
Closed

The t_ignore_newline method is unnecessarily used in this code file src/cql/lexer.py #1

vignesh1507 opened this issue Oct 4, 2024 · 1 comment · May be fixed by #2

Comments

@vignesh1507
Copy link

because the regular expression r"\n+" is already handled by the t_ignore = " \t\r\n\f\v" statement. This line already tells the lexer to ignore all whitespace, including newlines, so the t_ignore_newline method could be removed to avoid redundancy.
@vignesh1507 vignesh1507 mentioned this issue Oct 4, 2024
Open
@vignesh1507
Copy link
Author

resolved the issue #2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated the lexer.py code file. vignesh1507/cql-python
1 participant
@vignesh1507