]> granicus.if.org Git - php/commitdiff
Fixed signed/unsigned comparison warnings
authorDmitry Stogov <dmitry@zend.com>
Mon, 18 Feb 2013 09:03:29 +0000 (13:03 +0400)
committerDmitry Stogov <dmitry@zend.com>
Mon, 18 Feb 2013 09:03:29 +0000 (13:03 +0400)
Optimizer/block_pass.c
Optimizer/nop_removal.c
Optimizer/zend_optimizer_internal.h

index aeb8f18707ceb2adf01c19bfe6318fb842299ec3..966cf26fbc4c93a66866c3965d502405859f0343 100644 (file)
@@ -1182,7 +1182,7 @@ static void assemble_code_blocks(zend_code_block *blocks, zend_op_array *op_arra
 
 #if ZEND_EXTENSION_API_NO >= PHP_5_3_X_API_NO
        /* adjust early binding list */
-       if (op_array->early_binding != -1) {
+       if (op_array->early_binding != (zend_uint)-1) {
                zend_uint *opline_num = &op_array->early_binding;
                zend_op *end;
 
index 49ffe0e6e50fefae89742deea902e86cebee89e4..6c22bcf6fd5a4c4207c12cf90b2530075acd4de6 100644 (file)
@@ -92,10 +92,10 @@ static void nop_removal(zend_op_array *op_array)
                }
 
                /* update brk/cont array */
-               for (i=0; i<op_array->last_brk_cont; i++) {
-                       op_array->brk_cont_array[i].brk -= shiftlist[op_array->brk_cont_array[i].brk];
-                       op_array->brk_cont_array[i].cont -= shiftlist[op_array->brk_cont_array[i].cont];
-                       op_array->brk_cont_array[i].start -= shiftlist[op_array->brk_cont_array[i].start];
+               for (j=0; j<op_array->last_brk_cont; j++) {
+                       op_array->brk_cont_array[j].brk -= shiftlist[op_array->brk_cont_array[j].brk];
+                       op_array->brk_cont_array[j].cont -= shiftlist[op_array->brk_cont_array[j].cont];
+                       op_array->brk_cont_array[j].start -= shiftlist[op_array->brk_cont_array[j].start];
                }
 
                /* update try/catch array */
@@ -112,13 +112,13 @@ static void nop_removal(zend_op_array *op_array)
 
 #if ZEND_EXTENSION_API_NO >= PHP_5_3_X_API_NO
                /* update early binding list */
-               if (op_array->early_binding != -1) {
+               if (op_array->early_binding != (zend_uint)-1) {
                        zend_uint *opline_num = &op_array->early_binding;
 
                        do {
                                *opline_num -= shiftlist[*opline_num];
                                opline_num = &ZEND_RESULT(&op_array->opcodes[*opline_num]).opline_num;
-                       } while (*opline_num != -1);
+                       } while (*opline_num != (zend_uint)-1);
                }
 #endif
        }
index 113cbe98267813f7f675aef12e78e8d8a7f120aa..198ec20c4998c663e218b22336e5e4c2991f18a9 100644 (file)
@@ -25,7 +25,7 @@
 #include "ZendAccelerator.h"\r
 \r
 #if ZEND_EXTENSION_API_NO > PHP_5_4_X_API_NO\r
-# define VAR_NUM(v) (EX_TMP_VAR_NUM(0, 0) - EX_TMP_VAR(0, v))\r
+# define VAR_NUM(v) ((zend_uint)(EX_TMP_VAR_NUM(0, 0) - EX_TMP_VAR(0, v)))\r
 # define NUM_VAR(v) ((zend_uint)EX_TMP_VAR_NUM(0, v))\r
 #else\r
 # define VAR_NUM(v) ((v)/(sizeof(temp_variable)))\r