[2/2] widl: Fix generation of a typelib for winhttp.
Dmitry Timoshkov
dmitry at baikal.ru
Fri Oct 2 04:04:55 CDT 2015
Dmitry Timoshkov <dmitry at baikal.ru> wrote:
> Huw Davies <Huw.Davies at physics.ox.ac.uk> wrote:
>
> > > @@ -2307,9 +2306,7 @@ static void add_entry(msft_typelib_t *typelib, const statement_t *stmt)
> > > {
> > > const type_list_t *type_entry = stmt->u.type_list;
> > > for (; type_entry; type_entry = type_entry->next) {
> > > - /* if the type is public then add the typedef, otherwise attempt
> > > - * to add the aliased type */
> > > - if (is_attr(type_entry->type->attrs, ATTR_PUBLIC))
> > > + if (type_get_type(type_entry->type) == TYPE_BASIC)
> > > add_typedef_typeinfo(typelib, type_entry->type);
> > > else
> > > add_type_typeinfo(typelib, type_alias_get_aliasee(type_entry->type));
> >
> > Doesn't this essentially make [public] a nop? That seems wrong.
> >
> > The WinHttpRequestOption enum has a [uuid] attribute, so that probably
> > has something to do with it.
>
> I have tested typedefs with various types and was also wondering about
> [public] attribute: regardless of its existence midl generates identical
> typelibs.
Forgot to mention that [uuid] attribute also changes nothing for the enum
type generation.
--
Dmitry.
More information about the wine-devel
mailing list