[PATCH] String cleanup in filter

Ricardo Montania (via Code Review) gerrit at gerrit.libreoffice.org
Sat Dec 29 13:54:11 PST 2012


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1507

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/07/1507/1

String cleanup in filter

Change-Id: I2b8bd3535a5ba5323b9a17941c4345f4c1e0e822
---
M filter/inc/filter/msfilter/escherex.hxx
M filter/source/msfilter/escherex.cxx
M filter/source/msfilter/msocximex.cxx
M filter/source/msfilter/msoleexp.cxx
4 files changed, 246 insertions(+), 268 deletions(-)



diff --git a/filter/inc/filter/msfilter/escherex.hxx b/filter/inc/filter/msfilter/escherex.hxx
index 5182417..f4fb939 100644
--- a/filter/inc/filter/msfilter/escherex.hxx
+++ b/filter/inc/filter/msfilter/escherex.hxx
@@ -1001,13 +1001,13 @@
     static sal_Bool GetPropertyValue(
         ::com::sun::star::uno::Any& rAny,
         const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet > &,
-        const String& rPropertyName,
+        const OUString& rPropertyName,
         sal_Bool bTestPropertyAvailability = sal_False
     );
 
     static ::com::sun::star::beans::PropertyState GetPropertyState(
         const ::com::sun::star::uno::Reference < ::com::sun::star::beans::XPropertySet > &,
-        const String& rPropertyName
+        const OUString& rPropertyName
     );
 };
 
@@ -1249,7 +1249,7 @@
                     //             will be created, containing the BLIP directly (e.g. for Excel charts).
     sal_Bool    CreateGraphicProperties(
                     const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet > & rXPropSet,
-                    const String& rSource,
+                    const OUString& rSource,
                     const sal_Bool bCreateFillBitmap,
                     const sal_Bool bCreateCroppingAttributes = sal_False,
                     const sal_Bool bFillBitmapModeAllowed = sal_True
@@ -1641,7 +1641,7 @@
     virtual void AddChildAnchor( const Rectangle& rRectangle );
     virtual void AddClientAnchor( const Rectangle& rRectangle );
 
-    virtual sal_uInt32 EnterGroup( const String& rShapeName, const Rectangle* pBoundRect = 0 );
+    virtual sal_uInt32 EnterGroup( const OUString& rShapeName, const Rectangle* pBoundRect = 0 );
     sal_uInt32  EnterGroup( const Rectangle* pBoundRect = NULL );
     sal_uInt32  GetGroupLevel() const { return mnGroupLevel; };
     virtual sal_Bool SetGroupSnapRect( sal_uInt32 nGroupLevel, const Rectangle& rRect );
diff --git a/filter/source/msfilter/escherex.cxx b/filter/source/msfilter/escherex.cxx
index d078567..2f1807e 100644
--- a/filter/source/msfilter/escherex.cxx
+++ b/filter/source/msfilter/escherex.cxx
@@ -180,7 +180,7 @@
     AddOpt( nPropID, bBlib, nPropValue, NULL, 0 );
 }
 
-void EscherPropertyContainer::AddOpt( sal_uInt16 nPropID, const rtl::OUString& rString )
+void EscherPropertyContainer::AddOpt( sal_uInt16 nPropID, const OUString& rString )
 {
     sal_Int32 j, i, nLen = rString.getLength() * 2 + 2;
     sal_uInt8* pBuf = new sal_uInt8[ nLen ];
@@ -428,8 +428,7 @@
 {
     ::com::sun::star::uno::Any aAny;
     ::com::sun::star::awt::Gradient aGradient;
-    if ( EscherPropertyValueHelper::GetPropertyValue(
-            aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "FillGradient" ) ), sal_False ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "FillGradient" ), sal_False ) )
     {
         aGradient = *static_cast< const ::com::sun::star::awt::Gradient* >( aAny.getValue() );
     }
@@ -443,7 +442,7 @@
     ::com::sun::star::uno::Any aAny;
     AddOpt( ESCHER_Prop_WrapText, ESCHER_WrapNone );
     AddOpt( ESCHER_Prop_AnchorText, ESCHER_AnchorMiddle );
-    const rtl::OUString aPropName( String( RTL_CONSTASCII_USTRINGPARAM( "FillStyle" ) ) );
+    const OUString aPropName( "FillStyle" );
 
     if ( EscherPropertyValueHelper::GetPropertyValue(
             aAny, rXPropSet, aPropName, sal_False ) )
@@ -463,14 +462,14 @@
 
             case ::com::sun::star::drawing::FillStyle_BITMAP :
             {
-                CreateGraphicProperties( rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "FillBitmapURL" ) ), sal_True );
+                CreateGraphicProperties( rXPropSet, OUString( "FillBitmapURL" ), sal_True );
                 AddOpt( ESCHER_Prop_fNoFillHitTest, 0x140014 );
                 AddOpt( ESCHER_Prop_fillBackColor, nFillBackColor  );
             }
             break;
             case ::com::sun::star::drawing::FillStyle_HATCH :
             {
-                CreateGraphicProperties( rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "FillHatch" ) ), sal_True );
+                CreateGraphicProperties( rXPropSet, OUString( "FillHatch" ), sal_True );
             }
             break;
             case ::com::sun::star::drawing::FillStyle_SOLID :
@@ -481,8 +480,7 @@
                 if ( ePropState == ::com::sun::star::beans::PropertyState_DIRECT_VALUE )
                     AddOpt( ESCHER_Prop_fillType, ESCHER_FillSolid );
 
-                if ( EscherPropertyValueHelper::GetPropertyValue(
-                        aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "FillColor" ) ), sal_False ) )
+                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "FillColor" ), sal_False ) )
                 {
                     sal_uInt32 nFillColor = ImplGetColor( *((sal_uInt32*)aAny.getValue()) );
                     nFillBackColor = nFillColor ^ 0xffffff;
@@ -499,9 +497,8 @@
         if ( eFS != ::com::sun::star::drawing::FillStyle_NONE )
         {
             sal_uInt16 nTransparency = ( EscherPropertyValueHelper::GetPropertyValue(
-                                    aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "FillTransparence" ) ), sal_True ) )
-                                    ? *((sal_Int16*)aAny.getValue() )
-                                    : 0;
+                aAny, rXPropSet, OUString( "FillTransparence" ), sal_True ) )
+                ? *((sal_Int16*)aAny.getValue() ) : 0;
             if (  nTransparency )
                 AddOpt( ESCHER_Prop_fillOpacity, ( ( 100 - nTransparency ) << 16 ) / 100 );
         }
@@ -530,37 +527,37 @@
     sal_Bool bWordWrap          ( sal_False );
     sal_Bool bAutoGrowSize      ( sal_False );
 
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextWritingMode" ) ), sal_True ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextWritingMode" ), sal_True ) )
         aAny >>= eWM;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextVerticalAdjust" ) ), sal_True ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextVerticalAdjust" ), sal_True ) )
         aAny >>= eVA;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextHorizontalAdjust" ) ), sal_True ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextHorizontalAdjust" ), sal_True ) )
         aAny >>= eHA;
     if ( bIsCustomShape )
     {
-        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextWordWrap" ) ), sal_False ) )
+        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextWordWrap" ), sal_False ) )
             aAny >>= bWordWrap;
-        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextAutoGrowHeight" ) ), sal_True ) )
+        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextAutoGrowHeight" ), sal_True ) )
             aAny >>= bAutoGrowSize;
     }
     else if ( bIsTextFrame )
     {
-        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextAutoGrowWidth" ) ), sal_True ) )
+        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextAutoGrowWidth" ), sal_True ) )
             aAny >>= bAutoGrowWidth;
 
 // i63936 not setting autogrowheight, because otherwise
 // the minframeheight of the text will be ignored
 //
-//      if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextAutoGrowHeight" ) ), sal_True ) )
+//      if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, "TextAutoGrowHeight", sal_True ) )
 //          aAny >>= bAutoGrowHeight;
     }
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextLeftDistance" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextLeftDistance" ) ) )
         aAny >>= nLeft;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextUpperDistance" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextUpperDistance" ) ) )
         aAny >>= nTop;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextRightDistance" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextRightDistance" ) ) )
         aAny >>= nRight;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "TextLowerDistance" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "TextLowerDistance" ) ) )
         aAny >>= nBottom;
 
     ESCHER_AnchorText eAnchor = ESCHER_AnchorTop;
@@ -682,18 +679,18 @@
 
     // n#404221: In case of rotation we need to write the txtflTextFlow
     // attribute too.
