General observation: other tests in Wine explicitly check for specific hr values, is there a reason you have used the generic SUCCEEDED here instead of that approach?
In general we don't care what the return value on success is. Applications sometimes sometimes do check for particular failure conditions, but particular success conditions aren't checked nearly as often. --Juan