Page MenuHomePhabricator

Support class const dereferencing in xhpast
ClosedPublic

Authored by richardvanvelzen on Jun 17 2016, 9:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 8:34 AM
Unknown Object (File)
Sun, Mar 24, 9:51 AM
Unknown Object (File)
Sat, Mar 23, 3:10 PM
Unknown Object (File)
Wed, Mar 20, 6:06 PM
Unknown Object (File)
Mon, Mar 18, 6:08 PM
Unknown Object (File)
Mon, Mar 18, 4:31 PM
Unknown Object (File)
Sun, Mar 10, 9:47 AM
Unknown Object (File)
Tue, Mar 5, 8:00 AM
Subscribers

Details

Summary

Ref T4334. As it turns out, a major overhaul of the syntax wasn't needed.

This adds support for stuff like SomeClass::SOME_CONST[$idx] which was introduced in PHP 7.

Test Plan

Added a test case. Ran tests on libphutil and arcanist with both PHP 5 and PHP 7.

Diff Detail

Repository
rPHU libphutil
Branch
const-deref
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12701
Build 16150: arc lint + arc unit

Event Timeline

richardvanvelzen retitled this revision from to Support class const dereferencing in xhpast.
richardvanvelzen updated this object.
richardvanvelzen edited the test plan for this revision. (Show Details)
richardvanvelzen added a reviewer: epriestley.
This revision is now accepted and ready to land.Jun 17 2016, 11:43 AM
This revision was automatically updated to reflect the committed changes.