Re: [PATCH] FCI work for cabinet.dll [cabinet-fci-patch-02c.diff]

[Alexandre Julliard]

"Gerold J. Wucherpfennig"  writes:
> Please apply, it seems to be fine, so far.
>
> There is still some work to be done:
> - currently no support for big-endian machines
> - the ERF error structure aren't used on error
> - no real compression yet
> - unknown behaviour if files>4GB or cabinet >4GB
> - incorrect status information
> - check if the maximum size for a cabinet is too small to store any data
> - call pfnfcignc on exactly the same position as MS FCIAddFile in every case

You should get rid of all the PFCI_INT macro uses, that's making the
code really ugly and hard to read. You should do the cast once at the
start of the function (preferably as part of the handle validation
routine) and then pass around a pointer of the appropriate type.