Hi Kam!

Back to finding things to improve in LibSourcey, I've been quite interested in the FormWriter class behaviour. After several tests, it appears that the class isn't able to send multiple form parts.

I mean, the class relies on ProgressSignal to dispatch the reading progress of the parts. However, its fields are never initialised properly. So, one part just works because of this :

void update(int nread)
{
    assert(current <= total);
    current += nread;
    emit(progress());
}

The assertion is what's making the test to fail. Before adding the first part, current and total are both equal to 0, so the assertion succeeds. After this call, current would hold the size of the part that was just added, while total is still 0, since it is never updated anywhere (grep -r 'OutgoingProgress.total' . in src/http/ told me so), and the assertion fails on adding another part.

Although it causes a crash in debug mode, running this in release mode will indeed lead into bogus progress values, such as things greater than 1 (>100%). Also, this :

double progress() const { return (current / (total * 1.0)) * 100; }

will lead to a division by zero, returning NaN (or inf).

I am not quite sure how this could be fixed, so I just posted it here as an issue. I have an idea though, as total should be computed before writeMultipartChunk gets called.

That's kind of a minor bug, but I think that it is worth reporting, though.