More String / sal_Bool removal.

3 files changed, 23 insertions, 26 deletions

diff --git a/sc/source/filter/html/htmlimp.cxx b/sc/source/filter/html/htmlimp.cxx
index 2e79fddc2c13..4b2b1dc54ebe 100644
--- a/sc/source/filter/html/htmlimp.cxx
+++ b/sc/source/filter/html/htmlimp.cxx
@@ -206,7 +206,7 @@ void ScHTMLImport::WriteToDocument(
         // insert table number as name
         InsertRangeName( mpDoc, ScfTools::GetNameFromHTMLIndex( nTableId ), aNewRange );
         // insert table id as name
-        if( pTable->GetTableName().Len() )
+        if (!pTable->GetTableName().isEmpty())
         {
             String aName( ScfTools::GetNameFromHTMLName( pTable->GetTableName() ) );
             if (!mpDoc->GetRangeName()->findByName(aName))
diff --git a/sc/source/filter/html/htmlpars.cxx b/sc/source/filter/html/htmlpars.cxx
index 9b1a3ba78e1c..1108dd603180 100644
--- a/sc/source/filter/html/htmlpars.cxx
+++ b/sc/source/filter/html/htmlpars.cxx
@@ -239,7 +239,7 @@ ScHTMLLayoutParser::ScHTMLLayoutParser(
         nMetaCnt(0),
         nOffsetTolerance( SC_HTML_OFFSET_TOLERANCE_SMALL ),
         bTabInTabCell( false ),
-        bFirstRow( sal_True ),
+        bFirstRow( true ),
         bInCell( false ),
         bInTitle( false )
 {
@@ -910,7 +910,7 @@ IMPL_LINK( ScHTMLLayoutParser, HTMLImportHdl, ImportInfo*, pInfo )
                 {
                     bInCell = false;
                     NextRow( pInfo );
-                    bInCell = sal_True;
+                    bInCell = true;
                 }
                 CloseEntry( pInfo );
             }
@@ -973,7 +973,7 @@ void ScHTMLLayoutParser::TableDataOn( ImportInfo* pInfo )
         OSL_FAIL( "Dummbatz-Dok! <TH> oder <TD> ohne vorheriges <TABLE>" );
         TableOn( pInfo );
     }
-    bInCell = sal_True;
+    bInCell = true;
     sal_Bool bHorJustifyCenterTH = (pInfo->nToken == HTML_TABLEHEADER_ON);
     const HTMLOptions& rOptions = static_cast<HTMLParser*>(pInfo->pParser)->GetOptions();
     for (size_t i = 0, n = rOptions.size(); i < n; ++i)
@@ -1333,8 +1333,8 @@ void ScHTMLLayoutParser::TableOff( ImportInfo* pInfo )
             pActEntry = pE;
             delete pS;
         }
-        bTabInTabCell = sal_True;
-        bInCell = sal_True;
+        bTabInTabCell = true;
+        bInCell = true;
     }
     else
     {   // einfache Table beendet
@@ -1586,17 +1586,16 @@ void ScHTMLLayoutParser::ProcToken( ImportInfo* pInfo )
         break;
         case HTML_TITLE_ON:
         {
-            bInTitle = sal_True;
-            aString.Erase();
+            bInTitle = true;
+            aString = rtl::OUString();
         }
         break;
         case HTML_TITLE_OFF:
         {
-            if ( bInTitle && aString.Len() )
+            if ( bInTitle && !aString.isEmpty() )
             {
                 // Leerzeichen von Zeilenumbruechen raus
-                aString.EraseLeadingChars();
-                aString.EraseTrailingChars();
+                aString = aString.trim();
                 uno::Reference<document::XDocumentPropertiesSupplier> xDPS(
                     mpDoc->GetDocumentShell()->GetModel(),
                     uno::UNO_QUERY_THROW);
diff --git a/sc/source/filter/inc/htmlpars.hxx b/sc/source/filter/inc/htmlpars.hxx
index 9ba3d1063b42..bada5beaaa2a 100644
--- a/sc/source/filter/inc/htmlpars.hxx
+++ b/sc/source/filter/inc/htmlpars.hxx
@@ -168,9 +168,9 @@ class ScHTMLLayoutParser : public ScHTMLParser
 {
 private:
     Size                aPageSize;
-    String              aBaseURL;
+    rtl::OUString       aBaseURL;
     ScHTMLTableStack    aTableStack;
-    String              aString;
+    rtl::OUString       aString;
     ScRangeListRef      xLockedList;        // je Table
     Table*              pTables;
     ScHTMLColOffset*    pColOffset;
@@ -186,12 +186,10 @@ private:
     sal_uInt16              nColOffsetStart;    // Startwert je Table, in Pixel
     sal_uInt16              nMetaCnt;           // fuer ParseMetaOptions
     sal_uInt16              nOffsetTolerance;   // for use with SeekOffset and related
-    sal_Bool                bCalcWidthHeight;   // TRUE: calculate real column width
-                                            // FALSE: 1 html-col = 1 sc-col
-    sal_Bool                bTabInTabCell;
-    sal_Bool                bFirstRow;          // je Table, ob in erster Zeile
-    sal_Bool                bInCell;
-    sal_Bool                bInTitle;
+    bool                bTabInTabCell:1;
+    bool                bFirstRow:1;          // je Table, ob in erster Zeile
+    bool                bInCell:1;
+    bool                bInTitle:1;
 
     DECL_LINK( HTMLImportHdl, ImportInfo* );
     void                NewActEntry( ScEEParseEntry* );
@@ -388,7 +386,7 @@ public:
     virtual             ~ScHTMLTable();
 
     /** Returns the name of the table, specified in the TABLE tag. */
-    inline const String& GetTableName() const { return maTableName; }
+    inline const rtl::OUString& GetTableName() const { return maTableName; }
     /** Returns the unique identifier of the table. */
     inline ScHTMLTableId GetTableId() const { return maTableId.mnTableId; }
     /** Returns the table size. */
@@ -571,7 +569,7 @@ private:
 private:
     ScHTMLTable*        mpParentTable;      /// Pointer to parent table.
     ScHTMLTableMapPtr   mxNestedTables;     /// Table of nested HTML tables.
-    String              maTableName;        /// Table name from <table id> option.
+    rtl::OUString       maTableName;        /// Table name from <table id> option.
     ScHTMLTableAutoId   maTableId;          /// Unique identifier of this table.
     SfxItemSet          maTableItemSet;     /// Items for the entire table.
     SfxItemSetPtr       mxRowItemSet;       /// Items for the current table row.
@@ -589,11 +587,11 @@ private:
     ScHTMLPos           maCurrCell;         /// Address of current cell to fill.
     ScHTMLPos           maDocBasePos;       /// Resulting base address in a Calc document.
     ScHTMLParser*       mpParser;
-    bool                mbBorderOn;         /// true = Table borders on.
-    bool                mbPreFormText;      /// true = Table from preformatted text (<pre> tag).
-    bool                mbRowOn;            /// true = Inside of <tr> </tr>.
-    bool                mbDataOn;           /// true = Inside of <td> </td> or <th> </th>.
-    bool                mbPushEmptyLine;    /// true = Insert empty line before current entry.
+    bool                mbBorderOn:1;       /// true = Table borders on.
+    bool                mbPreFormText:1;    /// true = Table from preformatted text (<pre> tag).
+    bool                mbRowOn:1;          /// true = Inside of <tr> </tr>.
+    bool                mbDataOn:1;         /// true = Inside of <td> </td> or <th> </th>.
+    bool                mbPushEmptyLine:1;  /// true = Insert empty line before current entry.
 };
 
 // ----------------------------------------------------------------------------