Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2627

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/27/2627/1

Remove now obsolete RTL_CONSTASCII_(U)STRINGPARAM

Change-Id: I15ae64d621fd0a1fa178f9a4062b93520628bdb0
---
M vcl/unx/generic/app/i18n_im.cxx
M vcl/unx/generic/app/saldata.cxx
M vcl/unx/generic/app/salinst.cxx
M vcl/unx/generic/app/sm.cxx
4 files changed, 20 insertions(+), 20 deletions(-)



diff --git a/vcl/unx/generic/app/i18n_im.cxx b/vcl/unx/generic/app/i18n_im.cxx
index 9da48e8..430aea2 100644
--- a/vcl/unx/generic/app/i18n_im.cxx
+++ b/vcl/unx/generic/app/i18n_im.cxx
@@ -169,10 +169,10 @@
 static void
 SetSystemEnvironment( const rtl::OUString& rLocale )
 {
-    rtl::OUString LC_ALL_Var(RTL_CONSTASCII_USTRINGPARAM("LC_ALL"));
+    OUString LC_ALL_Var("LC_ALL");
     osl_setEnvironment(LC_ALL_Var.pData, rLocale.pData);
 
-    rtl::OUString LANG_Var(RTL_CONSTASCII_USTRINGPARAM("LANG"));
+    OUString LANG_Var("LANG");
     osl_setEnvironment(LANG_Var.pData, rLocale.pData);
 }
 #endif
@@ -229,13 +229,13 @@
             osl_setThreadTextEncoding (RTL_TEXTENCODING_ISO_8859_1);
             locale = SetSystemLocale( "en_US" );
             #ifdef SOLARIS
-            SetSystemEnvironment( rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("en_US")) );
+            SetSystemEnvironment( "en_US" );
             #endif
             if (! IsXWindowCompatibleLocale(locale))
             {
                 locale = SetSystemLocale( "C" );
                 #ifdef SOLARIS
-                SetSystemEnvironment( rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("C")) );
+                SetSystemEnvironment( "C" );
                 #endif
                 if (! IsXWindowCompatibleLocale(locale))
                     mbUseable = False;
@@ -366,7 +366,7 @@
 
         if ((maMethod == (XIM)NULL) && (getenv("XMODIFIERS") != NULL))
         {
-                rtl::OUString envVar(RTL_CONSTASCII_USTRINGPARAM("XMODIFIERS"));
+                OUString envVar("XMODIFIERS");
                 osl_clearEnvironment(envVar.pData);
                 XSetLocaleModifiers("");
                 maMethod = XOpenIM(pDisplay, NULL, NULL, NULL);
diff --git a/vcl/unx/generic/app/saldata.cxx b/vcl/unx/generic/app/saldata.cxx
index ed1f036..b7c7ee9 100644
--- a/vcl/unx/generic/app/saldata.cxx
+++ b/vcl/unx/generic/app/saldata.cxx
@@ -435,7 +435,7 @@
                  * the clipboard build another connection
                  * to the xserver using $DISPLAY
                  */
-                rtl::OUString envVar(RTL_CONSTASCII_USTRINGPARAM("DISPLAY"));
+                OUString envVar("DISPLAY");
                 osl_setEnvironment(envVar.pData, aParam.pData);
             }
             break;
diff --git a/vcl/unx/generic/app/salinst.cxx b/vcl/unx/generic/app/salinst.cxx
index ed8e216..b234b8b 100644
--- a/vcl/unx/generic/app/salinst.cxx
+++ b/vcl/unx/generic/app/salinst.cxx
@@ -192,7 +192,7 @@
     delete pFrame;
 }
 
