Minor Motif changes, made scrollsub sample work somehow.

[wxWidgets.git] / src / os2 / utils.cpp

diff --git a/src/os2/utils.cpp b/src/os2/utils.cpp
index 15fd47cea381f2f24d5132810d69ef25e46dec64..e837dcc66a569054d7137d2e2fd7215ced29ed13 100644 (file)
--- a/src/os2/utils.cpp
+++ b/src/os2/utils.cpp
@@ -46,7 +46,6 @@
 #include <netcons.h>
 #include <netbios.h>
 
 #include <netcons.h>
 #include <netbios.h>
 

-// In the WIN.INI file

 static const wxChar WX_SECTION[] = _T("wxWindows");
 static const wxChar eHOSTNAME[]  = _T("HostName");
 static const wxChar eUSERID[]    = _T("UserId");
 static const wxChar WX_SECTION[] = _T("wxWindows");
 static const wxChar eHOSTNAME[]  = _T("HostName");
 static const wxChar eUSERID[]    = _T("UserId");


@@ -58,87 +57,117 @@ static const wxChar eUSERNAME[]  = _T("UserName");
 // functions beyond those provided by WinSock
 
 // Get full hostname (eg. DoDo.BSn-Germany.crg.de)
 // functions beyond those provided by WinSock
 
 // Get full hostname (eg. DoDo.BSn-Germany.crg.de)

-bool wxGetHostName(wxChar *buf, int maxSize)
+bool wxGetHostName(
+  wxChar*                           zBuf
+, int                               nMaxSize
+)

 {
 #if wxUSE_NET_API
 {
 #if wxUSE_NET_API

-  char                               server[256];
-  char                               computer[256];
-  unsigned long                      ulLevel;
-  unsigned char*                     pbBuffer;
-  unsigned long                      ulBuffer;
-  unsigned long*                     pulTotalAvail;
-
-  NetBios32GetInfo( (const unsigned char*)server
-                   ,(const unsigned char*)computer
-                   ,ulLevel
-                   ,pbBuffer
-                   ,ulBuffer
-                   ,pulTotalAvail
+    char                            zServer[256];
+    char                            zComputer[256];
+    unsigned short                  nLevel = 0;
+    unsigned char*                  zBuffer;
+    unsigned short                  nBuffer;
+    unsigned short*                 pnTotalAvail;
+
+    NetBiosGetInfo( (const unsigned char*)zServer
+                   ,(const unsigned char*)zComputer
+                   ,nLevel
+                   ,zBuffer
+                   ,nBuffer
+                   ,pnTotalAvail

                   );
                   );

-  strcpy(buf, server);
+    strcpy(zBuf, zServer);

 #else
 #else

-  wxChar *sysname;
-  const wxChar *default_host = _T("noname");
-
-  if ((sysname = wxGetenv(_T("SYSTEM_NAME"))) == NULL)
-  {
-      // GetProfileString(WX_SECTION, eHOSTNAME, default_host, buf, maxSize - 1);
-  }
-  else
-      wxStrncpy(buf, sysname, maxSize - 1);
-  buf[maxSize] = _T('\0');
+    wxChar*                         zSysname;
+    const wxChar*                   zDefaultHost = _T("noname");
+
+    if ((zSysname = wxGetenv(_T("SYSTEM_NAME"))) == NULL)
+    {
+        ULONG n = ::PrfQueryProfileString( HINI_PROFILE
+                                          ,(PSZ)WX_SECTION
+                                          ,(PSZ)eHOSTNAME
+                                          ,(PSZ)zDefaultHost
+                                          ,(void*)zBuf
+                                          ,(ULONG)nMaxSize - 1
+                                         );
+    }
+    else
+        wxStrncpy(zBuf, zSysname, nMaxSize - 1);
+    zBuf[nMaxSize] = _T('\0');

 #endif
 #endif

-  return *buf ? TRUE : FALSE;
+    return *zBuf ? TRUE : FALSE;

 }
 
 // Get user ID e.g. jacs
 }
 
 // Get user ID e.g. jacs

-bool wxGetUserId(wxChar *buf, int maxSize)
+bool wxGetUserId(
+  wxChar*                           zBuf
+, int                               nMaxSize
+)

 {
 {

-  return(U32ELOCL((unsigned char*)buf, (unsigned long *)&maxSize));
+   return(U32ELOCL((unsigned char*)zBuf, (unsigned long *)&nMaxSize));

 }
 
 }
 

-bool wxGetUserName(wxChar *buf, int maxSize)
+bool wxGetUserName(
+  wxChar*                           zBuf
+, int                               nMaxSize
+)

 {
 #ifdef USE_NET_API
 {
 #ifdef USE_NET_API

-    wxGetUserId(buf, maxSize);
+    wxGetUserId( zBuf
+                ,nMaxSize
+               );

 #else
 #else

-    wxStrncpy(buf, _T("Unknown User"), maxSize);
+    wxStrncpy(zBuf, _T("Unknown User"), nMaxSize);

 #endif
 #endif

-  return TRUE;
+    return TRUE;

 }
 
 }
 

