Add class factories for filter streams. Also filters now follow the convention

[wxWidgets.git] / src / common / docview.cpp
diff --git a/src/common/docview.cpp b/src/common/docview.cpp

index 988fc0b7d5b951339d005f431acd7b5cd2329778..0e26b04b6b6428b15126f750cff26c30aeaedb04 100644 (file)
--- a/src/common/docview.cpp
+++ b/src/common/docview.cpp
@@ -17,30 +17,31 @@
  // headers
  // ----------------------------------------------------------------------------
  
  // headers
  // ----------------------------------------------------------------------------
  
-#ifdef __GNUG__
-    #pragma implementation "docview.h"
-#endif
-
  // 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
  
  #if wxUSE_DOC_VIEW_ARCHITECTURE
  
  #endif
  
  #if wxUSE_DOC_VIEW_ARCHITECTURE
  
+#include "wx/docview.h"
+
  #ifndef WX_PRECOMP
  #ifndef WX_PRECOMP
+    #include "wx/list.h"
      #include "wx/string.h"
      #include "wx/utils.h"
      #include "wx/app.h"
      #include "wx/dc.h"
      #include "wx/dialog.h"
      #include "wx/menu.h"
      #include "wx/string.h"
      #include "wx/utils.h"
      #include "wx/app.h"
      #include "wx/dc.h"
      #include "wx/dialog.h"
      #include "wx/menu.h"
-    #include "wx/list.h"
      #include "wx/filedlg.h"
      #include "wx/intl.h"
      #include "wx/log.h"
      #include "wx/filedlg.h"
      #include "wx/intl.h"
      #include "wx/log.h"
+    #include "wx/msgdlg.h"
+    #include "wx/mdi.h"
+    #include "wx/choicdlg.h"
  #endif
  
  #include "wx/ffile.h"
  #endif
  
  #include "wx/ffile.h"
@@ -49,38 +50,32 @@
      #include "wx/filename.h"
  #endif
  
      #include "wx/filename.h"
  #endif
  
-#ifdef __WXGTK__
-    #include "wx/mdi.h"
-#endif
-
  #if wxUSE_PRINTING_ARCHITECTURE
  #if wxUSE_PRINTING_ARCHITECTURE
-  #include "wx/prntbase.h"
-  #include "wx/printdlg.h"
+    #include "wx/prntbase.h"
+    #include "wx/printdlg.h"
  #endif
  
  #endif
  
-#include "wx/msgdlg.h"
-#include "wx/choicdlg.h"
-#include "wx/docview.h"
  #include "wx/confbase.h"
  #include "wx/file.h"
  #include "wx/cmdproc.h"
  #include "wx/confbase.h"
  #include "wx/file.h"
  #include "wx/cmdproc.h"
+#include "wx/tokenzr.h"
  
  #include <stdio.h>
  #include <string.h>
  
  #if wxUSE_STD_IOSTREAM
  
  #include <stdio.h>
  #include <string.h>
  
  #if wxUSE_STD_IOSTREAM
-  #include "wx/ioswrap.h"
-  #if wxUSE_IOSTREAMH
-    #include <fstream.h>
-  #else
-    #include <fstream>
-  #endif
+    #include "wx/ioswrap.h"
+    #if wxUSE_IOSTREAMH
+        #include <fstream.h>
+    #else
+        #include <fstream>
+    #endif
  #else
  #else
-  #include "wx/wfstream.h"
+    #include "wx/wfstream.h"
  #endif
  
  // ----------------------------------------------------------------------------
  #endif
  
  // ----------------------------------------------------------------------------
-// wxWindows macros
+// wxWidgets macros
  // ----------------------------------------------------------------------------
  
  IMPLEMENT_ABSTRACT_CLASS(wxDocument, wxEvtHandler)
  // ----------------------------------------------------------------------------
  
  IMPLEMENT_ABSTRACT_CLASS(wxDocument, wxEvtHandler)
@@ -101,6 +96,7 @@ IMPLEMENT_DYNAMIC_CLASS(wxFileHistory, wxObject)
  // ----------------------------------------------------------------------------
  
  static inline wxString FindExtension(const wxChar *path);
  // ----------------------------------------------------------------------------
  
  static inline wxString FindExtension(const wxChar *path);
+static wxWindow* wxFindSuitableParent(void);
  
  // ----------------------------------------------------------------------------
  // local constants
  
  // ----------------------------------------------------------------------------
  // local constants
@@ -132,16 +128,16 @@ static wxString FindExtension(const wxChar *path)
  
  wxDocument::wxDocument(wxDocument *parent)
  {
  
  wxDocument::wxDocument(wxDocument *parent)
  {
-    m_documentModified = FALSE;
+    m_documentModified = false;
      m_documentParent = parent;
      m_documentTemplate = (wxDocTemplate *) NULL;
      m_commandProcessor = (wxCommandProcessor*) NULL;
      m_documentParent = parent;
      m_documentTemplate = (wxDocTemplate *) NULL;
      m_commandProcessor = (wxCommandProcessor*) NULL;
-    m_savedYet = FALSE;
+    m_savedYet = false;
  }
  
  bool wxDocument::DeleteContents()
  {
  }
  
  bool wxDocument::DeleteContents()
  {
-    return TRUE;
+    return true;
  }
  
  wxDocument::~wxDocument()
  }
  
  wxDocument::~wxDocument()
@@ -165,16 +161,16 @@ bool wxDocument::Close()
      if (OnSaveModified())
          return OnCloseDocument();
      else
      if (OnSaveModified())
          return OnCloseDocument();
      else
-        return FALSE;
+        return false;
  }
  
  bool wxDocument::OnCloseDocument()
  {
      // Tell all views that we're about to close
  }
  
  bool wxDocument::OnCloseDocument()
  {
      // Tell all views that we're about to close
-    NotifyClosing();    
+    NotifyClosing();
      DeleteContents();
      DeleteContents();
-    Modify(FALSE);
-    return TRUE;
+    Modify(false);
+    return true;
  }
  
  // Note that this implicitly deletes the document when the last view is
  }
  
  // Note that this implicitly deletes the document when the last view is
@@ -183,24 +179,44 @@ bool wxDocument::DeleteAllViews()
  {
      wxDocManager* manager = GetDocumentManager();
  
  {
      wxDocManager* manager = GetDocumentManager();
  
-    wxNode *node = m_documentViews.GetFirst();
-    while (node)
+    // first check if all views agree to be closed
+    const wxList::iterator end = m_documentViews.end();
+    for ( wxList::iterator i = m_documentViews.begin(); i != end; ++i )
      {
      {
-        wxView *view = (wxView *)node->GetData();
-        if (!view->Close())
-            return FALSE;
+        wxView *view = (wxView *)*i;
+        if ( !view->Close() )
+            return false;
+    }
  
  
-        wxNode *next = node->GetNext();
+    // all views agreed to close, now do close them
+    if ( m_documentViews.empty() )
+    {
+        // normally the document would be implicitly deleted when the last view
+        // is, but if don't have any views, do it here instead
+        if ( manager && manager->GetDocuments().Member(this) )
+            delete this;
+    }
+    else // have views
+    {
+        // as we delete elements we iterate over, don't use the usual "from
+        // begin to end" loop
+        for ( ;; )
+        {
+            wxView *view = (wxView *)*m_documentViews.begin();
  
  
-        delete view; // Deletes node implicitly
-        node = next;
+            bool isLastOne = m_documentViews.size() == 1;
+
+            // this always deletes the node implicitly and if this is the last
+            // view also deletes this object itself (also implicitly, great),
+            // so we can't test for m_documentViews.empty() after calling this!
+            delete view;
+
+            if ( isLastOne )
+                break;
+        }
      }
      }
-    // If we haven't yet deleted the document (for example
-    // if there were no views) then delete it.
-    if (manager && manager->GetDocuments().Member(this))
-        delete this;
  
  
-    return TRUE;
+    return true;
  }
  
  wxView *wxDocument::GetFirstView() const
  }
  
  wxView *wxDocument::GetFirstView() const
