#At file:///home/kgeorge/mysql/bzr/B33812-5.1.26-release/
2659 Georgi Kodinov 2008-06-20
Bug#33812: mysql client incorrectly parsing DELIMITER
Remove unnecessary and incorrect code that tried to pull delimiter commands out of the
middle of statements
modified:
client/mysql.cc
mysql-test/r/mysql.result
mysql-test/t/mysql_delimiter.sql
per-file messages:
client/mysql.cc
Bug#33812: remove incorrect code
mysql-test/r/mysql.result
Bug#33812: test case
mysql-test/t/mysql_delimiter.sql
Bug#33812: test case
=== modified file 'client/mysql.cc'
--- a/client/mysql.cc 2008-06-19 03:12:28 +0000
+++ b/client/mysql.cc 2008-06-20 15:08:24 +0000
@@ -2101,37 +2101,6 @@ static bool add_line(String &buffer,char
continue;
}
}
- else if (!*ml_comment && !*in_string &&
- (end_of_line - pos) >= 10 &&
- !my_strnncoll(charset_info, (uchar*) pos, 10,
- (const uchar*) "delimiter ", 10))
- {
- // Flush previously accepted characters
- if (out != line)
- {
- buffer.append(line, (uint32) (out - line));
- out= line;
- }
-
- // Flush possible comments in the buffer
- if (!buffer.is_empty())
- {
- if (com_go(&buffer, 0) > 0) // < 0 is not fatal
- DBUG_RETURN(1);
- buffer.length(0);
- }
-
- /*
- Delimiter wants the get rest of the given line as argument to
- allow one to change ';' to ';;' and back
- */
- buffer.append(pos);
- if (com_delimiter(&buffer, pos) > 0)
- DBUG_RETURN(1);
-
- buffer.length(0);
- break;
- }
else if (!*ml_comment && !*in_string && is_prefix(pos, delimiter))
{
// Found a statement. Continue parsing after the delimiter
=== modified file 'mysql-test/r/mysql.result'
--- a/mysql-test/r/mysql.result 2007-11-26 18:50:43 +0000
+++ b/mysql-test/r/mysql.result 2008-06-20 15:08:24 +0000
@@ -38,6 +38,8 @@ t2
t3
Tables_in_test
t1
+delimiter
+1
_
Test delimiter : from command line
a
=== modified file 'mysql-test/t/mysql_delimiter.sql'
--- a/mysql-test/t/mysql_delimiter.sql 2006-11-22 06:27:06 +0000
+++ b/mysql-test/t/mysql_delimiter.sql 2008-06-20 15:08:24 +0000
@@ -59,3 +59,9 @@ source t/mysql_delimiter_19799.sql
use test//
show tables//
delimiter ; # Reset delimiter
+
+#
+# Bug #33812: mysql client incorrectly parsing DELIMITER
+#
+select a as delimiter from t1
+delimiter ; # Reset delimiter
| Thread |
|---|
| • bzr commit into mysql-5.1 branch (kgeorge:2659) Bug#33812 | Georgi Kodinov | 20 Jun |