-int wxKill(long pid, int sig)
+int wxKill(
+  long                              lPid
+, int                               nSig
+)

 {
 {

-  return 0;
+    return((int)::DosKillProcess(0, (PID)lPid));

 }
 
 //
 // Execute a program in an Interactive Shell
 //
 }
 
 //
 // Execute a program in an Interactive Shell
 //

-bool wxShell(const wxString& command)
+bool wxShell(
+  const wxString&                   rCommand
+)

 {
 {

-  wxChar *shell;
-  if ((shell = wxGetenv(_T("COMSPEC"))) == NULL)
-    shell = _T("\\CMD.EXE");
+    wxChar*                         zShell;
+
+    if ((zShell = wxGetenv(_T("COMSPEC"))) == NULL)
+        zShell = _T("\\CMD.EXE");

 
 

-  wxChar tmp[255];
-  if (command != "")
-    wxSprintf(tmp, "%s /c %s", shell, WXSTRINGCAST command);
-  else
-    wxStrcpy(tmp, shell);
+    wxChar                          zTmp[255];

 
 

-  return (wxExecute((wxChar *)tmp, FALSE) != 0);
+    if (rCommand != "")
+        wxSprintf( zTmp
+                  ,"%s /c %s"
+                  ,zShell
+                  ,WXSTRINGCAST rCommand
+                 );
+    else
+        wxStrcpy(zTmp, zShell);
+
+    return (wxExecute((wxChar*)zTmp, FALSE) != 0);

 }
 
 // Get free memory in bytes, or -1 if cannot determine amount (e.g. on UNIX)
 }
 
 // Get free memory in bytes, or -1 if cannot determine amount (e.g. on UNIX)

-long wxGetFreeMemory(void *memptr)
+long wxGetFreeMemory(
+  void*                             pMemptr
+)

 {
     ULONG                           lSize;
     ULONG                           lMemFlags;
     APIRET                          rc;
 
     lMemFlags = PAG_FREE;
 {
     ULONG                           lSize;
     ULONG                           lMemFlags;
     APIRET                          rc;
 
     lMemFlags = PAG_FREE;

-    rc = ::DosQueryMem(memptr, &lSize, &lMemFlags);
+    rc = ::DosQueryMem(pMemptr, &lSize, &lMemFlags);

     if (rc != 0)
         return -1L;
     return (long)lSize;
     if (rc != 0)
         return -1L;
     return (long)lSize;


@@ -149,24 +178,28 @@ static bool inTimer = FALSE;
 
 class wxSleepTimer: public wxTimer
 {
 
 class wxSleepTimer: public wxTimer
 {

- public:
-  inline void Notify()
-  {
-    inTimer = FALSE;
-    Stop();
-  }
+public:
+    inline void Notify()
+    {
+        inTimer = FALSE;
+        Stop();
+    }

 };
 
 };
 

-static wxTimer *wxTheSleepTimer = NULL;
+static wxTimer*                     wxTheSleepTimer = NULL;

 
 

-void wxUsleep(unsigned long milliseconds)
+void wxUsleep(
+  unsigned long                     ulMilliseconds
+)

 {
 {

-    ::DosSleep(milliseconds);
+    ::DosSleep(ulMilliseconds);

 }
 
 }
 

-void wxSleep(int nSecs)
+void wxSleep(
+  int                               nSecs
+)

 {
 {

-    ::DosSleep( 1000*nSecs );
+    ::DosSleep(1000 * nSecs);

 }
 
 // Consume all events until no more left
 }
 
 // Consume all events until no more left


@@ -176,48 +209,53 @@ void wxFlushEvents()
 }
 
 // Output a debug mess., in a system dependent fashion.
 }
 
 // Output a debug mess., in a system dependent fashion.

-void wxDebugMsg(const wxChar *fmt ...)
+void wxDebugMsg(
+  const wxChar*                     zFmt ...
+)

 {
 {

-  va_list ap;
-  static wxChar buffer[512];
-
-  if (!wxTheApp->GetWantDebugOutput())
-    return ;
-
-  va_start(ap, fmt);
-
-   sprintf(buffer,fmt,ap) ;
+    va_list                         vAp;
+    static wxChar                   zBuffer[512];

 
 

-  va_end(ap);
+    if (!wxTheApp->GetWantDebugOutput())
+        return ;
+    va_start(vAp, zFmt);
+    sprintf(zBuffer, zFmt, vAp) ;
+    va_end(vAp);

 }
 
 // Non-fatal error: pop up message box and (possibly) continue
 }
 
 // Non-fatal error: pop up message box and (possibly) continue

