fix warning about function hiding by using the 'On' prefix for event handlers

[wxWidgets.git] / samples / keyboard / keyboard.cpp

diff --git a/samples/keyboard/keyboard.cpp b/samples/keyboard/keyboard.cpp
index aca5749e2b5b3084adced9e55af3c9bbfd4a1cdb..5ea78192a1cfdcbf7b95bf99b976eaf46a396bf7 100644 (file)
--- a/samples/keyboard/keyboard.cpp
+++ b/samples/keyboard/keyboard.cpp
@@ -1,502 +1,371 @@
 /////////////////////////////////////////////////////////////////////////////
 // Name:        keyboard.cpp
 /////////////////////////////////////////////////////////////////////////////
 // Name:        keyboard.cpp

-// Purpose:     Keyboard wxWindows sample
+// Purpose:     Keyboard wxWidgets sample

 // Author:      Vadim Zeitlin
 // Author:      Vadim Zeitlin

-// Modified by:
+// Modified by: Marcin Wojdyr

 // Created:     07.04.02
 // RCS-ID:      $Id$
 // Copyright:   (c) 2002 Vadim Zeitlin
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Created:     07.04.02
 // RCS-ID:      $Id$
 // Copyright:   (c) 2002 Vadim Zeitlin
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-// ============================================================================
-// declarations
-// ============================================================================
-
-// ----------------------------------------------------------------------------
-// headers
-// ----------------------------------------------------------------------------
-
-// For compilers that support precompilation, includes "wx/wx.h".

 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
     #pragma hdrstop
 #endif
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
     #pragma hdrstop
 #endif

-
-// for all others, include the necessary headers (this file is usually all you
-// need because it includes almost all "standard" wxWindows headers)

 #ifndef WX_PRECOMP
     #include "wx/wx.h"
 #endif
 
 #ifndef WX_PRECOMP
     #include "wx/wx.h"
 #endif
 

-// ----------------------------------------------------------------------------
-// private classes
-// ----------------------------------------------------------------------------
-
-// Define a new application type, each program should derive a class from wxApp
-class MyApp : public wxApp
-{
-public:
-    // override base class virtuals
-    // ----------------------------
-
-    // this one is called on application startup and is a good place for the app
-    // initialization (doing it here and not in the ctor allows to have an error
-    // return: if OnInit() returns false, the application terminates)
-    virtual bool OnInit();
-};

 
 // Define a new frame type: this is going to be our main frame
 class MyFrame : public wxFrame
 {
 public:
 
 // Define a new frame type: this is going to be our main frame
 class MyFrame : public wxFrame
 {
 public:

-    // ctor(s)
-    MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size,
-            long style = wxDEFAULT_FRAME_STYLE);
-
-    ~MyFrame() { delete m_logTarget; }
-
-    // event handlers (these functions should _not_ be virtual)
-    void OnQuit(wxCommandEvent& event);
-    void OnAbout(wxCommandEvent& event);
-    void OnClear(wxCommandEvent& event);
-    void OnSkip(wxCommandEvent& event);
-    void OnShowRaw(wxCommandEvent& event);
-
-    void OnSize(wxSizeEvent& event);
+    MyFrame(const wxString& title);

 
 private:
 
 private:

-    wxLog *m_logTarget;
+    // event handlers
+    void OnQuit(wxCommandEvent& WXUNUSED(event)) { Close(true); }
+    void OnAbout(wxCommandEvent& event);
+    void OnClear(wxCommandEvent& WXUNUSED(event)) { m_logText->Clear(); }
+    void OnSkip(wxCommandEvent& event) { m_skip = event.IsChecked(); }
+    void OnKeyDown(wxKeyEvent& event) { LogEvent("KeyDown", event); }
+    void OnKeyUp(wxKeyEvent& event) { LogEvent("KeyUp", event); }
+    void OnChar(wxKeyEvent& event) { LogEvent("Char", event); }
+    void OnPaintInputWin(wxPaintEvent& event);

 
 

-    class TextWindow *m_winText;
-    wxListBox *m_lboxLog;
+    void LogEvent(const wxString& name, wxKeyEvent& event);

 
 

-    // any class wishing to process wxWindows events must use this macro
-    DECLARE_EVENT_TABLE()
+    wxTextCtrl *m_logText;
+    wxWindow *m_inputWin;
+    bool m_skip;

 };
 
 };
 

