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