QHttp in thread (QT4)

[wysota]

Well, I thought I needed to close the file properly to ensure my program did not keep it open. Is that not true?

It's true (although it's possible that QHttp closes it itself), but here you create a new file object instead of closing the one already open. You should get a pointer to a QIODevice you assigned when making the request and call close on it.

I moved the code I had before into a done slot and now I do not see that debug error any more. i will have to keep an eye on it to ensure that it does not return.

It shouldn't unless you start deleting the signalling object from a slot again.

I can not simply look up the request id in my table to ensure I am working on the correct entry since there could be duplicates, right?):

Every id is unique per object, so the pair (QHttp*, id) can't duplicate.

BTW. If you make a qobject_cast, you should check if it doesn't return 0. If you're sure that it is a QHttp* (like in this case), you can make a static_cast instead.

int index = httpPointers.indexOf(sendHttp);
httpPointers.removeAt(index);

int index = httpPointers.indexOf(sendHttp);
httpPointers.removeAt(index);

To copy to clipboard, switch view to plain text mode 

I think you can (but don't have to - your code should be slightly faster) substitute it with:

httpPointers.removeAll(sendHttp);

httpPointers.removeAll(sendHttp);

To copy to clipboard, switch view to plain text mode