cleaned IPC samples a bit

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@5433 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/samples/ipc/client.cpp b/samples/ipc/client.cpp
index 616ef99d80f091ae9611d5972f6f638c498ba036..99940537f19ba66d23cd090d5ec5e2364a98ee2e 100644 (file)
--- a/samples/ipc/client.cpp
+++ b/samples/ipc/client.cpp
@@ -9,15 +9,23 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

+// ============================================================================
+// declarations
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// headers
+// ----------------------------------------------------------------------------
+

 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP

-#include "wx/wx.h"
+    #include "wx/wx.h"

 #endif
 
 // Settings common to both executables: determines whether
 #endif
 
 // Settings common to both executables: determines whether


@@ -31,189 +39,191 @@
 
 #include "client.h"
 
 
 #include "client.h"
 

-MyFrame *frame = NULL;
+// ----------------------------------------------------------------------------
+// wxWin macros
+// ----------------------------------------------------------------------------

 
 IMPLEMENT_APP(MyApp)
 
 
 IMPLEMENT_APP(MyApp)
 

+BEGIN_EVENT_TABLE(MyFrame, wxFrame)
+    EVT_MENU(CLIENT_QUIT, MyFrame::OnExit)
+    EVT_MENU(CLIENT_EXECUTE, MyFrame::OnExecute)
+    EVT_MENU(CLIENT_POKE, MyFrame::OnPoke)
+    EVT_MENU(CLIENT_REQUEST, MyFrame::OnRequest)
+END_EVENT_TABLE()
+
+// ----------------------------------------------------------------------------
+// globals
+// ----------------------------------------------------------------------------
+

 char ipc_buffer[4000];
 wxListBox *the_list = NULL;
 
 MyConnection *the_connection = NULL;
 MyClient *my_client ;
 
 char ipc_buffer[4000];
 wxListBox *the_list = NULL;
 
 MyConnection *the_connection = NULL;
 MyClient *my_client ;
 

+// ============================================================================
+// implementation
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// MyApp
+// ----------------------------------------------------------------------------
+

 // The `main program' equivalent, creating the windows and returning the
 // main frame
 bool MyApp::OnInit()
 {
 // The `main program' equivalent, creating the windows and returning the
 // main frame
 bool MyApp::OnInit()
 {

-  // Create the main frame window
-  frame = new MyFrame(NULL, "Client", wxPoint(400, 0), wxSize(400, 300));
-
-  // Give it an icon
-  frame->SetIcon(wxICON(mondrian));
-
-  // Make a menubar
-  wxMenu *file_menu = new wxMenu;
-
-  file_menu->Append(CLIENT_EXECUTE, "Execute");
-  file_menu->Append(CLIENT_REQUEST, "Request");
-  file_menu->Append(CLIENT_POKE, "Poke");
-  file_menu->Append(CLIENT_QUIT, "Quit");
+    wxString server = "4242";
+#if wxUSE_DDE_FOR_SAMPLE
+    wxString hostName = wxGetHostName();
+#else
+    wxString hostName = "localhost";
+#endif

 
 

-  wxMenuBar *menu_bar = new wxMenuBar;
+    if (argc > 1)
+        server = argv[1];
+    if (argc > 2)
+        hostName = argv[2];

 
 

-  menu_bar->Append(file_menu, "File");
+    // Create a new client
+    my_client = new MyClient;
+    the_connection = (MyConnection *)my_client->MakeConnection(hostName, server, "IPC TEST");

 
 

-  // Associate the menu bar with the frame
-  frame->SetMenuBar(menu_bar);
+    while ( !the_connection )
+    {
+        if ( wxMessageBox("Failed to make connection to server.\nRetry?",
+                          "Client Demo Error",
+                          wxICON_ERROR | wxYES_NO | wxCANCEL ) != wxYES )
+        {
+            // no server
+            return FALSE;
+        }

 
 

-  // Make a panel
-  frame->panel = new wxPanel(frame, -1, wxPoint(0, 0), wxSize(400, 250));
-  the_list = new wxListBox(frame->panel, CLIENT_LISTBOX, wxPoint(5, 5), wxSize(150, 120));
-  the_list->Append("Apple");
-  the_list->Append("Pear");
-  the_list->Append("Orange");
-  the_list->Append("Banana");
-  the_list->Append("Fruit");
+        the_connection = (MyConnection *)my_client->MakeConnection(hostName, server, "IPC TEST");
+    }

 
 

-  frame->panel->Fit();
-  frame->Fit();
+    if (!the_connection->StartAdvise("Item"))
+        wxMessageBox("StartAdvise failed", "Client Demo Error");

 
 

-  wxString server = "4242";
+    // Create the main frame window
+    (new MyFrame(NULL, "Client"))->Show(TRUE);

 
 

-#if wxUSE_DDE_FOR_SAMPLE
-  wxString hostName = wxGetHostName();
-#else
-  wxString hostName = "localhost";
-#endif
-
-  if (argc > 1)
-    server = argv[1];
-  if (argc > 2)
-    hostName = argv[2];
-
-  // Create a new client
-  my_client = new MyClient;
-  the_connection = (MyConnection *)my_client->MakeConnection(hostName, server, "IPC TEST");
-
-  if (!the_connection)
-  {
-    wxMessageBox("Failed to make connection to server", "Client Demo Error");
-#ifdef __WXMSW__
-//    extern void wxPrintDDEError();
-//    wxPrintDDEError();
-#endif
-    return FALSE;
-  }
-  if (!the_connection->StartAdvise("Item"))
-    wxMessageBox("StartAdvise failed", "Client Demo Error");
-
-  frame->Show(TRUE);
-
-  return TRUE;
+    return TRUE;

 }
 
 int MyApp::OnExit()
 {
 }
 
 int MyApp::OnExit()
 {

-    if (my_client)
-      delete my_client ;
+    if (the_connection)
+    {
+        the_connection->Disconnect();
+    }
+
+    // will delete the connection too
+    delete my_client;

 
     return 0;
 }
 
 
     return 0;
 }
 

