[Libreoffice-commits] .: 2 commits - sfx2/inc sfx2/source sw/source vcl/inc vcl/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Oct 1 16:06:55 PDT 2012


 sfx2/inc/sfx2/sfxhelp.hxx        |   36 ++---
 sfx2/source/appl/newhelp.cxx     |   40 ++----
 sfx2/source/appl/newhelp.hxx     |    2 
 sfx2/source/appl/sfxhelp.cxx     |  236 +++++++++++++++++++--------------------
 sw/source/filter/ww8/ww8par2.cxx |    1 
 vcl/inc/helpwin.hxx              |   14 +-
 vcl/inc/vcl/help.hxx             |   32 ++---
 vcl/source/app/help.cxx          |   47 ++++---
 8 files changed, 200 insertions(+), 208 deletions(-)

New commits:
commit b966a09c2da9441961c93c44be556399575db849
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Oct 1 23:57:06 2012 +0100

    Resolves: fdo#54862 extra ++n causing merged cells to be skipped
    
    commit 567c1db25bd705faac44203e4a3d01d0f5e1385c reverted a pile
    of other commits, including 858b5b4f36a357fe7192e7c2ed9cc3cdfc81fd8f
    but didn't revert the ++n of that commit, leading to merge groups
    getting skipped
    
    Change-Id: Ie2b46ebc433df636ebe0092f4a992631b7f06a3f

diff --git a/sw/source/filter/ww8/ww8par2.cxx b/sw/source/filter/ww8/ww8par2.cxx
index a4336b9..83db83e 100644
--- a/sw/source/filter/ww8/ww8par2.cxx
+++ b/sw/source/filter/ww8/ww8par2.cxx
@@ -2752,7 +2752,6 @@ void WW8TabDesc::FinishSwTable()
                                                nRowSpan :
                                              ((-1) * (nRowSpan - n));
                     pCurrentBox->setRowSpan( nRowSpanSet );
-                    ++n;
                 }
             }
         }
commit d67dff5719b32c17f4bf02990de5d7a772db484c
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Oct 1 11:03:34 2012 +0100

    XubString->OUString
    
    Change-Id: Id64cdd70c2877ff71c5bc90286fab6b3d8ab7ad4

diff --git a/sfx2/inc/sfx2/sfxhelp.hxx b/sfx2/inc/sfx2/sfxhelp.hxx
index 12c58c2..9de00f0 100644
--- a/sfx2/inc/sfx2/sfxhelp.hxx
+++ b/sfx2/inc/sfx2/sfxhelp.hxx
@@ -19,45 +19,45 @@
 #ifndef _SFX_HELP_HXX
 #define _SFX_HELP_HXX
 
+#include <rtl/ustring.hxx>
 #include "sal/config.h"
 #include "sfx2/dllapi.h"
 #include "sal/types.h"
 #include <vcl/help.hxx>
-#include <tools/string.hxx>
 
 class SfxHelp_Impl;
 class SfxFrame;
 class SFX2_DLLPUBLIC SfxHelp : public Help
 {
-    String          aTicket;        // for Plugins
-    String          aUser;
-    String          aLanguageStr;
-    String          aCountryStr;
+    OUString        aTicket;        // for Plugins
+    OUString        aUser;
+    OUString        aLanguageStr;
+    OUString        aCountryStr;
     sal_Bool        bIsDebug;
     SfxHelp_Impl*   pImp;
 
 private:
-    SAL_DLLPRIVATE sal_Bool Start_Impl( const String& rURL, const Window* pWindow, const String& rKeyword );
-    SAL_DLLPRIVATE virtual sal_Bool SearchKeyword( const XubString& rKeyWord );
-    SAL_DLLPRIVATE virtual sal_Bool Start( const String& rURL, const Window* pWindow );
-    SAL_DLLPRIVATE virtual void OpenHelpAgent( const rtl::OString& sHelpId );
-    SAL_DLLPRIVATE String GetHelpModuleName_Impl();
-    SAL_DLLPRIVATE String CreateHelpURL_Impl( const String& aCommandURL, const String& rModuleName );
+    SAL_DLLPRIVATE sal_Bool Start_Impl( const OUString& rURL, const Window* pWindow, const OUString& rKeyword );
+    SAL_DLLPRIVATE virtual sal_Bool SearchKeyword( const OUString& rKeyWord );
+    SAL_DLLPRIVATE virtual sal_Bool Start( const OUString& rURL, const Window* pWindow );
+    SAL_DLLPRIVATE virtual void OpenHelpAgent( const OString& sHelpId );
+    SAL_DLLPRIVATE OUString GetHelpModuleName_Impl();
+    SAL_DLLPRIVATE OUString CreateHelpURL_Impl( const OUString& aCommandURL, const OUString& rModuleName );
 
 public:
     SfxHelp();
     ~SfxHelp();
 
-    inline void             SetTicket( const String& rTicket )  { aTicket = rTicket; }
-    inline void             SetUser( const String& rUser )      { aUser = rUser; }
+    inline void             SetTicket( const OUString& rTicket )  { aTicket = rTicket; }
+    inline void             SetUser( const OUString& rUser )      { aUser = rUser; }
 
-    virtual XubString       GetHelpText( const String&, const Window* pWindow );
+    virtual OUString        GetHelpText( const OUString&, const Window* pWindow );
 
-    static String           CreateHelpURL( const String& aCommandURL, const String& rModuleName );
+    static OUString         CreateHelpURL( const OUString& aCommandURL, const OUString& rModuleName );
     using Help::OpenHelpAgent;
-    static void             OpenHelpAgent( SfxFrame* pFrame, const rtl::OString& sHelpId );
-    static String           GetDefaultHelpModule();
-    static ::rtl::OUString  GetCurrentModuleIdentifier();
+    static void             OpenHelpAgent( SfxFrame* pFrame, const OString& sHelpId );
+    static OUString         GetDefaultHelpModule();
+    static OUString         GetCurrentModuleIdentifier();
 };
 
 #endif // #ifndef _SFX_HELP_HXX
