compilation fixes

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

diff --git a/src/html/htmlpars.cpp b/src/html/htmlpars.cpp
index 7986e9610158d48497e4686b44ee6794d28212ef..ab88ab7d46ee5acafb478b17056fe7d6b736dbc5 100644 (file)
--- a/src/html/htmlpars.cpp
+++ b/src/html/htmlpars.cpp
@@ -51,53 +51,53 @@ wxObject* wxHtmlParser::Parse(const wxString& source)
     return result;
 }
 
     return result;
 }
 

-
-

 void wxHtmlParser::InitParser(const wxString& source)
 {
 void wxHtmlParser::InitParser(const wxString& source)
 {

-    m_Source = source;
-    m_Cache = new wxHtmlTagsCache(m_Source);
+    SetSource(source);

 }
 }

-        
-        
-        
+

 void wxHtmlParser::DoneParser()
 {
     delete m_Cache;
     m_Cache = NULL;
 }
 
 void wxHtmlParser::DoneParser()
 {
     delete m_Cache;
     m_Cache = NULL;
 }
 

-
-
-#define wxHTML_MAX_BUFLEN 1024
+void wxHtmlParser::SetSource(const wxString& src)
+{
+    m_Source = src;
+    delete m_Cache;
+    m_Cache = new wxHtmlTagsCache(m_Source);
+}

 
 void wxHtmlParser::DoParsing(int begin_pos, int end_pos)
 {
 
 void wxHtmlParser::DoParsing(int begin_pos, int end_pos)
 {

-    char temp[wxHTML_BUFLEN], c;
+    if (end_pos <= begin_pos) return;
+
+    char c;
+    char *temp = new char[end_pos - begin_pos + 1];

     int i;
     int templen;
 
     templen = 0;
     i = begin_pos;
 
     int i;
     int templen;
 
     templen = 0;
     i = begin_pos;
 

-    while (i < end_pos) {
+    while (i < end_pos) 
+    {

         c = m_Source[(unsigned int) i];
 
         // continue building word:
         c = m_Source[(unsigned int) i];
 
         // continue building word:

-        if (c != '<') {
+        if (c != '<') 
+           {

             temp[templen++] = c;
             temp[templen++] = c;

-            if (templen == wxHTML_BUFLEN-1) {
-                temp[templen] = 0;
-                AddText(temp);
-                templen = 0;
-            }

             i++;
         }
 
             i++;
         }
 

-        else if (c == '<')  {
+        else if (c == '<')  
+           {

             wxHtmlTag tag(m_Source, i, end_pos, m_Cache);
 
             wxHtmlTag tag(m_Source, i, end_pos, m_Cache);
 

-            if (templen) {
+            if (templen) 
+               {

                 temp[templen] = 0;
                 AddText(temp);
                 templen = 0;
                 temp[templen] = 0;
                 AddText(temp);
                 templen = 0;


@@ -108,14 +108,14 @@ void wxHtmlParser::DoParsing(int begin_pos, int end_pos)
         }
     }
 
         }
     }
 

-    if (templen) { // last word of block :-(
+    if (templen) 
+    { // last word of block :-(

         temp[templen] = 0;
         AddText(temp);
     }
         temp[templen] = 0;
         AddText(temp);
     }

+    delete[] temp;

 }
 
 }
 

-
-

 void wxHtmlParser::AddTag(const wxHtmlTag& tag)
 {
     wxHtmlTagHandler *h;
 void wxHtmlParser::AddTag(const wxHtmlTag& tag)
 {
     wxHtmlTagHandler *h;


@@ -123,18 +123,17 @@ void wxHtmlParser::AddTag(const wxHtmlTag& tag)
 
     h = (wxHtmlTagHandler*) m_HandlersHash.Get(tag.GetName());
     if (h)
 
     h = (wxHtmlTagHandler*) m_HandlersHash.Get(tag.GetName());
     if (h)

-        inner = h -> HandleTag(tag);
-    if (!inner) {
+        inner = h->HandleTag(tag);
+    if (!inner) 
+    {

         if (tag.HasEnding())
             DoParsing(tag.GetBeginPos(), tag.GetEndPos1());
     }
 }
 
         if (tag.HasEnding())
             DoParsing(tag.GetBeginPos(), tag.GetEndPos1());
     }
 }
 

-
-

 void wxHtmlParser::AddTagHandler(wxHtmlTagHandler *handler)
 {
 void wxHtmlParser::AddTagHandler(wxHtmlTagHandler *handler)
 {

-    wxString s(handler -> GetSupportedTags());
+    wxString s(handler->GetSupportedTags());

     wxStringTokenizer tokenizer(s, ", ");
 
     while (tokenizer.HasMoreTokens())
     wxStringTokenizer tokenizer(s, ", ");
 
     while (tokenizer.HasMoreTokens())


@@ -143,48 +142,44 @@ void wxHtmlParser::AddTagHandler(wxHtmlTagHandler *handler)
     if (m_HandlersList.IndexOf(handler) == wxNOT_FOUND)
         m_HandlersList.Append(handler);
 
     if (m_HandlersList.IndexOf(handler) == wxNOT_FOUND)
         m_HandlersList.Append(handler);
 

-    handler -> SetParser(this);
+    handler->SetParser(this);

 }
 
 }
 

-
-

 void wxHtmlParser::PushTagHandler(wxHtmlTagHandler *handler, wxString tags)
 {
     wxStringTokenizer tokenizer(tags, ", ");
     wxString key;
 
 void wxHtmlParser::PushTagHandler(wxHtmlTagHandler *handler, wxString tags)
 {
     wxStringTokenizer tokenizer(tags, ", ");
     wxString key;
 

-    if (m_HandlersStack == NULL) {
+    if (m_HandlersStack == NULL) 
+    {

         m_HandlersStack = new wxList;
         m_HandlersStack = new wxList;

-        m_HandlersStack -> DeleteContents(TRUE);
+        m_HandlersStack->DeleteContents(TRUE);

     }
 
     }
 

-    m_HandlersStack -> Insert(new wxHashTable(m_HandlersHash));
+    m_HandlersStack->Insert(new wxHashTable(m_HandlersHash));

 
 

-    while (tokenizer.HasMoreTokens()) {
+    while (tokenizer.HasMoreTokens()) 
+    {

         key = tokenizer.NextToken();
         m_HandlersHash.Delete(key);
         m_HandlersHash.Put(key, handler);
     }
 }
 
         key = tokenizer.NextToken();
         m_HandlersHash.Delete(key);
         m_HandlersHash.Put(key, handler);
     }
 }
 

-
-

 void wxHtmlParser::PopTagHandler()
 {
     wxNode *first;
     
     if (m_HandlersStack == NULL || 
 void wxHtmlParser::PopTagHandler()
 {
     wxNode *first;
     
     if (m_HandlersStack == NULL || 

-        (first = m_HandlersStack -> GetFirst()) == NULL) 
+        (first = m_HandlersStack->GetFirst()) == NULL) 

     {
         wxLogWarning(_("Warning: attempt to remove HTML tag handler from empty stack."));
         return;
     }
     {
         wxLogWarning(_("Warning: attempt to remove HTML tag handler from empty stack."));
         return;
     }

-    m_HandlersHash = *((wxHashTable*) first -> GetData());
-    m_HandlersStack -> DeleteNode(first);
+    m_HandlersHash = *((wxHashTable*) first->GetData());
+    m_HandlersStack->DeleteNode(first);

 }
 
 }
 

-
-

 wxHtmlParser::~wxHtmlParser()
 {
     if (m_HandlersStack) delete m_HandlersStack;
 wxHtmlParser::~wxHtmlParser()
 {
     if (m_HandlersStack) delete m_HandlersStack;


@@ -193,8 +188,6 @@ wxHtmlParser::~wxHtmlParser()
     m_HandlersList.Clear();
 }
 
     m_HandlersList.Clear();
 }
 

-
-

 //-----------------------------------------------------------------------------
 // wxHtmlTagHandler
 //-----------------------------------------------------------------------------
 //-----------------------------------------------------------------------------
 // wxHtmlTagHandler
 //-----------------------------------------------------------------------------