Re: [PATCH v10 0/6] MR1948: windows.media.speech: Recognition session work the second.

[Bernhard Kölbl (＠besentv)]

On Wed Jan 25 12:05:53 2023 +0000, Jinoh Kang wrote:
> I think it's better to consolidate all resource cleanup to the `error:`
> label inside `recognizer_factory_Create`.  Alternatively, if we want to
> keep the `CloseHandle`/`Release` here, we should set the variable to
> `NULL` after closing the handle (or releasing the interface).  In
> general, we don't want to leave an object in an inconsistent state if we
> don't own it (e.g. it's passed by pointer as a parameter).

Yeah, I initially had the idea that every of these functions is self containing, but that can't possibly work out well.