Cast-qual warning fix

Alexandre Julliard julliard at winehq.org
Mon Nov 6 06:01:00 CST 2006


Andrew Talbot <Andrew.Talbot at talbotville.com> writes:

> In our case, MSIITERHANDLE is declared as a pointer to void, so
>
>     const MSIITERHANDLE *handle;
>
> is equivalent to
>
>     void *const *handle;
>
> whereas
>
>    MSICITERHANDLE *handle;
>
> is equivalent to
>
>    const void **handle;
>
> which, I believe, is what is required. Hence, my new type definition.

Actually from reading the code there doesn't seem to be a reason for
having the non-const variant, and also no reason to use a void*
instead of a proper type. The definition should probably be something
like "typedef const struct tagMSICOLUMNHASHENTRY *MSIITERHANDLE".

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list