-    if (bIsTextFrame) {
+    if (bIsTextFrame)
+    {
         sal_uInt16 nAngle = EscherPropertyValueHelper::GetPropertyValue(
-        aAny,
-        rXPropSet,
-        String( RTL_CONSTASCII_USTRINGPARAM( "RotateAngle" ) ),
-        sal_True )
-        ? (sal_uInt16)( ( *((sal_Int32*)aAny.getValue() ) ) + 5 ) / 10 : 0;
-        if (nAngle==900) {
-        AddOpt( ESCHER_Prop_txflTextFlow, ESCHER_txflBtoT );
+            aAny, rXPropSet, OUString( "RotateAngle" ), sal_True ) ?
+                (sal_uInt16)( ( *((sal_Int32*)aAny.getValue() ) ) + 5 ) / 10 : 0;
+        if (nAngle==900)
+        {
+            AddOpt( ESCHER_Prop_txflTextFlow, ESCHER_txflBtoT );
         }
-        if (nAngle==2700) {
-        AddOpt( ESCHER_Prop_txflTextFlow, ESCHER_txflTtoBA );
+        if (nAngle==2700)
+        {
+            AddOpt( ESCHER_Prop_txflTextFlow, ESCHER_txflTtoBA );
         }
     }
 }
@@ -702,13 +699,13 @@
     const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet > & rXPropSet,
         ESCHER_LineEnd& reLineEnd, sal_Int32& rnArrowLength, sal_Int32& rnArrowWidth )
 {
-    static String sLineStart    ( RTL_CONSTASCII_USTRINGPARAM( "LineStart" ) );
-    static String sLineStartName( RTL_CONSTASCII_USTRINGPARAM( "LineStartName" ) );
-    static String sLineEnd      ( RTL_CONSTASCII_USTRINGPARAM( "LineEnd" ) );
-    static String sLineEndName  ( RTL_CONSTASCII_USTRINGPARAM( "LineEndName" ) );
+    static OUString sLineStart    ( "LineStart" );
+    static OUString sLineStartName( "LineStartName" );
+    static OUString sLineEnd      ( "LineEnd" );
+    static OUString sLineEndName  ( "LineEndName" );
 
-    const String sLine      ( bLineStart ? sLineStart : sLineEnd );
-    const String sLineName  ( bLineStart ? sLineStartName : sLineEndName );
+    const OUString sLine      ( bLineStart ? sLineStart : sLineEnd );
+    const OUString sLineName  ( bLineStart ? sLineStartName : sLineEndName );
 
     sal_Bool bIsArrow = sal_False;
 
@@ -728,7 +725,7 @@
             if ( EscherPropertyValueHelper::GetPropertyValue(
                 aAny, rXPropSet, sLineName, sal_False ) )
             {
-                String          aArrowStartName = *(::rtl::OUString*)aAny.getValue();
+                OUString        aArrowStartName = *(OUString*)aAny.getValue();
                 sal_Int16       nWhich = bLineStart ? XATTR_LINESTART : XATTR_LINEEND;
 
                 OUString aApiName = SvxUnogetApiNameForItem(nWhich, aArrowStartName);
@@ -770,16 +767,16 @@
                 else if ( comphelper::string::getTokenCount(aArrowStartName, ' ') == 2 )
                 {
                     sal_Bool b = sal_True;
-                    String aArrowName( aArrowStartName.GetToken( 0, ' ' ) );
-                    if (  aArrowName.EqualsAscii( "msArrowEnd" ) )
+                    OUString aArrowName( aArrowStartName.getToken( 0, ' ' ) );
+                    if (  aArrowName == "msArrowEnd" )
                         reLineEnd = ESCHER_LineArrowEnd;
-                    else if (  aArrowName.EqualsAscii( "msArrowOpenEnd" ) )
+                    else if (  aArrowName == "msArrowOpenEnd" )
                         reLineEnd = ESCHER_LineArrowOpenEnd;
-                    else if ( aArrowName.EqualsAscii( "msArrowStealthEnd" ) )
+                    else if ( aArrowName == "msArrowStealthEnd" )
                         reLineEnd = ESCHER_LineArrowStealthEnd;
-                    else if ( aArrowName.EqualsAscii( "msArrowDiamondEnd" ) )
+                    else if ( aArrowName == "msArrowDiamondEnd" )
                         reLineEnd = ESCHER_LineArrowDiamondEnd;
-                    else if ( aArrowName.EqualsAscii( "msArrowOvalEnd" ) )
+                    else if ( aArrowName == "msArrowOvalEnd" )
                         reLineEnd = ESCHER_LineArrowOvalEnd;
                     else
                         b = sal_False;
@@ -787,8 +784,8 @@
                     // now we have the arrow, and try to determine the arrow size;
                     if ( b )
                     {
-                        String aArrowSize( aArrowStartName.GetToken( 1, ' ' ) );
-                        sal_Int32 nArrowSize = aArrowSize.ToInt32();
+                        OUString aArrowSize( aArrowStartName.getToken( 1, ' ' ) );
+                        sal_Int32 nArrowSize = aArrowSize.toInt32();
                         rnArrowWidth = ( nArrowSize - 1 ) / 3;
                         rnArrowLength = nArrowSize - ( rnArrowWidth * 3 ) - 1;
                     }
@@ -811,7 +808,7 @@
     sal_Int32 nArrowWidth;
 
     sal_Bool bSwapLineEnds = sal_False;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "CircleKind" ) ), sal_True ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "CircleKind" ), sal_True ) )
     {
         ::com::sun::star::drawing::CircleKind  eCircleKind;
         if ( aAny >>= eCircleKind )
@@ -836,7 +833,7 @@
     }
 
     // support LineCaps
-    if(EscherPropertyValueHelper::GetPropertyValue(aAny, rXPropSet, String(RTL_CONSTASCII_USTRINGPARAM("LineCap")), sal_False))
+    if(EscherPropertyValueHelper::GetPropertyValue(aAny, rXPropSet, OUString( "LineCap" ), sal_False))
     {
         ::com::sun::star::drawing::LineCap aLineCap(com::sun::star::drawing::LineCap_BUTT);
 
@@ -863,8 +860,7 @@
         }
     }
 
-    if ( EscherPropertyValueHelper::GetPropertyValue(
-        aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "LineStyle"  ) ), sal_False ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "LineStyle" ), sal_False ) )
     {
         ::com::sun::star::drawing::LineStyle eLS;
         if ( aAny >>= eLS )
@@ -877,8 +873,7 @@
 
                 case ::com::sun::star::drawing::LineStyle_DASH :
                 {
-                    if ( EscherPropertyValueHelper::GetPropertyValue(
-                        aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "LineDash" ) ), sal_False ) )
+                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "LineDash" ), sal_False ) )
                     {
                         ESCHER_LineDashing eDash = ESCHER_LineSolid;
                         ::com::sun::star::drawing::LineDash* pLineDash = (::com::sun::star::drawing::LineDash*)aAny.getValue();
@@ -933,8 +928,7 @@
                 break;
             }
         }
-        if ( EscherPropertyValueHelper::GetPropertyValue(
-            aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "LineColor"  ) ), sal_False ) )
+        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "LineColor" ), sal_False ) )
         {
             sal_uInt32 nLineColor = ImplGetColor( *((sal_uInt32*)aAny.getValue()) );
             AddOpt( ESCHER_Prop_lineColor, nLineColor );
@@ -942,16 +936,13 @@
         }
     }
 
-    sal_uInt32 nLineSize = ( EscherPropertyValueHelper::GetPropertyValue(
-        aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "LineWidth"  ) ), sal_False ) )
-        ? *((sal_uInt32*)aAny.getValue())
-        : 0;
+    sal_uInt32 nLineSize = ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "LineWidth" ), sal_False ) )
+        ? *((sal_uInt32*)aAny.getValue()) : 0;
     if ( nLineSize > 1 )
         AddOpt( ESCHER_Prop_lineWidth, nLineSize * 360 );       // 100TH MM -> PT , 1PT = 12700 EMU
 
     ESCHER_LineJoin eLineJoin = ESCHER_LineJoinMiter;
-    if ( EscherPropertyValueHelper::GetPropertyValue(
-        aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "LineJoint" ) ), sal_True ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "LineJoint" ), sal_True ) )
     {
         ::com::sun::star::drawing::LineJoint eLJ;
         if ( aAny >>= eLJ )
@@ -1007,25 +998,25 @@
     double fGamma = 1.0;
     sal_Int16 nTransparency = 0;
 
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "GraphicColorMode" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "GraphicColorMode" ) ) )
         aAny >>= eColorMode;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "AdjustLuminance" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "AdjustLuminance" ) ) )
         aAny >>= nLuminance;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "AdjustContrast" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "AdjustContrast" ) ) )
     {
         sal_Int16 nC = sal_Int16();
         aAny >>= nC;
         nContrast = nC;
     }
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "AdjustRed" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "AdjustRed" ) ) )
         aAny >>= nRed;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "AdjustGreen" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "AdjustGreen" ) ) )
         aAny >>= nGreen;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "AdjustBlue" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "AdjustBlue" ) ) )
         aAny >>= nBlue;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "Gamma" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "Gamma" ) ) )
         aAny >>= fGamma;