@@ -218,25 +234,25 @@ wxDocManager *wxDocument::GetDocumentManager() const
  bool wxDocument::OnNewDocument()
  {
      if (!OnSaveModified())
  bool wxDocument::OnNewDocument()
  {
      if (!OnSaveModified())
-        return FALSE;
+        return false;
  
  
-    if (OnCloseDocument()==FALSE) return FALSE;
+    if (OnCloseDocument()==false) return false;
      DeleteContents();
      DeleteContents();
-    Modify(FALSE);
-    SetDocumentSaved(FALSE);
+    Modify(false);
+    SetDocumentSaved(false);
  
      wxString name;
      GetDocumentManager()->MakeDefaultName(name);
      SetTitle(name);
  
      wxString name;
      GetDocumentManager()->MakeDefaultName(name);
      SetTitle(name);
-    SetFilename(name, TRUE);
+    SetFilename(name, true);
  
  
-    return TRUE;
+    return true;
  }
  
  bool wxDocument::Save()
  {
      if (!IsModified() && m_savedYet)
  }
  
  bool wxDocument::Save()
  {
      if (!IsModified() && m_savedYet)
-        return TRUE;
+        return true;
  
      if ( m_documentFile.empty() || !m_savedYet )
          return SaveAs();
  
      if ( m_documentFile.empty() || !m_savedYet )
          return SaveAs();
@@ -248,24 +264,55 @@ bool wxDocument::SaveAs()
  {
      wxDocTemplate *docTemplate = GetDocumentTemplate();
      if (!docTemplate)
  {
      wxDocTemplate *docTemplate = GetDocumentTemplate();
      if (!docTemplate)
-        return FALSE;
+        return false;
+
+#if defined(__WXMSW__) || defined(__WXGTK__) || defined(__WXMAC__)
+    wxString filter = docTemplate->GetDescription() + wxT(" (") + docTemplate->GetFileFilter() + wxT(")|") + docTemplate->GetFileFilter();
+
+    // Now see if there are some other template with identical view and document
+    // classes, whose filters may also be used.
+
+    if (docTemplate->GetViewClassInfo() && docTemplate->GetDocClassInfo())
+    {
+        wxList::compatibility_iterator node = wxDocManager::GetDocumentManager()->GetTemplates().GetFirst();
+        while (node)
+        {
+            wxDocTemplate *t = (wxDocTemplate*) node->GetData();
  
  
+            if (t->IsVisible() && t != docTemplate &&
+                t->GetViewClassInfo() == docTemplate->GetViewClassInfo() &&
+                t->GetDocClassInfo() == docTemplate->GetDocClassInfo())
+            {
+                // add a '|' to separate this filter from the previous one
+                if ( !filter.empty() )
+                    filter << wxT('|');
+
+                filter << t->GetDescription() << wxT(" (") << t->GetFileFilter() << wxT(") |")
+                       << t->GetFileFilter();
+            }
+
+            node = node->GetNext();
+        }
+    }
+#else
+    wxString filter = docTemplate->GetFileFilter() ;
+#endif
      wxString tmp = wxFileSelector(_("Save as"),
              docTemplate->GetDirectory(),
              wxFileNameFromPath(GetFilename()),
              docTemplate->GetDefaultExtension(),
      wxString tmp = wxFileSelector(_("Save as"),
              docTemplate->GetDirectory(),
              wxFileNameFromPath(GetFilename()),
              docTemplate->GetDefaultExtension(),
-            docTemplate->GetFileFilter(),
-            wxSAVE | wxOVERWRITE_PROMPT,
+            filter,
+            wxFD_SAVE | wxFD_OVERWRITE_PROMPT,
              GetDocumentWindow());
  
              GetDocumentWindow());
  
-    if (tmp.IsEmpty())
-        return FALSE;
+    if (tmp.empty())
+        return false;
  
      wxString fileName(tmp);
      wxString path, name, ext;
      wxSplitPath(fileName, & path, & name, & ext);
  
  
      wxString fileName(tmp);
      wxString path, name, ext;
      wxSplitPath(fileName, & path, & name, & ext);
  
-    if (ext.IsEmpty() || ext == wxT(""))
+    if (ext.empty())
      {
          fileName += wxT(".");
          fileName += docTemplate->GetDefaultExtension();
      {
          fileName += wxT(".");
          fileName += docTemplate->GetDefaultExtension();
@@ -274,10 +321,8 @@ bool wxDocument::SaveAs()
      SetFilename(fileName);
      SetTitle(wxFileNameFromPath(fileName));
  
      SetFilename(fileName);
      SetTitle(wxFileNameFromPath(fileName));
  
-    GetDocumentManager()->AddFileToHistory(fileName);
-
      // Notify the views that the filename has changed
      // Notify the views that the filename has changed
-    wxNode *node = m_documentViews.GetFirst();
+    wxList::compatibility_iterator node = m_documentViews.GetFirst();
      while (node)
      {
          wxView *view = (wxView *)node->GetData();
      while (node)
      {
          wxView *view = (wxView *)node->GetData();
@@ -285,93 +330,57 @@ bool wxDocument::SaveAs()
          node = node->GetNext();
      }
  
          node = node->GetNext();
      }
  
-    return OnSaveDocument(m_documentFile);
+    // Files that were not saved correctly are not added to the FileHistory.
+    if (!OnSaveDocument(m_documentFile))
+        return false;
+
+   // A file that doesn't use the default extension of its document template cannot be opened
+   // via the FileHistory, so we do not add it.
+   if (docTemplate->FileMatchesTemplate(fileName))
+   {
+       GetDocumentManager()->AddFileToHistory(fileName);
+   }
+   else
+   {
+       // The user will probably not be able to open the file again, so
+       // we could warn about the wrong file-extension here.
+   }
+   return true;
  }
  
  bool wxDocument::OnSaveDocument(const wxString& file)
  {
      if ( !file )
  }
  
  bool wxDocument::OnSaveDocument(const wxString& file)
  {
      if ( !file )
-        return FALSE;
+        return false;
  
  
-    wxString msgTitle;
-    if (wxTheApp->GetAppName() != wxT(""))
-        msgTitle = wxTheApp->GetAppName();
-    else
-        msgTitle = wxString(_("File error"));
+    if ( !DoSaveDocument(file) )
+        return false;
  
  
-#if wxUSE_STD_IOSTREAM
-    wxSTD ofstream store(file.mb_str());
-    if (store.fail() || store.bad())
-#else
-    wxFileOutputStream store(file);
-    if (store.GetLastError() != wxSTREAM_NO_ERROR)
-#endif
-    {
-        (void)wxMessageBox(_("Sorry, could not open this file for saving."), msgTitle, wxOK | wxICON_EXCLAMATION,
-                           GetDocumentWindow());
-        // Saving error
-        return FALSE;
-    }
-    if (!SaveObject(store))
-    {
-        (void)wxMessageBox(_("Sorry, could not save this file."), msgTitle, wxOK | wxICON_EXCLAMATION,
-                           GetDocumentWindow());
-        // Saving error
-        return FALSE;
-    }
-    Modify(FALSE);
+    Modify(false);
      SetFilename(file);
      SetFilename(file);
-    SetDocumentSaved(TRUE);
+    SetDocumentSaved(true);
  #ifdef __WXMAC__
      wxFileName fn(file) ;
      fn.MacSetDefaultTypeAndCreator() ;
  #endif
  #ifdef __WXMAC__
      wxFileName fn(file) ;
      fn.MacSetDefaultTypeAndCreator() ;
  #endif
-    return TRUE;
+    return true;
  }
  
  bool wxDocument::OnOpenDocument(const wxString& file)
  {
      if (!OnSaveModified())
  }
  
  bool wxDocument::OnOpenDocument(const wxString& file)
  {
      if (!OnSaveModified())
-        return FALSE;
+        return false;
  
  
-    wxString msgTitle;
-    if (wxTheApp->GetAppName() != wxT(""))
-        msgTitle = wxTheApp->GetAppName();
-    else
-        msgTitle = wxString(_("File error"));
+    if ( !DoOpenDocument(file) )
+        return false;
  
  
-#if wxUSE_STD_IOSTREAM
-    wxSTD ifstream store(file.mb_str());
-    if (store.fail() || store.bad())
-#else
-    wxFileInputStream store(file);
-    if (store.GetLastError() != wxSTREAM_NO_ERROR)
-#endif
-    {
-        (void)wxMessageBox(_("Sorry, could not open this file."), msgTitle, wxOK|wxICON_EXCLAMATION,
-                           GetDocumentWindow());
-        return FALSE;
-    }
-#if wxUSE_STD_IOSTREAM
-    LoadObject(store);
-    if ( !store && !store.eof() )
-#else
-    int res = LoadObject(store).GetLastError();
-    if ((res != wxSTREAM_NO_ERROR) &&
-        (res != wxSTREAM_EOF))
-#endif
-    {
-        (void)wxMessageBox(_("Sorry, could not open this file."), msgTitle, wxOK|wxICON_EXCLAMATION,
-                           GetDocumentWindow());
-        return FALSE;
-    }
-    SetFilename(file, TRUE);
-    Modify(FALSE);
-    m_savedYet = TRUE;
+    SetFilename(file, true);
+    Modify(false);
+    m_savedYet = true;
  
      UpdateAllViews();
  
  
      UpdateAllViews();
  
-    return TRUE;
+    return true;
  }
  
  #if wxUSE_STD_IOSTREAM
  }
  
  #if wxUSE_STD_IOSTREAM
@@ -394,27 +403,27 @@ wxOutputStream& wxDocument::SaveObject(wxOutputStream& stream)
  
  bool wxDocument::Revert()
  {
  
  bool wxDocument::Revert()
  {
-    return FALSE;
+    return false;
  }
  
  
  // Get title, or filename if no title, else unnamed
  bool wxDocument::GetPrintableName(wxString& buf) const
  {
  }
  
  
  // Get title, or filename if no title, else unnamed
  bool wxDocument::GetPrintableName(wxString& buf) const
  {
-    if (m_documentTitle != wxT(""))
+    if (!m_documentTitle.empty())
      {
          buf = m_documentTitle;
      {
          buf = m_documentTitle;
-        return TRUE;
+        return true;
      }
      }
-    else if (m_documentFile != wxT(""))
+    else if (!m_documentFile.empty())
      {
          buf = wxFileNameFromPath(m_documentFile);
      {
          buf = wxFileNameFromPath(m_documentFile);
-        return TRUE;
+        return true;
      }
      else
      {
          buf = _("unnamed");
      }
      else
      {
          buf = _("unnamed");
-        return TRUE;
+        return true;
      }
  }
  
      }
  }
  
@@ -432,7 +441,7 @@ wxCommandProcessor *wxDocument::OnCreateCommandProcessor()
      return new wxCommandProcessor;
  }
  
      return new wxCommandProcessor;
  }
  
-// TRUE if safe to close
+// true if safe to close
  bool wxDocument::OnSaveModified()
  {
      if (IsModified())
  bool wxDocument::OnSaveModified()
  {
      if (IsModified())
@@ -441,7 +450,7 @@ bool wxDocument::OnSaveModified()
          GetPrintableName(title);
  
          wxString msgTitle;
          GetPrintableName(title);
  
          wxString msgTitle;
-        if (wxTheApp->GetAppName() != wxT(""))
+        if (!wxTheApp->GetAppName().empty())
              msgTitle = wxTheApp->GetAppName();
          else
              msgTitle = wxString(_("Warning"));
              msgTitle = wxTheApp->GetAppName();
          else
              msgTitle = wxString(_("Warning"));
@@ -454,20 +463,20 @@ bool wxDocument::OnSaveModified()
                  GetDocumentWindow());
          if (res == wxNO)
          {
                  GetDocumentWindow());
          if (res == wxNO)
          {
-            Modify(FALSE);
-            return TRUE;
+            Modify(false);
+            return true;
          }
          else if (res == wxYES)
              return Save();
          else if (res == wxCANCEL)
          }
          else if (res == wxYES)
              return Save();
          else if (res == wxCANCEL)
-            return FALSE;
+            return false;
      }
      }
-    return TRUE;
+    return true;
  }
  
  bool wxDocument::Draw(wxDC& WXUNUSED(context))
  {
  }
  
  bool wxDocument::Draw(wxDC& WXUNUSED(context))
  {
-    return TRUE;
+    return true;
  }
  
  bool wxDocument::AddView(wxView *view)
  }
  
  bool wxDocument::AddView(wxView *view)
@@ -477,22 +486,22 @@ bool wxDocument::AddView(wxView *view)
          m_documentViews.Append(view);
          OnChangedViewList();
      }
          m_documentViews.Append(view);
          OnChangedViewList();
      }
-    return TRUE;
+    return true;
  }
  
  bool wxDocument::RemoveView(wxView *view)
  {
      (void)m_documentViews.DeleteObject(view);
      OnChangedViewList();
  }
  
  bool wxDocument::RemoveView(wxView *view)
  {
      (void)m_documentViews.DeleteObject(view);
      OnChangedViewList();
-    return TRUE;
+    return true;
  }
  
  bool wxDocument::OnCreate(const wxString& WXUNUSED(path), long flags)
  {
      if (GetDocumentTemplate()->CreateView(this, flags))
  }
  
  bool wxDocument::OnCreate(const wxString& WXUNUSED(path), long flags)
  {
      if (GetDocumentTemplate()->CreateView(this, flags))
-        return TRUE;
+        return true;
      else
      else
-        return FALSE;
+        return false;
  }
  
  // Called after a view is added or removed.
  }
  
  // Called after a view is added or removed.
