[Libreoffice-commits] core.git: l10ntools/source oox/source sdext/source tools/source vcl/unx

Noel Grandin noel at peralex.com
Fri Dec 5 01:35:52 PST 2014


 l10ntools/source/localize.cxx                  |    2 
 l10ntools/source/merge.cxx                     |    2 
 l10ntools/source/po.cxx                        |    4 
 oox/source/drawingml/customshapepresetdata.cxx |  106 ++++++++++++-------------
 sdext/source/pdfimport/wrapper/wrapper.cxx     |   16 +--
 tools/source/inet/inetmsg.cxx                  |   96 +++++++---------------
 vcl/unx/gtk/fpicker/SalGtkFilePicker.hxx       |    2 
 7 files changed, 98 insertions(+), 130 deletions(-)

New commits:
commit 0375504f7be34d857859dfbaa312501e0eaaaad1
Author: Noel Grandin <noel at peralex.com>
Date:   Thu Dec 4 14:19:41 2014 +0200

    fdo#38835 strip out OString globals
    
    Change-Id: Id2eb27132fd6c1734e50c02617ce1bbb75e294a0

diff --git a/l10ntools/source/localize.cxx b/l10ntools/source/localize.cxx
index 5546e9e..5c107a6 100644
--- a/l10ntools/source/localize.cxx
+++ b/l10ntools/source/localize.cxx
@@ -261,7 +261,7 @@ void handleFilesOfDir(
 }
 
 bool includeProject(const OString& rProject) {
-    static const OString projects[] = {
+    static const char *projects[] = {
         "accessibility",
         "avmedia",
         "basctl",
diff --git a/l10ntools/source/merge.cxx b/l10ntools/source/merge.cxx
index ee98009..a8323be 100644
--- a/l10ntools/source/merge.cxx
+++ b/l10ntools/source/merge.cxx
@@ -439,7 +439,7 @@ void MergeDataFile::InsertEntry(
 OString MergeDataFile::CreateKey(const OString& rTYP, const OString& rGID,
     const OString& rLID, const OString& rFilename, bool bCaseSensitive)
 {
-    static const OString sStroke('-');
+    static const char sStroke[] = "-";
     OString sKey( rTYP );
     sKey += sStroke;
     sKey += rGID;
diff --git a/l10ntools/source/po.cxx b/l10ntools/source/po.cxx
index bc599a0..a4e60b9 100644
--- a/l10ntools/source/po.cxx
+++ b/l10ntools/source/po.cxx
@@ -396,12 +396,12 @@ OString PoEntry::genKeyId(const OString& rGenerator)
     aCRC32.process_bytes(rGenerator.getStr(), rGenerator.getLength());
     sal_uInt32 nCRC = aCRC32.checksum();
     // Use simple ASCII characters, exclude I, l, 1 and O, 0 to avoid confusing IDs
-    static const OString sSymbols =
+    static const char sSymbols[] =
         "ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz23456789";
     char sKeyId[6];
     for( short nKeyInd = 0; nKeyInd < 5; ++nKeyInd )
     {
-        sKeyId[nKeyInd] = sSymbols[(nCRC & 63) % sSymbols.getLength()];
+        sKeyId[nKeyInd] = sSymbols[(nCRC & 63) % strlen(sSymbols)];
         nCRC >>= 6;
     }
     sKeyId[5] = '\0';
diff --git a/oox/source/drawingml/customshapepresetdata.cxx b/oox/source/drawingml/customshapepresetdata.cxx
index 2edc40c..945825e 100644
--- a/oox/source/drawingml/customshapepresetdata.cxx
+++ b/oox/source/drawingml/customshapepresetdata.cxx
@@ -29,16 +29,16 @@ void lcl_parseAdjustmentValue(comphelper::SequenceAsVector<drawing::EnhancedCust
     do
     {
         OString aToken = rValue.getToken(0, ',', nIndex).trim();
-        static const OString aNamePrefix("Name = \"");
-        static const OString aValuePrefix("Value = (any) { (long) ");
+        static const char aNamePrefix[] = "Name = \"";
+        static const char aValuePrefix[] = "Value = (any) { (long) ";
         if (aToken.startsWith(aNamePrefix))
         {
-            OString aName = aToken.copy(aNamePrefix.getLength(), aToken.getLength() - aNamePrefix.getLength() - strlen("\""));
+            OString aName = aToken.copy(strlen(aNamePrefix), aToken.getLength() - strlen(aNamePrefix) - strlen("\""));
             aAdjustmentValue.Name = OUString::fromUtf8(aName);
         }
         else if (aToken.startsWith(aValuePrefix))
         {
-            OString aValue = aToken.copy(aValuePrefix.getLength(), aToken.getLength() - aValuePrefix.getLength() - strlen(" }"));
+            OString aValue = aToken.copy(strlen(aValuePrefix), aToken.getLength() - strlen(aValuePrefix) - strlen(" }"));
             aAdjustmentValue.Value = uno::makeAny(aValue.toInt32());
         }
         else if (!aToken.startsWith("State = "))
@@ -77,27 +77,27 @@ drawing::EnhancedCustomShapeParameterPair lcl_parseEnhancedCustomShapeParameterP
     drawing::EnhancedCustomShapeParameterPair aPair;
     OString aToken = rValue;
     // We expect the followings here: First.Value, First.Type, Second.Value, Second.Type
-    static const OString aExpectedFVPrefix = "First = (com.sun.star.drawing.EnhancedCustomShapeParameter) { Value = (any) { (long) ";
+    static const char aExpectedFVPrefix[] = "First = (com.sun.star.drawing.EnhancedCustomShapeParameter) { Value = (any) { (long) ";
     assert(aToken.startsWith(aExpectedFVPrefix));
-    sal_Int32 nIndex = aExpectedFVPrefix.getLength();
+    sal_Int32 nIndex = strlen(aExpectedFVPrefix);
     aPair.First.Value = uno::makeAny(static_cast<sal_uInt32>(aToken.getToken(0, '}', nIndex).toInt32()));
 
-    static const OString aExpectedFTPrefix = ", Type = (short) ";
+    static const char aExpectedFTPrefix[] = ", Type = (short) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedFTPrefix));
-    nIndex = aExpectedFTPrefix.getLength();
+    nIndex = strlen(aExpectedFTPrefix);
     aPair.First.Type = static_cast<sal_uInt16>(aToken.getToken(0, '}', nIndex).toInt32());
 
-    static const OString aExpectedSVPrefix = ", Second = (com.sun.star.drawing.EnhancedCustomShapeParameter) { Value = (any) { (long) ";
+    static const char aExpectedSVPrefix[] = ", Second = (com.sun.star.drawing.EnhancedCustomShapeParameter) { Value = (any) { (long) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedSVPrefix));
-    nIndex = aExpectedSVPrefix.getLength();
+    nIndex = strlen(aExpectedSVPrefix);
     aPair.Second.Value = uno::makeAny(static_cast<sal_uInt32>(aToken.getToken(0, '}', nIndex).toInt32()));
 
-    static const OString aExpectedSTPrefix = ", Type = (short) ";
+    static const char aExpectedSTPrefix[] = ", Type = (short) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedSTPrefix));
-    nIndex = aExpectedSTPrefix.getLength();
+    nIndex = strlen(aExpectedSTPrefix);
     aPair.Second.Type = static_cast<sal_uInt16>(aToken.getToken(0, '}', nIndex).toInt32());
     return aPair;
 }
@@ -107,15 +107,15 @@ drawing::EnhancedCustomShapeSegment lcl_parseEnhancedCustomShapeSegment(const OS
     drawing::EnhancedCustomShapeSegment aSegment;
     OString aToken = rValue;
     // We expect the followings here: Command, Count
-    static const OString aExpectedCommandPrefix = "Command = (short) ";
+    static const char aExpectedCommandPrefix[] = "Command = (short) ";
     assert(aToken.startsWith(aExpectedCommandPrefix));
-    sal_Int32 nIndex = aExpectedCommandPrefix.getLength();
+    sal_Int32 nIndex = strlen(aExpectedCommandPrefix);
     aSegment.Command = static_cast<sal_Int16>(aToken.getToken(0, ',', nIndex).toInt32());
 
-    static const OString aExpectedCountPrefix = " Count = (short) ";
+    static const char aExpectedCountPrefix[] = " Count = (short) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedCountPrefix));
-    nIndex = aExpectedCountPrefix.getLength();
+    nIndex = strlen(aExpectedCountPrefix);
     aSegment.Count = static_cast<sal_Int16>(aToken.getToken(0, '}', nIndex).toInt32());
     return aSegment;
 }
@@ -125,27 +125,27 @@ awt::Rectangle lcl_parseRectangle(const OString& rValue)
     awt::Rectangle aRectangle;
     OString aToken = rValue;
     // We expect the followings here: X, Y, Width, Height
-    static const OString aExpectedXPrefix = "X = (long) ";
+    static const char aExpectedXPrefix[] = "X = (long) ";
     assert(aToken.startsWith(aExpectedXPrefix));
-    sal_Int32 nIndex = aExpectedXPrefix.getLength();
+    sal_Int32 nIndex = strlen(aExpectedXPrefix);
     aRectangle.X = static_cast<sal_Int32>(aToken.getToken(0, ',', nIndex).toInt32());
 
-    static const OString aExpectedYPrefix = " Y = (long) ";
+    static const char aExpectedYPrefix[] = " Y = (long) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedYPrefix));
-    nIndex = aExpectedYPrefix.getLength();
+    nIndex = strlen(aExpectedYPrefix);
     aRectangle.Y = static_cast<sal_Int32>(aToken.getToken(0, ',', nIndex).toInt32());
 
-    static const OString aExpectedWidthPrefix = " Width = (long) ";
+    static const char aExpectedWidthPrefix[] = " Width = (long) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedWidthPrefix));
-    nIndex = aExpectedWidthPrefix.getLength();
+    nIndex = strlen(aExpectedWidthPrefix);
     aRectangle.Width = static_cast<sal_Int32>(aToken.getToken(0, ',', nIndex).toInt32());
 
-    static const OString aExpectedHeightPrefix = " Height = (long) ";
+    static const char aExpectedHeightPrefix[] = " Height = (long) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedHeightPrefix));
-    nIndex = aExpectedHeightPrefix.getLength();
+    nIndex = strlen(aExpectedHeightPrefix);
     aRectangle.Width = static_cast<sal_Int32>(aToken.copy(nIndex).toInt32());
 
     return aRectangle;
@@ -156,15 +156,15 @@ awt::Size lcl_parseSize(const OString& rValue)
     awt::Size aSize;
     OString aToken = rValue;
     // We expect the followings here: Width, Height
-    static const OString aExpectedWidthPrefix = "Width = (long) ";
+    static const char aExpectedWidthPrefix[] = "Width = (long) ";
     assert(aToken.startsWith(aExpectedWidthPrefix));
-    sal_Int32 nIndex = aExpectedWidthPrefix.getLength();
+    sal_Int32 nIndex = strlen(aExpectedWidthPrefix);
     aSize.Width = static_cast<sal_Int32>(aToken.getToken(0, ',', nIndex).toInt32());
 
-    static const OString aExpectedHeightPrefix = " Height = (long) ";
+    static const char aExpectedHeightPrefix[] = " Height = (long) ";
     aToken = aToken.copy(nIndex);
     assert(aToken.startsWith(aExpectedHeightPrefix));
-    nIndex = aExpectedHeightPrefix.getLength();
+    nIndex = strlen(aExpectedHeightPrefix);
     aSize.Height = static_cast<sal_Int32>(aToken.copy(nIndex).toInt32());
 
     return aSize;
@@ -193,10 +193,10 @@ drawing::EnhancedCustomShapeTextFrame lcl_parseEnhancedCustomShapeTextFrame(cons
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("TopLeft = (com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ");
+            static const char aExpectedPrefix[] = "TopLeft = (com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" }"));
                 aTextFrame.TopLeft = lcl_parseEnhancedCustomShapeParameterPair(aToken);
             }
             else
@@ -206,10 +206,10 @@ drawing::EnhancedCustomShapeTextFrame lcl_parseEnhancedCustomShapeTextFrame(cons
     }
 
     OString aToken = rValue.copy(nStart, rValue.getLength() - nStart);
-    static const OString aExpectedPrefix("BottomRight = (com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ");
+    static const char aExpectedPrefix[] = "BottomRight = (com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ";
     if (aToken.startsWith(aExpectedPrefix))
     {
-        aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" }"));
+        aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" }"));
         aTextFrame.BottomRight = lcl_parseEnhancedCustomShapeParameterPair(aToken);
     }
     else
@@ -242,10 +242,10 @@ void lcl_parseHandlePosition(comphelper::SequenceAsVector<beans::PropertyValue>&
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("Value = (any) { (com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ");
+            static const char aExpectedPrefix[] = "Value = (any) { (com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" } }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" } }"));
 
                 beans::PropertyValue aPropertyValue;
                 aPropertyValue.Name = "Position";
@@ -283,21 +283,21 @@ void lcl_parseHandleRange(comphelper::SequenceAsVector<beans::PropertyValue>& rH
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("Value = (any) { (com.sun.star.drawing.EnhancedCustomShapeParameter) { ");
+            static const char aExpectedPrefix[] = "Value = (any) { (com.sun.star.drawing.EnhancedCustomShapeParameter) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
                 drawing::EnhancedCustomShapeParameter aParameter;
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" } }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" } }"));
                 // We expect the followings here: Value and Type
-                static const OString aExpectedVPrefix = "Value = (any) { (long) ";
+                static const char aExpectedVPrefix[] = "Value = (any) { (long) ";
                 assert(aToken.startsWith(aExpectedVPrefix));
-                sal_Int32 nIndex = aExpectedVPrefix.getLength();
+                sal_Int32 nIndex = strlen(aExpectedVPrefix);
                 aParameter.Value = uno::makeAny(aToken.getToken(0, '}', nIndex).toInt32());
 
-                static const OString aExpectedTPrefix = ", Type = (short) ";
+                static const char aExpectedTPrefix[] = ", Type = (short) ";
                 aToken = aToken.copy(nIndex);
                 assert(aToken.startsWith(aExpectedTPrefix));
-                nIndex = aExpectedTPrefix.getLength();
+                nIndex = strlen(aExpectedTPrefix);
                 aParameter.Type = static_cast<sal_Int16>(aToken.getToken(0, '}', nIndex).toInt32());
 
                 beans::PropertyValue aPropertyValue;
