Page MenuHomePhabricator
Differential D13994

Minor tidying of parser_nodes script
ClosedPublic
Actions

Authored by joshuaspence on Aug 26 2015, 8:02 AM.
Tags
None
Referenced Files
F18250333: D13994.id.diff
Thu, Aug 21, 3:55 PM
F18236904: D13994.diff
Wed, Aug 20, 8:37 PM
F17735546: D13994.id33818.diff
Jul 20 2025, 2:35 PM
F17702428: D13994.diff
Jul 15 2025, 10:58 PM
Unknown Object (File)
Jun 17 2025, 9:26 PM
Unknown Object (File)
Jun 17 2025, 3:21 AM
Unknown Object (File)
Jun 16 2025, 8:23 PM
Unknown Object (File)
May 28 2025, 2:17 AM
View All Files
Subscribers
epriestley

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPHUa00d404cb723: Minor tidying of parser_nodes script
Summary

Various minor improvements to parser_nodes.php, including:

  • Use libphutil functions instead of vanilla PHP.
  • phtize strings.
  • Use an array instead of a heredoc for representing the nodes.
Test Plan

Ran the script and compared ouptut.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 33818.Aug 26 2015, 8:02 AM
joshuaspence retitled this revision from to Minor tidying of parser_nodes script.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 26 2015, 8:02 AM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley requested changes to this revision.Aug 27 2015, 11:22 AM
epriestley added a reviewer: epriestley.

Seems basically fine but WIP.

This revision now requires changes to proceed.Aug 27 2015, 11:22 AM
joshuaspence updated this object.Aug 27 2015, 12:11 PM
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited edge metadata.
joshuaspence added inline comments.
support/xhpast/generate_nodes.php
1

This is wrong.

joshuaspence requested a review of this revision.Aug 27 2015, 12:53 PM
joshuaspence edited edge metadata.

It's good to go.

epriestley accepted this revision.Aug 27 2015, 2:50 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 27 2015, 2:50 PM
Closed by commit rPHUa00d404cb723: Minor tidying of parser_nodes script (authored by joshuaspence, committed by joshuaspence). · Explain WhyAug 27 2015, 9:03 PM
This revision was automatically updated to reflect the committed changes.
• cyn0228.ch added a task: T5375: Tokenizers fail in Firefox on Android.Sep 7 2015, 10:42 PM
chad removed a task: T5375: Tokenizers fail in Firefox on Android.Sep 7 2015, 10:45 PM

Revision Contents
Changeset List

PathSize
support/
xhpast/
285 lines
6 lines

Diff 33839

View Options

support/xhpast/generate_nodes.php

Loading...
View Options

support/xhpast/parser_nodes.php

Loading...
Phabricator · Built by Phacility