Annotate the unusual use of "$callback()" in "xsprintf()"
ClosedPublic
Actions

Authored by epriestley on Feb 3 2021, 10:19 PM.

Tags

None

Referenced Files

	F15429181: D21541.id.diff
	Mon, Mar 24, 1:03 AM

	F15424447: D21541.id51275.diff
	Sat, Mar 22, 10:28 PM

	F15383430: D21541.diff
	Fri, Mar 14, 4:41 PM

	F15378126: D21541.diff
	Thu, Mar 13, 1:00 PM

	Unknown Object (File)
	Feb 22 2025, 7:38 AM

	Unknown Object (File)
	Feb 14 2025, 1:52 AM

	Unknown Object (File)
	Feb 10 2025, 7:56 PM

	Unknown Object (File)
	Feb 7 2025, 6:31 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13588: PHP 8 Compatibility
Commits: rARCc1afa91f9ff8: Annotate the unusual use of "$callback()" in "xsprintf()"

Summary

Ref T13588. See D21500. This syntax is unusual and there are some hidden complexities involved; annotate them. See D21500 for more discussion.

Test Plan

Read text, reviewed D21500.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 3 2021, 10:19 PM

Harbormaster completed remote builds in B25089: Diff 51275.Feb 3 2021, 10:20 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 3 2021, 10:20 PM

epriestley requested review of this revision.

Closed by commit rARCc1afa91f9ff8: Annotate the unusual use of "$callback()" in "xsprintf()" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCc1afa91f9ff8: Annotate the unusual use of "$callback()" in "xsprintf()".

epriestley mentioned this in T13588: PHP 8 Compatibility.Feb 3 2021, 10:52 PM

Revision Contents
Changeset List

Path

Size

src/

xsprintf/

xsprintf.php

12 lines

Diff 51276

View Options

Annotate the unusual use of "$callback()" in "xsprintf()"ClosedPublicActions