URLDownloadToFile

[Mat]

I'm surprised HRESULT isn't unsigned. Honestly, I hate it when types are defined via typedef and #define. It would be significantly easier if types were called their types and only got a new name if they were different.

Things like this bug me:

typedef long LONG;

Has some ms programmer got an aversion to using lowercase? I get a nice blue colour for base types in vs... Now you make me look at boring black capitals.

[trancexx]

Already downloaded means the files there, and can be used. In 99% of cases that means success. If you know that you always want to download a new file then delete the file beforehand. If you look at it the other way round, if I wanted to treat S_FALSE as a success, I'd have to compare it against at least two values... If there were 200 different possible success codes then do you really want to test all of them?

I would still do if (SUCCEEDED(hr) && (hr != S_FALSE)) {}

We are talking programming, right? 90%