Now allow SetMenuBar to be called several times.

[wxWidgets.git] / src / msw / dde.cpp

diff --git a/src/msw/dde.cpp b/src/msw/dde.cpp
index f2af16d92471c5344034ee4f6797f3570cdf8bfa..ad4f8174a2d4a21d127ad4680aa4b1c9b0aef94b 100644 (file)
--- a/src/msw/dde.cpp
+++ b/src/msw/dde.cpp
@@ -34,15 +34,18 @@
 #include "wx/module.h"
 #include "wx/dde.h"
 
 #include "wx/module.h"
 #include "wx/dde.h"
 

+#include "wx/msw/private.h"
+#include <windows.h>
+#include <ddeml.h>
+

 #ifndef __TWIN32__
 #ifndef __TWIN32__

+#if !wxUSE_NORLANDER_HEADERS

 #ifdef __GNUWIN32__
 #include "wx/msw/gnuwin32/extra.h"
 #endif
 #endif
 #ifdef __GNUWIN32__
 #include "wx/msw/gnuwin32/extra.h"
 #endif
 #endif

+#endif

 
 

-#include "wx/msw/private.h"
-#include <windows.h>
-#include <ddeml.h>

 #include <string.h>
 
 #ifdef __WIN32__
 #include <string.h>
 
 #ifdef __WIN32__


