Fix two quick issues

First is allowing bind to take AbstractVector; this is needed for
DBInterface.executemany which passes a custom AbstractVector "view" type
for parameters.

Second is calling sqlite_reset before executing statements; this is
important for prepared statements because they may have been executed
before and never reset (they're usually reset by iterating results, but
we don't require users to fully consume cursor results).

Author: quinnj

src/SQLite.jl

@@ -181,7 +181,7 @@ function bind!(stmt::Stmt, params::Union{NamedTuple, Dict})
     end
 end
 
-function bind!(stmt::Stmt, values::Union{Vector, Tuple})
+function bind!(stmt::Stmt, values::Union{AbstractVector, Tuple})
     nparams = sqlite3_bind_parameter_count(stmt.handle)
     @assert nparams == length(values) "you must provide values for all query placeholders"
     for i in 1:nparams
@@ -306,6 +306,7 @@ The sqlite return status code is returned. To return results from a query, pleas
 function execute end
 
 function execute(stmt::Stmt, params=())
+    sqlite3_reset(stmt.handle)
     bind!(stmt, params)
     r = sqlite3_step(stmt.handle)
     stmt.status = r