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

Stephan Bergmann sbergman at redhat.com
Tue Feb 4 05:45:13 PST 2014


 cui/source/customize/acccfg.cxx |   10 ----------
 1 file changed, 10 deletions(-)

New commits:
commit e0e82c6d12938d3eb4ff63589980a781365640c7
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Feb 4 14:36:00 2014 +0100

    Handling ".uno:InsertSymbol?Symbols:string=" appears to be dead code
    
    4d3148e0b7ee607de9c86e9e9aa79e9cd423f1f1 "CWS-TOOLING: integrate CWS
    mba32issues01" had introduced it, along with code in
    SfxConfigGroupListBox_Impl::AddAndSelect to generate such URLs, but which had
    been commented out from the beginning (and since been removed), and all other
    places that mention .uno:InsertSymbol apparently use it without a query part (to
    run the special character dialog), and the string has 33 characters, not 34, so
    aRet would not contain the first character after Symbols:string=, which looks
    odd.
    
    Change-Id: Icdab24148b217531f2db65b1604678650180a22c

diff --git a/cui/source/customize/acccfg.cxx b/cui/source/customize/acccfg.cxx
index d555932..df3c474 100644
--- a/cui/source/customize/acccfg.cxx
+++ b/cui/source/customize/acccfg.cxx
@@ -1513,16 +1513,6 @@ OUString SfxAcceleratorConfigPage::GetLabel4Command(const OUString& sCommand)
         m_aStylesInfo.getLabel4Style(aStyle);
         return aStyle.sLabel;
     }
-    else
-    {
-        OUString aRet("Symbols: ");
-        sal_Int32 nPos = sCommand.indexOf(".uno:InsertSymbol?Symbols:string=");
-        if ( nPos == 0 )
-        {
-            aRet += sCommand.copy(34, sCommand.getLength()-34);
-            return aRet;
-        }
-    }
 
     return sCommand;
 }


More information about the Libreoffice-commits mailing list