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

Furkan Ahmet Kara (via logerrit) logerrit at kemper.freedesktop.org
Wed Mar 25 19:20:57 UTC 2020


 cui/source/dialogs/cuigaldlg.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 200fbb32c9d620e52c30f0016f11d595c1614185
Author:     Furkan Ahmet Kara <furkanahmetkara.fk at gmail.com>
AuthorDate: Tue Mar 24 00:07:29 2020 +0300
Commit:     Michael Stahl <michael.stahl at cib.de>
CommitDate: Wed Mar 25 20:20:22 2020 +0100

    tdf#114441: Convert use of sal_uLong to better integer types
    
    Change-Id: Ib4ad1a46a176b7043b1b87f430f442c13f3db8dc
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90941
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <michael.stahl at cib.de>

diff --git a/cui/source/dialogs/cuigaldlg.cxx b/cui/source/dialogs/cuigaldlg.cxx
index fcf0d4cb79ac..919fae2f3e5f 100644
--- a/cui/source/dialogs/cuigaldlg.cxx
+++ b/cui/source/dialogs/cuigaldlg.cxx
@@ -266,7 +266,7 @@ void TakeThread::execute()
         const sal_Int32 nPos = mpBrowser->bTakeAll ? i : aSelectedRows[i];
         const INetURLObject aURL( mpBrowser->aFoundList[ nPos ]);
 
-        mrTakenList.push_back( static_cast<sal_uLong>(nPos) );
+        mrTakenList.push_back( nPos );
 
         {
             SolarMutexGuard aGuard;
@@ -451,10 +451,10 @@ GalleryIdDialog::~GalleryIdDialog()
 IMPL_LINK_NOARG(GalleryIdDialog, ClickOkHdl, weld::Button&, void)
 {
     Gallery*    pGal = m_pThm->GetParent();
-    const sal_uLong nId = GetId();
+    const sal_uInt32 nId = GetId();
     bool        bDifferentThemeExists = false;
 
-    for( sal_uLong i = 0, nCount = pGal->GetThemeCount(); i < nCount && !bDifferentThemeExists; i++ )
+    for( size_t i = 0, nCount = pGal->GetThemeCount(); i < nCount && !bDifferentThemeExists; i++ )
     {
         const GalleryThemeEntry* pInfo = pGal->GetThemeInfo( i );
 


More information about the Libreoffice-commits mailing list