[Libreoffice-commits] .: 2 commits - svtools/source
Christina Rossmanith
crossmanith at kemper.freedesktop.org
Mon Mar 21 04:51:51 PDT 2011
svtools/source/svhtml/parhtml.cxx | 56 ++++++++++++++------------------------
1 file changed, 22 insertions(+), 34 deletions(-)
New commits:
commit ce35bd00685cb0a8fdabe08660aac599e14a461f
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Fri Mar 18 21:28:43 2011 +0100
Removed commented code
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index aac5cf6..58d1f5d 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -57,12 +57,10 @@ using namespace ::com::sun::star;
const sal_Int32 MAX_LEN( 1024L );
-//static sal_Unicode sTmpBuffer[ MAX_LEN+1 ];
const sal_Int32 MAX_MACRO_LEN( 1024 );
const sal_Int32 MAX_ENTITY_LEN( 8L );
-/* */
// Tabellen zum Umwandeln von Options-Werten in Strings
@@ -113,7 +111,6 @@ static HTMLOptionEnum const aTableRulesOptEnums[] =
SV_IMPL_PTRARR(HTMLOptions,HTMLOptionPtr)
-/* */
sal_uInt16 HTMLOption::GetEnum( const HTMLOptionEnum *pOptEnums, sal_uInt16 nDflt ) const
{
@@ -269,7 +266,7 @@ void HTMLOption::GetColor( Color& rColor ) const
xub_StrLen nPos = 0;
for( sal_uInt32 i=0; i<6; i++ )
{
- // MIB 26.06.97: Wie auch immer Netscape Farbwerte ermittelt,
+ // 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...
@@ -312,8 +309,6 @@ HTMLTableRules HTMLOption::GetTableRules() const
return (HTMLTableRules)GetEnum( aTableRulesOptEnums, HTML_TR_NONE );
}
-/* */
-
HTMLParser::HTMLParser( SvStream& rIn, int bReadNewDoc )
: SvParser( rIn )
{
@@ -571,8 +566,6 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
// wieder aufsetzen
sTmpBuffer.append( (sal_Unicode)'&' );
-// rInput.SeekRel( -(long)(++nPos*GetCharSize()) );
-// nlLinePos -= nPos;
DBG_ASSERT( rInput.Tell()-nStreamPos ==
(sal_uLong)(nPos+1)*GetCharSize(),
"Falsche Stream-Position" );
@@ -616,8 +609,6 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
{
// mit dem Zeichen wieder aufsetzen
nNextCh = '&';
-// rInput.SeekRel( -(long)(++nPos*GetCharSize()) );
-// nlLinePos -= nPos;
DBG_ASSERT( rInput.Tell()-nStreamPos ==
(sal_uLong)(nPos+1)*GetCharSize(),
"Falsche Stream-Position" );
@@ -649,8 +640,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
else
nNextCh = 0U;
}
- // MIB 03/02/2000: &{...};-JavaScript-Macros are not
- // supported any longer.
+ // &{...};-JavaScript-Macros are not supported any longer.
else if( IsParserWorking() )
{
sTmpBuffer.append( (sal_Unicode)'&' );
@@ -730,7 +720,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
case sal_Unicode(EOF):
if( rInput.IsEof() )
{
-// MIB 20.11.98: Das macht hier keinen Sinn, oder doch: Zumindest wird
+// Das macht hier keinen Sinn, oder doch: Zumindest wird
// abcä<EOF> nicht angezeigt, also lassen wir das in Zukunft.
// if( '>' != cBreak )
// eState = SVPAR_ACCEPTED;
@@ -1509,16 +1499,14 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
// Netscape achtet aber nur auf "=" und Leerzeichen (siehe
// Mozilla: PA_FetchRequestedNameValues in
// lipparse/pa_mdl.c
-// while( nPos < aToken.Len() &&
-// ( '-'==(c=aToken[nPos]) || isalnum(c) || '.'==c || '_'==c) )
while( nPos < aToken.Len() && '=' != (cChar=aToken.GetChar(nPos)) &&
HTML_ISPRINTABLE(cChar) && !HTML_ISSPACE(cChar) )
nPos++;
String sName( aToken.Copy( nStt, nPos-nStt ) );
-//JP 23.03.97: die PlugIns wollen die TokenName im "Original" haben
-// also nur fuers Suchen in UpperCase wandeln
+ // die PlugIns wollen die TokenName im "Original" haben
+ // also nur fuers Suchen in UpperCase wandeln
String sNameUpperCase( sName );
sNameUpperCase.ToUpperAscii();
commit 3cec6ebf1872deca2ef0c1ea68e4c9bc190d6d9e
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date: Fri Mar 18 21:19:34 2011 +0100
renamed variable bWeiter to bContinue
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index 5a8cdc5..aac5cf6 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -450,11 +450,11 @@ int HTMLParser::FilterToken( int nToken )
int HTMLParser::ScanText( const sal_Unicode cBreak )
{
::rtl::OUStringBuffer sTmpBuffer( MAX_LEN );
- int bWeiter = sal_True;
+ int bContinue = sal_True;
int bEqSignFound = sal_False;
sal_Unicode cQuote = 0U;
- while( bWeiter && IsParserWorking() )
+ while( bContinue && IsParserWorking() )
{
int bNextCh = sal_True;
switch( nNextCh )
@@ -734,7 +734,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
// abcä<EOF> nicht angezeigt, also lassen wir das in Zukunft.
// if( '>' != cBreak )
// eState = SVPAR_ACCEPTED;
- bWeiter = sal_False;
+ bContinue = sal_False;
}
else
{
@@ -747,7 +747,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
if( '>'==cBreak )
sTmpBuffer.append( nNextCh );
else
- bWeiter = sal_False; // Abbrechen, String zusammen
+ bContinue = sal_False; // Abbrechen, String zusammen
break;
case '\f':
@@ -759,7 +759,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
else
{
// sonst wird es ein eigenes Token
- bWeiter = sal_False;
+ bContinue = sal_False;
}
break;
@@ -773,7 +773,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
}
else if( bReadListing || bReadXMP || bReadPRE || bReadTextArea )
{
- bWeiter = sal_False;
+ bContinue = sal_False;
break;
}
// Bug 18984: CR-LF -> Blank
@@ -783,7 +783,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
if( '\t'==nNextCh && bReadPRE && '>'!=cBreak )
{
// In <PRE>: Tabs nach oben durchreichen
- bWeiter = sal_False;
+ bContinue = sal_False;
break;
}
// kein break
@@ -828,7 +828,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
bEqSignFound = sal_False;
if( (nNextCh==cBreak && !cQuote) ||
(sal_uLong(aToken.Len()) + MAX_LEN) > sal_uLong(STRING_MAXLEN & ~1 ))
- bWeiter = sal_False;
+ bContinue = sal_False;
else
{
do {
@@ -860,7 +860,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
if( MAX_LEN == sTmpBuffer.getLength() )
aToken += String(sTmpBuffer.makeStringAndClear());
- if( bWeiter && bNextCh )
+ if( bContinue && bNextCh )
nNextCh = GetNextChar();
}
@@ -887,10 +887,10 @@ int HTMLParser::_GetNextRawToken()
}
// per default geben wir HTML_RAWDATA zurueck
- int bWeiter = sal_True;
+ int bContinue = sal_True;
int nToken = HTML_RAWDATA;
SaveState( 0 );
- while( bWeiter && IsParserWorking() )
+ while( bContinue && IsParserWorking() )
{
int bNextCh = sal_True;
switch( nNextCh )
@@ -980,7 +980,7 @@ int HTMLParser::_GetNextRawToken()
// bisher gelesenen String zurueckgeben und dnach normal
// weitermachen
- bWeiter = sal_False;
+ bContinue = sal_False;
// nToken==0 heisst, dass _GetNextToken gleich weiterliest
if( !aToken.Len() && (bReadStyle || bReadScript) )
@@ -1049,19 +1049,19 @@ int HTMLParser::_GetNextRawToken()
nNextCh = GetNextChar();
if( nNextCh=='\n' )
nNextCh = GetNextChar();
- bWeiter = sal_False;
+ bContinue = sal_False;
break;
case '\n':
// \n beendet das aktuelle Text-Token (auch wenn es leer ist)
nNextCh = GetNextChar();
- bWeiter = sal_False;
+ bContinue = sal_False;
break;
case sal_Unicode(EOF):
// eof beendet das aktuelle Text-Token und tut so, als ob
// ein End-Token gelesen wurde
if( rInput.IsEof() )
{
- bWeiter = sal_False;
+ bContinue = sal_False;
if( aToken.Len() || sTmpBuffer.getLength() )
{
bEndTokenFound = sal_True;
@@ -1082,11 +1082,11 @@ int HTMLParser::_GetNextRawToken()
break;
}
- if( (!bWeiter && sTmpBuffer.getLength() > 0L) ||
+ if( (!bContinue && sTmpBuffer.getLength() > 0L) ||
MAX_LEN == sTmpBuffer.getLength() )
aToken += String(sTmpBuffer.makeStringAndClear());
- if( bWeiter && bNextCh )
+ if( bContinue && bNextCh )
nNextCh = GetNextChar();
}
More information about the Libreoffice-commits
mailing list