Split this out from other changes to keep things sane..

[wxWidgets.git] / src / generic / helpext.cpp

diff --git a/src/generic/helpext.cpp b/src/generic/helpext.cpp
index 56a293ca4259b0c878c82ceb17ce6327e6e5681d..f6e3add7b5afc49dd8035357537d100ac733858e 100644 (file)
--- a/src/generic/helpext.cpp
+++ b/src/generic/helpext.cpp
@@ -36,12 +36,21 @@
 #include <ctype.h>
 #include <sys/stat.h>
 
 #include <ctype.h>
 #include <sys/stat.h>
 

-#ifndef __WINDOWS__
+#if !defined(__WINDOWS__) && !defined(__OS2__)

     #include   <unistd.h>
 #endif
 
     #include   <unistd.h>
 #endif
 

+#ifdef __WXMSW__
+#include <windows.h>
+#endif
+

 IMPLEMENT_CLASS(wxExtHelpController, wxHTMLHelpControllerBase)
 
 IMPLEMENT_CLASS(wxExtHelpController, wxHTMLHelpControllerBase)
 

+/// Name of environment variable to set help browser.
+#define   WXEXTHELP_ENVVAR_BROWSER   wxT("WX_HELPBROWSER")
+/// Is browser a netscape browser?
+#define   WXEXTHELP_ENVVAR_BROWSERISNETSCAPE wxT("WX_HELPBROWSER_NS")
+

 /**
    This class implements help via an external browser.
    It requires the name of a directory containing the documentation
 /**
    This class implements help via an external browser.
    It requires the name of a directory containing the documentation


@@ -53,18 +62,17 @@ wxExtHelpController::wxExtHelpController(void)
    m_BrowserName = WXEXTHELP_DEFAULTBROWSER;
    m_BrowserIsNetscape = WXEXTHELP_DEFAULTBROWSER_IS_NETSCAPE;
 
    m_BrowserName = WXEXTHELP_DEFAULTBROWSER;
    m_BrowserIsNetscape = WXEXTHELP_DEFAULTBROWSER_IS_NETSCAPE;
 

-   char *browser = getenv(WXEXTHELP_ENVVAR_BROWSER);
+   wxChar *browser = wxGetenv(WXEXTHELP_ENVVAR_BROWSER);

    if(browser)
    {
       m_BrowserName = browser;
    if(browser)
    {
       m_BrowserName = browser;

-      browser = getenv(WXEXTHELP_ENVVAR_BROWSERISNETSCAPE);
-      m_BrowserIsNetscape = browser && (atoi(browser) != 0);
+      browser = wxGetenv(WXEXTHELP_ENVVAR_BROWSERISNETSCAPE);
+      m_BrowserIsNetscape = browser && (wxAtoi(browser) != 0);

    }
 }
 
 
    }
 }
 
 

-void
-wxExtHelpController::SetBrowser(wxString const & browsername, bool isNetscape)
+void wxExtHelpController::SetBrowser(wxString const & browsername, bool isNetscape)

 {
    m_BrowserName = browsername;
    m_BrowserIsNetscape = isNetscape;
 {
    m_BrowserName = browsername;
    m_BrowserIsNetscape = isNetscape;


@@ -77,15 +85,15 @@ void wxExtHelpController::SetViewer(const wxString& viewer, long flags)
 }
 
 bool
 }
 
 bool

-wxExtHelpController::DisplayHelp(wxString const &relativeURL)
+wxExtHelpController::DisplayHelp(const wxString &relativeURL)

 {
    wxBusyCursor b; // display a busy cursor
 
 
 {
    wxBusyCursor b; // display a busy cursor
 
 

-#ifdef __WXMSW__
+#if defined(__WXMSW__)

    wxString url;
    url << m_MapFile << '\\' << relativeURL.BeforeFirst('#');
    wxString url;
    url << m_MapFile << '\\' << relativeURL.BeforeFirst('#');

-   bool bOk = (int)ShellExecute(NULL, "open", url,
+   bool bOk = (int)ShellExecute(NULL, wxT("open"), url.c_str(),

                                 NULL, NULL, SW_SHOWNORMAL ) > 32;
    if ( !bOk )
    {
                                 NULL, NULL, SW_SHOWNORMAL ) > 32;
    if ( !bOk )
    {


@@ -94,31 +102,62 @@ wxExtHelpController::DisplayHelp(wxString const &relativeURL)
    }
    else
       return true;
    }
    else
       return true;

-#else
-   // assume UNIX
+
+#elif  defined(__WXPM__)
+
+   wxString url;
+   url << m_MapFile << '\\' << relativeURL.BeforeFirst('#');
+//   will have to fix for OS/2, later.....DW
+//   bool bOk = (int)ShellExecute(NULL, "open", url,
+//                                NULL, NULL, SW_SHOWNORMAL ) > 32;
+//   if ( !bOk )
+//   {
+//      wxLogSysError(_("Cannot open URL '%s'"), relativeURL.c_str());
+//      return false;
+//   }
+//   else
+      return TRUE;
+
+#elif defined(__DOS__)
+
+   wxString command;
+   command = m_BrowserName;
+   command << wxT(" file://")
+           << m_MapFile << WXEXTHELP_SEPARATOR << relativeURL;
+   return wxExecute(command) != 0;
+
+#else // UNIX

    wxString command;
 
    wxString command;
 

+#ifndef __EMX__

    if(m_BrowserIsNetscape) // try re-loading first
    {
       wxString lockfile;
       wxGetHomeDir(&lockfile);
    if(m_BrowserIsNetscape) // try re-loading first
    {
       wxString lockfile;
       wxGetHomeDir(&lockfile);

-      lockfile << WXEXTHELP_SEPARATOR << T(".netscape/lock");
+#ifdef __VMS__
+      lockfile << WXEXTHELP_SEPARATOR << wxT(".netscape]lock.");
+      struct stat statbuf;
+      if(stat(lockfile.fn_str(), &statbuf) == 0)
+#else
+      lockfile << WXEXTHELP_SEPARATOR << wxT(".netscape/lock");

       struct stat statbuf;
       if(lstat(lockfile.fn_str(), &statbuf) == 0)
       // cannot use wxFileExists, because it's a link pointing to a
       // non-existing location      if(wxFileExists(lockfile))
       struct stat statbuf;
       if(lstat(lockfile.fn_str(), &statbuf) == 0)
       // cannot use wxFileExists, because it's a link pointing to a
       // non-existing location      if(wxFileExists(lockfile))

+#endif

       {
          long success;
       {
          long success;

-         command << m_BrowserName << T(" -remote openURL(")
-                 << T("file://") << m_MapFile
-                 << WXEXTHELP_SEPARATOR << relativeURL << T(")");
+         command << m_BrowserName << wxT(" -remote openURL(")
+                 << wxT("file://") << m_MapFile
+                 << WXEXTHELP_SEPARATOR << relativeURL << wxT(")");

          success = wxExecute(command);
          if(success != 0 ) // returns PID on success
             return TRUE;
       }
    }
          success = wxExecute(command);
          if(success != 0 ) // returns PID on success
             return TRUE;
       }
    }

+#endif

    command = m_BrowserName;
    command = m_BrowserName;

-   command << T(" file://")
+   command << wxT(" file://")

            << m_MapFile << WXEXTHELP_SEPARATOR << relativeURL;
    return wxExecute(command) != 0;
 #endif
            << m_MapFile << WXEXTHELP_SEPARATOR << relativeURL;
    return wxExecute(command) != 0;
 #endif