Personally, I'd rather see it reported as a percentage of the "real" total file size, rather than just the percentage of the remaining bytes left to download. E.g., as David first described (where the % would start at something other than zero if a file is already partially downloaded).

I think this is more consistent with other download tools also, in the case where curl is automatically resuming a download partway through.

But, in the case where someone specifically requests a partial file download, it makes more sense to still show the % of the current request...

For what it's worth.....

- Kevin

-----Original Message-----
From: David Byron [mailto:DByron_at_everdreamcorp.com]
Sent: Monday, May 19, 2003 1:47 PM
To: 'curl-users_at_lists.sourceforge.net'
Subject: RE: bug/feature in progress?

David Byron wrote:

> $ ./curl http://www.gnu.org/manual/emacs-21.2/ps/emacs.ps.gz -O
--limit-rate
> 10k --continue-at -

> % Total % Received % Xferd Average Speed Time
Curr.
> Dload Upload Total Current Left
Speed
> 6 1005k 6 66058 0 0 9949 0 0:01:43 0:00:06 0:01:36
10223
>
> When I execute the download command again, both the % total
> and % received numbers start back from 0 again, even though
> the file is partially there. The total number of bytes seems
> to start at a non-zero number, but that one seems too
> big....Anyway, I'm more concerned about the percentage for now.

Replying to myself, I can now see that the total to download gets smaller
with each invocation. From that perspective, the % of total, and total
number of bytes are correct, as are the received numbers.

> I'll try this with 7.10.5 in the meantime, but the release
> notes don't make it sound like anything like this has
> changed.

It's the same with 7.10.5.

So, I guess I can pass the total size in the void * that gets passed to the
progress function. Are other people interested in having the progress
information reported like this? If so, it looks like calling
Curl_pgrsSetDownloadSize() gets more complicated than it currently is in
transfer.c (line 562 of rev 1.150). Would you consider a patch that adds a
CURLOption to support this? Any pointers to the questions I asked in my
previous post?

All the best,

-DB

--
David Byron                   dbyron_at_everdream.com
Everdream                     http://www.everdream.com
6591 Dumbarton Circle         voice:(510)818-5550
Fremont, CA  94555            fax:(510)818-5510
-------------------------------------------------------
This SF.net email is sponsored by: If flattening out C++ or Java
code to make your application fit in a relational database is painful, 
don't do it! Check out ObjectStore. Now part of Progress Software.
http://www.objectstore.net/sourceforge
-------------------------------------------------------
This SF.net email is sponsored by: If flattening out C++ or Java
code to make your application fit in a relational database is painful, 
don't do it! Check out ObjectStore. Now part of Progress Software.
http://www.objectstore.net/sourceforge