-    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "Transparency" ) ) ) )
+    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "Transparency" ) ) )
         aAny >>= nTransparency;
 
     if ( eColorMode == ::com::sun::star::drawing::ColorMode_WATERMARK )
@@ -1073,7 +1064,7 @@
             Size aCropSize(lcl_SizeToEmu(aPrefSize, aPrefMapMode));
             if ( aCropSize.Width() && aCropSize.Height() )
             {
-                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "GraphicCrop" ) ) ) )
+                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "GraphicCrop" ) ) )
                 {
                     ::com::sun::star::text::GraphicCrop aGraphCrop;
                     if ( aAny >>= aGraphCrop )
@@ -1113,13 +1104,13 @@
         sal_Bool bVal = false;
         ::com::sun::star::uno::Any aAny;
         sal_uInt32 nShapeAttr = 0;
-        EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "Visible" ) ), sal_True );
+        EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "Visible" ), sal_True );
         if ( aAny >>= bVal )
         {
             if ( !bVal )
                 nShapeAttr |= 0x20002;  // set fHidden = true
         }
-        EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "Printable" ) ), sal_True );
+        EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "Printable" ), sal_True );
         if ( aAny >>= bVal )
         {
             if ( !bVal )
@@ -1145,7 +1136,7 @@
             if ( pGraphic )
             {
                 GraphicObject aGraphicObject( *pGraphic );
-                rtl::OString aUniqueId( aGraphicObject.GetUniqueID() );
+                OString aUniqueId( aGraphicObject.GetUniqueID() );
                 if ( !aUniqueId.isEmpty() )
                 {
                     AddOpt( ESCHER_Prop_fillType, ESCHER_FillPicture );
@@ -1155,7 +1146,7 @@
                     {
                         ::com::sun::star::uno::Any aAny;
                         ::com::sun::star::awt::Rectangle* pVisArea = NULL;
-                        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "VisibleArea" ) ) ) )
+                        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "VisibleArea" ) ) )
                         {
                             pVisArea = new ::com::sun::star::awt::Rectangle;
                             aAny >>= (*pVisArea);
@@ -1178,7 +1169,7 @@
 }
 
 
-sal_Bool EscherPropertyContainer::ImplCreateEmbeddedBmp( const rtl::OString& rUniqueId )
+sal_Bool EscherPropertyContainer::ImplCreateEmbeddedBmp( const OString& rUniqueId )
 {
     if( !rUniqueId.isEmpty() )
     {
@@ -1200,19 +1191,19 @@
 }
 
 sal_Bool EscherPropertyContainer::CreateEmbeddedBitmapProperties(
-    const ::rtl::OUString& rBitmapUrl, ::com::sun::star::drawing::BitmapMode eBitmapMode )
+    const OUString& rBitmapUrl, ::com::sun::star::drawing::BitmapMode eBitmapMode )
 {
     sal_Bool bRetValue = sal_False;
-    String aVndUrl( RTL_CONSTASCII_USTRINGPARAM( "vnd.sun.star.GraphicObject:" ) );
-    String aBmpUrl( rBitmapUrl );
-    xub_StrLen nIndex = aBmpUrl.Search( aVndUrl, 0 );
+    OUString aVndUrl( "vnd.sun.star.GraphicObject:" );
+    OUString aBmpUrl( rBitmapUrl );
+    sal_Int32 nIndex = aBmpUrl.indexOf( aVndUrl, 0 );
     if( nIndex != STRING_NOTFOUND )
     {
         // note: += ist not defined for xub_StrLen -> conversion to int and back to xub_StrLen
-        nIndex = nIndex + aVndUrl.Len();
-        if( aBmpUrl.Len() > nIndex )
+        nIndex = nIndex + aVndUrl.getLength();
+        if( aBmpUrl.getLength() > nIndex )
         {
-            rtl::OString aUniqueId(rtl::OUStringToOString(aBmpUrl.Copy(nIndex, aBmpUrl.Len() - nIndex), RTL_TEXTENCODING_UTF8));
+            OString aUniqueId(OUStringToOString(aBmpUrl.copy(nIndex, aBmpUrl.getLength() - nIndex), RTL_TEXTENCODING_UTF8));
             bRetValue = ImplCreateEmbeddedBmp( aUniqueId );
             if( bRetValue )
             {
@@ -1256,7 +1247,7 @@
 sal_Bool EscherPropertyContainer::CreateEmbeddedHatchProperties( const ::com::sun::star::drawing::Hatch& rHatch, const Color& rBackColor, bool bFillBackground )
 {
     GraphicObject aGraphicObject = lclDrawHatch( rHatch, rBackColor, bFillBackground );
-    rtl::OString aUniqueId = aGraphicObject.GetUniqueID();
+    OString aUniqueId = aGraphicObject.GetUniqueID();
     sal_Bool bRetValue = ImplCreateEmbeddedBmp( aUniqueId );
     if ( bRetValue )
         AddOpt( ESCHER_Prop_fillType, ESCHER_FillTexture );
@@ -1266,7 +1257,7 @@
 
 sal_Bool EscherPropertyContainer::CreateGraphicProperties(
     const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet > & rXPropSet,
-        const String& rSource, const sal_Bool bCreateFillBitmap, const sal_Bool bCreateCroppingAttributes,
+        const OUString& rSource, const sal_Bool bCreateFillBitmap, const sal_Bool bCreateCroppingAttributes,
             const sal_Bool bFillBitmapModeAllowed )
 {
     sal_Bool        bRetValue = sal_False;
@@ -1276,8 +1267,8 @@
     sal_Bool        bRotate   = sal_True;
     GraphicAttr*    pGraphicAttr = NULL;
     GraphicObject   aGraphicObject;
-    String          aGraphicUrl;
-    rtl::OString    aUniqueId;
+    OUString        aGraphicUrl;
+    OString         aUniqueId;
     bool            bIsGraphicMtf(false);
 
     ::com::sun::star::drawing::BitmapMode   eBitmapMode( ::com::sun::star::drawing::BitmapMode_NO_REPEAT );
@@ -1286,7 +1277,7 @@
     if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, rSource ) )
     {
         sal_uInt16 nAngle = 0;
-        if ( rSource == String( RTL_CONSTASCII_USTRINGPARAM( "MetaFile" ) ) )
+        if ( rSource == "MetaFile" )
         {
             ::com::sun::star::uno::Sequence<sal_uInt8> aSeq = *(::com::sun::star::uno::Sequence<sal_uInt8>*)aAny.getValue();
             const sal_uInt8*    pAry = aSeq.getArray();
@@ -1308,7 +1299,7 @@
                 }
             }
         }
-        else if ( rSource == String( RTL_CONSTASCII_USTRINGPARAM( "Bitmap" ) ) )
+        else if ( rSource == "Bitmap" )
         {
             ::com::sun::star::uno::Reference< ::com::sun::star::awt::XBitmap >xBitmap;
             if ( ::cppu::extractInterface( xBitmap, aAny ) )
@@ -1324,29 +1315,27 @@
                 }
             }
         }
-        else if ( rSource == String( RTL_CONSTASCII_USTRINGPARAM( "FillBitmapURL" ) ) )
+        else if ( rSource == "FillBitmapURL" )
         {
-            aGraphicUrl = *(::rtl::OUString*)aAny.getValue();
+            aGraphicUrl = *(OUString*)aAny.getValue();
         }
-        else if ( rSource == String( RTL_CONSTASCII_USTRINGPARAM( "GraphicURL" ) ) )
+        else if ( rSource == "GraphicURL" )
         {
-            aGraphicUrl = *(::rtl::OUString*)aAny.getValue();
+            aGraphicUrl = *(OUString*)aAny.getValue();
             bCreateFillStyles = sal_True;
         }
-        else if ( rSource == String( RTL_CONSTASCII_USTRINGPARAM( "FillHatch" ) ) )
+        else if ( rSource == "FillHatch" )
         {
             ::com::sun::star::drawing::Hatch aHatch;
             if ( aAny >>= aHatch )
             {
                 Color aBackColor;
-                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                    String( RTL_CONSTASCII_USTRINGPARAM( "FillColor" ) ), sal_False ) )
+                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "FillColor" ), sal_False ) )
                 {
                     aBackColor = ImplGetColor( *((sal_uInt32*)aAny.getValue()), sal_False );
                 }
                 bool bFillBackground = false;
-                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                        String( RTL_CONSTASCII_USTRINGPARAM( "FillBackground" ) ), sal_True ) )
+                if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "FillBackground" ), sal_True ) )
                 {
                     aAny >>= bFillBackground;
                 }
