Use helper class instead of manually calling gtk_tree_path_free().

[wxWidgets.git] / src / richtext / richtexthtml.cpp

diff --git a/src/richtext/richtexthtml.cpp b/src/richtext/richtexthtml.cpp
index 6079da35c528fbb0039d0aa6fbcd2909ad995407..c5df3087979521a94c9ca22fc0b6890f2319064d 100644 (file)
--- a/src/richtext/richtexthtml.cpp
+++ b/src/richtext/richtexthtml.cpp
@@ -85,8 +85,7 @@ bool wxRichTextHTMLHandler::DoSaveFile(wxRichTextBuffer *buffer, wxOutputStream&
         customEncoding = new wxCSConv(GetEncoding());
         if (!customEncoding->IsOk())
         {
         customEncoding = new wxCSConv(GetEncoding());
         if (!customEncoding->IsOk())
         {

-            delete customEncoding;
-            customEncoding = NULL;
+            wxDELETE(customEncoding);

         }
     }
     if (customEncoding)
         }
     }
     if (customEncoding)


@@ -322,7 +321,7 @@ void wxRichTextHTMLHandler::BeginParagraphFormatting(const wxTextAttr& WXUNUSED(
             str << wxT(">");
 
             // TODO: convert to pixels
             str << wxT(">");
 
             // TODO: convert to pixels

-            int indentPixels = indentLeftMM*10/4;
+            int indentPixels = static_cast<int>(indentLeftMM*10/4);

 
             if ((GetFlags() & wxRICHTEXT_HANDLER_USE_CSS) == 0)
             {
 
             if ((GetFlags() & wxRICHTEXT_HANDLER_USE_CSS) == 0)
             {


@@ -331,8 +330,6 @@ void wxRichTextHTMLHandler::BeginParagraphFormatting(const wxTextAttr& WXUNUSED(
                 m_inTable = true;
             }
 
                 m_inTable = true;
             }
 

-            OutputFont(thisStyle, str);
-

             if (((GetFlags() & wxRICHTEXT_HANDLER_USE_CSS) == 0) && (thisStyle.GetLeftSubIndent() < 0))
             {
                 str << SymbolicIndent( - thisStyle.GetLeftSubIndent());
             if (((GetFlags() & wxRICHTEXT_HANDLER_USE_CSS) == 0) && (thisStyle.GetLeftSubIndent() < 0))
             {
                 str << SymbolicIndent( - thisStyle.GetLeftSubIndent());


@@ -366,16 +363,17 @@ void wxRichTextHTMLHandler::BeginParagraphFormatting(const wxTextAttr& WXUNUSED(
 
         str << wxT(">");
     }
 
         str << wxT(">");
     }

+    OutputFont(thisStyle, str);

 }
 
 /// End paragraph formatting
 void wxRichTextHTMLHandler::EndParagraphFormatting(const wxTextAttr& WXUNUSED(currentStyle), const wxTextAttr& thisStyle, wxTextOutputStream& stream)
 {
 }
 
 /// End paragraph formatting
 void wxRichTextHTMLHandler::EndParagraphFormatting(const wxTextAttr& WXUNUSED(currentStyle), const wxTextAttr& thisStyle, wxTextOutputStream& stream)
 {

+    if (thisStyle.HasFont())
+        stream << wxT("</font>");
+

     if (m_inTable)
     {
     if (m_inTable)
     {

-        if (thisStyle.HasFont())
-            stream << wxT("</font>");
-

         stream << wxT("</td></tr></table></p>\n");
         m_inTable = false;
     }
         stream << wxT("</td></tr></table></p>\n");
         m_inTable = false;
     }