Re: [PATCH 05/10][try 2] quartz/tests: Fix interface leaks in aggregation test.

[Michael Karcher]

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?

Regards,
  Michael Karcher

[PATCH 05/10][try 2] quartz/tests: Fix interface leaks in aggregation test.

[Alexander Dorofeyev]

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.