[PATCH (try2)] setupapi: Fix spec entries for handle parameters

Andrew Eikum aeikum at codeweavers.com
Thu Mar 31 07:15:18 CDT 2016


On Thu, Mar 31, 2016 at 04:34:27AM +0200, Sebastian Lackner wrote:
> On 30.03.2016 17:00, Andrew Eikum wrote:
> > Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
> > ---
> > 
> > try2: Turns out there's a whole bunch more of these. I think I got
> > most/all of them.
> > 
> >  dlls/setupapi/setupapi.spec | 118 ++++++++++++++++++++++----------------------
> >  1 file changed, 59 insertions(+), 59 deletions(-)
> 
> Is there any specific reason why you want to change that? Using "long"
> for handles is usually perfectly fine and it depends on the exact
> implementation details if its a pointer or just an index into a handle table.
> 

I thought that ptr and long were different widths on 64-bit, which
could lead to incorrect parameter dumping in relay logs. If that's not
the case, then ignore the patch.

Andrew



More information about the wine-devel mailing list