summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Widenius <monty@mariadb.org>2018-04-11 03:18:03 +0300
committerMichael Widenius <monty@mariadb.org>2018-04-16 20:16:43 +0300
commit3bae6a2b9886d185509dce0dacc489dda42152aa (patch)
tree6c02aa5520da92d77620278deec6727a4c940be2
parentddc5764303dd72f68d2c69cc574a6f42e8bcf86d (diff)
downloadmariadb-git-3bae6a2b9886d185509dce0dacc489dda42152aa.tar.gz
Remove not needed calls to print_error
-rw-r--r--sql/rpl_rli.cc6
-rw-r--r--sql/sql_acl.cc10
-rw-r--r--sql/sql_partition.cc5
3 files changed, 5 insertions, 16 deletions
diff --git a/sql/rpl_rli.cc b/sql/rpl_rli.cc
index c79a6440e84..6229f3e1faf 100644
--- a/sql/rpl_rli.cc
+++ b/sql/rpl_rli.cc
@@ -1537,11 +1537,9 @@ scan_one_gtid_slave_pos_table(THD *thd, HASH *hash, DYNAMIC_ARRAY *array,
goto end;
bitmap_set_all(table->read_set);
- if ((err= table->file->ha_rnd_init_with_error(1)))
- {
- table->file->print_error(err, MYF(0));
+ if (unlikely(err= table->file->ha_rnd_init_with_error(1)))
goto end;
- }
+
table_scanned= true;
for (;;)
{
diff --git a/sql/sql_acl.cc b/sql/sql_acl.cc
index ec195f82069..ffc109d7d26 100644
--- a/sql/sql_acl.cc
+++ b/sql/sql_acl.cc
@@ -9286,11 +9286,8 @@ static int handle_roles_mappings_table(TABLE *table, bool drop,
DBUG_PRINT("info", ("Rewriting entry in roles_mapping table: %s@%s",
user_from->user.str, user_from->host.str));
table->use_all_columns();
- if ((error= table->file->ha_rnd_init(1)))
- {
- table->file->print_error(error, MYF(0));
+ if (unlikely(error= table->file->ha_rnd_init_with_error(1)))
result= -1;
- }
else
{
while((error= table->file->ha_rnd_next(table->record[0])) !=
@@ -9451,11 +9448,8 @@ static int handle_grant_table(THD *thd, const Grant_table_base& grant_table,
And their host- and user fields are not consecutive.
Thus, we need to do a table scan to find all matching records.
*/
- if ((error= table->file->ha_rnd_init(1)))
- {
- table->file->print_error(error, MYF(0));
+ if (unlikely(error= table->file->ha_rnd_init_with_error(1)))
result= -1;
- }
else
{
#ifdef EXTRA_DEBUG
diff --git a/sql/sql_partition.cc b/sql/sql_partition.cc
index 09c59c862ad..94e90e1fc5e 100644
--- a/sql/sql_partition.cc
+++ b/sql/sql_partition.cc
@@ -4107,11 +4107,8 @@ bool verify_data_with_partition(TABLE *table, TABLE *part_table,
old_rec= part_table->record[0];
part_table->record[0]= table->record[0];
part_info->table->move_fields(part_info->full_part_field_array, table->record[0], old_rec);
- if ((error= file->ha_rnd_init(TRUE)))
- {
- file->print_error(error, MYF(0));
+ if (unlikely(error= file->ha_rnd_init_with_error(TRUE)))
goto err;
- }
do
{