-BEGIN_EVENT_TABLE(MyFrame, wxFrame)
-    EVT_MENU(CLIENT_QUIT, MyFrame::OnExit)
-    EVT_MENU(CLIENT_EXECUTE, MyFrame::OnExecute)
-    EVT_MENU(CLIENT_POKE, MyFrame::OnPoke)
-    EVT_MENU(CLIENT_REQUEST, MyFrame::OnRequest)
-    EVT_CLOSE(MyFrame::OnCloseWindow)
-END_EVENT_TABLE()
-

 // Define my frame constructor
 // Define my frame constructor

-MyFrame::MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos, const wxSize& size):
-  wxFrame(frame, -1, title, pos, size)
+MyFrame::MyFrame(wxFrame *frame, const wxString& title)
+        : wxFrame(frame, -1, title)

 {
     panel = NULL;
 {
     panel = NULL;

+
+    // Give it an icon
+    SetIcon(wxICON(mondrian));
+
+    // Make a menubar
+    wxMenu *file_menu = new wxMenu;
+
+    file_menu->Append(CLIENT_EXECUTE, "Execute");
+    file_menu->Append(CLIENT_REQUEST, "Request");
+    file_menu->Append(CLIENT_POKE, "Poke");
+    file_menu->Append(CLIENT_QUIT, "Quit");
+
+    wxMenuBar *menu_bar = new wxMenuBar;
+
+    menu_bar->Append(file_menu, "File");
+
+    // Associate the menu bar with the frame
+    SetMenuBar(menu_bar);
+
+    // Make a panel
+    panel = new wxPanel(this);
+    the_list = new wxListBox(panel, CLIENT_LISTBOX, wxPoint(5, 5));
+    the_list->Append("Apple");
+    the_list->Append("Pear");
+    the_list->Append("Orange");
+    the_list->Append("Banana");
+    the_list->Append("Fruit");
+
+    panel->Fit();
+    Fit();

 }
 
 void MyFrame::OnExecute(wxCommandEvent& event)
 {
 }
 
 void MyFrame::OnExecute(wxCommandEvent& event)
 {

-      if (the_connection)
+    if (the_connection)

         if (!the_connection->Execute("Hello from the client!"))
         if (!the_connection->Execute("Hello from the client!"))

-          wxMessageBox("Execute failed", "Client Demo Error");
+            wxMessageBox("Execute failed", "Client Demo Error");

 }
 
 void MyFrame::OnPoke(wxCommandEvent& event)
 {
 }
 
 void MyFrame::OnPoke(wxCommandEvent& event)
 {

-      if (the_connection)
+    if (the_connection)

         if (!the_connection->Poke("An item", "Some data to poke at the server!"))
         if (!the_connection->Poke("An item", "Some data to poke at the server!"))

-          wxMessageBox("Poke failed", "Client Demo Error");
+            wxMessageBox("Poke failed", "Client Demo Error");

 }
 
 void MyFrame::OnRequest(wxCommandEvent& event)
 {
 }
 
 void MyFrame::OnRequest(wxCommandEvent& event)
 {

-      if (the_connection)
-      {
+    if (the_connection)
+    {

         char *data = the_connection->Request("An item");
         if (data)
         char *data = the_connection->Request("An item");
         if (data)

-          wxMessageBox(data, "Client: Request", wxOK);
+            wxMessageBox(data, "Client: Request", wxOK);

         else
         else

-          wxMessageBox("Request failed", "Client Demo Error");
-      }
+            wxMessageBox("Request failed", "Client Demo Error");
+    }

 }
 
 void MyFrame::OnExit(wxCommandEvent& event)
 {
 }
 
 void MyFrame::OnExit(wxCommandEvent& event)
 {

-  if (the_connection)
-    the_connection->Disconnect();
-
-  this->Destroy();
+    Close();

 }
 
 }
 

