]> git.saurik.com Git - wxWidgets.git/blobdiff - src/common/utilscmn.cpp
Also allow key events for Shift-Tab when wxWANTS_CHARS style is used
[wxWidgets.git] / src / common / utilscmn.cpp
index ae8c7e79dc1349becec4e1f21bc2bde72cae63db..449bafe65fea735a240a8776d68455f8c0dd87dc 100644 (file)
 // headers
 // ----------------------------------------------------------------------------
 
 // headers
 // ----------------------------------------------------------------------------
 
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA) && !defined(__EMX__)
+// Some older compilers (such as EMX) cannot handle
+// #pragma interface/implementation correctly, iff 
+// #pragma implementation is used in _two_ translation
+// units (as created by e.g. event.cpp compiled for
+// libwx_base and event.cpp compiled for libwx_gui_core).
+// So we must not use those pragmas for those compilers in
+// such files.
     #pragma implementation "utils.h"
 #endif
 
     #pragma implementation "utils.h"
 #endif
 
 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP
-    #include "wx/defs.h"
+    #include "wx/app.h"
     #include "wx/string.h"
     #include "wx/utils.h"
     #include "wx/intl.h"
     #include "wx/log.h"
 
     #if wxUSE_GUI
     #include "wx/string.h"
     #include "wx/utils.h"
     #include "wx/intl.h"
     #include "wx/log.h"
 
     #if wxUSE_GUI
-        #include "wx/app.h"
         #include "wx/window.h"
         #include "wx/frame.h"
         #include "wx/menu.h"
         #include "wx/window.h"
         #include "wx/frame.h"
         #include "wx/menu.h"
     #endif // wxUSE_GUI
 #endif // WX_PRECOMP
 
     #endif // wxUSE_GUI
 #endif // WX_PRECOMP
 
-#ifndef __WIN16__
+#include "wx/apptrait.h"
+
 #include "wx/process.h"
 #include "wx/txtstrm.h"
 #include "wx/process.h"
 #include "wx/txtstrm.h"
+
+#if defined(__WXWINCE__) && wxUSE_DATETIME
+#include "wx/datetime.h"
 #endif
 
 #include <ctype.h>
 #endif
 
 #include <ctype.h>
     #include "wx/statusbr.h"
 #endif // wxUSE_GUI
 
     #include "wx/statusbr.h"
 #endif // wxUSE_GUI
 
+#ifndef __WXWINCE__
 #include <time.h>
 #include <time.h>
+#else
+#include "wx/msw/wince/time.h"
+#endif
 
 
-#ifndef __MWERKS__
+#if !defined(__MWERKS__) && !defined(__WXWINCE__)
     #include <sys/types.h>
     #include <sys/stat.h>
 #endif
 
     #include <sys/types.h>
     #include <sys/stat.h>
 #endif
 
-#ifdef __SALFORDC__
-    #include <clib.h>
-#endif
-
 #ifdef __WXMSW__
     #include "wx/msw/private.h"
 #endif
 
 #ifdef __WXMSW__
     #include "wx/msw/private.h"
 #endif
 
+#if wxUSE_BASE
+
 // ----------------------------------------------------------------------------
 // common data
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // common data
 // ----------------------------------------------------------------------------
 // implementation
 // ============================================================================
 
 // implementation
 // ============================================================================
 
+#if WXWIN_COMPATIBILITY_2_4
+
 wxChar *
 copystring (const wxChar *s)
 {
 wxChar *
 copystring (const wxChar *s)
 {
@@ -114,24 +128,16 @@ copystring (const wxChar *s)
   return news;
 }
 
   return news;
 }
 
-// Id generation
-static long wxCurrentId = 100;
+#endif // WXWIN_COMPATIBILITY_2_4
 
 
-long
-wxNewId (void)
-{
-  return wxCurrentId++;
-}
+// ----------------------------------------------------------------------------
+// String <-> Number conversions (deprecated)
+// ----------------------------------------------------------------------------
 
 
-long
-wxGetCurrentId(void) { return wxCurrentId; }
+#if WXWIN_COMPATIBILITY_2_4
 
 
-void
-wxRegisterId (long id)
-{
-  if (id >= wxCurrentId)
-    wxCurrentId = id + 1;
-}
+WXDLLIMPEXP_DATA_BASE(const wxChar *) wxFloatToStringStr = wxT("%.2f");
+WXDLLIMPEXP_DATA_BASE(const wxChar *) wxDoubleToStringStr = wxT("%.2f");
 
 void
 StringToFloat (const wxChar *s, float *number)
 
 void
 StringToFloat (const wxChar *s, float *number)
@@ -197,6 +203,8 @@ LongToString (long number)
   return buf;
 }
 
   return buf;
 }
 
+#endif // WXWIN_COMPATIBILITY_2_4
+
 // Array used in DecToHex conversion routine.
 static wxChar hexArray[] = wxT("0123456789ABCDEF");
 
 // Array used in DecToHex conversion routine.
 static wxChar hexArray[] = wxT("0123456789ABCDEF");
 
@@ -236,938 +244,693 @@ wxString wxDecToHex(int dec)
     return wxString(buf);
 }
 
     return wxString(buf);
 }
 
-#if WXWIN_COMPATIBILITY_2
-bool
-StringMatch (const wxChar *str1, const wxChar *str2, bool subString, bool exact)
-{
-  if (str1 == NULL || str2 == NULL)
-    return FALSE;
-  if (str1 == str2)
-    return TRUE;
-
-  if (subString)
-    {
-      int len1 = wxStrlen (str1);
-      int len2 = wxStrlen (str2);
-      int i;
-
-      // Search for str1 in str2
-      // Slow .... but acceptable for short strings
-      for (i = 0; i <= len2 - len1; i++)
-        {
-          if (wxStrnicmp (str1, str2 + i, len1) == 0)
-            return TRUE;
-        }
-    }
-  else if (exact)
-    {
-      if (wxStricmp (str1, str2) == 0)
-        return TRUE;
-    }
-  else
-    {
-      int len1 = wxStrlen (str1);
-      int len2 = wxStrlen (str2);
-
-      if (wxStrnicmp (str1, str2, wxMin (len1, len2)) == 0)
-        return TRUE;
-    }
-
-  return FALSE;
-}
-#endif
+// ----------------------------------------------------------------------------
+// misc functions
+// ----------------------------------------------------------------------------
 
 // Return the current date/time
 
 // Return the current date/time
