diff options
author | Sergey Petrunya <psergey@askmonty.org> | 2010-05-10 19:28:19 +0400 |
---|---|---|
committer | Sergey Petrunya <psergey@askmonty.org> | 2010-05-10 19:28:19 +0400 |
commit | b9688830ecb3ab528770aaa00a4b57c29859dc65 (patch) | |
tree | eec72d95c63cc6f14b3adacaebe51a471bd2c6cb /sql/item_subselect.cc | |
parent | b2c57cedbfd0f8ce2b4f67021a2aca5dedf9b6c9 (diff) | |
download | mariadb-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.cc | 7 |
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; } |