Paths

Table of Contentst

Remove unused class "PhutilConsoleConcatenatedView"
ClosedPublic
Actions

Authored by epriestley on Feb 16 2017, 12:47 PM.

Tags

None

Referenced Files

	F15534981: D17365.id41756.diff
	Thu, Apr 24, 2:53 AM

	F15517702: D17365.id.diff
	Sat, Apr 19, 7:53 AM

	F15516745: D17365.diff
	Fri, Apr 18, 9:01 PM

	F15493066: D17365.id41757.diff
	Sat, Apr 12, 9:17 PM

	F15492398: D17365.id41756.diff
	Sat, Apr 12, 2:02 PM

	F15469090: D17365.id41756.diff
	Fri, Apr 4, 8:51 AM

	F15442679: D17365.diff
	Mar 26 2025, 11:57 PM

	F15436893: D17365.id41759.diff
	Mar 25 2025, 4:44 PM

View All Files

Subscribers

None

Details

Reviewers

vrana
chad

Commits

rPHUe78b0df10881: Remove unused class "PhutilConsoleConcatenatedView"

Summary

See D14136. This class is never used anywhere and wouldn't work anyway.

I believe the thing it was intended to do (merging views into a single string) ended up in the base class, PhutilConsoleView.

Test Plan

Grepped for PhutilConsoleConcatenatedView in libphutil, Arcanist, and Phabricator.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Also arc liberate.

Harbormaster failed remote builds in B15652: Diff 41756!Feb 16 2017, 12:48 PM

vrana accepted this revision.Feb 16 2017, 12:48 PM

This revision is now accepted and ready to land.Feb 16 2017, 12:48 PM

Harbormaster completed remote builds in B15653: Diff 41757.Feb 16 2017, 12:49 PM

Closed by commit rPHUe78b0df10881: Remove unused class "PhutilConsoleConcatenatedView" (authored by epriestley, committed by epriestley). · Explain WhyFeb 16 2017, 12:50 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

console/

view/

PhutilConsoleConcatenatedView.php

Diff 41759

View Options

src/__phutil_library_map__.php