-// [volatile]
 wxString wxNow()
 {
 wxString wxNow()
 {
+#ifdef __WXWINCE__
+#if wxUSE_DATETIME
+    wxDateTime now = wxDateTime::Now();
+    return now.Format();
+#else
+    return wxEmptyString;
+#endif
+#else
     time_t now = time((time_t *) NULL);
     char *date = ctime(&now);
     date[24] = '\0';
     return wxString::FromAscii(date);
     time_t now = time((time_t *) NULL);
     char *date = ctime(&now);
     date[24] = '\0';
     return wxString::FromAscii(date);
+#endif
 }
 
 }
 
-#if wxUSE_GUI
+const wxChar *wxGetInstallPrefix()
+{
+    wxString prefix;
 
 
-#if wxUSE_MENUS
+    if ( wxGetEnv(wxT("WXPREFIX"), &prefix) )
+        return prefix.c_str();
+
+#ifdef wxINSTALL_PREFIX
+    return wxT(wxINSTALL_PREFIX);
+#else
+    return wxT("");
+#endif
+}
+
+wxString wxGetDataDir()
+{
+    wxString format = wxGetInstallPrefix();
+    format <<  wxFILE_SEP_PATH
+           << wxT("share") << wxFILE_SEP_PATH
+           << wxT("wx") << wxFILE_SEP_PATH
+           << wxT("%i.%i");
+    wxString dir;
+    dir.Printf(format.c_str(), wxMAJOR_VERSION, wxMINOR_VERSION);
+    return dir;
+}
+
+int wxGetOsVersion(int *verMaj, int *verMin)
+{
+    // we want this function to work even if there is no wxApp
+    wxConsoleAppTraits traitsConsole;
+    wxAppTraits *traits = wxTheApp ? wxTheApp->GetTraits() : NULL;
+    if ( ! traits )
+        traits = &traitsConsole;
+
+    wxToolkitInfo& info = traits->GetToolkitInfo();
+    if ( verMaj )
+        *verMaj = info.versionMajor;
+    if ( verMin )
+        *verMin = info.versionMinor;
+    return info.os;
+}
 
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
-// Menu accelerators related functions
+// network and user id functions
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 
-wxChar *wxStripMenuCodes(const wxChar *in, wxChar *out)
+// Get Full RFC822 style email address
+bool wxGetEmailAddress(wxChar *address, int maxSize)
 {
 {
-    wxString s = wxMenuItem::GetLabelFromText(in);
-    if ( out )
-    {
-        // go smash their buffer if it's not big enough - I love char * params
-        memcpy(out, s.c_str(), s.length() * sizeof(wxChar));
-    }
-    else
-    {
-        out = copystring(s);
-    }
+    wxString email = wxGetEmailAddress();
+    if ( !email )
+        return FALSE;
 
 
-    return out;
+    wxStrncpy(address, email, maxSize - 1);
+    address[maxSize - 1] = wxT('\0');
+
+    return TRUE;
 }
 
 }
 
-wxString wxStripMenuCodes(const wxString& in)
+wxString wxGetEmailAddress()
 {
 {
-    wxString out;
-
-    size_t len = in.length();
-    out.reserve(len);
+    wxString email;
 
 
-    for ( size_t n = 0; n < len; n++ )
+    wxString host = wxGetFullHostName();
+    if ( !!host )
     {
     {
-        wxChar ch = in[n];
-        if ( ch == _T('&') )
-        {
-            // skip it, it is used to introduce the accel char (or to quote
-            // itself in which case it should still be skipped): note that it
-            // can't be the last character of the string
-            if ( ++n == len )
-            {
-                wxLogDebug(_T("Invalid menu string '%s'"), in.c_str());
-            }
-            else
-            {
-                // use the next char instead
-                ch = in[n];
-            }
-        }
-        else if ( ch == _T('\t') )
+        wxString user = wxGetUserId();
+        if ( !!user )
         {
         {
-            // everything after TAB is accel string, exit the loop
-            break;
+            email << user << wxT('@') << host;
         }
         }
-
-        out += ch;
     }
 
     }
 
-    return out;
+    return email;
 }
 
 }
 
-#endif // wxUSE_MENUS
+wxString wxGetUserId()
+{
+    static const int maxLoginLen = 256; // FIXME arbitrary number
 
 
-// ----------------------------------------------------------------------------
-// Window search functions
-// ----------------------------------------------------------------------------
+    wxString buf;
+    bool ok = wxGetUserId(wxStringBuffer(buf, maxLoginLen), maxLoginLen);
 
 
-/*
- * If parent is non-NULL, look through children for a label or title
- * matching the specified string. If NULL, look through all top-level windows.
- *
- */
+    if ( !ok )
+        buf.Empty();
 
 
-wxWindow *
-wxFindWindowByLabel (const wxString& title, wxWindow * parent)
-{
-    return wxWindow::FindWindowByLabel( title, parent );
+    return buf;
 }
 
 }
 
+wxString wxGetUserName()
+{
+    static const int maxUserNameLen = 1024; // FIXME arbitrary number
+
+    wxString buf;
+    bool ok = wxGetUserName(wxStringBuffer(buf, maxUserNameLen), maxUserNameLen);
 
 
-/*
- * If parent is non-NULL, look through children for a name
- * matching the specified string. If NULL, look through all top-level windows.
- *
- */
+    if ( !ok )
+        buf.Empty();
 
 
-wxWindow *
-wxFindWindowByName (const wxString& name, wxWindow * parent)
-{
-    return wxWindow::FindWindowByName( name, parent );
+    return buf;
 }
 
 }
 
-// Returns menu item id or -1 if none.
-int
-wxFindMenuItemId (wxFrame * frame, const wxString& menuString, const wxString& itemString)
+wxString wxGetHostName()
 {
 {
-#if wxUSE_MENUS
-  wxMenuBar *menuBar = frame->GetMenuBar ();
-  if ( menuBar )
-      return menuBar->FindMenuItem (menuString, itemString);
-#endif // wxUSE_MENUS
+    static const size_t hostnameSize = 257;
 
 
-  return -1;
+    wxString buf;
+    bool ok = wxGetHostName(wxStringBuffer(buf, hostnameSize), hostnameSize);
+
+    if ( !ok )
+        buf.Empty();
+
+    return buf;
 }
 
 }
 
