widl: use base pointer type instead of default if not overwritten.

Rob Shearman robertshearman at gmail.com
Mon Jan 5 04:27:41 CST 2009


2009/1/4 Michael Karcher <wine at mkarcher.dialup.fu-berlin.de>:
> diff --git a/tools/widl/proxy.c b/tools/widl/proxy.c
> index f0e3f1d..0e5cbaf 100644
> --- a/tools/widl/proxy.c
> +++ b/tools/widl/proxy.c
> @@ -185,6 +185,11 @@ int needs_null_check(const var_t *v)
>
>      if (type)
>      {
> +      if (type->type == RPC_FC_FP ||
> +          type->type == RPC_FC_OP ||
> +          type->type == RPC_FC_UP)
> +        return 0;
> +
>        attrs = type->attrs;
>        type = type->ref;
>      }

Looks good, although we should always use the type set in the type
object, rather than trying to guess from the attributes. I plan to
clean this up in the near future so you don't need to change the patch
unless you want to.

-- 
Rob Shearman



More information about the wine-devel mailing list