mstefani at redhat.de
Thu Oct 31 11:04:20 CST 2002
On Thu, Oct 31, 2002 at 10:55:10AM -0600, Greg Turner wrote:
> On Thursday 31 October 2002 09:05 am, Michael Stefaniuc wrote:
> > -UINT WINAPI I_RpcWindowProc( HANDLE hWnd, UINT Message, UINT wParam,
> > ULONG lParam)
> >+UINT WINAPI I_RpcWindowProc( void *hWnd, UINT Message,
> > UINT wParam, ULONG lParam )
> > That's totaly broken. It's a problem in winapi_check and not in your
> > code. Well, their is a problem in your code too, i would change
> > HANDLE to HWND.
> > bye
> > michael
> Before I change it back on your advice, are you /sure/ about this?
> o rpcrt4 compiles STRICT, so how broken can the conflation of
> HANDLE and void* be? (HWND is DECLARE_HANDLE()'ed so I could see
> your point there.)
HANDLE is a void* therefor "compatible" to any other pointer, that's why
you don't get a warning.
> o from the Microsoft Platform SDK headers (October '02):
> ./rpcdcep.h-729-unsigned int
> ./rpcdcep.h-732- IN void * hWnd,
> ./rpcdcep.h-733- IN unsigned int Message,
> ./rpcdcep.h-734- IN unsigned int wParam,
> ./rpcdcep.h-735- IN unsigned long lParam
> ./rpcdcep.h-736- ) ;
> so, technically, isn't void* a /better/ choice?
*Sigh* a HWND should be HWND but m$ seems to have decided to do it in an
other way. I would say resend the patch but please change the Changelog.
Michael Stefaniuc Tel.: +49-711-96437-199
System Administration Fax.: +49-711-96437-111
Red Hat GmbH Email: mstefani at redhat.com
Hauptstaetterstr. 58 http://www.redhat.de/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 232 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20021031/2b4c2c30/attachment.pgp
More information about the wine-devel