@@ -213,7 +216,7 @@ wxDDEServer::wxDDEServer(void)
 bool wxDDEServer::Create(const wxString& server_name)
 {
   m_serviceName = server_name;
 bool wxDDEServer::Create(const wxString& server_name)
 {
   m_serviceName = server_name;

-  HSZ serviceName = DdeCreateStringHandle(DDEIdInst, (char*) (const char *)server_name, CP_WINANSI);
+  HSZ serviceName = DdeCreateStringHandle(DDEIdInst, WXSTRINGCAST server_name, CP_WINANSI);

 
   if (DdeNameService(DDEIdInst, serviceName, (HSZ) NULL, DNS_REGISTER) == 0)
   {
 
   if (DdeNameService(DDEIdInst, serviceName, (HSZ) NULL, DNS_REGISTER) == 0)
   {


@@ -225,9 +228,9 @@ bool wxDDEServer::Create(const wxString& server_name)
 
 wxDDEServer::~wxDDEServer(void)
 {
 
 wxDDEServer::~wxDDEServer(void)
 {

-  if (m_serviceName != "")
+  if (m_serviceName != wxT(""))

   {
   {

-    HSZ serviceName = DdeCreateStringHandle(DDEIdInst, (char*) (const char *)m_serviceName, CP_WINANSI);
+    HSZ serviceName = DdeCreateStringHandle(DDEIdInst, WXSTRINGCAST m_serviceName, CP_WINANSI);

     if (DdeNameService(DDEIdInst, serviceName, NULL, DNS_UNREGISTER) == 0)
     {
       DDEPrintError();
     if (DdeNameService(DDEIdInst, serviceName, NULL, DNS_UNREGISTER) == 0)
     {
       DDEPrintError();


@@ -323,8 +326,8 @@ bool wxDDEClient::ValidHost(const wxString& /* host */)
 
 wxConnectionBase *wxDDEClient::MakeConnection(const wxString& /* host */, const wxString& server_name, const wxString& topic)
 {
 
 wxConnectionBase *wxDDEClient::MakeConnection(const wxString& /* host */, const wxString& server_name, const wxString& topic)
 {

-  HSZ serviceName = DdeCreateStringHandle(DDEIdInst, (char*) (const char *)server_name, CP_WINANSI);
-  HSZ topic_atom = DdeCreateStringHandle(DDEIdInst, (char*) (const char *)topic, CP_WINANSI);
+  HSZ serviceName = DdeCreateStringHandle(DDEIdInst, WXSTRINGCAST server_name, CP_WINANSI);
+  HSZ topic_atom = DdeCreateStringHandle(DDEIdInst, WXSTRINGCAST topic, CP_WINANSI);

 
   HCONV hConv = DdeConnect(DDEIdInst, serviceName, topic_atom, (PCONVCONTEXT)NULL);
   if (hConv == (HCONV) NULL)
 
   HCONV hConv = DdeConnect(DDEIdInst, serviceName, topic_atom, (PCONVCONTEXT)NULL);
   if (hConv == (HCONV) NULL)


@@ -438,11 +441,11 @@ bool wxDDEConnection::Disconnect(void)
   return (DdeDisconnect((HCONV) m_hConv) != 0);
 }
 
   return (DdeDisconnect((HCONV) m_hConv) != 0);
 }
 

-bool wxDDEConnection::Execute(char *data, int size, wxIPCFormat format)
+bool wxDDEConnection::Execute(const wxChar *data, int size, wxIPCFormat format)

 {
   DWORD result;
   if (size < 0)
 {
   DWORD result;
   if (size < 0)

-    size = strlen(data);
+    size = wxStrlen(data);

 
   size ++;
 
 
   size ++;
 


@@ -470,11 +473,11 @@ char *wxDDEConnection::Request(const wxString& item, int *size, wxIPCFormat form
   else return NULL;
 }
 
   else return NULL;
 }
 

-bool wxDDEConnection::Poke(const wxString& item, char *data, int size, wxIPCFormat format)
+bool wxDDEConnection::Poke(const wxString& item, wxChar *data, int size, wxIPCFormat format)

 {
   DWORD result;
   if (size < 0)
 {
   DWORD result;
   if (size < 0)

-    size = strlen(data);
+    size = wxStrlen(data);

 
   size ++;
 
 
   size ++;
 


@@ -502,10 +505,10 @@ bool wxDDEConnection::StopAdvise(const wxString& item)
 }
 
 // Calls that SERVER can make
 }
 
 // Calls that SERVER can make

-bool wxDDEConnection::Advise(const wxString& item, char *data, int size, wxIPCFormat format)
+bool wxDDEConnection::Advise(const wxString& item, wxChar *data, int size, wxIPCFormat format)

 {
   if (size < 0)
 {
   if (size < 0)

-    size = strlen(data);
+    size = wxStrlen(data);

 
   size ++;
 
 
   size ++;
 


@@ -540,11 +543,11 @@ DWORD /* lData2 */)
   {
     case XTYP_CONNECT:
     {
   {
     case XTYP_CONNECT:
     {

-      char topic_buf[100];
-      char server_buf[100];
-      DdeQueryString(DDEIdInst, hsz1, (LPSTR)topic_buf, sizeof(topic_buf),
+      wxChar topic_buf[100];
+      wxChar server_buf[100];
+      DdeQueryString(DDEIdInst, hsz1, (LPTSTR)topic_buf, WXSIZEOF(topic_buf),

                      CP_WINANSI);
                      CP_WINANSI);

-      DdeQueryString(DDEIdInst, hsz2, (LPSTR)server_buf, sizeof(topic_buf),
+      DdeQueryString(DDEIdInst, hsz2, (LPTSTR)server_buf, WXSIZEOF(topic_buf),

                      CP_WINANSI);
       wxDDEServer *server = DDEFindServer(server_buf);
       if (server)
                      CP_WINANSI);
       wxDDEServer *server = DDEFindServer(server_buf);
       if (server)


@@ -558,7 +561,7 @@ DWORD /* lData2 */)
           connection->m_hConv = 0;
           connection->m_topicName = topic_buf;
           DDECurrentlyConnecting = connection;
           connection->m_hConv = 0;
           connection->m_topicName = topic_buf;
           DDECurrentlyConnecting = connection;

-          return (DDERETURN)TRUE;
+          return (DDERETURN)(DWORD)TRUE;

         }
       }
       else return (DDERETURN)0;
         }
       }
       else return (DDERETURN)0;


@@ -571,7 +574,7 @@ DWORD /* lData2 */)
       {
         DDECurrentlyConnecting->m_hConv = (WXHCONV) hConv;
         DDECurrentlyConnecting = NULL;
       {
         DDECurrentlyConnecting->m_hConv = (WXHCONV) hConv;
         DDECurrentlyConnecting = NULL;

-        return (DDERETURN)TRUE;
+        return (DDERETURN)(DWORD)TRUE;

       }
       else return 0;
       break;
       }
       else return 0;
       break;


@@ -583,7 +586,7 @@ DWORD /* lData2 */)
       if (connection && connection->OnDisconnect())
       {
         DDEDeleteConnection(hConv);  // Delete mapping: hConv => connection
       if (connection && connection->OnDisconnect())
       {
         DDEDeleteConnection(hConv);  // Delete mapping: hConv => connection

-        return (DDERETURN)TRUE;
+        return (DDERETURN)(DWORD)TRUE;

       }
       else return (DDERETURN)0;
       break;
       }
       else return (DDERETURN)0;
       break;


@@ -598,7 +601,7 @@ DWORD /* lData2 */)
         DWORD len = DdeGetData(hData, (LPBYTE)(connection->m_bufPtr), connection->m_bufSize, 0);
         DdeFreeDataHandle(hData);
         if (connection->OnExecute(connection->m_topicName, connection->m_bufPtr, (int)len, (wxIPCFormat) wFmt))
         DWORD len = DdeGetData(hData, (LPBYTE)(connection->m_bufPtr), connection->m_bufSize, 0);
         DdeFreeDataHandle(hData);
         if (connection->OnExecute(connection->m_topicName, connection->m_bufPtr, (int)len, (wxIPCFormat) wFmt))

-          return (DDERETURN)DDE_FACK;
+          return (DDERETURN)(DWORD)DDE_FACK;

         else
           return (DDERETURN)DDE_FNOTPROCESSED;
       } else return (DDERETURN)DDE_FNOTPROCESSED;
         else
           return (DDERETURN)DDE_FNOTPROCESSED;
       } else return (DDERETURN)DDE_FNOTPROCESSED;


@@ -611,8 +614,8 @@ DWORD /* lData2 */)
 
       if (connection)
       {
 
       if (connection)
       {

-        char item_name[200];
-        DdeQueryString(DDEIdInst, hsz2, (LPSTR)item_name, sizeof(item_name),
+        wxChar item_name[200];
+        DdeQueryString(DDEIdInst, hsz2, (LPTSTR)item_name, WXSIZEOF(item_name),

                      CP_WINANSI);
 
         int user_size = -1;
                      CP_WINANSI);
 
         int user_size = -1;


@@ -635,13 +638,13 @@ DWORD /* lData2 */)
 
       if (connection)
       {
 
       if (connection)
       {

-        char item_name[200];
-        DdeQueryString(DDEIdInst, hsz2, (LPSTR)item_name, sizeof(item_name),
+        wxChar item_name[200];
+        DdeQueryString(DDEIdInst, hsz2, (LPTSTR)item_name, WXSIZEOF(item_name),

                      CP_WINANSI);
         DWORD len = DdeGetData(hData, (LPBYTE)(connection->m_bufPtr), connection->m_bufSize, 0);
         DdeFreeDataHandle(hData);
         connection->OnPoke(connection->m_topicName, wxString(item_name), connection->m_bufPtr, (int)len, (wxIPCFormat) wFmt);
                      CP_WINANSI);
         DWORD len = DdeGetData(hData, (LPBYTE)(connection->m_bufPtr), connection->m_bufSize, 0);
         DdeFreeDataHandle(hData);
         connection->OnPoke(connection->m_topicName, wxString(item_name), connection->m_bufPtr, (int)len, (wxIPCFormat) wFmt);

-        return (DDERETURN)DDE_FACK;
+        return (DDERETURN)(DWORD)DDE_FACK;

       } else return (DDERETURN)DDE_FNOTPROCESSED;
       break;
     }
       } else return (DDERETURN)DDE_FNOTPROCESSED;
       break;
     }


@@ -652,11 +655,11 @@ DWORD /* lData2 */)
 
       if (connection)
       {
 
       if (connection)
       {

-        char item_name[200];
-        DdeQueryString(DDEIdInst, hsz2, (LPSTR)item_name, sizeof(item_name),
+        wxChar item_name[200];
+        DdeQueryString(DDEIdInst, hsz2, (LPTSTR)item_name, WXSIZEOF(item_name),

                      CP_WINANSI);
 
                      CP_WINANSI);
 

-        return (DDERETURN)connection->OnStartAdvise(connection->m_topicName, wxString(item_name));
+        return (DDERETURN)(DWORD)connection->OnStartAdvise(connection->m_topicName, wxString(item_name));

       } else return (DDERETURN)0;
       break;
     }
       } else return (DDERETURN)0;
       break;
     }


@@ -667,10 +670,10 @@ DWORD /* lData2 */)
 
       if (connection)
       {
 
       if (connection)
       {

-        char item_name[200];
-        DdeQueryString(DDEIdInst, hsz2, (LPSTR)item_name, sizeof(item_name),
+        wxChar item_name[200];
+        DdeQueryString(DDEIdInst, hsz2, (LPTSTR)item_name, WXSIZEOF(item_name),

                      CP_WINANSI);
                      CP_WINANSI);

-        return (DDERETURN)connection->OnStopAdvise(connection->m_topicName, wxString(item_name));
+        return (DDERETURN)(DWORD)connection->OnStopAdvise(connection->m_topicName, wxString(item_name));

       } else return (DDERETURN)0;
       break;
     }
       } else return (DDERETURN)0;
       break;
     }