-// Define the behaviour for the frame closing
-void MyFrame::OnCloseWindow(wxCloseEvent& event)
+wxConnectionBase *MyClient::OnMakeConnection()

 {
 {

-  if (the_connection)
-  {
-    the_connection->Disconnect();
-  }
-  this->Destroy();
+    return new MyConnection;

 }
 
 }
 

-MyClient::MyClient(void)
+MyConnection::MyConnection()
+            : wxConnection(ipc_buffer, WXSIZEOF(ipc_buffer))

 {
 }
 
 {
 }
 

-wxConnectionBase *MyClient::OnMakeConnection(void)
+MyConnection::~MyConnection()

 {
 {

-  return new MyConnection;
-}
-
-MyConnection::MyConnection(void):wxConnection(ipc_buffer, 3999)
-{
-}
-
-MyConnection::~MyConnection(void)
-{
-  the_connection = NULL;
+    the_connection = NULL;

 }
 
 bool MyConnection::OnAdvise(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format)
 {
 }
 
 bool MyConnection::OnAdvise(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format)
 {

-  if (the_list)
-  {
-    int n = the_list->FindString(data);
-    if (n > -1)
-      the_list->SetSelection(n);
-  }
-  return TRUE;
+    if (the_list)
+    {
+        int n = the_list->FindString(data);
+        if (n > -1)
+            the_list->SetSelection(n);
+    }
+    return TRUE;

 }
 
 bool MyConnection::OnDisconnect()
 {
 }
 
 bool MyConnection::OnDisconnect()
 {

-    frame->Destroy();
-
-    the_connection = NULL;
-    delete this;
+    wxWindow *win = wxTheApp->GetTopWindow();
+    if ( win )
+        win->Destroy();

 
     return TRUE;
 }
 
     return TRUE;
 }

diff --git a/samples/ipc/client.h b/samples/ipc/client.h
index 8017a29def07e89519527a55927994f2fec32c76..8dcdc7dbe4f71d069c7077e8fe2d1971d672b0cf 100644 (file)
--- a/samples/ipc/client.h
+++ b/samples/ipc/client.h
@@ -12,40 +12,42 @@
 // Define a new application
 class MyApp: public wxApp
 {
 // Define a new application
 class MyApp: public wxApp
 {

-  public:
-    bool OnInit();
-    int OnExit();
+public:
+    virtual bool OnInit();
+    virtual int OnExit();

 };
 
 // Define a new frame
 class MyFrame: public wxFrame
 {
 };
 
 // Define a new frame
 class MyFrame: public wxFrame
 {

-  public:
-    wxPanel *panel;
+public:
+    MyFrame(wxFrame *frame, const wxString& title);

 
 

-    MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos, const wxSize& size);
-    void OnCloseWindow(wxCloseEvent& event);

     void OnExit(wxCommandEvent& event);
     void OnExecute(wxCommandEvent& event);
     void OnPoke(wxCommandEvent& event);
     void OnRequest(wxCommandEvent& event);
     void OnExit(wxCommandEvent& event);
     void OnExecute(wxCommandEvent& event);
     void OnPoke(wxCommandEvent& event);
     void OnRequest(wxCommandEvent& event);

-DECLARE_EVENT_TABLE()
+
+private:
+    wxPanel *panel;
+
+    DECLARE_EVENT_TABLE()

 };
 
 class MyConnection: public wxConnection
 {
 };
 
 class MyConnection: public wxConnection
 {

- public:
-  MyConnection();
-  ~MyConnection();
-  bool OnAdvise(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format);
-  bool OnDisconnect();
+public:
+    MyConnection();
+    ~MyConnection();
+
+    bool OnAdvise(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format);
+    bool OnDisconnect();

 };
 
 class MyClient: public wxClient
 {
 };
 
 class MyClient: public wxClient
 {

- public:
-  MyClient();
-  wxConnectionBase *OnMakeConnection();
+public:
+    wxConnectionBase *OnMakeConnection();

 };
 
 #define CLIENT_QUIT     wxID_EXIT
 };
 
 #define CLIENT_QUIT     wxID_EXIT

