On Sunday, September 19, 2010 17:17:53 Alexandre Julliard wrote:
> Mike Frysinger writes:
> > i dont see how that would help.  the code is currently:
> > typedef struct ... {
> > 
> >     ...
> >     CHAR foo[1];
> > 
> > } ...;
> > 
> > it needs to be "foo[]".  ANYSIZE_ARRAY is defined as 1, so once the
> > preprocessor is done, we're right back where we started.  i cant change
> > the value of ANYSIZE_ARRAY to nothing because some code uses this define
> > in multiplication to calculate the size of objects.
> 
> True, you don't want to change the public one, but you can define a
> shell32-specific version of it. At least then it looks like a normal
> array declaration.

what do you mean by "public" ?  the header i'm changing (dlls/shell32/pidl.h) 
isnt exported during `make install` that i can see.  so only wine itself will 
be reading this header, and i think all consumers of it internally should be 
getting "foo[]".
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.



Reply via email to