diff --git a/sfx2/source/appl/newhelp.cxx b/sfx2/source/appl/newhelp.cxx
index f6da0ca..8e360c6 100644
--- a/sfx2/source/appl/newhelp.cxx
+++ b/sfx2/source/appl/newhelp.cxx
@@ -627,14 +627,11 @@ void IndexTabPage_Impl::InitializeIndex()
 
     try
     {
-        ::rtl::OUString aURL = HELP_URL;
-        aURL += ::rtl::OUString( sFactory );
+        OUStringBuffer aURL = HELP_URL;
+        aURL.append(sFactory);
+        AppendConfigToken(aURL, sal_True);
 
-        String aTemp = aURL;
-        AppendConfigToken( aTemp, sal_True );
-        aURL = aTemp;
-
-        Content aCnt( aURL, Reference< ::com::sun::star::ucb::XCommandEnvironment >(), comphelper::getProcessComponentContext() );
+        Content aCnt( aURL.makeStringAndClear(), Reference< ::com::sun::star::ucb::XCommandEnvironment >(), comphelper::getProcessComponentContext() );
         ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySetInfo > xInfo = aCnt.getProperties();
         if ( xInfo->hasPropertyByName( PROPERTY_ANCHORREF ) )
         {
@@ -1070,17 +1067,17 @@ IMPL_LINK_NOARG(SearchTabPage_Impl, SearchHdl)
         EnterWait();
         ClearSearchResults();
         RememberSearchText( aSearchText );
-        String aSearchURL = HELP_URL;
-        aSearchURL += aFactory;
-        aSearchURL += String( HELP_SEARCH_TAG );
+        OUStringBuffer aSearchURL(HELP_URL);
+        aSearchURL.append(aFactory);
+        aSearchURL.append(HELP_SEARCH_TAG);
         if ( !aFullWordsCB.IsChecked() )
             aSearchText = sfx2::PrepareSearchString( aSearchText, xBreakIterator, true );
-        aSearchURL += aSearchText;
-        AppendConfigToken( aSearchURL, sal_False );
+        aSearchURL.append(aSearchText);
+        AppendConfigToken(aSearchURL, sal_False);
         if ( aScopeCB.IsChecked() )
-            aSearchURL += DEFINE_CONST_UNICODE("&Scope=Heading");
-        Sequence< ::rtl::OUString > aFactories = SfxContentHelper::GetResultSet( aSearchURL );
-        const ::rtl::OUString* pFacs  = aFactories.getConstArray();
+            aSearchURL.append("&Scope=Heading");
+        Sequence< OUString > aFactories = SfxContentHelper::GetResultSet(aSearchURL.makeStringAndClear());
+        const OUString* pFacs  = aFactories.getConstArray();
         sal_uInt32 i, nCount = aFactories.getLength();
         for ( i = 0; i < nCount; ++i )
         {
@@ -1479,11 +1476,10 @@ void BookmarksTabPage_Impl::AddBookmarks( const String& rTitle, const String& rU
     sHelpURL.append(HELP_URL);
     sHelpURL.append(sFactory);
     sHelpURL.append(sContent);
-    String sURL = String(sHelpURL.makeStringAndClear());
-    AppendConfigToken(sURL, bUseQuestionMark);
+    AppendConfigToken(sHelpURL, bUseQuestionMark);
     if (!sAnchor.isEmpty())
-        sURL += String(sAnchor);
-    return ::rtl::OUString(sURL);
+        sHelpURL.append(sAnchor);
+    return sHelpURL.makeStringAndClear();
 }
 
 void SfxHelpWindow_Impl::loadHelpContent(const ::rtl::OUString& sHelpURL, sal_Bool bAddToHistory)
@@ -1597,9 +1593,9 @@ SfxHelpIndexWindow_Impl::~SfxHelpIndexWindow_Impl()
 
 void SfxHelpIndexWindow_Impl::Initialize()
 {
-    String aHelpURL = HELP_URL;
-    AppendConfigToken( aHelpURL, sal_True );
-    Sequence< ::rtl::OUString > aFactories = SfxContentHelper::GetResultSet( aHelpURL );
+    OUStringBuffer aHelpURL(HELP_URL);
+    AppendConfigToken(aHelpURL, sal_True);
+    Sequence< ::rtl::OUString > aFactories = SfxContentHelper::GetResultSet(aHelpURL.makeStringAndClear());
     const ::rtl::OUString* pFacs  = aFactories.getConstArray();
     sal_uInt32 i, nCount = aFactories.getLength();
     for ( i = 0; i < nCount; ++i )
diff --git a/sfx2/source/appl/newhelp.hxx b/sfx2/source/appl/newhelp.hxx
index 9ba6b69..6b60bb8 100644
--- a/sfx2/source/appl/newhelp.hxx
+++ b/sfx2/source/appl/newhelp.hxx
@@ -598,7 +598,7 @@ public:
 };
 
 /// Appends ?Language=xy&System=abc to the help URL in rURL
-void AppendConfigToken( String& rURL, sal_Bool bQuestionMark, const rtl::OUString &rLang = rtl::OUString() );
+void AppendConfigToken(OUStringBuffer& rURL, sal_Bool bQuestionMark, const OUString &rLang = rtl::OUString());
 
 #endif // #ifndef INCLUDED_SFX_NEWHELP_HXX
 
diff --git a/sfx2/source/appl/sfxhelp.cxx b/sfx2/source/appl/sfxhelp.cxx
index 0d89637..b2be3e3 100644
--- a/sfx2/source/appl/sfxhelp.cxx
+++ b/sfx2/source/appl/sfxhelp.cxx
@@ -98,15 +98,15 @@ void NoHelpErrorBox::RequestHelp( const HelpEvent& )
 
 #define STARTERLIST 0
 
-static bool impl_hasHelpInstalled( const rtl::OUString &rLang );
+static bool impl_hasHelpInstalled( const OUString &rLang );
 
 /// Return the locale we prefer for displaying help
-static rtl::OUString HelpLocaleString()
+static OUString HelpLocaleString()
 {
-    static rtl::OUString aLocaleStr;
+    static OUString aLocaleStr;
     if (aLocaleStr.isEmpty())
     {
-        const rtl::OUString aEnglish( "en"  );
+        const OUString aEnglish( "en"  );
         // detect installed locale
         aLocaleStr = utl::ConfigManager::getLocale();
         bool bOk = !aLocaleStr.isEmpty();
@@ -114,12 +114,12 @@ static rtl::OUString HelpLocaleString()
             aLocaleStr = aEnglish;
         else
         {
-            rtl::OUString aBaseInstallPath;
+            OUString aBaseInstallPath;
             utl::Bootstrap::locateBaseInstallation(aBaseInstallPath);
             static const char *szHelpPath = "/help/";
 
-            rtl::OUString sHelpPath = aBaseInstallPath +
-                rtl::OUString::createFromAscii(szHelpPath) + aLocaleStr;
+            OUString sHelpPath = aBaseInstallPath +
+                OUString::createFromAscii(szHelpPath) + aLocaleStr;
             osl::DirectoryItem aDirItem;
 
             if (!osl::DirectoryItem::get(sHelpPath, aDirItem) == osl::FileBase::E_None)
@@ -132,7 +132,7 @@ static rtl::OUString HelpLocaleString()
                     bOk = true;
                     sLang = sLang.Copy( 0, nSepPos );
                     sHelpPath = aBaseInstallPath +
-                        rtl::OUString::createFromAscii(szHelpPath) + sLang;
+                        OUString::createFromAscii(szHelpPath) + sLang;
                     if (!osl::DirectoryItem::get(sHelpPath, aDirItem) == osl::FileBase::E_None)
                         bOk = false;
                 }
@@ -146,40 +146,40 @@ static rtl::OUString HelpLocaleString()
     return aLocaleStr;
 }
 
-void AppendConfigToken( String& rURL, sal_Bool bQuestionMark, const rtl::OUString &rLang )
+void AppendConfigToken( OUStringBuffer& rURL, sal_Bool bQuestionMark, const OUString &rLang )
 {
-    ::rtl::OUString aLocaleStr( rLang );
+    OUString aLocaleStr( rLang );
     if ( aLocaleStr.isEmpty() )
         aLocaleStr = HelpLocaleString();
 
     // query part exists?
     if ( bQuestionMark )
         // no, so start with '?'
-        rURL += '?';
+        rURL.append('?');
     else
         // yes, so only append with '&'
-        rURL += '&';
+        rURL.append('&');
 
     // set parameters
-    rURL += DEFINE_CONST_UNICODE("Language=");
-    rURL += String( aLocaleStr );
-    rURL += DEFINE_CONST_UNICODE("&System=");
-    rURL += SvtHelpOptions().GetSystem();
-    rURL += DEFINE_CONST_UNICODE("&Version=");
-    rURL += utl::ConfigManager::getProductVersion();
+    rURL.append("Language=");
+    rURL.append(aLocaleStr);
+    rURL.append("&System=");
+    rURL.append(SvtHelpOptions().GetSystem());
+    rURL.append("&Version=");
+    rURL.append(utl::ConfigManager::getProductVersion());
 }
 
-sal_Bool GetHelpAnchor_Impl( const String& _rURL, String& _rAnchor )
+sal_Bool GetHelpAnchor_Impl( const OUString& _rURL, OUString& _rAnchor )
 {
     sal_Bool bRet = sal_False;
-    ::rtl::OUString sAnchor;
+    OUString sAnchor;
 
     try
     {
         ::ucbhelper::Content aCnt( INetURLObject( _rURL ).GetMainURL( INetURLObject::NO_DECODE ),
                              Reference< ::com::sun::star::ucb::XCommandEnvironment >(),
                              comphelper::getProcessComponentContext() );
-        if ( ( aCnt.getPropertyValue( ::rtl::OUString("AnchorName") ) >>= sAnchor ) )
+        if ( ( aCnt.getPropertyValue( OUString("AnchorName") ) >>= sAnchor ) )
         {
 
             if ( !sAnchor.isEmpty() )
@@ -203,18 +203,18 @@ sal_Bool GetHelpAnchor_Impl( const String& _rURL, String& _rAnchor )
 class SfxHelpOptions_Impl : public utl::ConfigItem
 {
 private:
-    std::set < rtl::OString > m_aIds;
+    std::set < OString > m_aIds;
 
 public:
                     SfxHelpOptions_Impl();
                     ~SfxHelpOptions_Impl();
 
-    bool            HasId( const rtl::OString& rId ) { return m_aIds.size() ? m_aIds.find( rId ) != m_aIds.end() : false; }
-    virtual void            Notify( const com::sun::star::uno::Sequence< rtl::OUString >& aPropertyNames );
+    bool            HasId( const OString& rId ) { return m_aIds.size() ? m_aIds.find( rId ) != m_aIds.end() : false; }
+    virtual void            Notify( const com::sun::star::uno::Sequence< OUString >& aPropertyNames );
     virtual void            Commit();
 };
 
-static Sequence< ::rtl::OUString > GetPropertyNames()
+static Sequence< OUString > GetPropertyNames()
 {
     static const char* aPropNames[] =
     {
@@ -222,20 +222,20 @@ static Sequence< ::rtl::OUString > GetPropertyNames()
     };
 
     const int nCount = sizeof( aPropNames ) / sizeof( const char* );
-    Sequence< ::rtl::OUString > aNames( nCount );
-    ::rtl::OUString* pNames = aNames.getArray();
-    ::rtl::OUString* pEnd   = pNames + aNames.getLength();
+    Sequence< OUString > aNames( nCount );
+    OUString* pNames = aNames.getArray();
+    OUString* pEnd   = pNames + aNames.getLength();
     int i = 0;
     for ( ; pNames != pEnd; ++pNames )
-        *pNames = ::rtl::OUString::createFromAscii( aPropNames[i++] );
+        *pNames = OUString::createFromAscii( aPropNames[i++] );
 
     return aNames;
 }
 
 SfxHelpOptions_Impl::SfxHelpOptions_Impl()
-    : ConfigItem( ::rtl::OUString("Office.SFX/Help") )
+    : ConfigItem( OUString("Office.SFX/Help") )
 {
-    Sequence< ::rtl::OUString > aNames = GetPropertyNames();
+    Sequence< OUString > aNames = GetPropertyNames();
     Sequence< Any > aValues = GetProperties( aNames );
     EnableNotification( aNames );
     const Any* pValues = aValues.getConstArray();
@@ -251,16 +251,16 @@ SfxHelpOptions_Impl::SfxHelpOptions_Impl()
                 {
                     case STARTERLIST :
                     {
-                        ::rtl::OUString aCodedList;
+                        OUString aCodedList;
                         if ( pValues[nProp] >>= aCodedList )
                         {
-                            rtl::OString aTmp(
-                                rtl::OUStringToOString(
-                                    aCodedList, RTL_TEXTENCODING_UTF8 ) );
+                            OString aTmp(
+                                OUStringToOString(
+                                    aCodedList, RTL_TEXTENCODING_UTF8));
                             sal_Int32 nIndex = 0;
                             do
                             {
-                                rtl::OString aToken = aTmp.getToken( 0, ',', nIndex );
+                                OString aToken = aTmp.getToken( 0, ',', nIndex );
                                 if ( !aToken.isEmpty() )
                                     m_aIds.insert( aToken );
                             }
@@ -287,7 +287,7 @@ SfxHelpOptions_Impl::~SfxHelpOptions_Impl()
 }
 
 
-void SfxHelpOptions_Impl::Notify( const com::sun::star::uno::Sequence< rtl::OUString >& )
+void SfxHelpOptions_Impl::Notify( const com::sun::star::uno::Sequence< OUString >& )
 {
 }
 
@@ -299,14 +299,14 @@ class SfxHelp_Impl
 {
 private:
     SfxHelpOptions_Impl*                m_pOpt;         // the options
-    ::std::vector< ::rtl::OUString >    m_aModulesList; // list of all installed modules
+    ::std::vector< OUString >    m_aModulesList; // list of all installed modules
 
 public:
     SfxHelp_Impl();
     ~SfxHelp_Impl();
 
     SfxHelpOptions_Impl*    GetOptions();
-    static String           GetHelpText( const rtl::OUString& aCommandURL, const String& rModule );
+    static String           GetHelpText( const OUString& aCommandURL, const String& rModule );
 };
 
 SfxHelp_Impl::SfxHelp_Impl() :
@@ -321,7 +321,7 @@ SfxHelp_Impl::~SfxHelp_Impl()
     delete m_pOpt;
 }
 
-String SfxHelp_Impl::GetHelpText( const rtl::OUString& aCommandURL, const String& rModule )
+String SfxHelp_Impl::GetHelpText( const OUString& aCommandURL, const String& rModule )
 {
     // create help url
     String aHelpURL = SfxHelp::CreateHelpURL( aCommandURL, rModule );
@@ -348,15 +348,15 @@ SfxHelp::SfxHelp() :
     // read the environment variable "HELP_DEBUG"
     // if it's set, you will see debug output on active help
     {
-        ::rtl::OUString sHelpDebug;
-        ::rtl::OUString sEnvVarName( "HELP_DEBUG"  );
+        OUString sHelpDebug;
+        OUString sEnvVarName( "HELP_DEBUG"  );
         osl_getEnvironment( sEnvVarName.pData, &sHelpDebug.pData );
         bIsDebug = !sHelpDebug.isEmpty();
     }
 
     pImp = new SfxHelp_Impl();
 
-    ::rtl::OUString aLocaleStr = HelpLocaleString();
+    OUString aLocaleStr = HelpLocaleString();
 
     sal_Int32 nSepPos = aLocaleStr.indexOf( '_' );
     if ( nSepPos != -1 )
@@ -384,9 +384,9 @@ SfxHelp::~SfxHelp()
     delete pImp;
 }
 
-::rtl::OUString getDefaultModule_Impl()
+OUString getDefaultModule_Impl()
 {
-    rtl::OUString sDefaultModule;
+    OUString sDefaultModule;
     SvtModuleOptions aModOpt;
     if ( aModOpt.IsModuleInstalled( SvtModuleOptions::E_SWRITER ) )
         sDefaultModule = DEFINE_CONST_UNICODE("swriter");
@@ -411,9 +411,9 @@ SfxHelp::~SfxHelp()
     return sDefaultModule;
 }
 
-::rtl::OUString getCurrentModuleIdentifier_Impl()
+OUString getCurrentModuleIdentifier_Impl()
 {
-    ::rtl::OUString sIdentifier;
+    OUString sIdentifier;
     Reference < XFrame > xCurrentFrame;
     Reference < XModuleManager2 > xModuleManager( ModuleManager::create(::comphelper::getProcessComponentContext()) );
     Reference < XDesktop > xDesktop( ::comphelper::getProcessServiceFactory()->createInstance(
@@ -440,11 +440,10 @@ SfxHelp::~SfxHelp()
     return sIdentifier;
 }
 
-String SfxHelp::GetHelpModuleName_Impl()
+OUString SfxHelp::GetHelpModuleName_Impl()
 {
-    String sModuleName;
-    rtl::OUString aFactoryShortName;
-    rtl::OUString aModuleIdentifier = getCurrentModuleIdentifier_Impl();
+    OUString aFactoryShortName;
+    OUString aModuleIdentifier = getCurrentModuleIdentifier_Impl();
 
     if ( !aModuleIdentifier.isEmpty() )
     {
@@ -469,18 +468,18 @@ String SfxHelp::GetHelpModuleName_Impl()
         }
     }
 
-    rtl::OUString sDefaultModule = getDefaultModule_Impl();
+    OUString sDefaultModule = getDefaultModule_Impl();
     if ( !aFactoryShortName.isEmpty() )
     {
         // Map some module identifiers to their "real" help module string.
         if ( aFactoryShortName == "chart2" )
-            aFactoryShortName = rtl::OUString( "schart"  );
+            aFactoryShortName = OUString( "schart"  );
         else if ( aFactoryShortName == "BasicIDE" )
-            aFactoryShortName = rtl::OUString( "sbasic"  );
+            aFactoryShortName = OUString( "sbasic"  );
         else if ( aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("sweb"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("sglobal"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("swxform")) )
-            aFactoryShortName = rtl::OUString( "swriter"  );
+            aFactoryShortName = OUString( "swriter"  );
         else if ( aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("dbquery"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("dbbrowser"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("dbrelation"))
@@ -490,7 +489,7 @@ String SfxHelp::GetHelpModuleName_Impl()
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("swreport"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("dbbrowser"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("swform")) )
-            aFactoryShortName = rtl::OUString( "sdatabase"  );
+            aFactoryShortName = OUString( "sdatabase"  );
         else if ( aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("sbibliography"))
                 || aFactoryShortName.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("StartModule")) )
             aFactoryShortName = sDefaultModule;
@@ -498,48 +497,46 @@ String SfxHelp::GetHelpModuleName_Impl()
     else
         aFactoryShortName = sDefaultModule;
 
-    sModuleName = String( aFactoryShortName );
-    return sModuleName;
+    return aFactoryShortName;
 }
 
-String  SfxHelp::CreateHelpURL_Impl( const String& aCommandURL, const String& rModuleName )
+OUString SfxHelp::CreateHelpURL_Impl( const OUString& aCommandURL, const OUString& rModuleName )
 {
     // build up the help URL
-    String aHelpURL;
+    OUStringBuffer aHelpURL("vnd.sun.star.help://");
     sal_Bool bHasAnchor = sal_False;
-    String aAnchor;
+    OUString aAnchor;
 
-    String aModuleName( rModuleName );
-    if ( aModuleName.Len() == 0 )
+    OUString aModuleName( rModuleName );
+    if (aModuleName.isEmpty())
         aModuleName = getDefaultModule_Impl();
 
-    aHelpURL = rtl::OUString("vnd.sun.star.help://");
-    aHelpURL += aModuleName;
+    aHelpURL.append(aModuleName);
 
-    if ( !aCommandURL.Len() )
-        aHelpURL += rtl::OUString("/start");
+    if ( aCommandURL.isEmpty() )
+        aHelpURL.append("/start");
     else
     {
-        aHelpURL += '/';
-        aHelpURL += String( rtl::Uri::encode( aCommandURL,
+        aHelpURL.append('/');
+        aHelpURL.append(rtl::Uri::encode(aCommandURL,
                                               rtl_UriCharClassRelSegment,
                                               rtl_UriEncodeKeepEscapes,
-                                              RTL_TEXTENCODING_UTF8 ));
+                                              RTL_TEXTENCODING_UTF8));
 
-        String aTempURL = aHelpURL;
+        OUStringBuffer aTempURL = aHelpURL;
         AppendConfigToken( aTempURL, sal_True );
-        bHasAnchor = GetHelpAnchor_Impl( aTempURL, aAnchor );
+        bHasAnchor = GetHelpAnchor_Impl(aTempURL.makeStringAndClear(), aAnchor);
     }
 
     AppendConfigToken( aHelpURL, sal_True );
 
     if ( bHasAnchor )
     {
-        aHelpURL += '#';
-        aHelpURL += aAnchor;
+        aHelpURL.append('#');
+        aHelpURL.append(aAnchor);
     }
 
-    return aHelpURL;
+    return aHelpURL.makeStringAndClear();
 }
 
 SfxHelpWindow_Impl* impl_createHelp(Reference< XFrame >& rHelpTask   ,
@@ -550,7 +547,7 @@ SfxHelpWindow_Impl* impl_createHelp(Reference< XFrame >& rHelpTask   ,
 
     // otherwhise - create new help task
     Reference< XFrame > xHelpTask = xDesktop->findFrame(
-        ::rtl::OUString(DEFINE_CONST_UNICODE("OFFICE_HELP_TASK")),
+        OUString("OFFICE_HELP_TASK"),
         FrameSearchFlag::TASKS | FrameSearchFlag::CREATE);
     if (!xHelpTask.is())
         return 0;
@@ -565,7 +562,7 @@ SfxHelpWindow_Impl* impl_createHelp(Reference< XFrame >& rHelpTask   ,
     if (xHelpTask->setComponent( xHelpWindow, Reference< XController >() ))
     {
         // Customize UI ...
-        xHelpTask->setName( ::rtl::OUString(DEFINE_CONST_UNICODE("OFFICE_HELP_TASK")) );
+        xHelpTask->setName( OUString("OFFICE_HELP_TASK") );
 
         Reference< XPropertySet > xProps(xHelpTask, UNO_QUERY);
         if (xProps.is())
@@ -579,42 +576,42 @@ SfxHelpWindow_Impl* impl_createHelp(Reference< XFrame >& rHelpTask   ,
 
         // This sub frame is created internaly (if we called new SfxHelpWindow_Impl() ...)
         // It should exist :-)
-        xHelpContent = xHelpTask->findFrame(::rtl::OUString(DEFINE_CONST_UNICODE("OFFICE_HELP")), FrameSearchFlag::CHILDREN);
+        xHelpContent = xHelpTask->findFrame(OUString("OFFICE_HELP"), FrameSearchFlag::CHILDREN);
     }
 
     if (!xHelpContent.is())
         delete pHelpWindow;
 
-    xHelpContent->setName(::rtl::OUString(DEFINE_CONST_UNICODE("OFFICE_HELP")));
+    xHelpContent->setName(OUString("OFFICE_HELP"));
 
     rHelpTask    = xHelpTask;
     rHelpContent = xHelpContent;
     return pHelpWindow;
 }
 
-XubString SfxHelp::GetHelpText( const String& aCommandURL, const Window* pWindow )
+OUString SfxHelp::GetHelpText( const OUString& aCommandURL, const Window* pWindow )
 {
-    String sModuleName = GetHelpModuleName_Impl();
-    String sHelpText = pImp->GetHelpText( aCommandURL, sModuleName );
+    OUString sModuleName = GetHelpModuleName_Impl();
+    OUString sHelpText = pImp->GetHelpText( aCommandURL, sModuleName );
 
-    rtl::OString aNewHelpId;
+    OString aNewHelpId;
 
-    if ( pWindow && !sHelpText.Len() )
+    if (pWindow && sHelpText.isEmpty())
     {
         // no help text found -> try with parent help id.
         Window* pParent = pWindow->GetParent();
         while ( pParent )
         {
             aNewHelpId = pParent->GetHelpId();
-            sHelpText = pImp->GetHelpText( rtl::OStringToOUString(aNewHelpId, RTL_TEXTENCODING_UTF8), sModuleName );
-            if ( sHelpText.Len() > 0 )
+            sHelpText = pImp->GetHelpText( OStringToOUString(aNewHelpId, RTL_TEXTENCODING_UTF8), sModuleName );
+            if (!sHelpText.isEmpty())
                 pParent = NULL;
             else
                 pParent = pParent->GetParent();
         }
 
-        if ( bIsDebug && !sHelpText.Len() )
-            aNewHelpId = rtl::OString();
+        if (bIsDebug && sHelpText.isEmpty())
+            aNewHelpId = OString();
     }
 
     // add some debug information?
@@ -627,7 +624,7 @@ XubString SfxHelp::GetHelpText( const String& aCommandURL, const Window* pWindow
         if ( !aNewHelpId.isEmpty() )
         {
             sHelpText += DEFINE_CONST_UNICODE(" - ");
-            sHelpText += String(rtl::OStringToOUString(aNewHelpId, RTL_TEXTENCODING_UTF8));
+            sHelpText += String(OStringToOUString(aNewHelpId, RTL_TEXTENCODING_UTF8));
         }
     }
 
@@ -635,23 +632,23 @@ XubString SfxHelp::GetHelpText( const String& aCommandURL, const Window* pWindow
 }
 
 /// Check for built-in help
-static bool impl_hasHelpInstalled( const rtl::OUString &rLang = rtl::OUString() )
+static bool impl_hasHelpInstalled( const OUString &rLang = OUString() )
 {
-    String aHelpRootURL( DEFINE_CONST_OUSTRING("vnd.sun.star.help://") );
-    AppendConfigToken( aHelpRootURL, sal_True, rLang );
-    Sequence< ::rtl::OUString > aFactories = SfxContentHelper::GetResultSet( aHelpRootURL );
+    OUStringBuffer aHelpRootURL("vnd.sun.star.help://");
+    AppendConfigToken(aHelpRootURL, sal_True, rLang);
+    Sequence< OUString > aFactories = SfxContentHelper::GetResultSet(aHelpRootURL.makeStringAndClear());
 
     return ( aFactories.getLength() != 0   );
 }
 
-sal_Bool SfxHelp::SearchKeyword( const XubString& rKeyword )
+sal_Bool SfxHelp::SearchKeyword( const OUString& rKeyword )
 {
     return Start_Impl( String(), NULL, rKeyword );
 }
 
-sal_Bool SfxHelp::Start( const String& rURL, const Window* pWindow )
+sal_Bool SfxHelp::Start( const OUString& rURL, const Window* pWindow )
 {
-    return Start_Impl( rURL, pWindow, String() );
+    return Start_Impl( rURL, pWindow, OUString() );
 }
 
 /// Redirect the vnd.sun.star.help:// urls to http://help.libreoffice.org
@@ -661,14 +658,14 @@ static bool impl_showOnlineHelp( const String& rURL )
     if ( rURL.Len() <= aInternal.Len() || rURL.Copy( 0, aInternal.Len() ) != aInternal )
         return false;
 
-    rtl::OUString aHelpLink( "http://help.libreoffice.org/"  );
+    OUString aHelpLink( "http://help.libreoffice.org/"  );
     aHelpLink += rURL.Copy( aInternal.Len() );
     try
     {
         Reference< XSystemShellExecute > xSystemShell(
                 SystemShellExecute::create(::comphelper::getProcessComponentContext()) );
 
-        xSystemShell->execute( aHelpLink, rtl::OUString(), SystemShellExecuteFlags::URIS_ONLY );
+        xSystemShell->execute( aHelpLink, OUString(), SystemShellExecuteFlags::URIS_ONLY );
         return true;
     }
     catch (const Exception&)
@@ -677,11 +674,11 @@ static bool impl_showOnlineHelp( const String& rURL )
     return false;
 }
 
-sal_Bool SfxHelp::Start_Impl( const String& rURL, const Window* pWindow, const String& rKeyword )
+sal_Bool SfxHelp::Start_Impl(const OUString& rURL, const Window* pWindow, const OUString& rKeyword)
 {
-    String aHelpRootURL( DEFINE_CONST_OUSTRING("vnd.sun.star.help://") );
-    AppendConfigToken( aHelpRootURL, sal_True);
-    Sequence< ::rtl::OUString > aFactories = SfxContentHelper::GetResultSet( aHelpRootURL );
+    OUStringBuffer aHelpRootURL("vnd.sun.star.help://");
+    AppendConfigToken(aHelpRootURL, sal_True);
+    Sequence< OUString > aFactories = SfxContentHelper::GetResultSet(aHelpRootURL.makeStringAndClear());
 
     /* rURL may be
         - a "real" URL
@@ -716,8 +713,8 @@ sal_Bool SfxHelp::Start_Impl( const String& rURL, const Window* pWindow, const S
                 Window* pParent = pWindow->GetParent();
                 while ( pParent )
                 {
-                    rtl::OString aHelpId = pParent->GetHelpId();
-                    aHelpURL = CreateHelpURL( rtl::OStringToOUString(aHelpId, RTL_TEXTENCODING_UTF8), aHelpModuleName );
+                    OString aHelpId = pParent->GetHelpId();
+                    aHelpURL = CreateHelpURL( OStringToOUString(aHelpId, RTL_TEXTENCODING_UTF8), aHelpModuleName );
                     if ( !SfxContentHelper::IsHelpErrorDocument( aHelpURL ) )
                     {
                         break;
@@ -756,10 +753,10 @@ sal_Bool SfxHelp::Start_Impl( const String& rURL, const Window* pWindow, const S
     // If not, create a new one and return access directly to the internal sub frame showing the help content
     // search must be done here; search one desktop level could return an arbitraty frame
     Reference< XFrame > xHelp = xDesktop->findFrame(
-        ::rtl::OUString(DEFINE_CONST_UNICODE("OFFICE_HELP_TASK")),
+        OUString("OFFICE_HELP_TASK"),
         FrameSearchFlag::CHILDREN);
     Reference< XFrame > xHelpContent = xDesktop->findFrame(
-        ::rtl::OUString(DEFINE_CONST_UNICODE("OFFICE_HELP")),
+        OUString("OFFICE_HELP"),
         FrameSearchFlag::CHILDREN);
 
     SfxHelpWindow_Impl* pHelpWindow = 0;
@@ -771,14 +768,14 @@ sal_Bool SfxHelp::Start_Impl( const String& rURL, const Window* pWindow, const S
         return sal_False;
 
 #ifdef DBG_UTIL
-    rtl::OStringBuffer aTmp(RTL_CONSTASCII_STRINGPARAM("SfxHelp: HelpId = "));
-    aTmp.append(rtl::OUStringToOString(aHelpURL, RTL_TEXTENCODING_UTF8));
+    OStringBuffer aTmp(RTL_CONSTASCII_STRINGPARAM("SfxHelp: HelpId = "));
+    aTmp.append(OUStringToOString(aHelpURL, RTL_TEXTENCODING_UTF8));
     OSL_TRACE( aTmp.getStr() );
 #endif
 
     pHelpWindow->SetHelpURL( aHelpURL );
     pHelpWindow->loadHelpContent(aHelpURL);
-    if ( rKeyword.Len() )
+    if (!rKeyword.isEmpty())
         pHelpWindow->OpenKeyword( rKeyword );
 
     Reference < ::com::sun::star::awt::XTopWindow > xTopWindow( xHelp->getContainerWindow(), UNO_QUERY );
@@ -788,23 +785,20 @@ sal_Bool SfxHelp::Start_Impl( const String& rURL, const Window* pWindow, const S
     return sal_True;
 }
 
-String SfxHelp::CreateHelpURL( const String& aCommandURL, const String& rModuleName )
+OUString SfxHelp::CreateHelpURL(const OUString& aCommandURL, const OUString& rModuleName)
 {
-    String aURL;
-    SfxHelp* pHelp = (static_cast< SfxHelp* >(Application::GetHelp()) );
-    if ( pHelp )
-        aURL = pHelp->CreateHelpURL_Impl( aCommandURL, rModuleName );
-    return aURL;
+    SfxHelp* pHelp = static_cast< SfxHelp* >(Application::GetHelp());
+    return pHelp ? pHelp->CreateHelpURL_Impl( aCommandURL, rModuleName ) : OUString();
 }
 
-void SfxHelp::OpenHelpAgent( SfxFrame*, const rtl::OString& sHelpId )
+void SfxHelp::OpenHelpAgent( SfxFrame*, const OString& sHelpId )
 {
     SfxHelp* pHelp = (static_cast< SfxHelp* >(Application::GetHelp()) );
     if ( pHelp )
         pHelp->OpenHelpAgent( sHelpId );
 }
 
-void SfxHelp::OpenHelpAgent( const rtl::OString& sHelpId )
+void SfxHelp::OpenHelpAgent( const OString& sHelpId )
 {
     if ( SvtHelpOptions().IsHelpAgentAutoStartMode() )
     {
@@ -815,7 +809,7 @@ void SfxHelp::OpenHelpAgent( const rtl::OString& sHelpId )
             try
             {
                 URL aURL;
-                aURL.Complete = CreateHelpURL_Impl( rtl::OStringToOUString(sHelpId, RTL_TEXTENCODING_UTF8), GetHelpModuleName_Impl() );
+                aURL.Complete = CreateHelpURL_Impl( OStringToOUString(sHelpId, RTL_TEXTENCODING_UTF8), GetHelpModuleName_Impl() );
                 Reference< XURLTransformer > xTrans( URLTransformer::create( ::comphelper::getProcessComponentContext() ) );
                 xTrans->parseStrict(aURL);
 
@@ -829,7 +823,7 @@ void SfxHelp::OpenHelpAgent( const rtl::OString& sHelpId )
                 Reference< XDispatch > xHelpDispatch;
                 if ( xDispProv.is() )
                     xHelpDispatch = xDispProv->queryDispatch(
-                        aURL, ::rtl::OUString("_helpagent"),
+                        aURL, OUString("_helpagent"),
                         FrameSearchFlag::PARENT | FrameSearchFlag::SELF );
 
                 DBG_ASSERT( xHelpDispatch.is(), "OpenHelpAgent: could not get a dispatcher!" );
@@ -843,12 +837,12 @@ void SfxHelp::OpenHelpAgent( const rtl::OString& sHelpId )
     }
 }
 
-String SfxHelp::GetDefaultHelpModule()
+OUString SfxHelp::GetDefaultHelpModule()
 {
     return getDefaultModule_Impl();
 }
 
-::rtl::OUString SfxHelp::GetCurrentModuleIdentifier()
+OUString SfxHelp::GetCurrentModuleIdentifier()
 {
     return getCurrentModuleIdentifier_Impl();
 }
diff --git a/vcl/inc/helpwin.hxx b/vcl/inc/helpwin.hxx
index 40a969f..7dd0c31 100644
--- a/vcl/inc/helpwin.hxx
+++ b/vcl/inc/helpwin.hxx
@@ -44,8 +44,8 @@ private:
 
     Rectangle           maTextRect; // For wrapped text in QuickHelp
 
-    String              maHelpText;
-    String              maStatusText;
+    OUString            maHelpText;
+    OUString            maStatusText;
 
     Timer               maShowTimer;
     Timer               maHideTimer;
@@ -61,16 +61,16 @@ protected:
     void                ImplShow();
 
 public:
-                        HelpTextWindow( Window* pParent, const String& rText, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle );
+                        HelpTextWindow( Window* pParent, const OUString& rText, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle );
                         ~HelpTextWindow();
 
-    const String&       GetHelpText() const { return maHelpText; }
-    void                SetHelpText( const String& rHelpText );
+    const OUString&     GetHelpText() const { return maHelpText; }
+    void                SetHelpText( const OUString& rHelpText );
     sal_uInt16          GetWinStyle() const { return mnHelpWinStyle; }
     sal_uInt16          GetStyle() const { return mnStyle; }
 
     // Nur merken:
-    void                SetStatusText( const String& rStatusText ) { maStatusText = rStatusText; }
+    void                SetStatusText( const OUString& rStatusText ) { maStatusText = rStatusText; }
     void                SetHelpArea( const Rectangle& rRect ) { maHelpArea = rRect; }
 
     void                ShowHelp( sal_uInt16 nDelayMode );
@@ -82,7 +82,7 @@ public:
 };
 
 void ImplShowHelpWindow( Window* pParent, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle,
-        const String& rHelpText, const String& rStatusText,
+        const OUString& rHelpText, const OUString& rStatusText,
         const Point& rScreenPos, const Rectangle* pHelpArea = NULL );
 void ImplDestroyHelpWindow( bool bUpdateHideTime );
 void ImplSetHelpWindowPos( Window* pHelpWindow, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle,
diff --git a/vcl/inc/vcl/help.hxx b/vcl/inc/vcl/help.hxx
index 1906bca..8753c3c 100644
--- a/vcl/inc/vcl/help.hxx
+++ b/vcl/inc/vcl/help.hxx
@@ -29,7 +29,7 @@
 #ifndef _SV_HELP_HXX
 #define _SV_HELP_HXX
 
-#include <tools/string.hxx>
+#include <rtl/ustring.hxx>
 #include <tools/solar.h>
 #include <vcl/dllapi.h>
 
@@ -69,19 +69,19 @@ class Window;
 class VCL_DLLPUBLIC Help
 {
 private:
-    String              maHelpFile;
+    OUString            maHelpFile;
 
 public:
                         Help();
     virtual             ~Help();
 
-    void                SetHelpFile( const String& rFileName ) { maHelpFile = rFileName; }
-    const String&       GetHelpFile() const { return maHelpFile; }
+    void                SetHelpFile( const OUString& rFileName ) { maHelpFile = rFileName; }
+    const OUString&     GetHelpFile() const { return maHelpFile; }
 
-    virtual sal_Bool    Start( const XubString& rHelpId, const Window* pWindow );
-    virtual sal_Bool    SearchKeyword( const XubString& rKeyWord );
-    virtual void        OpenHelpAgent( const rtl::OString& rHelpId );
-    virtual XubString   GetHelpText( const String& aHelpURL, const Window* pWindow );
+    virtual sal_Bool    Start( const OUString& rHelpId, const Window* pWindow );
+    virtual sal_Bool    SearchKeyword( const OUString& rKeyWord );
+    virtual void        OpenHelpAgent( const OString& rHelpId );
+    virtual OUString    GetHelpText( const OUString& aHelpURL, const Window* pWindow );
 
     static void         EnableContextHelp();
     static void         DisableContextHelp();
@@ -98,35 +98,35 @@ public:
     static sal_Bool     IsBalloonHelpEnabled();
     static sal_Bool     ShowBalloon( Window* pParent,
                                      const Point& rScreenPos,
-                                     const XubString& rHelpText );
+                                     const OUString& rHelpText );
     static sal_Bool     ShowBalloon( Window* pParent,
                                      const Point& rScreenPos,
                                      const Rectangle&,
-                                     const XubString& rHelpText );
+                                     const OUString& rHelpText );
 
     static void         EnableQuickHelp();
     static void         DisableQuickHelp();
     static sal_Bool     IsQuickHelpEnabled();
     static sal_Bool     ShowQuickHelp( Window* pParent,
                                        const Rectangle& rScreenRect,
-                                       const XubString& rHelpText,
-                                       const XubString& rLongHelpText,
+                                       const OUString& rHelpText,
+                                       const OUString& rLongHelpText,
                                        sal_uInt16 nStyle = 0 );
     static sal_Bool     ShowQuickHelp( Window* pParent,
                                        const Rectangle& rScreenRect,
-                                       const XubString& rHelpText,
+                                       const OUString& rHelpText,
                                        sal_uInt16 nStyle = 0 )
-                            { return Help::ShowQuickHelp( pParent, rScreenRect, rHelpText, XubString(), nStyle ); }
+                            { return Help::ShowQuickHelp( pParent, rScreenRect, rHelpText, OUString(), nStyle ); }
 
     static void         HideBalloonAndQuickHelp();
 
     static sal_uLong    ShowTip( Window* pParent,
                                  const Rectangle& rScreenRect,
-                                 const XubString& rText, sal_uInt16 nStyle = 0 );
+                                 const OUString& rText, sal_uInt16 nStyle = 0 );
     static void         UpdateTip( sal_uLong nId,
                                    Window* pParent,
                                    const Rectangle& rScreenRect,
-                                   const XubString& rText );
+                                   const OUString& rText );
     static void         HideTip( sal_uLong nId );
 };
 
diff --git a/vcl/source/app/help.cxx b/vcl/source/app/help.cxx
index 9cb7a03..18a6573 100644
--- a/vcl/source/app/help.cxx
+++ b/vcl/source/app/help.cxx
@@ -26,6 +26,7 @@
  *
  ************************************************************************/
 
+#include <comphelper/string.hxx>
 
 #include "tools/debug.hxx"
 #include "tools/diagnose_ex.h"
@@ -66,25 +67,25 @@ Help::~Help()
 
 // -----------------------------------------------------------------------
 
-void Help::OpenHelpAgent( const rtl::OString& )
+void Help::OpenHelpAgent( const OString& )
 {
 }
 
 // -----------------------------------------------------------------------
 
-sal_Bool Help::Start( const XubString&, const Window* )
+sal_Bool Help::Start( const OUString&, const Window* )
 {
     return sal_False;
 }
 
-sal_Bool Help::SearchKeyword( const XubString& )
+sal_Bool Help::SearchKeyword( const OUString& )
 {
     return sal_False;
 }
 
 // -----------------------------------------------------------------------
 
-XubString Help::GetHelpText( const String&, const Window* )
+OUString Help::GetHelpText( const OUString&, const Window* )
 {
     return ImplGetSVEmptyStr();
 }
@@ -193,7 +194,7 @@ sal_Bool Help::IsBalloonHelpEnabled()
 
 sal_Bool Help::ShowBalloon( Window* pParent,
                         const Point& rScreenPos,
-                        const XubString& rHelpText )
+                        const OUString& rHelpText )
 {
     ImplShowHelpWindow( pParent, HELPWINSTYLE_BALLOON, 0,
                         rHelpText, ImplGetSVEmptyStr(), rScreenPos );
@@ -205,7 +206,7 @@ sal_Bool Help::ShowBalloon( Window* pParent,
 
 sal_Bool Help::ShowBalloon( Window* pParent,
                         const Point& rScreenPos, const Rectangle& rRect,
-                        const XubString& rHelpText )
+                        const OUString& rHelpText )
 {
     ImplShowHelpWindow( pParent, HELPWINSTYLE_BALLOON, 0,
                         rHelpText, ImplGetSVEmptyStr(), rScreenPos, &rRect );
@@ -238,8 +239,8 @@ sal_Bool Help::IsQuickHelpEnabled()
 
 sal_Bool Help::ShowQuickHelp( Window* pParent,
                           const Rectangle& rScreenRect,
-                          const XubString& rHelpText,
-                          const XubString& rLongHelpText,
+                          const OUString& rHelpText,
+                          const OUString& rLongHelpText,
                           sal_uInt16 nStyle )
 {
     ImplShowHelpWindow( pParent, HELPWINSTYLE_QUICK, nStyle,
@@ -260,7 +261,7 @@ void Help::HideBalloonAndQuickHelp()
 // -----------------------------------------------------------------------
 
 sal_uIntPtr Help::ShowTip( Window* pParent, const Rectangle& rScreenRect,
-                     const XubString& rText, sal_uInt16 nStyle )
+                     const OUString& rText, sal_uInt16 nStyle )
 {
     sal_uInt16 nHelpWinStyle = ( ( nStyle & QUICKHELP_TIP_STYLE_BALLOON ) != 0 ) ? HELPWINSTYLE_BALLOON : HELPWINSTYLE_QUICK;
     HelpTextWindow* pHelpWin = new HelpTextWindow( pParent, rText, nHelpWinStyle, nStyle );
@@ -274,7 +275,7 @@ sal_uIntPtr Help::ShowTip( Window* pParent, const Rectangle& rScreenRect,
 
 // -----------------------------------------------------------------------
 
-void Help::UpdateTip( sal_uIntPtr nId, Window* pParent, const Rectangle& rScreenRect, const XubString& rText )
+void Help::UpdateTip( sal_uIntPtr nId, Window* pParent, const Rectangle& rScreenRect, const OUString& rText )
 {
     HelpTextWindow* pHelpWin = reinterpret_cast< HelpTextWindow* >( nId );
     ENSURE_OR_RETURN_VOID( pHelpWin != NULL, "Help::UpdateTip: invalid ID!" );
@@ -303,7 +304,7 @@ void Help::HideTip( sal_uLong nId )
 
 // =======================================================================
 
-HelpTextWindow::HelpTextWindow( Window* pParent, const XubString& rText, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle ) :
+HelpTextWindow::HelpTextWindow( Window* pParent, const OUString& rText, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle ) :
     //FloatingWindow( pParent->ImplGetFrameWindow(), WB_SYSTEMWINDOW ),
     FloatingWindow( pParent, WB_SYSTEMWINDOW|WB_TOOLTIPWIN ), // #105827# if we change the parent, mirroring will not work correctly when positioning this window
     maHelpText( rText )
@@ -367,10 +368,10 @@ HelpTextWindow::~HelpTextWindow()
 
 // -----------------------------------------------------------------------
 
-void HelpTextWindow::SetHelpText( const String& rHelpText )
+void HelpTextWindow::SetHelpText( const OUString& rHelpText )
 {
     maHelpText = rHelpText;
-    if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.Len() < HELPTEXTMAXLEN)
+    if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.getLength() < HELPTEXTMAXLEN)
     {
         Size aSize;
         aSize.Height() = GetTextHeight();
@@ -383,9 +384,11 @@ void HelpTextWindow::SetHelpText( const String& rHelpText )
     else // HELPWINSTYLE_BALLOON
     {
         Point       aTmpPoint;
-        sal_uInt16      nCharsInLine = 35 + ((maHelpText.Len()/100)*5);
-        XubString   aXXX;
-        aXXX.Fill( nCharsInLine, 'x' );   // average width to have all windows consistent
+        sal_Int32 nCharsInLine = 35 + ((maHelpText.getLength()/100)*5);
+        // average width to have all windows consistent
+        OUStringBuffer aBuf;
+        comphelper::string::padToLength(aBuf, nCharsInLine, 'x');
+        OUString aXXX = aBuf.makeStringAndClear();
         long nWidth = GetTextWidth( aXXX );
         Size aTmpSize( nWidth, 0x7FFFFFFF );
         Rectangle aTry1( aTmpPoint, aTmpSize );
@@ -428,11 +431,11 @@ void HelpTextWindow::Paint( const Rectangle& )
         Rectangle aCtrlRegion( Point( 0, 0 ), GetOutputSizePixel() );
         ImplControlValue    aControlValue;
         bNativeOK = DrawNativeControl( CTRL_TOOLTIP, PART_ENTIRE_CONTROL, aCtrlRegion,
-                                       0, aControlValue, rtl::OUString() );
+                                       0, aControlValue, OUString() );
     }
 
     // paint text
-    if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.Len() < HELPTEXTMAXLEN)
+    if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.getLength() < HELPTEXTMAXLEN)
     {
         if ( mnStyle & QUICKHELP_CTRLTEXT )
             DrawCtrlText( maTextRect.TopLeft(), maHelpText );
@@ -536,7 +539,7 @@ void HelpTextWindow::RequestHelp( const HelpEvent& /*rHEvt*/ )
 
 // -----------------------------------------------------------------------
 
-String HelpTextWindow::GetText() const
+XubString HelpTextWindow::GetText() const
 {
     return maHelpText;
 }
@@ -552,12 +555,12 @@ String HelpTextWindow::GetText() const
 // =======================================================================
 
 void ImplShowHelpWindow( Window* pParent, sal_uInt16 nHelpWinStyle, sal_uInt16 nStyle,
-                         const XubString& rHelpText, const XubString& rStatusText,
+                         const OUString& rHelpText, const OUString& rStatusText,
                          const Point& rScreenPos, const Rectangle* pHelpArea )
 {
     ImplSVData* pSVData = ImplGetSVData();
 
-    if( !rHelpText.Len() && !pSVData->maHelpData.mbRequestingHelp )
+    if (rHelpText.isEmpty() && !pSVData->maHelpData.mbRequestingHelp)
         return;
 
     HelpTextWindow* pHelpWin = pSVData->maHelpData.mpHelpWin;
@@ -602,7 +605,7 @@ void ImplShowHelpWindow( Window* pParent, sal_uInt16 nHelpWinStyle, sal_uInt16 n
         }
     }
 
-    if ( !pHelpWin && rHelpText.Len() )
+    if (!pHelpWin && !rHelpText.isEmpty())
     {
         sal_uLong nCurTime = Time::GetSystemTicks();
         if  (   ( ( nCurTime - pSVData->maHelpData.mnLastHelpHideTime ) < pParent->GetSettings().GetHelpSettings().GetTipDelay() )


More information about the Libreoffice-commits mailing list