Re: [PATCH 0/7] MR8670: Improve WinRT parser checks

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about tools/widl/parser.y:
> +static void check_propput_args(const var_t *func)
> +{
> - const var_t *arg;
> - const var_list_t *arg_list = type_function_get_args(func->declspec.type);
> - unsigned int count = 0;
> -
> - if (!is_attr(func->attrs, ATTR_PROPPUT)) return;
> -
> - LIST_FOR_EACH_ENTRY(arg, arg_list, const var_t, entry)
> - {
> -
>    count++;
> -
>    if (count > 1) error_at( &arg->where, "propput function '%s' has too many parameters\n", func->name );
> -
> -
>    if (is_attr(arg->attrs, ATTR_OUT))
> -
>        error_at( &arg->where, "parameter '%s' of function '%s' must be an IN parameter\n", arg->name, func->name );
> - }

This should probably also support arrays like the `propget` checks.