[PATCH] GetUrlCacheEntryInfo*: Handle null dest pointer when length is given

Jon Griffiths jon_p_griffiths at yahoo.com
Fri Jun 6 22:22:34 CDT 2008

> Internal functions should return an error code instead of
> calling SetLastError.

Perhaps in a general sense, but URLCache_CopyEntry already did
SetLastError(ERROR_INSUFFICIENT_BUFFER): setting it there means the
4 places that call it don't need to ^ the logic to set the 2
different errors themselves.

^ "repeat" is probably the word I was looking for there :-)


More information about the wine-devel mailing list