[PATCH 05/10][try 2] quartz/tests: Fix interface leaks in aggregation test.
Alexander Dorofeyev
alexd4 at inbox.lv
Thu Jul 17 13:44:12 CDT 2008
Michael Karcher wrote:
> Am Donnerstag, den 17.07.2008, 20:24 +0300 schrieb Alexander Dorofeyev:
>> + do
>> + {
>> + hr = IUnknown_Release(pUnkInner);
>> + } while (hr);
> Release returns a ULONG, not a HRESULT. Do we ignore the difference in
> Wine?
I usually do not, but that's how it happens in surrounding code there
(ADDREF_EXPECT macro etc). IMO this is no big deal, but if this is a causing
questions I guess I'll change it to use an ULONG variable and and fix the
existing code to do the same for the sake of consistency.
More information about the wine-devel
mailing list