Page MenuHomePhabricator

Update xhpast linter rules for new function AST format
ClosedPublic

Authored by richardvanvelzen on Apr 28 2016, 3:03 PM.
Tags
None
Referenced Files
F19823832: D15814.diff
Mar 7 2026, 3:44 PM
F19822346: D15814.diff
Mar 7 2026, 4:49 AM
F19721185: D15814.id38095.diff
Feb 12 2026, 10:50 PM
F19615029: D15814.diff
Feb 3 2026, 11:15 PM
F19571745: D15814.diff
Feb 1 2026, 2:44 AM
F19553645: D15814.id38095.diff
Jan 29 2026, 1:51 PM
F19520353: D15814.id.diff
Jan 16 2026, 1:12 PM
F19517752: D15814.diff
Jan 15 2026, 3:04 PM
Subscribers

Details

Summary

See D15678. A node containing a return type hint is added right before the statement body node. This updates all relevant linter rules to now retrieve the body from the correct index.

Test Plan

Ran arc unit --everything, inspected every single message to make sure all xhpast test cases succeeded. Also grepped for getChildByIndex(5 and getChildOfType(5.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

richardvanvelzen retitled this revision from to Update xhpast linter rules for new function AST format.
richardvanvelzen updated this object.
richardvanvelzen edited the test plan for this revision. (Show Details)
richardvanvelzen added a reviewer: epriestley.
epriestley edited edge metadata.

Those test cases are unrelated version issues.

This revision is now accepted and ready to land.Apr 29 2016, 12:24 PM
This revision was automatically updated to reflect the committed changes.