Tags

None

Referenced Files

	F14705164: D13858.id33482.diff
	Thu, Jan 16, 10:30 AM

	Unknown Object (File)
	Sun, Jan 12, 11:46 PM

	Unknown Object (File)
	Sun, Jan 12, 12:27 AM

	Unknown Object (File)
	Wed, Jan 1, 1:24 AM

	Unknown Object (File)
	Sun, Dec 29, 4:48 AM

	Unknown Object (File)
	Sun, Dec 29, 4:14 AM

	Unknown Object (File)
	Sat, Dec 28, 4:38 PM

	Unknown Object (File)
	Sat, Dec 28, 2:46 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