bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] [Bug-Wget][Patch] Implement --show-progress


From: Darshit Shah
Subject: Re: [Bug-wget] [Bug-Wget][Patch] Implement --show-progress
Date: Thu, 1 May 2014 23:14:18 +0200

Hello,

As Gisle pointed out in the other thread, the progress implementation had a
few flaws. I've mostly ironed them all out in this patch. This is a
slightly enhanced version of the patch I sent on the other thread related
to progress bar implementation.

In a gist, this patch:
1. Cleans up output in case of dot-style progress bar by not printing extra
newlines when not needed
2. Ensures all the columns are perfectly aligned in all (?) cases
3. Makes the filename column fixed width. This makes a much better image
when downloading multiple files in quiet mode.

Please do test this patch and I welcome any and all comments on it.


On Wed, Apr 30, 2014 at 9:39 PM, Giuseppe Scrivano <address@hidden>wrote:

> Darshit Shah <address@hidden> writes:
>
> > No, I haven't tried porting this code into parallel-wget yet. We'd
> > have to edit all the calls to fd_read_body() which I'm assuming have
> > shifted by a large amount so we'll have to manually merge it. This set
> > of patches is for master only.
>
> feel free to push your patch whenever you want :)
>
> Giuseppe
>



-- 
Thanking You,
Darshit Shah

Attachment: 0001-Aesthetic-changes-and-bug-fixes-for-progress-bar.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]