Page MenuHomePhabricator

Removed useless comments.
ClosedPublic

Authored by joshuaspence on Mar 2 2014, 3:10 AM.
Tags
None
Referenced Files
F13972271: D8382.diff
Thu, Oct 17, 6:05 PM
F13972263: D8382.id.diff
Thu, Oct 17, 6:02 PM
F13972257: D8382.id19918.diff
Thu, Oct 17, 6:00 PM
F13972251: D8382.id19932.diff
Thu, Oct 17, 5:58 PM
F13965754: D8382.diff
Wed, Oct 16, 4:07 AM
Unknown Object (File)
Sep 20 2024, 3:11 PM
Unknown Object (File)
Sep 14 2024, 11:53 PM
Unknown Object (File)
Sep 14 2024, 11:52 PM

Details

Summary

To me, it seems that these comments add no value. Personal opinion I suppose.

Test Plan

N/A

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley edited edge metadata.

I might have given some feedback like "document the options this linter takes", and whoever was contributing the patch did this, and then I missed it on accept. Or I might have just derped this up.

Since I want to deprecate .arcconfig options anyway, removing these fragments seems much cleaner to me. I agree that they're currently pretty useless. Generally, I'd like to push documentation aggressively toward being more terse, particularly after T988, which generally allows documentation to be inherited.

This revision is now accepted and ready to land.Mar 4 2014, 6:52 PM