bug-wget
[Top][All Lists]
Advanced

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

Re: [Bug-wget] --progress should not be overridden by --quiet


From: Gisle Vanem
Subject: Re: [Bug-wget] --progress should not be overridden by --quiet
Date: Thu, 17 Apr 2014 14:49:59 +0200

"Darshit Shah" <address@hidden> wrote:

progress bar to me. It does not change the existing output so we have
full backward compatibility with existing scripts, but allows the user
to explicitly display the progress bar if required.

I don't see your patch is considering the Windows console-title
progress indicator. In e.g. retr.c:

#ifdef WINDOWS
     if (toread > 0 && !opt.quiet)
       ws_percenttitle (100.0 *
                        (startpos + sum_read) / (startpos + toread));
#endif

Maybe this should test for this new option too? Something like:
     if (toread > 0 && !opt.quiet || opt.show_progress)
    ...

But mswindows.c always needs a 'curr_url' to show percentage.
I don't think 'ws_changetitle()' will be called as needed for this to
happen. The best IMHO would be to drop 'ws_changetitle()' and always have the current-URL as a parameter to 'ws_percenttitle()'.

--gv



reply via email to

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