Paths

Table of Contentst

Remove unused class "PhutilConsoleConcatenatedView"
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Dec 30, 5:44 AM

	Unknown Object (File)
	Tue, Dec 17, 7:22 AM

	Unknown Object (File)
	Thu, Dec 12, 5:14 AM

	Unknown Object (File)
	Wed, Dec 11, 8:06 AM

	Unknown Object (File)
	Sat, Dec 7, 2:11 AM

	Unknown Object (File)
	Fri, Dec 6, 6:38 PM

	Unknown Object (File)
	Fri, Dec 6, 5:43 PM

	Unknown Object (File)
	Dec 2 2024, 10:17 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