-// A log target which just redirects the messages to a listbox
-class LboxLogger : public wxLog
-{
-public:
-    LboxLogger(wxListBox *lbox, wxLog *logOld)
-    {
-        m_lbox = lbox;
-        //m_lbox->Disable(); -- looks ugly under MSW
-        m_logOld = logOld;
-    }
-
-    virtual ~LboxLogger()
-    {
-        wxLog::SetActiveTarget(m_logOld);
-    }

 
 

-private:
-    // implement sink functions
-    virtual void DoLog(wxLogLevel level, const wxChar *szString, time_t t)
-    {
-        // don't put trace messages into listbox or we can get into infinite
-        // recursion
-        if ( level == wxLOG_Trace )
-        {
-            if ( m_logOld )
-            {
-                // cast is needed to call protected method
-                ((LboxLogger *)m_logOld)->DoLog(level, szString, t);
-            }
-        }
-        else
-        {
-            wxLog::DoLog(level, szString, t);
-        }
-    }
-
-    virtual void DoLogString(const wxChar *szString, time_t t)
-    {
-        wxString msg;
-        TimeStamp(&msg);
-        msg += szString;
-
-        #ifdef __WXUNIVERSAL__
-            m_lbox->AppendAndEnsureVisible(msg);
-        #else // other ports don't have this method yet
-            m_lbox->Append(msg);
-            m_lbox->SetFirstItem(m_lbox->GetCount() - 1);
-        #endif
-    }
-
-    // the control we use
-    wxListBox *m_lbox;
-
-    // the old log target
-    wxLog *m_logOld;
-};
-
-class TextWindow : public wxWindow
+// Define a new application type, each program should derive a class from wxApp
+class MyApp : public wxApp

 {
 public:
 {
 public:

-    TextWindow(wxWindow *parent)
-        : wxWindow(parent, -1, wxDefaultPosition, wxDefaultSize,
-                   wxRAISED_BORDER)
+    // 'Main program' equivalent: the program execution "starts" here
+    virtual bool OnInit()

     {
     {

-        m_skip = TRUE;
-        m_showRaw = FALSE;
-
-        SetBackgroundColour(*wxBLUE);
-    }
-
-    void SetSkip(bool skip) { m_skip = skip; }
-    void SetShowRaw(bool show) { m_showRaw = show; }
+        // create the main application window
+        new MyFrame("Keyboard wxWidgets App");

 
 

-protected:
-    void OnKeyDown(wxKeyEvent& event) { LogEvent(_T("Key down"), event); }
-    void OnKeyUp(wxKeyEvent& event) { LogEvent(_T("Key up"), event); }
-    void OnChar(wxKeyEvent& event) { LogEvent(_T("Char"), event); }
-
-    void OnPaint(wxPaintEvent& event)
-    {
-        wxPaintDC dc(this);
-        dc.SetTextForeground(*wxWHITE);
-        dc.DrawLabel(_T("Press keys here"), GetClientRect(), wxALIGN_CENTER);
+        // If we returned false here, the application would exit immediately.
+        return true;

     }
     }

-
-private:
-    static inline wxChar GetChar(bool on, wxChar c) { return on ? c : _T('-'); }
-
-    void LogEvent(const wxChar *name, wxKeyEvent& event);
-
-    bool m_skip;
-    bool m_showRaw;
-
-    DECLARE_EVENT_TABLE()
-};
-
-BEGIN_EVENT_TABLE(TextWindow, wxWindow)
-    EVT_KEY_DOWN(TextWindow::OnKeyDown)
-    EVT_KEY_UP(TextWindow::OnKeyUp)
-    EVT_CHAR(TextWindow::OnChar)
-
-    EVT_PAINT(TextWindow::OnPaint)
-END_EVENT_TABLE()
-
-// ----------------------------------------------------------------------------
-// constants
-// ----------------------------------------------------------------------------
-
-// IDs for the controls and the menu commands
-enum
-{
-    // menu items
-    Keyboard_Quit = 1,
-
-    Keyboard_Clear,
-    Keyboard_Skip,
-    Keyboard_ShowRaw,
-
-    // it is important for the id corresponding to the "About" command to have
-    // this standard value as otherwise it won't be handled properly under Mac
-    // (where it is special and put into the "Apple" menu)
-    Keyboard_About = wxID_ABOUT

 };
 
 };
 

