[Libreoffice-commits] core.git: sfx2/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Dec 17 13:43:16 UTC 2018


 sfx2/source/appl/sfxhelp.cxx |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 0574f8241acbe07c87a014fb0c8f12d2f3a8b785
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Mon Dec 17 11:41:27 2018 +0100
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Mon Dec 17 14:42:57 2018 +0100

    Remove unnecessary OUString aInternal
    
    Change-Id: I37112fdb971a60012a4cb4f1aebfbb79dda3f92e
    Reviewed-on: https://gerrit.libreoffice.org/65258
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/sfx2/source/appl/sfxhelp.cxx b/sfx2/source/appl/sfxhelp.cxx
index 4a8929af5a31..7a3eb65a1867 100644
--- a/sfx2/source/appl/sfxhelp.cxx
+++ b/sfx2/source/appl/sfxhelp.cxx
@@ -721,10 +721,9 @@ static bool impl_showOnlineHelp( const OUString& rURL )
 static bool impl_showOfflineHelp( const OUString& rURL )
 {
     const OUString& aBaseInstallPath = getHelpRootURL();
-    OUString const aInternal( "vnd.sun.star.help://"  );
 
     OUString aHelpLink( aBaseInstallPath + "/index.html?" );
-    OUString aTarget = "Target=" + rURL.copy(aInternal.getLength());
+    OUString aTarget = "Target=" + rURL.copy(RTL_CONSTASCII_LENGTH("vnd.sun.star.help://"));
     aTarget = aTarget.replaceAll("%2F","/").replaceAll("?","&");
     aHelpLink += aTarget;
 


More information about the Libreoffice-commits mailing list