@@ -1357,31 +1346,30 @@
             }
         }
 
-        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "IsMirrored" ) ), sal_True ) )
+        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "IsMirrored" ), sal_True ) )
             aAny >>= bMirrored;
 
         if ( bCreateFillBitmap && bFillBitmapModeAllowed )
         {
-            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "FillBitmapMode" ) ), sal_True ) )
+            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "FillBitmapMode" ), sal_True ) )
                 aAny >>= eBitmapMode;
         }
         else
         {
-            nAngle = bRotate && EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                                                                             String( RTL_CONSTASCII_USTRINGPARAM( "RotateAngle" ) ), sal_True )
+            nAngle = bRotate && EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "RotateAngle" ), sal_True )
                 ? (sal_uInt16)( ( *((sal_Int32*)aAny.getValue() ) ) + 5 ) / 10
                 : 0;
         }
 
-        if ( aGraphicUrl.Len() )
+        if ( aGraphicUrl.getLength() )
         {
-            String aVndUrl( RTL_CONSTASCII_USTRINGPARAM( "vnd.sun.star.GraphicObject:" ) );
-            xub_StrLen nIndex = aGraphicUrl.Search( aVndUrl, 0 );
+            OUString aVndUrl( "vnd.sun.star.GraphicObject:" );
+            sal_Int32 nIndex = aGraphicUrl.indexOf( aVndUrl, 0 );
             if ( nIndex != STRING_NOTFOUND )
             {
-                nIndex = nIndex + aVndUrl.Len();
-                if ( aGraphicUrl.Len() > nIndex  )
-                    aUniqueId = rtl::OUStringToOString(aGraphicUrl.Copy(nIndex, aGraphicUrl.Len() - nIndex), RTL_TEXTENCODING_UTF8);
+                nIndex = nIndex + aVndUrl.getLength();
+                if ( aGraphicUrl.getLength() > nIndex  )
+                    aUniqueId = OUStringToOString(aGraphicUrl.copy(nIndex, aGraphicUrl.getLength() - nIndex), RTL_TEXTENCODING_UTF8);
             }
             else
             {
@@ -1428,11 +1416,11 @@
                 {
                     if ( pGraphicProvider )
                     {
-                        const rtl::OUString& rBaseURI( pGraphicProvider->GetBaseURI() );
+                        const OUString& rBaseURI( pGraphicProvider->GetBaseURI() );
                         INetURLObject aBaseURI( rBaseURI );
                         if( aBaseURI.GetProtocol() == aTmp.GetProtocol() )
                         {
-                            rtl::OUString aRelUrl( INetURLObject::GetRelURL( rBaseURI, aGraphicUrl,
+                            OUString aRelUrl( INetURLObject::GetRelURL( rBaseURI, aGraphicUrl,
                                                     INetURLObject::WAS_ENCODED, INetURLObject::DECODE_TO_IURI, RTL_TEXTENCODING_UTF8, INetURLObject::FSYS_DETECT ) );
                             if ( !aRelUrl.isEmpty() )
                                 aGraphicUrl = aRelUrl;
@@ -1442,7 +1430,7 @@
             }
         }
 
-        if ( aGraphicUrl.Len() || !aUniqueId.isEmpty() )
+        if ( aGraphicUrl.getLength() || !aUniqueId.isEmpty() )
         {
             if ( bMirrored || nAngle )
             {
@@ -1511,7 +1499,7 @@
             // write out link to graphic
             else
             {
-                OSL_ASSERT(aGraphicUrl.Len());
+                OSL_ASSERT(aGraphicUrl.getLength());
 
                 AddOpt( ESCHER_Prop_pibName, aGraphicUrl );
                 sal_uInt32  nPibFlags=0;
@@ -1535,9 +1523,9 @@
     ::com::sun::star::uno::Any aAny( rXShape->queryInterface(
         ::getCppuType( (const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet >*) 0 ) ));
 
-    String sPolyPolygonBezier( RTL_CONSTASCII_USTRINGPARAM( "PolyPolygonBezier" ) );
-    String sPolyPolygon     ( RTL_CONSTASCII_USTRINGPARAM( "PolyPolygon" ) );
-    String sPolygon         ( RTL_CONSTASCII_USTRINGPARAM( "Polygon" ) );
+    OUString sPolyPolygonBezier ( "PolyPolygonBezier" );
+    OUString sPolyPolygon       ( "PolyPolygon" );
+    OUString sPolygon           ( "Polygon" );
 
     if ( aAny >>= aXPropSet )
     {
@@ -1681,8 +1669,8 @@
                 ::com::sun::star::awt::Rectangle& rGeoRect,
                     Polygon* pPolygon )
 {
-    static String sPolyPolygonBezier( RTL_CONSTASCII_USTRINGPARAM( "PolyPolygonBezier" ) );
-    static String sPolyPolygon      ( RTL_CONSTASCII_USTRINGPARAM( "PolyPolygon" ) );
+    static OUString sPolyPolygonBezier  ( "PolyPolygonBezier" );
+    static OUString sPolyPolygon        ( "PolyPolygon" );
 
     sal_Bool    bRetValue = sal_True;
     sal_Bool    bLine = ( nFlags & ESCHER_CREATEPOLYGON_LINE ) != 0;
@@ -1828,11 +1816,11 @@
     EscherSolverContainer& rSolverContainer, ::com::sun::star::awt::Rectangle& rGeoRect,
             sal_uInt16& rShapeType, sal_uInt16& rShapeFlags )
 {
-    static String sEdgeKind             ( RTL_CONSTASCII_USTRINGPARAM( "EdgeKind" ) );
-    static String sEdgeStartPoint       ( RTL_CONSTASCII_USTRINGPARAM( "EdgeStartPoint" ) );
-    static String sEdgeEndPoint         ( RTL_CONSTASCII_USTRINGPARAM( "EdgeEndPoint" ) );
-    static String sEdgeStartConnection  ( RTL_CONSTASCII_USTRINGPARAM( "EdgeStartConnection" ) );
-    static String sEdgeEndConnection    ( RTL_CONSTASCII_USTRINGPARAM( "EdgeEndConnection" ) );
+    static OUString sEdgeKind             ( "EdgeKind" );
+    static OUString sEdgeStartPoint       ( "EdgeStartPoint" );
+    static OUString sEdgeEndPoint         ( "EdgeEndPoint" );
+    static OUString sEdgeStartConnection  ( "EdgeStartConnection" );
+    static OUString sEdgeEndConnection    ( "EdgeEndConnection" );
 
     sal_Bool bRetValue = sal_False;
     rShapeType = rShapeFlags = 0;
@@ -1934,25 +1922,20 @@
     sal_uInt32 nShadowFlags = 0x20000;
     if ( ( nLineFlags & 8 ) || ( nFillFlags & 0x10 ) || bGraphic )
     {
-        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                String( RTL_CONSTASCII_USTRINGPARAM( "Shadow" ) ), sal_True ) )
+        if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "Shadow" ), sal_True ) )
         {
             if ( aAny >>= bHasShadow )
             {
                 if ( bHasShadow )
                 {
                     nShadowFlags |= 2;
-                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                            String( RTL_CONSTASCII_USTRINGPARAM( "ShadowColor" ) ), sal_False ) )
+                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "ShadowColor" ), sal_False ) )
                         AddOpt( ESCHER_Prop_shadowColor, ImplGetColor( *((sal_uInt32*)aAny.getValue()) ) );
-                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                            String( RTL_CONSTASCII_USTRINGPARAM( "ShadowXDistance" ) ), sal_False ) )
+                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "ShadowXDistance" ), sal_False ) )
                         AddOpt( ESCHER_Prop_shadowOffsetX, *((sal_Int32*)aAny.getValue()) * 360 );
-                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                            String( RTL_CONSTASCII_USTRINGPARAM( "ShadowYDistance" ) ), sal_False ) )
+                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "ShadowYDistance" ), sal_False ) )
                         AddOpt( ESCHER_Prop_shadowOffsetY, *((sal_Int32*)aAny.getValue()) * 360 );
-                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet,
-                            String( RTL_CONSTASCII_USTRINGPARAM( "ShadowTransparence" ) ), sal_False ) )
+                    if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, OUString( "ShadowTransparence" ), sal_False ) )
                         AddOpt( ESCHER_Prop_shadowOpacity,  0x10000 - (((sal_uInt32)*((sal_uInt16*)aAny.getValue())) * 655 ) );
                 }
             }
