nanogui@linuxhacker.org

nanogui@linuxhacker.org


Subject: Re: [patch] windef.h enhancements for VxWorks
From: Greg Haerr
Date: Mon, 9 Oct 2000 20:12:04 -0600

Brent,
    I'm about to add some of your include file changes for
VxWorks - but I'm wondering why every internal win32
.h file needs to be wrapped with a total #ifdef wrapper.
These routines will never be included more than once
if I wrap windows.h, so I'm thinking of leaving out
the extraneous #ifdef wrappers.  OK?

: Here's the guts of the changes needed for the Microwindows
: port to VxWorks.  There are some typedef conflicts between
: the two systems (UCHAR/USHORT/ULONG/UINT/BOOL) which are
: handled with proper timing of includes and ifdefs.

Also, I don't like including the vxWorks.h from within
the win32-specific windefs.h.  Why is it needed in here?
It isn't required to compile the mwin/*.c files is it?

I do agree that perhaps the UCHAR and USHORT stuff
shouldn't be included if VXWORKS is #defined.

Regards,

Greg


:
: BAT
: --
: Brent A. Thompson, Next Level Communications <www.nlc.com>
: 1776 22nd Street #100, West Des Moines, IA 50266-1444
: EMail: <Thompson@nlc.com>, Phone: (515)991-3853
:


--------------------------------------------------------------------------------


: *** windef.h_0.88 Mon Oct  9 11:13:20 2000
: --- windef.h Mon Oct  9 14:15:22 2000
: ***************
: *** 5,12 ****
: --- 5,22 ----
:    * Win32 API base type definitions
:    */
:
: + #ifndef _WINDEF_H
:   #define _WINDEF_H
:
: +
: + #ifdef VXWORKS
: + /* Don't include the internal Tornado header file <ntcontext.h>, **
: + ** as the definitions in it conflict with these definitions.     */
: + #define __INCntcontexth
: + #include <vxWorks.h>
: + #endif
: +
: +
:   #ifndef NULL
:   #define NULL    ((void *)0)
:   #endif
: ***************
: *** 51,74 ****
:
:   #ifndef __ITRON_TYPES_h_ /* FIXME RTEMS hack*/
:   #ifndef COMMON_H /* MiniGUI hack*/
: - typedef int BOOL;
:   #endif
:   #endif
:
:   typedef unsigned char UCHAR;
:   typedef unsigned long ULONG;
:   typedef ULONG * PULONG;
: - typedef unsigned short USHORT;
:   typedef USHORT * PUSHORT;
:   typedef UCHAR * PUCHAR;
:   typedef char * PSZ;
:   typedef int                 INT;
: - typedef unsigned int        UINT;
:   typedef unsigned int        *PUINT;
:
:   typedef unsigned long       DWORD;
:   typedef unsigned char       BYTE;
:   typedef unsigned short      WORD;
:   typedef float               FLOAT;
:   typedef FLOAT               *PFLOAT;
:   typedef BOOL NEAR           *PBOOL;
: --- 61,88 ----
:
:   #ifndef __ITRON_TYPES_h_ /* FIXME RTEMS hack*/
:   #ifndef COMMON_H /* MiniGUI hack*/
:   #endif
:   #endif
:
: + #ifndef VXWORKS
:   typedef unsigned char UCHAR;
: + typedef unsigned short USHORT;
:   typedef unsigned long ULONG;
: + typedef unsigned int        UINT;
: + typedef int BOOL;
: + #endif
: +
:   typedef ULONG * PULONG;
:   typedef USHORT * PUSHORT;
:   typedef UCHAR * PUCHAR;
:   typedef char * PSZ;
:   typedef int                 INT;
:   typedef unsigned int        *PUINT;
:
:   typedef unsigned long       DWORD;
:   typedef unsigned char       BYTE;
:   typedef unsigned short      WORD;
: +
:   typedef float               FLOAT;
:   typedef FLOAT               *PFLOAT;
:   typedef BOOL NEAR           *PBOOL;
: ***************
: *** 157,159 ****
: --- 171,177 ----
:   typedef HANDLE HMENU;
:
:   typedef LRESULT (CALLBACK* WNDPROC)(HWND, UINT, WPARAM, LPARAM);
: +
: +
: + #endif /* _WINDEF_H */
: +
:
:


--------------------------------------------------------------------------------


: ---------------------------------------------------------------------
: To unsubscribe, e-mail: nanogui-unsubscribe@linuxhacker.org
: For additional commands, e-mail: nanogui-help@linuxhacker.org


nanogui@linuxhacker.org