Re: [PATCH v2 0/10] MR6966: windows.web: Partially implement IJsonValueStatics::Parse().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.web/json_value.c:
> -
> -
>    errno = 0;
> -
>    result = wcstod( json, &end );
> -
> -
>    if (errno == ERANGE) return WEB_E_INVALID_JSON_NUMBER;
> -
>    if (result || ( !errno && end != json ))
> -
>    {
> -
>        *parsed_number = result;
> -
>        *json_value_type = JsonValueType_Number;
> -
>        return S_OK;
> -
>    }
> - }
> -
> - for ( int i = 0; json[i] != '\0'; i++ )
> - {
> -
>    char c = json[i];

Unless it is important to validate the strings I would leave this for later, and return S_OK. Validation is also just a matter of actually parsing the array and objects, so implementing them would automatically end up validating the json string.