Moved the cleanup code to an EVT_WINDOW_DESTROY handler.

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

diff --git a/src/html/m_layout.cpp b/src/html/m_layout.cpp
index 4fd163e68bf7f456eec4731484263ced77d659c4..e828f01e1ddc6ac0a7ede27b8782fdea18b73265 100644 (file)
--- a/src/html/m_layout.cpp
+++ b/src/html/m_layout.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        mod_layout.cpp
+// Name:        m_layout.cpp

 // Purpose:     wxHtml module for basic paragraphs/layout handling
 // Author:      Vaclav Slavik
 // RCS-ID:      $Id$
 // Purpose:     wxHtml module for basic paragraphs/layout handling
 // Author:      Vaclav Slavik
 // RCS-ID:      $Id$


@@ -14,13 +14,12 @@
 
 
 #include "wx/defs.h"
 
 
 #include "wx/defs.h"

-#if wxUSE_HTML
+#if wxUSE_HTML && wxUSE_STREAMS

 #ifdef __BORDLANDC__
 #pragma hdrstop
 #endif
 
 #ifndef WXPRECOMP
 #ifdef __BORDLANDC__
 #pragma hdrstop
 #endif
 
 #ifndef WXPRECOMP

-#include "wx/wx.h"

 #endif
 
 
 #endif
 
 


@@ -29,19 +28,20 @@
 
 #include "wx/html/htmlwin.h"
 
 
 #include "wx/html/htmlwin.h"
 

-FORCE_LINK_ME(mod_layout)
+FORCE_LINK_ME(m_layout)

 
 
 TAG_HANDLER_BEGIN(P, "P")
 
     TAG_HANDLER_PROC(tag)
     {
 
 
 TAG_HANDLER_BEGIN(P, "P")
 
     TAG_HANDLER_PROC(tag)
     {

-        if (m_WParser -> GetContainer() -> GetFirstCell() != NULL) {
-            m_WParser -> CloseContainer();
-            m_WParser -> OpenContainer();
-    }
-        m_WParser -> GetContainer() -> SetIndent(m_WParser -> GetCharHeight(), wxHTML_INDENT_TOP);
-        m_WParser -> GetContainer() -> SetAlign(tag);
+        if (m_WParser->GetContainer()->GetFirstCell() != NULL)
+        {
+            m_WParser->CloseContainer();
+            m_WParser->OpenContainer();
+        }
+        m_WParser->GetContainer()->SetIndent(m_WParser->GetCharHeight(), wxHTML_INDENT_TOP);
+        m_WParser->GetContainer()->SetAlign(tag);

         return FALSE;
     }
 
         return FALSE;
     }
 


@@ -53,13 +53,14 @@ TAG_HANDLER_BEGIN(BR, "BR")
 
     TAG_HANDLER_PROC(tag)
     {
 
     TAG_HANDLER_PROC(tag)
     {

-        int al = m_WParser -> GetContainer() -> GetAlignHor();
+        int al = m_WParser->GetContainer()->GetAlignHor();

         wxHtmlContainerCell *c;
         wxHtmlContainerCell *c;

-    
-        m_WParser -> CloseContainer();
-        c = m_WParser -> OpenContainer();
-        c -> SetAlignHor(al);
-        c -> SetAlign(tag);
+
+        m_WParser->CloseContainer();
+        c = m_WParser->OpenContainer();
+        c->SetAlignHor(al);
+        c->SetAlign(tag);
+        c->SetMinHeight(m_WParser->GetCharHeight());

         return FALSE;
     }
 
         return FALSE;
     }
 