@@ -511,11 +520,11 @@ void wxDocument::OnChangedViewList()
  
  void wxDocument::UpdateAllViews(wxView *sender, wxObject *hint)
  {
  
  void wxDocument::UpdateAllViews(wxView *sender, wxObject *hint)
  {
-    wxNode *node = m_documentViews.GetFirst();
+    wxList::compatibility_iterator node = m_documentViews.GetFirst();
      while (node)
      {
          wxView *view = (wxView *)node->GetData();
      while (node)
      {
          wxView *view = (wxView *)node->GetData();
-               if (view != sender)
+        if (view != sender)
              view->OnUpdate(sender, hint);
          node = node->GetNext();
      }
              view->OnUpdate(sender, hint);
          node = node->GetNext();
      }
@@ -523,7 +532,7 @@ void wxDocument::UpdateAllViews(wxView *sender, wxObject *hint)
  
  void wxDocument::NotifyClosing()
  {
  
  void wxDocument::NotifyClosing()
  {
-    wxNode *node = m_documentViews.GetFirst();
+    wxList::compatibility_iterator node = m_documentViews.GetFirst();
      while (node)
      {
          wxView *view = (wxView *)node->GetData();
      while (node)
      {
          wxView *view = (wxView *)node->GetData();
@@ -538,7 +547,7 @@ void wxDocument::SetFilename(const wxString& filename, bool notifyViews)
      if ( notifyViews )
      {
          // Notify the views that the filename has changed
      if ( notifyViews )
      {
          // Notify the views that the filename has changed
-        wxNode *node = m_documentViews.GetFirst();
+        wxList::compatibility_iterator node = m_documentViews.GetFirst();
          while (node)
          {
              wxView *view = (wxView *)node->GetData();
          while (node)
          {
              wxView *view = (wxView *)node->GetData();
@@ -548,22 +557,77 @@ void wxDocument::SetFilename(const wxString& filename, bool notifyViews)
      }
  }
  
      }
  }
  
+bool wxDocument::DoSaveDocument(const wxString& file)
+{
+    wxString msgTitle;
+    if (!wxTheApp->GetAppName().empty())
+        msgTitle = wxTheApp->GetAppName();
+    else
+        msgTitle = wxString(_("File error"));
+
+#if wxUSE_STD_IOSTREAM
+    wxSTD ofstream store(file.mb_str(), wxSTD ios::binary);
+    if (store.fail() || store.bad())
+#else
+    wxFileOutputStream store(file);
+    if (store.GetLastError() != wxSTREAM_NO_ERROR)
+#endif
+    {
+        (void)wxMessageBox(_("Sorry, could not open this file for saving."), msgTitle, wxOK | wxICON_EXCLAMATION,
+                           GetDocumentWindow());
+        // Saving error
+        return false;
+    }
+    if (!SaveObject(store))
+    {
+        (void)wxMessageBox(_("Sorry, could not save this file."), msgTitle, wxOK | wxICON_EXCLAMATION,
+                           GetDocumentWindow());
+        // Saving error
+        return false;
+    }
+
+    return true;
+}
+
+bool wxDocument::DoOpenDocument(const wxString& file)
+{
+#if wxUSE_STD_IOSTREAM
+    wxSTD ifstream store(file.mb_str(), wxSTD ios::binary);
+    if (!store.fail() && !store.bad())
+#else
+    wxFileInputStream store(file);
+    if (store.GetLastError() == wxSTREAM_NO_ERROR)
+#endif
+    {
+#if wxUSE_STD_IOSTREAM
+        LoadObject(store);
+        if ( !!store || store.eof() )
+#else
+        int res = LoadObject(store).GetLastError();
+        if ( res == wxSTREAM_NO_ERROR || res == wxSTREAM_EOF )
+#endif
+            return true;
+    }
+
+    wxLogError(_("Sorry, could not open this file."));
+    return false;
+}
+
+
  // ----------------------------------------------------------------------------
  // Document view
  // ----------------------------------------------------------------------------
  
  wxView::wxView()
  {
  // ----------------------------------------------------------------------------
  // Document view
  // ----------------------------------------------------------------------------
  
  wxView::wxView()
  {
-    //  SetDocument(doc);
      m_viewDocument = (wxDocument*) NULL;
  
      m_viewDocument = (wxDocument*) NULL;
  
-    m_viewTypeName = wxT("");
      m_viewFrame = (wxFrame *) NULL;
  }
  
  wxView::~wxView()
  {
      m_viewFrame = (wxFrame *) NULL;
  }
  
  wxView::~wxView()
  {
-//    GetDocumentManager()->ActivateView(this, FALSE, TRUE);
+    GetDocumentManager()->ActivateView(this, false);
      m_viewDocument->RemoveView(this);
  }
  
      m_viewDocument->RemoveView(this);
  }
  
@@ -572,8 +636,8 @@ bool wxView::ProcessEvent(wxEvent& event)
  {
      if ( !GetDocument() || !GetDocument()->ProcessEvent(event) )
          return wxEvtHandler::ProcessEvent(event);
  {
      if ( !GetDocument() || !GetDocument()->ProcessEvent(event) )
          return wxEvtHandler::ProcessEvent(event);
-    else
-        return TRUE;
+
+    return true;
  }
  
  void wxView::OnActivateView(bool WXUNUSED(activate), wxView *WXUNUSED(activeView), wxView *WXUNUSED(deactiveView))
  }
  
  void wxView::OnActivateView(bool WXUNUSED(activate), wxView *WXUNUSED(activeView), wxView *WXUNUSED(deactiveView))
@@ -591,14 +655,18 @@ void wxView::OnUpdate(wxView *WXUNUSED(sender), wxObject *WXUNUSED(hint))
  
  void wxView::OnChangeFilename()
  {
  
  void wxView::OnChangeFilename()
  {
-    if (GetFrame() && GetDocument())
-    {
-        wxString title;
+    // GetFrame can return wxWindow rather than wxTopLevelWindow due to
+    // generic MDI implementation so use SetLabel rather than SetTitle.
+    // It should cause SetTitle() for top level windows.
+    wxWindow *win = GetFrame();
+    if (!win) return;
  
  
-        GetDocument()->GetPrintableName(title);
+    wxDocument *doc = GetDocument();
+    if (!doc) return;
  
  
-        GetFrame()->SetTitle(title);
-    }
+    wxString name;
+    doc->GetPrintableName(name);
+    win->SetLabel(name);
  }
  
  void wxView::SetDocument(wxDocument *doc)
  }
  
  void wxView::SetDocument(wxDocument *doc)
@@ -611,9 +679,9 @@ void wxView::SetDocument(wxDocument *doc)
  bool wxView::Close(bool deleteWindow)
  {
      if (OnClose(deleteWindow))
  bool wxView::Close(bool deleteWindow)
  {
      if (OnClose(deleteWindow))
-        return TRUE;
+        return true;
      else
      else
-        return FALSE;
+        return false;
  }
  
  void wxView::Activate(bool activate)
  }
  
  void wxView::Activate(bool activate)
@@ -627,7 +695,7 @@ void wxView::Activate(bool activate)
  
  bool wxView::OnClose(bool WXUNUSED(deleteWindow))
  {
  
  bool wxView::OnClose(bool WXUNUSED(deleteWindow))
  {
-    return GetDocument() ? GetDocument()->Close() : TRUE;
+    return GetDocument() ? GetDocument()->Close() : true;
  }
  
  #if wxUSE_PRINTING_ARCHITECTURE
  }
  
  #if wxUSE_PRINTING_ARCHITECTURE
@@ -674,29 +742,43 @@ wxDocTemplate::~wxDocTemplate()
  // Tries to dynamically construct an object of the right class.
  wxDocument *wxDocTemplate::CreateDocument(const wxString& path, long flags)
  {
  // Tries to dynamically construct an object of the right class.
  wxDocument *wxDocTemplate::CreateDocument(const wxString& path, long flags)
  {
-    if (!m_docClassInfo)
+    wxDocument *doc = DoCreateDocument();
+    if ( doc == NULL )
+        return (wxDocument *) NULL;
+
+    if (InitDocument(doc, path, flags))
+    {
+        return doc;
+    }
+    else
+    {
          return (wxDocument *) NULL;
          return (wxDocument *) NULL;
-    wxDocument *doc = (wxDocument *)m_docClassInfo->CreateObject();
+    }
+}
+
+bool wxDocTemplate::InitDocument(wxDocument* doc, const wxString& path, long flags)
+{
      doc->SetFilename(path);
      doc->SetDocumentTemplate(this);
      GetDocumentManager()->AddDocument(doc);
      doc->SetCommandProcessor(doc->OnCreateCommandProcessor());
  
      if (doc->OnCreate(path, flags))
      doc->SetFilename(path);
      doc->SetDocumentTemplate(this);
      GetDocumentManager()->AddDocument(doc);
      doc->SetCommandProcessor(doc->OnCreateCommandProcessor());
  
      if (doc->OnCreate(path, flags))
-        return doc;
+        return true;
      else
      {
          if (GetDocumentManager()->GetDocuments().Member(doc))
              doc->DeleteAllViews();
      else
      {
          if (GetDocumentManager()->GetDocuments().Member(doc))
              doc->DeleteAllViews();
-        return (wxDocument *) NULL;
+        return false;
      }
  }
  
  wxView *wxDocTemplate::CreateView(wxDocument *doc, long flags)
  {
      }
  }
  
  wxView *wxDocTemplate::CreateView(wxDocument *doc, long flags)
  {
-    if (!m_viewClassInfo)
+    wxView *view = DoCreateView();
+    if ( view == NULL )
          return (wxView *) NULL;
          return (wxView *) NULL;
-    wxView *view = (wxView *)m_viewClassInfo->CreateObject();
+
      view->SetDocument(doc);
      if (view->OnCreate(doc, flags))
      {
      view->SetDocument(doc);
      if (view->OnCreate(doc, flags))
      {
@@ -713,9 +795,36 @@ wxView *wxDocTemplate::CreateView(wxDocument *doc, long flags)
  // that of the template
  bool wxDocTemplate::FileMatchesTemplate(const wxString& path)
  {
  // that of the template
  bool wxDocTemplate::FileMatchesTemplate(const wxString& path)
  {
+    wxStringTokenizer parser (GetFileFilter(), wxT(";"));
+    wxString anything = wxT ("*");
+    while (parser.HasMoreTokens())
+    {
+        wxString filter = parser.GetNextToken();
+        wxString filterExt = FindExtension (filter);
+        if ( filter.IsSameAs (anything)    ||
+             filterExt.IsSameAs (anything) ||
+             filterExt.IsSameAs (FindExtension (path)) )
+            return true;
+    }
      return GetDefaultExtension().IsSameAs(FindExtension(path));
  }
  
      return GetDefaultExtension().IsSameAs(FindExtension(path));
  }
  
+wxDocument *wxDocTemplate::DoCreateDocument()
+{
+    if (!m_docClassInfo)
+        return (wxDocument *) NULL;
+
+    return (wxDocument *)m_docClassInfo->CreateObject();
+}
+
+wxView *wxDocTemplate::DoCreateView()
+{
+    if (!m_viewClassInfo)
+        return (wxView *) NULL;
+
+    return (wxView *)m_viewClassInfo->CreateObject();
+}
+
  // ----------------------------------------------------------------------------
  // wxDocManager
  // ----------------------------------------------------------------------------
  // ----------------------------------------------------------------------------
  // wxDocManager
  // ----------------------------------------------------------------------------
@@ -743,11 +852,9 @@ BEGIN_EVENT_TABLE(wxDocManager, wxEvtHandler)
  
  #if wxUSE_PRINTING_ARCHITECTURE
      EVT_MENU(wxID_PRINT, wxDocManager::OnPrint)
  
  #if wxUSE_PRINTING_ARCHITECTURE
      EVT_MENU(wxID_PRINT, wxDocManager::OnPrint)
-    EVT_MENU(wxID_PRINT_SETUP, wxDocManager::OnPrintSetup)
      EVT_MENU(wxID_PREVIEW, wxDocManager::OnPreview)
  
      EVT_UPDATE_UI(wxID_PRINT, wxDocManager::OnUpdatePrint)
      EVT_MENU(wxID_PREVIEW, wxDocManager::OnPreview)
  
      EVT_UPDATE_UI(wxID_PRINT, wxDocManager::OnUpdatePrint)
-    EVT_UPDATE_UI(wxID_PRINT_SETUP, wxDocManager::OnUpdatePrintSetup)
      EVT_UPDATE_UI(wxID_PREVIEW, wxDocManager::OnUpdatePreview)
  #endif
  END_EVENT_TABLE()
      EVT_UPDATE_UI(wxID_PREVIEW, wxDocManager::OnUpdatePreview)
  #endif
  END_EVENT_TABLE()
@@ -786,51 +893,53 @@ bool wxDocManager::CloseDocument(wxDocument* doc, bool force)
          // Check we're really deleted
          if (m_docs.Member(doc))
              delete doc;
          // Check we're really deleted
          if (m_docs.Member(doc))
              delete doc;
-        
-        return TRUE;
+
+        return true;
      }
      }
-    return FALSE;
+    return false;
  }
  
  bool wxDocManager::CloseDocuments(bool force)
  {
  }
  
  bool wxDocManager::CloseDocuments(bool force)
  {
-    wxNode *node = m_docs.GetFirst();
+    wxList::compatibility_iterator node = m_docs.GetFirst();
      while (node)
      {
          wxDocument *doc = (wxDocument *)node->GetData();
      while (node)
      {
          wxDocument *doc = (wxDocument *)node->GetData();
-        wxNode *next = node->GetNext();
-        
+        wxList::compatibility_iterator next = node->GetNext();
+
          if (!CloseDocument(doc, force))
          if (!CloseDocument(doc, force))
-            return FALSE;
+            return false;
  
          // This assumes that documents are not connected in
          // any way, i.e. deleting one document does NOT
          // delete another.
          node = next;
      }
  
          // This assumes that documents are not connected in
          // any way, i.e. deleting one document does NOT
          // delete another.
          node = next;
      }
-    return TRUE;
+    return true;
  }
  
  bool wxDocManager::Clear(bool force)
  {
      if (!CloseDocuments(force))
  }
  
  bool wxDocManager::Clear(bool force)
  {
      if (!CloseDocuments(force))
-        return FALSE;
+        return false;
+
+    m_currentView = NULL;
  
  
-    wxNode *node = m_templates.GetFirst();
+    wxList::compatibility_iterator node = m_templates.GetFirst();
      while (node)
      {
          wxDocTemplate *templ = (wxDocTemplate*) node->GetData();
      while (node)
      {
          wxDocTemplate *templ = (wxDocTemplate*) node->GetData();
-        wxNode* next = node->GetNext();
+        wxList::compatibility_iterator next = node->GetNext();
          delete templ;
          node = next;
      }
          delete templ;
          node = next;
      }
-    return TRUE;
+    return true;
  }
  
  bool wxDocManager::Initialize()
  {
      m_fileHistory = OnCreateFileHistory();
  }
  
  bool wxDocManager::Initialize()
  {
      m_fileHistory = OnCreateFileHistory();
-    return TRUE;
+    return true;
  }
  
  wxFileHistory *wxDocManager::OnCreateFileHistory()
  }
  
  wxFileHistory *wxDocManager::OnCreateFileHistory()
@@ -853,17 +962,17 @@ void wxDocManager::OnFileClose(wxCommandEvent& WXUNUSED(event))
  
  void wxDocManager::OnFileCloseAll(wxCommandEvent& WXUNUSED(event))
  {
  
  void wxDocManager::OnFileCloseAll(wxCommandEvent& WXUNUSED(event))
  {
-    CloseDocuments(FALSE);
+    CloseDocuments(false);
  }
  
  void wxDocManager::OnFileNew(wxCommandEvent& WXUNUSED(event))
  {
  }
  
  void wxDocManager::OnFileNew(wxCommandEvent& WXUNUSED(event))
  {
-    CreateDocument( wxT(""), wxDOC_NEW );
+    CreateDocument( wxEmptyString, wxDOC_NEW );
  }
  
  void wxDocManager::OnFileOpen(wxCommandEvent& WXUNUSED(event))
  {
  }
  
  void wxDocManager::OnFileOpen(wxCommandEvent& WXUNUSED(event))
  {
-    if ( !CreateDocument( wxT(""), 0) )
+    if ( !CreateDocument( wxEmptyString, 0) )
      {
          OnOpenFileFailure();
      }
      {
          OnOpenFileFailure();
      }
@@ -904,29 +1013,13 @@ void wxDocManager::OnPrint(wxCommandEvent& WXUNUSED(event))
      if (printout)
      {
          wxPrinter printer;
      if (printout)
      {
          wxPrinter printer;
-        printer.Print(view->GetFrame(), printout, TRUE);
+        printer.Print(view->GetFrame(), printout, true);
  
          delete printout;
      }
  #endif // wxUSE_PRINTING_ARCHITECTURE
  }
  
  
          delete printout;
      }
  #endif // wxUSE_PRINTING_ARCHITECTURE
  }
  
-void wxDocManager::OnPrintSetup(wxCommandEvent& WXUNUSED(event))
-{
-#if wxUSE_PRINTING_ARCHITECTURE
-    wxWindow *parentWin = wxTheApp->GetTopWindow();
-    wxView *view = GetCurrentView();
-    if (view)
-        parentWin = view->GetFrame();
-
-    wxPrintDialogData data;
-
-    wxPrintDialog printerDialog(parentWin, &data);
-    printerDialog.GetPrintDialogData().SetSetupDialog(TRUE);
-    printerDialog.ShowModal();
-#endif // wxUSE_PRINTING_ARCHITECTURE
-}
-
  void wxDocManager::OnPreview(wxCommandEvent& WXUNUSED(event))
  {
  #if wxUSE_PRINTING_ARCHITECTURE
  void wxDocManager::OnPreview(wxCommandEvent& WXUNUSED(event))
  {
  #if wxUSE_PRINTING_ARCHITECTURE
@@ -938,41 +1031,50 @@ void wxDocManager::OnPreview(wxCommandEvent& WXUNUSED(event))
      if (printout)
      {
          // Pass two printout objects: for preview, and possible printing.
      if (printout)
      {
          // Pass two printout objects: for preview, and possible printing.
-        wxPrintPreviewBase *preview = (wxPrintPreviewBase *) NULL;
-        preview = new wxPrintPreview(printout, view->OnCreatePrintout());
+        wxPrintPreviewBase *preview = new wxPrintPreview(printout, view->OnCreatePrintout());
+        if ( !preview->Ok() )
+        {
+            delete preview;
+            wxMessageBox( _("Sorry, print preview needs a printer to be installed.") );
+            return;
+        }
  
          wxPreviewFrame *frame = new wxPreviewFrame(preview, (wxFrame *)wxTheApp->GetTopWindow(), _("Print Preview"),
                  wxPoint(100, 100), wxSize(600, 650));
          frame->Centre(wxBOTH);
          frame->Initialize();
  
          wxPreviewFrame *frame = new wxPreviewFrame(preview, (wxFrame *)wxTheApp->GetTopWindow(), _("Print Preview"),
                  wxPoint(100, 100), wxSize(600, 650));
          frame->Centre(wxBOTH);
          frame->Initialize();
-        frame->Show(TRUE);
+        frame->Show(true);
      }
  #endif // wxUSE_PRINTING_ARCHITECTURE
  }
  
      }
  #endif // wxUSE_PRINTING_ARCHITECTURE
  }
  
-void wxDocManager::OnUndo(wxCommandEvent& WXUNUSED(event))
+void wxDocManager::OnUndo(wxCommandEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
      if (!doc)
          return;
      if (doc->GetCommandProcessor())
          doc->GetCommandProcessor()->Undo();
  {
      wxDocument *doc = GetCurrentDocument();
      if (!doc)
          return;
      if (doc->GetCommandProcessor())
          doc->GetCommandProcessor()->Undo();
+    else
+        event.Skip();
  }
  
  }
  
-void wxDocManager::OnRedo(wxCommandEvent& WXUNUSED(event))
+void wxDocManager::OnRedo(wxCommandEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
      if (!doc)
          return;
      if (doc->GetCommandProcessor())
          doc->GetCommandProcessor()->Redo();
  {
      wxDocument *doc = GetCurrentDocument();
      if (!doc)
          return;
      if (doc->GetCommandProcessor())
          doc->GetCommandProcessor()->Redo();
+    else
+        event.Skip();
  }
  
  // Handlers for UI update commands
  
  void wxDocManager::OnUpdateFileOpen(wxUpdateUIEvent& event)
  {
  }
  
  // Handlers for UI update commands
  
  void wxDocManager::OnUpdateFileOpen(wxUpdateUIEvent& event)
  {
-    event.Enable( TRUE );
+    event.Enable( true );
  }
  
  void wxDocManager::OnUpdateFileClose(wxUpdateUIEvent& event)
  }
  
  void wxDocManager::OnUpdateFileClose(wxUpdateUIEvent& event)
@@ -989,7 +1091,7 @@ void wxDocManager::OnUpdateFileRevert(wxUpdateUIEvent& event)
  
  void wxDocManager::OnUpdateFileNew(wxUpdateUIEvent& event)
  {
  
  void wxDocManager::OnUpdateFileNew(wxUpdateUIEvent& event)
  {
-    event.Enable( TRUE );
+    event.Enable( true );
  }
  
  void wxDocManager::OnUpdateFileSave(wxUpdateUIEvent& event)
  }
  
  void wxDocManager::OnUpdateFileSave(wxUpdateUIEvent& event)
@@ -1007,17 +1109,29 @@ void wxDocManager::OnUpdateFileSaveAs(wxUpdateUIEvent& event)
  void wxDocManager::OnUpdateUndo(wxUpdateUIEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
  void wxDocManager::OnUpdateUndo(wxUpdateUIEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
-    event.Enable( (doc && doc->GetCommandProcessor() && doc->GetCommandProcessor()->CanUndo()) );
-    if (doc && doc->GetCommandProcessor())
+    if (!doc)
+        event.Enable(false);
+    else if (!doc->GetCommandProcessor())
+        event.Skip();
+    else
+    {
+        event.Enable( doc->GetCommandProcessor()->CanUndo() );
          doc->GetCommandProcessor()->SetMenuStrings();
          doc->GetCommandProcessor()->SetMenuStrings();
+    }
  }
  
  void wxDocManager::OnUpdateRedo(wxUpdateUIEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
  }
  
  void wxDocManager::OnUpdateRedo(wxUpdateUIEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
-    event.Enable( (doc && doc->GetCommandProcessor() && doc->GetCommandProcessor()->CanRedo()) );
-    if (doc && doc->GetCommandProcessor())
+    if (!doc)
+        event.Enable(false);
+    else if (!doc->GetCommandProcessor())
+        event.Skip();
+    else
+    {
+        event.Enable( doc->GetCommandProcessor()->CanRedo() );
          doc->GetCommandProcessor()->SetMenuStrings();
          doc->GetCommandProcessor()->SetMenuStrings();
+    }
  }
  
  void wxDocManager::OnUpdatePrint(wxUpdateUIEvent& event)
  }
  
  void wxDocManager::OnUpdatePrint(wxUpdateUIEvent& event)
@@ -1026,11 +1140,6 @@ void wxDocManager::OnUpdatePrint(wxUpdateUIEvent& event)
      event.Enable( (doc != (wxDocument*) NULL) );
  }
  
      event.Enable( (doc != (wxDocument*) NULL) );
  }
  
-void wxDocManager::OnUpdatePrintSetup(wxUpdateUIEvent& event)
-{
-    event.Enable( TRUE );
-}
-
  void wxDocManager::OnUpdatePreview(wxUpdateUIEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
  void wxDocManager::OnUpdatePreview(wxUpdateUIEvent& event)
  {
      wxDocument *doc = GetCurrentDocument();
@@ -1056,7 +1165,7 @@ bool wxDocManager::ProcessEvent(wxEvent& event)
      if (view)
      {
          if (view->ProcessEvent(event))
      if (view)
      {
          if (view->ProcessEvent(event))
-            return TRUE;
+            return true;
      }
      return wxEvtHandler::ProcessEvent(event);
  }
      }
      return wxEvtHandler::ProcessEvent(event);
  }
@@ -1080,7 +1189,7 @@ wxDocument *wxDocManager::CreateDocument(const wxString& path, long flags)
          delete[] templates;
          return (wxDocument *) NULL;
      }
          delete[] templates;
          return (wxDocument *) NULL;
      }
-    
+
      wxDocument* docToClose = NULL;
  
      // If we've reached the max number of docs, close the
      wxDocument* docToClose = NULL;
  
      // If we've reached the max number of docs, close the
@@ -1098,22 +1207,27 @@ wxDocument *wxDocManager::CreateDocument(const wxString& path, long flags)
          {
              if (docToClose)
              {
          {
              if (docToClose)
              {
-                if (!CloseDocument(docToClose, FALSE))
+                if (!CloseDocument(docToClose, false))
                  {
                      delete[] templates;
                      return NULL;
                  }
              }
                  {
                      delete[] templates;
                      return NULL;
                  }
              }
-            
+
              wxDocTemplate *temp = templates[0];
              delete[] templates;
              wxDocument *newDoc = temp->CreateDocument(path, flags);
              wxDocTemplate *temp = templates[0];
              delete[] templates;
              wxDocument *newDoc = temp->CreateDocument(path, flags);
-            
+
              if (newDoc)
              {
                  newDoc->SetDocumentName(temp->GetDocumentName());
                  newDoc->SetDocumentTemplate(temp);
              if (newDoc)
              {
                  newDoc->SetDocumentName(temp->GetDocumentName());
                  newDoc->SetDocumentTemplate(temp);
-                newDoc->OnNewDocument();
+                if (!newDoc->OnNewDocument() )
+                {
+                     // Document is implicitly deleted by DeleteAllViews
+                     newDoc->DeleteAllViews();
+                     return NULL;
+                }
              }
              return newDoc;
          }
              }
              return newDoc;
          }
@@ -1124,19 +1238,24 @@ wxDocument *wxDocManager::CreateDocument(const wxString& path, long flags)
          {
              if (docToClose)
              {
          {
              if (docToClose)
              {
-                if (!CloseDocument(docToClose, FALSE))
+                if (!CloseDocument(docToClose, false))
                  {
                      return NULL;
                  }
              }
                  {
                      return NULL;
                  }
              }
-            
+
              wxDocument *newDoc = temp->CreateDocument(path, flags);
  
              if (newDoc)
              {
                  newDoc->SetDocumentName(temp->GetDocumentName());
                  newDoc->SetDocumentTemplate(temp);
              wxDocument *newDoc = temp->CreateDocument(path, flags);
  
              if (newDoc)
              {
                  newDoc->SetDocumentName(temp->GetDocumentName());
                  newDoc->SetDocumentTemplate(temp);
-                newDoc->OnNewDocument();
+                if (!newDoc->OnNewDocument() )
+                {
+                     // Document is implicitly deleted by DeleteAllViews
+                     newDoc->DeleteAllViews();
+                     return NULL;
+                }
              }
              return newDoc;
          }
              }
              return newDoc;
          }
@@ -1145,14 +1264,22 @@ wxDocument *wxDocManager::CreateDocument(const wxString& path, long flags)
      }
  
      // Existing document
      }
  
      // Existing document
