Re: [PATCH 1/3] tools/widl: A structure containing a BOGUS_ARRAY should be a BOGUS_STRUCT.

[Alexandre Julliard]

Huw Davies  writes:
> @@ -280,9 +282,10 @@ unsigned char get_struct_fc(const type_t *type)
>          if (get_enum_fc(t) == RPC_FC_ENUM16)
>              return RPC_FC_BOGUS_STRUCT;
>          break;
> - case TGT_POINTER:
> case TGT_ARRAY:
> -
>    if (get_pointer_fc(t, field->attrs, FALSE) == RPC_FC_RP || pointer_size != 4)
> -
>    if(get_array_fc(t) == RPC_FC_BOGUS_ARRAY) return RPC_FC_BOGUS_STRUCT;
> - case TGT_POINTER:
> -
>    if(get_pointer_fc(t, field->attrs, FALSE) == RPC_FC_RP || pointer_size != 4)

This doesn't seem to match the midl behavior. For instance in the
server.idl test puints_t becomes a bogus struct, but it's a simple
struct with midl.