Re: [PATCH 3/5] d2d1: Implement property parsing for RegisterEffectFromStream().

[Nikolay Sivov]

On 6/6/22 10:35, Ziqing Hui wrote:
> struct d2d_effect_info
>   {
>       const CLSID *clsid;
>       UINT32 default_input_count;
>       UINT32 min_inputs;
>       UINT32 max_inputs;
> +    WCHAR *display_name;
> +    WCHAR *author;
> +    WCHAR *category;
> +    WCHAR *description;
>   };
> Shouldn't these also be expressed as properties?
> +static D2D1_PROPERTY_TYPE str_to_property_type(const WCHAR *str)
> +{
> +    unsigned int i;
> +
> +    static const WCHAR *type_str[] =
> +    {
> +        L"", L"string", L"bool", L"uint32", L"int32", L"float", L"vector2", L"vector3",
> +        L"vector4", L"blob", L"iunknown", L"enum", L"array", L"clsid", L"matrix3x2",
> +        L"matrix4x3", L"matrix4x4", L"matrix5x4", L"colorcontext",
> +    };
> +
> +    for (i = 1; i < ARRAY_SIZE(type_str); ++i)
> +    {
> +        if (!wcscmp(str, type_str[i]))
> +            return i;
> +    }
> +
> +    return D2D1_PROPERTY_TYPE_UNKNOWN;
> +}
> This uses numeric values of D2D1_PROPERTY_TYPE, that deserves at least a 
> comment.