Page MenuHomePhabricator

Fix a lint renderering issue
ClosedPublic

Authored by joshuaspence on Jan 11 2015, 9:02 AM.
Tags
None
Referenced Files
F18281160: D11319.id27188.diff
Fri, Aug 22, 10:40 PM
F18281155: D11319.id27189.diff
Fri, Aug 22, 10:40 PM
Unknown Object (File)
May 19 2025, 4:31 PM
Unknown Object (File)
May 19 2025, 4:21 PM
Unknown Object (File)
May 19 2025, 4:20 PM
Unknown Object (File)
May 19 2025, 4:19 PM
Unknown Object (File)
May 8 2025, 11:15 AM
Unknown Object (File)
Apr 28 2025, 7:21 AM
Subscribers

Details

Summary

Currently there is an issue when renderering linter messages with ArcanistConsoleLintRenderer if $message->getChar() returns 0 (i.e. the value is unset).

Test Plan

Before

lint_renderer_(before) (122×453 px, 8 KB)

After

lint_renderer_(after) (123×308 px, 7 KB)

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Fix a lint renderering issue.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence edited edge metadata.

Minor whitespace change

epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 11 2015, 4:57 PM
This revision was automatically updated to reflect the committed changes.