[Libreoffice-commits] .: 9 commits - svtools/source
Christina Rossmanith
crossmanith at kemper.freedesktop.org
Sun Jun 5 04:28:30 PDT 2011
svtools/source/svhtml/htmlkywd.cxx | 103 ++++------
svtools/source/svhtml/htmlsupp.cxx | 14 -
svtools/source/svhtml/parhtml.cxx | 356 ++++++++++++++++---------------------
3 files changed, 208 insertions(+), 265 deletions(-)
New commits:
commit b247a329c5ca9f8cd1e84fbaca3456be71a87fbf
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Tue May 24 10:50:21 2011 +0200
Removed lines from merge conflicts
diff --git a/svtools/source/svhtml/htmlkywd.cxx b/svtools/source/svhtml/htmlkywd.cxx
index ef30c0d..0b40a6f 100644
--- a/svtools/source/svhtml/htmlkywd.cxx
+++ b/svtools/source/svhtml/htmlkywd.cxx
@@ -549,15 +549,7 @@ static HTML_CharEntry aHTMLCharNameTab[] = {
extern "C"
{
-<<<<<<< HEAD
-static int
-#if defined( WNT )
- __cdecl
-#endif
- HTMLCharNameCompare( const void *pFirst, const void *pSecond)
-=======
static int SAL_CALL HTMLCharNameCompare( const void *pFirst, const void *pSecond)
->>>>>>> Use SAL_CALL / refurbish #include's (IWYU) / sal_Bool->bool
{
int nRet = 0;
if( USHRT_MAX == ((HTML_CharEntry*)pFirst)->cChar )
@@ -1006,15 +998,7 @@ static HTML_ColorEntry aHTMLColorNameTab[] = {
extern "C"
{
-<<<<<<< HEAD
-static int
-#if defined( WNT )
- __cdecl
-#endif
- HTMLColorNameCompare( const void *pFirst, const void *pSecond)
-=======
static int SAL_CALL HTMLColorNameCompare( const void *pFirst, const void *pSecond)
->>>>>>> Use SAL_CALL / refurbish #include's (IWYU) / sal_Bool->bool
{
int nRet = 0;
if( HTML_NO_COLOR == ((HTML_ColorEntry*)pFirst)->nColor )
commit c9c86a76dd8ccd3d1494639cb0bc46ca58c5bb90
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Tue May 17 20:55:42 2011 +0200
Use SAL_CALL / refurbish #include's (IWYU) / sal_Bool->bool
diff --git a/svtools/source/svhtml/htmlkywd.cxx b/svtools/source/svhtml/htmlkywd.cxx
index 578b5c0..ef30c0d 100644
--- a/svtools/source/svhtml/htmlkywd.cxx
+++ b/svtools/source/svhtml/htmlkywd.cxx
@@ -29,14 +29,14 @@
// MARKER(update_precomp.py): autogen include statement, do not remove
#include "precompiled_svtools.hxx"
-#include <ctype.h>
-#include <stdlib.h>
-#include <string.h>
-#include <limits.h>
-
-#include <svtools/svparser.hxx>
-#include <svtools/htmlkywd.hxx>
+#include <limits.h> // for USHRT_MAX
+#include <sal/types.h> // for sal_Char, sal_Unicode, etc
+#include <stdlib.h> // for bsearch, qsort
+#include <string.h> // for strcmp
#include <svtools/htmltokn.h>
+#include <tools/solar.h> // for String
+#include <svtools/htmlkywd.hxx>
+#include <tools/string.hxx> // for String::CompareToAscii, etc
// Table has still to be sorted
struct HTML_TokenEntry
@@ -50,7 +50,7 @@ struct HTML_TokenEntry
};
// Flag: RTF token table has already been sorted
-static int bSortKeyWords = sal_False;
+static bool bSortKeyWords = false;
static HTML_TokenEntry aHTMLTokenTab[] = {
{{OOO_STRING_SVTOOLS_HTML_area}, HTML_AREA}, // Netscape 2.0
@@ -203,11 +203,7 @@ static HTML_TokenEntry aHTMLTokenTab[] = {
extern "C"
{
-static int
-#if defined( WNT )
- __cdecl
-#endif
- HTMLKeyCompare( const void *pFirst, const void *pSecond)
+static int SAL_CALL HTMLKeyCompare( const void *pFirst, const void *pSecond)
{
int nRet = 0;
if( -1 == ((HTML_TokenEntry*)pFirst)->nToken )
@@ -242,7 +238,7 @@ int GetHTMLToken( const String& rName )
sizeof( aHTMLTokenTab ) / sizeof( HTML_TokenEntry ),
sizeof( HTML_TokenEntry ),
HTMLKeyCompare );
- bSortKeyWords = sal_True;
+ bSortKeyWords = true;
}
int nRet = 0;
@@ -275,7 +271,7 @@ struct HTML_CharEntry
};
// Flag: RTF token table has already been sorted
-static int bSortCharKeyWords = sal_False;
+static bool bSortCharKeyWords = false;
static HTML_CharEntry aHTMLCharNameTab[] = {
{{OOO_STRING_SVTOOLS_HTML_C_lt}, 60},
@@ -553,11 +549,15 @@ static HTML_CharEntry aHTMLCharNameTab[] = {
extern "C"
{
+<<<<<<< HEAD
static int
#if defined( WNT )
__cdecl
#endif
HTMLCharNameCompare( const void *pFirst, const void *pSecond)
+=======
+static int SAL_CALL HTMLCharNameCompare( const void *pFirst, const void *pSecond)
+>>>>>>> Use SAL_CALL / refurbish #include's (IWYU) / sal_Bool->bool
{
int nRet = 0;
if( USHRT_MAX == ((HTML_CharEntry*)pFirst)->cChar )
@@ -592,7 +592,7 @@ sal_Unicode GetHTMLCharName( const String& rName )
sizeof( aHTMLCharNameTab ) / sizeof( HTML_CharEntry ),
sizeof( HTML_CharEntry ),
HTMLCharNameCompare );
- bSortCharKeyWords = sal_True;
+ bSortCharKeyWords = true;
}
sal_Unicode cRet = 0;
@@ -611,7 +611,7 @@ sal_Unicode GetHTMLCharName( const String& rName )
}
// Flag: Options table has already been sorted
-static int bSortOptionKeyWords = sal_False;
+static bool bSortOptionKeyWords = false;
static HTML_TokenEntry aHTMLOptionTab[] = {
@@ -823,7 +823,7 @@ int GetHTMLOption( const String& rName )
sizeof( aHTMLOptionTab ) / sizeof( HTML_TokenEntry ),
sizeof( HTML_TokenEntry ),
HTMLKeyCompare );
- bSortOptionKeyWords = sal_True;
+ bSortOptionKeyWords = true;
}
int nRet = HTML_O_UNKNOWN;
@@ -853,7 +853,7 @@ struct HTML_ColorEntry
};
// Flag: color table has already been sorted
-static int bSortColorKeyWords = sal_False;
+static bool bSortColorKeyWords = false;
#define HTML_NO_COLOR 0xffffffffUL
@@ -1006,11 +1006,15 @@ static HTML_ColorEntry aHTMLColorNameTab[] = {
extern "C"
{
+<<<<<<< HEAD
static int
#if defined( WNT )
__cdecl
#endif
HTMLColorNameCompare( const void *pFirst, const void *pSecond)
+=======
+static int SAL_CALL HTMLColorNameCompare( const void *pFirst, const void *pSecond)
+>>>>>>> Use SAL_CALL / refurbish #include's (IWYU) / sal_Bool->bool
{
int nRet = 0;
if( HTML_NO_COLOR == ((HTML_ColorEntry*)pFirst)->nColor )
@@ -1045,7 +1049,7 @@ sal_uInt32 GetHTMLColor( const String& rName )
sizeof( aHTMLColorNameTab ) / sizeof( HTML_ColorEntry ),
sizeof( HTML_ColorEntry ),
HTMLColorNameCompare );
- bSortColorKeyWords = sal_True;
+ bSortColorKeyWords = true;
}
sal_uInt32 nRet = HTML_NO_COLOR;
commit b8e4fae544147f9af83ec811bac49662481305f4
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Tue May 17 09:26:35 2011 +0200
Changed HTML_ColorEntry.nColor to sal_uInt32
diff --git a/svtools/source/svhtml/htmlkywd.cxx b/svtools/source/svhtml/htmlkywd.cxx
index ae7e930..578b5c0 100644
--- a/svtools/source/svhtml/htmlkywd.cxx
+++ b/svtools/source/svhtml/htmlkywd.cxx
@@ -849,7 +849,7 @@ struct HTML_ColorEntry
const sal_Char* sName;
const String *pUName;
};
- sal_uLong nColor;
+ sal_uInt32 nColor;
};
// Flag: color table has already been sorted
@@ -1037,7 +1037,7 @@ static int
}
-sal_uLong GetHTMLColor( const String& rName )
+sal_uInt32 GetHTMLColor( const String& rName )
{
if( !bSortColorKeyWords )
{
@@ -1048,7 +1048,7 @@ sal_uLong GetHTMLColor( const String& rName )
bSortColorKeyWords = sal_True;
}
- sal_uLong nRet = ULONG_MAX;
+ sal_uInt32 nRet = HTML_NO_COLOR;
void* pFound;
HTML_ColorEntry aSrch;
aSrch.pUName = &rName;
commit 88948b20686923168bfefb158ea6e1fe96dc859e
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Thu May 12 11:18:14 2011 +0200
Added HTML_TOKEN_ONOFF test
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index 77bdb75..66ef97c 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -1225,8 +1225,9 @@ int HTMLParser::_GetNextToken()
// fdo#34666 fdo#36080 fdo#36390: closing "/>"?:
// return HTML_<TOKEN>_OFF instead of HTML_<TOKEN>_ON
- if (aToken.Len() >= 1 && '/' == aToken.GetChar(aToken.Len()-1)) {
- nRet += 1; // HTML_<TOKEN>_ON -> HTML_<TOKEN>_OFF;
+ if ((HTML_TOKEN_ONOFF & nRet) && (aToken.Len() >= 1) &&
+ ('/' == aToken.GetChar(aToken.Len()-1))) {
+ ++nRet; // HTML_<TOKEN>_ON -> HTML_<TOKEN>_OFF;
}
if( sal_Unicode(EOF) == nNextCh && rInput.IsEof() )
{
commit dd068979a28b83aa83573a60f7e52d5113b054de
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Fri May 6 22:10:43 2011 +0200
Translated comments
diff --git a/svtools/source/svhtml/htmlkywd.cxx b/svtools/source/svhtml/htmlkywd.cxx
index a85f348..ae7e930 100644
--- a/svtools/source/svhtml/htmlkywd.cxx
+++ b/svtools/source/svhtml/htmlkywd.cxx
@@ -38,7 +38,7 @@
#include <svtools/htmlkywd.hxx>
#include <svtools/htmltokn.h>
-// die Tabelle muss noch sortiert werden
+// Table has still to be sorted
struct HTML_TokenEntry
{
union
@@ -49,7 +49,7 @@ struct HTML_TokenEntry
int nToken;
};
-// Flag: RTF-Token Tabelle wurde schon sortiert
+// Flag: RTF token table has already been sorted
static int bSortKeyWords = sal_False;
static HTML_TokenEntry aHTMLTokenTab[] = {
@@ -264,8 +264,6 @@ int GetHTMLToken( const String& rName )
return nRet;
}
-/* */
-
struct HTML_CharEntry
{
union
@@ -276,7 +274,7 @@ struct HTML_CharEntry
sal_Unicode cChar;
};
-// Flag: RTF-Token Tabelle wurde schon sortiert
+// Flag: RTF token table has already been sorted
static int bSortCharKeyWords = sal_False;
static HTML_CharEntry aHTMLCharNameTab[] = {
@@ -350,7 +348,7 @@ static HTML_CharEntry aHTMLCharNameTab[] = {
{{OOO_STRING_SVTOOLS_HTML_S_thorn}, 254},
{{OOO_STRING_SVTOOLS_HTML_S_yuml}, 255},
-// Sonderzeichen
+// special characters
{{OOO_STRING_SVTOOLS_HTML_S_acute}, 180},
{{OOO_STRING_SVTOOLS_HTML_S_brvbar}, 166},
{{OOO_STRING_SVTOOLS_HTML_S_cedil}, 184},
@@ -384,7 +382,7 @@ static HTML_CharEntry aHTMLCharNameTab[] = {
{{OOO_STRING_SVTOOLS_HTML_S_uml}, 168},
{{OOO_STRING_SVTOOLS_HTML_S_yen}, 165},
-// Netscape kennt noch ein paar in Grossbuchstaben ...
+// Netscape has some in upper case...
{{OOO_STRING_SVTOOLS_HTML_C_LT}, 60},
{{OOO_STRING_SVTOOLS_HTML_C_GT}, 62},
{{OOO_STRING_SVTOOLS_HTML_C_AMP}, 38},
@@ -392,7 +390,7 @@ static HTML_CharEntry aHTMLCharNameTab[] = {
{{OOO_STRING_SVTOOLS_HTML_S_COPY}, 169},
{{OOO_STRING_SVTOOLS_HTML_S_REG}, 174},
-// Sonderzeichen, die zu Tokens konvertiert werden !!!
+// special characters, which will be converted to tokens !!!
{{OOO_STRING_SVTOOLS_HTML_S_nbsp}, 1},
{{OOO_STRING_SVTOOLS_HTML_S_shy}, 2},
@@ -612,14 +610,12 @@ sal_Unicode GetHTMLCharName( const String& rName )
return cRet;
}
-/* */
-
-// Flag: Optionen-Tabelle wurde schon sortiert
+// Flag: Options table has already been sorted
static int bSortOptionKeyWords = sal_False;
static HTML_TokenEntry aHTMLOptionTab[] = {
-// Attribute ohne Wert
+// Attributes without value
{{OOO_STRING_SVTOOLS_HTML_O_box}, HTML_O_BOX},
{{OOO_STRING_SVTOOLS_HTML_O_checked}, HTML_O_CHECKED},
{{OOO_STRING_SVTOOLS_HTML_O_compact}, HTML_O_COMPACT},
@@ -641,7 +637,7 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_selected}, HTML_O_SELECTED},
{{OOO_STRING_SVTOOLS_HTML_O_shapes}, HTML_O_SHAPES},
-// Attribute mit einem String als Wert
+// Attributes with a string value
{{OOO_STRING_SVTOOLS_HTML_O_above}, HTML_O_ABOVE},
{{OOO_STRING_SVTOOLS_HTML_O_accept}, HTML_O_ACCEPT},
{{OOO_STRING_SVTOOLS_HTML_O_accesskey}, HTML_O_ACCESSKEY},
@@ -682,14 +678,14 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_sdlibrary}, HTML_O_SDLIBRARY},
{{OOO_STRING_SVTOOLS_HTML_O_sdmodule}, HTML_O_SDMODULE},
-// Attribute mit einem SGML-Identifier als Wert
+// Attributes with a SGML identifier value
{{OOO_STRING_SVTOOLS_HTML_O_from}, HTML_O_FROM},
{{OOO_STRING_SVTOOLS_HTML_O_id}, HTML_O_ID},
{{OOO_STRING_SVTOOLS_HTML_O_target}, HTML_O_TARGET}, // Netscape 2.0
{{OOO_STRING_SVTOOLS_HTML_O_to}, HTML_O_TO},
{{OOO_STRING_SVTOOLS_HTML_O_until}, HTML_O_UNTIL},
-// Attribute mit einem URI als Wert
+// Attributes with an URI value
{{OOO_STRING_SVTOOLS_HTML_O_action}, HTML_O_ACTION},
{{OOO_STRING_SVTOOLS_HTML_O_archive}, HTML_O_ARCHIVE},
{{OOO_STRING_SVTOOLS_HTML_O_background}, HTML_O_BACKGROUND},
@@ -705,11 +701,11 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_src}, HTML_O_SRC},
{{OOO_STRING_SVTOOLS_HTML_O_usemap}, HTML_O_USEMAP}, // Netscape 2.0
-// Attribute mit Entity-Namen als Wert
+// Attributes with entity name value
{{OOO_STRING_SVTOOLS_HTML_O_dingbat}, HTML_O_DINGBAT},
{{OOO_STRING_SVTOOLS_HTML_O_sym}, HTML_O_SYM},
-// Attribute mit einer Farbe als Wert (alle Netscape)
+// Attributes with a color value (all Netscape versions)
{{OOO_STRING_SVTOOLS_HTML_O_alink}, HTML_O_ALINK},
{{OOO_STRING_SVTOOLS_HTML_O_bgcolor}, HTML_O_BGCOLOR},
{{OOO_STRING_SVTOOLS_HTML_O_bordercolor}, HTML_O_BORDERCOLOR}, // IExplorer 2.0
@@ -720,7 +716,7 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_text}, HTML_O_TEXT},
{{OOO_STRING_SVTOOLS_HTML_O_vlink}, HTML_O_VLINK},
-// Attribute mit einem numerischen Wert
+// Attributes with a numerical value
{{OOO_STRING_SVTOOLS_HTML_O_border}, HTML_O_BORDER},
{{OOO_STRING_SVTOOLS_HTML_O_cellspacing},HTML_O_CELLSPACING}, // HTML 3 Table Model Draft
{{OOO_STRING_SVTOOLS_HTML_O_cellpadding},HTML_O_CELLPADDING}, // HTML 3 Table Model Draft
@@ -758,7 +754,7 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_y}, HTML_O_Y},
{{OOO_STRING_SVTOOLS_HTML_O_zindex}, HTML_O_ZINDEX},
-// Attribute mit Enum-Werten
+// Attributes with enum values
{{OOO_STRING_SVTOOLS_HTML_O_bgproperties}, HTML_O_BGPROPERTIES}, // IExplorer 2.0
{{OOO_STRING_SVTOOLS_HTML_O_behavior}, HTML_O_BEHAVIOR}, // IExplorer 2.0
{{OOO_STRING_SVTOOLS_HTML_O_clear}, HTML_O_CLEAR},
@@ -781,7 +777,7 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_wrap}, HTML_O_WRAP},
{{OOO_STRING_SVTOOLS_HTML_O_visibility}, HTML_O_VISIBILITY},
-// Attribute mit Script-Code als Wert
+// Attributes with script code value
{{OOO_STRING_SVTOOLS_HTML_O_onblur}, HTML_O_ONBLUR}, // JavaScript
{{OOO_STRING_SVTOOLS_HTML_O_onchange}, HTML_O_ONCHANGE}, // JavaScript
{{OOO_STRING_SVTOOLS_HTML_O_onclick}, HTML_O_ONCLICK}, // JavaScript
@@ -810,7 +806,7 @@ static HTML_TokenEntry aHTMLOptionTab[] = {
{{OOO_STRING_SVTOOLS_HTML_O_SDonerror}, HTML_O_SDONERROR}, // StarBasic
{{OOO_STRING_SVTOOLS_HTML_O_SDonmouseout}, HTML_O_SDONMOUSEOUT}, // StarBasic
-// Attribute mit Kontext-abhaengigen Werten
+// Attributes with context sensitive values
{{OOO_STRING_SVTOOLS_HTML_O_align}, HTML_O_ALIGN},
{{OOO_STRING_SVTOOLS_HTML_O_cols}, HTML_O_COLS}, // Netscape 2.0 vs HTML 2.0
{{OOO_STRING_SVTOOLS_HTML_O_rows}, HTML_O_ROWS}, // Netscape 2.0 vs HTML 2.0
@@ -845,9 +841,7 @@ int GetHTMLOption( const String& rName )
return nRet;
}
-/* */
-// Flag: Farb-Tabelle wurde schon sortiert
struct HTML_ColorEntry
{
union
@@ -858,15 +852,14 @@ struct HTML_ColorEntry
sal_uLong nColor;
};
+// Flag: color table has already been sorted
static int bSortColorKeyWords = sal_False;
#define HTML_NO_COLOR 0xffffffffUL
-// die Farbnamen werden nicht exportiert
-// Sie stammen aus "http://www.uio.no/~mnbjerke/colors_w.html"
-// und scheinen im Gegensatz zu denen aus
-// "http://www.infi.net/wwwimages/colorindex.html"
-// zu stimmen
+// Color names are not exported (source:
+// "http://www.uio.no/~mnbjerke/colors_w.html")
+// "http://www.infi.net/wwwimages/colorindex.html" seem to be buggy.
static HTML_ColorEntry aHTMLColorNameTab[] = {
{ { "ALICEBLUE" }, 0x00f0f8ffUL },
{ { "ANTIQUEWHITE" }, 0x00faebd7UL },
commit 6e8fbb060fb2f608dba6c35351e7c24d37423667
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Fri May 6 21:35:35 2011 +0200
Removed comments/commented code
diff --git a/svtools/source/svhtml/htmlsupp.cxx b/svtools/source/svhtml/htmlsupp.cxx
index f76dd8f..9cc1093 100644
--- a/svtools/source/svhtml/htmlsupp.cxx
+++ b/svtools/source/svhtml/htmlsupp.cxx
@@ -41,19 +41,13 @@
#include <svtools/htmltokn.h>
#include <svtools/htmlkywd.hxx>
-/* */
-
// Table for converting option values into strings
-
static HTMLOptionEnum const aScriptLangOptEnums[] =
{
{ OOO_STRING_SVTOOLS_HTML_LG_starbasic, HTML_SL_STARBASIC },
{ OOO_STRING_SVTOOLS_HTML_LG_javascript, HTML_SL_JAVASCRIPT },
{ OOO_STRING_SVTOOLS_HTML_LG_javascript11,HTML_SL_JAVASCRIPT },
{ OOO_STRING_SVTOOLS_HTML_LG_livescript, HTML_SL_JAVASCRIPT },
-// { OOO_STRING_SVTOOLS_HTML_LG_unused_javascript, HTML_SL_UNUSEDJS },
-// { OOO_STRING_SVTOOLS_HTML_LG_vbscript, HTML_SL_VBSCRIPT },
-// { OOO_STRING_SVTOOLS_HTML_LG_starone, HTML_SL_STARONE },
{ 0, 0 }
};
commit 43f16e8961c0cb8a65141c7bb9caaf97999152d4
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Fri May 6 21:34:07 2011 +0200
Translated comments
diff --git a/svtools/source/svhtml/htmlsupp.cxx b/svtools/source/svhtml/htmlsupp.cxx
index 23e1caf..f76dd8f 100644
--- a/svtools/source/svhtml/htmlsupp.cxx
+++ b/svtools/source/svhtml/htmlsupp.cxx
@@ -43,7 +43,7 @@
/* */
-// Tabellen zum Umwandeln von Options-Werten in Strings
+// Table for converting option values into strings
static HTMLOptionEnum const aScriptLangOptEnums[] =
{
@@ -116,14 +116,14 @@ void HTMLParser::RemoveSGMLComment( String &rString, sal_Bool bFull )
rString.Erase( rString.Len()-1 );
- // SGML-Kommentare entfernen
+ // remove SGML comments
if( rString.Len() >= 4 &&
rString.CompareToAscii( "<!--", 4 ) == COMPARE_EQUAL )
{
xub_StrLen nPos = 3;
if( bFull )
{
- // die gesamte Zeile !
+ // the whole line
nPos = 4;
while( nPos < rString.Len() &&
( ( c = rString.GetChar( nPos )) != '\r' && c != '\n' ) )
@@ -144,7 +144,7 @@ void HTMLParser::RemoveSGMLComment( String &rString, sal_Bool bFull )
rString.Erase( rString.Len()-3 );
if( bFull )
{
- // auch noch ein "//" oder "'" und ggf CR/LF davor
+ // "//" or "'", maybe preceding CR/LF
rString.EraseTrailingChars();
xub_StrLen nDel = 0, nLen = rString.Len();
if( nLen >= 2 &&
commit 21535e7eba963b3613181ddfb74c50a825484299
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Sat Apr 23 22:25:26 2011 +0200
Bug 36390 <title /> breaks html filter fixed
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index aa760fa..77bdb75 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -372,7 +372,7 @@ int HTMLParser::FilterToken( int nToken )
{
case sal_Unicode(EOF):
nToken = 0;
- break; // nicht verschicken
+ break; // don't pass
case HTML_HEAD_OFF:
bIsInBody = sal_True;
@@ -393,7 +393,7 @@ int HTMLParser::FilterToken( int nToken )
case HTML_HTML_OFF:
nToken = 0;
bReadPRE = bReadListing = bReadXMP = sal_False;
- break; // HTML_ON wurde auch nicht verschickt !
+ break; // HTML_ON hasn't been passed either !
case HTML_PREFORMTXT_ON:
StartPRE();
@@ -1223,12 +1223,10 @@ int HTMLParser::_GetNextToken()
{
ScanText( '>' );
- // fdo#34666: closing "/>"?:
- // return HTML_UNKNOWNCONTROL_OFF instead of
- // HTML_UNKNOWNCONTROL_ON
+ // fdo#34666 fdo#36080 fdo#36390: closing "/>"?:
+ // return HTML_<TOKEN>_OFF instead of HTML_<TOKEN>_ON
if (aToken.Len() >= 1 && '/' == aToken.GetChar(aToken.Len()-1)) {
- if (HTML_UNKNOWNCONTROL_ON == nRet)
- nRet = HTML_UNKNOWNCONTROL_OFF;
+ nRet += 1; // HTML_<TOKEN>_ON -> HTML_<TOKEN>_OFF;
}
if( sal_Unicode(EOF) == nNextCh && rInput.IsEof() )
{
@@ -1351,7 +1349,6 @@ int HTMLParser::_GetNextToken()
bReadStyle = sal_False;
break;
}
-
}
}
break;
@@ -1648,7 +1645,7 @@ int HTMLParser::FilterPRE( int nToken )
{
xub_StrLen nSpaces = sal::static_int_cast< xub_StrLen >(
8 - (nPre_LinePos % 8));
- DBG_ASSERT( !aToken.Len(), "Wieso ist das Token nicht leer?" );
+ DBG_ASSERT( !aToken.Len(), "Why is the token not empty?" );
aToken.Expand( nSpaces, ' ' );
nPre_LinePos += nSpaces;
nToken = HTML_TEXTTOKEN;
commit 2aa674b8f97ce2f4cc58b06a0a0ed30743923b92
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Fri Apr 22 22:13:32 2011 +0200
Translated comments German->English
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index 58d1f5d..aa760fa 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -62,7 +62,7 @@ const sal_Int32 MAX_MACRO_LEN( 1024 );
const sal_Int32 MAX_ENTITY_LEN( 8L );
-// Tabellen zum Umwandeln von Options-Werten in Strings
+// Tables to convert option values into strings
// <INPUT TYPE=xxx>
static HTMLOptionEnum const aInputTypeOptEnums[] =
@@ -152,7 +152,7 @@ HTMLOption::HTMLOption( sal_uInt16 nTok, const String& rToken,
, nToken( nTok )
{
DBG_ASSERT( nToken>=HTML_OPTION_START && nToken<HTML_OPTION_END,
- "HTMLOption: unbekanntes Token" );
+ "HTMLOption: unknown token" );
}
sal_uInt32 HTMLOption::GetNumber() const
@@ -162,7 +162,7 @@ sal_uInt32 HTMLOption::GetNumber() const
(nToken>=HTML_OPTION_CONTEXT_START &&
nToken<HTML_OPTION_CONTEXT_END) ||
nToken==HTML_O_VALUE,
- "GetNumber: Option ist nicht numerisch" );
+ "GetNumber: Option not numerical" );
String aTmp( aValue );
aTmp.EraseLeadingChars();
sal_Int32 nTmp = aTmp.ToInt32();
@@ -173,7 +173,7 @@ sal_Int32 HTMLOption::GetSNumber() const
{
DBG_ASSERT( (nToken>=HTML_OPTION_NUMBER_START && nToken<HTML_OPTION_NUMBER_END) ||
(nToken>=HTML_OPTION_CONTEXT_START && nToken<HTML_OPTION_CONTEXT_END),
- "GetSNumber: Option ist nicht numerisch" );
+ "GetSNumber: Option not numerical" );
String aTmp( aValue );
aTmp.EraseLeadingChars();
return aTmp.ToInt32();
@@ -186,8 +186,8 @@ void HTMLOption::GetNumbers( SvULongs &rLongs, sal_Bool bSpaceDelim ) const
if( bSpaceDelim )
{
- // das ist ein sehr stark vereinfachter Scanner. Er sucht einfach
- // alle Tiffern aus dem String
+ // This is a very simplified scanner: it only searches all
+ // numerals in the string.
sal_Bool bInNum = sal_False;
sal_uLong nNum = 0;
for( xub_StrLen i=0; i<aValue.Len(); i++ )
@@ -213,8 +213,8 @@ void HTMLOption::GetNumbers( SvULongs &rLongs, sal_Bool bSpaceDelim ) const
}
else
{
- // hier wird auf die korrekte Trennung der Zahlen durch ',' geachtet
- // und auch mal eine 0 eingefuegt
+ // Check whether numbers are separated by ',' and
+ // insert 0 if necessary
xub_StrLen nPos = 0;
while( nPos < aValue.Len() )
{
@@ -252,7 +252,7 @@ void HTMLOption::GetNumbers( SvULongs &rLongs, sal_Bool bSpaceDelim ) const
void HTMLOption::GetColor( Color& rColor ) const
{
DBG_ASSERT( (nToken>=HTML_OPTION_COLOR_START && nToken<HTML_OPTION_COLOR_END) || nToken==HTML_O_SIZE,
- "GetColor: Option spezifiziert keine Farbe" );
+ "GetColor: Option is not a color." );
String aTmp( aValue );
aTmp.ToUpperAscii();
@@ -266,10 +266,8 @@ void HTMLOption::GetColor( Color& rColor ) const
xub_StrLen nPos = 0;
for( sal_uInt32 i=0; i<6; i++ )
{
- // Wie auch immer Netscape Farbwerte ermittelt,
- // maximal drei Zeichen, die kleiner als '0' sind werden
- // ignoriert. Bug #40901# stimmt damit. Mal schauen, was sich
- // irgendwelche HTML-Autoren noch so einfallen lassen...
+ // Whatever Netscape does to get color values,
+ // at maximum three characters < '0' are ignored.
register sal_Unicode c = nPos<aTmp.Len() ? aTmp.GetChar( nPos++ )
: '0';
if( c < '0' )
@@ -293,19 +291,19 @@ void HTMLOption::GetColor( Color& rColor ) const
HTMLInputType HTMLOption::GetInputType() const
{
- DBG_ASSERT( nToken==HTML_O_TYPE, "GetInputType: Option nicht TYPE" );
+ DBG_ASSERT( nToken==HTML_O_TYPE, "GetInputType: Option not TYPE" );
return (HTMLInputType)GetEnum( aInputTypeOptEnums, HTML_IT_TEXT );
}
HTMLTableFrame HTMLOption::GetTableFrame() const
{
- DBG_ASSERT( nToken==HTML_O_FRAME, "GetTableFrame: Option nicht FRAME" );
+ DBG_ASSERT( nToken==HTML_O_FRAME, "GetTableFrame: Option not FRAME" );
return (HTMLTableFrame)GetEnum( aTableFrameOptEnums, HTML_TF_VOID );
}
HTMLTableRules HTMLOption::GetTableRules() const
{
- DBG_ASSERT( nToken==HTML_O_RULES, "GetTableRules: Option nicht RULES" );
+ DBG_ASSERT( nToken==HTML_O_RULES, "GetTableRules: Option not RULES" );
return (HTMLTableRules)GetEnum( aTableRulesOptEnums, HTML_TR_NONE );
}
@@ -343,7 +341,7 @@ SvParserState HTMLParser::CallParser()
AddRef();
Continue( 0 );
if( SVPAR_PENDING != eState )
- ReleaseRef(); // dann brauchen wir den Parser nicht mehr!
+ ReleaseRef(); // Parser not needed anymore
return eState;
}
@@ -362,8 +360,8 @@ void HTMLParser::Continue( int nToken )
NextToken( nToken );
if( IsParserWorking() )
- SaveState( 0 ); // bis hierhin abgearbeitet,
- // weiter mit neuem Token!
+ SaveState( 0 ); // continue with new token
+
nToken = GetNextToken();
}
}
@@ -505,7 +503,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
(sal_Char)cChar, eSrcEnc );
if( 0U == cChar )
{
- // #73398#: If the character could not be
+ // If the character could not be
// converted, because a conversion is not
// available, do no conversion at all.
cChar = cOrig;
@@ -533,14 +531,13 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
String sEntity( sEntityBuffer.getStr(), nPos );
cChar = GetHTMLCharName( sEntity );
- // nicht gefunden ( == 0 ), dann Klartext
- // oder ein Zeichen das als Attribut eingefuegt
- // wird
+ // not found ( == 0 ): plain text
+ // or a character which is inserted as attribute
if( 0U == cChar && ';' != nNextCh )
{
DBG_ASSERT( rInput.Tell() - nStreamPos ==
(sal_uLong)(nPos+1L)*GetCharSize(),
- "UTF-8 geht hier schief" );
+ "UTF-8 is failing here" );
for( xub_StrLen i=nPos-1L; i>1L; i-- )
{
nNextCh = sEntityBuffer[i];
@@ -559,19 +556,18 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
}
}
- if( !cChar ) // unbekanntes Zeichen?
+ if( !cChar ) // unknown character?
{
- // dann im Stream zurueck, das '&' als Zeichen
- // einfuegen und mit dem nachfolgenden Zeichen
- // wieder aufsetzen
+ // back in stream, insert '&'
+ // and restart with next character
sTmpBuffer.append( (sal_Unicode)'&' );
DBG_ASSERT( rInput.Tell()-nStreamPos ==
(sal_uLong)(nPos+1)*GetCharSize(),
- "Falsche Stream-Position" );
+ "Wrong stream position" );
DBG_ASSERT( nlLinePos-nLinePos ==
(sal_uLong)(nPos+1),
- "Falsche Zeilen-Position" );
+ "Wrong line position" );
rInput.Seek( nStreamPos );
nlLinePos = nLinePos;
ClearTxtConvContext();
@@ -585,44 +581,43 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
{
if( '>' == cBreak )
{
- // Wenn der Inhalt eines Tags gelesen wird,
- // muessen wir ein Space bzw. - daraus machen
+ // When reading the content of a tag we have
+ // to change it to ' ' or '-'
switch( cChar )
{
case 1U: cChar = ' '; break;
case 2U: cChar = '-'; break;
default:
DBG_ASSERT( cChar==1U,
- "\0x00 sollte doch schon laengt abgefangen sein!" );
+ "\0x00 should be handled already!" );
break;
}
}
else
{
- // Wenn kein Tag gescannt wird, enstprechendes
- // Token zurueckgeben
+ // If not scanning a tag return token
aToken +=
String( sTmpBuffer.makeStringAndClear() );
if( cChar )
{
if( aToken.Len() )
{
- // mit dem Zeichen wieder aufsetzen
+ // restart with character
nNextCh = '&';
DBG_ASSERT( rInput.Tell()-nStreamPos ==
(sal_uLong)(nPos+1)*GetCharSize(),
- "Falsche Stream-Position" );
+ "Wrong stream position" );
DBG_ASSERT( nlLinePos-nLinePos ==
(sal_uLong)(nPos+1),
- "Falsche Zeilen-Position" );
+ "Wrong line position" );
rInput.Seek( nStreamPos );
nlLinePos = nLinePos;
ClearTxtConvContext();
return HTML_TEXTTOKEN;
}
- // Hack: _GetNextChar soll nicht das
- // naechste Zeichen lesen
+ // Hack: _GetNextChar shall not read the
+ // next character
if( ';' != nNextCh )
aToken += ' ';
if( 1U == cChar )
@@ -652,12 +647,11 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
if( cBreak=='>' && (cChar=='\\' || cChar=='\'' ||
cChar=='\"' || cChar==' ') )
{
- // ' und " mussen innerhalb von Tags mit einem
- // gekennzeichnet werden, um sie von ' und " als Klammern
- // um Optionen zu unterscheiden. Logischerweise muss
- // deshalb auch ein \ gekeenzeichnet werden. Ausserdem
- // schuetzen wir ein Space, weil es kein Trennzeichen
- // zwischen Optionen ist.
+ // ' and " have to be escaped withing tags to separate
+ // them from ' and " enclosing options.
+ // \ has to be escaped as well.
+ // Space is protected because it's not a delimiter between
+ // options.
sTmpBuffer.append( (sal_Unicode)'\\' );
if( MAX_LEN == sTmpBuffer.getLength() )
aToken += String(sTmpBuffer.makeStringAndClear());
@@ -669,14 +663,13 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
}
else if( SVPAR_PENDING==eState && '>'!=cBreak )
{
- // Mit dem '&' Zeichen wieder aufsetzen, der Rest
- // wird als Texttoken zurueckgegeben.
+ // Restart with '&', the remainder is returned as
+ // text token.
if( aToken.Len() || sTmpBuffer.getLength() )
{
- // Der bisherige Text wird von _GetNextChar()
- // zurueckgegeben und beim naechsten Aufruf wird
- // ein neues Zeichen gelesen. Also muessen wir uns
- // noch vor das & stellen.
+ // _GetNextChar() returns the previous text and
+ // during the next execution a new character is read.
+ // Thus we have to position in front of the '&'.
nNextCh = 0U;
rInput.Seek( nStreamPos-(sal_uInt32)GetCharSize() );
nlLinePos = nLinePos-1;
@@ -720,10 +713,6 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
case sal_Unicode(EOF):
if( rInput.IsEof() )
{
-// Das macht hier keinen Sinn, oder doch: Zumindest wird
-// abcä<EOF> nicht angezeigt, also lassen wir das in Zukunft.
-// if( '>' != cBreak )
-// eState = SVPAR_ACCEPTED;
bContinue = sal_False;
}
else
@@ -737,18 +726,18 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
if( '>'==cBreak )
sTmpBuffer.append( nNextCh );
else
- bContinue = sal_False; // Abbrechen, String zusammen
+ bContinue = sal_False; // break, String zusammen
break;
case '\f':
if( '>' == cBreak )
{
- // Beim Scannen von Optionen wie ein Space behandeln
+ // If scanning options treat it like a space, ...
sTmpBuffer.append( (sal_Unicode)' ' );
}
else
{
- // sonst wird es ein eigenes Token
+ // otherwise it's a separate token.
bContinue = sal_False;
}
break;
@@ -757,7 +746,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
case '\n':
if( '>'==cBreak )
{
- // #26979# cr/lf in Tag wird in _GetNextToken() behandeln
+ // cr/lf in tag is handled in _GetNextToken()
sTmpBuffer.append( nNextCh );
break;
}
@@ -766,9 +755,8 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
bContinue = sal_False;
break;
}
- // Bug 18984: CR-LF -> Blank
- // Folge von CR/LF/BLANK/TAB nur in ein Blank wandeln
- // kein break!!
+ // Reduce sequence of CR/LF/BLANK/TAB to a single blank
+ // no break!!
case '\t':
if( '\t'==nNextCh && bReadPRE && '>'!=cBreak )
{
@@ -776,7 +764,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
bContinue = sal_False;
break;
}
- // kein break
+ // no break
case '\x0b':
if( '\x0b'==nNextCh && (bReadPRE || bReadXMP ||bReadListing) &&
'>'!=cBreak )
@@ -784,27 +772,26 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
break;
}
nNextCh = ' ';
- // kein break;
+ // no break;
case ' ':
sTmpBuffer.append( nNextCh );
if( '>'!=cBreak && (!bReadListing && !bReadXMP &&
!bReadPRE && !bReadTextArea) )
{
- // alle Folgen von Blanks/Tabs/CR/LF zu einem Blank umwandeln
+ // Reduce sequences of Blanks/Tabs/CR/LF to a single blank
do {
if( sal_Unicode(EOF) == (nNextCh = GetNextChar()) &&
rInput.IsEof() )
{
if( aToken.Len() || sTmpBuffer.getLength() > 1L )
{
- // ausser den Blanks wurde noch etwas geselen
+ // Have seen s.th. aside from blanks?
aToken += String(sTmpBuffer.makeStringAndClear());
return HTML_TEXTTOKEN;
}
else
- // nur Blanks gelesen: dann darf kein Text
- // mehr zurueckgegeben werden und _GetNextToken
- // muss auf EOF laufen
+ // Only read blanks: no text must be returned
+ // and _GetNextToken has to read until EOF
return 0;
}
} while ( ' ' == nNextCh || '\t' == nNextCh ||
@@ -822,7 +809,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
else
{
do {
- // alle anderen Zeichen kommen in den Text
+ // All remaining characters make their way into the text.
sTmpBuffer.append( nNextCh );
if( MAX_LEN == sTmpBuffer.getLength() )
{
@@ -866,8 +853,8 @@ int HTMLParser::_GetNextRawToken()
if( bEndTokenFound )
{
- // beim letzten Aufruf haben wir das End-Token bereits gefunden,
- // deshalb muessen wir es nicht noch einmal suchen
+ // During the last execution we already found the end token,
+ // thus we don't have to search it again.
bReadScript = sal_False;
bReadStyle = sal_False;
aEndToken.Erase();
@@ -876,7 +863,7 @@ int HTMLParser::_GetNextRawToken()
return 0;
}
- // per default geben wir HTML_RAWDATA zurueck
+ // Default return value: HTML_RAWDATA
int bContinue = sal_True;
int nToken = HTML_RAWDATA;
SaveState( 0 );
@@ -887,17 +874,17 @@ int HTMLParser::_GetNextRawToken()
{
case '<':
{
- // Vielleicht haben wir das Ende erreicht
+ // Maybe we've reached the end.
- // das bisher gelesene erstmal retten
+ // Save what we have read previously...
aToken += String(sTmpBuffer.makeStringAndClear());
- // und die Position im Stream merken
+ // and remember position in stream.
sal_uLong nStreamPos = rInput.Tell();
sal_uLong nLineNr = GetLineNr();
sal_uLong nLinePos = GetLinePos();
- // Start eines End-Token?
+ // Start of an end token?
int bOffState = sal_False;
if( '/' == (nNextCh = GetNextChar()) )
{
@@ -910,7 +897,7 @@ int HTMLParser::_GetNextRawToken()
nNextCh = GetNextChar();
}
- // jetzt die Buchstaben danach lesen
+ // Read following letters
while( (HTML_ISALPHA(nNextCh) || '-'==nNextCh) &&
IsParserWorking() && sTmpBuffer.getLength() < MAX_LEN )
{
@@ -934,10 +921,9 @@ int HTMLParser::_GetNextRawToken()
}
else
{
- // ein Script muss mit "</SCRIPT>" aufhoehren, wobei
- // wir es mit dem ">" aus sicherheitsgruenden
- // erstmal nicht so genau nehmen
- bDone = bOffState && // '>'==nNextCh &&
+ // A script has to end with "</SCRIPT>". But
+ // ">" is optional for security reasons
+ bDone = bOffState &&
COMPARE_EQUAL == ( bReadScript
? aTok.CompareToAscii(OOO_STRING_SVTOOLS_HTML_script)
: aTok.CompareTo(aEndToken) );
@@ -946,14 +932,13 @@ int HTMLParser::_GetNextRawToken()
if( bReadComment && '>'==nNextCh && aTok.Len() >= 2 &&
aTok.Copy( aTok.Len()-2 ).EqualsAscii( "--" ) )
{
- // hier ist ein Kommentar der Art <!-----> zuende
+ // End of comment of style <!----->
bReadComment = sal_False;
}
}
else
{
- // ein Style-Sheet kann mit </STYLE>, </HEAD> oder
- // <BODY> aughoehren
+ // Style sheets can be closed by </STYLE>, </HEAD> or <BODY>
if( bOffState )
bDone = aTok.CompareToAscii(OOO_STRING_SVTOOLS_HTML_style)
== COMPARE_EQUAL ||
@@ -966,17 +951,16 @@ int HTMLParser::_GetNextRawToken()
if( bDone )
{
- // das war's, jetzt muessen wir gegebenenfalls den
- // bisher gelesenen String zurueckgeben und dnach normal
- // weitermachen
+ // Done! Return the previously read string (if requested)
+ // and continue.
bContinue = sal_False;
- // nToken==0 heisst, dass _GetNextToken gleich weiterliest
+ // nToken==0 means, _GetNextToken continues to read
if( !aToken.Len() && (bReadStyle || bReadScript) )
{
- // wir koennen sofort die Umgebung beeden und
- // das End-Token parsen
+ // Immediately close environment (or context?)
+ // and parse the end token
bReadScript = sal_False;
bReadStyle = sal_False;
aEndToken.Erase();
@@ -984,25 +968,24 @@ int HTMLParser::_GetNextRawToken()
}
else
{
- // wir muessen bReadScript/bReadStyle noch am
- // Leben lassen und koennen erst beim naechsten
- // mal das End-Token Parsen
+ // Keep bReadScript/bReadStyle alive
+ // and parse end token during next execution
bEndTokenFound = sal_True;
}
- // jetzt fahren wir im Stream auf das '<' zurueck
+ // Move backwards in stream to '<'
rInput.Seek( nStreamPos );
SetLineNr( nLineNr );
SetLinePos( nLinePos );
ClearTxtConvContext();
nNextCh = '<';
- // den String wollen wir nicht an das Token haengen
+ // Don't append string to token.
sTmpBuffer.setLength( 0L );
}
else
{
- // "</" merken, alles andere steht noch im buffer
+ // remember "</" , everything else we find in the buffer
aToken += (sal_Unicode)'<';
if( bOffState )
aToken += (sal_Unicode)'/';
@@ -1035,20 +1018,20 @@ int HTMLParser::_GetNextRawToken()
break;
case '\r':
- // \r\n? beendet das aktuelle Text-Token (auch wenn es leer ist)
+ // \r\n? closes the current text token (even if it's empty)
nNextCh = GetNextChar();
if( nNextCh=='\n' )
nNextCh = GetNextChar();
bContinue = sal_False;
break;
case '\n':
- // \n beendet das aktuelle Text-Token (auch wenn es leer ist)
+ // \n closes the current text token (even if it's empty)
nNextCh = GetNextChar();
bContinue = sal_False;
break;
case sal_Unicode(EOF):
- // eof beendet das aktuelle Text-Token und tut so, als ob
- // ein End-Token gelesen wurde
+ // eof closes the current text token and behaves like having read
+ // an end token
if( rInput.IsEof() )
{
bContinue = sal_False;
@@ -1065,9 +1048,9 @@ int HTMLParser::_GetNextRawToken()
}
break;
}
- // kein break
+ // no break
default:
- // alle anderen Zeichen landen im Buffer
+ // all remaining characters are appended to the buffer
sTmpBuffer.append( nNextCh );
break;
}
@@ -1088,26 +1071,26 @@ int HTMLParser::_GetNextRawToken()
return nToken;
}
-// scanne das naechste Token,
+// Scan next token
int HTMLParser::_GetNextToken()
{
int nRet = 0;
sSaveToken.Erase();
- // die Optionen loeschen
+ // Delete options
if( pOptions->Count() )
pOptions->DeleteAndDestroy( 0, pOptions->Count() );
- if( !IsParserWorking() ) // wenn schon Fehler, dann nicht weiter!
+ if( !IsParserWorking() ) // Don't continue if already an error occured
return 0;
sal_Bool bReadNextCharSave = bReadNextChar;
if( bReadNextChar )
{
DBG_ASSERT( !bEndTokenFound,
- "</SCRIPT> gelesen und trotzdem noch ein Zeichen lesen?" );
+ "Read a character despite </SCRIPT> was read?" );
nNextCh = GetNextChar();
- if( !IsParserWorking() ) // wenn schon Fehler, dann nicht weiter!
+ if( !IsParserWorking() ) // Don't continue if already an error occured
return 0;
bReadNextChar = sal_False;
}
@@ -1135,7 +1118,7 @@ int HTMLParser::_GetNextToken()
bOffState = sal_True;
nNextCh = GetNextChar();
}
- if( HTML_ISALPHA( nNextCh ) || '!'==nNextCh ) // fix #26984#
+ if( HTML_ISALPHA( nNextCh ) || '!'==nNextCh )
{
::rtl::OUStringBuffer sTmpBuffer;
do {
@@ -1149,7 +1132,7 @@ int HTMLParser::_GetNextToken()
if( sTmpBuffer.getLength() )
aToken += String(sTmpBuffer.makeStringAndClear());
- // Blanks ueberlesen
+ // Skip blanks
while( HTML_ISSPACE( nNextCh ) && IsParserWorking() )
nNextCh = GetNextChar();
@@ -1160,35 +1143,33 @@ int HTMLParser::_GetNextToken()
break;
}
- // suche das Token in der Tabelle:
+ // Search token in table:
sSaveToken = aToken;
aToken.ToUpperAscii();
if( 0 == (nRet = GetHTMLToken( aToken )) )
- // Unknown Control
+ // Unknown control
nRet = HTML_UNKNOWNCONTROL_ON;
- // Wenn es ein Token zum ausschalten ist ...
+ // If it's a token which can be switched off...
if( bOffState )
{
if( HTML_TOKEN_ONOFF & nRet )
{
- // und es ein Off-Token gibt, das daraus machen
+ // and there is an off token, return off token instead
++nRet;
}
else if( HTML_LINEBREAK!=nRet )
{
- // und es kein Off-Token gibt, ein unbekanntes
- // Token daraus machen (ausser </BR>, das wird
- // wie <BR> behandelt
+ // and there is no off token, return unknown token.
+ // (except for </BR>, that is treated like <BR>)
nRet = HTML_UNKNOWNCONTROL_OFF;
}
}
if( nRet == HTML_COMMENT )
{
- // fix: sSaveToken wegen Gross-/Kleinschreibung
- // als Anfang des Kommentars benutzen und ein
- // Space anhaengen.
+ // fix: due to being case sensitive use sSaveToken as start of comment
+ // and append a blank.
aToken = sSaveToken;
if( '>'!=nNextCh )
aToken += (sal_Unicode)' ';
@@ -1198,8 +1179,7 @@ int HTMLParser::_GetNextToken()
xub_StrLen nCStrLen = 0;
sal_Bool bDone = sal_False;
- // bis zum schliessenden --> lesen. wenn keins gefunden
- // wurde beim der ersten > wieder aufsetzen
+ // Read until closing -->. If not found restart at first >
while( !bDone && !rInput.IsEof() && IsParserWorking() )
{
if( '>'==nNextCh )
@@ -1234,11 +1214,11 @@ int HTMLParser::_GetNextToken()
}
else
{
- // den TokenString koennen wir jetzt verwerfen
+ // TokenString not needed anymore
aToken.Erase();
}
- // dann lesen wir mal alles bis zur schliessenden '>'
+ // Read until closing '>'
if( '>' != nNextCh && IsParserWorking() )
{
ScanText( '>' );
@@ -1252,8 +1232,8 @@ int HTMLParser::_GetNextToken()
}
if( sal_Unicode(EOF) == nNextCh && rInput.IsEof() )
{
- // zurueck hinter die < gehen und dort neu
- // aufsetzen, das < als Text zurueckgeben
+ // Move back in front of < and restart there.
+ // Return < as text.
rInput.Seek( nStreamPos );
SetLineNr( nLineNr );
SetLinePos( nLinePos );
@@ -1277,8 +1257,8 @@ int HTMLParser::_GetNextToken()
ScanText( '>' );
if( sal_Unicode(EOF) == nNextCh && rInput.IsEof() )
{
- // zurueck hinter die < gehen und dort neu
- // aufsetzen, das < als Text zurueckgeben
+ // Move back in front of < and restart there.
+ // Return < as text.
rInput.Seek( nStreamPos );
SetLineNr( nLineNr );
SetLinePos( nLinePos );
@@ -1302,8 +1282,7 @@ int HTMLParser::_GetNextToken()
sal_uLong nCLineNr = GetLineNr(), nCLinePos = GetLinePos();
sal_Bool bDone = sal_False;
- // bis zum schliessenden %> lesen. wenn keins gefunden
- // wurde beim der ersten > wieder aufsetzen
+ // Read until closing %>. If not found restart at first >.
while( !bDone && !rInput.IsEof() && IsParserWorking() )
{
bDone = '>'==nNextCh && aToken.Len() >= 1 &&
@@ -1358,9 +1337,8 @@ int HTMLParser::_GetNextToken()
if( !bReadTextArea )
{
bReadScript = sal_False;
- // JavaScript kann den Stream veraendern
- // also muss das letzte Zeichen nochmals
- // gelesen werden
+ // JavaScript might modify the stream,
+ // thus the last character has to be read again.
bReadNextChar = sal_True;
bNextCh = sal_False;
}
@@ -1386,14 +1364,14 @@ int HTMLParser::_GetNextToken()
}
else
{
- // normalen Text lesen
+ // Read normal text.
goto scan_text;
}
break;
case '\f':
- // Form-Feeds werden jetzt extra nach oben gereicht
- nRet = HTML_LINEFEEDCHAR; // !!! eigentlich FORMFEEDCHAR
+ // form feeds are passed upwards separately
+ nRet = HTML_LINEFEEDCHAR; // !!! should be FORMFEEDCHAR
break;
case '\n':
@@ -1410,24 +1388,24 @@ int HTMLParser::_GetNextToken()
nRet = HTML_NEWPARA;
break;
}
- // kein break !
+ // no break !
case '\t':
if( bReadPRE )
{
nRet = HTML_TABCHAR;
break;
}
- // kein break !
+ // no break !
case ' ':
- // kein break !
+ // no break !
default:
scan_text:
- // es folgt "normaler" Text
+ // "normal" text to come
nRet = ScanText();
bNextCh = 0 == aToken.Len();
- // der Text sollte noch verarbeitet werden
+ // the text should be processed
if( !bNextCh && eState == SVPAR_PENDING )
{
eState = SVPAR_WORKING;
@@ -1450,7 +1428,7 @@ scan_text:
} while( !nRet && SVPAR_WORKING == eState );
if( SVPAR_PENDING == eState )
- nRet = -1; // irgendwas ungueltiges
+ nRet = -1; // s.th. invalid
return nRet;
}
@@ -1476,18 +1454,17 @@ void HTMLParser::UnescapeToken()
}
}
-// hole die Optionen
const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
{
- // wenn die Option fuer das aktuelle Token schon einmal
- // geholt wurden, geben wir sie noch einmal zurueck
+ // If the options for the current token have already been returned,
+ // return them once again.
if( pOptions->Count() )
return pOptions;
xub_StrLen nPos = 0;
while( nPos < aToken.Len() )
{
- // ein Zeichen ? Dann faengt hier eine Option an
+ // A letter? Option beginning here.
if( HTML_ISALPHA( aToken.GetChar(nPos) ) )
{
int nToken;
@@ -1495,24 +1472,22 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
xub_StrLen nStt = nPos;
sal_Unicode cChar = 0;
- // Eigentlich sind hier nur ganz bestimmte Zeichen erlaubt.
- // Netscape achtet aber nur auf "=" und Leerzeichen (siehe
- // Mozilla: PA_FetchRequestedNameValues in
- // lipparse/pa_mdl.c
+ // Actually only certain characters allowed.
+ // Netscape only looks for "=" and white space (c.f.
+ // Mozilla: PA_FetchRequestedNameValues in lipparse/pa_mdl.c)
while( nPos < aToken.Len() && '=' != (cChar=aToken.GetChar(nPos)) &&
HTML_ISPRINTABLE(cChar) && !HTML_ISSPACE(cChar) )
nPos++;
String sName( aToken.Copy( nStt, nPos-nStt ) );
- // die PlugIns wollen die TokenName im "Original" haben
- // also nur fuers Suchen in UpperCase wandeln
+ // PlugIns require original token name. Convert to upper case only for searching.
String sNameUpperCase( sName );
sNameUpperCase.ToUpperAscii();
- nToken = GetHTMLOption( sNameUpperCase ); // der Name ist fertig
+ nToken = GetHTMLOption( sNameUpperCase ); // Name is ready
DBG_ASSERTWARNING( nToken!=HTML_O_UNKNOWN,
- "GetOption: unbekannte HTML-Option" );
+ "GetOption: unknown HTML option" );
sal_Bool bStripCRLF = (nToken < HTML_OPTION_SCRIPT_START ||
nToken >= HTML_OPTION_SCRIPT_END) &&
(!pNoConvertToken || nToken != *pNoConvertToken);
@@ -1522,7 +1497,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
HTML_ISSPACE(cChar) ) )
nPos++;
- // hat die Option auch einen Wert?
+ // Option with value?
if( nPos!=aToken.Len() && '='==cChar )
{
nPos++;
@@ -1583,8 +1558,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
}
else
{
- // hier sind wir etwas laxer als der
- // Standard und erlauben alles druckbare
+ // More liberal than the standard: allow all printable characters
sal_Bool bEscape = sal_False;
sal_Bool bDone = sal_False;
while( nPos < aToken.Len() && !bDone )
@@ -1633,7 +1607,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
}
}
- // Wir kennen das Token und koennen es Speichern
+ // Token is known and can be saved
HTMLOption *pOption =
new HTMLOption(
sal::static_int_cast< sal_uInt16 >(nToken), sName, aValue );
@@ -1642,7 +1616,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
}
else
- // white space un unerwartete Zeichen ignorieren wie
+ // Ignore white space and unexpected characters
nPos++;
}
@@ -1654,12 +1628,12 @@ int HTMLParser::FilterPRE( int nToken )
switch( nToken )
{
#ifdef HTML_BEHAVIOUR
- // diese werden laut Definition zu LFs
+ // These become LFs according to the definition
case HTML_PARABREAK_ON:
case HTML_LINEBREAK:
nToken = HTML_NEWPARA;
#else
- // in Netscape zeigen sie aber nur in nicht-leeren Absaetzen Wirkung
+ // in Netscape they only have impact in not empty paragraphs
case HTML_PARABREAK_ON:
nToken = HTML_LINEBREAK;
case HTML_LINEBREAK:
@@ -1680,7 +1654,7 @@ int HTMLParser::FilterPRE( int nToken )
nToken = HTML_TEXTTOKEN;
}
break;
- // diese bleiben erhalten
+ // Keep those
case HTML_TEXTTOKEN:
nPre_LinePos += aToken.Len();
break;
@@ -1815,7 +1789,7 @@ int HTMLParser::FilterPRE( int nToken )
break;
- // der Rest wird als unbekanntes Token behandelt
+ // The remainder is treated as an unknown token.
default:
if( nToken )
{
@@ -1842,7 +1816,7 @@ int HTMLParser::FilterXMP( int nToken )
case HTML_TEXTTOKEN:
case HTML_NONBREAKSPACE:
case HTML_SOFTHYPH:
- break; // bleiben erhalten
+ break; // kept
default:
if( nToken )
@@ -1883,7 +1857,7 @@ int HTMLParser::FilterListing( int nToken )
case HTML_TEXTTOKEN:
case HTML_NONBREAKSPACE:
case HTML_SOFTHYPH:
- break; // bleiben erhalten
+ break; // kept
default:
if( nToken )
@@ -1905,16 +1879,14 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
sal_Bool bSwitchToUCS2,
rtl_TextEncoding eEnc )
{
- // Einer der folgenden regulaeren Ausdrucke muss sich auf den String
- // anwenden lassen, damit das Dok ein HTML-Dokument ist.
+ // If the string matches one of the following regular expressions then
+ // the document is a HTML document.
//
// ^[^<]*<[^ \t]*[> \t]
// -------
// ^<!
//
- // wobei der unterstrichene Teilausdruck einem HTML-Token
- // ensprechen muss
-
+ // where the underlined subexpression has to be a HTML token
ByteString sCmp;
sal_Bool bUCS2B = sal_False;
if( bSwitchToUCS2 )
@@ -1971,14 +1943,13 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
sCmp.ToUpperAscii();
- // Ein HTML-Dokument muss in der ersten Zeile ein '<' besitzen
+ // A HTML document must have a '<' in the first line
xub_StrLen nStart = sCmp.Search( '<' );
if( STRING_NOTFOUND == nStart )
return sal_False;
nStart++;
- // danach duerfen beliebige andere Zeichen bis zu einem blank oder
- // '>' kommen
+ // followed by arbitrary characters followed by a blank or '>'
sal_Char c;
xub_StrLen nPos;
for( nPos = nStart; nPos<sCmp.Len(); nPos++ )
@@ -1987,31 +1958,30 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
break;
}
- // wenn das Dokeument hinter dem < aufhoert ist es wohl kein HTML
+ // If the document ends after < it's no HTML
if( nPos==nStart )
return sal_False;
- // die Zeichenkette nach dem '<' muss ausserdem ein bekanntes
- // HTML Token sein. Damit die Ausgabe eines DOS-dir-Befehls nicht
- // als HTML interpretiert wird, wird ein <DIR> jedoch nicht als HTML
- // interpretiert.
+ // the string following '<' has to be a known HTML token.
+ // <DIR> is not interpreted as HTML. Otherwise the output of the DOS command "DIR"
+ // could be interpreted as HTML.
String sTest( sCmp.Copy( nStart, nPos-nStart ), RTL_TEXTENCODING_ASCII_US );
int nTok = GetHTMLToken( sTest );
if( 0 != nTok && HTML_DIRLIST_ON != nTok )
return sal_True;
- // oder es handelt sich um ein "<!" ganz am Anfang der Datei (fix #27092#)
+ // "<!" at the very beginning of the file?
if( nStart == 1 && '!' == sCmp.GetChar( 1 ) )
return sal_True;
- // oder wir finden irgendwo ein <HTML> in den ersten 80 Zeichen
+ // <HTML> somewhere in the first 80 characters of the document
nStart = sCmp.Search( OOO_STRING_SVTOOLS_HTML_html );
if( nStart!=STRING_NOTFOUND &&
nStart>0 && '<'==sCmp.GetChar(nStart-1) &&
nStart+4 < sCmp.Len() && '>'==sCmp.GetChar(nStart+4) )
return sal_True;
- // sonst ist es wohl doch eher kein HTML-Dokument
+ // Else it's rather not a HTML document
return sal_False;
}
@@ -2170,7 +2140,7 @@ bool HTMLParser::ParseMetaOptionsImpl(
if ( bHTTPEquiv && i_pHTTPHeader )
{
- // #57232#: Netscape seems to just ignore a closing ", so we do too
+ // Netscape seems to just ignore a closing ", so we do too
if ( aContent.Len() && '"' == aContent.GetChar( aContent.Len()-1 ) )
{
aContent.Erase( aContent.Len() - 1 );
@@ -2289,7 +2259,7 @@ bool HTMLParser::ParseMetaOptions(
rtl_isOctetTextEncoding( eEnc ) &&
rtl_isOctetTextEncoding( GetSrcEncoding() ) )
{
- eEnc = GetExtendedCompatibilityTextEncoding( eEnc ); // #89973#
+ eEnc = GetExtendedCompatibilityTextEncoding( eEnc );
SetSrcEncoding( eEnc );
}
More information about the Libreoffice-commits
mailing list