[Libreoffice-commits] .: framework/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Sat Oct 6 15:14:43 PDT 2012


 framework/source/uielement/langselectionmenucontroller.cxx |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit 8113588c35ae494ccdcbb864a7f0f9e08697118a
Author: jailletc36 <christophe.jaillet at wanadoo.fr>
Date:   Sat Oct 6 22:23:29 2012 +0200

    cppcheck: unreadVariable.
    
    aLangMap is a std::map that is written but never used, so drop it.
    
    Change-Id: I8f5184c58fbe812beab8580349535920645e062e
    Reviewed-on: https://gerrit.libreoffice.org/781
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Tested-by: Markus Mohrhard <markus.mohrhard at googlemail.com>

diff --git a/framework/source/uielement/langselectionmenucontroller.cxx b/framework/source/uielement/langselectionmenucontroller.cxx
index 1dc52ed..44605f0 100644
--- a/framework/source/uielement/langselectionmenucontroller.cxx
+++ b/framework/source/uielement/langselectionmenucontroller.cxx
@@ -251,7 +251,6 @@ void LanguageSelectionMenuController::fillPopupMenu( Reference< css::awt::XPopup
     sal_Int16 nItemId = 1;  // in this control the item id is not important for executing the command
     const OUString sAsterix("*");  // multiple languages in current selection
     const OUString sEmpty;  // 'no language found' from language guessing
-    std::map< sal_Int16, OUString > aLangMap;
     std::set< OUString >::const_iterator it;
     for (it = aLangItems.begin(); it != aLangItems.end(); ++it)
     {
@@ -269,7 +268,6 @@ void LanguageSelectionMenuController::fillPopupMenu( Reference< css::awt::XPopup
                 //make a sign for the current language
                 pPopupMenu->CheckItem( nItemId, sal_True );
             }
-            aLangMap[ nItemId ] = rStr;
             ++nItemId;
         }
     }


More information about the Libreoffice-commits mailing list