swoole_table warning free.

Author: matyhtf

examples/table/set.php

@@ -21,10 +21,16 @@ function child1($worker)
 //master
 sleep(1);
 $s = microtime(true);
-for($i =0; $i < 1000; $i++)
-{
-    $arr = $table->get('[email protected]');
-}
+//for($i =0; $i < 1000; $i++)
+//{
+//    $arr = $table->get('[email protected]');
+//	var_dump($arr);
+//}
+
+var_dump($table->get('[email protected]'));
+var_dump($table->get('[email protected]'));
+var_dump($table->get('[email protected]'));
+
 echo "use: ".((microtime(true) - $s) * 1000)."ms\n";
 //var_dump($table->get('[email protected]'));
 sleep(100000);

include/table.h

@@ -142,16 +142,16 @@ static sw_inline void swTableRow_set_value(swTableRow *row, swTableColumn * col,
     switch(col->type)
     {
     case SW_TABLE_INT8:
-        *(int8_t *)(row->data + col->index) = *(int8_t*) value;
+        memcpy(row->data + col->index, value, 1);
         break;
     case SW_TABLE_INT16:
-        *(int16_t *)(row->data + col->index) = *(int16_t*) value;
+        memcpy(row->data + col->index, value, 2);
         break;
     case SW_TABLE_INT32:
-        *(int32_t *)(row->data + col->index) = *(int32_t*) value;
+        memcpy(row->data + col->index, value, 4);
         break;
     case SW_TABLE_INT64:
-        *(int64_t *)(row->data + col->index) = *(int64_t*) value;
+        memcpy(row->data + col->index, value, 8);
         break;
     case SW_TABLE_FLOAT:
         memcpy(row->data + col->index, value, sizeof(double));
@@ -162,7 +162,7 @@ static sw_inline void swTableRow_set_value(swTableRow *row, swTableColumn * col,
             swWarn("string is too long.");
             vlen = col->size - sizeof(swTable_string_length_t);
         }
-        *(swTable_string_length_t *)(row->data + col->index) = vlen;
+        memcpy(row->data + col->index, &vlen, sizeof(swTable_string_length_t));
         memcpy(row->data + col->index + sizeof(swTable_string_length_t), value, vlen);
         break;
     }

swoole_table.c

@@ -87,6 +87,9 @@ static void php_swoole_table_row2array(swTable *table, swTableRow *row, zval *re
     array_init(return_value);
 
     swTableColumn *col = NULL;
+    swTable_string_length_t vlen = 0;
+    double dval = 0;
+    int64_t lval = 0;
     char *k;
 
     sw_spinlock(&row->lock);
@@ -99,30 +102,29 @@ static void php_swoole_table_row2array(swTable *table, swTableRow *row, zval *re
         }
         if (col->type == SW_TABLE_STRING)
         {
-            swTable_string_length_t vlen = *(swTable_string_length_t *) (row->data + col->index);
+            memcpy(&vlen, row->data + col->index, sizeof(swTable_string_length_t));
             add_assoc_stringl_ex(return_value, col->name->str, col->name->length + 1, row->data + col->index + sizeof(swTable_string_length_t), vlen, 1);
         }
         else if (col->type == SW_TABLE_FLOAT)
         {
-            double dval = *(double *) (row->data + col->index);
+            memcpy(&dval, row->data + col->index, sizeof(dval));
             add_assoc_double_ex(return_value, col->name->str, col->name->length + 1, dval);
         }
         else
         {
-            int64_t lval;
             switch (col->type)
             {
             case SW_TABLE_INT8:
-                lval = *(int8_t *) (row->data + col->index);
+                memcpy(&lval, row->data + col->index, 1);
                 break;
             case SW_TABLE_INT16:
-                lval = *(int16_t *) (row->data + col->index);
+                memcpy(&lval, row->data + col->index, 2);
                 break;
             case SW_TABLE_INT32:
-                lval = *(int32_t *) (row->data + col->index);
+                memcpy(&lval, row->data + col->index, 4);
                 break;
             default:
-                lval = *(int64_t *) (row->data + col->index);
+                memcpy(&lval, row->data + col->index, 8);
                 break;
             }
             add_assoc_long_ex(return_value, col->name->str, col->name->length + 1, lval);