Page MenuHomePhabricator
Differential D21173

Accommodate PHP 7.4 changes to certain "preg_match_all()" calls
ClosedPublic
Actions

Authored by epriestley on Apr 26 2020, 3:32 PM.
Tags
None
Referenced Files
F18999553: D21173.diff
Thu, Nov 20, 9:23 PM
F18985458: D21173.id50414.diff
Mon, Nov 17, 1:03 PM
F18842996: D21173.id50414.diff
Tue, Oct 28, 8:21 PM
F18838193: D21173.id.diff
Mon, Oct 27, 9:37 AM
F18831143: D21173.id50415.diff
Sat, Oct 25, 11:13 AM
F18822314: D21173.diff
Thu, Oct 23, 5:49 AM
F18760038: D21173.id50414.diff
Oct 6 2025, 7:40 AM
F18751781: D21173.diff
Oct 4 2025, 12:38 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13518: PHP 7.4 "preg_match_all()" + "PREG_OFFSET_CAPTURE" behavior has changed
Commits
rARC5efbe22c2afa: (stable) Accommodate PHP 7.4 changes to certain "preg_match_all()" calls
rARCb81818b287db: Accommodate PHP 7.4 changes to certain "preg_match_all()" calls
Summary

Ref T13518. The result format of this call changed in PHP 7.4, which causes us to emit "-1" matches because "-1" survives array_filter().

Filter results in a way that should survive both result formats.

Test Plan

Ran arc unit --everything under PHP 7.4.

Diff Detail

Repository
rARC Arcanist
Branch
php741
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24237
Build 33383: Run Core Tests
Build 33382: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 26 2020, 3:32 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Apr 26 2020, 3:32 PM
Harbormaster failed remote builds in B24233: Diff 50414!
epriestley updated this revision to Diff 50415.Apr 26 2020, 3:35 PM
  • Swap order to work better under PHP 7.3 and earlier.
Harbormaster returned this revision to the author for changes because remote builds failed.Apr 26 2020, 3:35 PM
Harbormaster failed remote builds in B24234: Diff 50415!
epriestley updated this revision to Diff 50418.Apr 26 2020, 3:38 PM
  • Actually look at the PHP 7.3x output before writing code for it.
  • Third try's the charm!
Harbormaster completed remote builds in B24237: Diff 50418.Apr 26 2020, 3:39 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 26 2020, 3:39 PM
epriestley requested review of this revision.
Closed by commit rARCb81818b287db: Accommodate PHP 7.4 changes to certain "preg_match_all()" calls (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rARCb81818b287db: Accommodate PHP 7.4 changes to certain "preg_match_all()" calls.
epriestley added a commit: rARC5efbe22c2afa: (stable) Accommodate PHP 7.4 changes to certain "preg_match_all()" calls.

Revision Contents
Changeset List

PathSize
src/
parser/
xhpast/
api/
21 lines

Diff 50418

View Options

src/parser/xhpast/api/XHPASTNode.php

Loading...
Phabricator · Built by Phacility