-// ----------------------------------------------------------------------------
-// event tables and other macros for wxWindows
-// ----------------------------------------------------------------------------
-
-// the event tables connect the wxWindows events with the functions (event
-// handlers) which process them. It can be also done at run-time, but for the
-// simple menu events like this the static method is much simpler.
-BEGIN_EVENT_TABLE(MyFrame, wxFrame)
-    EVT_MENU(Keyboard_Quit,  MyFrame::OnQuit)
-    EVT_MENU(Keyboard_About, MyFrame::OnAbout)
-
-    EVT_MENU(Keyboard_Clear, MyFrame::OnClear)
-    EVT_MENU(Keyboard_Skip, MyFrame::OnSkip)
-    EVT_MENU(Keyboard_ShowRaw, MyFrame::OnShowRaw)
-
-    EVT_SIZE(MyFrame::OnSize)
-END_EVENT_TABLE()
-
-// Create a new application object: this macro will allow wxWindows to create
+// Create a new application object: this macro will allow wxWidgets to create

 // the application object during program execution (it's better than using a
 // static object for many reasons) and also declares the accessor function
 // wxGetApp() which will return the reference of the right type (i.e. MyApp and
 // not wxApp)
 IMPLEMENT_APP(MyApp)
 
 // the application object during program execution (it's better than using a
 // static object for many reasons) and also declares the accessor function
 // wxGetApp() which will return the reference of the right type (i.e. MyApp and
 // not wxApp)
 IMPLEMENT_APP(MyApp)
 

+

 // ============================================================================
 // implementation
 // ============================================================================
 
 // ============================================================================
 // implementation
 // ============================================================================
 

-// ----------------------------------------------------------------------------
-// the application class
-// ----------------------------------------------------------------------------
-
-// 'Main program' equivalent: the program execution "starts" here
-bool MyApp::OnInit()
-{
-    // create the main application window
-    MyFrame *frame = new MyFrame(_T("Keyboard wxWindows App"),
-                                 wxPoint(50, 50), wxSize(450, 340));
-
-    // and show it (the frames, unlike simple controls, are not shown when
-    // created initially)
-    frame->Show(TRUE);
-
-    // success: wxApp::OnRun() will be called which will enter the main message
-    // loop and the application will run. If we returned FALSE here, the
-    // application would exit immediately.
-    return TRUE;
-}
-
-// ----------------------------------------------------------------------------
-// main frame
-// ----------------------------------------------------------------------------
-

 // frame constructor
 // frame constructor

