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.
Tags
None
Referenced Files
F15437680: D21290.id50699.diff
Tue, Mar 25, 9:00 PM
F15415877: D21290.id.diff
Thu, Mar 20, 8:04 AM
F15334857: D21290.diff
Sat, Mar 8, 12:42 PM
Unknown Object (File)
Feb 26 2025, 3:31 AM
Unknown Object (File)
Feb 19 2025, 8:41 PM
Unknown Object (File)
Feb 14 2025, 1:42 PM
Unknown Object (File)
Feb 14 2025, 2:09 AM
Unknown Object (File)
Feb 12 2025, 9:41 PM
Subscribers
None

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
Branch
white1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24519
Build 33789: Run Core Tests
Build 33788: arc lint + arc unit