-static void getServerDirectories( std::list< rtl::OString >& o_rFontPaths )
+static void getServerDirectories( std::list< OString >& o_rFontPaths )
 {
 #ifdef LINUX
     /*
@@ -201,7 +201,7 @@
     static const char* pCommands[] = {
         "/usr/sbin/chkfontpath 2>/dev/null", "chkfontpath 2>/dev/null"
     };
-    ::std::list< rtl::OString > aLines;
+    ::std::list< OString > aLines;
 
     for( unsigned int i = 0; i < SAL_N_ELEMENTS(pCommands); i++ )
     {
@@ -225,7 +225,7 @@
         }
     }
 
-    for( ::std::list< rtl::OString >::iterator it = aLines.begin(); it != aLines.end(); ++it )
+    for( ::std::list< OString >::iterator it = aLines.begin(); it != aLines.end(); ++it )
     {
         if( ! access( it->getStr(), F_OK ) )
         {
@@ -242,7 +242,7 @@
 
 
 
-void X11SalInstance::FillFontPathList( std::list< rtl::OString >& o_rFontPaths )
+void X11SalInstance::FillFontPathList( std::list< OString >& o_rFontPaths )
 {
     Display *pDisplay = GetGenericData()->GetSalDisplay()->GetDisplay();
 
@@ -301,7 +301,7 @@
         // TODO: replace environment variables
         while( aStream.IsOpen() && ! aStream.IsEof() )
         {
-            rtl::OString aLine;
+            OString aLine;
             aStream.ReadLine( aLine );
             psp::normPath( aLine );
             // try to avoid bad fonts in some cases
@@ -316,11 +316,11 @@
 
 extern "C" { static void SAL_CALL thisModule() {} }
 
-void X11SalInstance::AddToRecentDocumentList(const rtl::OUString& rFileUrl, const rtl::OUString& 
rMimeType)
+void X11SalInstance::AddToRecentDocumentList(const OUString& rFileUrl, const OUString& rMimeType)
 {
-    const rtl::OUString 
SYM_ADD_TO_RECENTLY_USED_FILE_LIST(RTL_CONSTASCII_USTRINGPARAM("add_to_recently_used_file_list"));
-    const rtl::OUString LIB_RECENT_FILE(RTL_CONSTASCII_USTRINGPARAM("librecentfile.so"));
-    typedef void (*PFUNC_ADD_TO_RECENTLY_USED_LIST)(const rtl::OUString&, const rtl::OUString&);
+    const OUString SYM_ADD_TO_RECENTLY_USED_FILE_LIST("add_to_recently_used_file_list");
+    const OUString LIB_RECENT_FILE("librecentfile.so");
+    typedef void (*PFUNC_ADD_TO_RECENTLY_USED_LIST)(const OUString&, const OUString&);
 
     PFUNC_ADD_TO_RECENTLY_USED_LIST add_to_recently_used_file_list = 0;
 
diff --git a/vcl/unx/generic/app/sm.cxx b/vcl/unx/generic/app/sm.cxx
index 6d4b5e5..7cb72e8 100644
--- a/vcl/unx/generic/app/sm.cxx
+++ b/vcl/unx/generic/app/sm.cxx
@@ -189,12 +189,12 @@
         pSmProps[ 2 ].vals      = new SmPropValue[3];
         pSmProps[ 2 ].vals[0].length    = aExec.getLength()+1;
         pSmProps[ 2 ].vals[0].value = strdup( aExec.getStr() );
-        rtl::OStringBuffer aRestartOption;
-        aRestartOption.append(RTL_CONSTASCII_STRINGPARAM("--session="));
+        OStringBuffer aRestartOption;
+        aRestartOption.append("--session=");
         aRestartOption.append(SessionManagerClient::getSessionID());
         pSmProps[ 2 ].vals[1].length    = aRestartOption.getLength()+1;
         pSmProps[ 2 ].vals[1].value = strdup(aRestartOption.getStr());
-        rtl::OString aRestartOptionNoLogo(RTL_CONSTASCII_STRINGPARAM("--nologo"));
+        OString aRestartOptionNoLogo("--nologo");
         pSmProps[ 2 ].vals[2].length    = aRestartOptionNoLogo.getLength()+1;
         pSmProps[ 2 ].vals[2].value = strdup(aRestartOptionNoLogo.getStr());
 
@@ -204,7 +204,7 @@
         if( aSec )
         {
             osl_getUserName( aSec, &aUserName.pData );
-            aUser = rtl::OUStringToOString( aUserName, osl_getThreadTextEncoding() );
+            aUser = OUStringToOString( aUserName, osl_getThreadTextEncoding() );
             osl_freeSecurityHandle( aSec );
         }
 
@@ -451,7 +451,7 @@
                 SAL_INFO("vcl.sm", "SmcOpenConnection failed: " << aErrBuf);
             else
                 SAL_INFO("vcl.sm", "SmcOpenConnection succeeded, client ID is " << pClientID );
-            m_aClientID = rtl::OString(pClientID);
+            m_aClientID = OString(pClientID);
             free( pClientID );
             pClientID = NULL;
         }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I15ae64d621fd0a1fa178f9a4062b93520628bdb0
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Chris Sherlock <chris.sherlock79@gmail.com>


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.