From 787fa9325f353cc5d8d0db4f96a07502758f2c0c Mon Sep 17 00:00:00 2001
From: Davi Arnaut <Davi.Arnaut@Sun.COM>
Date: Thu, 2 Oct 2008 16:13:42 -0300
Subject: [PATCH] Remove usage of the _db_strict_keyword_ in favor of the more
 reliable DBUG_EXECUTE_IF macro. The usage of the former caused breakage in
 other trees as it got removed from the dbug library.

---
 sql/sql_base.cc | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/sql/sql_base.cc b/sql/sql_base.cc
index 49efc72ba68..ad9af040dbb 100644
--- a/sql/sql_base.cc
+++ b/sql/sql_base.cc
@@ -3720,6 +3720,18 @@ void assign_new_table_id(TABLE_SHARE *share)
   DBUG_VOID_RETURN;
 }
 
+/* Cause a spurious statement reprepare for debug purposes. */
+static inline bool inject_reprepare(THD *thd)
+{
+  if (thd->m_reprepare_observer && thd->stmt_arena->is_reprepared == FALSE)
+  {
+    thd->m_reprepare_observer->report_error(thd);
+    return TRUE;
+  }
+
+  return FALSE;
+}
+
 /**
   Compare metadata versions of an element obtained from the table
   definition cache and its corresponding node in the parse tree.
@@ -3773,15 +3785,7 @@ check_and_update_table_version(THD *thd,
     tables->set_table_ref_id(table_share);
   }
 
-#ifndef DBUG_OFF
-  /* Spuriously reprepare each statement. */
-  if (_db_strict_keyword_("reprepare_each_statement") &&
-      thd->m_reprepare_observer && thd->stmt_arena->is_reprepared == FALSE)
-  {
-    thd->m_reprepare_observer->report_error(thd);
-    return TRUE;
-  }
-#endif
+  DBUG_EXECUTE_IF("reprepare_each_statement", return inject_reprepare(thd););
 
   return FALSE;
 }