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

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/windows.web/json_value.c:
> static HRESULT WINAPI json_value_statics_CreateStringValue( IJsonValueStatics *iface, HSTRING input, IJsonValue **value )
>  {
> -    FIXME( "iface %p, input %s, value %p stub!\n", iface, debugstr_hstring( input ), value );
> -    return E_NOTIMPL;
> +    struct json_value *impl;
> +
> +    TRACE( "iface %p, input %s, value %p\n", iface, debugstr_hstring( input ), value );
> +
> +    if (!value) return E_POINTER;
> +    if (!(impl = calloc( 1, sizeof(*impl) ))) return E_OUTOFMEMORY;
> +
> +    impl->IJsonValue_iface.lpVtbl = &json_value_vtbl;
> +    impl->ref = 1;
> +    WindowsDuplicateString( input, &impl->string_value );

```suggestion:-0+0
    if (FAILED(hr = WindowsDuplicateString( input, &impl->string_value )))
    {
         free( impl );
         return hr;
    }
```

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