Page MenuHomePhabricator

Improve parsing of `use` statements
ClosedPublic

Authored by joshuaspence on Nov 19 2015, 8:50 AM.
Tags
None
Referenced Files
F19896287: D14518.id35128.diff
Sun, Mar 29, 2:42 AM
F19896286: D14518.id35128.diff
Sun, Mar 29, 2:41 AM
F19876595: D14518.id35128.diff
Mon, Mar 16, 12:09 PM
F19810553: D14518.id35120.diff
Mar 3 2026, 8:02 PM
F19706947: D14518.id.diff
Feb 11 2026, 10:17 AM
F19684271: D14518.diff
Feb 9 2026, 3:41 AM
F19684269: D14518.diff
Feb 9 2026, 3:36 AM
F19627224: D14518.diff
Feb 4 2026, 9:00 PM
Subscribers

Details

Summary

The way that we parse use statements at the moment is a bit odd. For example, consider the following example:

<?php

use A\B\C;
use X\Y\Z;

This is currently parsed as:

xhpast-before.png (336×182 px, 8 KB)

After this change, it is instead parsed as:

xhpast-after.png (279×166 px, 7 KB)

Test Plan

Added a test case.

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8930
Build 10474: Run Core Tests
Build 10473: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Improve parsing on `use` statements.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 19 2015, 4:34 PM
joshuaspence retitled this revision from Improve parsing on `use` statements to Improve parsing of `use` statements.Nov 19 2015, 7:47 PM
joshuaspence edited edge metadata.
This revision was automatically updated to reflect the committed changes.