Page MenuHomePhabricator

Fix more PHP 8.1 "strlen(null)" callsites
Needs ReviewPublic

Authored by vm on Jun 7 2022, 7:48 PM.
Tags
None
Referenced Files
F18998612: D21853.id.diff
Nov 20 2025, 7:13 PM
F18850640: D21853.diff
Oct 30 2025, 6:24 PM
F18808408: D21853.id.diff
Oct 19 2025, 6:22 AM
F18803235: D21853.id52085.diff
Oct 17 2025, 10:17 PM
F18792615: D21853.diff
Oct 16 2025, 7:57 PM
F18782235: D21853.id52085.diff
Oct 13 2025, 1:00 AM
F18647568: D21853.diff
Sep 19 2025, 1:30 PM
F18632323: D21853.id.diff
Sep 16 2025, 5:35 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T13588: PHP 8 Compatibility
Summary

Noticed some failures with a custom UnitEngine after upgrading to
PHP 8.1
Was able to work around it by setting empty strings but this should fix the
underlying problem.

Test Plan

arc unit no longer fails with no namespace set on UnitTestResult

Diff Detail

Repository
rARC Arcanist
Branch
fix_8.1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25761
Build 35589: arc lint + arc unit