Minor post-commit fixes (WL#7199):

GlebShchepa · GlebShchepa · commit a849f59cea21 · 2014-03-28T14:47:09.000+04:00
* sql/item_func.cc: initialize Item_func::args with tmp_args if arg_count == 0 to make valgrind happy when it traces "for(... = args + arg_count...)" in Item::walk();
* sql/parse_tree_nodes.h: remove the trailing comma in the emum definition to make some C++ compilers happy.
diff --git a/sql/item_func.cc b/sql/item_func.cc
@@ -118,7 +118,10 @@ Item_func::Item_func(const POS &pos, PT_item_list *opt_list)
   : super(pos), allowed_arg_cols(1)
 {
   if (opt_list == NULL)
+  {
+    args= tmp_arg;
     arg_count= 0;
+  }
   else
     set_arguments(opt_list->value, true);
 }
diff --git a/sql/parse_tree_nodes.h b/sql/parse_tree_nodes.h
@@ -273,7 +273,7 @@ enum PT_join_table_type
   JTT_RIGHT             = 0x10,
 
   JTT_NATURAL_LEFT      = JTT_NATURAL | JTT_LEFT,
-  JTT_NATURAL_RIGHT     = JTT_NATURAL | JTT_RIGHT,
+  JTT_NATURAL_RIGHT     = JTT_NATURAL | JTT_RIGHT
 };
 
 

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,10 @@ Item_func::Item_func(const POS &pos, PT_item_list *opt_list)`
`118`	`118`	`: super(pos), allowed_arg_cols(1)`
`119`	`119`	`{`
`120`	`120`	`if (opt_list == NULL)`
	`121`	`+ {`
	`122`	`+ args= tmp_arg;`
`121`	`123`	`arg_count= 0;`
	`124`	`+ }`
`122`	`125`	`else`
`123`	`126`	`set_arguments(opt_list->value, true);`
`124`	`127`	`}`