[PATCH] Change j type to size_t

Anthony Youngman (via Code Review) gerrit at gerrit.libreoffice.org
Fri Apr 26 01:20:54 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3616

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/16/3616/1

Change j type to size_t

Change-Id: If04e33228162baab392906903a574f1a6e7a6110
---
M connectivity/source/drivers/postgresql/pq_connection.cxx
1 file changed, 1 insertion(+), 3 deletions(-)



diff --git a/connectivity/source/drivers/postgresql/pq_connection.cxx b/connectivity/source/drivers/postgresql/pq_connection.cxx
index f0d9e03..4f830a1 100644
--- a/connectivity/source/drivers/postgresql/pq_connection.cxx
+++ b/connectivity/source/drivers/postgresql/pq_connection.cxx
@@ -496,8 +496,7 @@
     for( int i = 0; i < args.getLength() ; ++i )
     {
         bool append = false;
-//        for( int j = 0; j < (int) ( sizeof( keyword_list ) / sizeof( char * )); j++)
-        for( int j = 0; j < (int) SAL_N_ELEMENTS( keyword_list ); j++)
+        for( size_t j = 0; j < SAL_N_ELEMENTS( keyword_list ); j++)
         {
             if( args[i].Name.equalsIgnoreAsciiCaseAscii( keyword_list[j] ))
             {
@@ -505,7 +504,6 @@
                 append = true;
                 break;
             }
-            append = false;
         }
 
         if( append )

-- 
To view, visit https://gerrit.libreoffice.org/3616
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If04e33228162baab392906903a574f1a6e7a6110
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Anthony Youngman <anthony at youngman.org.uk>



More information about the LibreOffice mailing list