-// Try to find the deepest child that contains 'pt'.
-// We go backwards, to try to allow for controls that are spacially
-// within other controls, but are still siblings (e.g. buttons within
-// static boxes). Static boxes are likely to be created _before_ controls
-// that sit inside them.
-wxWindow* wxFindWindowAtPoint(wxWindow* win, const wxPoint& pt)
+wxString wxGetFullHostName()
 {
 {
-    if (!win->IsShown())
-        return NULL;
+    static const size_t hostnameSize = 257;
 
 
-    // Hack for wxNotebook case: at least in wxGTK, all pages
-    // claim to be shown, so we must only deal with the selected one.
-#if wxUSE_NOTEBOOK
-    if (win->IsKindOf(CLASSINFO(wxNotebook)))
-    {
-      wxNotebook* nb = (wxNotebook*) win;
-      int sel = nb->GetSelection();
-      if (sel >= 0)
-      {
-        wxWindow* child = nb->GetPage(sel);
-        wxWindow* foundWin = wxFindWindowAtPoint(child, pt);
-        if (foundWin)
-           return foundWin;
-      }
-    }
-#endif
+    wxString buf;
+    bool ok = wxGetFullHostName(wxStringBuffer(buf, hostnameSize), hostnameSize);
 
 
-    /* Doesn't work
-    // Frame case
-    else if (win->IsKindOf(CLASSINFO(wxFrame)))
-    {
-      // Pseudo-children that may not be mentioned in the child list
-      wxWindowList extraChildren;
-      wxFrame* frame = (wxFrame*) win;
-      if (frame->GetStatusBar())
-        extraChildren.Append(frame->GetStatusBar());
-      if (frame->GetToolBar())
-        extraChildren.Append(frame->GetToolBar());
-
-      wxNode* node = extraChildren.GetFirst();
-      while (node)
-      {
-          wxWindow* child = (wxWindow*) node->GetData();
-          wxWindow* foundWin = wxFindWindowAtPoint(child, pt);
-          if (foundWin)
-            return foundWin;
-          node = node->Next();
-      }
-    }
-    */
+    if ( !ok )
+        buf.Empty();
 
 
-    wxWindowList::Node  *node = win->GetChildren().GetLast();
-    while (node)
-    {
-        wxWindow* child = node->GetData();
-        wxWindow* foundWin = wxFindWindowAtPoint(child, pt);
-        if (foundWin)
-          return foundWin;
-        node = node->GetPrevious();
-    }
+    return buf;
+}
 
 
-    wxPoint pos = win->GetPosition();
-    wxSize sz = win->GetSize();
-    if (win->GetParent())
-    {
-        pos = win->GetParent()->ClientToScreen(pos);
-    }
+wxString wxGetHomeDir()
+{
+    wxString home;
+    wxGetHomeDir(&home);
 
 
-    wxRect rect(pos, sz);
-    if (rect.Inside(pt))
-        return win;
-    else
-        return NULL;
+    return home;
 }
 
 }
 