diff --git a/samples/ipc/server.cpp b/samples/ipc/server.cpp
index 3dfb36562fed1c6fda1073d85b86d17302210622..4763f877e8669bbcbcb599fe49fc4b768a1b164c 100644 (file)
--- a/samples/ipc/server.cpp
+++ b/samples/ipc/server.cpp
@@ -1,6 +1,6 @@
 /////////////////////////////////////////////////////////////////////////////
 // Name:        server.cpp
 /////////////////////////////////////////////////////////////////////////////
 // Name:        server.cpp

-// Purpose:     DDE sample: server
+// Purpose:     IPC sample: server

 // Author:      Julian Smart
 // Modified by:
 // Created:     25/01/99
 // Author:      Julian Smart
 // Modified by:
 // Created:     25/01/99


@@ -9,101 +9,126 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

+// ============================================================================
+// declarations
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// headers
+// ----------------------------------------------------------------------------
+

 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP

-#include "wx/wx.h"
+    #include "wx/wx.h"

 #endif
 
 // Settings common to both executables: determines whether
 // we're using TCP/IP or real DDE.
 #endif
 
 // Settings common to both executables: determines whether
 // we're using TCP/IP or real DDE.

-

 #include "ddesetup.h"
 
 #if defined(__WXGTK__) || defined(__WXMOTIF__)
 #include "ddesetup.h"
 
 #if defined(__WXGTK__) || defined(__WXMOTIF__)

-#include "mondrian.xpm"
+    #include "mondrian.xpm"

 #endif
 
 #include "server.h"
 
 #endif
 
 #include "server.h"
 

-MyFrame *frame = NULL;
+// ----------------------------------------------------------------------------
+// wxWin macros
+// ----------------------------------------------------------------------------

 
 IMPLEMENT_APP(MyApp)
 
 
 IMPLEMENT_APP(MyApp)
 

+BEGIN_EVENT_TABLE(MyFrame, wxFrame)
+    EVT_LISTBOX(SERVER_LISTBOX, MyFrame::OnListBoxClick)
+END_EVENT_TABLE()
+
+BEGIN_EVENT_TABLE(IPCDialogBox, wxDialog)
+    EVT_BUTTON(SERVER_QUIT_BUTTON, IPCDialogBox::OnQuit)
+END_EVENT_TABLE()
+
+// ----------------------------------------------------------------------------
+// global variables
+// ----------------------------------------------------------------------------
+

 char ipc_buffer[4000];
 MyConnection *the_connection = NULL;
 char ipc_buffer[4000];
 MyConnection *the_connection = NULL;

