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
Fix: no-multiple-empty-lines crash on space after last \n (fixes #8401) #8402
Conversation
Previously, no-multiple-empty-lines would crash if it tried to remove a trailing newline followed by a space from the end of a file. This commit fixes the crash.
@not-an-aardvark, thanks for your PR! By analyzing the history of the files in this pull request, we identified @RRMoelker, @alberto and @gcochard to be potential reviewers. |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
lib/rules/no-multiple-empty-lines.js
Outdated
*/ | ||
const lineNumberAfterRemovedLines = lineNumber - maxAllowed; | ||
const rangeEnd = lineNumberAfterRemovedLines <= allLines.length | ||
? sourceCode.getIndexFromLoc({ line: lineNumber - maxAllowed, column: 0 }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: but u can use the variable lineNumberAfterRemovedLines
here also.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, fixed.
LGTM |
What is the purpose of this pull request? (put an "X" next to item)
[x] Bug fix (#8401)
What changes did you make? (Give an overview)
Previously, no-multiple-empty-lines would crash if it tried to remove a trailing newline followed by a space from the end of a file. This commit fixes the crash.
Is there anything you'd like reviewers to focus on?
Nothing in particular