added cw6 fixes and patrick's patches

[wxWidgets.git] / src / html / helpctrl.cpp

diff --git a/src/html/helpctrl.cpp b/src/html/helpctrl.cpp
index 6d6583db509a9465598925bd0c12334a9f3937ff..26911db463e8e8a0b507e50dd5b2dfc883a99543 100644 (file)
--- a/src/html/helpctrl.cpp
+++ b/src/html/helpctrl.cpp
@@ -22,7 +22,7 @@
 
 #include "wx/defs.h"
 
 
 #include "wx/defs.h"
 

-#if wxUSE_HTML
+#if wxUSE_HTML && wxUSE_STREAMS

 
 #include "wx/html/helpctrl.h"
 #include "wx/wx.h"
 
 #include "wx/html/helpctrl.h"
 #include "wx/wx.h"


@@ -30,12 +30,6 @@
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlHelpController, wxHelpControllerBase)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlHelpController, wxHelpControllerBase)
 

-#if 0
-BEGIN_EVENT_TABLE(wxHtmlHelpController, wxEvtHandler)
-EVT_CLOSE(wxHtmlHelpController::OnCloseFrame)
-END_EVENT_TABLE()
-#endif
-

 wxHtmlHelpController::wxHtmlHelpController(int style)
 {
     m_helpFrame = NULL;
 wxHtmlHelpController::wxHtmlHelpController(int style)
 {
     m_helpFrame = NULL;


@@ -47,7 +41,8 @@ wxHtmlHelpController::wxHtmlHelpController(int style)
 
 wxHtmlHelpController::~wxHtmlHelpController()
 {
 
 wxHtmlHelpController::~wxHtmlHelpController()
 {

-    WriteCustomization(m_Config, m_ConfigRoot);
+    if (m_Config)
+        WriteCustomization(m_Config, m_ConfigRoot);

     if (m_helpFrame)
         DestroyHelpWindow();
 }
     if (m_helpFrame)
         DestroyHelpWindow();
 }


@@ -84,7 +79,8 @@ bool wxHtmlHelpController::AddBook(const wxString& book, bool show_wait_msg)
 #if wxUSE_BUSYINFO
     wxBusyInfo* busy = NULL;
     wxString info;
 #if wxUSE_BUSYINFO
     wxBusyInfo* busy = NULL;
     wxString info;

-    if (show_wait_msg) {
+    if (show_wait_msg) 
+    {

         info.Printf(_("Adding book %s"), book.c_str());
         busy = new wxBusyInfo(info);
     }
         info.Printf(_("Adding book %s"), book.c_str());
         busy = new wxBusyInfo(info);
     }


@@ -107,7 +103,8 @@ wxHtmlHelpFrame *wxHtmlHelpController::CreateHelpFrame(wxHtmlHelpData *data)
 
 void wxHtmlHelpController::CreateHelpWindow()
 {
 
 void wxHtmlHelpController::CreateHelpWindow()
 {

-    if (m_helpFrame) {
+    if (m_helpFrame)
+    {

         m_helpFrame->Raise();
         return ;
     }
         m_helpFrame->Raise();
         return ;
     }


@@ -121,7 +118,6 @@ void wxHtmlHelpController::CreateHelpWindow()
 
     m_helpFrame = CreateHelpFrame(&m_helpData);
     m_helpFrame->SetController(this);
 
     m_helpFrame = CreateHelpFrame(&m_helpData);
     m_helpFrame->SetController(this);

-//    m_helpFrame->PushEventHandler(this);

 
     if (m_Config)
         m_helpFrame->UseConfig(m_Config, m_ConfigRoot);
 
     if (m_Config)
         m_helpFrame->UseConfig(m_Config, m_ConfigRoot);


@@ -135,14 +131,14 @@ void wxHtmlHelpController::ReadCustomization(wxConfigBase* cfg, const wxString&
 {
     /* should not be called by the user; call UseConfig, and the controller
      * will do the rest */
 {
     /* should not be called by the user; call UseConfig, and the controller
      * will do the rest */

-    if (m_helpFrame)
+    if (m_helpFrame && cfg)

         m_helpFrame->ReadCustomization(cfg, path);
 }
 
 void wxHtmlHelpController::WriteCustomization(wxConfigBase* cfg, const wxString& path)
 {
     /* typically called by the controllers OnCloseFrame handler */
         m_helpFrame->ReadCustomization(cfg, path);
 }
 
 void wxHtmlHelpController::WriteCustomization(wxConfigBase* cfg, const wxString& path)
 {
     /* typically called by the controllers OnCloseFrame handler */

-    if (m_helpFrame)
+    if (m_helpFrame && cfg)

         m_helpFrame->WriteCustomization(cfg, path);
 }
 
         m_helpFrame->WriteCustomization(cfg, path);
 }
 


@@ -150,7 +146,7 @@ void wxHtmlHelpController::UseConfig(wxConfigBase *config, const wxString& rootp
 {
     m_Config = config;
     m_ConfigRoot = rootpath;
 {
     m_Config = config;
     m_ConfigRoot = rootpath;

-    if (m_helpFrame) m_helpFrame -> UseConfig(config, rootpath);
+    if (m_helpFrame) m_helpFrame->UseConfig(config, rootpath);

     ReadCustomization(config, rootpath);
 }
 
     ReadCustomization(config, rootpath);
 }
 


@@ -162,16 +158,16 @@ bool wxHtmlHelpController::Initialize(const wxString& file)
     wxSplitPath(file, & dir, & filename, & ext);
 
     if (!dir.IsEmpty())
     wxSplitPath(file, & dir, & filename, & ext);
 
     if (!dir.IsEmpty())

-        dir = dir + wxString(_("/"));
+        dir = dir + wxString(wxT("/"));

 
     // Try to find a suitable file
 
     // Try to find a suitable file

-    wxString actualFilename = dir + filename + wxString(_(".zip"));
+    wxString actualFilename = dir + filename + wxString(wxT(".zip"));

     if (!wxFileExists(actualFilename))
     {
     if (!wxFileExists(actualFilename))
     {

-        actualFilename = dir + filename + wxString(_(".htb"));
+        actualFilename = dir + filename + wxString(wxT(".htb"));

         if (!wxFileExists(actualFilename))
         {
         if (!wxFileExists(actualFilename))
         {

-            actualFilename = dir + filename + wxString(_(".hhp"));
+            actualFilename = dir + filename + wxString(wxT(".hhp"));

             if (!wxFileExists(actualFilename))
                 return FALSE;
         }
             if (!wxFileExists(actualFilename))
                 return FALSE;
         }