fixed   handling in wxHtmlWinParser, broken by TAB-handling changes (bug #1957041)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@53456 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/html/winpars.cpp b/src/html/winpars.cpp
index 9b5adbdfb17dfff7824b924963a9e6d35c253ffb..f21cf9e8fba4ea531c95d3553d3971e39e46aecf 100644 (file)
--- a/src/html/winpars.cpp
+++ b/src/html/winpars.cpp
@@ -345,15 +345,18 @@ wxFSFile *wxHtmlWinParser::OpenURL(wxHtmlURLType type,
     return GetFS()->OpenFile(myurl, flags);
 }
 
     return GetFS()->OpenFile(myurl, flags);
 }
 

+#define NBSP_UNICODE_VALUE  (wxChar(160))
+#if !wxUSE_UNICODE
+    #define CUR_NBSP_VALUE m_nbsp
+#else
+    #define CUR_NBSP_VALUE NBSP_UNICODE_VALUE
+#endif
+

 void wxHtmlWinParser::AddText(const wxString& txt)
 {
 void wxHtmlWinParser::AddText(const wxString& txt)
 {

-    #define NBSP_UNICODE_VALUE  (wxChar(160))

 #if !wxUSE_UNICODE
     if ( m_nbsp == 0 )
         m_nbsp = GetEntitiesParser()->GetCharForCode(NBSP_UNICODE_VALUE);
 #if !wxUSE_UNICODE
     if ( m_nbsp == 0 )
         m_nbsp = GetEntitiesParser()->GetCharForCode(NBSP_UNICODE_VALUE);

-    #define CUR_NBSP_VALUE m_nbsp
-#else
-    #define CUR_NBSP_VALUE NBSP_UNICODE_VALUE

 #endif
 
     if ( m_whitespaceMode == Whitespace_Normal )
 #endif
 
     if ( m_whitespaceMode == Whitespace_Normal )


@@ -385,7 +388,7 @@ void wxHtmlWinParser::AddText(const wxString& txt)
         while (i < end)
         {
             size_t x = 0;
         while (i < end)
         {
             size_t x = 0;

-            wxChar d = *i;
+            const wxChar d = temp[templen++] = *i;

             if ((d == wxT('\n')) || (d == wxT('\r')) || (d == wxT(' ')) || (d == wxT('\t')))
             {
                 ++i, ++x;
             if ((d == wxT('\n')) || (d == wxT('\r')) || (d == wxT(' ')) || (d == wxT('\t')))
             {
                 ++i, ++x;


@@ -402,11 +405,6 @@ void wxHtmlWinParser::AddText(const wxString& txt)
                 ++i;
             }
 
                 ++i;
             }
 

-            if (d == CUR_NBSP_VALUE)
-                d = ' ';
-
-            temp[templen++] = d;
-

             if (x)
             {
                 temp[templen-1] = wxT(' ');
             if (x)
             {
                 temp[templen-1] = wxT(' ');


@@ -445,6 +443,12 @@ void wxHtmlWinParser::FlushWordBuf(wxChar *buf, int& len)
 {
     buf[len] = 0;
 
 {
     buf[len] = 0;
 

+    for ( int i = 0; i < len; i++ )
+    {
+        if ( buf[i] == CUR_NBSP_VALUE )
+            buf[i] = ' ';
+    }
+

 #if !wxUSE_UNICODE
     if (m_EncConv)
         m_EncConv->Convert(buf);
 #if !wxUSE_UNICODE
     if (m_EncConv)
         m_EncConv->Convert(buf);