-MyFrame::MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size, long style)
-       : wxFrame(NULL, -1, title, pos, size, style),
-         m_winText(NULL)
+MyFrame::MyFrame(const wxString& title)
+       : wxFrame(NULL, wxID_ANY, title),
+         m_inputWin(NULL),
+         m_skip(true)

 {
 {

-#if wxUSE_MENUS
+    // IDs for menu items
+    enum
+    {
+        QuitID = wxID_EXIT,
+        ClearID = wxID_CLEAR,
+        SkipID = 100,
+    };
+

     // create a menu bar
     wxMenu *menuFile = new wxMenu;
     // create a menu bar
     wxMenu *menuFile = new wxMenu;

-    menuFile->Append(Keyboard_Clear, _T("&Clear log\tCtrl-L"));
+
+    menuFile->Append(ClearID, "&Clear log\tCtrl-L");
+    menuFile->AppendSeparator();
+
+    menuFile->AppendCheckItem(SkipID, "Call event.&Skip()\tCtrl-S");
+    menuFile->Check(SkipID, true);

     menuFile->AppendSeparator();
     menuFile->AppendSeparator();

-    menuFile->Append(Keyboard_Quit, _T("E&xit\tAlt-X"), _T("Quit this program"));

 
 

-    wxMenu *menuKeys = new wxMenu;
-    menuKeys->AppendCheckItem(Keyboard_ShowRaw, _T("Show &raw keys\tCtrl-R"));
-    menuKeys->AppendSeparator();
-    menuKeys->AppendCheckItem(Keyboard_Skip, _T("&Skip key down\tCtrl-S"));
+    menuFile->Append(QuitID, "E&xit\tAlt-X", "Quit this program");

 
     // the "About" item should be in the help menu
     wxMenu *menuHelp = new wxMenu;
 
     // the "About" item should be in the help menu
     wxMenu *menuHelp = new wxMenu;

-    menuHelp->Append(Keyboard_About, _T("&About...\tF1"), _T("Show about dialog"));
+    menuHelp->Append(wxID_ABOUT, "&About...\tF1", "Show about dialog");

 
     // now append the freshly created menu to the menu bar...
     wxMenuBar *menuBar = new wxMenuBar();
 
     // now append the freshly created menu to the menu bar...
     wxMenuBar *menuBar = new wxMenuBar();

-    menuBar->Append(menuFile, _T("&File"));
-    menuBar->Append(menuKeys, _T("&Keys"));
-    menuBar->Append(menuHelp, _T("&Help"));
+    menuBar->Append(menuFile, "&File");
+    menuBar->Append(menuHelp, "&Help");

 
     // ... and attach this menu bar to the frame
     SetMenuBar(menuBar);
 
 
     // ... and attach this menu bar to the frame
     SetMenuBar(menuBar);
 

-    menuBar->Check(Keyboard_Skip, TRUE);
-
-#ifndef wxHAS_RAW_KEY_CODES
-    menuBar->Enable(Keyboard_ShowRaw, FALSE);
-#endif // !wxHAS_RAW_KEY_CODES
-#endif // wxUSE_MENUS
-
-    m_winText = new TextWindow(this);
-    m_lboxLog = new wxListBox(this, -1);
-
-    m_logTarget = new LboxLogger(m_lboxLog, wxLog::GetActiveTarget());
-    wxLog::SetActiveTarget(m_logTarget);
-
-#if wxUSE_STATUSBAR
-    // create a status bar just for fun (by default with 1 pane only)
-    CreateStatusBar(2);
-    SetStatusText(_T("Welcome to wxWindows!"));
-#endif // wxUSE_STATUSBAR
+    m_inputWin = new wxWindow(this, wxID_ANY, wxDefaultPosition, wxSize(-1, 50),
+                              wxRAISED_BORDER),
+    m_inputWin->SetBackgroundColour(*wxBLUE);
+
+    wxTextCtrl *headerText = new wxTextCtrl(this, wxID_ANY, "",
+                                            wxDefaultPosition, wxDefaultSize,
+                                            wxTE_READONLY);
+    headerText->SetValue(
+               " event          key     KeyCode mod   UnicodeKey  "
+               "  RawKeyCode RawKeyFlags");
+
+
+    m_logText = new wxTextCtrl(this, wxID_ANY, "", 
+                               wxDefaultPosition, wxDefaultSize,
+                               wxTE_MULTILINE|wxTE_READONLY|wxHSCROLL);
+
+    // set monospace font to have output in nice columns
+    wxFont font(10, wxFONTFAMILY_TELETYPE, 
+                wxFONTSTYLE_NORMAL, wxFONTWEIGHT_NORMAL);
+    headerText->SetFont(font);
+    m_logText->SetFont(font);
+
+    // layout
+    wxBoxSizer *sizer = new wxBoxSizer(wxVERTICAL);
+    sizer->Add(m_inputWin, wxSizerFlags().Expand());
+    sizer->Add(headerText, wxSizerFlags().Expand());
+    sizer->Add(m_logText, wxSizerFlags(1).Expand());
+    SetSizerAndFit(sizer);
+
+    // set size and position on screen
+    SetSize(700, 340);
+    CentreOnScreen();
+
+    // connect menu event handlers
+
+    Connect(QuitID, wxEVT_COMMAND_MENU_SELECTED,
+            wxCommandEventHandler(MyFrame::OnQuit));
+
+    Connect(wxID_ABOUT, wxEVT_COMMAND_MENU_SELECTED,
+            wxCommandEventHandler(MyFrame::OnAbout));
+
+    Connect(ClearID, wxEVT_COMMAND_MENU_SELECTED,
+            wxCommandEventHandler(MyFrame::OnClear));
+
+    Connect(SkipID, wxEVT_COMMAND_MENU_SELECTED,
+            wxCommandEventHandler(MyFrame::OnSkip));
+
+    // connect event handlers for the blue input window 
+    m_inputWin->Connect(wxEVT_KEY_DOWN, wxKeyEventHandler(MyFrame::OnKeyDown),
+                        NULL, this);
+    m_inputWin->Connect(wxEVT_KEY_UP, wxKeyEventHandler(MyFrame::OnKeyUp),
+                        NULL, this);
+    m_inputWin->Connect(wxEVT_CHAR, wxKeyEventHandler(MyFrame::OnChar),
+                        NULL, this);
+    m_inputWin->Connect(wxEVT_PAINT, 
+                        wxPaintEventHandler(MyFrame::OnPaintInputWin),
+                        NULL, this);
+
+    // and show itself (the frames, unlike simple controls, are not shown when
+    // created initially)
+    Show(true);

 }
 
 // event handlers
 
 }
 
 // event handlers
 