-void wxError(const wxString& msg, const wxString& title)
-{
-  wxSprintf(wxBuffer, "%s\nContinue?", WXSTRINGCAST msg);
-  if (::WinMessageBox( HWND_DESKTOP
-                      ,NULL
-                      ,(PSZ)wxBuffer
-                      ,(PSZ)WXSTRINGCAST title
-                      ,0
-                      ,MB_ICONEXCLAMATION | MB_YESNO
-                     ) == MBID_YES)
+void wxError(
+  const wxString&                   rMsg
+, const wxString&                   rTitle
+)
+{
+    wxSprintf(wxBuffer, "%s\nContinue?", WXSTRINGCAST rMsg);
+    if (::WinMessageBox( HWND_DESKTOP
+                        ,NULL
+                        ,(PSZ)wxBuffer
+                        ,(PSZ)WXSTRINGCAST rTitle
+                        ,0
+                        ,MB_ICONEXCLAMATION | MB_YESNO
+                       ) == MBID_YES)

     wxExit();
 }
 
 // Fatal error: pop up message box and abort
     wxExit();
 }
 
 // Fatal error: pop up message box and abort

-void wxFatalError(const wxString& rMsg, const wxString& rTitle)
+void wxFatalError(
+  const wxString&                   rMsg
+, const wxString&                   rTitle
+)

 {
 {

-    unsigned long                   rc;
+    unsigned long                   ulRc;

 
 

-    rc = ::WinMessageBox( HWND_DESKTOP
-                         ,NULL
-                         ,WXSTRINGCAST rMsg
-                         ,WXSTRINGCAST rTitle
-                         ,0
-                         ,MB_NOICON | MB_OK
-                        );
-    DosExit(EXIT_PROCESS, rc);
+    ulRc = ::WinMessageBox( HWND_DESKTOP
+                           ,NULL
+                           ,WXSTRINGCAST rMsg
+                           ,WXSTRINGCAST rTitle
+                           ,0
+                           ,MB_NOICON | MB_OK
+                          );
+    DosExit(EXIT_PROCESS, ulRc);

 }
 
 // Emit a beeeeeep
 }
 
 // Emit a beeeeeep


@@ -228,93 +266,134 @@ void wxBell()
 
 // Chris Breeze 27/5/98: revised WIN32 code to
 // detect WindowsNT correctly
 
 // Chris Breeze 27/5/98: revised WIN32 code to
 // detect WindowsNT correctly

-int wxGetOsVersion(int *majorVsn, int *minorVsn)
-{
-  ULONG                 aulSysInfo[QSV_MAX] = {0};
-
-  if (DosQuerySysInfo( 1L
-                      ,QSV_MAX
-                      ,(PVOID)aulSysInfo
-                      ,sizeof(ULONG) * QSV_MAX
-                     ))
-  {
-     *majorVsn = aulSysInfo[QSV_VERSION_MAJOR];
-     *minorVsn = aulSysInfo[QSV_VERSION_MINOR];
-     return wxWINDOWS_OS2;
-  }
-  return wxWINDOWS; // error if we get here, return generic value
+int wxGetOsVersion(
+  int*                              pMajorVsn
+, int*                              pMinorVsn
+)
+{
+    ULONG                           ulSysInfo[QSV_MAX] = {0};
+
+    if (::DosQuerySysInfo( 1L
+                          ,QSV_MAX
+                          ,(PVOID)ulSysInfo
+                          ,sizeof(ULONG) * QSV_MAX
+                         ))
+    {
+        *pMajorVsn = ulSysInfo[QSV_VERSION_MAJOR];
+        *pMinorVsn = ulSysInfo[QSV_VERSION_MINOR];
+        return wxWINDOWS_OS2;
+    }
+    return wxWINDOWS; // error if we get here, return generic value

 }
 
 // Reading and writing resources (eg WIN.INI, .Xdefaults)
 }
 
 // Reading and writing resources (eg WIN.INI, .Xdefaults)

-// TODO: Ability to read and write to an INI file
-

 #if wxUSE_RESOURCES
 #if wxUSE_RESOURCES

