[Libreoffice-commits] .: 4 commits - svtools/inc svtools/source

Kohei Yoshida kohei at kemper.freedesktop.org
Fri Jul 22 19:06:27 PDT 2011


 svtools/inc/svtools/parhtml.hxx    |  110 ++++++-------
 svtools/source/svhtml/htmlsupp.cxx |   22 +-
 svtools/source/svhtml/parhtml.cxx  |  299 ++++++++++++++++++-------------------
 3 files changed, 214 insertions(+), 217 deletions(-)

New commits:
commit 490480f174b00d5f4d86c612193a5919bdc50ec2
Author: Kohei Yoshida <kyoshida at novell.com>
Date:   Fri Jul 22 15:39:18 2011 -0400

    Converted HTMLOptions to boost::ptr_vector.

diff --git a/svtools/inc/svtools/parhtml.hxx b/svtools/inc/svtools/parhtml.hxx
index 9a30e04..a13567e 100644
--- a/svtools/inc/svtools/parhtml.hxx
+++ b/svtools/inc/svtools/parhtml.hxx
@@ -35,6 +35,7 @@
 #include <svl/svarray.hxx>
 #include <svtools/svparser.hxx>
 
+#include <boost/ptr_container/ptr_vector.hpp>
 
 namespace com { namespace sun { namespace star {
     namespace document {
@@ -130,11 +131,13 @@ public:
     //SvxAdjust GetAdjust() const;						// <P,TH,TD ALIGN=>
 };
 
-typedef HTMLOption* HTMLOptionPtr;
-SV_DECL_PTRARR(HTMLOptions,HTMLOptionPtr,16,16)
+typedef ::boost::ptr_vector<HTMLOption> HTMLOptions;
 
 class SVT_DLLPUBLIC HTMLParser : public SvParser
 {
+private:
+    mutable HTMLOptions maOptions; // die Optionen des Start-Tags
+
     bool bNewDoc        : 1;        // neues Doc lesen ?
     bool bIsInHeader    : 1;        // scanne Header-Bereich
     bool bIsInBody      : 1;        // scanne Body-Bereich
@@ -152,7 +155,6 @@ class SVT_DLLPUBLIC HTMLParser : public SvParser
 
     sal_uInt32 nPre_LinePos;			// Pos in der Line im PRE-Tag
 
-    HTMLOptions *pOptions;			// die Optionen des Start-Tags
     String aEndToken;
 
 protected:
@@ -223,7 +225,7 @@ public:
     // Ermitteln der Optionen. pNoConvertToken ist das optionale Token
     // einer Option, fuer die CR/LFs nicht aus dem Wert der Option
     // geloescht werden.
-    const HTMLOptions *GetOptions( sal_uInt16 *pNoConvertToken=0 ) const;
+    const HTMLOptions& GetOptions( sal_uInt16 *pNoConvertToken=0 ) const;
 
     // fuers asynchrone lesen aus dem SvStream
     virtual void Continue( int nToken );
@@ -241,7 +243,7 @@ private:
     bool ParseMetaOptionsImpl( const ::com::sun::star::uno::Reference<
                 ::com::sun::star::document::XDocumentProperties>&,
             SvKeyValueIterator*,
-            const HTMLOptions*,
+            const HTMLOptions&,
             rtl_TextEncoding& rEnc );
 
 public:
@@ -259,7 +261,7 @@ public:
                       bool bSwitchToUCS2 = false,
                       rtl_TextEncoding eEnc=RTL_TEXTENCODING_DONTKNOW );
 
-    sal_Bool ParseScriptOptions( String& rLangString, const String&, HTMLScriptLanguage& rLang,
+    bool ParseScriptOptions( String& rLangString, const String&, HTMLScriptLanguage& rLang,
                              String& rSrc, String& rLibrary, String& rModule );
 
     // Einen Kommentar um den Inhalt von <SCRIPT> oder <STYLE> entfernen
diff --git a/svtools/source/svhtml/htmlsupp.cxx b/svtools/source/svhtml/htmlsupp.cxx
index 9cc1093..e0f255a 100644
--- a/svtools/source/svhtml/htmlsupp.cxx
+++ b/svtools/source/svhtml/htmlsupp.cxx
@@ -51,13 +51,13 @@ static HTMLOptionEnum const aScriptLangOptEnums[] =
     { 0,					0					}
 };
 
-sal_Bool HTMLParser::ParseScriptOptions( String& rLangString, const String& rBaseURL,
+bool HTMLParser::ParseScriptOptions( String& rLangString, const String& rBaseURL,
                                      HTMLScriptLanguage& rLang,
                                      String& rSrc,
                                      String& rLibrary,
                                      String& rModule )
 {
-    const HTMLOptions *pScriptOptions = GetOptions();
+    const HTMLOptions& aScriptOptions = GetOptions();
 
     rLangString.Erase();
     rLang = HTML_SL_JAVASCRIPT;
@@ -65,16 +65,16 @@ sal_Bool HTMLParser::ParseScriptOptions( String& rLangString, const String& rBas
     rLibrary.Erase();
     rModule.Erase();
 
-    for( sal_uInt16 i = pScriptOptions->Count(); i; )
+    for( size_t i = aScriptOptions.size(); i; )
     {
-        const HTMLOption *pOption = (*pScriptOptions)[ --i ];
-        switch( pOption->GetToken() )
+        const HTMLOption& aOption = aScriptOptions[--i];
+        switch( aOption.GetToken() )
         {
         case HTML_O_LANGUAGE:
             {
-                rLangString = pOption->GetString();
+                rLangString = aOption.GetString();
                 sal_uInt16 nLang;
-                if( pOption->GetEnum( nLang, aScriptLangOptEnums ) )
+                if( aOption.GetEnum( nLang, aScriptLangOptEnums ) )
                     rLang = (HTMLScriptLanguage)nLang;
                 else
                     rLang = HTML_SL_UNKNOWN;
@@ -82,19 +82,19 @@ sal_Bool HTMLParser::ParseScriptOptions( String& rLangString, const String& rBas
             break;
 
         case HTML_O_SRC:
-            rSrc = INetURLObject::GetAbsURL( rBaseURL, pOption->GetString() );
+            rSrc = INetURLObject::GetAbsURL( rBaseURL, aOption.GetString() );
             break;
         case HTML_O_SDLIBRARY:
-            rLibrary = pOption->GetString();
+            rLibrary = aOption.GetString();
             break;
 
         case HTML_O_SDMODULE:
-            rModule = pOption->GetString();
+            rModule = aOption.GetString();
             break;
         }
     }
 
-    return sal_True;
+    return true;
 }
 
 void HTMLParser::RemoveSGMLComment( String &rString, sal_Bool bFull )
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index ec3d457..c676b2c 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -52,6 +52,7 @@
 #include <svtools/htmltokn.h>
 #include <svtools/htmlkywd.hxx>
 
+#include <memory>
 
 using namespace ::com::sun::star;
 
@@ -108,10 +109,6 @@ static HTMLOptionEnum const aTableRulesOptEnums[] =
     { 0,				0				}
 };
 
-
-SV_IMPL_PTRARR(HTMLOptions,HTMLOptionPtr)
-
-
 sal_uInt16 HTMLOption::GetEnum( const HTMLOptionEnum *pOptEnums, sal_uInt16 nDflt ) const
 {
     sal_uInt16 nValue = nDflt;
@@ -323,17 +320,12 @@ HTMLParser::HTMLParser( SvStream& rIn, bool bReadNewDoc ) :
     bReadNextChar(false),
     bReadComment(false)
 {
-    pOptions = new HTMLOptions;
-
     //#i76649, default to UTF-8 for HTML unless we know differently
     SetSrcEncoding(RTL_TEXTENCODING_UTF8);
 }
 
 HTMLParser::~HTMLParser()
 {
-    if( pOptions && pOptions->Count() )
-        pOptions->DeleteAndDestroy( 0, pOptions->Count() );
-    delete pOptions;
 }
 
 SvParserState HTMLParser::CallParser()
@@ -1085,8 +1077,8 @@ int HTMLParser::_GetNextToken()
     sSaveToken.Erase();
 
     // Delete options
-    if( pOptions->Count() )
-        pOptions->DeleteAndDestroy( 0, pOptions->Count() );
+    if (!maOptions.empty())
+        maOptions.clear();
 
     if( !IsParserWorking() )		// Don't continue if already an error occured
         return 0;
@@ -1459,12 +1451,12 @@ void HTMLParser::UnescapeToken()
     }
 }
 
-const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
+const HTMLOptions& HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
 {
     // If the options for the current token have already been returned,
     // return them once again.
-    if( pOptions->Count() )
-        return pOptions;
+    if (!maOptions.empty())
+        return maOptions;
 
     xub_StrLen nPos = 0;
     while( nPos < aToken.Len() )
@@ -1613,11 +1605,10 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
             }
 
             // Token is known and can be saved
-            HTMLOption *pOption =
-                new HTMLOption(
-                    sal::static_int_cast< sal_uInt16 >(nToken), sName, aValue );
+            std::auto_ptr<HTMLOption> pOption(
+                new HTMLOption(sal::static_int_cast<sal_uInt16>(nToken), sName, aValue));
 
-            pOptions->Insert( pOption, pOptions->Count() );
+            maOptions.push_back(pOption);
 
         }
         else
@@ -1625,7 +1616,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
             nPos++;
     }
 
-    return pOptions;
+    return maOptions;
 }
 
 int HTMLParser::FilterPRE( int nToken )
@@ -2100,32 +2091,32 @@ void HTMLParser::AddMetaUserDefined( ::rtl::OUString const & )
 bool HTMLParser::ParseMetaOptionsImpl(
         const uno::Reference<document::XDocumentProperties> & i_xDocProps,
         SvKeyValueIterator *i_pHTTPHeader,
-        const HTMLOptions *i_pOptions,
+        const HTMLOptions& aOptions,
         rtl_TextEncoding& o_rEnc )
 {
     String aName, aContent;
     sal_uInt16 nAction = HTML_META_NONE;
     bool bHTTPEquiv = false, bChanged = false;
 
-    for ( sal_uInt16 i = i_pOptions->Count(); i; )
+    for ( size_t i = aOptions.size(); i; )
     {
-        const HTMLOption *pOption = (*i_pOptions)[ --i ];
-        switch ( pOption->GetToken() )
+        const HTMLOption& aOption = aOptions[--i];
+        switch ( aOption.GetToken() )
         {
             case HTML_O_NAME:
-                aName = pOption->GetString();
+                aName = aOption.GetString();
                 if ( HTML_META_NONE==nAction )
                 {
-                    pOption->GetEnum( nAction, aHTMLMetaNameTable );
+                    aOption.GetEnum( nAction, aHTMLMetaNameTable );
                 }
                 break;
             case HTML_O_HTTPEQUIV:
-                aName = pOption->GetString();
-                pOption->GetEnum( nAction, aHTMLMetaNameTable );
+                aName = aOption.GetString();
+                aOption.GetEnum( nAction, aHTMLMetaNameTable );
                 bHTTPEquiv = true;
                 break;
             case HTML_O_CONTENT:
-                aContent = pOption->GetString();
+                aContent = aOption.GetString();
                 break;
         }
     }
commit 41e14e97b88b21897ebdd6b8ba0803f4e0b0442b
Author: Kohei Yoshida <kyoshida at novell.com>
Date:   Fri Jul 22 15:13:14 2011 -0400

    Initialize members in the initializer of the ctor.

diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index 1c3e64a..ec3d457 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -307,15 +307,22 @@ HTMLTableRules HTMLOption::GetTableRules() const
     return (HTMLTableRules)GetEnum( aTableRulesOptEnums, HTML_TR_NONE );
 }
 
-HTMLParser::HTMLParser( SvStream& rIn, bool bReadNewDoc )
-    : SvParser( rIn )
+HTMLParser::HTMLParser( SvStream& rIn, bool bReadNewDoc ) :
+    SvParser( rIn ),
+    bNewDoc(bReadNewDoc),
+    bIsInHeader(true),
+    bIsInBody(false),
+    bReadListing(false),
+    bReadXMP(false),
+    bReadPRE(false),
+    bReadTextArea(false),
+    bReadScript(false),
+    bReadStyle(false),
+    bEndTokenFound(false),
+    bPre_IgnoreNewPara(false),
+    bReadNextChar(false),
+    bReadComment(false)
 {
-    bNewDoc = bReadNewDoc;
-    bReadListing = bReadXMP = bReadPRE = bReadTextArea =
-        bReadScript = bReadStyle =
-        bEndTokenFound = bIsInBody = bReadNextChar =
-        bReadComment = false;
-    bIsInHeader = true;
     pOptions = new HTMLOptions;
 
     //#i76649, default to UTF-8 for HTML unless we know differently
commit 18d38a5c2849f81e6100d22a13a1b182af79aa93
Author: Kohei Yoshida <kyoshida at novell.com>
Date:   Fri Jul 22 15:05:09 2011 -0400

    Removed commented-out methods.

diff --git a/svtools/inc/svtools/parhtml.hxx b/svtools/inc/svtools/parhtml.hxx
index 569968b..9a30e04 100644
--- a/svtools/inc/svtools/parhtml.hxx
+++ b/svtools/inc/svtools/parhtml.hxx
@@ -226,8 +226,6 @@ public:
     const HTMLOptions *GetOptions( sal_uInt16 *pNoConvertToken=0 ) const;
 
     // fuers asynchrone lesen aus dem SvStream
-//	virtual void SaveState( int nToken );
-//	virtual void RestoreState();
     virtual void Continue( int nToken );
 
 
commit 93ccf0e56f51ba102b328ec014e05a2b66ecf0bc
Author: Kohei Yoshida <kyoshida at novell.com>
Date:   Fri Jul 22 15:02:30 2011 -0400

    sal_Bool to bool.

diff --git a/svtools/inc/svtools/parhtml.hxx b/svtools/inc/svtools/parhtml.hxx
index 8f48cde..569968b 100644
--- a/svtools/inc/svtools/parhtml.hxx
+++ b/svtools/inc/svtools/parhtml.hxx
@@ -115,13 +115,13 @@ public:
     sal_uInt32 GetNumber() const;                          	// ... als Zahl
     sal_Int32 GetSNumber() const;                          	// ... als Zahl
     void GetNumbers( SvULongs &rLongs, 					// ... als Zahlen
-                     sal_Bool bSpaceDelim=sal_False ) const;
+                     bool bSpaceDelim=false ) const;
     void GetColor( Color& ) const;						// ... als Farbe
 
     // ... als Enum pOptEnums ist ein HTMLOptionEnum-Array
     sal_uInt16 GetEnum( const HTMLOptionEnum *pOptEnums,
                         sal_uInt16 nDflt=0 ) const;
-    sal_Bool GetEnum( sal_uInt16 &rEnum, const HTMLOptionEnum *pOptEnums ) const;
+    bool GetEnum( sal_uInt16 &rEnum, const HTMLOptionEnum *pOptEnums ) const;
 
     // ... und als ein par spezielle Enums
     HTMLInputType GetInputType() const;				   	// <INPUT TYPE=...>
@@ -135,20 +135,20 @@ SV_DECL_PTRARR(HTMLOptions,HTMLOptionPtr,16,16)
 
 class SVT_DLLPUBLIC HTMLParser : public SvParser
 {
-    sal_Bool bNewDoc 		: 1;		// neues Doc lesen ?
-    sal_Bool bIsInHeader 	: 1;		// scanne Header-Bereich
-    sal_Bool bIsInBody 		: 1;		// scanne Body-Bereich
-    sal_Bool bReadListing	: 1;		// Lese Listings
-    sal_Bool bReadXMP		: 1;		// Lese XMP
-    sal_Bool bReadPRE		: 1;		// Lese preformatted Text
-    sal_Bool bReadTextArea	: 1;		// Lese TEXTAREA
-    sal_Bool bReadScript	: 1;		// Lesen von <SCRIPT>
-    sal_Bool bReadStyle		: 1;		// Lesen von <STYLE>
-    sal_Bool bEndTokenFound : 1;		// </SCRIPT> oder </STYLE> gefunden
-
-    sal_Bool bPre_IgnoreNewPara : 1;	// Flags fuers lesen von PRE-Absaetzen
-    sal_Bool bReadNextChar : 1;			// sal_True: NextChar nochmals lesen (JavaScript!)
-    sal_Bool bReadComment : 1;			// sal_True: NextChar nochmals lesen (JavaScript!)
+    bool bNewDoc        : 1;        // neues Doc lesen ?
+    bool bIsInHeader    : 1;        // scanne Header-Bereich
+    bool bIsInBody      : 1;        // scanne Body-Bereich
+    bool bReadListing   : 1;        // Lese Listings
+    bool bReadXMP       : 1;        // Lese XMP
+    bool bReadPRE       : 1;        // Lese preformatted Text
+    bool bReadTextArea  : 1;        // Lese TEXTAREA
+    bool bReadScript    : 1;        // Lesen von <SCRIPT>
+    bool bReadStyle     : 1;        // Lesen von <STYLE>
+    bool bEndTokenFound : 1;        // </SCRIPT> oder </STYLE> gefunden
+
+    bool bPre_IgnoreNewPara : 1;    // Flags fuers lesen von PRE-Absaetzen
+    bool bReadNextChar : 1;         // true: NextChar nochmals lesen (JavaScript!)
+    bool bReadComment : 1;          // true: NextChar nochmals lesen (JavaScript!)
 
     sal_uInt32 nPre_LinePos;			// Pos in der Line im PRE-Tag
 
@@ -167,43 +167,43 @@ protected:
 
     virtual ~HTMLParser();
 
-    void FinishHeader( sal_Bool bBody ) { bIsInHeader = sal_False; bIsInBody = bBody; }
+    void FinishHeader( bool bBody ) { bIsInHeader = false; bIsInBody = bBody; }
 
 public:
-    HTMLParser( SvStream& rIn, int bReadNewDoc = sal_True );
+    HTMLParser( SvStream& rIn, bool bReadNewDoc = true );
 
     virtual SvParserState CallParser();   // Aufruf des Parsers
 
-    sal_Bool IsNewDoc() const 		{ return bNewDoc; }
-    sal_Bool IsInHeader() const 	{ return bIsInHeader; }
-    sal_Bool IsInBody() const 		{ return bIsInBody; }
-    sal_Bool IsValidSyntax() const 	{ return sal_True; }
-    sal_Bool IsReadListing() const 	{ return bReadListing; }
-    sal_Bool IsReadXMP() const 		{ return bReadXMP; }
-    sal_Bool IsReadPRE() const 		{ return bReadPRE; }
-    sal_Bool IsReadScript() const 	{ return bReadScript; }
-    sal_Bool IsReadStyle() const 	{ return bReadStyle; }
+    bool IsNewDoc() const 		{ return bNewDoc; }
+    bool IsInHeader() const 	{ return bIsInHeader; }
+    bool IsInBody() const       { return bIsInBody; }
+    bool IsValidSyntax() const  { return true; }
+    bool IsReadListing() const  { return bReadListing; }
+    bool IsReadXMP() const      { return bReadXMP; }
+    bool IsReadPRE() const      { return bReadPRE; }
+    bool IsReadScript() const   { return bReadScript; }
+    bool IsReadStyle() const    { return bReadStyle; }
 
-    void SetReadNextChar() 		{ bReadNextChar = sal_True; }
+    void SetReadNextChar() 		{ bReadNextChar = true; }
 
     // PRE-/LISTING oder XMP-Modus starten/beenden oder Tags entsprechend
     // filtern
-    inline void StartPRE( sal_Bool bRestart=sal_False );
-    void FinishPRE() { bReadPRE = sal_False; }
+    inline void StartPRE( bool bRestart=false );
+    void FinishPRE() { bReadPRE = false; }
     int FilterPRE( int nToken );
 
-    inline void StartListing( sal_Bool bRestart=sal_False );
-    void FinishListing() { bReadListing = sal_False; }
+    inline void StartListing( bool bRestart=false );
+    void FinishListing() { bReadListing = false; }
     int FilterListing( int nToken );
 
-    inline void StartXMP( sal_Bool bRestart=sal_False );
-    void FinishXMP() { bReadXMP = sal_False; }
+    inline void StartXMP( bool bRestart=false );
+    void FinishXMP() { bReadXMP = false; }
     int FilterXMP( int nToken );
 
-    void FinishTextArea() { bReadTextArea = sal_False; }
+    void FinishTextArea() { bReadTextArea = false; }
 
     // PRE-/LSITING- und XMP-Modus beenden
-    void FinishPREListingXMP() { bReadPRE = bReadListing = bReadXMP = sal_False; }
+    void FinishPREListingXMP() { bReadPRE = bReadListing = bReadXMP = false; }
 
     // Das aktuelle Token dem aktuellen Modus (PRE, XMP, ...) entsprechend
     // Filtern und die Flags setzen. Wird von Continue aufgerufen, bevor
@@ -213,7 +213,7 @@ public:
 
     // Scannen eines Scripts beenden (sollte nur unmittelbar nach dem
     // Lesen eines <SCRIPT> aufgerufen werden
-    void EndScanScript() { bReadScript = sal_False; }
+    void EndScanScript() { bReadScript = false; }
 
     void ReadRawData( const sal_Char *pEndToken ) { aEndToken.AssignAscii(pEndToken); }
 
@@ -254,11 +254,11 @@ public:
 
     // Ist der uebergebene 0-terminierte String (vermutlich) der Anfang
     // eines HTML-Files? Er sollte mind. 80 Zeichen lang sein.
-    // Mit Ausnahme des Falls, dass SwitchToUCS2==sal_False und
+    // Mit Ausnahme des Falls, dass SwitchToUCS2==false und
     // SVPAR_CS_DONTKNOW uebergeben wird muss der String mit zwei(!)
     // 0-Bytes an einer geraden(!) Position terminiert sein.
     static bool IsHTMLFormat( const sal_Char* pHeader,
-                      sal_Bool bSwitchToUCS2 = sal_False,
+                      bool bSwitchToUCS2 = false,
                       rtl_TextEncoding eEnc=RTL_TEXTENCODING_DONTKNOW );
 
     sal_Bool ParseScriptOptions( String& rLangString, const String&, HTMLScriptLanguage& rLang,
@@ -269,28 +269,28 @@ public:
     // entfernt (fuer JavaSript)
     static void RemoveSGMLComment( String &rString, sal_Bool bFull );
 
-    static sal_Bool InternalImgToPrivateURL( String& rURL );
+    static bool InternalImgToPrivateURL( String& rURL );
     static rtl_TextEncoding GetEncodingByHttpHeader( SvKeyValueIterator *pHTTPHeader );
-    sal_Bool SetEncodingByHTTPHeader( SvKeyValueIterator *pHTTPHeader );
+    bool SetEncodingByHTTPHeader( SvKeyValueIterator *pHTTPHeader );
 };
 
-inline void HTMLParser::StartPRE( sal_Bool bRestart )
+inline void HTMLParser::StartPRE( bool bRestart )
 {
-    bReadPRE = sal_True;
+    bReadPRE = true;
     bPre_IgnoreNewPara = !bRestart;
     nPre_LinePos = 0UL;
 }
 
-inline void HTMLParser::StartListing( sal_Bool bRestart )
+inline void HTMLParser::StartListing( bool bRestart )
 {
-    bReadListing = sal_True;
+    bReadListing = true;
     bPre_IgnoreNewPara = !bRestart;
     nPre_LinePos = 0UL;
 }
 
-inline void HTMLParser::StartXMP( sal_Bool bRestart )
+inline void HTMLParser::StartXMP( bool bRestart )
 {
-    bReadXMP = sal_True;
+    bReadXMP = true;
     bPre_IgnoreNewPara = !bRestart;
     nPre_LinePos = 0UL;
 }
diff --git a/svtools/source/svhtml/parhtml.cxx b/svtools/source/svhtml/parhtml.cxx
index 1de6b51..1c3e64a 100644
--- a/svtools/source/svhtml/parhtml.cxx
+++ b/svtools/source/svhtml/parhtml.cxx
@@ -128,7 +128,7 @@ sal_uInt16 HTMLOption::GetEnum( const HTMLOptionEnum *pOptEnums, sal_uInt16 nDfl
     return nValue;
 }
 
-sal_Bool HTMLOption::GetEnum( sal_uInt16 &rEnum, const HTMLOptionEnum *pOptEnums ) const
+bool HTMLOption::GetEnum( sal_uInt16 &rEnum, const HTMLOptionEnum *pOptEnums ) const
 {
     while( pOptEnums->pName )
     {
@@ -179,7 +179,7 @@ sal_Int32 HTMLOption::GetSNumber() const
     return aTmp.ToInt32();
 }
 
-void HTMLOption::GetNumbers( SvULongs &rLongs, sal_Bool bSpaceDelim ) const
+void HTMLOption::GetNumbers( SvULongs &rLongs, bool bSpaceDelim ) const
 {
     if( rLongs.Count() )
         rLongs.Remove( 0, rLongs.Count() );
@@ -188,7 +188,7 @@ void HTMLOption::GetNumbers( SvULongs &rLongs, sal_Bool bSpaceDelim ) const
     {
         // This is a very simplified scanner: it only searches all
         // numerals in the string.
-        sal_Bool bInNum = sal_False;
+        bool bInNum = false;
         sal_uLong nNum = 0;
         for( xub_StrLen i=0; i<aValue.Len(); i++ )
         {
@@ -197,12 +197,12 @@ void HTMLOption::GetNumbers( SvULongs &rLongs, sal_Bool bSpaceDelim ) const
             {
                 nNum *= 10;
                 nNum += (c - '0');
-                bInNum = sal_True;
+                bInNum = true;
             }
             else if( bInNum )
             {
                 rLongs.Insert( nNum, rLongs.Count() );
-                bInNum = sal_False;
+                bInNum = false;
                 nNum = 0;
             }
         }
@@ -307,15 +307,15 @@ HTMLTableRules HTMLOption::GetTableRules() const
     return (HTMLTableRules)GetEnum( aTableRulesOptEnums, HTML_TR_NONE );
 }
 
-HTMLParser::HTMLParser( SvStream& rIn, int bReadNewDoc )
+HTMLParser::HTMLParser( SvStream& rIn, bool bReadNewDoc )
     : SvParser( rIn )
 {
     bNewDoc = bReadNewDoc;
     bReadListing = bReadXMP = bReadPRE = bReadTextArea =
         bReadScript = bReadStyle =
         bEndTokenFound = bIsInBody = bReadNextChar =
-        bReadComment = sal_False;
-    bIsInHeader = sal_True;
+        bReadComment = false;
+    bIsInHeader = true;
     pOptions = new HTMLOptions;
 
     //#i76649, default to UTF-8 for HTML unless we know differently
@@ -336,7 +336,7 @@ SvParserState HTMLParser::CallParser()
     SaveState( 0 );
 
     nPre_LinePos = 0;
-    bPre_IgnoreNewPara = sal_False;
+    bPre_IgnoreNewPara = false;
 
     AddRef();
     Continue( 0 );
@@ -375,24 +375,24 @@ int HTMLParser::FilterToken( int nToken )
         break;			// don't pass
 
     case HTML_HEAD_OFF:
-        bIsInBody = sal_True;
+        bIsInBody = true;
     case HTML_HEAD_ON:
         bIsInHeader = HTML_HEAD_ON == nToken;
         break;
 
     case HTML_BODY_ON:
     case HTML_FRAMESET_ON:
-        bIsInHeader = sal_False;
+        bIsInHeader = false;
         bIsInBody = HTML_BODY_ON == nToken;
         break;
 
     case HTML_BODY_OFF:
-        bIsInBody = bReadPRE = bReadListing = bReadXMP = sal_False;
+        bIsInBody = bReadPRE = bReadListing = bReadXMP = false;
         break;
 
     case HTML_HTML_OFF:
         nToken = 0;
-        bReadPRE = bReadListing = bReadXMP = sal_False;
+        bReadPRE = bReadListing = bReadXMP = false;
         break;		// HTML_ON hasn't been passed either !
 
     case HTML_PREFORMTXT_ON:
@@ -443,17 +443,17 @@ int HTMLParser::FilterToken( int nToken )
 int HTMLParser::ScanText( const sal_Unicode cBreak )
 {
     ::rtl::OUStringBuffer sTmpBuffer( MAX_LEN );
-    int bContinue = sal_True;
-    int bEqSignFound = sal_False;
+    int bContinue = true;
+    int bEqSignFound = false;
     sal_Unicode cQuote = 0U;
 
     while( bContinue && IsParserWorking() )
     {
-        int bNextCh = sal_True;
+        int bNextCh = true;
         switch( nNextCh )
         {
         case '&':
-            bEqSignFound = sal_False;
+            bEqSignFound = false;
             if( bReadXMP )
                 sTmpBuffer.append( (sal_Unicode)'&' );
             else
@@ -465,8 +465,8 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
                 if( '#' == (nNextCh = GetNextChar()) )
                 {
                     nNextCh = GetNextChar();
-                    const sal_Bool bIsHex( 'x' == nNextCh );
-                    const sal_Bool bIsDecOrHex( bIsHex || HTML_ISDIGIT(nNextCh) );
+                    const bool bIsHex( 'x' == nNextCh );
+                    const bool bIsDecOrHex( bIsHex || HTML_ISDIGIT(nNextCh) );
                     if ( bIsDecOrHex )
                     {
                         if ( bIsHex )
@@ -639,7 +639,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
                 else if( IsParserWorking() )
                 {
                     sTmpBuffer.append( (sal_Unicode)'&' );
-                    bNextCh = sal_False;
+                    bNextCh = false;
                     break;
                 }
 
@@ -674,15 +674,15 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
                         rInput.Seek( nStreamPos-(sal_uInt32)GetCharSize() );
                         nlLinePos = nLinePos-1;
                         ClearTxtConvContext();
-                        bReadNextChar = sal_True;
+                        bReadNextChar = true;
                     }
-                    bNextCh = sal_False;
+                    bNextCh = false;
                 }
             }
             break;
         case '=':
             if( '>'==cBreak && !cQuote )
-                bEqSignFound = sal_True;
+                bEqSignFound = true;
             sTmpBuffer.append( nNextCh );
             break;
 
@@ -707,13 +707,13 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
                     cQuote = 0U;
             }
             sTmpBuffer.append( nNextCh );
-            bEqSignFound = sal_False;
+            bEqSignFound = false;
             break;
 
         case sal_Unicode(EOF):
             if( rInput.IsEof() )
             {
-                bContinue = sal_False;
+                bContinue = false;
             }
             else
             {
@@ -722,11 +722,11 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
             break;
 
         case '<':
-            bEqSignFound = sal_False;
+            bEqSignFound = false;
             if( '>'==cBreak )
                 sTmpBuffer.append( nNextCh );
             else
-                bContinue = sal_False;		// break, String zusammen
+                bContinue = false;		// break, String zusammen
             break;
 
         case '\f':
@@ -738,7 +738,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
             else
             {
                 // otherwise it's a separate token.
-                bContinue = sal_False;
+                bContinue = false;
             }
             break;
 
@@ -752,7 +752,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
             }
             else if( bReadListing || bReadXMP || bReadPRE || bReadTextArea )
             {
-                bContinue = sal_False;
+                bContinue = false;
                 break;
             }
             // Reduce sequence of CR/LF/BLANK/TAB to a single blank
@@ -761,7 +761,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
             if( '\t'==nNextCh && bReadPRE && '>'!=cBreak )
             {
                 // In <PRE>: Tabs nach oben durchreichen
-                bContinue = sal_False;
+                bContinue = false;
                 break;
             }
             // no break
@@ -797,15 +797,15 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
                 } while ( ' ' == nNextCh || '\t' == nNextCh ||
                           '\r' == nNextCh || '\n' == nNextCh ||
                           '\x0b' == nNextCh );
-                bNextCh = sal_False;
+                bNextCh = false;
             }
             break;
 
         default:
-            bEqSignFound = sal_False;
+            bEqSignFound = false;
             if( (nNextCh==cBreak && !cQuote) ||
                 (sal_uLong(aToken.Len()) + MAX_LEN) > sal_uLong(STRING_MAXLEN & ~1 ))
-                bContinue = sal_False;
+                bContinue = false;
             else
             {
                 do {
@@ -830,7 +830,7 @@ int HTMLParser::ScanText( const sal_Unicode cBreak )
                         return HTML_TEXTTOKEN;
                     }
                 } while( HTML_ISALPHA( nNextCh ) || HTML_ISDIGIT( nNextCh ) );
-                bNextCh = sal_False;
+                bNextCh = false;
             }
         }
 
@@ -855,21 +855,21 @@ int HTMLParser::_GetNextRawToken()
     {
         // 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;
+        bReadScript = false;
+        bReadStyle = false;
         aEndToken.Erase();
-        bEndTokenFound = sal_False;
+        bEndTokenFound = false;
 
         return 0;
     }
 
     // Default return value: HTML_RAWDATA
-    int bContinue = sal_True;
+    int bContinue = true;
     int nToken = HTML_RAWDATA;
     SaveState( 0 );
     while( bContinue && IsParserWorking() )
     {
-        int bNextCh = sal_True;
+        int bNextCh = true;
         switch( nNextCh )
         {
         case '<':
@@ -885,10 +885,10 @@ int HTMLParser::_GetNextRawToken()
                 sal_uLong nLinePos = GetLinePos();
 
                 // Start of an end token?
-                int bOffState = sal_False;
+                int bOffState = false;
                 if( '/' == (nNextCh = GetNextChar()) )
                 {
-                    bOffState = sal_True;
+                    bOffState = true;
                     nNextCh = GetNextChar();
                 }
                 else if( '!' == nNextCh )
@@ -909,7 +909,7 @@ int HTMLParser::_GetNextRawToken()
                              sal::static_int_cast< xub_StrLen >(
                                  sTmpBuffer.getLength()) );
                 aTok.ToUpperAscii();
-                sal_Bool bDone = sal_False;
+                bool bDone = false;
                 if( bReadScript || aEndToken.Len() )
                 {
                     if( !bReadComment )
@@ -917,7 +917,7 @@ int HTMLParser::_GetNextRawToken()
                         if( aTok.CompareToAscii( OOO_STRING_SVTOOLS_HTML_comment, 3 )
                                 == COMPARE_EQUAL )
                         {
-                            bReadComment = sal_True;
+                            bReadComment = true;
                         }
                         else
                         {
@@ -933,7 +933,7 @@ int HTMLParser::_GetNextRawToken()
                         aTok.Copy( aTok.Len()-2 ).EqualsAscii( "--" ) )
                     {
                         // End of comment of style <!----->
-                        bReadComment = sal_False;
+                        bReadComment = false;
                     }
                 }
                 else
@@ -954,15 +954,15 @@ int HTMLParser::_GetNextRawToken()
                     // Done! Return the previously read string (if requested)
                     // and continue.
 
-                    bContinue = sal_False;
+                    bContinue = false;
 
                     // nToken==0 means, _GetNextToken continues to read
                     if( !aToken.Len() && (bReadStyle || bReadScript) )
                     {
                         // Immediately close environment (or context?)
                         // and parse the end token
-                        bReadScript = sal_False;
-                        bReadStyle = sal_False;
+                        bReadScript = false;
+                        bReadStyle = false;
                         aEndToken.Erase();
                         nToken = 0;
                     }
@@ -970,7 +970,7 @@ int HTMLParser::_GetNextRawToken()
                     {
                         // Keep bReadScript/bReadStyle alive
                         // and parse end token during next execution
-                        bEndTokenFound = sal_True;
+                        bEndTokenFound = true;
                     }
 
                     // Move backwards in stream to '<'
@@ -990,7 +990,7 @@ int HTMLParser::_GetNextRawToken()
                     if( bOffState )
                         aToken += (sal_Unicode)'/';
 
-                    bNextCh = sal_False;
+                    bNextCh = false;
                 }
             }
             break;
@@ -998,11 +998,11 @@ int HTMLParser::_GetNextRawToken()
             sTmpBuffer.append( nNextCh );
             if( bReadComment )
             {
-                sal_Bool bTwoMinus = sal_False;
+                bool bTwoMinus = false;
                 nNextCh = GetNextChar();
                 while( '-' == nNextCh && IsParserWorking() )
                 {
-                    bTwoMinus = sal_True;
+                    bTwoMinus = true;
 
                     if( MAX_LEN == sTmpBuffer.getLength() )
                         aToken += String(sTmpBuffer.makeStringAndClear());
@@ -1011,9 +1011,9 @@ int HTMLParser::_GetNextRawToken()
                 }
 
                 if( '>' == nNextCh && IsParserWorking() && bTwoMinus )
-                    bReadComment = sal_False;
+                    bReadComment = false;
 
-                bNextCh = sal_False;
+                bNextCh = false;
             }
             break;
 
@@ -1022,27 +1022,27 @@ int HTMLParser::_GetNextRawToken()
             nNextCh = GetNextChar();
             if( nNextCh=='\n' )
                 nNextCh = GetNextChar();
-            bContinue = sal_False;
+            bContinue = false;
             break;
         case '\n':
             // \n closes the current text token (even if it's empty)
             nNextCh = GetNextChar();
-            bContinue = sal_False;
+            bContinue = false;
             break;
         case sal_Unicode(EOF):
             // eof closes the current text token and behaves like having read
             // an end token
             if( rInput.IsEof() )
             {
-                bContinue = sal_False;
+                bContinue = false;
                 if( aToken.Len() || sTmpBuffer.getLength() )
                 {
-                    bEndTokenFound = sal_True;
+                    bEndTokenFound = true;
                 }
                 else
                 {
-                    bReadScript = sal_False;
-                    bReadStyle = sal_False;
+                    bReadScript = false;
+                    bReadStyle = false;
                     aEndToken.Erase();
                     nToken = 0;
                 }
@@ -1084,7 +1084,7 @@ int HTMLParser::_GetNextToken()
     if( !IsParserWorking() )		// Don't continue if already an error occured
         return 0;
 
-    sal_Bool bReadNextCharSave = bReadNextChar;
+    bool bReadNextCharSave = bReadNextChar;
     if( bReadNextChar )
     {
         DBG_ASSERT( !bEndTokenFound,
@@ -1092,7 +1092,7 @@ int HTMLParser::_GetNextToken()
         nNextCh = GetNextChar();
         if( !IsParserWorking() )		// Don't continue if already an error occured
             return 0;
-        bReadNextChar = sal_False;
+        bReadNextChar = false;
     }
 
     if( bReadScript || bReadStyle || aEndToken.Len() )
@@ -1103,7 +1103,7 @@ int HTMLParser::_GetNextToken()
     }
 
     do {
-        int bNextCh = sal_True;
+        int bNextCh = true;
         switch( nNextCh )
         {
         case '<':
@@ -1112,10 +1112,10 @@ int HTMLParser::_GetNextToken()
                 sal_uLong nLineNr = GetLineNr();
                 sal_uLong nLinePos = GetLinePos();
 
-                int bOffState = sal_False;
+                int bOffState = false;
                 if( '/' == (nNextCh = GetNextChar()) )
                 {
-                    bOffState = sal_True;
+                    bOffState = true;
                     nNextCh = GetNextChar();
                 }
                 if( HTML_ISALPHA( nNextCh ) || '!'==nNextCh )
@@ -1178,7 +1178,7 @@ int HTMLParser::_GetNextToken()
                         sal_uLong nCLinePos = 0;
                         xub_StrLen nCStrLen = 0;
 
-                        sal_Bool bDone = sal_False;
+                        bool bDone = false;
                         // Read until closing -->. If not found restart at first >
                         while( !bDone && !rInput.IsEof() && IsParserWorking() )
                         {
@@ -1241,7 +1241,7 @@ int HTMLParser::_GetNextToken()
                             aToken = '<';
                             nRet = HTML_TEXTTOKEN;
                             nNextCh = GetNextChar();
-                            bNextCh = sal_False;
+                            bNextCh = false;
                             break;
                         }
                     }
@@ -1266,7 +1266,7 @@ int HTMLParser::_GetNextToken()
                             aToken = '<';
                             nRet = HTML_TEXTTOKEN;
                             nNextCh = GetNextChar();
-                            bNextCh = sal_False;
+                            bNextCh = false;
                             break;
                         }
                         if( SVPAR_PENDING == eState )
@@ -1280,7 +1280,7 @@ int HTMLParser::_GetNextToken()
                         sal_uLong nCStreamPos = rInput.Tell();
                         sal_uLong nCLineNr = GetLineNr(), nCLinePos = GetLinePos();
 
-                        sal_Bool bDone = sal_False;
+                        bool bDone = false;
                         // Read until closing %>. If not found restart at first >.
                         while( !bDone && !rInput.IsEof() && IsParserWorking() )
                         {
@@ -1312,7 +1312,7 @@ int HTMLParser::_GetNextToken()
                     {
                         aToken = '<';
                         nRet = HTML_TEXTTOKEN;
-                        bNextCh = sal_False;
+                        bNextCh = false;
                         break;
                     }
                 }
@@ -1323,31 +1323,31 @@ int HTMLParser::_GetNextToken()
                     switch( nRet )
                     {
                     case HTML_TEXTAREA_ON:
-                        bReadTextArea = sal_True;
+                        bReadTextArea = true;
                         break;
                     case HTML_TEXTAREA_OFF:
-                        bReadTextArea = sal_False;
+                        bReadTextArea = false;
                         break;
                     case HTML_SCRIPT_ON:
                         if( !bReadTextArea )
-                            bReadScript = sal_True;
+                            bReadScript = true;
                         break;
                     case HTML_SCRIPT_OFF:
                         if( !bReadTextArea )
                         {
-                            bReadScript = sal_False;
+                            bReadScript = false;
                             // JavaScript might modify the stream,
                             // thus the last character has to be read again.
-                            bReadNextChar = sal_True;
-                            bNextCh = sal_False;
+                            bReadNextChar = true;
+                            bNextCh = false;
                         }
                         break;
 
                     case HTML_STYLE_ON:
-                        bReadStyle = sal_True;
+                        bReadStyle = true;
                         break;
                     case HTML_STYLE_OFF:
-                        bReadStyle = sal_False;
+                        bReadStyle = false;
                         break;
                     }
                 }
@@ -1380,7 +1380,7 @@ int HTMLParser::_GetNextToken()
                 if( ( '\n' != nNextCh || '\r' != c ) &&
                     ( '\r' != nNextCh || '\n' != c ) )
                 {
-                    bNextCh = sal_False;
+                    bNextCh = false;
                     nNextCh = c;
                 }
                 nRet = HTML_NEWPARA;
@@ -1407,7 +1407,7 @@ scan_text:
             if( !bNextCh && eState == SVPAR_PENDING )
             {
                 eState = SVPAR_WORKING;
-                bReadNextChar = sal_True;
+                bReadNextChar = true;
             }
 
             break;
@@ -1418,7 +1418,7 @@ scan_text:
             nNextCh = GetNextChar();
             if( SVPAR_PENDING == eState && nRet && HTML_TEXTTOKEN != nRet )
             {
-                bReadNextChar = sal_True;
+                bReadNextChar = true;
                 eState = SVPAR_WORKING;
             }
         }
@@ -1435,15 +1435,15 @@ void HTMLParser::UnescapeToken()
 {
     xub_StrLen nPos=0;
 
-    sal_Bool bEscape = sal_False;
+    bool bEscape = false;
     while( nPos < aToken.Len() )
     {
-        sal_Bool bOldEscape = bEscape;
-        bEscape = sal_False;
+        bool bOldEscape = bEscape;
+        bEscape = false;
         if( '\\'==aToken.GetChar(nPos) && !bOldEscape )
         {
             aToken.Erase( nPos, 1 );
-            bEscape = sal_True;
+            bEscape = true;
         }
         else
         {
@@ -1486,7 +1486,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
             nToken = GetHTMLOption( sNameUpperCase ); // Name is ready
             DBG_ASSERTWARNING( nToken!=HTML_O_UNKNOWN,
                         "GetOption: unknown HTML option" );
-            sal_Bool bStripCRLF = (nToken < HTML_OPTION_SCRIPT_START ||
+            bool bStripCRLF = (nToken < HTML_OPTION_SCRIPT_START ||
                                nToken >= HTML_OPTION_SCRIPT_END) &&
                               (!pNoConvertToken || nToken != *pNoConvertToken);
 
@@ -1513,12 +1513,12 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
                     {
                         sal_Unicode cEnd = cChar;
                         nPos++; nStt++;
-                        sal_Bool bDone = sal_False;
-                        sal_Bool bEscape = sal_False;
+                        bool bDone = false;
+                        bool bEscape = false;
                         while( nPos < aToken.Len() && !bDone )
                         {
-                            sal_Bool bOldEscape = bEscape;
-                            bEscape = sal_False;
+                            bool bOldEscape = bEscape;
+                            bEscape = false;
                             cChar = aToken.GetChar(nPos);
                             switch( cChar )
                             {
@@ -1537,7 +1537,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
                                 else
                                 {
                                     ((String &)aToken).Erase( nPos, 1 );
-                                    bEscape = sal_True;
+                                    bEscape = true;
                                 }
                                 break;
                             case '"':
@@ -1557,12 +1557,12 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
                     else
                     {
                         // More liberal than the standard: allow all printable characters
-                        sal_Bool bEscape = sal_False;
-                        sal_Bool bDone = sal_False;
+                        bool bEscape = false;
+                        bool bDone = false;
                         while( nPos < aToken.Len() && !bDone )
                         {
-                            sal_Bool bOldEscape = bEscape;
-                            bEscape = sal_False;
+                            bool bOldEscape = bEscape;
+                            bEscape = false;
                             sal_Unicode c = aToken.GetChar(nPos);
                             switch( c )
                             {
@@ -1575,7 +1575,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
                             case '\t':
                             case '\r':
                             case '\n':
-                                bDone = sal_True;
+                                bDone = true;
                                 break;
 
                             case '\\':
@@ -1586,7 +1586,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
                                 else
                                 {
                                     ((String &)aToken).Erase( nPos, 1 );
-                                    bEscape = sal_True;
+                                    bEscape = true;
                                 }
                                 break;
 
@@ -1594,7 +1594,7 @@ const HTMLOptions *HTMLParser::GetOptions( sal_uInt16 *pNoConvertToken ) const
                                 if( HTML_ISPRINTABLE( c ) )
                                     nPos++, nLen++;
                                 else
-                                    bDone = sal_True;
+                                    bDone = true;
                                 break;
                             }
                         }
@@ -1799,7 +1799,7 @@ int HTMLParser::FilterPRE( int nToken )
         break;
     }
 
-    bPre_IgnoreNewPara = sal_False;
+    bPre_IgnoreNewPara = false;
 
     return nToken;
 }
@@ -1840,7 +1840,7 @@ int HTMLParser::FilterXMP( int nToken )
         break;
     }
 
-    bPre_IgnoreNewPara = sal_False;
+    bPre_IgnoreNewPara = false;
 
     return nToken;
 }
@@ -1868,13 +1868,13 @@ int HTMLParser::FilterListing( int nToken )
         break;
     }
 
-    bPre_IgnoreNewPara = sal_False;
+    bPre_IgnoreNewPara = false;
 
     return nToken;
 }
 
 bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
-                               sal_Bool bSwitchToUCS2,
+                               bool bSwitchToUCS2,
                                rtl_TextEncoding eEnc )
 {
     // If the string matches one of the following regular expressions then
@@ -1886,14 +1886,14 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
     //
     // where the underlined subexpression has to be a HTML token
     ByteString sCmp;
-    sal_Bool bUCS2B = sal_False;
+    bool bUCS2B = false;
     if( bSwitchToUCS2 )
     {
         if( 0xfeU == (sal_uChar)pHeader[0] &&
             0xffU == (sal_uChar)pHeader[1] )
         {
             eEnc = RTL_TEXTENCODING_UCS2;
-            bUCS2B = sal_True;
+            bUCS2B = true;
         }
         else if( 0xffU == (sal_uChar)pHeader[0] &&
                  0xfeU == (sal_uChar)pHeader[1] )
@@ -1911,7 +1911,7 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
        )
     {
         if( 0xfe == (sal_uChar)pHeader[0] )
-            bUCS2B = sal_True;
+            bUCS2B = true;
 
         xub_StrLen nLen;
         for( nLen = 2;
@@ -1944,7 +1944,7 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
     // A HTML document must have a '<' in the first line
     xub_StrLen nStart = sCmp.Search( '<' );
     if( STRING_NOTFOUND  == nStart )
-        return sal_False;
+        return false;
     nStart++;
 
     // followed by arbitrary characters followed by a blank or '>'
@@ -1958,7 +1958,7 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
 
     // If the document ends after < it's no HTML
     if( nPos==nStart )
-        return sal_False;
+        return false;
 
     // the string following '<' has to be a known HTML token.
     // <DIR> is not interpreted as HTML. Otherwise the output of the DOS command "DIR"
@@ -1966,30 +1966,30 @@ bool HTMLParser::IsHTMLFormat( const sal_Char* pHeader,
     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;
+        return true;
 
     // "<!" at the very beginning of the file?
     if( nStart == 1 && '!' == sCmp.GetChar( 1 ) )
-        return sal_True;
+        return true;
 
     // <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;
+        return true;
 
     // Else it's rather not a HTML document
-    return sal_False;
+    return false;
 }
 
-sal_Bool HTMLParser::InternalImgToPrivateURL( String& rURL )
+bool HTMLParser::InternalImgToPrivateURL( String& rURL )
 {
     if( rURL.Len() < 19 || 'i' != rURL.GetChar(0) ||
         rURL.CompareToAscii( OOO_STRING_SVTOOLS_HTML_internal_gopher, 9 ) != COMPARE_EQUAL )
-        return sal_False;
+        return false;
 
-    sal_Bool bFound = sal_False;
+    bool bFound = false;
 
     if( rURL.CompareToAscii( OOO_STRING_SVTOOLS_HTML_internal_gopher,16) == COMPARE_EQUAL )
     {
@@ -2289,7 +2289,7 @@ rtl_TextEncoding HTMLParser::GetEncodingByHttpHeader( SvKeyValueIterator *pHTTPH
     if( pHTTPHeader )
     {
         SvKeyValue aKV;
-        for( sal_Bool bCont = pHTTPHeader->GetFirst( aKV ); bCont;
+        for( bool bCont = pHTTPHeader->GetFirst( aKV ); bCont;
              bCont = pHTTPHeader->GetNext( aKV ) )
         {
             if( aKV.GetKey().EqualsIgnoreCaseAscii( OOO_STRING_SVTOOLS_HTML_META_content_type ) )
@@ -2304,15 +2304,14 @@ rtl_TextEncoding HTMLParser::GetEncodingByHttpHeader( SvKeyValueIterator *pHTTPH
     return eRet;
 }
 
-sal_Bool HTMLParser::SetEncodingByHTTPHeader(
-                                SvKeyValueIterator *pHTTPHeader )
+bool HTMLParser::SetEncodingByHTTPHeader( SvKeyValueIterator *pHTTPHeader )
 {
-    sal_Bool bRet = sal_False;
+    bool bRet = false;
     rtl_TextEncoding eEnc = HTMLParser::GetEncodingByHttpHeader( pHTTPHeader );
     if(RTL_TEXTENCODING_DONTKNOW != eEnc)
     {
         SetSrcEncoding( eEnc );
-        bRet = sal_True;
+        bRet = true;
     }
     return bRet;
 }


More information about the Libreoffice-commits mailing list