Background download Thread Progress Windows problem

[railjonrogut]

As a test (and only a test) try putting this in after you increment the progress value:

MessageManager::getInstance()->runDispatchLoopUntil (200);

Rail

[MBO]

The problem is you probably have not initialized ‘progress = 0’. Then you’re adding values to ‘progress’ in the run() but until ‘progress’ reaches 1% you will wait for eternity.

Edit: Ah, sorry, I see it’s in the constructor. I’ve got almost identical code in my apps and it works perfectly fine.

[jules]

progress += i / j;

maybe you meant

progress = i / j;

[danielrudrich]

Shouldn‘t i be converted to float first before dividing? Otherwise it will return 0 al the time. Or am I wrong here?

Edit: sorry, j is double so never mind

[jules]

Personally I’d have written it with an explicit cast to double, just to make sure that’s clear to a reader, e.g.

progress = currentItem / (double) totalNumItems;

[mikolaj91]

I think it was clear to reader, i was testing with hardcoded values also.
MessageManager::getInstance()->runDispatchLoopUntil (200);
works for testing in simple loop, but returns assertion when running in thread as it is not a Message thread.