[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