Fix a crash in wxExecute() in wxMSW too.

[wxWidgets.git] / samples / taskbar / tbtest.cpp

diff --git a/samples/taskbar/tbtest.cpp b/samples/taskbar/tbtest.cpp
index 9256009cdd99e75ab814c25c40dd526fff4f54c2..c31128f06b17e9346907f6c94c0bc0d496ca37ce 100644 (file)
--- a/samples/taskbar/tbtest.cpp
+++ b/samples/taskbar/tbtest.cpp
@@ -29,7 +29,7 @@
 #endif
 
 // the application icon (under Windows and OS/2 it is in resources)
 #endif
 
 // the application icon (under Windows and OS/2 it is in resources)

-#if defined(__WXGTK__) || defined(__WXMOTIF__) || defined(__WXMAC__) || defined(__WXMGL__) || defined(__WXX11__)
+#ifndef wxHAS_IMAGES_IN_RESOURCES

     #include "../sample.xpm"
 #endif
 
     #include "../sample.xpm"
 #endif
 


@@ -103,31 +103,28 @@ MyDialog::MyDialog(const wxString& title)
                       (
                         this,
                         wxID_ANY,
                       (
                         this,
                         wxID_ANY,

-                        _T("Press 'Hide me' to hide this window, Exit to quit.")
+                        wxT("Press 'Hide me' to hide this window, Exit to quit.")

                       ), flags);
 
     sizerTop->Add(new wxStaticText
                       (
                         this,
                         wxID_ANY,
                       ), flags);
 
     sizerTop->Add(new wxStaticText
                       (
                         this,
                         wxID_ANY,

-                        _T("Double-click on the taskbar icon to show me again.")
+                        wxT("Double-click on the taskbar icon to show me again.")

                       ), flags);
 
     sizerTop->AddStretchSpacer()->SetMinSize(200, 50);
 
     wxSizer * const sizerBtns = new wxBoxSizer(wxHORIZONTAL);
                       ), flags);
 
     sizerTop->AddStretchSpacer()->SetMinSize(200, 50);
 
     wxSizer * const sizerBtns = new wxBoxSizer(wxHORIZONTAL);

-    sizerBtns->Add(new wxButton(this, wxID_ABOUT, _T("&About")), flags);
-    sizerBtns->Add(new wxButton(this, wxID_OK, _T("&Hide")), flags);
-    sizerBtns->Add(new wxButton(this, wxID_EXIT, _T("E&xit")), flags);
+    sizerBtns->Add(new wxButton(this, wxID_ABOUT, wxT("&About")), flags);
+    sizerBtns->Add(new wxButton(this, wxID_OK, wxT("&Hide")), flags);
+    sizerBtns->Add(new wxButton(this, wxID_EXIT, wxT("E&xit")), flags);

 
     sizerTop->Add(sizerBtns, flags.Align(wxALIGN_CENTER_HORIZONTAL));
     SetSizerAndFit(sizerTop);
     Centre();
 
     m_taskBarIcon = new MyTaskBarIcon();
 
     sizerTop->Add(sizerBtns, flags.Align(wxALIGN_CENTER_HORIZONTAL));
     SetSizerAndFit(sizerTop);
     Centre();
 
     m_taskBarIcon = new MyTaskBarIcon();

-#if defined(__WXCOCOA__)
-    m_dockIcon = new MyTaskBarIcon(wxTaskBarIcon::DOCK);
-#endif

 
     // we should be able to show up to 128 characters on recent Windows versions
     // (and 64 on Win9x)
 
     // we should be able to show up to 128 characters on recent Windows versions
     // (and 64 on Win9x)


@@ -139,6 +136,14 @@ MyDialog::MyDialog(const wxString& title)
     {
         wxLogError(wxT("Could not set icon."));
     }
     {
         wxLogError(wxT("Could not set icon."));
     }

+
+#if defined(__WXOSX__) && wxOSX_USE_COCOA
+    m_dockIcon = new MyTaskBarIcon(wxTBI_DOCK);
+    if ( !m_dockIcon->SetIcon(wxICON(sample)) )
+    {
+        wxLogError(wxT("Could not set icon."));
+    }
+#endif

 }
 
 MyDialog::~MyDialog()
 }
 
 MyDialog::~MyDialog()


@@ -247,21 +252,25 @@ void MyTaskBarIcon::OnMenuSub(wxCommandEvent&)
 wxMenu *MyTaskBarIcon::CreatePopupMenu()
 {
     wxMenu *menu = new wxMenu;
 wxMenu *MyTaskBarIcon::CreatePopupMenu()
 {
     wxMenu *menu = new wxMenu;

-    menu->Append(PU_RESTORE, _T("&Restore main window"));
+    menu->Append(PU_RESTORE, wxT("&Restore main window"));

     menu->AppendSeparator();
     menu->AppendSeparator();

-    menu->Append(PU_NEW_ICON, _T("&Set New Icon"));
+    menu->Append(PU_NEW_ICON, wxT("&Set New Icon"));

     menu->AppendSeparator();
     menu->AppendSeparator();

-    menu->AppendCheckItem(PU_CHECKMARK, _T("Test &check mark"));
+    menu->AppendCheckItem(PU_CHECKMARK, wxT("Test &check mark"));

     menu->AppendSeparator();
     wxMenu *submenu = new wxMenu;
     menu->AppendSeparator();
     wxMenu *submenu = new wxMenu;

-    submenu->Append(PU_SUB1, _T("One submenu"));
+    submenu->Append(PU_SUB1, wxT("One submenu"));

     submenu->AppendSeparator();
     submenu->AppendSeparator();

-    submenu->Append(PU_SUB2, _T("Another submenu"));
-    menu->Append(PU_SUBMAIN, _T("Submenu"), submenu);
-#ifndef __WXMAC_OSX__ /*Mac has built-in quit menu*/
-    menu->AppendSeparator();
-    menu->Append(PU_EXIT,    _T("E&xit"));
+    submenu->Append(PU_SUB2, wxT("Another submenu"));
+    menu->Append(PU_SUBMAIN, wxT("Submenu"), submenu);
+    /* OSX has built-in quit menu for the dock menu, but not for the status item */
+#ifdef __WXOSX__ 
+    if ( OSXIsStatusItem() )

 #endif
 #endif

+    {
+        menu->AppendSeparator();
+        menu->Append(PU_EXIT,    wxT("E&xit"));
+    }

     return menu;
 }
 
     return menu;
 }