-    wxDocTemplate *temp = (wxDocTemplate *) NULL;
+    wxDocTemplate *temp;
  
  
-    wxString path2(wxT(""));
-    if (path != wxT(""))
-        path2 = path;
+    wxString path2 = path;
  
      if (flags & wxDOC_SILENT)
  
      if (flags & wxDOC_SILENT)
+    {
          temp = FindTemplateForPath(path2);
          temp = FindTemplateForPath(path2);
+        if (!temp)
+        {
+            // Since we do not add files with non-default extensions to the FileHistory this
+            // can only happen if the application changes the allowed templates in runtime.
+            (void)wxMessageBox(_("Sorry, the format for this file is unknown."),
+                                _("Open File"),
+                               wxOK | wxICON_EXCLAMATION, wxFindSuitableParent());
+        }
+    }
      else
          temp = SelectDocumentPath(templates, n, path2, flags);
  
      else
          temp = SelectDocumentPath(templates, n, path2, flags);
  
@@ -1162,12 +1289,34 @@ wxDocument *wxDocManager::CreateDocument(const wxString& path, long flags)
      {
          if (docToClose)
          {
      {
          if (docToClose)
          {
-            if (!CloseDocument(docToClose, FALSE))
+            if (!CloseDocument(docToClose, false))
              {
                  return NULL;
              }
          }
              {
                  return NULL;
              }
          }
-        
+
+        //see if this file is already open
+        for (size_t i = 0; i < GetDocuments().GetCount(); ++i)
+        {
+            wxDocument* currentDoc = (wxDocument*)(GetDocuments().Item(i)->GetData());
+#ifdef __WXMSW__
+            //file paths are case-insensitive on Windows
+            if (path2.CmpNoCase(currentDoc->GetFilename()) == 0)
+#else
+            if (path2.Cmp(currentDoc->GetFilename()) == 0)
+#endif
+            {
+                //file already open. Just activate it and return
+                if (currentDoc->GetFirstView())
+                {
+                    ActivateView(currentDoc->GetFirstView(), true);
+                    if (currentDoc->GetDocumentWindow())
+                        currentDoc->GetDocumentWindow()->SetFocus();
+                    return currentDoc;
+                }
+            }
+        }
+
          wxDocument *newDoc = temp->CreateDocument(path2, flags);
          if (newDoc)
          {
          wxDocument *newDoc = temp->CreateDocument(path2, flags);
          if (newDoc)
          {
@@ -1179,7 +1328,11 @@ wxDocument *wxDocManager::CreateDocument(const wxString& path, long flags)
                  // delete newDoc; // Implicitly deleted by DeleteAllViews
                  return (wxDocument *) NULL;
              }
                  // delete newDoc; // Implicitly deleted by DeleteAllViews
                  return (wxDocument *) NULL;
              }
-            AddFileToHistory(path2);
+            // A file that doesn't use the default extension of its document
+            // template cannot be opened via the FileHistory, so we do not
+            // add it.
+            if (temp->FileMatchesTemplate(path2))
+                AddFileToHistory(path2);
          }
          return newDoc;
      }
          }
          return newDoc;
      }
