Changed to Reset(bool clear_bound_vars)

sql/statement.h

Index: sql/statement.h
diff --git a/sql/statement.h b/sql/statement.h
index 92d1ef2fb472c44f15434ac60772b1561d6ab009..501b8b3735f1f5f83f62e838cc0c88647242f097 100644
--- a/sql/statement.h
+++ b/sql/statement.h
@@ -91,6 +91,10 @@ class SQL_EXPORT Statement {
   // the bound variables and any current result row.
   void Reset();
 
+  // Resets the statement to its initial condition without clearing the bound
+  // variables.
+  void ResetWithoutClearingBoundVariables();

[brettw, 2012/04/24 23:12:30]

This name seems awkward and out of place. It looks like Reset is actually used
very rarely. How about just adding a param to that?
  void Reset(bool clear_bound_vars);

[michaelbai, 2012/04/25 17:10:52]

Done.

[Scott Hess - ex-Googler, 2012/07/03 00:34:34]

In my imagination, ResetWithoutClearingBoundVariables() would have been used
just about once, and then only in a case where someone nominally knew what they
were doing (not true of most Reset() callers).  So right now I'm having a
two-minute hate for this brettw person.

+
   // Returns true if the last executed thing in this statement succeeded. If
   // there was no last executed thing or the statement is invalid, this will
   // return false.