-bool wxWriteResource(const wxString& section, const wxString& entry, const wxString& value, const wxString& file)
+bool wxWriteResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, const wxString&                   rValue
+, const wxString&                   rFile
+)

 {
     HAB                             hab;
     HINI                            hIni;
 
 {
     HAB                             hab;
     HINI                            hIni;
 

-    if (file != "")
+    if (rFile != "")

     {
     {

-        hIni = ::PrfOpenProfile(hab, (PSZ)WXSTRINGCAST file);
+        hIni = ::PrfOpenProfile(hab, (PSZ)WXSTRINGCAST rFile);

         if (hIni != 0L)
         {
             return (::PrfWriteProfileString( hIni
         if (hIni != 0L)
         {
             return (::PrfWriteProfileString( hIni

-                                            ,(PSZ)WXSTRINGCAST section
-                                            ,(PSZ)WXSTRINGCAST entry
-                                            ,(PSZ)WXSTRINGCAST value
+                                            ,(PSZ)WXSTRINGCAST rSection
+                                            ,(PSZ)WXSTRINGCAST rEntry
+                                            ,(PSZ)WXSTRINGCAST rValue

                                            ));
         }
     }
     else
         return (::PrfWriteProfileString( HINI_PROFILE
                                            ));
         }
     }
     else
         return (::PrfWriteProfileString( HINI_PROFILE

-                                        ,(PSZ)WXSTRINGCAST section
-                                        ,(PSZ)WXSTRINGCAST entry
-                                        ,(PSZ)WXSTRINGCAST value
+                                        ,(PSZ)WXSTRINGCAST rSection
+                                        ,(PSZ)WXSTRINGCAST rEntry
+                                        ,(PSZ)WXSTRINGCAST rValue

                                        ));
     return FALSE;
 }
 
                                        ));
     return FALSE;
 }
 

-bool wxWriteResource(const wxString& section, const wxString& entry, float value, const wxString& file)
+bool wxWriteResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, float                             fValue
+, const wxString&                   rFile
+)

 {
 {

-    wxChar buf[50];
-    wxSprintf(buf, "%.4f", value);
-    return wxWriteResource(section, entry, buf, file);
+    wxChar                          zBuf[50];
+
+    wxSprintf(zBuf, "%.4f", fValue);
+    return wxWriteResource( rSection
+                           ,rEntry
+                           ,zBuf
+                           ,rFile
+                          );

 }
 
 }
 

-bool wxWriteResource(const wxString& section, const wxString& entry, long value, const wxString& file)
+bool wxWriteResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, long                              lValue
+, const wxString&                   rFile
+)

 {
 {

-    wxChar buf[50];
-    wxSprintf(buf, "%ld", value);
-    return wxWriteResource(section, entry, buf, file);
+    wxChar                          zBuf[50];
+
+    wxSprintf(zBuf, "%ld", lValue);
+    return wxWriteResource( rSection
+                           ,rEntry
+                           ,zBuf
+                           ,rFile
+                          );

 }
 
 }
 

-bool wxWriteResource(const wxString& section, const wxString& entry, int value, const wxString& file)
+bool wxWriteResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, int                               lValue
+, const wxString&                   rFile
+)

 {
 {

-    wxChar buf[50];
-    wxSprintf(buf, "%d", value);
-    return wxWriteResource(section, entry, buf, file);
+    wxChar                          zBuf[50];
+
+    wxSprintf(zBuf, "%d", lValue);
+    return wxWriteResource( rSection
+                           ,rEntry
+                           ,zBuf
+                           ,rFile
+                          );

 }
 
 }
 

