Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Conversation

smashwilson
Copy link
Contributor

If the final row of a range passed to scanInRange is empty, then the break statement shouldn't trigger on a match at the range's end, because a zero-length match is a valid and expected result.

Manifesting in Atom as atom/atom#13489.

@maxbrunsfeld
Copy link
Contributor

Great explanation. Thanks for the fix @smashwilson!

@maxbrunsfeld maxbrunsfeld merged commit c12a47e into master Dec 28, 2016
@maxbrunsfeld maxbrunsfeld deleted the aw-scaninrange-empty branch December 28, 2016 18:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants