Page MenuHomePhabricator

Fix an issue when rendering a lint message which removes whitespace at the end of a file
ClosedPublic

Authored by epriestley on May 27 2020, 6:26 PM.

Details

Summary

Ref T13543. If a file ends in spaces and no newline, we'll emit a message suggesting removal of the spaces. This will effectively remove the line, but the code will then attempt to highlight text within the line.

Prior to D21044 this continued without raising an error and produced a reasonable result, but it now fatals. Insetad, don't try to highlight lines which no longer exist.

Test Plan

See T13543 for details.

Diff Detail

Repository
rARC Arcanist
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.May 27 2020, 6:26 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 27 2020, 6:26 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.