Re: [PATCH 0/3] MR8507: widl: Drop an incorrect check on runtimeclass definitions.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about tools/widl/typetree.c:
> runtimeclass->attrs = check_runtimeclass_attrs(runtimeclass->name, attrs);
>      runtimeclass->details.runtimeclass.ifaces = ifaces;
>      define_type(runtimeclass, where);
> -    if (!type_runtimeclass_get_default_iface(runtimeclass, FALSE) &&
> -        !get_attrp(runtimeclass->attrs, ATTR_STATIC))
> -        error_loc("runtimeclass %s must have a default interface or static factory\n", runtimeclass->name);

This might not be the right place to check for it but midl definitely emits an error for such cases, and it should be moved where appropriate, not removed.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8507#note_108822