-MyServer *my_server ;
+
+// ============================================================================
+// implementation
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// MyApp
+// ----------------------------------------------------------------------------

 
 bool MyApp::OnInit()
 {
 
 bool MyApp::OnInit()
 {

-  // Create the main frame window
-  frame = new MyFrame(NULL, "Server", wxDefaultPosition, wxSize(400, 500));
+    // Create the main frame window
+    (new MyFrame(NULL, "Server"))->Show();

 
 

-  frame->CreateStatusBar();
+    // create the server object
+    wxString server_name = "4242";
+    if (argc > 1)
+        server_name = argv[1];

 
 

-  // Give it an icon
-  frame->SetIcon(wxICON(mondrian));
+    // Create a new server
+    m_server = new MyServer;
+    m_server->Create(server_name);

 
 

-  // Make a menubar
-  wxMenu *file_menu = new wxMenu;
+    return TRUE;
+}

 
 

-  file_menu->Append(SERVER_QUIT, "&Exit");
+int MyApp::OnExit()
+{
+    delete m_server;
+
+    return 0;
+}

 
 

-  wxMenuBar *menu_bar = new wxMenuBar;
+// ----------------------------------------------------------------------------
+// MyFrame
+// ----------------------------------------------------------------------------

 
 

-  menu_bar->Append(file_menu, "&File");
+// Define my frame constructor
+MyFrame::MyFrame(wxFrame *frame, const wxString& title)
+       : wxFrame(frame, -1, title)
+{
+    panel = NULL;

 
 

-  // Associate the menu bar with the frame
-  frame->SetMenuBar(menu_bar);
+    CreateStatusBar();

 
 

-  // Make a panel
-  frame->panel = new wxPanel(frame, 0, 0, 400, 250);
-  wxListBox *list = new wxListBox(frame->panel, SERVER_LISTBOX,
-                                  wxPoint(5, 5), wxSize(150, 120));
-  list->Append("Apple");
-  list->Append("Pear");
-  list->Append("Orange");
-  list->Append("Banana");
-  list->Append("Fruit");
+    // Give it an icon
+    SetIcon(wxICON(mondrian));

 
 

-  frame->panel->Fit();
-  frame->Fit();
+    // Make a menubar
+    wxMenu *file_menu = new wxMenu;

 
 

-  wxString server_name = "4242";
-  if (argc > 1)
-    server_name = argv[1];
+    file_menu->Append(SERVER_QUIT, "&Exit");

 
 

-  // Create a new server
-  my_server = new MyServer;
-  my_server->Create(server_name);
-  frame->Show(TRUE);
+    wxMenuBar *menu_bar = new wxMenuBar;

 
 

-  return TRUE;
-}
+    menu_bar->Append(file_menu, "&File");

 
 

-BEGIN_EVENT_TABLE(MyFrame, wxFrame)
-    EVT_MENU(SERVER_QUIT, MyFrame::OnExit)
-    EVT_CLOSE(MyFrame::OnCloseWindow)
-    EVT_LISTBOX(SERVER_LISTBOX, MyFrame::OnListBoxClick)
-END_EVENT_TABLE()
+    // Associate the menu bar with the frame
+    SetMenuBar(menu_bar);

 
 

-// Define my frame constructor
-MyFrame::MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos, const wxSize& size):
-  wxFrame(frame, -1, title, pos, size)
-{
-  panel = NULL;
-}
+    // Make a panel
+    panel = new wxPanel(this);
+    wxListBox *list = new wxListBox(panel, SERVER_LISTBOX, wxPoint(5, 5));
+    list->Append("Apple");
+    list->Append("Pear");
+    list->Append("Orange");
+    list->Append("Banana");
+    list->Append("Fruit");

 
 

-void MyFrame::OnExit(wxCommandEvent& event)
-{
-    if (my_server)
-        delete my_server;
-    this->Destroy();
+    panel->Fit();
+    Fit();

 }
 
 // Set the client process's listbox to this item
 }
 
 // Set the client process's listbox to this item


@@ -115,84 +140,96 @@ void MyFrame::OnListBoxClick(wxCommandEvent& event)
         wxString value = listBox->GetStringSelection();
         if (the_connection)
         {
         wxString value = listBox->GetStringSelection();
         if (the_connection)
         {

-            the_connection->Advise("Item", (char*) (const char*) value);
+            the_connection->Advise("Item", (wxChar *)value.c_str());

         }
     }
 }
 
         }
     }
 }
 

-void MyFrame::OnCloseWindow(wxCloseEvent& event)
-{
-    if (my_server)
-        delete my_server;
-    this->Destroy();
-}
-
-BEGIN_EVENT_TABLE(IPCDialogBox, wxDialog)
-    EVT_BUTTON(SERVER_QUIT_BUTTON, IPCDialogBox::OnQuit)
-END_EVENT_TABLE()
+// ----------------------------------------------------------------------------
+// IPCDialogBox
+// ----------------------------------------------------------------------------

 
 