@@ -2046,8 +2029,8 @@
 {
     if ( pCustoShape )
     {
-        uno::Sequence< rtl::OUString > sEquationSource;
-        const rtl::OUString sEquations( "Equations"  );
+        uno::Sequence< OUString > sEquationSource;
+        const OUString sEquations( "Equations"  );
         SdrCustomShapeGeometryItem& rGeometryItem = (SdrCustomShapeGeometryItem&)(const SdrCustomShapeGeometryItem&)
             pCustoShape->GetMergedItem( SDRATTR_CUSTOMSHAPE_GEOMETRY );
         const uno::Any* pAny = ((SdrCustomShapeGeometryItem&)rGeometryItem).GetPropertyValueByName( sEquations );
@@ -2177,19 +2160,19 @@
     if ( aXPropSet.is() )
     {
         SdrObjCustomShape* pCustoShape = (SdrObjCustomShape*)GetSdrObjectFromXShape( rXShape );
-        const rtl::OUString sCustomShapeGeometry( "CustomShapeGeometry"  );
+        const OUString sCustomShapeGeometry( "CustomShapeGeometry"  );
         uno::Any aGeoPropSet = aXPropSet->getPropertyValue( sCustomShapeGeometry );
         uno::Sequence< beans::PropertyValue > aGeoPropSeq;
         if ( aGeoPropSet >>= aGeoPropSeq )
         {
-            const rtl::OUString sViewBox            ( "ViewBox"  );
-            const rtl::OUString sTextRotateAngle    ( "TextRotateAngle"  );
-            const rtl::OUString sExtrusion          ( "Extrusion"  );
-            const rtl::OUString sEquations          ( "Equations"  );
-            const rtl::OUString sPath               ( "Path"  );
-            const rtl::OUString sTextPath           ( "TextPath"  );
-            const rtl::OUString sHandles            ( "Handles"  );
-            const rtl::OUString sAdjustmentValues   ( "AdjustmentValues"  );
+            const OUString sViewBox            ( "ViewBox"  );
+            const OUString sTextRotateAngle    ( "TextRotateAngle"  );
+            const OUString sExtrusion          ( "Extrusion"  );
+            const OUString sEquations          ( "Equations"  );
+            const OUString sPath               ( "Path"  );
+            const OUString sTextPath           ( "TextPath"  );
+            const OUString sHandles            ( "Handles"  );
+            const OUString sAdjustmentValues   ( "AdjustmentValues"  );
 
             const beans::PropertyValue* pAdjustmentValuesProp = NULL;
             sal_Int32 nAdjustmentsWhichNeedsToBeConverted = 0;
@@ -2254,28 +2237,28 @@
                         for ( r = 0; r < nrCount; r++ )
                         {
                             const beans::PropertyValue& rrProp = aExtrusionPropSeq[ r ];
-                            const rtl::OUString sExtrusionBrightness            ( "Brightness"  );
-                            const rtl::OUString sExtrusionDepth                 ( "Depth"  );
-                            const rtl::OUString sExtrusionDiffusion             ( "Diffusion"  );
-                            const rtl::OUString sExtrusionNumberOfLineSegments  ( "NumberOfLineSegments"  );
-                            const rtl::OUString sExtrusionLightFace             ( "LightFace"  );
-                            const rtl::OUString sExtrusionFirstLightHarsh       ( "FirstLightHarsh"  );
-                            const rtl::OUString sExtrusionSecondLightHarsh      ( "SecondLightHarsh"  );
-                            const rtl::OUString sExtrusionFirstLightLevel       ( "FirstLightLevel"  );
-                            const rtl::OUString sExtrusionSecondLightLevel      ( "SecondLightLevel"  );
-                            const rtl::OUString sExtrusionFirstLightDirection   ( "FirstLightDirection"  );
-                            const rtl::OUString sExtrusionSecondLightDirection  ( "SecondLightDirection"  );
-                            const rtl::OUString sExtrusionMetal                 ( "Metal"  );
-                            const rtl::OUString sExtrusionShadeMode             ( "ShadeMode"  );
-                            const rtl::OUString sExtrusionRotateAngle           ( "RotateAngle"  );
-                            const rtl::OUString sExtrusionRotationCenter        ( "RotationCenter"  );
-                            const rtl::OUString sExtrusionShininess             ( "Shininess"  );
-                            const rtl::OUString sExtrusionSkew                  ( "Skew"  );
-                            const rtl::OUString sExtrusionSpecularity           ( "Specularity"  );
-                            const rtl::OUString sExtrusionProjectionMode        ( "ProjectionMode"  );
-                            const rtl::OUString sExtrusionViewPoint             ( "ViewPoint"  );
-                            const rtl::OUString sExtrusionOrigin                ( "Origin"  );
-                            const rtl::OUString sExtrusionColor                 ( "Color"  );
+                            const OUString sExtrusionBrightness            ( "Brightness"  );
+                            const OUString sExtrusionDepth                 ( "Depth"  );
+                            const OUString sExtrusionDiffusion             ( "Diffusion"  );
+                            const OUString sExtrusionNumberOfLineSegments  ( "NumberOfLineSegments"  );
+                            const OUString sExtrusionLightFace             ( "LightFace"  );
+                            const OUString sExtrusionFirstLightHarsh       ( "FirstLightHarsh"  );
+                            const OUString sExtrusionSecondLightHarsh      ( "SecondLightHarsh"  );
+                            const OUString sExtrusionFirstLightLevel       ( "FirstLightLevel"  );
+                            const OUString sExtrusionSecondLightLevel      ( "SecondLightLevel"  );
+                            const OUString sExtrusionFirstLightDirection   ( "FirstLightDirection"  );
+                            const OUString sExtrusionSecondLightDirection  ( "SecondLightDirection"  );
+                            const OUString sExtrusionMetal                 ( "Metal"  );
+                            const OUString sExtrusionShadeMode             ( "ShadeMode"  );
+                            const OUString sExtrusionRotateAngle           ( "RotateAngle"  );
+                            const OUString sExtrusionRotationCenter        ( "RotationCenter"  );
+                            const OUString sExtrusionShininess             ( "Shininess"  );
+                            const OUString sExtrusionSkew                  ( "Skew"  );
+                            const OUString sExtrusionSpecularity           ( "Specularity"  );
+                            const OUString sExtrusionProjectionMode        ( "ProjectionMode"  );
+                            const OUString sExtrusionViewPoint             ( "ViewPoint"  );
+                            const OUString sExtrusionOrigin                ( "Origin"  );
+                            const OUString sExtrusionColor                 ( "Color"  );
 
                             if ( rrProp.Name.equals( sExtrusion ) )
                             {
@@ -2523,8 +2506,7 @@
                                     {
                                         nLightFaceFlags |= 2;
                                         uno::Any aFillColor2;
-                                        if ( EscherPropertyValueHelper::GetPropertyValue( aFillColor2, aXPropSet,
-                                            String( RTL_CONSTASCII_USTRINGPARAM( "FillColor2" ) ), sal_True ) )
+                                        if ( EscherPropertyValueHelper::GetPropertyValue( aFillColor2, aXPropSet, OUString( "FillColor2" ), sal_True ) )
                                         {
                                             sal_uInt32 nFillColor = ImplGetColor( *((sal_uInt32*)aFillColor2.getValue()) );
                                             AddOpt( DFF_Prop_c3DExtrusionColor, nFillColor );
@@ -2590,16 +2572,16 @@
                         for ( r = 0; r < nrCount; r++ )
                         {
                             const beans::PropertyValue& rrProp = aPathPropSeq[ r ];
-                            const rtl::OUString sPathExtrusionAllowed               ( "ExtrusionAllowed"  );
-                            const rtl::OUString sPathConcentricGradientFillAllowed  ( "ConcentricGradientFillAllowed"  );
-                            const rtl::OUString sPathTextPathAllowed                ( "TextPathAllowed"  );
-                            const rtl::OUString sPathCoordinates                    ( "Coordinates"  );
-                            const rtl::OUString sPathGluePoints                     ( "GluePoints"  );
-                            const rtl::OUString sPathGluePointType                  ( "GluePointType"  );
-                            const rtl::OUString sPathSegments                       ( "Segments"  );
-                            const rtl::OUString sPathStretchX                       ( "StretchX"  );
-                            const rtl::OUString sPathStretchY                       ( "StretchY"  );
-                            const rtl::OUString sPathTextFrames                     ( "TextFrames"  );
+                            const OUString sPathExtrusionAllowed               ( "ExtrusionAllowed"  );
+                            const OUString sPathConcentricGradientFillAllowed  ( "ConcentricGradientFillAllowed"  );
+                            const OUString sPathTextPathAllowed                ( "TextPathAllowed"  );
+                            const OUString sPathCoordinates                    ( "Coordinates"  );
+                            const OUString sPathGluePoints                     ( "GluePoints"  );
+                            const OUString sPathGluePointType                  ( "GluePointType"  );
+                            const OUString sPathSegments                       ( "Segments"  );
+                            const OUString sPathStretchX                       ( "StretchX"  );
+                            const OUString sPathStretchY                       ( "StretchY"  );
+                            const OUString sPathTextFrames                     ( "TextFrames"  );
 
                             if ( rrProp.Name.equals( sPathExtrusionAllowed ) )
                             {
@@ -2907,9 +2889,9 @@
                         for ( r = 0; r < nrCount; r++ )
                         {
                             const beans::PropertyValue& rrProp = aTextPathPropSeq[ r ];
-                            const rtl::OUString sTextPathMode       ( "TextPathMode"  );
-                            const rtl::OUString sTextPathScaleX     ( "ScaleX"  );
-                            const rtl::OUString sSameLetterHeights  ( "SameLetterHeights"  );
+                            const OUString sTextPathMode       ( "TextPathMode"  );
+                            const OUString sTextPathScaleX     ( "ScaleX"  );
+                            const OUString sSameLetterHeights  ( "SameLetterHeights"  );
 
                             if ( rrProp.Name.equals( sTextPath ) )
                             {
@@ -2971,24 +2953,24 @@
                         if ( nTextPathFlags & 0x4000 )      // Is FontWork ?
                         {
                             // FontWork Text
-                            rtl::OUString aText;
+                            OUString aText;
                             uno::Reference< text::XSimpleText > xText( rXShape, uno::UNO_QUERY );
                             if ( xText.is() )
                                 aText = xText->getString();
                             if ( aText.isEmpty() )
-                                aText = ::rtl::OUString( "your text" );   // TODO: moving into a resource
+                                aText = "your text";   // TODO: moving into a resource
                             AddOpt( DFF_Prop_gtextUNICODE, aText );
 
                             // FontWork Font
-                            rtl::OUString aFontName;
-                            const rtl::OUString sCharFontName           ( "CharFontName"  );
+                            OUString aFontName;
+                            const OUString sCharFontName( "CharFontName"  );
                             uno::Any aAny = aXPropSet->getPropertyValue( sCharFontName );
                             aAny >>= aFontName;
                             if ( aFontName.isEmpty() )
-                                aFontName = ::rtl::OUString( "Arial Black" );
+                                aFontName = "Arial Black";
                             AddOpt( DFF_Prop_gtextFont, aFontName );
 
-                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "CharScaleWidth" ) ), sal_True ) )
+                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "CharScaleWidth" ), sal_True ) )
                             {
                                 sal_Int16 nCharScaleWidth = 100;
                                 if ( aAny >>= nCharScaleWidth )
@@ -3000,7 +2982,7 @@
                                     }
                                 }
                             }
-                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "CharKerning" ) ), sal_True ) )
+                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "CharKerning" ), sal_True ) )
                             {
                                 sal_Int16 nCharKerning = sal_Int16();
                                 if ( aAny >>= nCharKerning )
@@ -3012,7 +2994,7 @@
                                         nTextPathFlags &=~0x1000;
                                 }
                             }