@@ -696,14 +699,14 @@ DWORD /* lData2 */)
 
       if (connection)
       {
 
       if (connection)
       {

-        char item_name[200];
-        DdeQueryString(DDEIdInst, hsz2, (LPSTR)item_name, sizeof(item_name),
+        wxChar item_name[200];
+        DdeQueryString(DDEIdInst, hsz2, (LPTSTR)item_name, WXSIZEOF(item_name),

                      CP_WINANSI);
 
         DWORD len = DdeGetData(hData, (LPBYTE)(connection->m_bufPtr), connection->m_bufSize, 0);
         DdeFreeDataHandle(hData);
         if (connection->OnAdvise(connection->m_topicName, wxString(item_name), connection->m_bufPtr, (int)len, (wxIPCFormat) wFmt))
                      CP_WINANSI);
 
         DWORD len = DdeGetData(hData, (LPBYTE)(connection->m_bufPtr), connection->m_bufSize, 0);
         DdeFreeDataHandle(hData);
         if (connection->OnAdvise(connection->m_topicName, wxString(item_name), connection->m_bufPtr, (int)len, (wxIPCFormat) wFmt))

-          return (DDERETURN)DDE_FACK;
+          return (DDERETURN)(DWORD)DDE_FACK;

         else
           return (DDERETURN)DDE_FNOTPROCESSED;
       } else return (DDERETURN)DDE_FNOTPROCESSED;
         else
           return (DDERETURN)DDE_FNOTPROCESSED;
       } else return (DDERETURN)DDE_FNOTPROCESSED;


