reset the tooltip text before changing it, this apparently prevents a spurious redraw...

[wxWidgets.git] / src / html / htmlfilt.cpp
diff --git a/src/html/htmlfilt.cpp b/src/html/htmlfilt.cpp

index 85cc8b74b23ee885646ea43f343b8989d9d4322c..9aaa86d4ebadd4146ff018785f214fbadf66012c 100644 (file)
--- a/src/html/htmlfilt.cpp
+++ b/src/html/htmlfilt.cpp
@@ -1,63 +1,37 @@
  /////////////////////////////////////////////////////////////////////////////
-// Name:        htmlfilt.cpp
+// Name:        src/html/htmlfilt.cpp
  // Purpose:     wxHtmlFilter - input filter for translating into HTML format
  // Author:      Vaclav Slavik
  // RCS-ID:      $Id$
  // Copyright:   (c) 1999 Vaclav Slavik
-// Licence:     wxWindows Licence
+// Licence:     wxWindows licence
  /////////////////////////////////////////////////////////////////////////////
  
-
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#pragma implementation "htmlfilt.h"
-#endif
-
  #include "wx/wxprec.h"
  
-#include "wx/defs.h"
-#if wxUSE_HTML && wxUSE_STREAMS
-
  #ifdef __BORLANDC__
-#pragma hdrstop
+    #pragma hdrstop
  #endif
  
-#ifndef WXPRECOMP
+#if wxUSE_HTML && wxUSE_STREAMS
+
+#ifndef WX_PRECOMP
      #include "wx/log.h"
      #include "wx/intl.h"
  #endif
  
  #include "wx/strconv.h"
+#include "wx/sstream.h"
  #include "wx/html/htmlfilt.h"
  #include "wx/html/htmlwin.h"
  
-// utility function: read a wxString from a wxInputStream
+// utility function: read entire contents of an wxInputStream into a wxString
+//
+// TODO: error handling?
  static void ReadString(wxString& str, wxInputStream* s, wxMBConv& conv)
  {
-    size_t streamSize = s->GetSize();
-
-    if (streamSize == ~(size_t)0)
-    {
-        const size_t bufSize = 4095;
-        char buffer[bufSize+1];
-        size_t lastRead;
-
-        do
-        {
-            s->Read(buffer, bufSize);
-            lastRead = s->LastRead();
-            buffer[lastRead] = 0;
-            str.Append(wxString(buffer, conv));
-        }
-        while (lastRead == bufSize);
-    }
-    else
-    {
-        char* src = new char[streamSize+1];
-        s->Read(src, streamSize);
-        src[streamSize] = 0;
-        str = wxString(src, conv);
-        delete[] src;
-    }
+    wxStringOutputStream out(&str, conv);
+    s->Read(out);
  }
  
  /*
@@ -77,7 +51,7 @@ IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterPlainText, wxHtmlFilter)
  
  bool wxHtmlFilterPlainText::CanRead(const wxFSFile& WXUNUSED(file)) const
  {
-    return TRUE;
+    return true;
  }
  
  
@@ -90,9 +64,9 @@ wxString wxHtmlFilterPlainText::ReadFile(const wxFSFile& file) const
      if (s == NULL) return wxEmptyString;
      ReadString(doc, s, wxConvISO8859_1);
  
-    doc.Replace(wxT("&"), wxT("&amp;"), TRUE);
-    doc.Replace(wxT("<"), wxT("&lt;"), TRUE);
-    doc.Replace(wxT(">"), wxT("&gt;"), TRUE);
+    doc.Replace(wxT("&"), wxT("&amp;"), true);
+    doc.Replace(wxT("<"), wxT("&lt;"), true);
+    doc.Replace(wxT(">"), wxT("&gt;"), true);
      doc2 = wxT("<HTML><BODY><PRE>\n") + doc + wxT("\n</PRE></BODY></HTML>");
      return doc2;
  }
@@ -165,12 +139,12 @@ wxString wxHtmlFilterHTML::ReadFile(const wxFSFile& file) const
          return wxEmptyString;
      }
  
-    // NB: We convert input file to wchar_t here in Unicode mode, based on 
+    // NB: We convert input file to wchar_t here in Unicode mode, based on
      //     either Content-Type header or <meta> tags. In ANSI mode, we don't
      //     do it as it is done by wxHtmlParser (for this reason, we add <meta>
      //     tag if we used Content-Type header).
  #if wxUSE_UNICODE
-    int charsetPos;    
+    int charsetPos;
      if ((charsetPos = file.GetMimeType().Find(_T("; charset="))) != wxNOT_FOUND)
      {
          wxString charset = file.GetMimeType().Mid(charsetPos + 10);
@@ -179,15 +153,19 @@ wxString wxHtmlFilterHTML::ReadFile(const wxFSFile& file) const
      }
      else
      {
-        wxString tmpdoc;
-        ReadString(tmpdoc, s, wxConvISO8859_1);
+        size_t size = s->GetSize();
+        wxCharBuffer buf( size+1 );
+        s->Read( buf.data(), size );
+        *(buf.data() + size) = 0;
+        wxString tmpdoc( buf, wxConvISO8859_1);
+        
          wxString charset = wxHtmlParser::ExtractCharsetInformation(tmpdoc);
          if (charset.empty())
              doc = tmpdoc;
          else
          {
              wxCSConv conv(charset);
-            doc = wxString(tmpdoc.mb_str(wxConvISO8859_1), conv);
+            doc = wxString( buf, conv );
          }
      }
  #else // !wxUSE_UNICODE
@@ -219,7 +197,7 @@ class wxHtmlFilterModule : public wxModule
          {
              wxHtmlWindow::AddFilter(new wxHtmlFilterHTML);
              wxHtmlWindow::AddFilter(new wxHtmlFilterImage);
-            return TRUE;
+            return true;
          }
          virtual void OnExit() {}
  };