-                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "CharPosture" ) ), sal_True ) )
+                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "CharPosture" ), sal_True ) )
                             {
                                 awt::FontSlant eFontSlant;
                                 if ( aAny >>= eFontSlant )
@@ -3024,7 +3006,7 @@
                                         nTextPathFlags &=~0x10;
                                 }
                             }
-                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, String( RTL_CONSTASCII_USTRINGPARAM( "CharWeight" ) ), sal_True ) )
+                            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "CharWeight" ), sal_True ) )
                             {
                                 float fFontWidth = 0;
                                 if ( aAny >>= fFontWidth )
@@ -3075,17 +3057,17 @@
                                     {
                                         const beans::PropertyValue& rPropVal = rPropSeq[ j ];
 
-                                        const rtl::OUString sPosition           ( "Position"  );
-                                        const rtl::OUString sMirroredX          ( "MirroredX"  );
-                                        const rtl::OUString sMirroredY          ( "MirroredY"  );
-                                        const rtl::OUString sSwitched           ( "Switched"  );
-                                        const rtl::OUString sPolar              ( "Polar"  );
-                                        const rtl::OUString sRadiusRangeMinimum ( "RadiusRangeMinimum"  );
-                                        const rtl::OUString sRadiusRangeMaximum ( "RadiusRangeMaximum"  );
-                                        const rtl::OUString sRangeXMinimum      ( "RangeXMinimum"  );
-                                        const rtl::OUString sRangeXMaximum      ( "RangeXMaximum"  );
-                                        const rtl::OUString sRangeYMinimum      ( "RangeYMinimum"  );
-                                        const rtl::OUString sRangeYMaximum      ( "RangeYMaximum"  );
+                                        const OUString sPosition           ( "Position"  );
+                                        const OUString sMirroredX          ( "MirroredX"  );
+                                        const OUString sMirroredY          ( "MirroredY"  );
+                                        const OUString sSwitched           ( "Switched"  );
+                                        const OUString sPolar              ( "Polar"  );
+                                        const OUString sRadiusRangeMinimum ( "RadiusRangeMinimum"  );
+                                        const OUString sRadiusRangeMaximum ( "RadiusRangeMaximum"  );
+                                        const OUString sRangeXMinimum      ( "RangeXMinimum"  );
+                                        const OUString sRangeXMaximum      ( "RangeXMaximum"  );
+                                        const OUString sRangeYMinimum      ( "RangeYMinimum"  );
+                                        const OUString sRangeYMaximum      ( "RangeYMaximum"  );
 
                                         if ( rPropVal.Name.equals( sPosition ) )
                                         {
@@ -3252,7 +3234,7 @@
     }
 }
 
-MSO_SPT EscherPropertyContainer::GetCustomShapeType( const uno::Reference< drawing::XShape > & rXShape, sal_uInt32& nMirrorFlags, rtl::OUString& rShapeType )
+MSO_SPT EscherPropertyContainer::GetCustomShapeType( const uno::Reference< drawing::XShape > & rXShape, sal_uInt32& nMirrorFlags, OUString& rShapeType )
 {
     MSO_SPT eShapeType = mso_sptNil;
     nMirrorFlags = 0;
@@ -3299,7 +3281,7 @@
 
 MSO_SPT EscherPropertyContainer::GetCustomShapeType( const uno::Reference< drawing::XShape > & rXShape, sal_uInt32& nMirrorFlags )
 {
-    rtl::OUString aShapeType;
+    OUString aShapeType;
     return GetCustomShapeType( rXShape, nMirrorFlags, aShapeType );
 }
 
@@ -3385,7 +3367,7 @@
 sal_Bool EscherPropertyValueHelper::GetPropertyValue(
     ::com::sun::star::uno::Any& rAny,
     const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet > & rXPropSet,
-    const String& rString,
+    const OUString& rString,
     sal_Bool bTestPropertyAvailability
 ) {
     sal_Bool bRetValue = sal_True;
@@ -3422,7 +3404,7 @@
 
 ::com::sun::star::beans::PropertyState EscherPropertyValueHelper::GetPropertyState(
     const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet > & rXPropSet,
-        const String& rPropertyName )
+        const OUString& rPropertyName )
 {
     ::com::sun::star::beans::PropertyState eRetValue = ::com::sun::star::beans::PropertyState_AMBIGUOUS_VALUE;
     try
@@ -3438,7 +3420,7 @@
     return eRetValue;
 }
 
-EscherBlibEntry::EscherBlibEntry( sal_uInt32 nPictureOffset, const GraphicObject& rObject, const rtl::OString& rId,
+EscherBlibEntry::EscherBlibEntry( sal_uInt32 nPictureOffset, const GraphicObject& rObject, const OString& rId,
                                         const GraphicAttr* pGraphicAttr ) :
     mnPictureOffset ( nPictureOffset ),
     mnRefCount      ( 1 ),
@@ -3676,7 +3658,7 @@
     return bInRange;
 }
 
-sal_uInt32 EscherGraphicProvider::GetBlibID( SvStream& rPicOutStrm, const rtl::OString& rId,
+sal_uInt32 EscherGraphicProvider::GetBlibID( SvStream& rPicOutStrm, const OString& rId,
                                             const Rectangle& /* rBoundRect */, const com::sun::star::awt::Rectangle* pVisArea, const GraphicAttr* pGraphicAttr )
 {
     sal_uInt32          nBlibId = 0;
@@ -3756,9 +3738,10 @@
                 {   // to store a animation, a gif has to be included into the msOG chunk of a png  #I5583#
                     GraphicFilter &rFilter = GraphicFilter::GetGraphicFilter();
                     SvMemoryStream  aGIFStream;
-                    aGIFStream.Write(RTL_CONSTASCII_STRINGPARAM("MSOFFICE9.0"));
-                    nErrCode = rFilter.ExportGraphic( aGraphic, String(), aGIFStream,
-                        rFilter.GetExportFormatNumberForShortName( String( RTL_CONSTASCII_USTRINGPARAM( "GIF" ) ) ), NULL );
+                    const char* pString = "MSOFFICE9.0";
+                    aGIFStream.Write( pString, strlen(pString) );
+                    nErrCode = rFilter.ExportGraphic( aGraphic, OUString(), aGIFStream,
+                        rFilter.GetExportFormatNumberForShortName( OUString( "GIF" ) ), NULL );
                     com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue > aFilterData( 1 );
                     com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue > aAdditionalChunkSequence( 1 );
                     sal_uInt32 nGIFSreamLen = aGIFStream.Tell();
@@ -3767,14 +3750,14 @@
                     aGIFStream.Seek( STREAM_SEEK_TO_BEGIN );
                     aGIFStream.Read( pSeq, nGIFSreamLen );
                     com::sun::star::beans::PropertyValue aChunkProp, aFilterProp;
-                    aChunkProp.Name = String( RTL_CONSTASCII_USTRINGPARAM( "msOG" ) );
+                    aChunkProp.Name = OUString( "msOG" );
                     aChunkProp.Value <<= aGIFSeq;
                     aAdditionalChunkSequence[ 0 ] = aChunkProp;
-                    aFilterProp.Name = String( RTL_CONSTASCII_USTRINGPARAM( "AdditionalChunks" ) );
+                    aFilterProp.Name = OUString( "AdditionalChunks" );
                     aFilterProp.Value <<= aAdditionalChunkSequence;
                     aFilterData[ 0 ] = aFilterProp;
-                    nErrCode = rFilter.ExportGraphic( aGraphic, String(), aStream,
-                        rFilter.GetExportFormatNumberForShortName( String( RTL_CONSTASCII_USTRINGPARAM( "PNG" ) ) ), &aFilterData );
+                    nErrCode = rFilter.ExportGraphic( aGraphic, OUString(), aStream,
+                        rFilter.GetExportFormatNumberForShortName( OUString( "PNG" ) ), &aFilterData );
                 }
                 if ( nErrCode == ERRCODE_NONE )
                 {
@@ -3949,22 +3932,21 @@
     ::com::sun::star::uno::Reference< ::com::sun::star::drawing::XShape >
         aXShape( ( bFirst ) ? mXConnectToA : mXConnectToB );
 
-    rtl::OUString aString(aXShape->getShapeType());
-    rtl::OStringBuffer aBuf(rtl::OUStringToOString(aString, RTL_TEXTENCODING_UTF8));
+    OUString aString(aXShape->getShapeType());
+    OStringBuffer aBuf(OUStringToOString(aString, RTL_TEXTENCODING_UTF8));
     aBuf.remove( 0, 13 );   // removing "com.sun.star."
     sal_Int16 nPos = aBuf.toString().indexOf("Shape");
     aBuf.remove(nPos, 5);
-    rtl::OString aType = aBuf.makeStringAndClear();
+    OString aType = aBuf.makeStringAndClear();
 
     ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet >
         aPropertySet( aXShape, ::com::sun::star::uno::UNO_QUERY );
 
-    if (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.PolyPolygon")) || aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.PolyLine")))
+    if ((aType == OString( "drawing.PolyPolygon" )) || (aType == OString( "drawing.PolyLine"  )))
     {
         if ( aPropertySet.is() )
         {
-            if ( EscherPropertyValueHelper::GetPropertyValue( aAny,
-                    aPropertySet, String( RTL_CONSTASCII_USTRINGPARAM( "PolyPolygon" ) ) ) )
+            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aPropertySet, "PolyPolygon" ) )
             {
                 ::com::sun::star::drawing::PointSequenceSequence* pSourcePolyPolygon =
                     (::com::sun::star::drawing::PointSequenceSequence*)aAny.getValue();
@@ -3999,15 +3981,14 @@
             }
         }
     }
-    else if ( (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.OpenBezier"))) || (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.OpenFreeHand"))) || (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.PolyLinePath")))
-        || (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.ClosedBezier"))) || ( aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.ClosedFreeHand"))) || (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.PolyPolygonPath"))) )
+    else if ((aType == OString( "drawing.OpenBezier" )) || (aType == OString( "drawing.OpenFreeHand" )) || (aType == OString( "drawing.PolyLinePath" ))
+        || (aType == OString( "drawing.ClosedBezier" )) || ( aType == OString( "drawing.ClosedFreeHand" )) || (aType == OString( "drawing.PolyPolygonPath" )) )
     {
         ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet >
             aPropertySet2( aXShape, ::com::sun::star::uno::UNO_QUERY );
         if ( aPropertySet2.is() )
         {
-            if ( EscherPropertyValueHelper::GetPropertyValue( aAny,
-                    aPropertySet2, String( RTL_CONSTASCII_USTRINGPARAM( "PolyPolygonBezier" ) ) ) )
+            if ( EscherPropertyValueHelper::GetPropertyValue( aAny, aPropertySet2, "PolyPolygonBezier" ) )
             {
                 ::com::sun::star::drawing::PolyPolygonBezierCoords* pSourcePolyPolygon =
                     (::com::sun::star::drawing::PolyPolygonBezierCoords*)aAny.getValue();
@@ -4066,11 +4047,11 @@
                 SdrCustomShapeGeometryItem& rGeometryItem = (SdrCustomShapeGeometryItem&)(const SdrCustomShapeGeometryItem&)
                     pCustoShape->GetMergedItem( SDRATTR_CUSTOMSHAPE_GEOMETRY );
 
-                const rtl::OUString sPath( "Path"  );
-                const rtl::OUString sType( "Type"  );
-                const rtl::OUString sGluePointType( "GluePointType"  );
+                const OUString sPath( "Path"  );
+                const OUString sType( "Type"  );
+                const OUString sGluePointType( "GluePointType"  );
 
-                rtl::OUString sShapeType;
+                OUString sShapeType;
                 uno::Any* pType = rGeometryItem.GetPropertyValueByName( sType );
                 if ( pType )
                     *pType >>= sShapeType;
@@ -4152,15 +4133,13 @@
             aPoly[ 2 ] = Point( aCenter.X(), aRect.Bottom() );
             aPoly[ 3 ] = Point( aRect.Right(), aCenter.Y() );
 
-            sal_Int32 nAngle = ( EscherPropertyValueHelper::GetPropertyValue( aAny,
-                aPropertySet, String( RTL_CONSTASCII_USTRINGPARAM( "RotateAngle" ) ), sal_True ) )
-                    ? *((sal_Int32*)aAny.getValue() )
-                    : 0;
+            sal_Int32 nAngle = ( EscherPropertyValueHelper::GetPropertyValue( aAny, aPropertySet, OUString( "RotateAngle" ), sal_True ) )
+                    ? *((sal_Int32*)aAny.getValue() ) : 0;
             if ( nAngle )
                 aPoly.Rotate( aRect.TopLeft(), (sal_uInt16)( ( nAngle + 5 ) / 10 ) );
             nRule = GetClosestPoint( aPoly, aRefPoint );
 
-            if (aType.equalsL(RTL_CONSTASCII_STRINGPARAM("drawing.Ellipse")))
+            if (aType == OString( "drawing.Ellipse" ))
                 nRule <<= 1;    // In PPT an ellipse has 8 ways to connect
         }
     }
@@ -4721,7 +4700,7 @@
     return NULL;
 }
 
-sal_uInt32 EscherEx::EnterGroup( const String& rShapeName, const Rectangle* pBoundRect )
+sal_uInt32 EscherEx::EnterGroup( const OUString& rShapeName, const Rectangle* pBoundRect )
 {
     Rectangle aRect;
     if( pBoundRect )
@@ -4749,7 +4728,7 @@
         aPropOpt.AddOpt( ESCHER_Prop_dxWrapDistRight, 0 );
 
         // #i51348# shape name
-        if( rShapeName.Len() > 0 )
+        if( rShapeName.getLength() > 0 )
             aPropOpt.AddOpt( ESCHER_Prop_wzName, rShapeName );
 
         Commit( aPropOpt, aRect );
@@ -4771,7 +4750,7 @@
 
 sal_uInt32 EscherEx::EnterGroup( const Rectangle* pBoundRect )
 {
-    return EnterGroup( String::EmptyString(), pBoundRect );
+    return EnterGroup( OUString(), pBoundRect );
 }
 
 sal_Bool EscherEx::SetGroupSnapRect( sal_uInt32 nGroupLevel, const Rectangle& rRect )
diff --git a/filter/source/msfilter/msocximex.cxx b/filter/source/msfilter/msocximex.cxx
index f62cb70..6816b87 100644
--- a/filter/source/msfilter/msocximex.cxx
+++ b/filter/source/msfilter/msocximex.cxx
@@ -30,9 +30,9 @@
 using namespace ::com::sun::star;
 using namespace ::rtl;
 
-#define C2U(cChar)  rtl::OUString(cChar)
+#define C2U(cChar) OUString(cChar)
 
-static char sWW8_form[] = "WW-Standard";
+OUString sWW8_form( "WW-Standard" );
 
 SvxMSConvertOCXControls::SvxMSConvertOCXControls( const uno::Reference< frame::XModel >& rxModel) : mxModel(rxModel)
 {
@@ -106,13 +106,13 @@
 
             // Das Formular bekommt einen Namen wie "WW-Standard[n]" und
             // wird in jedem Fall neu angelegt.
-            UniString sName( sWW8_form, RTL_TEXTENCODING_MS_1252 );
+            OUString sName( sWW8_form );
             sal_uInt16 n = 0;
 
             while( xNameCont->hasByName( sName ) )
             {
-                sName.AssignAscii( sWW8_form );
-                sName += String::CreateFromInt32( ++n );
+                sName = sWW8_form;
+                sName += OUString::valueOf( static_cast<sal_Int32>(++n) );
             }
 
             const uno::Reference< lang::XMultiServiceFactory > &rServiceFactory
@@ -121,8 +121,8 @@
                 return xFormComps;
 
             uno::Reference< uno::XInterface >  xCreate =
-                rServiceFactory->createInstance(C2U(
-                    "com.sun.star.form.component.Form"));
+                rServiceFactory->createInstance( C2U(
+                    "com.sun.star.form.component.Form") );
             if( xCreate.is() )
             {
                 uno::Reference< beans::XPropertySet > xFormPropSet( xCreate,
diff --git a/filter/source/msfilter/msoleexp.cxx b/filter/source/msfilter/msoleexp.cxx
index 1235daa..b8864e2 100644
--- a/filter/source/msfilter/msoleexp.cxx
+++ b/filter/source/msfilter/msoleexp.cxx
@@ -64,21 +64,21 @@
     return SvGlobalName();
 }
 
-String GetStorageType( const SvGlobalName& aEmbName )
+OUString GetStorageType( const SvGlobalName& aEmbName )
 {
     if ( aEmbName == SvGlobalName( SO3_SM_OLE_EMBED_CLASSID_8 ) )
-        return rtl::OUString("LibreOffice.MathDocument.1");
+        return OUString( "LibreOffice.MathDocument.1" );
     else if ( aEmbName == SvGlobalName( SO3_SW_OLE_EMBED_CLASSID_8 ) )
-        return rtl::OUString("LibreOffice.WriterDocument.1");
+        return OUString( "LibreOffice.WriterDocument.1" );
     else if ( aEmbName == SvGlobalName( SO3_SC_OLE_EMBED_CLASSID_8 ) )
-        return rtl::OUString("LibreOffice.CalcDocument.1");
+        return OUString( "LibreOffice.CalcDocument.1" );
     else if ( aEmbName == SvGlobalName( SO3_SDRAW_OLE_EMBED_CLASSID_8 ) )
-        return rtl::OUString("LibreOffice.DrawDocument.1");
+        return OUString( "LibreOffice.DrawDocument.1" );
     else if ( aEmbName == SvGlobalName( SO3_SIMPRESS_OLE_EMBED_CLASSID_8 ) )
-        return rtl::OUString("LibreOffice.ImpressDocument.1");
+        return OUString( "LibreOffice.ImpressDocument.1" );
     else if ( aEmbName == SvGlobalName( SO3_SCH_OLE_EMBED_CLASSID_8 ) )
-        return rtl::OUString("LibreOffice.ChartDocument.1");
-    return rtl::OUString();
+        return OUString("LibreOffice.ChartDocument.1");
+    return OUString();
 }
 
 sal_Bool UseOldMSExport()
@@ -88,16 +88,15 @@
             comphelper::getProcessComponentContext()));
     try {
         uno::Sequence< uno::Any > aArg( 1 );
-        aArg[0] <<= rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "/org.openoffice.Office.Common/InternalMSExport") );
+        aArg[0] <<= OUString( "/org.openoffice.Office.Common/InternalMSExport" );
         uno::Reference< container::XNameAccess > xNameAccess(
             xProvider->createInstanceWithArguments(
-                rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "com.sun.star.configuration.ConfigurationUpdateAccess" ) ),
+                "com.sun.star.configuration.ConfigurationUpdateAccess",
                 aArg ),
             uno::UNO_QUERY );
         if ( xNameAccess.is() )
         {
-            uno::Any aResult = xNameAccess->getByName(
-                rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "UseOldExport" ) ) );
+            uno::Any aResult = xNameAccess->getByName( "UseOldExport" );
 
             sal_Bool bResult = sal_Bool();
             if ( aResult >>= bResult )
