trying to make QHttp synchronous or threaded

[wysota]

By the way... This:

while(!someFlagActive) qApp->processEvents();

while(!someFlagActive) qApp->processEvents();

To copy to clipboard, switch view to plain text mode 

is equivalent to this (just designed worse :P):

QHttp *http;
//...
QEventLoop loop;
connect(http, SIGNAL(done(bool)), &loop, SLOT(quit())); // you can use a different signal here
loop.exec(QEventLoop::AllEvents|QEventLoop::WaitForMoreEvents);
doSomethingAfterDoneIsEmitted();

QHttp *http;
//...
QEventLoop loop;
connect(http, SIGNAL(done(bool)), &loop, SLOT(quit())); // you can use a different signal here
loop.exec(QEventLoop::AllEvents|QEventLoop::WaitForMoreEvents);
doSomethingAfterDoneIsEmitted();

To copy to clipboard, switch view to plain text mode 

Remember that done() will be processed before the loop exits!

[ber_44]

> probably "reqId.value(reqId_)" returns an invalid value

No, I printed it, it's just fine. The crash was caused by something else... Now I'm just getting
free(): invalid pointer 0x40016ca0!
after returning from the SET_HOST / SET_USER case (request is queued to happen after HttpWindow::HttpWindow()

But this does not bother me too much. My program goes on to the first download, BUT when I print file->bytesAvailable() in the GET_N_BYTES case it is zero-size. However, the file does get downloaded just fine, as I see it after the program finishes.

> If you want to force "synchronousness", use a busy loop
> QEventLoop loop;
> connect(http, SIGNAL(done(bool)), &loop, SLOT(quit())); // you can use a different signal here
> loop.exec(QEventLoop::AllEvents|QEventLoop::WaitFo rMoreEvents);
> doSomethingAfterDoneIsEmitted();

That seems to be the solution. IMHO we just replaced the original idea of using QThread::wait() with a local event loop (the program won't execute the next statement until that local event loop has finished).