hi

Dan Harkless wrote:
> 
> "[EMAIL PROTECTED]" <[EMAIL PROTECTED]> writes:
> > Back in wget 1.5.3 I used to run the wget-new-percentage patch with much
> > delight, and I felt a bit sadden when I forced myself to at last remove
> > 1.5.3 in favour for 1.6 that still didn't have this highly useful feature.
> >
> > now, wget1.7-dev got a variant of this feature, however there is still
> > some work to be done I believe:
> >
> > ==> RETR xv-3.10a-13.i386.rpm ... done.
> > Length: 1,083,219 [820,419 to go] (unauthoritative)
> >
> >           [ skipping 250K ]
> >   250K ,,,,,,.... .......... .......... .......... .......... 28% @-305780.35 B/s
> >   300K .......... .......... .......... .......... .......... 33% @ 195.31 KB/s
> >
> > -305780.35B/s ?
> >
> > Since it is a dev-release this sort of thing is to expect of course, but
> > let's compare to the new-percentage-patch (I'll just copy from your url
> > since I don't have the time to patch an old wget :/ )
> >
> > HTTP request sent, awaiting response... 200 OK
> > Length: 235,173 [application/x-tar]
> >
> >     0K .......... .......... .......... .......... .......... 21%   0:00 7.31M
> >    50K .......... .......... .......... .......... .......... 43%   0:00 8.53M
> >
> > The output isn't exactly identical - for one you don't get the ugly "@"
> > after each line (yes I call it ugly, using "@" as an abbreviation for "at"
> > is not only wrong,
> 
> How is it wrong?  "@" is used as an abbreviation for "at" all the time.
> 
> > it doesn't look very good visually in my opinion) -
> 
> Personally I'd rather optimize for display clarity than display
> attractiveness, but it's true that with the "/s" on there, the "@" isn't
> strictly necessary to make it clear the figure is a bandwidth measurement.
> I think new-percentage-patch's output may be too terse, though.  Easy for
> all those unlabeled numbers to confuse non-experts.

  `Too terse' is probably correct but actually the only thing missing
  in the output is `minutes ETA' (or something like this). Everyone
  can check briefly the man page and/or will figure it out easy I think. 
  And finally we're have a 80-columns that should keep in -- I cannot
  afford to wrap the line or change the default 50K dot line... :)

> 
> I think a bigger problem is how the current display can go over 80 columns,
> which I think is much worse than is clutter due to " @ " (though removing
> that would help the line wrap problem as well, of course).

  Wget-new-percentage does not wrap lines and still reports info
  very good -- ETA in (6 chars) and speed (5 chars)...
  The source is so simple and short and finally works as it is
  expected (quite close to the current implementation in the CVS)
  so I cannot see why not include it? It worked with the CVS version
  from a week ago (well perhaps a bit more) and with 1.6.
  
  Anyway I will fix it to support latest wget and distribute it 
  alone if not accepted for the official wget release...

> 
> ---------------------------------------------------------------
> Dan Harkless            | To help prevent SPAM contamination,
> GNU Wget co-maintainer  | please do not mention this email
> http://sunsite.dk/wget/ | address in Usenet posts -- thank you.

P! Vladi.
-- 
Vladi Belperchinov-Shabanski <[EMAIL PROTECTED]> <[EMAIL PROTECTED]>
Personal home page at http://www.biscom.net/~cade
DataMax Ltd. http://www.datamax.bg
No tears to cry, no feelings left...

Reply via email to