@@ -174,7 +173,7 @@
                     // flags for checking if conversion is wanted at all (SaveOptions?!)
                     if( GetFlags() & pArr->nFlag )
                     {
-                        pExpFilter = SfxFilterMatcher().GetFilter4FilterName(rtl::OUString::createFromAscii(pArr->pFilterNm));
+                        pExpFilter = SfxFilterMatcher().GetFilter4FilterName(OUString::createFromAscii(pArr->pFilterNm));
                         break;
                     }
                 }
@@ -192,15 +191,15 @@
             //TODO/LATER: a "StoreTo" method at embedded object would be nice
             uno::Sequence < beans::PropertyValue > aSeq(2);
             SvStream* pStream = new SvMemoryStream;
-            aSeq[0].Name = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "OutputStream" ));
+            aSeq[0].Name = OUString( "OutputStream" );
             ::uno::Reference < io::XOutputStream > xOut = new ::utl::OOutputStreamWrapper( *pStream );
             aSeq[0].Value <<= xOut;
-            aSeq[1].Name = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "FilterName" ) );
-            aSeq[1].Value <<= ::rtl::OUString( pExpFilter->GetName() );
+            aSeq[1].Name = OUString( "FilterName" );
+            aSeq[1].Value <<= OUString( pExpFilter->GetName() );
             uno::Reference < frame::XStorable > xStor( rObj->getComponent(), uno::UNO_QUERY );
         try
         {
-            xStor->storeToURL( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "private:stream" )), aSeq );
+            xStor->storeToURL( "private:stream", aSeq );
         }
         catch( const uno::Exception& ) {} // #TODO really handle exceptions - interactionalhandler etc. ?
 
