Re: [PATCH v2 0/7] MR5992: windows.web: Implement IJsonValueStatics::CreateStringValue().

[Mohamad Al-Jaf (＠maljaf)]

On Fri Jul  5 04:37:23 2024 +0000, Rémi Bernon wrote:
> ```suggestion:-0+0
>     hr = WindowsCreateString( json_object_name, wcslen( json_object_name
> ), &str );
>     ok( hr == S_OK, "got hr %#lx.\n", hr );
>     hr = RoActivateInstance( str, &inspectable );
> ```
> I'd suggest to keep the `WindowsDeleteString( str );` above and
> duplicate the string creation. Makes the cleanup easier to track
> (because right now you're leaking the string if the class isn't present).

Good catch, updated.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5992#note_75162