another segfault fix

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

index 0f5fc11366aae060807e9cf4eea4cb7b5b0fdfd1..6cf959cf5f1bcb788e129e68d1fa204ebb2e5ecd 100644 (file)
--- a/src/html/htmlwin.cpp
+++ b/src/html/htmlwin.cpp
@@ -72,8 +72,7 @@ WX_DEFINE_LIST(wxHtmlProcessorList);
  //-----------------------------------------------------------------------------
  
  
-wxHtmlWindow::wxHtmlWindow(wxWindow *parent, wxWindowID id, const wxPoint& pos, const wxSize& size,
-                long style, const wxString& name) : wxScrolledWindow(parent, id, pos, size, style | wxVSCROLL | wxHSCROLL, name)
+void wxHtmlWindow::Init()
  {
      m_tmpMouseMoved = FALSE;
      m_tmpLastLink = NULL;
@@ -87,17 +86,28 @@ wxHtmlWindow::wxHtmlWindow(wxWindow *parent, wxWindowID id, const wxPoint& pos,
      m_Cell = NULL;
      m_Parser = new wxHtmlWinParser(this);
      m_Parser->SetFS(m_FS);
-    SetBorders(10);
      m_HistoryPos = -1;
      m_HistoryOn = TRUE;
      m_History = new wxHtmlHistoryArray;
      m_Processors = NULL;
+    m_Style = 0;
+    SetBorders(10);
+}
+
+bool wxHtmlWindow::Create(wxWindow *parent, wxWindowID id, 
+                          const wxPoint& pos, const wxSize& size,
+                          long style, const wxString& name) 
+{
+    if (!wxScrolledWindow::Create(parent, id, pos, size, 
+                                  style | wxVSCROLL | wxHSCROLL, name))
+        return FALSE;
+
      m_Style = style;
      SetPage(wxT("<html><body></body></html>"));
+    return TRUE;
  }
  
  
-
  wxHtmlWindow::~wxHtmlWindow()
  {
      HistoryClear();
@@ -632,23 +642,18 @@ void wxHtmlWindow::OnCellMouseHover(wxHtmlCell * WXUNUSED(cell),
  
  void wxHtmlWindow::OnDraw(wxDC& dc)
  {
-    int x, y;
-    wxRegionIterator upd(GetUpdateRegion()); // get the update rect list
-    int v_y, v_h;
+    if (m_tmpCanDrawLocks > 0 || m_Cell == NULL) return;
  
-    if (m_tmpCanDrawLocks > 0) return;
+    int x, y;
+    wxRect rect = GetUpdateRegion().GetBox();
  
      dc.SetMapMode(wxMM_TEXT);
      dc.SetBackgroundMode(wxTRANSPARENT);
      GetViewStart(&x, &y);
  
-    while (upd)
-    {
-        v_y = upd.GetY();
-        v_h = upd.GetH();
-        if (m_Cell) m_Cell->Draw(dc, 0, 0, y * wxHTML_SCROLL_STEP + v_y, y * wxHTML_SCROLL_STEP + v_h + v_y);
-        upd++;
-    }
+    m_Cell->Draw(dc, 0, 0, 
+                 y * wxHTML_SCROLL_STEP + rect.GetTop(), 
+                 y * wxHTML_SCROLL_STEP + rect.GetBottom());
  }
  
  
@@ -785,4 +790,4 @@ IMPLEMENT_DYNAMIC_CLASS(wxHtmlWinModule, wxModule)
  #include "wx/html/forcelnk.h"
  FORCE_WXHTML_MODULES()
  
-#endif
-\ No newline at end of file
+#endif