update based on the most recent version in vlib/db/sqlite/ in V 0.4.12 a26a42d

Author: spytheman

orm.v

@@ -133,7 +133,7 @@ fn bind(stmt Stmt, mut c &int, data orm.Primitive) int {
 			err = stmt.bind_i64(c, i64(data))
 		}
 		f32, f64 {
-			err = stmt.bind_f64(c, unsafe { *(&f64(&data)) })
+			err = stmt.bind_f64(c, f64(data))
 		}
 		string {
 			err = stmt.bind_text(c, data)
@@ -167,7 +167,9 @@ fn (stmt Stmt) sqlite_select_column(idx int, typ int) !orm.Primitive {
 		return stmt.get_int(idx) or { return orm.Null{} }
 	} else if typ in orm.num64 {
 		return stmt.get_i64(idx) or { return orm.Null{} }
-	} else if typ in orm.float {
+	} else if typ == typeof[f32]().idx {
+		return f32(stmt.get_f64(idx) or { return orm.Null{} })
+	} else if typ == typeof[f64]().idx {
 		return stmt.get_f64(idx) or { return orm.Null{} }
 	} else if typ == orm.type_string {
 		if v := stmt.get_text(idx) {

sqlite_vfs_lowlevel_test.v

@@ -1,3 +1,4 @@
+// vtest build: present_sqlite3?
 import sqlite
 import rand
 
@@ -281,3 +282,38 @@ fn example_vfs_getlasterror(vfs &sqlite.Sqlite3_vfs, i int, o &char) int {
 	}
 	return sqlite.sqlite_ok
 }
+
+////////////////////////////////////////////////
+
+struct Human {
+	name string
+	age  f32
+}
+
+fn check_connect_full_default_vfs(vfs_name string) ! {
+	mut db := sqlite.connect_full(':memory:', [.readwrite, .create, .fullmutex], '')!
+	sql db {
+		create table Human
+	}!
+	h := Human{'Bilbo', 56}
+	sql db {
+		insert h into Human
+	}!
+	res := sql db {
+		select from Human
+	}!
+	db.close()!
+	assert res.len == 1
+	assert res[0] == h
+}
+
+fn test_connect_full_default_vfs() {
+	// passing '' here as vfs_name should work everywhere, and it should be equivalent
+	// to 'unix' or 'win32', depending on the current system:
+	check_connect_full_default_vfs('')!
+	$if windows {
+		check_connect_full_default_vfs('win32')!
+	} $else {
+		check_connect_full_default_vfs('unix')!
+	}
+}

vfs_lowlevel.c.v

@@ -143,6 +143,8 @@ pub enum OpenModeFlag {
 
 // connect_full Opens connection to sqlite database. It gives more control than `open`.
 // Flags give control over readonly and create decisions. Specific VFS can be chosen.
+// Pass '' for vfs_name, if you want to use the default VFS for the current platform,
+// (which is equivalent to 'win32' on Windows, and 'unix' on !Windows systems).
 pub fn connect_full(path string, mode_flags []OpenModeFlag, vfs_name string) !DB {
 	db := &C.sqlite3(unsafe { nil })
 
@@ -152,7 +154,11 @@ pub fn connect_full(path string, mode_flags []OpenModeFlag, vfs_name string) !DB
 		flags = flags | int(flag)
 	}
 
-	code := C.sqlite3_open_v2(&char(path.str), &db, flags, vfs_name.str)
+	mut pstr := unsafe { &char(0) }
+	if vfs_name != '' {
+		pstr = vfs_name.str
+	}
+	code := C.sqlite3_open_v2(&char(path.str), &db, flags, pstr)
 	if code != 0 {
 		return &SQLError{
 			msg:  unsafe { cstring_to_vstring(&char(C.sqlite3_errstr(code))) }