Paths

Table of Contentst

Declare tokenCache on AASTNode
ClosedPublic
Actions

Authored by epriestley on Feb 27 2016, 3:25 PM.

Tags

None

Referenced Files

	F13236125: D15358.id37035.diff
	Tue, May 21, 8:16 AM

	F13216853: D15358.id37034.diff
	Sat, May 18, 2:39 AM

	F13196573: D15358.diff
	Sun, May 12, 11:23 PM

	F13192092: D15358.diff
	Sun, May 12, 5:08 AM

	Unknown Object (File)
	Fri, May 3, 5:41 AM

	Unknown Object (File)
	Sat, Apr 27, 2:37 PM

	Unknown Object (File)
	Thu, Apr 25, 1:20 AM

	Unknown Object (File)
	Mon, Apr 22, 2:23 AM

View All Files

Subscribers

None

Details

Reviewers

joshuaspence
chad

Commits

rPHU6ef9295e388d: Declare tokenCache on AASTNode

Summary

See discussion in T10462. Because this class extends Phobject, calling $this->undeclaredProperty = $value; throws.

Declare $tokenCache.

Test Plan

Added debugging statements, saw change cause cache hits to occur.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37034.Feb 27 2016, 3:25 PM

epriestley retitled this revision from to Declare tokenCache on AASTNode.

epriestley updated this object.

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

epriestley added reviewers: chad, joshuaspence.

epriestley mentioned this in T10462: AASTNode in libphutil missing tokenCache declaration.Feb 27 2016, 3:27 PM

chad accepted this revision.Feb 27 2016, 3:29 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Feb 27 2016, 3:29 PM

Closed by commit rPHU6ef9295e388d: Declare tokenCache on AASTNode (authored by epriestley, committed by epriestley). · Explain WhyFeb 27 2016, 4:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

parser/

aast/

api/

AASTNode.php

1 line

Diff 37035

View Options

src/parser/aast/api/AASTNode.php

Declare tokenCache on AASTNodeClosedPublicActions