Page MenuHomePhabricator

Improve atomizing of PHP `@return` annotation
ClosedPublic

Authored by joshuaspence on Jan 22 2015, 8:04 PM.
Tags
None
Referenced Files
F13106574: D11469.diff
Sat, Apr 27, 8:35 PM
Unknown Object (File)
Thu, Apr 25, 3:55 AM
Unknown Object (File)
Wed, Apr 17, 12:17 PM
Unknown Object (File)
Fri, Mar 29, 6:35 AM
Unknown Object (File)
Mar 28 2024, 9:12 PM
Unknown Object (File)
Mar 28 2024, 9:12 PM
Unknown Object (File)
Mar 28 2024, 8:50 PM
Unknown Object (File)
Mar 25 2024, 11:37 AM
Subscribers

Details

Summary

Fixes T6890. This doesn't feel like a perfect solution, but I can't think of any cases in which this will produce the wrong result either.

Test Plan

Ran ./bin/diviner generate and checked the generated documentation for PhabricatorCommonPasswords::loadWordlist(). The return type was corrected shown as map<string, bool>.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3988
Build 4001: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Improve atomizing of PHP `@return` annotation.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

Yeah, this was my thought too. Feels sketchy, but I also couldn't come up with any cases where it produces the wrong behavior.

This revision is now accepted and ready to land.Jan 22 2015, 8:17 PM
This revision was automatically updated to reflect the committed changes.