cgosqlite: cleanup two unnecessary unsafe casts

raggi · raggi · commit 7ab00bdc315f · 2026-01-06T14:37:00.000-08:00
The type is already compatible directly out of the API so there's no
need for the cast dance in these calls.

Updates #cleanup
diff --git a/cgosqlite/cgosqlite.go b/cgosqlite/cgosqlite.go
@@ -244,11 +244,11 @@ func (stmt *Stmt) StartTimer() {
 }
 
 func (stmt *Stmt) ColumnDatabaseName(col int) string {
-	return C.GoString((*C.char)(unsafe.Pointer(C.sqlite3_column_database_name(stmt.stmt, C.int(col)))))
+	return C.GoString(C.sqlite3_column_database_name(stmt.stmt, C.int(col)))
 }
 
 func (stmt *Stmt) ColumnTableName(col int) string {
-	return C.GoString((*C.char)(unsafe.Pointer(C.sqlite3_column_table_name(stmt.stmt, C.int(col)))))
+	return C.GoString(C.sqlite3_column_table_name(stmt.stmt, C.int(col)))
 }
 
 func (stmt *Stmt) Step(colType []sqliteh.ColumnType) (row bool, err error) {

Original file line number	Diff line number	Diff line change
`@@ -244,11 +244,11 @@ func (stmt *Stmt) StartTimer() {`
`244`	`244`	`}`
`245`	`245`
`246`	`246`	`func (stmt *Stmt) ColumnDatabaseName(col int) string {`
`247`		`- return C.GoString((*C.char)(unsafe.Pointer(C.sqlite3_column_database_name(stmt.stmt, C.int(col)))))`
	`247`	`+ return C.GoString(C.sqlite3_column_database_name(stmt.stmt, C.int(col)))`
`248`	`248`	`}`
`249`	`249`
`250`	`250`	`func (stmt *Stmt) ColumnTableName(col int) string {`
`251`		`- return C.GoString((*C.char)(unsafe.Pointer(C.sqlite3_column_table_name(stmt.stmt, C.int(col)))))`
	`251`	`+ return C.GoString(C.sqlite3_column_table_name(stmt.stmt, C.int(col)))`
`252`	`252`	`}`
`253`	`253`
`254`	`254`	`func (stmt *Stmt) Step(colType []sqliteh.ColumnType) (row bool, err error) {`