@@ -71,27 +72,30 @@ TAG_HANDLER_BEGIN(CENTER, "CENTER")
 
     TAG_HANDLER_PROC(tag)
     {
 
     TAG_HANDLER_PROC(tag)
     {

-        int old = m_WParser -> GetAlign();
-        wxHtmlContainerCell *c = m_WParser -> GetContainer();
-
-        m_WParser -> SetAlign(wxHTML_ALIGN_CENTER);
-        if (c -> GetFirstCell() != NULL) {
-            m_WParser -> CloseContainer();
-            m_WParser -> OpenContainer();
+        int old = m_WParser->GetAlign();
+        wxHtmlContainerCell *c = m_WParser->GetContainer();
+
+        m_WParser->SetAlign(wxHTML_ALIGN_CENTER);
+        if (c->GetFirstCell() != NULL)
+        {
+            m_WParser->CloseContainer();
+            m_WParser->OpenContainer();

         }
         else
         }
         else

-            c -> SetAlignHor(wxHTML_ALIGN_CENTER);
+            c->SetAlignHor(wxHTML_ALIGN_CENTER);

 
 

-        if (tag.HasEnding()) {
+        if (tag.HasEnding())
+        {

             ParseInner(tag);
 
             ParseInner(tag);
 

-            m_WParser -> SetAlign(old);
-            if (c -> GetFirstCell() != NULL) {
-                m_WParser -> CloseContainer();
-                m_WParser -> OpenContainer();
+            m_WParser->SetAlign(old);
+            if (c->GetFirstCell() != NULL)
+            {
+                m_WParser->CloseContainer();
+                m_WParser->OpenContainer();

             }
             else
             }
             else

-                c -> SetAlignHor(old);
+                c->SetAlignHor(old);

 
             return TRUE;
         }
 
             return TRUE;
         }


@@ -106,29 +110,32 @@ TAG_HANDLER_BEGIN(DIV, "DIV")
 
     TAG_HANDLER_PROC(tag)
     {
 
     TAG_HANDLER_PROC(tag)
     {

-        int old = m_WParser -> GetAlign();
-        wxHtmlContainerCell *c = m_WParser -> GetContainer();
-        if (c -> GetFirstCell() != NULL) {
-            m_WParser -> CloseContainer();
-            m_WParser -> OpenContainer();
-            c = m_WParser -> GetContainer();
-            c -> SetAlign(tag);
-            m_WParser -> SetAlign(c -> GetAlignHor());
+        int old = m_WParser->GetAlign();
+        wxHtmlContainerCell *c = m_WParser->GetContainer();
+        if (c->GetFirstCell() != NULL)
+        {
+            m_WParser->CloseContainer();
+            m_WParser->OpenContainer();
+            c = m_WParser->GetContainer();
+            c->SetAlign(tag);
+            m_WParser->SetAlign(c->GetAlignHor());

         }
         }

-        else {
-            c -> SetAlign(tag);
-        m_WParser -> SetAlign(c -> GetAlignHor());
+        else
+        {
+            c->SetAlign(tag);
+            m_WParser->SetAlign(c->GetAlignHor());

         }
 
         ParseInner(tag);
 
         }
 
         ParseInner(tag);
 

-        m_WParser -> SetAlign(old);
-        if (c -> GetFirstCell() != NULL) {
-            m_WParser -> CloseContainer();
-            m_WParser -> OpenContainer();
+        m_WParser->SetAlign(old);
+        if (c->GetFirstCell() != NULL)
+        {
+            m_WParser->CloseContainer();
+            m_WParser->OpenContainer();

         }
         else
         }
         else

-            c -> SetAlignHor(old);
+            c->SetAlignHor(old);

 
         return TRUE;
     }
 
         return TRUE;
     }


@@ -142,14 +149,14 @@ TAG_HANDLER_BEGIN(TITLE, "TITLE")
 
     TAG_HANDLER_PROC(tag)
     {
 
     TAG_HANDLER_PROC(tag)
     {

-        if (m_WParser -> GetWindow()) {
-            wxHtmlWindow *wfr = (wxHtmlWindow*)(m_WParser -> GetWindow());
-            if (wfr) {
-                wxString title = "";
-                wxString *src = m_WParser -> GetSource();
-
-                for (int i = tag.GetBeginPos(); i < tag.GetEndPos1(); i++) title += (*src)[i];
-                wfr -> SetTitle(title);
+        if (m_WParser->GetWindow())
+        {
+            wxHtmlWindow *wfr = (wxHtmlWindow*)(m_WParser->GetWindow());
+            if (wfr)
+            {
+                const wxString& src = *m_WParser->GetSource();
+                wfr->OnSetTitle(src.Mid(tag.GetBeginPos(), 
+                                        tag.GetEndPos1()-tag.GetBeginPos()));

             }
         }
         return TRUE;
             }
         }
         return TRUE;


@@ -164,31 +171,23 @@ TAG_HANDLER_BEGIN(BODY, "BODY")
 
     TAG_HANDLER_PROC(tag)
     {
 
     TAG_HANDLER_PROC(tag)
     {

-        unsigned long tmp;

         wxColour clr;
 
         wxColour clr;
 

-        if (tag.HasParam("TEXT")) {
-            if (tag.ScanParam("TEXT", "#%lX", &tmp) == 1) {
-                clr = wxColour((tmp & 0xFF0000) >> 16 , (tmp & 0x00FF00) >> 8, (tmp & 0x0000FF));
-                m_WParser -> SetActualColor(clr);
-                m_WParser -> GetContainer() -> InsertCell(new wxHtmlColourCell(clr));
-            }
-       }
-
-        if (tag.HasParam("LINK")) {
-            if (tag.ScanParam("LINK", "#%lX", &tmp) == 1) {
-                clr = wxColour((tmp & 0xFF0000) >> 16 , (tmp & 0x00FF00) >> 8, (tmp & 0x0000FF));
-                m_WParser -> SetLinkColor(clr);
-           }
+        if (tag.GetParamAsColour(wxT("TEXT"), &clr))
+        {
+            m_WParser->SetActualColor(clr);
+            m_WParser->GetContainer()->InsertCell(new wxHtmlColourCell(clr));

         }
 
         }
 

-        if (tag.HasParam("BGCOLOR")) {
-            if (tag.ScanParam("BGCOLOR", "#%lX", &tmp) == 1) {
-                clr = wxColour((tmp & 0xFF0000) >> 16 , (tmp & 0x00FF00) >> 8, (tmp & 0x0000FF));
-                m_WParser -> GetContainer() -> InsertCell(new wxHtmlColourCell(clr, wxHTML_CLR_BACKGROUND));
-                if (m_WParser -> GetWindow() != NULL)
-                    m_WParser -> GetWindow() -> SetBackgroundColour(clr);
-           }
+        if (tag.GetParamAsColour(wxT("LINK"), &clr))
+            m_WParser->SetLinkColor(clr);
+
+        if (tag.GetParamAsColour(wxT("BGCOLOR"), &clr))
+        {
+            m_WParser->GetContainer()->InsertCell(
+                new wxHtmlColourCell(clr, wxHTML_CLR_BACKGROUND));
+            if (m_WParser->GetWindow() != NULL)
+                m_WParser->GetWindow()->SetBackgroundColour(clr);

         }
         return FALSE;
     }
         }
         return FALSE;
     }


@@ -202,20 +201,22 @@ TAG_HANDLER_BEGIN(BLOCKQUOTE, "BLOCKQUOTE")
     TAG_HANDLER_PROC(tag)
     {
         wxHtmlContainerCell *c;
     TAG_HANDLER_PROC(tag)
     {
         wxHtmlContainerCell *c;

-    
-        m_WParser -> CloseContainer();
-        c = m_WParser -> OpenContainer();
-        if (c -> GetAlignHor() == wxHTML_ALIGN_RIGHT)
-            c -> SetIndent(5 * m_WParser -> GetCharWidth(), wxHTML_INDENT_RIGHT);
+
+        m_WParser->CloseContainer();
+        c = m_WParser->OpenContainer();
+
+        if (c->GetAlignHor() == wxHTML_ALIGN_RIGHT)
+            c->SetIndent(5 * m_WParser->GetCharWidth(), wxHTML_INDENT_RIGHT);

         else
         else

-            c -> SetIndent(5 * m_WParser -> GetCharWidth(), wxHTML_INDENT_LEFT);
-        c -> SetIndent(m_WParser -> GetCharHeight(), wxHTML_INDENT_TOP);
-        m_WParser -> OpenContainer();
+            c->SetIndent(5 * m_WParser->GetCharWidth(), wxHTML_INDENT_LEFT);
+
+        c->SetIndent(m_WParser->GetCharHeight(), wxHTML_INDENT_TOP);
+        m_WParser->OpenContainer();

         ParseInner(tag);
         ParseInner(tag);

-        c = m_WParser -> CloseContainer();
-        c -> SetIndent(m_WParser -> GetCharHeight(), wxHTML_INDENT_BOTTOM);
-        m_WParser -> CloseContainer();
-        m_WParser -> OpenContainer();
+        c = m_WParser->CloseContainer();
+        c->SetIndent(m_WParser->GetCharHeight(), wxHTML_INDENT_BOTTOM);
+        m_WParser->CloseContainer();
+        m_WParser->OpenContainer();

         return TRUE;
     }
 
         return TRUE;
     }