lmdb 0.7.0 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c098384d91381d0f7c66a4794f88ea55c1c8a1e54d60480a75a379704407252d
-  data.tar.gz: 11c4e5b74f9b552ad9f89d5a189bf812340ab409b43b54361cb84f7d47dfd906
+  metadata.gz: 27c33b768709ded58c1414c9102a0a4024219abb305e875279b705d6d784aa68
+  data.tar.gz: 6d9b3f61f5ec981329fe42b6c310aca2760d5569f72202cbbe590258d1693591
 SHA512:
-  metadata.gz: da3b52d345951ad527b67ce32998c749b17267b7b0d075f695e841d1b2a0891df4e0064e021e7db9ac21f41c0b81a4fb590ff88a98250675e15ad14baf185680
-  data.tar.gz: 12ae851f60db43dec61730f8497a06bf952e56242bff2e8fe64153a852c34582441556320cb1627fa65b7c29dfe417ba312c929d242511c0ab9c3888b12075b4
+  metadata.gz: 8e1816c955d847948075046de78e382316ea599e56090bde14c669b4318cb12fc42d0c90cfed3089dded7be185c2589b99d756b343ae126c4e2c6f425e19451e
+  data.tar.gz: 2bab39420ba0d6243bf261239a1264694e8be367f2b221522b3e596acfc3697225190cd8b6ba41fc34596e73731cc7f870312e13b15f36ae66f9041186268eb0

data/ext/lmdb_ext/lmdb_ext.c

@@ -137,6 +137,28 @@ static VALUE transaction_is_readonly(VALUE self) {
     return (transaction->flags & MDB_RDONLY) ? Qtrue : Qfalse;
 }
 
+/**
+ * @overload finished?
+ *   @note This predicate is considered *unstable*; do not get used to it.
+ *   @return [false,true] whether the transaction is finished.
+ */
+static VALUE transaction_is_finished(VALUE self) {
+  TRANSACTION(self, transaction);
+  // MDB_TXN_FINISHED
+  return (transaction->flags & 0x01) ? Qtrue : Qfalse;
+}
+
+/**
+ * @overload error?
+ *   @note This predicate is considered *unstable*; do not get used to it.
+ *   @return [false,true] whether the transaction incurred an error.
+ */
+static VALUE transaction_is_error(VALUE self) {
+  TRANSACTION(self, transaction);
+  // MDB_TXN_ERROR
+  return (transaction->flags & 0x02) ? Qtrue : Qfalse;
+}
+
 
 static void transaction_finish(VALUE self, int commit) {
     TRANSACTION(self, transaction);
@@ -312,16 +334,21 @@ static VALUE with_transaction(VALUE venv, VALUE(*fn)(VALUE), VALUE arg, int flag
    */
 
   if (tparent && flags & MDB_RDONLY) {
+    // We are reusing the parent transaction.
+
     int exception;
     VALUE ret = rb_protect(fn, NIL_P(arg) ? vparent : arg, &exception);
+
     if (exception) {
-      if (vparent == environment_active_txn(venv))
-        transaction_abort(vparent);
+      // this is a cargo cult; i just copied it from below
+      if (vparent == environment_active_txn(venv)) transaction_abort(vparent);
       rb_jump_tag(exception);
     }
     return ret;
   }
   else {
+    // We are creating a new transaction.
+
     // XXX note this is a cursed goto loop that could almost certainly
     // be rewritten as a do-while
   retry:
@@ -1802,6 +1829,8 @@ void Init_lmdb_ext() {
         rb_define_method(cTransaction, "abort", transaction_abort, 0);
         rb_define_method(cTransaction, "env", transaction_env, 0);
         rb_define_method(cTransaction, "readonly?", transaction_is_readonly, 0);
+        rb_define_method(cTransaction, "finished?", transaction_is_finished, 0);
+        rb_define_method(cTransaction, "error?", transaction_is_error, 0);
 
         /**
          * Document-class: LMDB::Cursor

data/lib/lmdb/database.rb

@@ -126,7 +126,7 @@ module LMDB
 
       ret = false
       # read-only txn was having trouble being nested inside a read-write
-      maybe_txn true do
+      maybe_txn(true) do
       # env.transaction true do
       # env.transaction do
         cursor { |c| ret = !!c.set(key, value) }
@@ -145,11 +145,20 @@ module LMDB
     # @return [void]
     #
     def put?(key, value = nil, **options)
+      # early bailout
+      return if value.nil?
+
       flags = { (dupsort? ? :nodupdata : :nooverwrite) => true }
-      begin
-        put key, value, **options.merge(flags)
-      rescue LMDB::Error::KEYEXIST
-        nil
+
+      env.transaction do |txn|
+        # begin
+        put(key, value, **options.merge(flags)) unless has?(key, value)
+        # rescue LMDB::Error::KEYEXIST
+        #   # this should never be reached lol
+        #   # warn 'lol'
+        #   txn.abort
+        #   nil
+        # end
       end
     end
 
@@ -163,10 +172,13 @@ module LMDB
     # @return [void]
     #
     def delete?(key, value = nil)
-      begin
-        delete key, value
-      rescue LMDB::Error::NOTFOUND
-        nil
+      env.transaction do |txn|
+        begin
+          delete key, value
+        rescue LMDB::Error::NOTFOUND
+          txn.abort
+          nil
+        end
       end
     end
 

data/lib/lmdb/version.rb

@@ -1,3 +1,3 @@
 module LMDB
-  VERSION = '0.7.0'.freeze
+  VERSION = '0.7.2'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lmdb
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.7.2
 platform: ruby
 authors:
 - Daniel Mendler
 - Dorian Taylor
 bindir: bin
 cert_chain: []
-date: 2025-11-23 00:00:00.000000000 Z
+date: 2026-05-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -143,7 +143,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.3
+rubygems_version: 3.6.7
 specification_version: 4
 summary: Ruby bindings to Lightning MDB
 test_files: