Re: [PATCH 0/5] MR8340: widl: Add rows for propget methods.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about tools/widl/metadata.c:
> +static void add_propget_method( const type_t *iface, const var_t *method )
> +{
> - static const BYTE sig_prop[] = { SIG_TYPE_HASTHIS | SIG_TYPE_PROPERTY, 0, ELEMENT_TYPE_I4 };
> - UINT methoddef, property, sig_size, paramlist, attrs;
> - BYTE sig[256];
> - char *name;
> -
> - property = add_property_row( 0, add_string(method->name), add_blob(sig_prop, sizeof(sig_prop)) );
> - method->declspec.type->md.property = property;
> - add_propertymap_row( iface->md.def, property );
> -
> - paramlist = add_method_params_step2( type_function_get_args(method->declspec.type) );
> - sig_size = make_method_sig( method, sig );
> -
> - attrs = METHOD_ATTR_FAMANDASSEM | METHOD_ATTR_FAMILY   | METHOD_ATTR_VIRTUAL  | METHOD_ATTR_HIDEBYSIG |

What about `METHOD_ATTR_PUBLIC` instead of `METHOD_ATTR_FAMANDASSEM | METHOD_ATTR_FAMILY`?