@@ -716,7 +719,7 @@ DWORD /* lData2 */)
 // Atom table stuff
 static HSZ DDEAddAtom(const wxString& string)
 {
 // Atom table stuff
 static HSZ DDEAddAtom(const wxString& string)
 {

-  HSZ atom = DdeCreateStringHandle(DDEIdInst, (char*) (const char *)string, CP_WINANSI);
+  HSZ atom = DdeCreateStringHandle(DDEIdInst, WXSTRINGCAST string, CP_WINANSI);

   wxAtomTable.Append(string, (wxObject *)atom);
   return atom;
 }
   wxAtomTable.Append(string, (wxObject *)atom);
   return atom;
 }


@@ -735,68 +738,68 @@ static HSZ DDEGetAtom(const wxString& string)
 
 void DDEPrintError(void)
 {
 
 void DDEPrintError(void)
 {

-  char *err = NULL;
+  wxChar *err = NULL;

   switch (DdeGetLastError(DDEIdInst))
   {
     case DMLERR_ADVACKTIMEOUT:
   switch (DdeGetLastError(DDEIdInst))
   {
     case DMLERR_ADVACKTIMEOUT:

-      err = "A request for a synchronous advise transaction has timed out.";
+      err = wxT("A request for a synchronous advise transaction has timed out.");

       break;
     case DMLERR_BUSY:
       break;
     case DMLERR_BUSY:

-      err = "The response to the transaction caused the DDE_FBUSY bit to be set.";
+      err = wxT("The response to the transaction caused the DDE_FBUSY bit to be set.");

       break;
     case DMLERR_DATAACKTIMEOUT:
       break;
     case DMLERR_DATAACKTIMEOUT:

-      err = "A request for a synchronous data transaction has timed out.";
+      err = wxT("A request for a synchronous data transaction has timed out.");

       break;
     case DMLERR_DLL_NOT_INITIALIZED:
       break;
     case DMLERR_DLL_NOT_INITIALIZED:

-      err = "A DDEML function was called without first calling the DdeInitialize function,\n\ror an invalid instance identifier\n\rwas passed to a DDEML function.";
+      err = wxT("A DDEML function was called without first calling the DdeInitialize function,\n\ror an invalid instance identifier\n\rwas passed to a DDEML function.");

       break;
     case DMLERR_DLL_USAGE:
       break;
     case DMLERR_DLL_USAGE:

-      err = "An application initialized as APPCLASS_MONITOR has\n\rattempted to perform a DDE transaction,\n\ror an application initialized as APPCMD_CLIENTONLY has \n\rattempted to perform server transactions.";
+      err = wxT("An application initialized as APPCLASS_MONITOR has\n\rattempted to perform a DDE transaction,\n\ror an application initialized as APPCMD_CLIENTONLY has \n\rattempted to perform server transactions.");

       break;
     case DMLERR_EXECACKTIMEOUT:
       break;
     case DMLERR_EXECACKTIMEOUT:

-      err = "A request for a synchronous execute transaction has timed out.";
+      err = wxT("A request for a synchronous execute transaction has timed out.");

       break;
     case DMLERR_INVALIDPARAMETER:
       break;
     case DMLERR_INVALIDPARAMETER:

-      err = "A parameter failed to be validated by the DDEML.";
+      err = wxT("A parameter failed to be validated by the DDEML.");

       break;
     case DMLERR_LOW_MEMORY:
       break;
     case DMLERR_LOW_MEMORY:

-      err = "A DDEML application has created a prolonged race condition.";
+      err = wxT("A DDEML application has created a prolonged race condition.");

       break;
     case DMLERR_MEMORY_ERROR:
       break;
     case DMLERR_MEMORY_ERROR:

-      err = "A memory allocation failed.";
+      err = wxT("A memory allocation failed.");

       break;
     case DMLERR_NO_CONV_ESTABLISHED:
       break;
     case DMLERR_NO_CONV_ESTABLISHED:

-      err = "A client's attempt to establish a conversation has failed.";
+      err = wxT("A client's attempt to establish a conversation has failed.");

       break;
     case DMLERR_NOTPROCESSED:
       break;
     case DMLERR_NOTPROCESSED:

-      err = "A transaction failed.";
+      err = wxT("A transaction failed.");

       break;
     case DMLERR_POKEACKTIMEOUT:
       break;
     case DMLERR_POKEACKTIMEOUT:

-      err = "A request for a synchronous poke transaction has timed out.";
+      err = wxT("A request for a synchronous poke transaction has timed out.");

       break;
     case DMLERR_POSTMSG_FAILED:
       break;
     case DMLERR_POSTMSG_FAILED:

-      err = "An internal call to the PostMessage function has failed. ";
+      err = wxT("An internal call to the PostMessage function has failed. ");

       break;
     case DMLERR_REENTRANCY:
       break;
     case DMLERR_REENTRANCY:

-      err = "Reentrancy problem.";
+      err = wxT("Reentrancy problem.");

       break;
     case DMLERR_SERVER_DIED:
       break;
     case DMLERR_SERVER_DIED:

-      err = "A server-side transaction was attempted on a conversation\n\rthat was terminated by the client, or the server\n\rterminated before completing a transaction.";
+      err = wxT("A server-side transaction was attempted on a conversation\n\rthat was terminated by the client, or the server\n\rterminated before completing a transaction.");

       break;
     case DMLERR_SYS_ERROR:
       break;
     case DMLERR_SYS_ERROR:

-      err = "An internal error has occurred in the DDEML.";
+      err = wxT("An internal error has occurred in the DDEML.");

       break;
     case DMLERR_UNADVACKTIMEOUT:
       break;
     case DMLERR_UNADVACKTIMEOUT:

-      err = "A request to end an advise transaction has timed out.";
+      err = wxT("A request to end an advise transaction has timed out.");

       break;
     case DMLERR_UNFOUND_QUEUE_ID:
       break;
     case DMLERR_UNFOUND_QUEUE_ID:

-      err = "An invalid transaction identifier was passed to a DDEML function.\n\rOnce the application has returned from an XTYP_XACT_COMPLETE callback,\n\rthe transaction identifier for that callback is no longer valid.";
+      err = wxT("An invalid transaction identifier was passed to a DDEML function.\n\rOnce the application has returned from an XTYP_XACT_COMPLETE callback,\n\rthe transaction identifier for that callback is no longer valid.");

       break;
     default:
       break;
     default:

-      err = "Unrecognised error type.";
+      err = wxT("Unrecognised error type.");

       break;
   }
       break;
   }

-  MessageBox((HWND) NULL, (LPCSTR)err, "DDE Error", MB_OK | MB_ICONINFORMATION);
+  MessageBox((HWND) NULL, (LPCTSTR)err, wxT("DDE Error"), MB_OK | MB_ICONINFORMATION);

 }
 
 #endif
 }
 
 #endif