-IPCDialogBox::IPCDialogBox(wxFrame *parent, const wxString& title,
-                         const wxPoint& pos, const wxSize& size, MyConnection *the_connection):
-                     wxDialog(parent, -1, title, pos, size)
+IPCDialogBox::IPCDialogBox(wxWindow *parent, const wxString& title,
+                           const wxPoint& pos, const wxSize& size,
+                           MyConnection *connection)
+            : wxDialog(parent, -1, title, pos, size)

 {
 {

-  connection = the_connection;
-  (void)new wxButton(this, SERVER_QUIT_BUTTON, "Quit this connection", wxPoint(5, 5));
-  Fit();
+    m_connection = connection;
+    (void)new wxButton(this, SERVER_QUIT_BUTTON, "Quit this connection",
+                       wxPoint(5, 5));
+    Fit();

 }
 
 void IPCDialogBox::OnQuit(wxCommandEvent& event)
 {
 }
 
 void IPCDialogBox::OnQuit(wxCommandEvent& event)
 {

-  connection->Disconnect();
-  delete connection;
+    m_connection->Disconnect();
+    delete m_connection;

 }
 
 }
 

+// ----------------------------------------------------------------------------
+// MyServer
+// ----------------------------------------------------------------------------
+

 wxConnectionBase *MyServer::OnAcceptConnection(const wxString& topic)
 {
 wxConnectionBase *MyServer::OnAcceptConnection(const wxString& topic)
 {

-  if (strcmp(topic, "STDIO") != 0 && strcmp(topic, "IPC TEST") == 0)
-    return new MyConnection(ipc_buffer, 4000);
-  else
-    return NULL;
+    if (strcmp(topic, "STDIO") != 0 && strcmp(topic, "IPC TEST") == 0)
+        return new MyConnection(ipc_buffer, WXSIZEOF(ipc_buffer));
+    else
+        return NULL;

 }
 
 }
 

-MyConnection::MyConnection(char *buf, int size):wxConnection(buf, size)
+// ----------------------------------------------------------------------------
+// MyConnection
+// ----------------------------------------------------------------------------
+
+MyConnection::MyConnection(char *buf, int size)
+            : wxConnection(buf, size)

 {
 {

-  dialog = new IPCDialogBox(frame, "Connection", wxPoint(100, 100), wxSize(500, 500), this);
-  dialog->Show(TRUE);
-  the_connection = this;
+    dialog = new IPCDialogBox(wxTheApp->GetTopWindow(), "Connection",
+                              wxPoint(100, 100), wxSize(500, 500), this);
+    dialog->Show(TRUE);
+    the_connection = this;

 }
 
 }
 

-MyConnection::~MyConnection(void)
+MyConnection::~MyConnection()

 {
 {

-  if (the_connection)
-  {
-    dialog->Destroy();
-    the_connection = NULL;
-  }
+    if (the_connection)
+    {
+        dialog->Destroy();
+        the_connection = NULL;
+    }

 }
 
 }
 

-bool MyConnection::OnExecute(const wxString& topic, char *data, int size, wxIPCFormat format)
+bool MyConnection::OnExecute(const wxString& WXUNUSED(topic),
+                             char *data,
+                             int WXUNUSED(size),
+                             wxIPCFormat WXUNUSED(format))

 {
 {

-  char buf[300];
-  sprintf(buf, "Execute command: %s", data);
-  frame->SetStatusText(buf);
-  return TRUE;
+    wxLogStatus("Execute command: %s", data);
+    return TRUE;

 }
 
 }
 

-bool MyConnection::OnPoke(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format)
+bool MyConnection::OnPoke(const wxString& WXUNUSED(topic),
+                          const wxString& item,
+                          char *data,
+                          int WXUNUSED(size),
+                          wxIPCFormat WXUNUSED(format))

 {
 {

-  char buf[300];
-  sprintf(buf, "Poke command: %s", data);
-  frame->SetStatusText(buf);
-  return TRUE;
+    wxLogStatus("Poke command: %s = %s", item, data);
+    return TRUE;

 }
 
 }
 

