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

[Rishabh Kumar]

 svx/source/tbxctrls/PaletteManager.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 74a017e27d04be13f072d03cab3a623a6b6d43ae
Author: Rishabh Kumar <kris.kr296 at gmail.com>
Date:   Sat Jul 30 09:22:45 2016 +0530

    Use static_cast for type casting and fix loop variant size
    
    Change-Id: Icc41693730ec8a68e3f3d68adff57fee02b7a689
    Reviewed-on: https://gerrit.libreoffice.org/27714
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Rishabh Kumar <kris.kr296 at yahoo.in>

diff --git a/svx/source/tbxctrls/PaletteManager.cxx b/svx/source/tbxctrls/PaletteManager.cxx
index e14c172..74a87f7 100644
--- a/svx/source/tbxctrls/PaletteManager.cxx
+++ b/svx/source/tbxctrls/PaletteManager.cxx
@@ -228,9 +228,9 @@ void PaletteManager::AddRecentColor(const Color& rRecentColor)
     if( maRecentColors.size() > mnMaxRecentColors )
         maRecentColors.pop_back();
     css::uno::Sequence< sal_Int32 > aColorList(maRecentColors.size());
-    for(sal_uInt16 i = 0;i < maRecentColors.size();i++)
+    for(sal_uInt32 i = 0;i < maRecentColors.size();i++)
     {
-        aColorList[i] = (int)maRecentColors[i].GetColor();
+        aColorList[i] = static_cast<sal_Int32>(maRecentColors[i].GetColor());
     }
     std::shared_ptr<comphelper::ConfigurationChanges> batch(comphelper::ConfigurationChanges::create(m_context));
     officecfg::Office::Common::UserColors::RecentColor::set(aColorList, batch);