darcs

Issue 1709 incorrect progress report numbering?

Title incorrect progress report numbering?
Priority bug Status resolved
Milestone 2.4.x Resolved in
Superseder Nosy List Quiark, darcs-devel, dmitry.kurochkin, kowey
Assigned To Quiark
Topics Regression, UI

Created on 2009-12-02.14:03:29 by kowey, last changed 2010-06-15.21:30:59 by admin.

Messages
msg9517 (view) Author: kowey Date: 2009-12-02.14:03:17
Hi Roman,

I noticed this surprising progress reporting text when doing a darcs check on
the darcs darcs repository.  

Replaying patch 468 done, 8009 queued.
Replaying patch 938 done, 8009 queued.
Replaying patch 1382 done, 8009 queued.
Replaying patch 1755 done, 8009 queued.
Replaying patch 2301 done, 8009 queued.
Replaying patch 2605 done, 8009 queued.
Replaying patch 3179 done, 8009 queued.
Replaying patch 3565 done, 8009 queued.
Replaying patch 4130 done, 8009 queued.
Replaying patch 4296 done, 8009 queued.
Replaying patch 4437 done, 8009 queued.
Replaying patch 4620 done, 8009 queued.
Replaying patch 4725 done, 8009 queued.
Replaying patch 4994 done, 8009 queued.
Replaying patch 5261 done, 8009 queued.
Replaying patch 5459 done, 8009 queued.
Replaying patch 5708 done, 8009 queued.
Replaying patch 6030 done, 8009 queued.
Replaying patch 6394 done, 8009 queued.
Replaying patch 6560 done, 8009 queued.
Replaying patch 6844 done, 8009 queued.
Replaying patch 7171 done, 8009 queued.
Replaying patch 7651 done, 8009 queued.
Replaying patch 7841 done, 8009 queued.

It looks like the "N queued" is incorrectly reporting the current total and not
the different between the total and the number completed.  Or am I just
misinterpreting "queued" here to mean "currently remaining" vs "total that was
queued up?"

So the first question is: what should "queued" mean?
Do you have any time to look into this?
msg9519 (view) Author: Quiark Date: 2009-12-02.18:42:09
Well, basically I don't have time for anything :)

Anyway, "queued x" should mean that there's at the moment "x" items to do and 
it should go down to 0. The short version of the report was wrong, I already 
sent a patch which corrects it. For some reason it worked fine on my machine...
msg9562 (view) Author: Quiark Date: 2009-12-08.15:10:23
The following patch updated the status of issue1709 to be resolved:

* resolve issue1709: Short version of progress reporting was wrong. 
Ignore-this: 51065958e4b46269e82e95876e5a82a2
msg10210 (view) Author: Quiark Date: 2010-03-15.11:00:37
The following patch updated the status of issue1709 to be resolved:

* resolve issue1709: Short version of progress reporting was wrong. 
Ignore-this: 51065958e4b46269e82e95876e5a82a2
History
Date User Action Args
2009-12-02 14:03:29koweycreate
2009-12-02 18:42:11Quiarksetmessages: + msg9519
2009-12-06 01:36:24ganeshlinkpatch105 issues
2009-12-08 15:10:29Quiarksetstatus: needs-reproduction -> resolved
messages: + msg9562
2010-03-15 11:00:41Quiarksetmessages: + msg10210
2010-06-15 21:30:57adminsetmilestone: 2.4.x
2010-06-15 21:30:59adminsettopic: - Target-2.4