-char *MyConnection::OnRequest(const wxString& topic, const wxString& item, int *size, wxIPCFormat format)
+char *MyConnection::OnRequest(const wxString& WXUNUSED(topic),
+                              const wxString& WXUNUSED(item),
+                              int * WXUNUSED(size),
+                              wxIPCFormat WXUNUSED(format))

 {
 {

-  return "Here, have your data, client!";
+    return "Here, have your data, client!";

 }
 
 }
 

-bool MyConnection::OnStartAdvise(const wxString& topic, const wxString& item)
+bool MyConnection::OnStartAdvise(const wxString& WXUNUSED(topic),
+                                 const wxString& WXUNUSED(item))

 {
 {

-  return TRUE;
+    return TRUE;

 }
 
 }
 

diff --git a/samples/ipc/server.h b/samples/ipc/server.h
index 81aa31697bcc30bd79f5bd768bd3810fc960102a..03b358b1f46a7b26e1b01414d5eda5a26197a32b 100644 (file)
--- a/samples/ipc/server.h
+++ b/samples/ipc/server.h
@@ -10,41 +10,47 @@
 /////////////////////////////////////////////////////////////////////////////
 
 // Define a new application
 /////////////////////////////////////////////////////////////////////////////
 
 // Define a new application

-class MyApp: public wxApp
+class MyServer;
+class MyApp : public wxApp

 {
 {

-  public:
-    bool OnInit();
+public:
+    virtual bool OnInit();
+    virtual int OnExit();
+
+private:
+    MyServer *m_server;

 };
 
 DECLARE_APP(MyApp)
 
 // Define a new frame
 };
 
 DECLARE_APP(MyApp)
 
 // Define a new frame

-class MyFrame: public wxFrame
+class MyFrame : public wxFrame

 {
 {

-  public:
-    wxPanel *panel;
-
-    MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos, const wxSize& size);
+public:
+    MyFrame(wxFrame *frame, const wxString& title);

 
 

-    void OnCloseWindow(wxCloseEvent& event);
-    void OnExit(wxCommandEvent& event);

     void OnListBoxClick(wxCommandEvent& event);
     void OnListBoxClick(wxCommandEvent& event);

-DECLARE_EVENT_TABLE()
+
+private:
+    wxPanel *panel;
+
+    DECLARE_EVENT_TABLE()

 };
 
 class IPCDialogBox;
 };
 
 class IPCDialogBox;

-class MyConnection: public wxConnection
+class MyConnection : public wxConnection

 {
 {

- public:
-  IPCDialogBox *dialog;
+public:
+    MyConnection(char *buf, int size);
+    ~MyConnection();

 
 

-  MyConnection(char *buf, int size);
-  ~MyConnection();
+    bool OnExecute(const wxString& topic, char *data, int size, wxIPCFormat format);
+    char *OnRequest(const wxString& topic, const wxString& item, int *size, wxIPCFormat format);
+    bool OnPoke(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format);
+    bool OnStartAdvise(const wxString& topic, const wxString& item);

 
 

-  bool OnExecute(const wxString& topic, char *data, int size, wxIPCFormat format);
-  char *OnRequest(const wxString& topic, const wxString& item, int *size, wxIPCFormat format);
-  bool OnPoke(const wxString& topic, const wxString& item, char *data, int size, wxIPCFormat format);
-  bool OnStartAdvise(const wxString& topic, const wxString& item);
+private:
+    IPCDialogBox *dialog;

 };
 
 class MyServer: public wxServer
 };
 
 class MyServer: public wxServer


@@ -56,13 +62,18 @@ public:
 class IPCDialogBox: public wxDialog
 {
 public:
 class IPCDialogBox: public wxDialog
 {
 public:

-    MyConnection *connection;
-    IPCDialogBox(wxFrame *parent, const wxString& title,
-                         const wxPoint& pos, const wxSize& size, MyConnection *the_connection);
+    IPCDialogBox(wxWindow *parent,
+                 const wxString& title,
+                 const wxPoint& pos,
+                 const wxSize& size,
+                 MyConnection *the_connection);

 
     void OnQuit(wxCommandEvent& event);
 
 
     void OnQuit(wxCommandEvent& event);
 

-DECLARE_EVENT_TABLE()
+private:
+    MyConnection *m_connection;
+
+    DECLARE_EVENT_TABLE()

 };
 
 #define SERVER_QUIT         wxID_EXIT
 };
 
 #define SERVER_QUIT         wxID_EXIT