[PATCH] progress: skip progress estimate when current exceeds total

timeless timeless at gmail.com
Sun Mar 13 09:13:57 CDT 2011


On Sun, Mar 13, 2011 at 3:50 PM, Augie Fackler <durin42 at gmail.com> wrote:
>> +        interval = pos - initialpos
>
> can we rename this to rate?

we can definitely rename it to something.

it's really 'current progress', there's no time yet. i guess
'displacement' is the right term.

>> +        delta = total - pos
>
> and this to remaining?

yes, that sounds better

> I think those renames would make the code clearer. I had a little trouble unpacking what was going on.


More information about the Mercurial-devel mailing list