Code cleanups during code reviews

Ensure we get error if INSERT IGNORE ... SELECT fails
Fixed wrong key_part->key_length usage in index_merge


client/mysql.cc:
  Code cleanups & simply optimizations
mysql-test/r/information_schema.result:
  Safety
mysql-test/t/information_schema.test:
  Safety
sql/ha_ndbcluster.cc:
  Code cleanups
sql/item.cc:
  Code cleanups
sql/item_subselect.cc:
  Code cleanups
sql/item_sum.cc:
  Code cleanups
sql/opt_range.cc:
  Made get_index_only_read_time() static (instad of inline) to increase portability (function was not declared before use)
  Simple optimization
  Fixed wrong key_part->key_length usage in index_merge
  Removed not used variable n_used_covered
  Indentation fixes & comment cleanups
sql/parse_file.cc:
  Code cleanups
sql/sql_base.cc:
  Code cleanups
sql/sql_bitmap.h:
  Added missing return
sql/sql_insert.cc:
  Ensure we get error if INSERT IGNORE ... SELECT fails
sql/sql_select.cc:
  Code cleanups
sql/sql_show.cc:
  Safety fix if a LOT of errors are ignored
sql/sql_update.cc:
  Code cleanups
sql/table.cc:
  Code cleanups
sql/table.h:
  Code cleanups
sql/uniques.cc:
  Code cleanups
strings/decimal.c:
  Simple optimization
  Code cleanups

1 files changed, 2 insertions, 2 deletions

diff --git a/sql/parse_file.cc b/sql/parse_file.cc
index 6c3a81384a6..7cc563901d2 100644
--- a/sql/parse_file.cc
+++ b/sql/parse_file.cc
@@ -475,7 +475,7 @@ read_escaped_string(char *ptr, char *eol, LEX_STRING *str)
 {
   char *write_pos= str->str;
 
-  for(; ptr < eol; ptr++, write_pos++)
+  for (; ptr < eol; ptr++, write_pos++)
   {
     char c= *ptr;
     if (c == '\\')
@@ -635,7 +635,7 @@ File_parser::parse(gptr base, MEM_ROOT *mem_root,
       File_option *parameter= parameters+first_param,
 	*parameters_end= parameters+required;
       int len= 0;
-      for(; parameter < parameters_end; parameter++)
+      for (; parameter < parameters_end; parameter++)
       {
 	len= parameter->name.length;
 	// check length