[haiku-commits] haiku: hrev48171 - src/bin/pkgman
- From: pulkomandy@xxxxxxxxxxxxx
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Thu, 30 Oct 2014 23:21:06 +0100 (CET)
hrev48171 adds 1 changeset to branch 'master'
old head: a5ce4678c054489520b8e2a7c0910da48a5d1521
new head: 87d0590f6091f16a32dfa672888f050de2e0266b
overview: http://cgit.haiku-os.org/haiku/log/?qt=range&q=87d0590+%5Ea5ce467
----------------------------------------------------------------------------
87d0590: pkgman: better download progress bar
* Use Unicode block drawing characters to provide sub-character accuracy
* Also display the download progress percentage as a number
When downloading big packages this helps seeing that the bar continues
moving and that the download is not stalled.
[ Adrien Destugues <pulkomandy@xxxxxxxxx> ]
----------------------------------------------------------------------------
Revision: hrev48171
Commit: 87d0590f6091f16a32dfa672888f050de2e0266b
URL: http://cgit.haiku-os.org/haiku/commit/?id=87d0590
Author: Adrien Destugues <pulkomandy@xxxxxxxxx>
Date: Thu Oct 30 22:17:26 2014 UTC
----------------------------------------------------------------------------
2 files changed, 34 insertions(+), 11 deletions(-)
src/bin/pkgman/PackageManager.cpp | 44 +++++++++++++++++++++++++++--------
src/bin/pkgman/PackageManager.h | 1 -
----------------------------------------------------------------------------
diff --git a/src/bin/pkgman/PackageManager.cpp
b/src/bin/pkgman/PackageManager.cpp
index 6a5e6ad..e9ecb1e 100644
--- a/src/bin/pkgman/PackageManager.cpp
+++ b/src/bin/pkgman/PackageManager.cpp
@@ -30,7 +30,6 @@ PackageManager::PackageManager(BPackageInstallationLocation
location,
BPackageManager::UserInteractionHandler(),
fDecisionProvider(interactive),
fClientInstallationInterface(),
- fPreviousDownloadPercentage(0),
fInteractive(interactive)
{
}
@@ -172,7 +171,6 @@ void
PackageManager::ProgressPackageDownloadStarted(const char* packageName)
{
printf("Downloading %s...\n", packageName);
- fPreviousDownloadPercentage = 0;
}
@@ -180,16 +178,42 @@ void
PackageManager::ProgressPackageDownloadActive(const char* packageName,
float completionPercentage)
{
- int32 currentPercentage = int32(completionPercentage * 100);
- int32 difference = currentPercentage - fPreviousDownloadPercentage;
-
- while (difference >= 2) {
- printf("#");
- difference -= 2;
+ static const char* progressChars[] = {
+ "\xE2\x96\x8F",
+ "\xE2\x96\x8E",
+ "\xE2\x96\x8D",
+ "\xE2\x96\x8C",
+ "\xE2\x96\x8B",
+ "\xE2\x96\x8A",
+ "\xE2\x96\x89",
+ "\xE2\x96\x88",
+ };
+
+ const int width = 70;
+
+ int position;
+ int ipart = (int)(completionPercentage * width);
+ int fpart = (int)(((completionPercentage * width) - ipart) * 8);
+
+ printf("\r"); // erase the line
+
+ for (position = 0; position < width; position++) {
+ if (position < ipart) {
+ // This part is fully downloaded, show a full block
+ printf(progressChars[7]);
+ } else if (position > ipart) {
+ // This part is not downloaded, show a space
+ printf(" ");
+ } else {
+ // This part is partially downloaded
+ printf(progressChars[fpart]);
+ }
}
- fflush(stdout);
- fPreviousDownloadPercentage = currentPercentage - difference;
+ // Also print the progress percentage
+ printf(" %d%%", (int)(completionPercentage * 100));
+
+ fflush(stdout);
}
diff --git a/src/bin/pkgman/PackageManager.h b/src/bin/pkgman/PackageManager.h
index 494912b..eb64aa4 100644
--- a/src/bin/pkgman/PackageManager.h
+++ b/src/bin/pkgman/PackageManager.h
@@ -70,7 +70,6 @@ private:
DecisionProvider fDecisionProvider;
BPackageManager::ClientInstallationInterface
fClientInstallationInterface;
- int32
fPreviousDownloadPercentage;
bool fInteractive;
};
Other related posts:
