summaryrefslogtreecommitdiff
path: root/sql/create_options.cc
diff options
context:
space:
mode:
authorMarko Mäkelä <marko.makela@mariadb.com>2019-08-30 12:51:37 +0300
committerMarko Mäkelä <marko.makela@mariadb.com>2019-08-30 14:11:59 +0300
commitf42a23178edee5dd1fb6e0f76bbe88ea2a98a045 (patch)
tree471f5d552f9b285bf2ba5940854783ea3708bbdf /sql/create_options.cc
parenta379f151b18e4bc7e4c26be5f5e1254a59e27bfa (diff)
downloadmariadb-git-f42a23178edee5dd1fb6e0f76bbe88ea2a98a045.tar.gz
MDEV-20425: Fix -Wimplicit-fallthrough
With --skip-debug-assert, DBUG_ASSERT(false) will allow execution to continue. Hence, we will need /* fall through */ after them. Some DBUG_ASSERT(0) were replaced by break; when the switch () statement was followed by DBUG_ASSERT(0).
Diffstat (limited to 'sql/create_options.cc')
-rw-r--r--sql/create_options.cc5
1 files changed, 3 insertions, 2 deletions
diff --git a/sql/create_options.cc b/sql/create_options.cc
index 5adcb2f1e9e..a8d997efaf4 100644
--- a/sql/create_options.cc
+++ b/sql/create_options.cc
@@ -1,4 +1,4 @@
-/* Copyright (C) 2010, 2017, MariaDB Corporation Ab
+/* Copyright (C) 2010, 2019, MariaDB Corporation.
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
@@ -132,7 +132,8 @@ static bool set_one_value(ha_create_table_option *opt,
switch (opt->type)
{
case HA_OPTION_TYPE_SYSVAR:
- DBUG_ASSERT(0); // HA_OPTION_TYPE_SYSVAR's are replaced in resolve_sysvars()
+ // HA_OPTION_TYPE_SYSVAR's are replaced in resolve_sysvars()
+ break; // to DBUG_ASSERT(0)
case HA_OPTION_TYPE_ULL:
{
ulonglong *val= (ulonglong*)value_ptr(base, opt);