db.sqlite: allow for mut db := sqlite.connect_full(":memory:", [.readwrite, .create, .fullmutex], "")! defaulting to unix/win32 depending on the platform

spytheman · spytheman · commit a26a42dcb288 · 2025-11-20T14:34:28.000+02:00
diff --git a/vlib/db/sqlite/sqlite_vfs_lowlevel_test.v b/vlib/db/sqlite/sqlite_vfs_lowlevel_test.v
@@ -282,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')!
+	}
+}
diff --git a/vlib/db/sqlite/vfs_lowlevel.c.v b/vlib/db/sqlite/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))) }
