don't cache the result of IsAlwaysConnected() and don't call IsOnline() unnecessarily...

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

index c0a56d98918b500ccffc20c391b52d657c8d722b..0482dc49cd0c869e8b460d88a53d093028a78d5d 100644 (file)
--- a/src/html/htmlfilt.cpp
+++ b/src/html/htmlfilt.cpp
@@ -9,24 +9,55 @@
  
  
  #ifdef __GNUG__
-#pragma implementation
+#pragma implementation "htmlfilt.h"
  #endif
  
  #include "wx/wxprec.h"
  
-#if wxUSE_HTML
+#include "wx/defs.h"
+#if wxUSE_HTML && wxUSE_STREAMS
  
  #ifdef __BORDLANDC__
  #pragma hdrstop
  #endif
  
  #ifndef WXPRECOMP
-#include "wx/wx.h"
+    #include "wx/log.h"
+    #include "wx/intl.h"
  #endif
  
  #include "wx/html/htmlfilt.h"
  #include "wx/html/htmlwin.h"
  
+// utility function: read a wxString from a wxInputStream
+void wxPrivate_ReadString(wxString& str, wxInputStream* s)
+{
+    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(buffer);
+        }
+        while(lastRead == bufSize);
+    }
+    else
+    {
+        char* src = new char[streamSize+1];
+        s->Read(src, streamSize);
+        src[streamSize] = 0;
+        str = src;
+        delete [] src;
+    }
+}
  
  /*
  
@@ -53,18 +84,14 @@ bool wxHtmlFilterPlainText::CanRead(const wxFSFile& WXUNUSED(file)) const
  wxString wxHtmlFilterPlainText::ReadFile(const wxFSFile& file) const
  {
      wxInputStream *s = file.GetStream();
-    char *src;
      wxString doc, doc2;
  
      if (s == NULL) return wxEmptyString;
-    src = new char[s -> GetSize()+1];
-    src[s -> GetSize()] = 0;
-    s -> Read(src, s -> GetSize());
-    doc = src;
-    delete [] src;
-
-    doc.Replace("<", "&lt;", TRUE);
-    doc.Replace(">", "&gt;", TRUE);
+    wxPrivate_ReadString(doc, s);
+
+    doc.Replace(wxT("&"), wxT("&amp;"), TRUE);
+    doc.Replace(wxT("<"), wxT("&lt;"), TRUE);
+    doc.Replace(wxT(">"), wxT("&gt;"), TRUE);
      doc2 = "<HTML><BODY><PRE>\n" + doc + "\n</PRE></BODY></HTML>";
      return doc2;
  }
@@ -93,7 +120,7 @@ IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterImage, wxHtmlFilter)
  
  bool wxHtmlFilterImage::CanRead(const wxFSFile& file) const
  {
-    return (file.GetMimeType().Left(6) == "image/");
+    return (file.GetMimeType().Left(6) == wxT("image/"));
  }
  
  
@@ -129,7 +156,7 @@ bool wxHtmlFilterHTML::CanRead(const wxFSFile& file) const
  // This is true in most case but some page can return:
  // "text/html; char-encoding=...."
  // So we use Find instead
-  return (file.GetMimeType().Find(T("text/html")) == 0);
+  return (file.GetMimeType().Find(wxT("text/html")) == 0);
  }
  
  
@@ -137,15 +164,22 @@ bool wxHtmlFilterHTML::CanRead(const wxFSFile& file) const
  wxString wxHtmlFilterHTML::ReadFile(const wxFSFile& file) const
  {
      wxInputStream *s = file.GetStream();
-    char *src;
      wxString doc;
  
-    if (s == NULL) return wxEmptyString;
-    src = new char[s -> GetSize() + 1];
-    src[s -> GetSize()] = 0;
-    s -> Read(src, s -> GetSize());
-    doc = src;
-    delete[] src;
+    if (s == NULL)
+    {
+        wxLogError(_("Cannot open HTML document: %s"), file.GetLocation().c_str());
+        return wxEmptyString;
+    }
+    wxPrivate_ReadString(doc, s);
+
+    // add meta tag if we obtained this through http:
+    if (file.GetMimeType().Find(_T("; charset=")) == 0)
+    {
+        wxString s(_T("<meta http-equiv=\"Content-Type\" content=\""));
+        s << file.GetMimeType() <<  _T("\">");
+        return s+doc;
+    }
  
      return doc;
  }