post-post-review fix (SCRUM)

sql/item.cc:
  post-post-review fix
sql/item_cmpfunc.h:
  post-post-review fix
sql/item_row.cc:
  post-post-review fix
sql/item_subselect.cc:
  post-post-review fix

sql/item.cc

@@ -1290,9 +1290,10 @@ bool Item_cache_row::setup(Item * item)
   for (uint i= 0; i < item_count; i++)
   {
     Item *el= item->el(i);
-    if (!(values[i]= Item_cache::get_cache(el->result_type())))
+    Item_cache *tmp;
+    if (!(tmp= values[i]= Item_cache::get_cache(el->result_type())))
       return 1;
-    values[i]->setup(el);
+    tmp->setup(el);
   }
   return 0;
 }

sql/item_cmpfunc.h

@@ -552,12 +552,14 @@ class cmp_item_row :public cmp_item
   ~cmp_item_row()
   {
     if (comparators)
+    {
       for (uint i= 0; i < n; i++)
       {
 	if (comparators[i])
 	  delete comparators[i];
       }
     }
+  }
   void store_value(Item *item);
   int cmp(Item *arg);
   int compare(cmp_item *arg);

sql/item_row.cc

@@ -55,6 +55,7 @@ bool Item_row::fix_fields(THD *thd, TABLE_LIST *tabl, Item **ref)
       return 1;
     used_tables_cache |= items[i]->used_tables();
     if (const_item_cache&= items[i]->const_item() && !with_null)
+    {
       if (items[i]->cols() > 1)
 	with_null|= items[i]->null_inside();
       else
@@ -62,6 +63,7 @@ bool Item_row::fix_fields(THD *thd, TABLE_LIST *tabl, Item **ref)
 	items[i]->val_int();
 	with_null|= items[i]->null_value;
       }
+    }
     maybe_null|= items[i]->maybe_null;
   }
   return 0;

sql/item_subselect.cc

@@ -604,7 +604,7 @@ void Item_in_subselect::row_value_transformer(THD *thd,
     }
 
     if (sl->having || sl->with_sum_func || sl->group_list.first ||
-	!sl->table_list.elements || !sl->table_list.elements)
+	!sl->table_list.elements)
       sl->having= and_items(sl->having, item);
     else
       sl->where= and_items(sl->where, item);