Page MenuHomePhabricator
Differential D12292

Mark some AAST methods as final
ClosedPublic
Actions

Authored by joshuaspence on Apr 5 2015, 10:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 17, 1:37 AM
Unknown Object (File)
Sat, Jun 6, 9:54 PM
Unknown Object (File)
Sat, Jun 6, 8:18 AM
Unknown Object (File)
Sat, Jun 6, 2:44 AM
Unknown Object (File)
Wed, Jun 3, 11:16 PM
Unknown Object (File)
Jun 2 2026, 12:01 AM
Unknown Object (File)
Mar 29 2026, 10:33 AM
Unknown Object (File)
Feb 15 2026, 9:03 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPHU37119b5a2140: Mark some AAST methods as final
Summary

Self-explanatory. I don't think any subclasses should reasonably override these methods.

Test Plan

N/A

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 29495.Apr 5 2015, 10:23 AM
joshuaspence retitled this revision from to Mark some AAST methods as final.
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 TranscriptApr 5 2015, 10:23 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.Apr 5 2015, 12:15 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/parser/xhpast/api/XHPASTNode.php
233–235

This has no callsites?

This revision is now accepted and ready to land.Apr 5 2015, 12:15 PM
joshuaspence added inline comments.Apr 5 2015, 12:16 PM
src/parser/xhpast/api/XHPASTNode.php
233–235

This method is identical to the implementation in AASTNode.

joshuaspence updated this revision to Diff 29521.Apr 5 2015, 12:44 PM
joshuaspence edited edge metadata.

Rebase

Closed by commit rPHU37119b5a2140: Mark some AAST methods as final (authored by joshuaspence, committed by joshuaspence). · Explain WhyApr 5 2015, 12:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
parser/
aast/
api/
22 lines
8 lines
20 lines
xhpast/
api/
4 lines

Diff 29522

View Options

src/parser/aast/api/AASTNode.php

Loading...
View Options

src/parser/aast/api/AASTToken.php

Loading...
View Options

src/parser/aast/api/AASTTree.php

Loading...
View Options

src/parser/xhpast/api/XHPASTNode.php

Loading...
Phabricator · Built by Phacility