@@ -316,11 +316,11 @@ void lcl_parseHandleRange(comphelper::SequenceAsVector<beans::PropertyValue>& rH
 // Parses a string like: Name = "RefY", Handle = (long) 0, Value = (any) { (long) 0 }, State = (com.sun.star.beans.PropertyState) DIRECT_VALUE
 void lcl_parseHandleRef(comphelper::SequenceAsVector<beans::PropertyValue>& rHandle, const OString& rValue, const OUString& rName)
 {
-    static const OString aExpectedXPrefix("Name = \"RefX\", Handle = (long) 0, Value = (any) { (long) ");
-    static const OString aExpectedYPrefix("Name = \"RefY\", Handle = (long) 0, Value = (any) { (long) ");
+    static const char aExpectedXPrefix[] = "Name = \"RefX\", Handle = (long) 0, Value = (any) { (long) ";
+    static const char aExpectedYPrefix[] = "Name = \"RefY\", Handle = (long) 0, Value = (any) { (long) ";
     if (rValue.startsWith(aExpectedXPrefix) || rValue.startsWith(aExpectedYPrefix))
     {
-        sal_Int32 nIndex = aExpectedXPrefix.getLength();
+        sal_Int32 nIndex = strlen(aExpectedXPrefix);
         beans::PropertyValue aPropertyValue;
         aPropertyValue.Name = rName;
         // We only expect a Value here
@@ -470,10 +470,10 @@ void lcl_parsePathCoordinates(comphelper::SequenceAsVector<beans::PropertyValue>
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("Value = (any) { ([]com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ");
+            static const char aExpectedPrefix[] = "Value = (any) { ([]com.sun.star.drawing.EnhancedCustomShapeParameterPair) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" } }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" } }"));
                 lcl_parsePathCoordinateValues(rPath, aToken);
             }
             else if (!aToken.startsWith("Name =") && !aToken.startsWith("Handle ="))
