Clear bindings before the statement is put in the cache.

README.md

@@ -30,6 +30,7 @@ $ cp ~/Downloads/sqlite-amalgamation-xxx/sqlite3.{c,h} $GOPATH/src/github.com/gw
 ### Features (not supported by database/sql/driver):
 
 * Dynamic type: currently, the SQLite3 manifest typing is respected. There is no use of the column declared type to guess the target/go type when scanning. On your side, you should try to not break column affinity rules (such as declaring a column with TIMESTAMP type (NUMERIC affinity) storing values with '2006-01-02T15:04:05.999Z07:00' format (TEXT type))...
+* Named bind parameters.
 * Partial scan: scan values may be partially scanned (by index or name) or skipped/ignored by passing nil pointer(s).
 * Null value: by default, empty string and zero time are bound to null for prepared statement's parameters (no need for NullString, NullTime but still supported).
 

cache.go

@@ -41,10 +41,6 @@ func (c *cache) find(sql string) *Stmt {
 		s := e.Value.(*Stmt)
 		if s.SQL() == sql { // TODO s.SQL() may have been trimmed by SQLite
 			c.l.Remove(e)
-			if err := s.ClearBindings(); err != nil {
-				s.finalize()
-				return nil
-			}
 			return s
 		}
 	}
@@ -60,6 +56,10 @@ func (c *cache) release(s *Stmt) error {
 		s.finalize()
 		return err
 	}
+	if err := s.ClearBindings(); err != nil {
+		s.finalize()
+		return nil
+	}
 	c.m.Lock()
 	defer c.m.Unlock()
 	c.l.PushFront(s)