summaryrefslogtreecommitdiff
path: root/sql/item_subselect.cc
diff options
context:
space:
mode:
authorSergey Petrunya <psergey@askmonty.org>2010-05-10 19:28:19 +0400
committerSergey Petrunya <psergey@askmonty.org>2010-05-10 19:28:19 +0400
commitb9688830ecb3ab528770aaa00a4b57c29859dc65 (patch)
treeeec72d95c63cc6f14b3adacaebe51a471bd2c6cb /sql/item_subselect.cc
parentb2c57cedbfd0f8ce2b4f67021a2aca5dedf9b6c9 (diff)
downloadmariadb-git-b9688830ecb3ab528770aaa00a4b57c29859dc65.tar.gz
Subquery optimizations:
- Better comments - Use more appropriate return types for functions - Provide handling where it was missing.
Diffstat (limited to 'sql/item_subselect.cc')
-rw-r--r--sql/item_subselect.cc7
1 files changed, 4 insertions, 3 deletions
diff --git a/sql/item_subselect.cc b/sql/item_subselect.cc
index 4f80f2a61f9..1e782120867 100644
--- a/sql/item_subselect.cc
+++ b/sql/item_subselect.cc
@@ -188,7 +188,7 @@ bool Item_subselect::fix_fields(THD *thd_param, Item **ref)
{
// all transformation is done (used by prepared statements)
changed= 1;
- inside_first_fix_fields= FALSE;
+ inside_first_fix_fields= FALSE;
// all transformation is done (used by prepared statements)
@@ -217,13 +217,13 @@ bool Item_subselect::fix_fields(THD *thd_param, Item **ref)
if (!(*ref)->fixed)
res= (*ref)->fix_fields(thd, ref);
goto end;
-//psergey-merge: done_first_fix_fields= FALSE;
+
}
// Is it one field subselect?
if (engine->cols() > max_columns)
{
my_error(ER_OPERAND_COLUMNS, MYF(0), 1);
-//psergey-merge: done_first_fix_fields= FALSE;
+
goto end;
}
fix_length_and_dec();
@@ -241,6 +241,7 @@ bool Item_subselect::fix_fields(THD *thd_param, Item **ref)
end:
done_first_fix_fields= FALSE;
+ inside_first_fix_fields= FALSE;
thd->where= save_where;
return res;
}