@@ -1240,7 +1393,7 @@ void wxDocManager::DeleteTemplate(wxDocTemplate *WXUNUSED(temp), long WXUNUSED(f
  // Not yet implemented
  bool wxDocManager::FlushDoc(wxDocument *WXUNUSED(doc))
  {
  // Not yet implemented
  bool wxDocManager::FlushDoc(wxDocument *WXUNUSED(doc))
  {
-    return FALSE;
+    return false;
  }
  
  wxDocument *wxDocManager::GetCurrentDocument() const
  }
  
  wxDocument *wxDocManager::GetCurrentDocument() const
@@ -1258,7 +1411,7 @@ bool wxDocManager::MakeDefaultName(wxString& name)
      name.Printf(_("unnamed%d"), m_defaultDocumentNameCounter);
      m_defaultDocumentNameCounter++;
  
      name.Printf(_("unnamed%d"), m_defaultDocumentNameCounter);
      m_defaultDocumentNameCounter++;
  
-    return TRUE;
+    return true;
  }
  
  // Make a frame title (override this to do something different)
  }
  
  // Make a frame title (override this to do something different)
@@ -1346,12 +1499,9 @@ void wxDocManager::FileHistoryAddFilesToMenu()
          m_fileHistory->AddFilesToMenu();
  }
  
          m_fileHistory->AddFilesToMenu();
  }
  
-size_t wxDocManager::GetNoHistoryFiles() const
+size_t wxDocManager::GetHistoryFilesCount() const
  {
  {
-    if (m_fileHistory)
-        return m_fileHistory->GetNoHistoryFiles();
-    else
-        return 0;
+    return m_fileHistory ? m_fileHistory->GetCount() : 0;
  }
  
  
  }
  
  
@@ -1397,9 +1547,8 @@ static wxWindow* wxFindSuitableParent()
  }
  
  // Prompts user to open a file, using file specs in templates.
  }
  
  // Prompts user to open a file, using file specs in templates.
