Skip to content

Commit 5cbbfe9

Browse files
committed
cleanup: remove duplicate code
1 parent 918e470 commit 5cbbfe9

File tree

2 files changed

+15
-24
lines changed

2 files changed

+15
-24
lines changed

sql/sql_partition.cc

Lines changed: 14 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1913,6 +1913,19 @@ static int add_subpartition_by(File fptr)
19131913
return err + add_partition_by(fptr);
19141914
}
19151915

1916+
static int add_name_string(File fptr, const char *name)
1917+
{
1918+
int err;
1919+
String name_string("", 0, system_charset_info);
1920+
THD *thd= current_thd;
1921+
ulonglong save_options= thd->variables.option_bits;
1922+
thd->variables.option_bits&= ~OPTION_QUOTE_SHOW_CREATE;
1923+
append_identifier(thd, &name_string, name, strlen(name));
1924+
thd->variables.option_bits= save_options;
1925+
err= add_string_object(fptr, &name_string);
1926+
return err;
1927+
}
1928+
19161929
static int add_part_field_list(File fptr, List<char> field_list)
19171930
{
19181931
uint i, num_fields;
@@ -1924,15 +1937,7 @@ static int add_part_field_list(File fptr, List<char> field_list)
19241937
err+= add_begin_parenthesis(fptr);
19251938
while (i < num_fields)
19261939
{
1927-
const char *field_str= part_it++;
1928-
String field_string("", 0, system_charset_info);
1929-
THD *thd= current_thd;
1930-
ulonglong save_options= thd->variables.option_bits;
1931-
thd->variables.option_bits&= ~OPTION_QUOTE_SHOW_CREATE;
1932-
append_identifier(thd, &field_string, field_str,
1933-
strlen(field_str));
1934-
thd->variables.option_bits= save_options;
1935-
err+= add_string_object(fptr, &field_string);
1940+
err+= add_name_string(fptr, part_it++);
19361941
if (i != (num_fields-1))
19371942
err+= add_comma(fptr);
19381943
i++;
@@ -1941,20 +1946,6 @@ static int add_part_field_list(File fptr, List<char> field_list)
19411946
return err;
19421947
}
19431948

1944-
static int add_name_string(File fptr, const char *name)
1945-
{
1946-
int err;
1947-
String name_string("", 0, system_charset_info);
1948-
THD *thd= current_thd;
1949-
ulonglong save_options= thd->variables.option_bits;
1950-
thd->variables.option_bits&= ~OPTION_QUOTE_SHOW_CREATE;
1951-
append_identifier(thd, &name_string, name,
1952-
strlen(name));
1953-
thd->variables.option_bits= save_options;
1954-
err= add_string_object(fptr, &name_string);
1955-
return err;
1956-
}
1957-
19581949
static int add_int(File fptr, longlong number)
19591950
{
19601951
char buff[32];

sql/sql_priv.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@
121121
#define OPTION_AUTOCOMMIT (1ULL << 8) // THD, user
122122
#define OPTION_BIG_SELECTS (1ULL << 9) // THD, user
123123
#define OPTION_LOG_OFF (1ULL << 10) // THD, user
124-
#define OPTION_QUOTE_SHOW_CREATE (1ULL << 11) // THD, user, unused
124+
#define OPTION_QUOTE_SHOW_CREATE (1ULL << 11) // THD, user
125125
#define TMP_TABLE_ALL_COLUMNS (1ULL << 12) // SELECT, intern
126126
#define OPTION_WARNINGS (1ULL << 13) // THD, user
127127
#define OPTION_AUTO_IS_NULL (1ULL << 14) // THD, user, binlog

0 commit comments

Comments
 (0)