Paths

Table of Contentst

Differential D9363

Don't redraw the console progress bar every time
ClosedPublic
Actions

Authored by epriestley on Jun 3 2014, 6:20 PM.

Tags

None

Referenced Files

	F14014479: D9363.id22297.diff
	Sun, Nov 3, 5:02 AM

	F14006262: D9363.id22297.diff
	Mon, Oct 28, 4:30 AM

	F13983212: D9363.id.diff
	Sun, Oct 20, 4:51 AM

	F13961838: D9363.id22294.diff
	Tue, Oct 15, 6:36 AM

	Unknown Object (File)
	Mon, Oct 7, 11:00 AM

	Unknown Object (File)
	Sep 20 2024, 10:58 PM

	Unknown Object (File)
	Sep 5 2024, 5:58 PM

	Unknown Object (File)
	Sep 4 2024, 1:05 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T5255: Batch up SQL operations in the `./bin/repository parents` script

Commits

rPHU1e025a87a4cb: Don't redraw the console progress bar every time

Summary

lulululuul see D9361

Test Plan

derp derp derp

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 22294.Jun 3 2014, 6:20 PM

epriestley retitled this revision from to Don't redraw the console progress bar every time.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: joshuaspence.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 3 2014, 6:20 PM

Only set lastUpdate if we actually draw.

Harbormaster completed remote builds in B829: Diff 22294.Jun 3 2014, 6:21 PM

Harbormaster completed remote builds in B830: Diff 22295.

Slightly smoother bar (100ms / update instead of 200ms).

Harbormaster completed remote builds in B831: Diff 22296.Jun 3 2014, 6:23 PM

LGTM.

This makes the progress bar a little jerky, but I'd rather the progress bar be jerky than have the ./bin/repository parents script run 127 times slower.

This revision is now accepted and ready to land.Jun 3 2014, 6:24 PM

Closed by commit rPHU1e025a87a4cb (authored by @epriestley).

Thanks for the fix

epriestley mentioned this in D19506: Add --quiet flag to rebuild-identities.Aug 2 2018, 4:14 PM

Revision Contents
Changeset List

Path

Size

src/

console/

PhutilConsoleProgressBar.php

8 lines

Diff 22297

src/console/PhutilConsoleProgressBar.php

Loading...