[patch] allow seeking in linear files
Pavel Tsekov
ptsekov at gmx.net
Tue Jan 24 14:12:25 UTC 2006
Hello Jindrich,
On Tue, 24 Jan 2006, Jindrich Makovicka wrote:
> On 1/23/06, Pavel Tsekov <ptsekov at gmx.net> wrote:
> >
> > So now to the patch - first I'd like to comment the last hunk of your patch:
> >
> > > - n_read_total, ctx->progress_bytes);
> > > + n_read_total + ctx->do_reget, ctx->progress_bytes);
> >
> > > - file_progress_show (ctx, n_read_total, file_size);
> > > + file_progress_show (ctx, n_read_total + ctx->do_reget,
> > > file_size);
> >
> > I think it would be better if you substract 'ctx->do_reget' from the last
> > argument (the total) instead of adding it to the second argument (the number
> > of bytes read so far). This way the user won't see the progress bar
> > semi-filled right after the start of the copy operation . What do you think
> > ?
>
> I chose this variant intentionally, as it is common eg. in the
> download managers - the user can immediately see that the download
> operation actually resumes, and it does not start from the very
> beginning.
I've just commited this part of the patch.
More information about the mc-devel
mailing list