Page MenuHomePhabricator

Improve parsing of `use` statements
ClosedPublic

Authored by joshuaspence on Nov 19 2015, 8:50 AM.
Tags
None
Referenced Files
F15580026: D14518.id35120.diff
Wed, May 7, 3:01 AM
F15543735: D14518.id.diff
Sat, Apr 26, 12:43 AM
F15538980: D14518.id.diff
Thu, Apr 24, 10:43 PM
F15538979: D14518.diff
Thu, Apr 24, 10:43 PM
F15528835: D14518.id35120.diff
Tue, Apr 22, 2:18 PM
F15526485: D14518.id.diff
Mon, Apr 21, 9:08 PM
F15526484: D14518.diff
Mon, Apr 21, 9:08 PM
F15526239: D14518.id35128.diff
Mon, Apr 21, 8:13 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.