fixing warnings

Author: tenderlove

ext/sqlite3/statement.c

@@ -63,6 +63,8 @@ static VALUE initialize(VALUE self, VALUE db, VALUE sql)
 
   rb_iv_set(self, "@connection", db);
   rb_iv_set(self, "@remainder", rb_str_new2(tail));
+  rb_iv_set(self, "@columns", Qnil);
+  rb_iv_set(self, "@types", Qnil);
 
   return self;
 }

lib/sqlite3/database.rb

@@ -110,7 +110,7 @@ def execute( sql, *bind_vars )
         if block_given?
           result.each { |row| yield row }
         else
-          return result.inject( [] ) { |arr,row| arr << row; arr }
+          result.inject( [] ) { |arr,row| arr << row; arr }
         end
       end
     end

lib/sqlite3/statement.rb

@@ -65,7 +65,7 @@ def bind_params( *bind_vars )
     # See also #bind_params, #execute!.
     def execute( *bind_vars )
       must_be_open!
-      reset! if active?
+      reset! if active? || done?
 
       bind_params(*bind_vars) unless bind_vars.empty?
       @results = ResultSet.new(@connection, self)
@@ -96,7 +96,7 @@ def execute! *bind_vars, &block
     # Returns true if the statement is currently active, meaning it has an
     # open result set.
     def active?
-      not @results.nil?
+      !done?
     end
 
     # Return an array of the column names for this statement. Note that this
@@ -120,7 +120,7 @@ def each
     # makes it a (potentially) expensive operation.
     def types
       must_be_open!
-      get_metadata unless defined?(@types)
+      get_metadata unless @types
       @types
     end