Merge pull request stephencelis#1170 from stephencelis/revert-blob-changes

Revert blob changes

Author: jberkel

CHANGELOG.md

@@ -1,3 +1,8 @@
+0.14.1 (01-11-2022), [diff][diff-0.14.1]
+========================================
+
+* Reverted `Blob` changes (See [#1167][] for rationale).
+
 0.14.0 (27-10-2022), [diff][diff-0.14.0]
 ========================================
 For breaking changes, see [Upgrading.md](Documentation/Upgrading.md).
@@ -129,6 +134,7 @@ For breaking changes, see [Upgrading.md](Documentation/Upgrading.md).
 [diff-0.13.2]: https://github.com/stephencelis/SQLite.swift/compare/0.13.1...0.13.2
 [diff-0.13.3]: https://github.com/stephencelis/SQLite.swift/compare/0.13.2...0.13.3
 [diff-0.14.0]: https://github.com/stephencelis/SQLite.swift/compare/0.13.3...0.14.0
+[diff-0.14.1]: https://github.com/stephencelis/SQLite.swift/compare/0.14.0...0.14.1
 
 [#30]: https://github.com/stephencelis/SQLite.swift/issues/30
 [#142]: https://github.com/stephencelis/SQLite.swift/issues/142

Documentation/Index.md

@@ -90,7 +90,7 @@ process of downloading, compiling, and linking dependencies.
 
   ```swift
   dependencies: [
-    .package(url: "https://github.com/stephencelis/SQLite.swift.git", from: "0.14.0")
+    .package(url: "https://github.com/stephencelis/SQLite.swift.git", from: "0.14.1")
   ]
   ```
 
@@ -111,7 +111,7 @@ install SQLite.swift with Carthage:
  2. Update your Cartfile to include the following:
 
     ```ruby
-    github "stephencelis/SQLite.swift" ~> 0.14.0
+    github "stephencelis/SQLite.swift" ~> 0.14.1
     ```
 
  3. Run `carthage update` and [add the appropriate framework][Carthage Usage].
@@ -141,7 +141,7 @@ install SQLite.swift with Carthage:
     use_frameworks!
 
     target 'YourAppTargetName' do
-        pod 'SQLite.swift', '~> 0.14.0'
+        pod 'SQLite.swift', '~> 0.14.1'
     end
     ```
 
@@ -155,7 +155,7 @@ with the OS you can require the `standalone` subspec:
 
 ```ruby
 target 'YourAppTargetName' do
-  pod 'SQLite.swift/standalone', '~> 0.14.0'
+  pod 'SQLite.swift/standalone', '~> 0.14.1'
 end
 ```
 
@@ -165,7 +165,7 @@ dependency to sqlite3 or one of its subspecs:
 
 ```ruby
 target 'YourAppTargetName' do
-  pod 'SQLite.swift/standalone', '~> 0.14.0'
+  pod 'SQLite.swift/standalone', '~> 0.14.1'
   pod 'sqlite3/fts5', '= 3.15.0'  # SQLite 3.15.0 with FTS5 enabled
 end
 ```
@@ -181,7 +181,7 @@ If you want to use [SQLCipher][] with SQLite.swift you can require the
 target 'YourAppTargetName' do
   # Make sure you only require the subspec, otherwise you app might link against
   # the system SQLite, which means the SQLCipher-specific methods won't work.
-  pod 'SQLite.swift/SQLCipher', '~> 0.14.0'
+  pod 'SQLite.swift/SQLCipher', '~> 0.14.1'
 end
 ```
 

README.md

@@ -131,7 +131,7 @@ Swift code.
 
   ```swift
   dependencies: [
-      .package(url: "https://github.com/stephencelis/SQLite.swift.git", from: "0.14.0")
+      .package(url: "https://github.com/stephencelis/SQLite.swift.git", from: "0.14.1")
   ]
   ```
 
@@ -155,7 +155,7 @@ install SQLite.swift with Carthage:
  2. Update your Cartfile to include the following:
 
     ```ruby
-    github "stephencelis/SQLite.swift" ~> 0.14.0
+    github "stephencelis/SQLite.swift" ~> 0.14.1
     ```
 
  3. Run `carthage update` and

SQLite.swift.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name             = "SQLite.swift"
-  s.version          = "0.14.0"
+  s.version          = "0.14.1"
   s.summary          = "A type-safe, Swift-language layer over SQLite3."
 
   s.description      = <<-DESC

Sources/SQLite/Core/Blob.swift

@@ -21,64 +21,36 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 // THE SOFTWARE.
 //
-import Foundation
 
-public final class Blob {
+public struct Blob {
 
-    public let data: NSData
+    public let bytes: [UInt8]
 
-    public var bytes: UnsafePointer<UInt8> {
-        data.bytes.assumingMemoryBound(to: UInt8.self)
+    public init(bytes: [UInt8]) {
+        self.bytes = bytes
     }
 
-    public var length: Int {
-        data.count
-    }
-
-    public convenience init(bytes: [UInt8]) {
-        guard bytes.count > 0 else {
-            self.init(data: NSData())
-            return
-        }
-        self.init(data: NSData(bytes: bytes, length: bytes.count))
-    }
-
-    public convenience init(bytes: UnsafeRawPointer, length: Int) {
-        self.init(data: NSData(bytes: bytes, length: length))
-    }
-
-    public init(data: NSData) {
-        precondition(!(data is NSMutableData), "Blob cannot be initialized with mutable data")
-        self.data = data
+    public init(bytes: UnsafeRawPointer, length: Int) {
+        let i8bufptr = UnsafeBufferPointer(start: bytes.assumingMemoryBound(to: UInt8.self), count: length)
+        self.init(bytes: [UInt8](i8bufptr))
     }
 
     public func toHex() -> String {
-        guard length > 0 else { return "" }
-
-        var hex = ""
-        for idx in 0..<length {
-            let byte = bytes.advanced(by: idx).pointee
-            if byte < 16 {
-                hex += "0"
-            }
-            hex += String(byte, radix: 16, uppercase: false)
-        }
-        return hex
+        bytes.map {
+           ($0 < 16 ? "0" : "") + String($0, radix: 16, uppercase: false)
+        }.joined(separator: "")
     }
 }
 
 extension Blob: CustomStringConvertible {
-
     public var description: String {
         "x'\(toHex())'"
     }
-
 }
 
 extension Blob: Equatable {
-
 }
 
 public func ==(lhs: Blob, rhs: Blob) -> Bool {
-    lhs.data == rhs.data
+    lhs.bytes == rhs.bytes
 }

Sources/SQLite/Core/Connection.swift

@@ -741,7 +741,7 @@ extension Context {
     func set(result: Binding?) {
         switch result {
         case let blob as Blob:
-            sqlite3_result_blob(self, blob.bytes, Int32(blob.length), nil)
+            sqlite3_result_blob(self, blob.bytes, Int32(blob.bytes.count), nil)
         case let double as Double:
             sqlite3_result_double(self, double)
         case let int as Int64:

Sources/SQLite/Core/Statement.swift

@@ -103,10 +103,10 @@ public final class Statement {
         switch value {
         case .none:
             sqlite3_bind_null(handle, Int32(idx))
-        case let value as Blob where value.length == 0:
+        case let value as Blob where value.bytes.count == 0:
             sqlite3_bind_zeroblob(handle, Int32(idx), 0)
         case let value as Blob:
-            sqlite3_bind_blob(handle, Int32(idx), value.bytes, Int32(value.length), SQLITE_TRANSIENT)
+            sqlite3_bind_blob(handle, Int32(idx), value.bytes, Int32(value.bytes.count), SQLITE_TRANSIENT)
         case let value as Double:
             sqlite3_bind_double(handle, Int32(idx), value)
         case let value as Int64:

Sources/SQLite/Extensions/Cipher.swift

@@ -32,7 +32,7 @@ extension Connection {
     }
 
     public func key(_ key: Blob, db: String = "main") throws {
-        try _key_v2(db: db, keyPointer: key.bytes, keySize: key.length)
+        try _key_v2(db: db, keyPointer: key.bytes, keySize: key.bytes.count)
     }
 
     /// Same as `key(_ key: String, db: String = "main")`, running "PRAGMA cipher_migrate;"
@@ -53,7 +53,7 @@ extension Connection {
 
     /// Same as `[`keyAndMigrate(_ key: String, db: String = "main")` accepting byte array as key
     public func keyAndMigrate(_ key: Blob, db: String = "main") throws {
-        try _key_v2(db: db, keyPointer: key.bytes, keySize: key.length, migrate: true)
+        try _key_v2(db: db, keyPointer: key.bytes, keySize: key.bytes.count, migrate: true)
     }
 
     /// Change the key on an open database. NB: only works if the database is already encrypted.
@@ -68,7 +68,7 @@ extension Connection {
     }
 
     public func rekey(_ key: Blob, db: String = "main") throws {
-        try _rekey_v2(db: db, keyPointer: key.bytes, keySize: key.length)
+        try _rekey_v2(db: db, keyPointer: key.bytes, keySize: key.bytes.count)
     }
 
     /// Converts a non-encrypted database to an encrypted one.

Sources/SQLite/Foundation.swift

@@ -31,11 +31,13 @@ extension Data: Value {
     }
 
     public static func fromDatatypeValue(_ dataValue: Blob) -> Data {
-        dataValue.data as Data
+        Data(dataValue.bytes)
     }
 
     public var datatypeValue: Blob {
-        Blob(data: self as NSData)
+        withUnsafeBytes { (pointer: UnsafeRawBufferPointer) -> Blob in
+            Blob(bytes: pointer.baseAddress!, length: count)
+        }
     }
 
 }

Tests/SPM/Package.swift

@@ -15,7 +15,7 @@ let package = Package(
         // for testing from same repository
         .package(path: "../..")
         // normally this would be:
-        // .package(url: "https://github.com/stephencelis/SQLite.swift.git", from: "0.14.0")
+        // .package(url: "https://github.com/stephencelis/SQLite.swift.git", from: "0.14.1")
     ],
     targets: [
         .target(