-bool wxGetResource(const wxString& section, const wxString& entry, wxChar **value, const wxString& file)
+bool wxGetResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, wxChar**                          ppValue
+, const wxString&                   rFile
+)

 {
     HAB                             hab;
     HINI                            hIni;
 {
     HAB                             hab;
     HINI                            hIni;

-    static const wxChar             defunkt[] = _T("$$default");
+    static const wxChar             zDefunkt[] = _T("$$default");

 
 

-    if (file != "")
+    if (rFile != "")

     {
     {

-        hIni = ::PrfOpenProfile(hab, (PSZ)WXSTRINGCAST file);
+        hIni = ::PrfOpenProfile(hab, (PSZ)WXSTRINGCAST rFile);

         if (hIni != 0L)
         {
             ULONG n = ::PrfQueryProfileString( hIni
         if (hIni != 0L)
         {
             ULONG n = ::PrfQueryProfileString( hIni

-                                              ,(PSZ)WXSTRINGCAST section
-                                              ,(PSZ)WXSTRINGCAST entry
-                                              ,(PSZ)defunkt
+                                              ,(PSZ)WXSTRINGCAST rSection
+                                              ,(PSZ)WXSTRINGCAST rEntry
+                                              ,(PSZ)zDefunkt

                                               ,(void*)wxBuffer
                                               ,1000
                                              );
                                               ,(void*)wxBuffer
                                               ,1000
                                              );

-            if (n == 0L || wxStrcmp(wxBuffer, defunkt) == 0)
+            if (n == 0L || wxStrcmp(wxBuffer, zDefunkt) == 0)

                 return FALSE;
         }
         else
                 return FALSE;
         }
         else


@@ -323,58 +402,88 @@ bool wxGetResource(const wxString& section, const wxString& entry, wxChar **valu
     else
     {
         ULONG n = ::PrfQueryProfileString( HINI_PROFILE
     else
     {
         ULONG n = ::PrfQueryProfileString( HINI_PROFILE

-                                          ,(PSZ)WXSTRINGCAST section
-                                          ,(PSZ)WXSTRINGCAST entry
-                                          ,(PSZ)defunkt
+                                          ,(PSZ)WXSTRINGCAST rSection
+                                          ,(PSZ)WXSTRINGCAST rEntry
+                                          ,(PSZ)zDefunkt

                                           ,(void*)wxBuffer
                                           ,1000
                                          );
                                           ,(void*)wxBuffer
                                           ,1000
                                          );

-        if (n == 0L || wxStrcmp(wxBuffer, defunkt) == 0)
+        if (n == 0L || wxStrcmp(wxBuffer, zDefunkt) == 0)

             return FALSE;
     }
             return FALSE;
     }

-    if (*value)
-        delete[] (*value);
-    *value = copystring(wxBuffer);
+    if (*ppValue)
+        delete[] (*ppValue);
+    *ppValue = copystring(wxBuffer);

     return TRUE;
 }
 
     return TRUE;
 }
 

-bool wxGetResource(const wxString& section, const wxString& entry, float *value, const wxString& file)
+bool wxGetResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, float*                            pValue
+, const wxString&                   rFile
+)

 {
 {

-  wxChar *s = NULL;
-  bool succ = wxGetResource(section, entry, (wxChar **)&s, file);
-  if (succ)
-  {
-    *value = (float)wxStrtod(s, NULL);
-    delete[] s;
-    return TRUE;
-  }
-  else return FALSE;
+    wxChar*                         zStr = NULL;
+    bool                            bSucc = wxGetResource( rSection
+                                                          ,rEntry
+                                                          ,(wxChar **)&zStr
+                                                          ,rFile
+                                                         );
+
+    if (bSucc)
+    {
+        *pValue = (float)wxStrtod(zStr, NULL);
+        delete[] zStr;
+        return TRUE;
+    }
+    else return FALSE;

 }
 
 }
 

-bool wxGetResource(const wxString& section, const wxString& entry, long *value, const wxString& file)
+bool wxGetResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, long*                             pValue
+, const wxString&                   rFile
+)

 {
 {

-  wxChar *s = NULL;
-  bool succ = wxGetResource(section, entry, (wxChar **)&s, file);
-  if (succ)
-  {
-    *value = wxStrtol(s, NULL, 10);
-    delete[] s;
-    return TRUE;
-  }
-  else return FALSE;
+    wxChar*                           zStr = NULL;
+    bool                              bSucc = wxGetResource( rSection
+                                                            ,rEntry
+                                                            ,(wxChar **)&zStr
+                                                            ,rFile
+                                                           );
+
+    if (bSucc)
+    {
+        *pValue = wxStrtol(zStr, NULL, 10);
+        delete[] zStr;
+        return TRUE;
+    }
+    else return FALSE;

 }
 
 }
 

-bool wxGetResource(const wxString& section, const wxString& entry, int *value, const wxString& file)
+bool wxGetResource(
+  const wxString&                   rSection
+, const wxString&                   rEntry
+, int*                              pValue
+, const wxString&                   rFile
+)

 {
 {

-  wxChar *s = NULL;
-  bool succ = wxGetResource(section, entry, (wxChar **)&s, file);
-  if (succ)
-  {
-    *value = (int)wxStrtol(s, NULL, 10);
-    delete[] s;
-    return TRUE;
-  }
-  else return FALSE;
+    wxChar*                         zStr = NULL;
+    bool                            bSucc = wxGetResource( rSection
+                                                          ,rEntry
+                                                          ,(wxChar **)&zStr
+                                                          ,rFile
+                                                         );
+
+    if (bSucc)
+    {
+        *pValue = (int)wxStrtol(zStr, NULL, 10);
+        delete[] zStr;
+        return TRUE;
+    }
+    else return FALSE;

 }
 #endif // wxUSE_RESOURCES
 
 }
 #endif // wxUSE_RESOURCES
 


@@ -387,11 +496,13 @@ HCURSOR gs_wxBusyCursorOld = 0;  // old cursor
 static int gs_wxBusyCursorCount = 0;
 
 // Set the cursor to the busy cursor for all windows
 static int gs_wxBusyCursorCount = 0;
 
 // Set the cursor to the busy cursor for all windows

-void wxBeginBusyCursor(wxCursor *cursor)
+void wxBeginBusyCursor(
+  wxCursor*                         pCursor
+)

 {
     if ( gs_wxBusyCursorCount++ == 0 )
     {
 {
     if ( gs_wxBusyCursorCount++ == 0 )
     {

-        gs_wxBusyCursor = (HCURSOR)cursor->GetHCURSOR();
+        gs_wxBusyCursor = (HCURSOR)pCursor->GetHCURSOR();

         ::WinSetPointer(HWND_DESKTOP, (HPOINTER)gs_wxBusyCursor);
     }
     //else: nothing to do, already set
         ::WinSetPointer(HWND_DESKTOP, (HPOINTER)gs_wxBusyCursor);
     }
     //else: nothing to do, already set


@@ -400,10 +511,11 @@ void wxBeginBusyCursor(wxCursor *cursor)
 // Restore cursor to normal
 void wxEndBusyCursor()
 {
 // Restore cursor to normal
 void wxEndBusyCursor()
 {

-    wxCHECK_RET( gs_wxBusyCursorCount > 0,
-                 _T("no matching wxBeginBusyCursor() for wxEndBusyCursor()"));
+    wxCHECK_RET( gs_wxBusyCursorCount > 0
+                ,_T("no matching wxBeginBusyCursor() for wxEndBusyCursor()")
+               );

 
 

-    if ( --gs_wxBusyCursorCount == 0 )
+    if (--gs_wxBusyCursorCount == 0)

     {
         ::WinSetPointer(HWND_DESKTOP, (HPOINTER)gs_wxBusyCursorOld);
         gs_wxBusyCursorOld = 0;
     {
         ::WinSetPointer(HWND_DESKTOP, (HPOINTER)gs_wxBusyCursorOld);
         gs_wxBusyCursorOld = 0;


@@ -413,218 +525,209 @@ void wxEndBusyCursor()
 // TRUE if we're between the above two calls
 bool wxIsBusy()
 {
 // TRUE if we're between the above two calls
 bool wxIsBusy()
 {

-  return (gs_wxBusyCursorCount > 0);
+    return (gs_wxBusyCursorCount > 0);

 }
 
 // ---------------------------------------------------------------------------
 }
 
 // ---------------------------------------------------------------------------

-const wxChar* wxGetHomeDir(wxString *pstr)
+const wxChar* wxGetHomeDir(
+  wxString*                         pStr
+)

 {
 {

-  wxString& strDir = *pstr;
+    wxString&                       rStrDir = *pStr;

 
 

-  // OS/2 has no idea about home,
-  // so use the working directory instead?
+    // OS/2 has no idea about home,
+    // so use the working directory instead?

 
 

-  // 256 was taken from os2def.h
+    // 256 was taken from os2def.h

 #ifndef MAX_PATH
 #  define MAX_PATH  256
 #endif
 
 #ifndef MAX_PATH
 #  define MAX_PATH  256
 #endif
 

+    char                            zDirName[256];
+    ULONG                           ulDirLen;

 
 

-    char     DirName[256];
-    ULONG    DirLen;
-
-    ::DosQueryCurrentDir( 0, DirName, &DirLen);
-    strDir = DirName;
-  return strDir.c_str();
+    ::DosQueryCurrentDir(0, zDirName, &ulDirLen);
+    rStrDir = zDirName;
+    return rStrDir.c_str();

 }
 
 }
 

-// Hack for MS-DOS
-wxChar *wxGetUserHome (const wxString& user)
+// Hack for OS/2
+wxChar* wxGetUserHome (
+  const wxString&                   rUser
+)

 {
 {

-  wxChar *home;
-  wxString user1(user);
+    wxChar*                         zHome;
+    wxString                        sUser1(rUser);

 
 

-  if (user1 != _T("")) {
-    wxChar tmp[64];
-    if (wxGetUserId(tmp, sizeof(tmp)/sizeof(char))) {
-      // Guests belong in the temp dir
-      if (wxStricmp(tmp, _T("annonymous")) == 0) {
-        if ((home = wxGetenv(_T("TMP"))) != NULL ||
-            (home = wxGetenv(_T("TMPDIR"))) != NULL ||
-            (home = wxGetenv(_T("TEMP"))) != NULL)
-            return *home ? home : (wxChar*)_T("\\");
-      }
-      if (wxStricmp(tmp, WXSTRINGCAST user1) == 0)
-        user1 = _T("");
-    }
-  }
-  if (user1 == _T(""))
-    if ((home = wxGetenv(_T("HOME"))) != NULL)
+    if (sUser1 != _T(""))

     {
     {

-      wxStrcpy(wxBuffer, home);
-      Unix2DosFilename(wxBuffer);
-      return wxBuffer;
+        wxChar                      zTmp[64];
+
+        if (wxGetUserId( zTmp
+                        ,sizeof(zTmp)/sizeof(char)
+                       ))
+        {
+            // Guests belong in the temp dir
+            if (wxStricmp(zTmp, _T("annonymous")) == 0)
+            {
+                if ((zHome = wxGetenv(_T("TMP"))) != NULL    ||
+                    (zHome = wxGetenv(_T("TMPDIR"))) != NULL ||
+                    (zHome = wxGetenv(_T("TEMP"))) != NULL)
+                    return *zHome ? zHome : (wxChar*)_T("\\");
+            }
+            if (wxStricmp(zTmp, WXSTRINGCAST sUser1) == 0)
+                sUser1 = _T("");
+        }

     }
     }

-  return NULL; // No home known!
+    if (sUser1 == _T(""))
+        if ((zHome = wxGetenv(_T("HOME"))) != NULL)
+        {
+            wxStrcpy(wxBuffer, zHome);
+            Unix2DosFilename(wxBuffer);
+            return wxBuffer;
+        }
+        return NULL; // No home known!

 }
 
 // Check whether this window wants to process messages, e.g. Stop button
 // in long calculations.
 }
 
 // Check whether this window wants to process messages, e.g. Stop button
 // in long calculations.

-bool wxCheckForInterrupt(wxWindow *wnd)
+bool wxCheckForInterrupt(
+  wxWindow*                         pWnd
+)

 {
 {

-  if(wnd){
-    QMSG            msg;
-    HAB             hab;
-    HWND            hwndFilter;
+    if(pWnd)
+    {
+        QMSG                        vMsg;
+        HAB                         hab;
+        HWND                        hwndFilter;
+        HWND                        hwndWin= (HWND) pWnd->GetHWND();

 
 

-    HWND win= (HWND) wnd->GetHWND();
-    while(::WinPeekMsg(hab,&msg,hwndFilter,0,0,PM_REMOVE))
+        while(::WinPeekMsg(hab, &vMsg, hwndFilter, 0, 0, PM_REMOVE))
+        {
+            ::WinDispatchMsg(hab, &vMsg);
+        }
+        return TRUE;//*** temporary?
+    }
+    else

     {
     {

-      ::WinDispatchMsg( hab, &msg );
+        wxFAIL_MSG(_T("pWnd==NULL !!!"));
+        return FALSE;//*** temporary?

     }
     }

-    return TRUE;//*** temporary?
-  }
-  else{
-    wxFAIL_MSG(_T("wnd==NULL !!!"));
-
-    return FALSE;//*** temporary?
-  }
-}
-
-wxChar *wxLoadUserResource(const wxString& resourceName, const wxString& resourceType)
-{
-  wxChar *s = NULL;
-
-/*
-*  How to in PM?
-*
-* #if !defined(__WIN32__) || defined(__TWIN32__)
-*   HRSRC hResource = ::FindResource(wxGetInstance(), WXSTRINGCAST resourceName, WXSTRINGCAST resourceType);
-* #else
-* #ifdef UNICODE
-*   HRSRC hResource = ::FindResourceW(wxGetInstance(), WXSTRINGCAST resourceName, WXSTRINGCAST resourceType);
-* #else
-*   HRSRC hResource = ::FindResourceA(wxGetInstance(), WXSTRINGCAST resourceName, WXSTRINGCAST resourceType);
-* #endif
-* #endif
-*
-*   if (hResource == 0)
-*     return NULL;
-*   HGLOBAL hData = ::LoadResource(wxGetInstance(), hResource);
-*   if (hData == 0)
-*     return NULL;
-*   wxChar *theText = (wxChar *)LockResource(hData);
-*   if (!theText)
-*     return NULL;
-*
-* s = copystring(theText);
-*/
-  return s;
-}
-
-void wxGetMousePosition( int* x, int* y )
-{
-    POINTL pt;
-    ::WinQueryPointerPos( HWND_DESKTOP, & pt );
-    *x = pt.x;
-    *y = pt.y;
+}
+
+void wxGetMousePosition(
+  int*                              pX
+, int*                              pY
+)
+{
+    POINTL                          vPt;
+
+    ::WinQueryPointerPos(HWND_DESKTOP, &vPt);
+    *pX = vPt.x;
+    *pY = vPt.y;

 };
 
 // Return TRUE if we have a colour display
 bool wxColourDisplay()
 {
 };
 
 // Return TRUE if we have a colour display
 bool wxColourDisplay()
 {

-    bool flag;
-    // TODO:  use DosQueryDevCaps to figure it out
-    return flag;
+    HPS                             hpsScreen;
+    HDC                             hdcScreen;
+    LONG                            lColors;
+
+    hpsScreen = ::WinGetScreenPS(HWND_DESKTOP);
+    hdcScreen = ::GpiQueryDevice(hpsScreen);
+    ::DevQueryCaps(hdcScreen, CAPS_COLORS, 1L, &lColors);
+    return(lColors > 1L);

 }
 
 // Returns depth of screen
 int wxDisplayDepth()
 {
 }
 
 // Returns depth of screen
 int wxDisplayDepth()
 {

-    HDC                             hDc = ::WinOpenWindowDC((HWND)NULL);
-    long                            lArray[CAPS_COLOR_BITCOUNT];
-    int                             nPlanes;
-    int                             nBitsPerPixel;
-    int                             nDepth;
-
-    if(DevQueryCaps( hDc
-                    ,CAPS_FAMILY
-                    ,CAPS_COLOR_BITCOUNT
-                    ,lArray
-                   ))
-    {
-        nPlanes = (int)lArray[CAPS_COLOR_PLANES];
-        nBitsPerPixel = (int)lArray[CAPS_COLOR_BITCOUNT];
-        nDepth = nPlanes * nBitsPerPixel;
-    }
-    DevCloseDC(hDc);
+    HPS                             hpsScreen;
+    HDC                             hdcScreen;
+    LONG                            lPlanes;
+    LONG                            lBitsPerPixel;
+    LONG                            nDepth;
+
+    hpsScreen = ::WinGetScreenPS(HWND_DESKTOP);
+    hdcScreen = ::GpiQueryDevice(hpsScreen);
+    ::DevQueryCaps(hdcScreen, CAPS_COLOR_PLANES, 1L, &lPlanes);
+    ::DevQueryCaps(hdcScreen, CAPS_COLOR_BITCOUNT, 1L, &lBitsPerPixel);
+
+    nDepth = (int)(lPlanes * lBitsPerPixel);
+    DevCloseDC(hdcScreen);

     return (nDepth);
 }
 
 // Get size of display
     return (nDepth);
 }
 
 // Get size of display

-void wxDisplaySize(int *width, int *height)
+void wxDisplaySize(
+  int*                              pWidth
+, int*                              pHeight
+)

 {
 {

-    HDC                             hDc = ::WinOpenWindowDC((HWND)NULL);
-    long                            lArray[CAPS_HEIGHT];
+    HPS                             hpsScreen;
+    HDC                             hdcScreen;

 
 

-    if(DevQueryCaps( hDc
-                    ,CAPS_FAMILY
-                    ,CAPS_HEIGHT
-                    ,lArray
-                   ))
-    {
-        *width  = (int)lArray[CAPS_WIDTH];
-        *height = (int)lArray[CAPS_HEIGHT];
-    }
-    DevCloseDC(hDc);
+    hpsScreen = ::WinGetScreenPS(HWND_DESKTOP);
+    hdcScreen = ::GpiQueryDevice(hpsScreen);
+    ::DevQueryCaps(hdcScreen, CAPS_WIDTH, 1L, (PLONG)pWidth);
+    ::DevQueryCaps(hdcScreen, CAPS_HEIGHT, 1L, (PLONG)pHeight);
+    DevCloseDC(hdcScreen);

 }
 
 }
 

-bool wxDirExists(const wxString& dir)
+bool wxDirExists(
+  const wxString&                   rDir
+)

 {
 {

-   // TODO:  Control program file stuff
-   return TRUE;
+    return (::DosSetCurrentDir(WXSTRINGCAST rDir));

 }
 
 // ---------------------------------------------------------------------------
 // window information functions
 // ---------------------------------------------------------------------------
 
 }
 
 // ---------------------------------------------------------------------------
 // window information functions
 // ---------------------------------------------------------------------------
 

-wxString WXDLLEXPORT wxGetWindowText(WXHWND hWnd)
+wxString WXDLLEXPORT wxGetWindowText(
+  WXHWND                            hWnd
+)

 {
 {

-    wxString str;
-    long len = ::WinQueryWindowTextLength((HWND)hWnd) + 1;
-    ::WinQueryWindowText((HWND)hWnd, len, str.GetWriteBuf((int)len));
-    str.UngetWriteBuf();
+    wxString                        vStr;
+    long                            lLen = ::WinQueryWindowTextLength((HWND)hWnd) + 1;
+
+    ::WinQueryWindowText((HWND)hWnd, lLen, vStr.GetWriteBuf((int)lLen));
+    vStr.UngetWriteBuf();

 
 

-    return str;
+    return vStr;

 }
 
 }
 

-wxString WXDLLEXPORT wxGetWindowClass(WXHWND hWnd)
+wxString WXDLLEXPORT wxGetWindowClass(
+  WXHWND                            hWnd
+)

 {
 {

-    wxString str;
-
-    int len = 256; // some starting value
+    wxString                        vStr;
+    int                             nLen = 256; // some starting value

 
     for ( ;; )
     {
 
     for ( ;; )
     {

-        int count = ::WinQueryClassName((HWND)hWnd, len, str.GetWriteBuf(len));
+        int                         nCount = ::WinQueryClassName((HWND)hWnd, nLen, vStr.GetWriteBuf(nLen));

 
 

-        str.UngetWriteBuf();
-        if ( count == len )
+        vStr.UngetWriteBuf();
+        if (nCount == nLen )

         {
             // the class name might have been truncated, retry with larger
             // buffer
         {
             // the class name might have been truncated, retry with larger
             // buffer

-            len *= 2;
+            nLen *= 2;

         }
         else
         {
             break;
         }
     }
         }
         else
         {
             break;
         }
     }

-    return str;
+    return vStr;

 }
 
 }
 

-WXWORD WXDLLEXPORT wxGetWindowId(WXHWND hWnd)
+WXWORD WXDLLEXPORT wxGetWindowId(
+  WXHWND                            hWnd
+)

 {
     return ::WinQueryWindowUShort((HWND)hWnd, QWS_ID);
 }
 {
     return ::WinQueryWindowUShort((HWND)hWnd, QWS_ID);
 }