Clenup code, move more to common (#20)

sqlite-open-helper/src/androidMain/kotlin/ru/pixnews/wasm/sqlite/open/helper/WasmSqliteOpenHelperFactoryBuilder.kt

@@ -12,6 +12,8 @@ import ru.pixnews.wasm.sqlite.open.helper.dsl.WasmSqliteOpenHelperFactoryConfigB
 import ru.pixnews.wasm.sqlite.open.helper.embedder.SqliteEmbedder
 import ru.pixnews.wasm.sqlite.open.helper.embedder.SqliteEmbedderConfig
 import ru.pixnews.wasm.sqlite.open.helper.embedder.WasmSqliteCommonConfig
+import ru.pixnews.wasm.sqlite.open.helper.internal.CloseGuard
+import ru.pixnews.wasm.sqlite.open.helper.internal.CloseGuard.Reporter
 import ru.pixnews.wasm.sqlite.open.helper.path.JvmDatabasePathResolver
 
 /**
@@ -28,6 +30,9 @@ public fun <E : SqliteEmbedderConfig> WasmSqliteOpenHelperFactory(
     val commonConfig = object : WasmSqliteCommonConfig {
         override val logger: Logger = config.logger
     }
+
+    setupCloseGuard(config.logger)
+
     return WasmSqliteOpenHelperFactory(
         pathResolver = config.pathResolver,
         defaultLocale = config.locale,
@@ -37,3 +42,10 @@ public fun <E : SqliteEmbedderConfig> WasmSqliteOpenHelperFactory(
         configurationOptions = config.configurationOptions,
     )
 }
+
+private fun setupCloseGuard(rootLogger: Logger) {
+    val logger = rootLogger.withTag("SQLite")
+    CloseGuard.reporter = Reporter { message, allocationSite ->
+        logger.w(allocationSite, message::toString)
+    }
+}

sqlite-open-helper/src/androidMain/kotlin/ru/pixnews/wasm/sqlite/open/helper/internal/SQLiteConnectionPool.kt

@@ -202,9 +202,9 @@ internal class SQLiteConnectionPool<CP : Sqlite3ConnectionPtr, SP : Sqlite3State
         val foreignKeyModeChanged = configuration.foreignKeyConstraintsEnabled !=
                 this.configuration.foreignKeyConstraintsEnabled
         if (foreignKeyModeChanged &&
-// Foreign key constraints can only be changed if there are no transactions
-// in progress.  To make this clear, we throw an exception if there are
-// any acquired connections.
+            // Foreign key constraints can only be changed if there are no transactions
+            // in progress.  To make this clear, we throw an exception if there are
+            // any acquired connections.
             acquiredConnections.isNotEmpty()
         ) {
             throw IllegalStateException(

sqlite-open-helper/src/androidMain/kotlin/ru/pixnews/wasm/sqlite/open/helper/internal/SQLiteCursor.kt

@@ -74,20 +74,17 @@ internal class SQLiteCursor(
     /**
      * The cursor window owned by this cursor.
      */
-    var window: CursorWindow? = null
+    private var window: CursorWindow? = null
         /**
          * Sets a new cursor window for the cursor to use.
          *
-         *
          * The cursor takes ownership of the provided cursor window; the cursor window
          * will be closed when the cursor is closed or when the cursor adopts a new
          * cursor window.
          *
-         *
          * If the cursor previously had a cursor window, then it is closed when the
          * new cursor window is assigned.
          *
-         *
          * @param newWindow The new cursor window, typically a remote cursor window.
          */
         set(newWindow) {