wininet: Remove the custom thread pool implementation and use
rob at codeweavers.com
Thu Jan 4 14:27:57 CST 2007
Thanks for looking through the patch.
Jacek Caban wrote:
> Why do you copy lpRequest to workRequest? You could use lpRequest
> in asyncproc and WININET_Release and call HeapFree before return.
> Then workRequest may be removed.
1. This was how the code did it before - I copied it from
2. The stack is already allocated, so using it instead of the heap makes
sense since it allows the block in the heap to be used again immediately.
Yes, it's two more lines of code, but I can't see a convincing reason to
change it from how it was before. Can you?
More information about the wine-devel