widl: Fix the is_array_type function.

Dan Hipschman dsh at linux.ucla.edu
Tue Jun 5 15:11:50 CDT 2007


On Tue, Jun 05, 2007 at 07:50:42PM +0100, Robert Shearman wrote:
> 
> Previously, an array of pointers wouldn't be detected by this function.
> ---
>  tools/widl/header.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tools/widl/header.h b/tools/widl/header.h
> index 57ae6b4..2165db7 100644
> --- a/tools/widl/header.h
> +++ b/tools/widl/header.h
> @@ -69,7 +69,7 @@ static inline int is_string_type(const a
>  static inline int is_array_type(const attr_list_t *attrs, const type_t *type, const array_dims_t *array)
>  {
>      return ((last_ptr(type) && !array && is_attr(attrs, ATTR_SIZEIS)) ||
> -            (!is_ptr(type) && array));
> +            array);
>  }
>  
>  #endif 

This and your next patch are in the same area as the one I just sent in
yesterday.  Actually, my patch completely removes this function.  Can
you try doing whatever you're working on with my last patch applied and
see if that fixes your problems?  Generating the format string for
arrays of pointers is going to required a little more work because
FIXED_REPEAT, etc. isn't implemented yet.

My last patch set:

http://winehq.org/pipermail/wine-patches/2007-June/040007.html
http://winehq.org/pipermail/wine-patches/2007-June/040008.html

Thanks,
Dan




More information about the wine-devel mailing list