(resend)[1/5]hidclass.sys: Implement helper IoCallMinidriver

Andrew Eikum aeikum at codeweavers.com
Fri Jul 10 13:24:09 CDT 2015


On Fri, Jul 10, 2015 at 12:48:36PM -0500, Aric Stewart wrote:
> On 7/10/15 12:21 PM, Dmitry Timoshkov wrote:
> > Aric Stewart <aric at codeweavers.com> wrote:
> > 
> >> +HRESULT IoCallMinidriver(ULONG code, PDEVICE_OBJECT device, PVOID in_buff, ULONG in_size, PVOID out_buff, ULONG out_size)
> > 
> > What 'HRESULT' is supposed to represent in the kernel space? (Especially
> > when actually an NTSTATUS is being returned). That applies to all patches
> > in the series. Also it would be nice to get rid of all LPJUNK style types
> > used all over the place.
> > 
> 
> Thanks for identifying the HRESULT thing. I need to fix that.
> 
> Ok question, Is there a desire for me to remove all the PXXXX and such with *XXXX style?  Looking at windows driver coding style there seemed to be a strong tendency to use PXXXX so that is what I did.  Did i miss the memo that said that WINE was going to be all *XXXX?
> 

I prefer removing them, but not strongly enough to make you re-do
everything. Up to you.

Andrew



More information about the wine-devel mailing list