lmdb 0.7.0 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c098384d91381d0f7c66a4794f88ea55c1c8a1e54d60480a75a379704407252d
-  data.tar.gz: 11c4e5b74f9b552ad9f89d5a189bf812340ab409b43b54361cb84f7d47dfd906
+  metadata.gz: ec26115e9cc3611f3305c308a5a26959e0f2ff6438c191968a562c85808fb6ed
+  data.tar.gz: 866b32d018335fa61dbefdc30234c976573a307ad11ae31c260a902cba6812e8
 SHA512:
-  metadata.gz: da3b52d345951ad527b67ce32998c749b17267b7b0d075f695e841d1b2a0891df4e0064e021e7db9ac21f41c0b81a4fb590ff88a98250675e15ad14baf185680
-  data.tar.gz: 12ae851f60db43dec61730f8497a06bf952e56242bff2e8fe64153a852c34582441556320cb1627fa65b7c29dfe417ba312c929d242511c0ab9c3888b12075b4
+  metadata.gz: '0356608ed8065cb4065a752fff8e4aa657fc29c38630e45d278c7edcad4fd211f8129e87eba2e323f2f72a8751b6a43ead97501ece2c8a504cfc9eb0e3b0174c'
+  data.tar.gz: 7f1e02c62e406c108bd7db8875cf528c2ec43cca0beb2ab1ae6380a71b3054b74b66beddd98404e3de93de0928d9221aca9e73c6b309caa5734a0ef78d2d1313

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,11 +126,8 @@ module LMDB
 
       ret = false
       # read-only txn was having trouble being nested inside a read-write
-      maybe_txn true do
-      # env.transaction true do
-      # env.transaction do
-        cursor { |c| ret = !!c.set(key, value) }
-      end
+      maybe_txn(true) { cursor { |c| ret = !!c.set(key, value) } }
+
       ret
     end
 
@@ -145,16 +142,19 @@ 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|
+        put(key, value, **options.merge(flags)) unless has?(key, value)
       end
     end
 
     # Delete the key (and optional value pair) if it exists; do not
     # complain about missing keys.
+    #
     # @param key [#to_s] The key of the record
     # @param value [#to_s, nil] The optional value
     #
@@ -163,11 +163,7 @@ module LMDB
     # @return [void]
     #
     def delete?(key, value = nil)
-      begin
-        delete key, value
-      rescue LMDB::Error::NOTFOUND
-        nil
-      end
+      env.transaction { |txn| delete(key, value) if has?(key, value) }
     end
 
     # Return how many records there are in this database.

data/lib/lmdb/version.rb

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

data/spec/helper.rb

@@ -1,12 +1,12 @@
 require 'lmdb'
 require 'rspec'
 require 'fileutils'
+require 'tmpdir'
 
 # for valgrind
 at_exit { GC.start }
 
-SPEC_ROOT = File.dirname(__FILE__)
-TEMP_ROOT = File.join(SPEC_ROOT, 'tmp')
+TEMP_ROOT = File.join(Dir.tmpdir, 'lmdb-spec')
 
 module LMDB::SpecHelper
   def mkpath(name = 'env')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lmdb
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.7.3
 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: