summaryrefslogtreecommitdiff
path: root/sql/item_cmpfunc.cc
diff options
context:
space:
mode:
authorunknown <monty@mysql.com>2003-11-20 22:07:02 +0200
committerunknown <monty@mysql.com>2003-11-20 22:07:02 +0200
commit671cc0a57491e8084b06a7aa6153f0d05c8ae6ef (patch)
tree84c181f583221c9570fd4312ba281b4e58555248 /sql/item_cmpfunc.cc
parentbaae790c963d695613c290e8f9f8ce5ca0853b33 (diff)
parent35da5e43fbd3653c39358669365e2c329e1e555c (diff)
downloadmariadb-git-671cc0a57491e8084b06a7aa6153f0d05c8ae6ef.tar.gz
Merge bk-internal.mysql.com:/home/bk/mysql-4.1
into mysql.com:/my/mysql-4.1 sql/item_cmpfunc.cc: Auto merged
Diffstat (limited to 'sql/item_cmpfunc.cc')
-rw-r--r--sql/item_cmpfunc.cc13
1 files changed, 7 insertions, 6 deletions
diff --git a/sql/item_cmpfunc.cc b/sql/item_cmpfunc.cc
index 8f3a1b78cd3..db80428290a 100644
--- a/sql/item_cmpfunc.cc
+++ b/sql/item_cmpfunc.cc
@@ -397,13 +397,12 @@ int Arg_comparator::compare_row()
int Arg_comparator::compare_e_row()
{
- int res= 0;
(*a)->bring_value();
(*b)->bring_value();
uint n= (*a)->cols();
for (uint i= 0; i<n; i++)
{
- if ((res= !comparators[i].compare()))
+ if (!comparators[i].compare())
return 0;
}
return 1;
@@ -1096,8 +1095,10 @@ void Item_func_case::fix_length_and_dec()
return;
- // Aggregate first expression and all THEN expression types
- // and collations when string comparison
+ /*
+ Aggregate first expression and all THEN expression types
+ and collations when string comparison
+ */
if (first_expr_num != -1)
{
agg[0]= args[first_expr_num];
@@ -1110,7 +1111,7 @@ void Item_func_case::fix_length_and_dec()
return;
}
- if (!else_expr_num != -1 || args[else_expr_num]->maybe_null)
+ if (else_expr_num == -1 || args[else_expr_num]->maybe_null)
maybe_null=1;
max_length=0;
@@ -1127,6 +1128,7 @@ void Item_func_case::fix_length_and_dec()
}
}
+
/* TODO: Fix this so that it prints the whole CASE expression */
void Item_func_case::print(String *str)
@@ -1538,7 +1540,6 @@ void Item_func_in::fix_length_and_dec()
{
switch (cmp_type) {
case STRING_RESULT:
- uint i;
array=new in_string(arg_count-1,(qsort2_cmp) srtcmp_in,
cmp_collation.collation);
break;