-void MyFrame::OnQuit(wxCommandEvent& WXUNUSED(event))
-{
-    // TRUE is to force the frame to close
-    Close(TRUE);
-}
-

 void MyFrame::OnAbout(wxCommandEvent& WXUNUSED(event))
 {
 void MyFrame::OnAbout(wxCommandEvent& WXUNUSED(event))
 {

-    wxString msg = _T("Demonstrates keyboard event processing in wxWindows\n")
-                   _T("© 2002 Vadim Zeitlin");
-
-    wxMessageBox(msg, _T("About wxKeyboard"), wxOK | wxICON_INFORMATION, this);
+    wxMessageBox("Demonstrates keyboard event processing in wxWidgets\n"
+                 "(c) 2002 Vadim Zeitlin\n"
+                 "(c) 2008 Marcin Wojdyr",
+                 "About wxWidgets Keyboard Sample",
+                 wxOK | wxICON_INFORMATION, this);

 }
 
 }
 

-void MyFrame::OnClear(wxCommandEvent& event)
+void MyFrame::OnPaintInputWin(wxPaintEvent& WXUNUSED(event))

 {
 {

-    m_lboxLog->Clear();
+    wxPaintDC dc(m_inputWin);
+    dc.SetTextForeground(*wxWHITE);
+    wxFont font(*wxSWISS_FONT);
+    font.SetWeight(wxFONTWEIGHT_BOLD);
+    font.SetPointSize(font.GetPointSize() + 2);
+    dc.SetFont(font);
+
+    dc.DrawLabel("Press keys here", 
+                 m_inputWin->GetClientRect(), wxALIGN_CENTER);

 }
 
 }
 

-void MyFrame::OnSkip(wxCommandEvent& event)
-{
-    m_winText->SetSkip(event.IsChecked());
-}

 
 

-void MyFrame::OnShowRaw(wxCommandEvent& event)
+// helper function that returns textual description of wx virtual keycode
+const char* GetVirtualKeyCodeName(int keycode)

 {
 {

-    m_winText->SetShowRaw(event.IsChecked());
+    switch ( keycode )
+    {
+#define WXK_(x) \
+        case WXK_##x: return #x; 
+
+        WXK_(BACK)
+        WXK_(TAB)
+        WXK_(RETURN)
+        WXK_(ESCAPE)
+        WXK_(SPACE)
+        WXK_(DELETE)
+        WXK_(START)
+        WXK_(LBUTTON)
+        WXK_(RBUTTON)
+        WXK_(CANCEL)
+        WXK_(MBUTTON)
+        WXK_(CLEAR)
+        WXK_(SHIFT)
+        WXK_(ALT)
+        WXK_(CONTROL)
+        WXK_(MENU)
+        WXK_(PAUSE)
+        WXK_(CAPITAL)
+        WXK_(END)
+        WXK_(HOME)
+        WXK_(LEFT)
+        WXK_(UP)
+        WXK_(RIGHT)
+        WXK_(DOWN)
+        WXK_(SELECT)
+        WXK_(PRINT)
+        WXK_(EXECUTE)
+        WXK_(SNAPSHOT)
+        WXK_(INSERT)
+        WXK_(HELP)
+        WXK_(NUMPAD0)
+        WXK_(NUMPAD1)
+        WXK_(NUMPAD2)
+        WXK_(NUMPAD3)
+        WXK_(NUMPAD4)
+        WXK_(NUMPAD5)
+        WXK_(NUMPAD6)
+        WXK_(NUMPAD7)
+        WXK_(NUMPAD8)
+        WXK_(NUMPAD9)
+        WXK_(MULTIPLY)
+        WXK_(ADD)
+        WXK_(SEPARATOR)
+        WXK_(SUBTRACT)
+        WXK_(DECIMAL)
+        WXK_(DIVIDE)
+        WXK_(F1)
+        WXK_(F2)
+        WXK_(F3)
+        WXK_(F4)
+        WXK_(F5)
+        WXK_(F6)
+        WXK_(F7)
+        WXK_(F8)
+        WXK_(F9)
+        WXK_(F10)
+        WXK_(F11)
+        WXK_(F12)
+        WXK_(F13)
+        WXK_(F14)
+        WXK_(F15)
+        WXK_(F16)
+        WXK_(F17)
+        WXK_(F18)
+        WXK_(F19)
+        WXK_(F20)
+        WXK_(F21)
+        WXK_(F22)
+        WXK_(F23)
+        WXK_(F24)
+        WXK_(NUMLOCK)
+        WXK_(SCROLL)
+        WXK_(PAGEUP)
+        WXK_(PAGEDOWN)
+        WXK_(NUMPAD_SPACE)
+        WXK_(NUMPAD_TAB)
+        WXK_(NUMPAD_ENTER)
+        WXK_(NUMPAD_F1)
+        WXK_(NUMPAD_F2)
+        WXK_(NUMPAD_F3)
+        WXK_(NUMPAD_F4)
+        WXK_(NUMPAD_HOME)
+        WXK_(NUMPAD_LEFT)
+        WXK_(NUMPAD_UP)
+        WXK_(NUMPAD_RIGHT)
+        WXK_(NUMPAD_DOWN)
+        WXK_(NUMPAD_PAGEUP)
+        WXK_(NUMPAD_PAGEDOWN)
+        WXK_(NUMPAD_END)
+        WXK_(NUMPAD_BEGIN)
+        WXK_(NUMPAD_INSERT)
+        WXK_(NUMPAD_DELETE)
+        WXK_(NUMPAD_EQUAL)
+        WXK_(NUMPAD_MULTIPLY)
+        WXK_(NUMPAD_ADD)
+        WXK_(NUMPAD_SEPARATOR)
+        WXK_(NUMPAD_SUBTRACT)
+        WXK_(NUMPAD_DECIMAL)
+        WXK_(NUMPAD_DIVIDE)
+#undef WXK_
+    default: 
+        return NULL; 
+    }

 }
 
 }
 