@@ -226,7 +225,7 @@
                                 SOT_FORMATSTR_ID_EMBEDDED_OBJ_OLE,
                                 GetStorageType( aEmbName ) );
             SotStorageStreamRef xExtStm = rDestStg.OpenSotStream(
-                                            rtl::OUString("properties_stream"),
+                                            OUString( "properties_stream" ),
                                             STREAM_STD_READWRITE);
 
             sal_Bool bExtentSuccess = sal_False;
@@ -281,7 +280,7 @@
             if ( bExtentSuccess )
             {
                 SotStorageStreamRef xEmbStm = rDestStg.OpenSotStream(
-                                                rtl::OUString("package_stream"),
+                                                OUString( "package_stream" ),
                                                 STREAM_STD_READWRITE);
                 if( !xEmbStm->GetError() )
                 {
@@ -292,11 +291,11 @@
                         //TODO/LATER: is stream instead of outputstream a better choice?!
                         //TODO/LATER: a "StoreTo" method at embedded object would be nice
                         uno::Sequence < beans::PropertyValue > aSeq(1);
-                        aSeq[0].Name = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "OutputStream" ));
+                        aSeq[0].Name = OUString( "OutputStream" );
                         ::uno::Reference < io::XOutputStream > xOut = new ::utl::OOutputStreamWrapper( *xEmbStm );
                         aSeq[0].Value <<= xOut;
                         uno::Reference < frame::XStorable > xStor( rObj->getComponent(), uno::UNO_QUERY );
-                        xStor->storeToURL( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "private:stream" )), aSeq );
+                        xStor->storeToURL( "private:stream", aSeq );
                     }
                     catch( const uno::Exception& )
                     {
@@ -321,7 +320,7 @@
         if ( xPers.is() )
         {
             uno::Sequence < beans::PropertyValue > aEmptySeq;
-            ::rtl::OUString aTempName( RTL_CONSTASCII_USTRINGPARAM( "bla" ));
+            OUString aTempName( "bla" );
             try
             {
                 xPers->storeToEntry( xStor, aTempName, aEmptySeq, aEmptySeq );
@@ -336,7 +335,7 @@
     }
 
     //We never need this stream: See #99809# and #i2179#
-    rDestStg.Remove(rtl::OUString(SVEXT_PERSIST_STREAM));
+    rDestStg.Remove( OUString(SVEXT_PERSIST_STREAM) );
 }
 
 

-- 
To view, visit https://gerrit.libreoffice.org/1507
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2b8bd3535a5ba5323b9a17941c4345f4c1e0e822
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania <ricardo at linuxafundo.com.br>



More information about the LibreOffice mailing list