-wxWindow* wxGenericFindWindowAtPoint(const wxPoint& pt)
+#if 0
+
+wxString wxGetCurrentDir()
 {
 {
-    // Go backwards through the list since windows
-    // on top are likely to have been appended most
-    // recently.
-    wxWindowList::Node  *node = wxTopLevelWindows.GetLast();
-    while (node)
+    wxString dir;
+    size_t len = 1024;
+    bool ok;
+    do
     {
     {
-        wxWindow* win = node->GetData();
-        wxWindow* found = wxFindWindowAtPoint(win, pt);
-        if (found)
-            return found;
-        node = node->GetPrevious();
-    }
-    return NULL;
-}
-
-#endif // wxUSE_GUI
-
-/*
-On Fri, 21 Jul 1995, Paul Craven wrote:
-
-> Is there a way to find the path of running program's executable? I can get
-> my home directory, and the current directory, but I don't know how to get the
-> executable directory.
->
-
-The code below (warty as it is), does what you want on most Unix,
-DOS, and Mac platforms (it's from the ALS Prolog main).
-
-|| Ken Bowen      Applied Logic Systems, Inc.         PO Box 180,
-||====            Voice:  +1 (617)965-9191            Newton Centre,
-||                FAX:    +1 (617)965-1636            MA  02159  USA
-                  Email:  ken@als.com        WWW: http://www.als.com
-------------------------------------------------------------------------
-*/
-
-// This code is commented out but it may be integrated with wxWin at
-// a later date, after testing. Thanks Ken!
-#if 0
+        ok = getcwd(dir.GetWriteBuf(len + 1), len) != NULL;
+        dir.UngetWriteBuf();
 
 
-/*--------------------------------------------------------------------*
- | whereami is given a filename f in the form:  whereami(argv[0])
- | It returns the directory in which the executable file (containing
- | this code [main.c] ) may be found.  A dot will be returned to indicate
- | the current directory.
- *--------------------------------------------------------------------*/
+        if ( !ok )
+        {
+            if ( errno != ERANGE )
+            {
+                wxLogSysError(_T("Failed to get current directory"));
 
 
-static void
-whereami(name)
-    char *name;
-{
-    register char *cutoff = NULL;        /* stifle -Wall */
-    register char *s;
-    register char *t;
-    int   cc;
-    char  ebuf[4096];
-
-    /*
-     * See if the file is accessible either through the current directory
-     * or through an absolute path.
-     */
-
-    if (access(name, R_OK) == 0) {
-
-        /*-------------------------------------------------------------*
-         * The file was accessible without any other work.  But the current
-         * working directory might change on us, so if it was accessible
-         * through the cwd, then we should get it for later accesses.
-         *-------------------------------------------------------------*/
-
-        t = imagedir;
-        if (!absolute_pathname(name)) {
-#if defined(__DOS__) || defined(__WIN32__)
-            int   drive;
-            char *newrbuf;
-
-            newrbuf = imagedir;
-#ifndef __DJGPP__
-            if (*(name + 1) == ':') {
-                if (*name >= 'a' && *name <= 'z')
-                    drive = (int) (*name - 'a' + 1);
-                else
-                    drive = (int) (*name - 'A' + 1);
-                *newrbuf++ = *name;
-                *newrbuf++ = *(name + 1);
-                *newrbuf++ = DIR_SEPARATOR;
-            }
-            else {
-                drive = 0;
-                *newrbuf++ = DIR_SEPARATOR;
-            }
-            if (getcwd(newrbuf, drive) == 0) {        /* } */
-#else
-            if (getcwd(newrbuf, 1024) == 0) {        /* } */
-#endif
-#else  /* DOS */
-#ifdef HAVE_GETWD
-            if (getwd(imagedir) == 0) {                /* } */
-#else  /* !HAVE_GETWD */
-            if (getcwd(imagedir, 1024) == 0) {
-#endif /* !HAVE_GETWD */
-#endif /* DOS */
-                fatal_error(FE_GETCWD, 0);
+                return wxEmptyString;
             }
             }
-            for (; *t; t++)        /* Set t to end of buffer */
-                ;
-            if (*(t - 1) == DIR_SEPARATOR)        /* leave slash if already
-                                                 * last char
-                                                 */
-                cutoff = t - 1;
-            else {
-                cutoff = t;        /* otherwise put one in */
-                *t++ = DIR_SEPARATOR;
+            else
+            {
+                // buffer was too small, retry with a larger one
+                len *= 2;
             }
         }
             }
         }
-#if (!defined(__MAC__) && !defined(__DJGPP__) && !defined(__GO32__) && !defined(__WIN32__))
-        else
-                (*t++ = DIR_SEPARATOR);
-#endif
+        //else: ok
+    } while ( !ok );
 
 
-        /*-------------------------------------------------------------*
-         * Copy the rest of the string and set the cutoff if it was not
-         * already set.  If the first character of name is a slash, cutoff
-         * is not presently set but will be on the first iteration of the
-         * loop below.
-         *-------------------------------------------------------------*/
-
-        for ((*name == DIR_SEPARATOR ? (s = name+1) : (s = name));;) {
-            if (*s == DIR_SEPARATOR)
-                        cutoff = t;
-            if (!(*t++ = *s++))
-                        break;
-        }
+    return dir;
+}
 
 
-    }
-    else {
+#endif // 0
 
 
-        /*-------------------------------------------------------------*
-         * Get the path list from the environment.  If the path list is
-         * inaccessible for any reason, leave with fatal error.
-         *-------------------------------------------------------------*/
+// ----------------------------------------------------------------------------
+// wxExecute
+// ----------------------------------------------------------------------------
 
 
-#ifdef __MAC__
-        if ((s = getenv("Commands")) == (char *) 0)
-#else
-        if ((s = getenv("PATH")) == (char *) 0)
-#endif
-            fatal_error(FE_PATH, 0);
-
-        /*
-         * Copy path list into ebuf and set the source pointer to the
-         * beginning of this buffer.
-         */
-
-        strcpy(ebuf, s);
-        s = ebuf;
-
-        for (;;) {
-            t = imagedir;
-            while (*s && *s != PATH_SEPARATOR)
-                *t++ = *s++;
-            if (t > imagedir && *(t - 1) == DIR_SEPARATOR)
-                ;                /* do nothing -- slash already is in place */
-            else
-                *t++ = DIR_SEPARATOR;        /* put in the slash */
-            cutoff = t - 1;        /* set cutoff */
-            strcpy(t, name);
-            if (access(imagedir, R_OK) == 0)
-                break;
-
-            if (*s)
-                s++;                /* advance source pointer */
-            else
-                fatal_error(FE_INFND, 0);
-        }
+// wxDoExecuteWithCapture() helper: reads an entire stream into one array
+//
+// returns TRUE if ok, FALSE if error
+#if wxUSE_STREAMS
+static bool ReadAll(wxInputStream *is, wxArrayString& output)
+{
+    wxCHECK_MSG( is, FALSE, _T("NULL stream in wxExecute()?") );
 
 
-    }
+    // the stream could be already at EOF or in wxSTREAM_BROKEN_PIPE state
+    is->Reset();
 
 
-    /*-------------------------------------------------------------*
-     | At this point the full pathname should exist in imagedir and
-     | cutoff should be set to the final slash.  We must now determine
-     | whether the file name is a symbolic link or not and chase it down
-     | if it is.  Note that we reuse ebuf for getting the link.
-     *-------------------------------------------------------------*/
-
-#ifdef HAVE_SYMLINK
-    while ((cc = readlink(imagedir, ebuf, 512)) != -1) {
-        ebuf[cc] = 0;
-        s = ebuf;
-        if (*s == DIR_SEPARATOR) {
-            t = imagedir;
-        }
-        else {
-            t = cutoff + 1;
+    wxTextInputStream tis(*is);
+
+    bool cont = TRUE;
+    while ( cont )
+    {
+        wxString line = tis.ReadLine();
+        if ( is->Eof() )
+            break;
+
+        if ( !*is )
+        {
+            cont = FALSE;
         }
         }
-        for (;;) {
-            if (*s == DIR_SEPARATOR)
-                cutoff = t;        /* mark the last slash seen */
-            if (!(*t++ = *s++))        /* copy the character */
-                break;
+        else
+        {
+            output.Add(line);
         }
     }
 
         }
     }
 
-#endif /* HAVE_SYMLINK */
-
-    strcpy(imagename, cutoff + 1);        /* keep the image name */
-    *(cutoff + 1) = 0;                /* chop off the filename part */
+    return cont;
 }
 }
+#endif // wxUSE_STREAMS
 
 
-#endif
-
-#if wxUSE_GUI
-
-// ----------------------------------------------------------------------------
-// GUI helpers
-// ----------------------------------------------------------------------------
+// this is a private function because it hasn't a clean interface: the first
+// array is passed by reference, the second by pointer - instead we have 2
+// public versions of wxExecute() below
+static long wxDoExecuteWithCapture(const wxString& command,
+                                   wxArrayString& output,
+                                   wxArrayString* error)
+{
+    // create a wxProcess which will capture the output
+    wxProcess *process = new wxProcess;
+    process->Redirect();
 
 
-/*
- * N.B. these convenience functions must be separate from msgdlgg.cpp, textdlgg.cpp
- * since otherwise the generic code may be pulled in unnecessarily.
- */
+    long rc = wxExecute(command, wxEXEC_SYNC, process);
 
 
-#if wxUSE_MSGDLG
+#if wxUSE_STREAMS
+    if ( rc != -1 )
+    {
+        if ( !ReadAll(process->GetInputStream(), output) )
+            rc = -1;
 
 
-int wxMessageBox(const wxString& message, const wxString& caption, long style,
-                 wxWindow *parent, int WXUNUSED(x), int WXUNUSED(y) )
-{
-    wxMessageDialog dialog(parent, message, caption, style);
+        if ( error )
+        {
+            if ( !ReadAll(process->GetErrorStream(), *error) )
+                rc = -1;
+        }
 
 
-    int ans = dialog.ShowModal();
-    switch ( ans )
-    {
-        case wxID_OK:
-            return wxOK;
-        case wxID_YES:
-            return wxYES;
-        case wxID_NO:
-            return wxNO;
-        case wxID_CANCEL:
-            return wxCANCEL;
     }
     }
+#endif // wxUSE_STREAMS
 
 
-    wxFAIL_MSG( _T("unexpected return code from wxMessageDialog") );
+    delete process;
 
 
-    return wxCANCEL;
+    return rc;
 }
 
 }
 
-#endif // wxUSE_MSGDLG
-
-#if wxUSE_TEXTDLG
-
-wxString wxGetTextFromUser(const wxString& message, const wxString& caption,
-                        const wxString& defaultValue, wxWindow *parent,
-                        int x, int y, bool WXUNUSED(centre) )
+long wxExecute(const wxString& command, wxArrayString& output)
 {
 {
-    wxString str;
-    wxTextEntryDialog dialog(parent, message, caption, defaultValue, wxOK|wxCANCEL, wxPoint(x, y));
-    if (dialog.ShowModal() == wxID_OK)
-    {
-        str = dialog.GetValue();
-    }
-
-    return str;
+    return wxDoExecuteWithCapture(command, output, NULL);
 }
 
 }
 
-wxString wxGetPasswordFromUser(const wxString& message,
-                               const wxString& caption,
-                               const wxString& defaultValue,
-                               wxWindow *parent)
+long wxExecute(const wxString& command,
+               wxArrayString& output,
+               wxArrayString& error)
 {
 {
-    wxString str;
-    wxTextEntryDialog dialog(parent, message, caption, defaultValue,
-                             wxOK | wxCANCEL | wxTE_PASSWORD);
-    if ( dialog.ShowModal() == wxID_OK )
-    {
-        str = dialog.GetValue();
-    }
-
-    return str;
+    return wxDoExecuteWithCapture(command, output, &error);
 }
 
 }
 
-#endif // wxUSE_TEXTDLG
-
-#if wxUSE_COLOURDLG
+// ----------------------------------------------------------------------------
+// wxApp::Yield() wrappers for backwards compatibility
+// ----------------------------------------------------------------------------
 
 
-wxColour wxGetColourFromUser(wxWindow *parent, const wxColour& colInit)
+bool wxYield()
 {
 {
-    wxColourData data;
-    data.SetChooseFull(TRUE);
-    if ( colInit.Ok() )
-    {
-        data.SetColour((wxColour &)colInit); // const_cast
-    }
-
-    wxColour colRet;
-    wxColourDialog dialog(parent, &data);
-    if ( dialog.ShowModal() == wxID_OK )
-    {
-        colRet = dialog.GetColourData().GetColour();
-    }
-    //else: leave it invalid
+    return wxTheApp && wxTheApp->Yield();
+}
 
 
-    return colRet;
+bool wxYieldIfNeeded()
+{
+    return wxTheApp && wxTheApp->Yield(TRUE);
 }
 
 }
 
-#endif // wxUSE_COLOURDLG
+#endif // wxUSE_BASE
 
 
-#if wxUSE_FONTDLG
+// ============================================================================
+// GUI-only functions from now on
+// ============================================================================
 
 
-wxFont wxGetFontFromUser(wxWindow *parent, const wxFont& fontInit)
-{
-    wxFontData data;
-    if ( fontInit.Ok() )
-    {
-        data.SetInitialFont(fontInit);
-    }
+#if wxUSE_GUI
 
 
-    wxFont fontRet;
-    wxFontDialog dialog(parent, data);
-    if ( dialog.ShowModal() == wxID_OK )
-    {
-        fontRet = dialog.GetFontData().GetChosenFont();
-    }
-    //else: leave it invalid
+// Id generation
+static long wxCurrentId = 100;
 
 
-    return fontRet;
+long
+wxNewId (void)
+{
+  return wxCurrentId++;
 }
 
 }
 
-#endif // wxUSE_FONTDLG
-// ----------------------------------------------------------------------------
-// missing C RTL functions (FIXME shouldn't be here at all)
-// ----------------------------------------------------------------------------
+long
+wxGetCurrentId(void) { return wxCurrentId; }
 
 
-#if defined( __MWERKS__ ) && !defined(__MACH__)
-char *strdup(const char *s)
-{
-        return strcpy( (char*) malloc( strlen( s ) + 1 ) , s ) ;
-}
-int isascii( int c )
+void
+wxRegisterId (long id)
 {
 {
-        return ( c >= 0 && c < 128 ) ;
+  if (id >= wxCurrentId)
+    wxCurrentId = id + 1;
 }
 }
-#endif // __MWERKS__
+
+#if wxUSE_MENUS
 
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
-// wxSafeYield and supporting functions
+// Menu accelerators related functions
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 
-void wxEnableTopLevelWindows(bool enable)
+wxChar *wxStripMenuCodes(const wxChar *in, wxChar *out)
 {
 {
-    wxWindowList::Node *node;
-    for ( node = wxTopLevelWindows.GetFirst(); node; node = node->GetNext() )
-        node->GetData()->Enable(enable);
+    wxString s = wxMenuItem::GetLabelFromText(in);
+    if ( out )
+    {
+        // go smash their buffer if it's not big enough - I love char * params
+        memcpy(out, s.c_str(), s.length() * sizeof(wxChar));
+    }
+    else
+    {
+        // MYcopystring - for easier search...
+        out = new wxChar[s.length() + 1];
+        wxStrcpy(out, s.c_str());
+    }
+
+    return out;
 }
 
 }
 
-wxWindowDisabler::wxWindowDisabler(wxWindow *winToSkip)
+wxString wxStripMenuCodes(const wxString& in)
 {
 {
-    // remember the top level windows which were already disabled, so that we
-    // don't reenable them later
-    m_winDisabled = NULL;
+    wxString out;
 
 
-    wxWindowList::Node *node;
-    for ( node = wxTopLevelWindows.GetFirst(); node; node = node->GetNext() )
-    {
-        wxWindow *winTop = node->GetData();
-        if ( winTop == winToSkip )
-            continue;
+    size_t len = in.length();
+    out.reserve(len);
 
 
-        if ( winTop->IsEnabled() )
-        {
-            winTop->Disable();
-        }
-        else
+    for ( size_t n = 0; n < len; n++ )
+    {
+        wxChar ch = in[n];
+        if ( ch == _T('&') )
         {
         {
-            if ( !m_winDisabled )
+            // skip it, it is used to introduce the accel char (or to quote
+            // itself in which case it should still be skipped): note that it
+            // can't be the last character of the string
+            if ( ++n == len )
             {
             {
-                m_winDisabled = new wxWindowList;
+                wxLogDebug(_T("Invalid menu string '%s'"), in.c_str());
+            }
+            else
+            {
+                // use the next char instead
+                ch = in[n];
             }
             }
-
-            m_winDisabled->Append(winTop);
         }
         }
-    }
-}
-
-wxWindowDisabler::~wxWindowDisabler()
-{
-    wxWindowList::Node *node;
-    for ( node = wxTopLevelWindows.GetFirst(); node; node = node->GetNext() )
-    {
-        wxWindow *winTop = node->GetData();
-        if ( !m_winDisabled || !m_winDisabled->Find(winTop) )
+        else if ( ch == _T('\t') )
         {
         {
-            winTop->Enable();
+            // everything after TAB is accel string, exit the loop
+            break;
         }
         }
-        //else: had been already disabled, don't reenable
+
+        out += ch;
     }
 
     }
 
-    delete m_winDisabled;
+    return out;
 }
 
 }
 
-// Yield to other apps/messages and disable user input to all windows except
-// the given one
-bool wxSafeYield(wxWindow *win, bool onlyIfNeeded)
-{
-    wxWindowDisabler wd(win);
-
-    bool rc;
-    if (onlyIfNeeded)
-        rc = wxYieldIfNeeded();
-    else
-        rc = wxYield();
-
-    return rc;
-}
+#endif // wxUSE_MENUS
 
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
-// misc functions
+// Window search functions
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 
-// Don't synthesize KeyUp events holding down a key and producing KeyDown
-// events with autorepeat. On by default and always on in wxMSW. wxGTK version
-// in utilsgtk.cpp.
-#ifndef __WXGTK__
-bool wxSetDetectableAutoRepeat( bool WXUNUSED(flag) )
-{
-    return TRUE;    // detectable auto-repeat is the only mode MSW supports
-}
-#endif // !wxGTK
-
-#endif // wxUSE_GUI
+/*
+ * If parent is non-NULL, look through children for a label or title
+ * matching the specified string. If NULL, look through all top-level windows.
+ *
+ */
 
 
-const wxChar *wxGetInstallPrefix()
+wxWindow *
+wxFindWindowByLabel (const wxString& title, wxWindow * parent)
 {
 {
-    wxString prefix;
+    return wxWindow::FindWindowByLabel( title, parent );
+}
 
 
-    if ( wxGetEnv(wxT("WXPREFIX"), &prefix) )
-        return prefix.c_str();
 
 
-#ifdef wxINSTALL_PREFIX
-    return wxT(wxINSTALL_PREFIX);
-#else
-    return wxT("");
-#endif
-}
+/*
+ * If parent is non-NULL, look through children for a name
+ * matching the specified string. If NULL, look through all top-level windows.
+ *
+ */
 
 
-wxString wxGetDataDir()
+wxWindow *
+wxFindWindowByName (const wxString& name, wxWindow * parent)
 {
 {
-    wxString format = wxGetInstallPrefix();
-    format <<  wxFILE_SEP_PATH
-           << wxT("share") << wxFILE_SEP_PATH
-           << wxT("wx") << wxFILE_SEP_PATH
-           << wxT("%i.%i");
-    wxString dir;
-    dir.Printf(format.c_str(), wxMAJOR_VERSION, wxMINOR_VERSION);
-    return dir;
+    return wxWindow::FindWindowByName( name, parent );
 }
 
 }
 
-
-// ----------------------------------------------------------------------------
-// network and user id functions
-// ----------------------------------------------------------------------------
-
-// Get Full RFC822 style email address
-bool wxGetEmailAddress(wxChar *address, int maxSize)
+// Returns menu item id or -1 if none.
+int
+wxFindMenuItemId (wxFrame * frame, const wxString& menuString, const wxString& itemString)
 {
 {
-    wxString email = wxGetEmailAddress();
-    if ( !email )
-        return FALSE;
-
-    wxStrncpy(address, email, maxSize - 1);
-    address[maxSize - 1] = wxT('\0');
+#if wxUSE_MENUS
+  wxMenuBar *menuBar = frame->GetMenuBar ();
+  if ( menuBar )
+      return menuBar->FindMenuItem (menuString, itemString);
+#endif // wxUSE_MENUS
 
 
-    return TRUE;
+  return -1;
 }
 
 }
 
-wxString wxGetEmailAddress()
+// Try to find the deepest child that contains 'pt'.
+// We go backwards, to try to allow for controls that are spacially
+// within other controls, but are still siblings (e.g. buttons within
+// static boxes). Static boxes are likely to be created _before_ controls
+// that sit inside them.
+wxWindow* wxFindWindowAtPoint(wxWindow* win, const wxPoint& pt)
 {
 {
-    wxString email;
+    if (!win->IsShown())
+        return NULL;
 
 
-    wxString host = wxGetFullHostName();
-    if ( !!host )
+    // Hack for wxNotebook case: at least in wxGTK, all pages
+    // claim to be shown, so we must only deal with the selected one.
+#if wxUSE_NOTEBOOK
+    if (win->IsKindOf(CLASSINFO(wxNotebook)))
     {
     {
-        wxString user = wxGetUserId();
-        if ( !!user )
-        {
-            email << user << wxT('@') << host;
-        }
+      wxNotebook* nb = (wxNotebook*) win;
+      int sel = nb->GetSelection();
+      if (sel >= 0)
+      {
+        wxWindow* child = nb->GetPage(sel);
+        wxWindow* foundWin = wxFindWindowAtPoint(child, pt);
+        if (foundWin)
+           return foundWin;
+      }
+    }
+#endif
+
+    wxWindowList::compatibility_iterator node = win->GetChildren().GetLast();
+    while (node)
+    {
+        wxWindow* child = node->GetData();
+        wxWindow* foundWin = wxFindWindowAtPoint(child, pt);
+        if (foundWin)
+          return foundWin;
+        node = node->GetPrevious();
     }
 
     }
 
-    return email;
+    wxPoint pos = win->GetPosition();
+    wxSize sz = win->GetSize();
+    if (win->GetParent())
+    {
+        pos = win->GetParent()->ClientToScreen(pos);
+    }
+
+    wxRect rect(pos, sz);
+    if (rect.Inside(pt))
+        return win;
+    else
+        return NULL;
 }
 
 }
 
-wxString wxGetUserId()
+wxWindow* wxGenericFindWindowAtPoint(const wxPoint& pt)
 {
 {
-    static const int maxLoginLen = 256; // FIXME arbitrary number
+    // Go backwards through the list since windows
+    // on top are likely to have been appended most
+    // recently.
+    wxWindowList::compatibility_iterator node = wxTopLevelWindows.GetLast();
+    while (node)
+    {
+        wxWindow* win = node->GetData();
+        wxWindow* found = wxFindWindowAtPoint(win, pt);
+        if (found)
+            return found;
+        node = node->GetPrevious();
+    }
+    return NULL;
+}
 
 
-    wxString buf;
-    bool ok = wxGetUserId(buf.GetWriteBuf(maxLoginLen), maxLoginLen);
-    buf.UngetWriteBuf();
+// ----------------------------------------------------------------------------
+// GUI helpers
+// ----------------------------------------------------------------------------
 
 
-    if ( !ok )
-        buf.Empty();
+/*
+ * N.B. these convenience functions must be separate from msgdlgg.cpp, textdlgg.cpp
+ * since otherwise the generic code may be pulled in unnecessarily.
+ */
 
 
-    return buf;
-}
+#if wxUSE_MSGDLG
 
 
-wxString wxGetUserName()
+int wxMessageBox(const wxString& message, const wxString& caption, long style,
+                 wxWindow *parent, int WXUNUSED(x), int WXUNUSED(y) )
 {
 {
-    static const int maxUserNameLen = 1024; // FIXME arbitrary number
+    wxMessageDialog dialog(parent, message, caption, style);
 
 
-    wxString buf;
-    bool ok = wxGetUserName(buf.GetWriteBuf(maxUserNameLen), maxUserNameLen);
-    buf.UngetWriteBuf();
+    int ans = dialog.ShowModal();
+    switch ( ans )
+    {
+        case wxID_OK:
+            return wxOK;
+        case wxID_YES:
+            return wxYES;
+        case wxID_NO:
+            return wxNO;
+        case wxID_CANCEL:
+            return wxCANCEL;
+    }
 
 
-    if ( !ok )
-        buf.Empty();
+    wxFAIL_MSG( _T("unexpected return code from wxMessageDialog") );
 
 
-    return buf;
+    return wxCANCEL;
 }
 
 }
 
-wxString wxGetHostName()
-{
-    static const size_t hostnameSize = 257;
-
-    wxString buf;
-    bool ok = wxGetHostName(buf.GetWriteBuf(hostnameSize), hostnameSize);
+#endif // wxUSE_MSGDLG
 
 
-    buf.UngetWriteBuf();
+#if wxUSE_TEXTDLG
 
 
-    if ( !ok )
-        buf.Empty();
+wxString wxGetTextFromUser(const wxString& message, const wxString& caption,
+                        const wxString& defaultValue, wxWindow *parent,
+                        int x, int y, bool WXUNUSED(centre) )
+{
+    wxString str;
+    wxTextEntryDialog dialog(parent, message, caption, defaultValue, wxOK|wxCANCEL, wxPoint(x, y));
+    if (dialog.ShowModal() == wxID_OK)
+    {
+        str = dialog.GetValue();
+    }
 
 
-    return buf;
+    return str;
 }
 
 }
 
-wxString wxGetFullHostName()
+wxString wxGetPasswordFromUser(const wxString& message,
+                               const wxString& caption,
+                               const wxString& defaultValue,
+                               wxWindow *parent)
 {
 {
-    static const size_t hostnameSize = 257;
-
-    wxString buf;
-    bool ok = wxGetFullHostName(buf.GetWriteBuf(hostnameSize), hostnameSize);
+    wxString str;
+    wxTextEntryDialog dialog(parent, message, caption, defaultValue,
+                             wxOK | wxCANCEL | wxTE_PASSWORD);
+    if ( dialog.ShowModal() == wxID_OK )
+    {
+        str = dialog.GetValue();
+    }
 
 
-    buf.UngetWriteBuf();
+    return str;
+}
 
 
-    if ( !ok )
-        buf.Empty();
+#endif // wxUSE_TEXTDLG
 
 
-    return buf;
-}
+#if wxUSE_COLOURDLG
 
 
-wxString wxGetHomeDir()
+wxColour wxGetColourFromUser(wxWindow *parent, const wxColour& colInit)
 {
 {
-    wxString home;
-    wxGetHomeDir(&home);
+    wxColourData data;
+    data.SetChooseFull(TRUE);
+    if ( colInit.Ok() )
+    {
+        data.SetColour((wxColour &)colInit); // const_cast
+    }
 
 
-    return home;
+    wxColour colRet;
+    wxColourDialog dialog(parent, &data);
+    if ( dialog.ShowModal() == wxID_OK )
+    {
+        colRet = dialog.GetColourData().GetColour();
+    }
+    //else: leave it invalid
+
+    return colRet;
 }
 
 }
 
-#if 0
+#endif // wxUSE_COLOURDLG
 
 
-wxString wxGetCurrentDir()
+#if wxUSE_FONTDLG
+
+wxFont wxGetFontFromUser(wxWindow *parent, const wxFont& fontInit)
 {
 {
-    wxString dir;
-    size_t len = 1024;
-    bool ok;
-    do
+    wxFontData data;
+    if ( fontInit.Ok() )
     {
     {
-        ok = getcwd(dir.GetWriteBuf(len + 1), len) != NULL;
-        dir.UngetWriteBuf();
-
-        if ( !ok )
-        {
-            if ( errno != ERANGE )
-            {
-                wxLogSysError(_T("Failed to get current directory"));
+        data.SetInitialFont(fontInit);
+    }
 
 
-                return wxEmptyString;
-            }
-            else
-            {
-                // buffer was too small, retry with a larger one
-                len *= 2;
-            }
-        }
-        //else: ok
-    } while ( !ok );
+    wxFont fontRet;
+    wxFontDialog dialog(parent, data);
+    if ( dialog.ShowModal() == wxID_OK )
+    {
+        fontRet = dialog.GetFontData().GetChosenFont();
+    }
+    //else: leave it invalid
 
 
-    return dir;
+    return fontRet;
 }
 
 }
 
-#endif // 0
+#endif // wxUSE_FONTDLG
 
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
-// wxExecute
+// wxSafeYield and supporting functions
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 
-// wxDoExecuteWithCapture() helper: reads an entire stream into one array
-//
-// returns TRUE if ok, FALSE if error
-#if wxUSE_STREAMS
-static bool ReadAll(wxInputStream *is, wxArrayString& output)
+void wxEnableTopLevelWindows(bool enable)
 {
 {
-    wxCHECK_MSG( is, FALSE, _T("NULL stream in wxExecute()?") );
-
-    // the stream could be already at EOF or in wxSTREAM_BROKEN_PIPE state
-    is->Reset();
+    wxWindowList::compatibility_iterator node;
+    for ( node = wxTopLevelWindows.GetFirst(); node; node = node->GetNext() )
+        node->GetData()->Enable(enable);
+}
 
 
-    wxTextInputStream tis(*is);
+wxWindowDisabler::wxWindowDisabler(wxWindow *winToSkip)
+{
+    // remember the top level windows which were already disabled, so that we
+    // don't reenable them later
+    m_winDisabled = NULL;
 
 
-    bool cont = TRUE;
-    while ( cont )
+    wxWindowList::compatibility_iterator node;
+    for ( node = wxTopLevelWindows.GetFirst(); node; node = node->GetNext() )
     {
     {
-        wxString line = tis.ReadLine();
-        if ( is->Eof() )
-            break;
+        wxWindow *winTop = node->GetData();
+        if ( winTop == winToSkip )
+            continue;
 
 
-        if ( !*is )
+        // we don't need to disable the hidden or already disabled windows
+        if ( winTop->IsEnabled() && winTop->IsShown() )
         {
         {
-            cont = FALSE;
+            winTop->Disable();
         }
         else
         {
         }
         else
         {
-            output.Add(line);
+            if ( !m_winDisabled )
+            {
+                m_winDisabled = new wxWindowList;
+            }
+
+            m_winDisabled->Append(winTop);
         }
     }
         }
     }
-
-    return cont;
 }
 }
-#endif // wxUSE_STREAMS
 
 
-// this is a private function because it hasn't a clean interface: the first
-// array is passed by reference, the second by pointer - instead we have 2
-// public versions of wxExecute() below
-static long wxDoExecuteWithCapture(const wxString& command,
-                                   wxArrayString& output,
-                                   wxArrayString* error)
+wxWindowDisabler::~wxWindowDisabler()
 {
 {
-#ifdef __WIN16__
-    wxFAIL_MSG("Sorry, this version of wxExecute not implemented on WIN16.");
-
-    return 0;
-#else // !Win16
-    // create a wxProcess which will capture the output
-    wxProcess *process = new wxProcess;
-    process->Redirect();
-
-    long rc = wxExecute(command, wxEXEC_SYNC, process);
-
-#if wxUSE_STREAMS
-    if ( rc != -1 )
+    wxWindowList::compatibility_iterator node;
+    for ( node = wxTopLevelWindows.GetFirst(); node; node = node->GetNext() )
     {
     {
-        if ( !ReadAll(process->GetInputStream(), output) )
-            rc = -1;
-
-        if ( error )
+        wxWindow *winTop = node->GetData();
+        if ( !m_winDisabled || !m_winDisabled->Find(winTop) )
         {
         {
-            if ( !ReadAll(process->GetErrorStream(), *error) )
-                rc = -1;
+            winTop->Enable();
         }
         }
-
+        //else: had been already disabled, don't reenable
     }
     }
-#endif // wxUSE_STREAMS
-
-    delete process;
-
-    return rc;
-#endif // IO redirection supoprted
-}
 
 
-long wxExecute(const wxString& command, wxArrayString& output)
-{
-    return wxDoExecuteWithCapture(command, output, NULL);
+    delete m_winDisabled;
 }
 
 }
 
-long wxExecute(const wxString& command,
-               wxArrayString& output,
-               wxArrayString& error)
+// Yield to other apps/messages and disable user input to all windows except
+// the given one
+bool wxSafeYield(wxWindow *win, bool onlyIfNeeded)
 {
 {
-    return wxDoExecuteWithCapture(command, output, &error);
-}
+    wxWindowDisabler wd(win);
 
 
-// ----------------------------------------------------------------------------
-// wxApp::Yield() wrappers for backwards compatibility
-// ----------------------------------------------------------------------------
+    bool rc;
+    if (onlyIfNeeded)
+        rc = wxYieldIfNeeded();
+    else
+        rc = wxYield();
 
 
-bool wxYield()
-{
-#if wxUSE_GUI
-    return wxTheApp && wxTheApp->Yield();
-#else
-    return FALSE;
-#endif
+    return rc;
 }
 
 }
 
-bool wxYieldIfNeeded()
+// Don't synthesize KeyUp events holding down a key and producing KeyDown
+// events with autorepeat. On by default and always on in wxMSW. wxGTK version
+// in utilsgtk.cpp.
+#ifndef __WXGTK__
+bool wxSetDetectableAutoRepeat( bool WXUNUSED(flag) )
 {
 {
-#if wxUSE_GUI
-    return wxTheApp && wxTheApp->Yield(TRUE);
-#else
-    return FALSE;
-#endif
+    return TRUE;    // detectable auto-repeat is the only mode MSW supports
 }
 }
+#endif // !wxGTK
+
+#endif // wxUSE_GUI