@@ -534,10 +534,10 @@ void lcl_parsePathSegments(comphelper::SequenceAsVector<beans::PropertyValue>& r
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("Value = (any) { ([]com.sun.star.drawing.EnhancedCustomShapeSegment) { ");
+            static const char aExpectedPrefix[] = "Value = (any) { ([]com.sun.star.drawing.EnhancedCustomShapeSegment) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" } }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" } }"));
                 lcl_parsePathSegmentValues(rPath, aToken);
             }
             else if (!aToken.startsWith("Name =") && !aToken.startsWith("Handle ="))
@@ -598,10 +598,10 @@ void lcl_parsePathTextFrames(comphelper::SequenceAsVector<beans::PropertyValue>&
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("Value = (any) { ([]com.sun.star.drawing.EnhancedCustomShapeTextFrame) { ");
+            static const char aExpectedPrefix[] = "Value = (any) { ([]com.sun.star.drawing.EnhancedCustomShapeTextFrame) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" } }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" } }"));
                 lcl_parsePathTextFrameValues(rPath, aToken);
             }
             else if (!aToken.startsWith("Name =") && !aToken.startsWith("Handle ="))
@@ -660,10 +660,10 @@ void lcl_parsePathSubViewSize(comphelper::SequenceAsVector<beans::PropertyValue>
         else if (rValue[i] == ',' && !bIgnore)
         {
             OString aToken = rValue.copy(nStart, i - nStart);
-            static const OString aExpectedPrefix("Value = (any) { ([]com.sun.star.awt.Size) { ");
+            static const char aExpectedPrefix[] = "Value = (any) { ([]com.sun.star.awt.Size) { ";
             if (aToken.startsWith(aExpectedPrefix))
             {
-                aToken = aToken.copy(aExpectedPrefix.getLength(), aToken.getLength() - aExpectedPrefix.getLength() - strlen(" } }"));
+                aToken = aToken.copy(strlen(aExpectedPrefix), aToken.getLength() - strlen(aExpectedPrefix) - strlen(" } }"));
                 lcl_parsePathSubViewSizeValues(rPath, aToken);
             }
             else if (!aToken.startsWith("Name =") && !aToken.startsWith("Handle ="))
@@ -727,14 +727,14 @@ void CustomShapeProperties::initializePresetDataMap()
     bool bFirst = true;
     while (bNotDone)
     {
-        static const OString aCommentPrefix("/* ");
+        static const char aCommentPrefix[] = "/* ";
         if (aLine.startsWith(aCommentPrefix))
         {
             if (bFirst)
                 bFirst = false;
             else
                 maPresetDataMap[StaticTokenMap::get().getTokenFromUnicode(aName)] = aPropertyMap;
-            aName = OUString::fromUtf8(aLine.copy(aCommentPrefix.getLength(), aLine.getLength() - aCommentPrefix.getLength() - strlen(" */")));
+            aName = OUString::fromUtf8(aLine.copy(strlen(aCommentPrefix), aLine.getLength() - strlen(aCommentPrefix) - strlen(" */")));
         }
         else
         {
diff --git a/sdext/source/pdfimport/wrapper/wrapper.cxx b/sdext/source/pdfimport/wrapper/wrapper.cxx
index 79d64fd..c393724 100644
--- a/sdext/source/pdfimport/wrapper/wrapper.cxx
+++ b/sdext/source/pdfimport/wrapper/wrapper.cxx
@@ -677,14 +677,14 @@ void Parser::readFont()
 
 uno::Sequence<beans::PropertyValue> Parser::readImageImpl()
 {
-    static const OString aJpegMarker( "JPEG" );
-    static const OString aPbmMarker( "PBM" );
-    static const OString aPpmMarker( "PPM" );
-    static const OString aPngMarker( "PNG" );
-    static const char aJpegFile[] = "DUMMY.JPEG";
-    static const char aPbmFile[] = "DUMMY.PBM";
-    static const char aPpmFile[] = "DUMMY.PPM";
-    static const char aPngFile[] = "DUMMY.PNG";
+    static const char aJpegMarker[] = "JPEG";
+    static const char aPbmMarker[]  = "PBM";
+    static const char aPpmMarker[]  = "PPM";
+    static const char aPngMarker[]  = "PNG";
+    static const char aJpegFile[]   = "DUMMY.JPEG";
+    static const char aPbmFile[]    = "DUMMY.PBM";
+    static const char aPpmFile[]    = "DUMMY.PPM";
+    static const char aPngFile[]    = "DUMMY.PNG";
 
     OString aToken = readNextToken();
     const sal_Int32 nImageSize( readInt32() );
diff --git a/tools/source/inet/inetmsg.cxx b/tools/source/inet/inetmsg.cxx
index cc3989a..42c7a5f 100644
--- a/tools/source/inet/inetmsg.cxx
+++ b/tools/source/inet/inetmsg.cxx
@@ -143,40 +143,25 @@ SvStream& INetMessage::operator>> (SvStream& rStrm)
     return rStrm;
 }
 
-namespace
-{
-    struct ImplINetRFC822MessageHeaderDataImpl
-    {
-        const OString* operator()()
-        {
-            static const OString _ImplINetRFC822MessageHeaderData[] =
-            {
-                OString("BCC"),
-                OString("CC"),
-                OString("Comments"),
-                OString("Date"),
-                OString("From"),
-                OString("In-Reply-To"),
-                OString("Keywords"),
-                OString("Message-ID"),
-                OString("References"),
-                OString("Reply-To"),
-                OString("Return-Path"),
-                OString("Subject"),
-                OString("Sender"),
-                OString("To"),
-                OString("X-Mailer"),
-                OString("Return-Receipt-To")
-            };
-            return &_ImplINetRFC822MessageHeaderData[0];
-        }
-    };
-
-    struct ImplINetRFC822MessageHeaderData
-        : public rtl::StaticAggregate< const OString, ImplINetRFC822MessageHeaderDataImpl > {};
-}
-
-#define HDR(n) ImplINetRFC822MessageHeaderData::get()[(n)]
+static const char * ImplINetRFC822MessageHeaderData[] =
+{
+    "BCC",
+    "CC",
+    "Comments",
+    "Date",
+    "From",
+    "In-Reply-To",
+    "Keywords",
+    "Message-ID",
+    "References",
+    "Reply-To",
+    "Return-Path",
+    "Subject",
+    "Sender",
+    "To",
+    "X-Mailer",
+    "Return-Receipt-To"
+};
 
 enum _ImplINetRFC822MessageHeaderState
 {
@@ -592,7 +577,7 @@ sal_uIntPtr INetRFC822Message::SetHeaderField (
             case INETMSG_RFC822_OK:
                 pData = pStop;
                 SetHeaderField_Impl (
-                    INetMessageHeader (HDR(nIdx), rHeader.GetValue()),
+                    INetMessageHeader( ImplINetRFC822MessageHeaderData[nIdx], rHeader.GetValue() ),
                     m_nIndex[nIdx]);
                 nNewIndex = m_nIndex[nIdx];
                 break;
@@ -630,30 +615,15 @@ SvStream& INetRFC822Message::operator>> (SvStream& rStrm)
     return rStrm;
 }
 
-namespace
+static const char* ImplINetMIMEMessageHeaderData[] =
 {
-    struct ImplINetMIMEMessageHeaderDataImpl
-    {
-        const OString* operator()()
-        {
-            static const OString _ImplINetMIMEMessageHeaderData[] =
-            {
-                OString("MIME-Version"),
-                OString("Content-Description"),
-                OString("Content-Disposition"),
-                OString("Content-ID"),
-                OString("Content-Type"),
-                OString("Content-Transfer-Encoding")
-            };
-            return &_ImplINetMIMEMessageHeaderData[0];
-        }
-    };
-
-    struct ImplINetMIMEMessageHeaderData
-        : public rtl::StaticAggregate< const OString, ImplINetMIMEMessageHeaderDataImpl > {};
-}
-
-#define MIMEHDR(n) ImplINetMIMEMessageHeaderData::get()[(n)]
+    "MIME-Version",
+    "Content-Description",
+    "Content-Disposition",
+    "Content-ID",
+    "Content-Type",
+    "Content-Transfer-Encoding"
+};
 
 enum _ImplINetMIMEMessageHeaderState
 {
@@ -871,7 +841,7 @@ sal_uIntPtr INetMIMEMessage::SetHeaderField (
             case INETMSG_MIME_OK:
                 pData = pStop;
                 SetHeaderField_Impl (
-                    INetMessageHeader (MIMEHDR(nIdx), rHeader.GetValue()),
+                    INetMessageHeader( ImplINetMIMEMessageHeaderData[nIdx], rHeader.GetValue()),
                     m_nIndex[nIdx]);
                 nNewIndex = m_nIndex[nIdx];
                 break;
@@ -890,7 +860,7 @@ void INetMIMEMessage::SetMIMEVersion (const OUString& rVersion)
 {
     SetHeaderField_Impl (
         INetMIME::HEADER_FIELD_TEXT,
-        MIMEHDR(INETMSG_MIME_VERSION), rVersion,
+        ImplINetMIMEMessageHeaderData[INETMSG_MIME_VERSION], rVersion,
         m_nIndex[INETMSG_MIME_VERSION]);
 }
 
@@ -898,7 +868,7 @@ void INetMIMEMessage::SetContentDisposition (const OUString& rDisposition)
 {
     SetHeaderField_Impl (
         INetMIME::HEADER_FIELD_TEXT,
-        MIMEHDR(INETMSG_MIME_CONTENT_DISPOSITION), rDisposition,
+        ImplINetMIMEMessageHeaderData[INETMSG_MIME_CONTENT_DISPOSITION], rDisposition,
         m_nIndex[INETMSG_MIME_CONTENT_DISPOSITION]);
 }
 
@@ -906,7 +876,7 @@ void INetMIMEMessage::SetContentType (const OUString& rType)
 {
     SetHeaderField_Impl (
         INetMIME::HEADER_FIELD_TEXT,
-        MIMEHDR(INETMSG_MIME_CONTENT_TYPE), rType,
+        ImplINetMIMEMessageHeaderData[INETMSG_MIME_CONTENT_TYPE], rType,
         m_nIndex[INETMSG_MIME_CONTENT_TYPE]);
 }
 
@@ -915,7 +885,7 @@ void INetMIMEMessage::SetContentTransferEncoding (
 {
     SetHeaderField_Impl (
         INetMIME::HEADER_FIELD_TEXT,
-        MIMEHDR(INETMSG_MIME_CONTENT_TRANSFER_ENCODING), rEncoding,
+        ImplINetMIMEMessageHeaderData[INETMSG_MIME_CONTENT_TRANSFER_ENCODING], rEncoding,
         m_nIndex[INETMSG_MIME_CONTENT_TRANSFER_ENCODING]);
 }
 
diff --git a/vcl/unx/gtk/fpicker/SalGtkFilePicker.hxx b/vcl/unx/gtk/fpicker/SalGtkFilePicker.hxx
index b4f4dbd..586dff1 100644
--- a/vcl/unx/gtk/fpicker/SalGtkFilePicker.hxx
+++ b/vcl/unx/gtk/fpicker/SalGtkFilePicker.hxx
@@ -213,8 +213,6 @@ class SalGtkFilePicker : public SalGtkPicker, public SalGtkFilePicker_Base
         bool mbToggleVisibility[TOGGLE_LAST];
         bool mbToggleChecked[TOGGLE_LAST];
 
-        static const OString m_ToggleLabels[TOGGLE_LAST];
-
         enum {
             PLAY,
             BUTTON_LAST };


More information about the Libreoffice-commits mailing list