-// How to implement in wxWindows? Must extend the file selector
-// dialog or implement own; OR match the extension to the
-// template extension.
+// Must extend the file selector dialog or implement own; OR
+// match the extension to the template extension.
  
  wxDocTemplate *wxDocManager::SelectDocumentPath(wxDocTemplate **templates,
  #if defined(__WXMSW__) || defined(__WXGTK__) || defined(__WXMAC__)
  
  wxDocTemplate *wxDocManager::SelectDocumentPath(wxDocTemplate **templates,
  #if defined(__WXMSW__) || defined(__WXGTK__) || defined(__WXMAC__)
@@ -1421,7 +1570,7 @@ wxDocTemplate *wxDocManager::SelectDocumentPath(wxDocTemplate **templates,
          if (templates[i]->IsVisible())
          {
              // add a '|' to separate this filter from the previous one
          if (templates[i]->IsVisible())
          {
              // add a '|' to separate this filter from the previous one
-            if ( !descrBuf.IsEmpty() )
+            if ( !descrBuf.empty() )
                  descrBuf << wxT('|');
  
              descrBuf << templates[i]->GetDescription()
                  descrBuf << wxT('|');
  
              descrBuf << templates[i]->GetDescription()
@@ -1439,19 +1588,19 @@ wxDocTemplate *wxDocManager::SelectDocumentPath(wxDocTemplate **templates,
  
      wxString pathTmp = wxFileSelectorEx(_("Select a file"),
                                          m_lastDirectory,
  
      wxString pathTmp = wxFileSelectorEx(_("Select a file"),
                                          m_lastDirectory,
-                                        wxT(""),
+                                        wxEmptyString,
                                          &FilterIndex,
                                          descrBuf,
                                          0,
                                          parent);
  
      wxDocTemplate *theTemplate = (wxDocTemplate *)NULL;
                                          &FilterIndex,
                                          descrBuf,
                                          0,
                                          parent);
  
      wxDocTemplate *theTemplate = (wxDocTemplate *)NULL;
-    if (!pathTmp.IsEmpty())
+    if (!pathTmp.empty())
      {
          if (!wxFileExists(pathTmp))
          {
              wxString msgTitle;
      {
          if (!wxFileExists(pathTmp))
          {
              wxString msgTitle;
-            if (!wxTheApp->GetAppName().IsEmpty())
+            if (!wxTheApp->GetAppName().empty())
                  msgTitle = wxTheApp->GetAppName();
              else
                  msgTitle = wxString(_("File error"));
                  msgTitle = wxTheApp->GetAppName();
              else
                  msgTitle = wxString(_("File error"));
@@ -1459,7 +1608,7 @@ wxDocTemplate *wxDocManager::SelectDocumentPath(wxDocTemplate **templates,
              (void)wxMessageBox(_("Sorry, could not open this file."), msgTitle, wxOK | wxICON_EXCLAMATION,
                  parent);
  
              (void)wxMessageBox(_("Sorry, could not open this file."), msgTitle, wxOK | wxICON_EXCLAMATION,
                  parent);
  
-            path = wxT("");
+            path = wxEmptyString;
              return (wxDocTemplate *) NULL;
          }
          m_lastDirectory = wxPathOnly(pathTmp);
              return (wxDocTemplate *) NULL;
          }
          m_lastDirectory = wxPathOnly(pathTmp);
@@ -1472,85 +1621,72 @@ wxDocTemplate *wxDocManager::SelectDocumentPath(wxDocTemplate **templates,
              theTemplate = templates[FilterIndex];
          if ( !theTemplate )
              theTemplate = FindTemplateForPath(path);
              theTemplate = templates[FilterIndex];
          if ( !theTemplate )
              theTemplate = FindTemplateForPath(path);
+        if ( !theTemplate )
+        {
+            // Since we do not add files with non-default extensions to the FileHistory this
+            // can only happen if the application changes the allowed templates in runtime.
+            (void)wxMessageBox(_("Sorry, the format for this file is unknown."),
+                                _("Open File"),
+                                wxOK | wxICON_EXCLAMATION, wxFindSuitableParent());
+        }
      }
      else
      {
      }
      else
      {
-        path = wxT("");
+        path = wxEmptyString;
      }
  
      return theTemplate;
      }
  
      return theTemplate;
-
-#if 0
-    // In all other windowing systems, until we have more advanced
-    // file selectors, we must select the document type (template) first, and
-    // _then_ pop up the file selector.
-    wxDocTemplate *temp = SelectDocumentType(templates, noTemplates);
-    if (!temp)
-        return (wxDocTemplate *) NULL;
-
-    wxChar *pathTmp = wxFileSelector(_("Select a file"), wxT(""), wxT(""),
-            temp->GetDefaultExtension(),
-            temp->GetFileFilter(),
-            0, wxTheApp->GetTopWindow());
-
-    if (pathTmp)
-    {
-        path = pathTmp;
-        return temp;
-    }
-    else
-        return (wxDocTemplate *) NULL;
-#endif // 0
  }
  
  wxDocTemplate *wxDocManager::SelectDocumentType(wxDocTemplate **templates,
                                                  int noTemplates, bool sort)
  {
  }
  
  wxDocTemplate *wxDocManager::SelectDocumentType(wxDocTemplate **templates,
                                                  int noTemplates, bool sort)
  {
-    wxArrayString strings(sort);
+    wxArrayString strings;
      wxDocTemplate **data = new wxDocTemplate *[noTemplates];
      int i;
      int n = 0;
      wxDocTemplate **data = new wxDocTemplate *[noTemplates];
      int i;
      int n = 0;
-        
-       for (i = 0; i < noTemplates; i++)
-       {
-               if (templates[i]->IsVisible())
-               {
-               int j;
-            bool want = TRUE;
-                       for (j = 0; j < n; j++)
-                       {
+
+    for (i = 0; i < noTemplates; i++)
+    {
+        if (templates[i]->IsVisible())
+        {
+            int j;
+            bool want = true;
+            for (j = 0; j < n; j++)
+            {
                  //filter out NOT unique documents + view combinations
                  //filter out NOT unique documents + view combinations
-                               if ( templates[i]->m_docTypeName == data[j]->m_docTypeName &&
+                if ( templates[i]->m_docTypeName == data[j]->m_docTypeName &&
                       templates[i]->m_viewTypeName == data[j]->m_viewTypeName
                     )
                       templates[i]->m_viewTypeName == data[j]->m_viewTypeName
                     )
-                    want = FALSE;
-                       }
+                    want = false;
+            }
  
              if ( want )
  
              if ( want )
-                       {
-                       strings.Add(templates[i]->m_description);
-
-                               data[n] = templates[i];
-                               n ++;
-                       }
-               }
-       }  // for
-
-       if (sort)
-       {
-               // Yes, this will be slow, but template lists
-               // are typically short.
-               int j;
-               n = strings.Count();
-               for (i = 0; i < n; i++)
-               {
-                       for (j = 0; j < noTemplates; j++)
-                       {
-                               if (strings[i] == templates[j]->m_description)
-                                       data[i] = templates[j];
-                       }
-               }
-       }
+            {
+                strings.Add(templates[i]->m_description);
+
+                data[n] = templates[i];
+                n ++;
+            }
+        }
+    }  // for
+
+    if (sort)
+    {
+        strings.Sort(); // ascending sort
+        // Yes, this will be slow, but template lists
+        // are typically short.
+        int j;
+        n = strings.Count();
+        for (i = 0; i < n; i++)
+        {
+            for (j = 0; j < noTemplates; j++)
+            {
+                if (strings[i] == templates[j]->m_description)
+                    data[i] = templates[j];
+            }
+        }
+    }
  
      wxDocTemplate *theTemplate;
  
  
      wxDocTemplate *theTemplate;
  
@@ -1586,49 +1722,50 @@ wxDocTemplate *wxDocManager::SelectDocumentType(wxDocTemplate **templates,
  wxDocTemplate *wxDocManager::SelectViewType(wxDocTemplate **templates,
                                              int noTemplates, bool sort)
  {
  wxDocTemplate *wxDocManager::SelectViewType(wxDocTemplate **templates,
                                              int noTemplates, bool sort)
  {
-    wxArrayString strings(sort);
+    wxArrayString strings;
      wxDocTemplate **data = new wxDocTemplate *[noTemplates];
      int i;
      int n = 0;
      wxDocTemplate **data = new wxDocTemplate *[noTemplates];
      int i;
      int n = 0;
-        
+
      for (i = 0; i < noTemplates; i++)
      {
          wxDocTemplate *templ = templates[i];
          if ( templ->IsVisible() && !templ->GetViewName().empty() )
          {
      for (i = 0; i < noTemplates; i++)
      {
          wxDocTemplate *templ = templates[i];
          if ( templ->IsVisible() && !templ->GetViewName().empty() )
          {
-               int j;
-            bool want = TRUE;
-                       for (j = 0; j < n; j++)
-                       {
+            int j;
+            bool want = true;
+            for (j = 0; j < n; j++)
+            {
                  //filter out NOT unique views
                  //filter out NOT unique views
-                               if ( templates[i]->m_viewTypeName == data[j]->m_viewTypeName )
-                    want = FALSE;
-                       }
+                if ( templates[i]->m_viewTypeName == data[j]->m_viewTypeName )
+                    want = false;
+            }
  
              if ( want )
              {
  
              if ( want )
              {
-                       strings.Add(templ->m_viewTypeName);
-                               data[n] = templ;
-                               n ++;
-                       }
+                strings.Add(templ->m_viewTypeName);
+                data[n] = templ;
+                n ++;
+            }
          }
      }
  
          }
      }
  
-       if (sort)
-       {
-               // Yes, this will be slow, but template lists
-               // are typically short.
-               int j;
-               n = strings.Count();
-               for (i = 0; i < n; i++)
-               {
-                       for (j = 0; j < noTemplates; j++)
-                       {
-                               if (strings[i] == templates[j]->m_viewTypeName)
-                                       data[i] = templates[j];
-                       }
-               }
-       }
+    if (sort)
+    {
+        strings.Sort(); // ascending sort
+        // Yes, this will be slow, but template lists
+        // are typically short.
+        int j;
+        n = strings.Count();
+        for (i = 0; i < n; i++)
+        {
+            for (j = 0; j < noTemplates; j++)
+            {
+                if (strings[i] == templates[j]->m_viewTypeName)
+                    data[i] = templates[j];
+            }
+        }
+    }
  
      wxDocTemplate *theTemplate;
  
  
      wxDocTemplate *theTemplate;
  
@@ -1684,25 +1821,19 @@ void wxDocManager::RemoveDocument(wxDocument *doc)
  
  // Views or windows should inform the document manager
  // when a view is going in or out of focus
  
  // Views or windows should inform the document manager
  // when a view is going in or out of focus
-void wxDocManager::ActivateView(wxView *view, bool activate, bool WXUNUSED(deleting))
-{
-    // If we're deactiving, and if we're not actually deleting the view, then
-    // don't reset the current view because we may be going to
-    // a window without a view.
-    // WHAT DID I MEAN BY THAT EXACTLY?
-    /*
-       if (deleting)
-       {
-       if (m_currentView == view)
-       m_currentView = NULL;
-       }
-       else
-     */
-    {
-        if (activate)
-            m_currentView = view;
-        else
+void wxDocManager::ActivateView(wxView *view, bool activate)
+{
+    if ( activate )
+    {
+        m_currentView = view;
+    }
+    else // deactivate
+    {
+        if ( m_currentView == view )
+        {
+            // don't keep stale pointer
              m_currentView = (wxView *) NULL;
              m_currentView = (wxView *) NULL;
+        }
      }
  }
  
      }
  }
  
@@ -1732,15 +1863,11 @@ wxDocChildFrame::wxDocChildFrame(wxDocument *doc,
          view->SetFrame(this);
  }
  
          view->SetFrame(this);
  }
  
-wxDocChildFrame::~wxDocChildFrame()
-{
-}
-
  // Extend event processing to search the view's event table
  bool wxDocChildFrame::ProcessEvent(wxEvent& event)
  {
      if (m_childView)
  // Extend event processing to search the view's event table
  bool wxDocChildFrame::ProcessEvent(wxEvent& event)
  {
      if (m_childView)
-        m_childView->Activate(TRUE);
+        m_childView->Activate(true);
  
      if ( !m_childView || ! m_childView->ProcessEvent(event) )
      {
  
      if ( !m_childView || ! m_childView->ProcessEvent(event) )
      {
@@ -1748,10 +1875,10 @@ bool wxDocChildFrame::ProcessEvent(wxEvent& event)
          if (!event.IsKindOf(CLASSINFO(wxCommandEvent)) || !GetParent() || !GetParent()->ProcessEvent(event))
              return wxEvtHandler::ProcessEvent(event);
          else
          if (!event.IsKindOf(CLASSINFO(wxCommandEvent)) || !GetParent() || !GetParent()->ProcessEvent(event))
              return wxEvtHandler::ProcessEvent(event);
          else
-            return TRUE;
+            return true;
      }
      else
      }
      else
-        return TRUE;
+        return true;
  }
  
  void wxDocChildFrame::OnActivate(wxActivateEvent& event)
  }
  
  void wxDocChildFrame::OnActivate(wxActivateEvent& event)
@@ -1766,15 +1893,13 @@ void wxDocChildFrame::OnCloseWindow(wxCloseEvent& event)
  {
      if (m_childView)
      {
  {
      if (m_childView)
      {
-        bool ans = FALSE;
-        if (!event.CanVeto())
-            ans = TRUE; // Must delete.
-        else
-            ans = m_childView->Close(FALSE); // FALSE means don't delete associated window
+        bool ans = event.CanVeto()
+                    ? m_childView->Close(false) // false means don't delete associated window
+                    : true; // Must delete.
  
          if (ans)
          {
  
          if (ans)
          {
-            m_childView->Activate(FALSE);
+            m_childView->Activate(false);
              delete m_childView;
              m_childView = (wxView *) NULL;
              m_childDocument = (wxDocument *) NULL;
              delete m_childView;
              m_childView = (wxView *) NULL;
              m_childDocument = (wxDocument *) NULL;
@@ -1798,6 +1923,11 @@ BEGIN_EVENT_TABLE(wxDocParentFrame, wxFrame)
      EVT_CLOSE(wxDocParentFrame::OnCloseWindow)
  END_EVENT_TABLE()
  
      EVT_CLOSE(wxDocParentFrame::OnCloseWindow)
  END_EVENT_TABLE()
  
+wxDocParentFrame::wxDocParentFrame()
+{
+    m_docManager = NULL;
+}
+
  wxDocParentFrame::wxDocParentFrame(wxDocManager *manager,
                                     wxFrame *frame,
                                     wxWindowID id,
  wxDocParentFrame::wxDocParentFrame(wxDocManager *manager,
                                     wxFrame *frame,
                                     wxWindowID id,
@@ -1811,6 +1941,19 @@ wxDocParentFrame::wxDocParentFrame(wxDocManager *manager,
      m_docManager = manager;
  }
  
      m_docManager = manager;
  }
  
+bool wxDocParentFrame::Create(wxDocManager *manager,
+                              wxFrame *frame,
+                              wxWindowID id,
+                              const wxString& title,
+                              const wxPoint& pos,
+                              const wxSize& size,
+                              long style,
+                              const wxString& name)
+{
+    m_docManager = manager;
+    return base_type::Create(frame, id, title, pos, size, style, name);
+}
+
  void wxDocParentFrame::OnExit(wxCommandEvent& WXUNUSED(event))
  {
      Close();
  void wxDocParentFrame::OnExit(wxCommandEvent& WXUNUSED(event))
  {
      Close();
@@ -1820,13 +1963,20 @@ void wxDocParentFrame::OnMRUFile(wxCommandEvent& event)
  {
      int n = event.GetId() - wxID_FILE1;  // the index in MRU list
      wxString filename(m_docManager->GetHistoryFile(n));
  {
      int n = event.GetId() - wxID_FILE1;  // the index in MRU list
      wxString filename(m_docManager->GetHistoryFile(n));
-    if ( !filename.IsEmpty() )
+    if ( !filename.empty() )
      {
          // verify that the file exists before doing anything else
          if ( wxFile::Exists(filename) )
          {
              // try to open it
      {
          // verify that the file exists before doing anything else
          if ( wxFile::Exists(filename) )
          {
              // try to open it
-            (void)m_docManager->CreateDocument(filename, wxDOC_SILENT);
+            if (!m_docManager->CreateDocument(filename, wxDOC_SILENT))
+            {
+                // remove the file from the MRU list. The user should already be notified.
+                m_docManager->RemoveFileFromHistory(n);
+
+                wxLogError(_("The file '%s' couldn't be opened.\nIt has been removed from the most recently used files list."),
+                       filename.c_str());
+            }
          }
          else
          {
          }
          else
          {
@@ -1847,7 +1997,7 @@ bool wxDocParentFrame::ProcessEvent(wxEvent& event)
      if (!m_docManager || !m_docManager->ProcessEvent(event))
          return wxEvtHandler::ProcessEvent(event);
      else
      if (!m_docManager || !m_docManager->ProcessEvent(event))
          return wxEvtHandler::ProcessEvent(event);
      else
-        return TRUE;
+        return true;
  }
  
  // Define the behaviour for the frame closing
  }
  
  // Define the behaviour for the frame closing
@@ -1877,8 +2027,10 @@ bool wxDocPrintout::OnPrintPage(int WXUNUSED(page))
      // Get the logical pixels per inch of screen and printer
      int ppiScreenX, ppiScreenY;
      GetPPIScreen(&ppiScreenX, &ppiScreenY);
      // Get the logical pixels per inch of screen and printer
      int ppiScreenX, ppiScreenY;
      GetPPIScreen(&ppiScreenX, &ppiScreenY);
+    wxUnusedVar(ppiScreenY);
      int ppiPrinterX, ppiPrinterY;
      GetPPIPrinter(&ppiPrinterX, &ppiPrinterY);
      int ppiPrinterX, ppiPrinterY;
      GetPPIPrinter(&ppiPrinterX, &ppiPrinterY);
+    wxUnusedVar(ppiPrinterY);
  
      // This scales the DC so that the printout roughly represents the
      // the screen scaling. The text point size _should_ be the right size
  
      // This scales the DC so that the printout roughly represents the
      // the screen scaling. The text point size _should_ be the right size
@@ -1893,6 +2045,7 @@ bool wxDocPrintout::OnPrintPage(int WXUNUSED(page))
      int w, h;
      dc->GetSize(&w, &h);
      GetPageSizePixels(&pageWidth, &pageHeight);
      int w, h;
      dc->GetSize(&w, &h);
      GetPageSizePixels(&pageWidth, &pageHeight);
+    wxUnusedVar(pageHeight);
  
      // If printer pageWidth == current DC width, then this doesn't
      // change. But w might be the preview bitmap width, so scale down.
  
      // If printer pageWidth == current DC width, then this doesn't
      // change. But w might be the preview bitmap width, so scale down.
@@ -1903,7 +2056,7 @@ bool wxDocPrintout::OnPrintPage(int WXUNUSED(page))
      {
          m_printoutView->OnDraw(dc);
      }
      {
          m_printoutView->OnDraw(dc);
      }
-    return TRUE;
+    return true;
  }
  
  bool wxDocPrintout::HasPage(int pageNum)
  }
  
  bool wxDocPrintout::HasPage(int pageNum)
@@ -1914,9 +2067,9 @@ bool wxDocPrintout::HasPage(int pageNum)
  bool wxDocPrintout::OnBeginDocument(int startPage, int endPage)
  {
      if (!wxPrintout::OnBeginDocument(startPage, endPage))
  bool wxDocPrintout::OnBeginDocument(int startPage, int endPage)
  {
      if (!wxPrintout::OnBeginDocument(startPage, endPage))
-        return FALSE;
+        return false;
  
  
-    return TRUE;
+    return true;
  }
  
  void wxDocPrintout::GetPageInfo(int *minPage, int *maxPage, int *selPageFrom, int *selPageTo)
  }
  
  void wxDocPrintout::GetPageInfo(int *minPage, int *maxPage, int *selPageFrom, int *selPageTo)
@@ -1933,6 +2086,18 @@ void wxDocPrintout::GetPageInfo(int *minPage, int *maxPage, int *selPageFrom, in
  // File history processor
  // ----------------------------------------------------------------------------
  
  // File history processor
  // ----------------------------------------------------------------------------
  
+static inline wxChar* MYcopystring(const wxString& s)
+{
+    wxChar* copy = new wxChar[s.length() + 1];
+    return wxStrcpy(copy, s.c_str());
+}
+
+static inline wxChar* MYcopystring(const wxChar* s)
+{
+    wxChar* copy = new wxChar[wxStrlen(s) + 1];
+    return wxStrcpy(copy, s);
+}
+
  wxFileHistory::wxFileHistory(size_t maxFiles, wxWindowID idBase)
  {
      m_fileMaxFiles = maxFiles;
  wxFileHistory::wxFileHistory(size_t maxFiles, wxWindowID idBase)
  {
      m_fileMaxFiles = maxFiles;
@@ -1957,7 +2122,14 @@ void wxFileHistory::AddFileToHistory(const wxString& file)
      // Check we don't already have this file
      for (i = 0; i < m_fileHistoryN; i++)
      {
      // Check we don't already have this file
      for (i = 0; i < m_fileHistoryN; i++)
      {
-        if ( m_fileHistory[i] && (file == m_fileHistory[i]) )
+#if defined( __WXMSW__ ) // Add any other OSes with case insensitive file names
+        wxString testString;
+        if ( m_fileHistory[i] )
+            testString = m_fileHistory[i];
+        if ( m_fileHistory[i] && ( file.Lower() == testString.Lower() ) )
+#else
+        if ( m_fileHistory[i] && ( file == m_fileHistory[i] ) )
+#endif
          {
              // we do have it, move it to the top of the history
              RemoveFileFromHistory (i);
          {
              // we do have it, move it to the top of the history
              RemoveFileFromHistory (i);
@@ -1978,7 +2150,7 @@ void wxFileHistory::AddFileToHistory(const wxString& file)
      // Move existing files (if any) down so we can insert file at beginning.
      if (m_fileHistoryN < m_fileMaxFiles)
      {
      // Move existing files (if any) down so we can insert file at beginning.
      if (m_fileHistoryN < m_fileMaxFiles)
      {
-        wxNode* node = m_fileMenus.GetFirst();
+        wxList::compatibility_iterator node = m_fileMenus.GetFirst();
          while (node)
          {
              wxMenu* menu = (wxMenu*) node->GetData();
          while (node)
          {
              wxMenu* menu = (wxMenu*) node->GetData();
@@ -1996,7 +2168,7 @@ void wxFileHistory::AddFileToHistory(const wxString& file)
      {
          m_fileHistory[i] = m_fileHistory[i-1];
      }
      {
          m_fileHistory[i] = m_fileHistory[i-1];
      }
-    m_fileHistory[0] = copystring(file);
+    m_fileHistory[0] = MYcopystring(file);
  
      // this is the directory of the last opened file
      wxString pathCurrent;
  
      // this is the directory of the last opened file
      wxString pathCurrent;
@@ -2021,9 +2193,11 @@ void wxFileHistory::AddFileToHistory(const wxString& file)
                  pathInMenu = m_fileHistory[i];
              }
  
                  pathInMenu = m_fileHistory[i];
              }
  
+            // we need to quote '&' characters which are used for mnemonics
+            pathInMenu.Replace(_T("&"), _T("&&"));
              wxString buf;
              buf.Printf(s_MRUEntryFormat, i + 1, pathInMenu.c_str());
              wxString buf;
              buf.Printf(s_MRUEntryFormat, i + 1, pathInMenu.c_str());
-            wxNode* node = m_fileMenus.GetFirst();
+            wxList::compatibility_iterator node = m_fileMenus.GetFirst();
              while (node)
              {
                  wxMenu* menu = (wxMenu*) node->GetData();
              while (node)
              {
                  wxMenu* menu = (wxMenu*) node->GetData();
@@ -2048,24 +2222,23 @@ void wxFileHistory::RemoveFileFromHistory(size_t i)
          m_fileHistory[j] = m_fileHistory[j + 1];
      }
  
          m_fileHistory[j] = m_fileHistory[j + 1];
      }
  
-    wxNode* node = m_fileMenus.GetFirst();
+    wxList::compatibility_iterator node = m_fileMenus.GetFirst();
      while ( node )
      {
      while ( node )
      {
-         wxMenu* menu = (wxMenu*) node->GetData();
-
+        wxMenu* menu = (wxMenu*) node->GetData();
  
  
-         // shuffle filenames up
-         wxString buf;
-         for ( j = i; j < m_fileHistoryN - 1; j++ )
-         {
-             buf.Printf(s_MRUEntryFormat, j + 1, m_fileHistory[j]);
-             menu->SetLabel(m_idBase + j, buf);
-         }
+        // shuffle filenames up
+        wxString buf;
+        for ( j = i; j < m_fileHistoryN - 1; j++ )
+        {
+            buf.Printf(s_MRUEntryFormat, j + 1, m_fileHistory[j]);
+            menu->SetLabel(m_idBase + j, buf);
+        }
  
  
-         node = node->GetNext();
+        node = node->GetNext();
  
          // delete the last menu item which is unused now
  
          // delete the last menu item which is unused now
-        wxWindowID lastItemId = m_idBase + m_fileHistoryN - 1;
+        wxWindowID lastItemId = m_idBase + wx_truncate_cast(wxWindowID, m_fileHistoryN) - 1;
          if (menu->FindItem(lastItemId))
          {
              menu->Delete(lastItemId);
          if (menu->FindItem(lastItemId))
          {
              menu->Delete(lastItemId);
@@ -2074,10 +2247,10 @@ void wxFileHistory::RemoveFileFromHistory(size_t i)
          // delete the last separator too if no more files are left
          if ( m_fileHistoryN == 1 )
          {
          // delete the last separator too if no more files are left
          if ( m_fileHistoryN == 1 )
          {
-            wxMenuItemList::Node *node = menu->GetMenuItems().GetLast();
-            if ( node )
+            wxMenuItemList::compatibility_iterator nodeLast = menu->GetMenuItems().GetLast();
+            if ( nodeLast )
              {
              {
-                wxMenuItem *menuItem = node->GetData();
+                wxMenuItem *menuItem = nodeLast->GetData();
                  if ( menuItem->IsSeparator() )
                  {
                      menu->Delete(menuItem);
                  if ( menuItem->IsSeparator() )
                  {
                      menu->Delete(menuItem);
@@ -2124,12 +2297,12 @@ void wxFileHistory::Load(wxConfigBase& config)
      wxString buf;
      buf.Printf(wxT("file%d"), (int)m_fileHistoryN+1);
      wxString historyFile;
      wxString buf;
      buf.Printf(wxT("file%d"), (int)m_fileHistoryN+1);
      wxString historyFile;
-    while ((m_fileHistoryN < m_fileMaxFiles) && config.Read(buf, &historyFile) && (historyFile != wxT("")))
+    while ((m_fileHistoryN < m_fileMaxFiles) && config.Read(buf, &historyFile) && (!historyFile.empty()))
      {
      {
-        m_fileHistory[m_fileHistoryN] = copystring((const wxChar*) historyFile);
+        m_fileHistory[m_fileHistoryN] = MYcopystring((const wxChar*) historyFile);
          m_fileHistoryN ++;
          buf.Printf(wxT("file%d"), (int)m_fileHistoryN+1);
          m_fileHistoryN ++;
          buf.Printf(wxT("file%d"), (int)m_fileHistoryN+1);
-        historyFile = wxT("");
+        historyFile = wxEmptyString;
      }
      AddFilesToMenu();
  }
      }
      AddFilesToMenu();
  }
@@ -2137,11 +2310,14 @@ void wxFileHistory::Load(wxConfigBase& config)
  void wxFileHistory::Save(wxConfigBase& config)
  {
      size_t i;
  void wxFileHistory::Save(wxConfigBase& config)
  {
      size_t i;
-    for (i = 0; i < m_fileHistoryN; i++)
+    for (i = 0; i < m_fileMaxFiles; i++)
      {
          wxString buf;
          buf.Printf(wxT("file%d"), (int)i+1);
      {
          wxString buf;
          buf.Printf(wxT("file%d"), (int)i+1);
-        config.Write(buf, wxString(m_fileHistory[i]));
+        if (i < m_fileHistoryN)
+            config.Write(buf, wxString(m_fileHistory[i]));
+        else
+            config.Write(buf, wxEmptyString);
      }
  }
  #endif // wxUSE_CONFIG
      }
  }
  #endif // wxUSE_CONFIG
@@ -2150,7 +2326,7 @@ void wxFileHistory::AddFilesToMenu()
  {
      if (m_fileHistoryN > 0)
      {
  {
      if (m_fileHistoryN > 0)
      {
-        wxNode* node = m_fileMenus.GetFirst();
+        wxList::compatibility_iterator node = m_fileMenus.GetFirst();
          while (node)
          {
              wxMenu* menu = (wxMenu*) node->GetData();
          while (node)
          {
              wxMenu* menu = (wxMenu*) node->GetData();
@@ -2207,7 +2383,7 @@ bool wxTransferFileToStream(const wxString& filename, wxSTD ostream& stream)
  {
      wxFFile file(filename, _T("rb"));
      if ( !file.IsOpened() )
  {
      wxFFile file(filename, _T("rb"));
      if ( !file.IsOpened() )
-        return FALSE;
+        return false;
  
      char buf[4096];
  
  
      char buf[4096];
  
@@ -2216,22 +2392,22 @@ bool wxTransferFileToStream(const wxString& filename, wxSTD ostream& stream)
      {
          nRead = file.Read(buf, WXSIZEOF(buf));
          if ( file.Error() )
      {
          nRead = file.Read(buf, WXSIZEOF(buf));
          if ( file.Error() )
-            return FALSE;
+            return false;
  
          stream.write(buf, nRead);
          if ( !stream )
  
          stream.write(buf, nRead);
          if ( !stream )
-            return FALSE;
+            return false;
      }
      while ( !file.Eof() );
  
      }
      while ( !file.Eof() );
  
-    return TRUE;
+    return true;
  }
  
  bool wxTransferStreamToFile(wxSTD istream& stream, const wxString& filename)
  {
      wxFFile file(filename, _T("wb"));
      if ( !file.IsOpened() )
  }
  
  bool wxTransferStreamToFile(wxSTD istream& stream, const wxString& filename)
  {
      wxFFile file(filename, _T("wb"));
      if ( !file.IsOpened() )
-        return FALSE;
+        return false;
  
      char buf[4096];
      do
  
      char buf[4096];
      do
@@ -2240,12 +2416,12 @@ bool wxTransferStreamToFile(wxSTD istream& stream, const wxString& filename)
          if ( !stream.bad() ) // fail may be set on EOF, don't use operator!()
          {
              if ( !file.Write(buf, stream.gcount()) )
          if ( !stream.bad() ) // fail may be set on EOF, don't use operator!()
          {
              if ( !file.Write(buf, stream.gcount()) )
-                return FALSE;
+                return false;
          }
      }
      while ( !stream.eof() );
  
          }
      }
      while ( !stream.eof() );
  
-    return TRUE;
+    return true;
  }
  
  #else // !wxUSE_STD_IOSTREAM
  }
  
  #else // !wxUSE_STD_IOSTREAM
@@ -2254,7 +2430,7 @@ bool wxTransferFileToStream(const wxString& filename, wxOutputStream& stream)
  {
      wxFFile file(filename, _T("rb"));
      if ( !file.IsOpened() )
  {
      wxFFile file(filename, _T("rb"));
      if ( !file.IsOpened() )
-        return FALSE;
+        return false;
  
      char buf[4096];
  
  
      char buf[4096];
  
@@ -2263,22 +2439,22 @@ bool wxTransferFileToStream(const wxString& filename, wxOutputStream& stream)
      {
          nRead = file.Read(buf, WXSIZEOF(buf));
          if ( file.Error() )
      {
          nRead = file.Read(buf, WXSIZEOF(buf));
          if ( file.Error() )
-            return FALSE;
+            return false;
  
          stream.Write(buf, nRead);
          if ( !stream )
  
          stream.Write(buf, nRead);
          if ( !stream )
-            return FALSE;
+            return false;
      }
      while ( !file.Eof() );
  
      }
      while ( !file.Eof() );
  
-    return TRUE;
+    return true;
  }
  
  bool wxTransferStreamToFile(wxInputStream& stream, const wxString& filename)
  {
      wxFFile file(filename, _T("wb"));
      if ( !file.IsOpened() )
  }
  
  bool wxTransferStreamToFile(wxInputStream& stream, const wxString& filename)
  {
      wxFFile file(filename, _T("wb"));
      if ( !file.IsOpened() )
-        return FALSE;
+        return false;
  
      char buf[4096];
      do
  
      char buf[4096];
      do
@@ -2287,14 +2463,13 @@ bool wxTransferStreamToFile(wxInputStream& stream, const wxString& filename)
  
          const size_t nRead = stream.LastRead();
          if ( !nRead || !file.Write(buf, nRead) )
  
          const size_t nRead = stream.LastRead();
          if ( !nRead || !file.Write(buf, nRead) )
-            return FALSE;
+            return false;
      }
      while ( !stream.Eof() );
  
      }
      while ( !stream.Eof() );
  
-    return TRUE;
+    return true;
  }
  
  #endif // wxUSE_STD_IOSTREAM/!wxUSE_STD_IOSTREAM
  
  #endif // wxUSE_DOC_VIEW_ARCHITECTURE
  }
  
  #endif // wxUSE_STD_IOSTREAM/!wxUSE_STD_IOSTREAM
  
  #endif // wxUSE_DOC_VIEW_ARCHITECTURE
-