Re: [PATCH v2 0/1] MR423: ieframe fix

[David Kahurani (＠kahurani)]

On Mon Jul 11 17:59:39 2022 +0000, Nikolay Sivov wrote:
> VariantClear() seems more fitting. Also, you don't need this
> duplication. Just use FAILED(hres) to return 0 or DDE_FACK.

Hello.

Thanks for the review.

Yes, VariatClear() would indeed be more fitting but I used SysFreeString instead for consistency with the rest of the file. Should I ignore that? Or...maybe look into migrating the whole module?