-void MyFrame::OnSize(wxSizeEvent& WXUNUSED(event))
+// helper function that returns textual description of key in the event
+wxString GetKeyName(const wxKeyEvent &event)

 {
 {

-    if ( m_winText )
-    {
-        wxSize size = GetClientSize();
-        m_winText->SetSize(0, 0, size.x, 50);
-        m_lboxLog->SetSize(0, 51, size.x, size.y - 50);
-    }
+    int keycode = event.GetKeyCode();
+    const char* virt = GetVirtualKeyCodeName(keycode);
+    if ( virt )
+        return virt;
+    if ( keycode > 0 && keycode < 27 )
+        return wxString::Format("Ctrl-%c", (unsigned char)('A' + keycode - 1));
+    if ( keycode >= 27 && keycode < 128 )
+        return wxString::Format("'%c'", (unsigned char)keycode);
+#if wxUSE_UNICODE
+    return wxString::Format("'%c'", event.GetUnicodeKey());
+#else
+    return "unknown";
+#endif

 }
 
 }
 

-// ----------------------------------------------------------------------------
-// TextWindow
-// ----------------------------------------------------------------------------

 
 

-void TextWindow::LogEvent(const wxChar *name, wxKeyEvent& event)
+void MyFrame::LogEvent(const wxString& name, wxKeyEvent& event)

 {
 {

-    wxString key;
-    long keycode = event.KeyCode();
-    switch ( keycode )
-    {
-        case WXK_BACK: key = _T("BACK"); break;
-        case WXK_TAB: key = _T("TAB"); break;
-        case WXK_RETURN: key = _T("RETURN"); break;
-        case WXK_ESCAPE: key = _T("ESCAPE"); break;
-        case WXK_SPACE: key = _T("SPACE"); break;
-        case WXK_DELETE: key = _T("DELETE"); break;
-        case WXK_START: key = _T("START"); break;
-        case WXK_LBUTTON: key = _T("LBUTTON"); break;
-        case WXK_RBUTTON: key = _T("RBUTTON"); break;
-        case WXK_CANCEL: key = _T("CANCEL"); break;
-        case WXK_MBUTTON: key = _T("MBUTTON"); break;
-        case WXK_CLEAR: key = _T("CLEAR"); break;
-        case WXK_SHIFT: key = _T("SHIFT"); break;
-        case WXK_ALT: key = _T("ALT"); break;
-        case WXK_CONTROL: key = _T("CONTROL"); break;
-        case WXK_MENU: key = _T("MENU"); break;
-        case WXK_PAUSE: key = _T("PAUSE"); break;
-        case WXK_CAPITAL: key = _T("CAPITAL"); break;
-        case WXK_PRIOR: key = _T("PRIOR"); break;
-        case WXK_NEXT: key = _T("NEXT"); break;
-        case WXK_END: key = _T("END"); break;
-        case WXK_HOME: key = _T("HOME"); break;
-        case WXK_LEFT: key = _T("LEFT"); break;
-        case WXK_UP: key = _T("UP"); break;
-        case WXK_RIGHT: key = _T("RIGHT"); break;
-        case WXK_DOWN: key = _T("DOWN"); break;
-        case WXK_SELECT: key = _T("SELECT"); break;
-        case WXK_PRINT: key = _T("PRINT"); break;
-        case WXK_EXECUTE: key = _T("EXECUTE"); break;
-        case WXK_SNAPSHOT: key = _T("SNAPSHOT"); break;
-        case WXK_INSERT: key = _T("INSERT"); break;
-        case WXK_HELP: key = _T("HELP"); break;
-        case WXK_NUMPAD0: key = _T("NUMPAD0"); break;
-        case WXK_NUMPAD1: key = _T("NUMPAD1"); break;
-        case WXK_NUMPAD2: key = _T("NUMPAD2"); break;
-        case WXK_NUMPAD3: key = _T("NUMPAD3"); break;
-        case WXK_NUMPAD4: key = _T("NUMPAD4"); break;
-        case WXK_NUMPAD5: key = _T("NUMPAD5"); break;
-        case WXK_NUMPAD6: key = _T("NUMPAD6"); break;
-        case WXK_NUMPAD7: key = _T("NUMPAD7"); break;
-        case WXK_NUMPAD8: key = _T("NUMPAD8"); break;
-        case WXK_NUMPAD9: key = _T("NUMPAD9"); break;
-        case WXK_MULTIPLY: key = _T("MULTIPLY"); break;
-        case WXK_ADD: key = _T("ADD"); break;
-        case WXK_SEPARATOR: key = _T("SEPARATOR"); break;
-        case WXK_SUBTRACT: key = _T("SUBTRACT"); break;
-        case WXK_DECIMAL: key = _T("DECIMAL"); break;
-        case WXK_DIVIDE: key = _T("DIVIDE"); break;
-        case WXK_F1: key = _T("F1"); break;
-        case WXK_F2: key = _T("F2"); break;
-        case WXK_F3: key = _T("F3"); break;
-        case WXK_F4: key = _T("F4"); break;
-        case WXK_F5: key = _T("F5"); break;
-        case WXK_F6: key = _T("F6"); break;
-        case WXK_F7: key = _T("F7"); break;
-        case WXK_F8: key = _T("F8"); break;
-        case WXK_F9: key = _T("F9"); break;
-        case WXK_F10: key = _T("F10"); break;
-        case WXK_F11: key = _T("F11"); break;
-        case WXK_F12: key = _T("F12"); break;
-        case WXK_F13: key = _T("F13"); break;
-        case WXK_F14: key = _T("F14"); break;
-        case WXK_F15: key = _T("F15"); break;
-        case WXK_F16: key = _T("F16"); break;
-        case WXK_F17: key = _T("F17"); break;
-        case WXK_F18: key = _T("F18"); break;
-        case WXK_F19: key = _T("F19"); break;
-        case WXK_F20: key = _T("F20"); break;
-        case WXK_F21: key = _T("F21"); break;
-        case WXK_F22: key = _T("F22"); break;
-        case WXK_F23: key = _T("F23"); break;
-        case WXK_F24: key = _T("F24"); break;
-        case WXK_NUMLOCK: key = _T("NUMLOCK"); break;
-        case WXK_SCROLL: key = _T("SCROLL"); break;
-        case WXK_PAGEUP: key = _T("PAGEUP"); break;
-        case WXK_PAGEDOWN: key = _T("PAGEDOWN"); break;
-        case WXK_NUMPAD_SPACE: key = _T("NUMPAD_SPACE"); break;
-        case WXK_NUMPAD_TAB: key = _T("NUMPAD_TAB"); break;
-        case WXK_NUMPAD_ENTER: key = _T("NUMPAD_ENTER"); break;
-        case WXK_NUMPAD_F1: key = _T("NUMPAD_F1"); break;
-        case WXK_NUMPAD_F2: key = _T("NUMPAD_F2"); break;
-        case WXK_NUMPAD_F3: key = _T("NUMPAD_F3"); break;
-        case WXK_NUMPAD_F4: key = _T("NUMPAD_F4"); break;
-        case WXK_NUMPAD_HOME: key = _T("NUMPAD_HOME"); break;
-        case WXK_NUMPAD_LEFT: key = _T("NUMPAD_LEFT"); break;
-        case WXK_NUMPAD_UP: key = _T("NUMPAD_UP"); break;
-        case WXK_NUMPAD_RIGHT: key = _T("NUMPAD_RIGHT"); break;
-        case WXK_NUMPAD_DOWN: key = _T("NUMPAD_DOWN"); break;
-        case WXK_NUMPAD_PRIOR: key = _T("NUMPAD_PRIOR"); break;
-        case WXK_NUMPAD_PAGEUP: key = _T("NUMPAD_PAGEUP"); break;
-        case WXK_NUMPAD_PAGEDOWN: key = _T("NUMPAD_PAGEDOWN"); break;
-        case WXK_NUMPAD_END: key = _T("NUMPAD_END"); break;
-        case WXK_NUMPAD_BEGIN: key = _T("NUMPAD_BEGIN"); break;
-        case WXK_NUMPAD_INSERT: key = _T("NUMPAD_INSERT"); break;
-        case WXK_NUMPAD_DELETE: key = _T("NUMPAD_DELETE"); break;
-        case WXK_NUMPAD_EQUAL: key = _T("NUMPAD_EQUAL"); break;
-        case WXK_NUMPAD_MULTIPLY: key = _T("NUMPAD_MULTIPLY"); break;
-        case WXK_NUMPAD_ADD: key = _T("NUMPAD_ADD"); break;
-        case WXK_NUMPAD_SEPARATOR: key = _T("NUMPAD_SEPARATOR"); break;
-        case WXK_NUMPAD_SUBTRACT: key = _T("NUMPAD_SUBTRACT"); break;
-        case WXK_NUMPAD_DECIMAL: key = _T("NUMPAD_DECIMAL"); break;
-        case WXK_NUMPAD_DIVIDE: key = _T("NUMPAD_DIVIDE"); break;
-
-        default:
-        {
-            if ( keycode < 256 )
-            {
-                if ( keycode == 0 )
-                    key.Printf(_T("NUL"));
-                else if ( keycode < 27 )
-                    key.Printf(_T("Ctrl-%c"),
-                                (unsigned char)(_T('A') + keycode - 1));
-                else
-                    key.Printf(_T("'%c'"), (unsigned char)keycode);
-            }
-            else
-            {
-                key.Printf(_T("unknown (%ld)"), keycode);
-            }
-        }
-    }
-

     wxString msg;
     wxString msg;

-    msg.Printf(_T("%s event: %s (flags = %c%c%c%c)"),
+    // event  key_name  KeyCode  modifiers  Unicode  raw_code raw_flags 
+    msg.Printf("%7s %15s %5d   %c%c%c%c"
+#if wxUSE_UNICODE
+                   "%5d (U+%04x)"
+#else
+                   "    none   "
+#endif
+#ifdef wxHAS_RAW_KEY_CODES
+                   "  %7lu    0x%lx"
+#else
+                   "  not-set    not-set"
+#endif
+                   "\n",

                name,
                name,

-               key.c_str(),
-               GetChar(event.ControlDown(), _T('C')),
-               GetChar(event.AltDown(), _T('A')),
-               GetChar(event.ShiftDown(), _T('S')),
-               GetChar(event.MetaDown(), _T('M')));
-
-    if ( m_showRaw )
-    {
-        msg += wxString::Format(_T(" (raw key code/flags: %lu and 0x%lx)"),
-                                (unsigned long)event.GetRawKeyCode(),
-                                (unsigned long)event.GetRawKeyFlags());
-    }
+               GetKeyName(event),
+               event.GetKeyCode(),
+               event.ControlDown() ? 'C' : '-',
+               event.AltDown()     ? 'A' : '-',
+               event.ShiftDown()   ? 'S' : '-',
+               event.MetaDown()    ? 'M' : '-'
+#if wxUSE_UNICODE
+               , event.GetUnicodeKey()
+               , event.GetUnicodeKey()
+#endif
+#ifdef wxHAS_RAW_KEY_CODES
+               , (unsigned long) event.GetRawKeyCode()
+               , (unsigned long) event.GetRawKeyFlags()
+#endif
+               );

 
 

-    wxLogMessage(msg);
+    m_logText->AppendText(msg);

 
     if ( m_skip )
 
     if ( m_skip )

-    {

         event.Skip();
         event.Skip();

-    }

 }
 }

+
+