Vincent Povirk wrote:
Maybe you know this, but this patch does too many things and will need to be split before sending to wine-patches.
I suspected that, and a rework is in progress (the patches originally eaten by the list need that badly, too).
- if (!psps && !cb) /* special case: empty storage */
Is it really not enough to have a size of 0? The buffer has to be NULL as well?
I don't think so, size 0 should be enough. I'll add a test.
- /* MSDN mandates generic error codes */
MSDN's lists of error codes aren't necessarily exhaustive.
Oops. I generally claim to be aware of this fact.