Tags

None

Referenced Files

	F15418902: D13858.id33482.diff
	Fri, Mar 21, 2:15 AM

	F15408686: D13858.id33484.diff
	Wed, Mar 19, 12:49 AM

	F15400059: D13858.id33485.diff
	Mon, Mar 17, 9:25 AM

	F15380573: D13858.id33473.diff
	Fri, Mar 14, 3:40 AM

	F15338365: D13858.id33474.diff
	Sun, Mar 9, 10:39 AM

	F15312317: D13858.id33484.diff
	Thu, Mar 6, 2:53 PM

	F15312316: D13858.id33482.diff
	Thu, Mar 6, 2:53 PM

	F15312314: D13858.id33473.diff
	Thu, Mar 6, 2:53 PM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHUd052c489a283: Include parentheses in n_ARRAY_VALUE_LIST node

Summary

Currently parsing array(1, 2, 3) results in a n_ARRAY_VALUE_LIST with a concrete string of 1, 2, 3. Compare this to list($x, $y) which results in a n_ASSIGNMENT_LIST with a concrete string of ($x, $y).

Test Plan

See next diff.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 33473.Aug 11 2015, 10:54 AM

joshuaspence retitled this revision from to Include parentheses in n_ARRAY_VALUE_LIST node.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 11 2015, 10:54 AM

Herald added a subscriber: Korvin. · View Herald Transcript

Fix for list($i, list($j, $k)) = y()

joshuaspence added a child revision: D13859: Improve linter rules for array formatting.Aug 11 2015, 11:42 AM

Also bump version?

This revision is now accepted and ready to land.Aug 11 2015, 12:28 PM

Update tests

Bump XHPAST version

Closed by commit rPHUd052c489a283: Include parentheses in n_ARRAY_VALUE_LIST node (authored by joshuaspence, committed by joshuaspence). · Explain WhyAug 11 2015, 12:34 PM

This revision was automatically updated to reflect the committed changes.

Include parentheses in n_ARRAY_VALUE_LIST node
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 33485

src/parser/xhpast/tests/data/php-array-dereference.expect

src/parser/xhpast/tests/data/php-literal-index.php.expect

src/parser/xhpast/tests/data/php-short-array.php.expect

src/parser/xhpast/tests/data/php-syntax.php.expect

src/parser/xhpast/bin/PhutilXHPASTBinary.php

support/xhpast/parser.y

support/xhpast/parser.yacc.cpp

support/xhpast/xhpast.cpp

Include parentheses in n_ARRAY_VALUE_LIST nodeClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 33485

src/parser/xhpast/__tests__/data/php-array-dereference.expect

src/parser/xhpast/__tests__/data/php-literal-index.php.expect

src/parser/xhpast/__tests__/data/php-short-array.php.expect

src/parser/xhpast/__tests__/data/php-syntax.php.expect

src/parser/xhpast/bin/PhutilXHPASTBinary.php

support/xhpast/parser.y

support/xhpast/parser.yacc.cpp

support/xhpast/xhpast.cpp

Include parentheses in n_ARRAY_VALUE_LIST node
ClosedPublic
Actions

Revision Contents
Changeset List

src/parser/xhpast/tests/data/php-array-dereference.expect

src/parser/xhpast/tests/data/php-literal-index.php.expect

src/parser/xhpast/tests/data/php-short-array.php.expect

src/parser/xhpast/tests/data/php-syntax.php.expect