omnizip 0.3.23 → 0.3.24

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 61cf26f9e324600f286b236ab80e02d720059f32bca2fa3c71640ba3abc4355a
-  data.tar.gz: fa189e081e402ecb4d084490e7874402faacf231a296cd009a15f218fcd9453e
+  metadata.gz: a0119d9a2a28f5bc71cc0aa63f142b373196b8ca3f4abc0db93a71e0a9e45051
+  data.tar.gz: b3f6b837610a9005322e25dc5fd9d79a34529b9cd29d2c4d634efb3d64ad50fa
 SHA512:
-  metadata.gz: 308660d6095a909d72c528bdfb5e828e76d320172b979a53ef7401aa2c1c12a560a0afd57f49991eb5190f87deca1c15cd798e8abae89bd3341ed3bb43c39b3d
-  data.tar.gz: 29ac9066539f2d8cef3973a2eb9cfe15a549a5efead04a1f5c88f28587fb6a2d23ac28c401c1c4187f5c226bfe4b531e3565523400dcd183adc83c842cd2d5dc
+  metadata.gz: dfa90d09bbe46eed96bec5565d1751e2bed9d47ee0e9a52b587c3cbad87aaf650e1083a4e8e9d3a22b7089ff0cf218386a2f55f637691f0de363d7646c114b3a
+  data.tar.gz: 5fecb94fb627f48411717bd7dcc4c9243f86b4030b4a24a951d8f0816a123e6ec7288cd4c6920e70a89fa0297e0d082389370341d4792d87985c6198ec7c2bbe

data/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.24] - 2026-08-26
+
+### Changed
+- LZMA2 encoder hot paths avoid per-call allocations: the
+  range-encoder symbol drain reuses its 10 KB scratch buffer
+  (profiles showed String#* at ~half of encoder CPU), and the optimal
+  parser compares 8-byte windows and selects best matches without
+  intermediate strings/arrays. ~180k fewer object allocations and
+  ~94 MB less churn per 138 KB compressed, with byte-identical
+  output; covers both the xz and 7-Zip paths.
+
 ## [0.3.23] - 2026-08-26
 
 ### Changed

data/lib/omnizip/algorithms/lzma/optimal_encoder.rb

@@ -67,11 +67,12 @@ module Omnizip
             ].min
 
             len = 2
-            # Compare 8 bytes at a time using 64-bit integers
+            # Compare 8 bytes at a time using 64-bit integers read
+            # without intermediate strings (byteslice + unpack1
+            # allocates two Strings per step and dominated GC time).
             while len + 8 <= max_match_len
-              # Read 8 bytes as little-endian 64-bit integers
-              v1 = buf.byteslice(buf_pos + len, 8).unpack1("Q<")
-              v2 = buf.byteslice(buf_back_index + len, 8).unpack1("Q<")
+              v1 = read8(buf, buf_pos + len)
+              v2 = read8(buf, buf_back_index + len)
               break if v1 != v2
 
               len += 8
@@ -104,7 +105,15 @@ module Omnizip
           # A match may reference anything before `position`: bytes already
           # committed to the dictionary *and* bytes decoded earlier within
           # the chunk currently being encoded.
-          best_normal = matches.max_by { |m| [m.length, m.distance] }
+          best_normal = nil
+          matches.each do |m|
+            next if best_normal &&
+              (m.length < best_normal.length ||
+                (m.length == best_normal.length &&
+                  m.distance <= best_normal.distance))
+
+            best_normal = m
+          end
 
           if best_normal && best_normal.length >= nice_len &&
               best_normal.distance <= position
@@ -126,6 +135,18 @@ module Omnizip
           end
         end
 
+        # Allocation-free little-endian 8-byte read.
+        def read8(buf, pos)
+          buf.getbyte(pos) |
+            (buf.getbyte(pos + 1) << 8) |
+            (buf.getbyte(pos + 2) << 16) |
+            (buf.getbyte(pos + 3) << 24) |
+            (buf.getbyte(pos + 4) << 32) |
+            (buf.getbyte(pos + 5) << 40) |
+            (buf.getbyte(pos + 6) << 48) |
+            (buf.getbyte(pos + 7) << 56)
+        end
+
         # Constants
         REPS = 4
         MATCH_LEN_MAX = 273 # From lzma.h

data/lib/omnizip/implementations/xz_utils/lzma2/encoder.rb

@@ -339,8 +339,11 @@ module Omnizip
           def encode_queued_symbols(encoder, output)
             return if encoder.none?
 
-            # Create temporary buffer for encoding
-            temp_buffer = "\0" * 10000
+            # Reused scratch buffer: allocating a fresh 10 KB String
+            # per drain showed up as half the encoder's runtime via
+            # GC (String#* in profiles).
+            @symbol_buffer ||= "\0".b * 10_000
+            temp_buffer = @symbol_buffer
             out_pos = Omnizip::Algorithms::LZMA::IntRef.new(0)
 
             # Track size before encoding

data/lib/omnizip/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Omnizip
-  VERSION = "0.3.23"
+  VERSION = "0.3.24"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.23
+  version: 0.3.24
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-08-25 00:00:00.000000000 Z
+date: 2026-08-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64