Page MenuHomePhabricator

Apply linter fixes
ClosedPublic

Authored by joshuaspence on Jun 10 2014, 1:01 AM.
Tags
None
Referenced Files
F14842618: D9449.diff
Sun, Feb 2, 12:40 AM
Unknown Object (File)
Mon, Jan 20, 4:37 PM
Unknown Object (File)
Mon, Jan 20, 2:52 PM
Unknown Object (File)
Mon, Jan 20, 11:00 AM
Unknown Object (File)
Sat, Jan 18, 2:26 AM
Unknown Object (File)
Sat, Jan 11, 9:47 PM
Unknown Object (File)
Tue, Jan 7, 3:09 PM
Unknown Object (File)
Tue, Jan 7, 2:55 PM
Subscribers

Details

Summary

Apply various linter fixes.

Test Plan

Eyeballed it.

Diff Detail

Repository
rJX Javelin
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Apply linter fixes.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

The Diviner stuff is very old code for the very old Diviner. I think it's OK to leave it for now, since it hasn't had a replacement written and can still technically run with old Diviner.

This revision is now accepted and ready to land.Jun 10 2014, 5:59 PM
epriestley updated this revision to Diff 22571.

Closed by commit rJXbdb2af92d8c4 (authored by @joshuaspence, committed by @epriestley).