minor fix

Author: codehz

src/easy_sqlite3/bindings.nim

@@ -446,9 +446,10 @@ proc getParameterIndex*(st: ref Statement, name: string): int =
   if result == 0:
     raise newException(SQLiteError, "Unknown parameter " & name)
 
+{.push inline.}
+
 proc `[]=`*(st: ref Statement, idx: int, blob: openarray[byte]) =
-  st.raw.check_sqlite_stmt sqlite3_bind_blob64(st.raw, idx, blob.unsafeAddr,
-      blob.len, TransientDestructor)
+  st.raw.check_sqlite_stmt sqlite3_bind_blob64(st.raw, idx, blob.unsafeAddr, blob.len, TransientDestructor)
 
 proc `[]=`*(st: ref Statement, idx: int, val: SomeFloat) =
   st.raw.check_sqlite_stmt sqlite3_bind_double(st.raw, idx, float64 val)
@@ -465,7 +466,7 @@ proc `[]=`*(st: ref Statement, idx: int, val: string) =
 proc reset*(st: ref Statement) =
   st.raw.check_sqlite_stmt sqlite3_reset(st.raw)
 
-proc step*(st: ref Statement): bool =
+proc step*(st: ref Statement): bool {.inline.} =
   let res = sqlite3_step(st.raw)
   case res:
   of sr_row: true
@@ -489,12 +490,10 @@ proc getColumn*(st: ref Statement, idx: int, T: typedesc[seq[byte]]): seq[byte]
   result = newSeq[byte]l
   copyMem(addr result[0], p, l)
 
-proc getColumn*(st: ref Statement, idx: int, T: typedesc[
-    SomeFloat]): SomeFloat =
+proc getColumn*(st: ref Statement, idx: int, T: typedesc[SomeFloat]): SomeFloat =
   cast[T](sqlite3_column_double(st.raw, idx))
 
-proc getColumn*(st: ref Statement, idx: int, T: typedesc[
-    SomeOrdinal]): SomeOrdinal =
+proc getColumn*(st: ref Statement, idx: int, T: typedesc[SomeOrdinal]): SomeOrdinal =
   cast[T](sqlite3_column_int64(st.raw, idx))
 
 proc getColumn*(st: ref Statement, idx: int, T: typedesc[string]): string =
@@ -515,6 +514,8 @@ proc unpack*[T: tuple](st: ref Statement, _: typedesc[T]): T =
     value = st.getColumn(idx, type(value))
     idx.inc
 
+{.pop.}
+
 proc exec*(db: var Database, sql: string): int {.discardable.} =
   var st = db.fetchStatement(sql)
   defer: st.reset()

tests/test_basic.nim

@@ -33,6 +33,7 @@ test "full":
   db.create_table()
   for name, value in dataset:
     db.insert_data name, value
+  db.exec "VACUUM"
   for name, value in db.iterate_data():
     check name in dataset
     check dataset[name] == value

tests/test_thread.nim

@@ -28,6 +28,7 @@ proc connectDatabase(): Database =
 
 var gdb = connectDatabase()
 gdb.create_table()
+gdb.exec "VACUUM"
 
 const COUNT = 100000
 
@@ -38,7 +39,7 @@ proc worker_fn() {.thread.} =
   for _ in 0..<COUNT:
     let val = r.rand(1048576)
     # increase the chance of collision
-    if val < 1024:
+    if val > 1024:
       sleep(1)
     when useMemFs:
       tdb.insert_data(val)