omnizip 0.3.13 → 0.3.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3e86ca6410983afcc2259af21021ae0049d2b5f4def32975d9fa1568297417e3
-  data.tar.gz: b52224b6ef3a5b2dab3f56c9117daf36b679ecda95983cafd62ddd7d7f05b35a
+  metadata.gz: 3d76166f4e6d65cb67526c079369395377bc4d30af75d52b9d83374f99721071
+  data.tar.gz: c47f5c274906ca6d92293606add01a44bfdb4e3f2f499647608bd2ac0629b98f
 SHA512:
-  metadata.gz: 3b74c21559d4c813b83c82d80957c97d3f57f0d512cdb1895a92f869bd27594f5c0b60ae108179431e50f52bb44bfbb795343facb2de8f853258e483f4b63e75
-  data.tar.gz: ff4f463c54686edc825f514bb74c527d33a0a9319e2952ac628984392eb5acfb26ce77077a12ae711ddb44a419361a93e3336ae43d8c85a23d01e3efd445a029
+  metadata.gz: 4cc7cbf7d9ce6763e6648abc3415b588a5a79df8e6b15fdfea5a6db962531b7c584628c1fe1b2c5c604d4113d05f9120bd8eb61a1fba70acdbfa324b3e621a2f
+  data.tar.gz: b796a58122e8d1b9ce208aafbf2b64826789e34fd8d165f05bab54ebd74b9169bdb1814bb8f83f2209bc09abad465a5b5b73c12be8570e3b2429a3288119563f

data/lib/omnizip/algorithm.rb

@@ -23,6 +23,8 @@ module Omnizip
   # the required methods. Algorithms are automatically registered with the
   # AlgorithmRegistry when defined.
   class Algorithm
+    require "stringio"
+
     attr_reader :options, :filter
 
     # Initialize algorithm with options.
@@ -38,7 +40,7 @@ module Omnizip
     # The filter will be applied before compression and reversed after
     # decompression. Returns self for method chaining.
     #
-    # @param filter [Filters::FilterBase, FilterPipeline] Filter or
+    # @param filter [Omnizip::Filter, FilterPipeline] Filter or
     #   pipeline to use
     # @return [self] For method chaining
     def with_filter(filter)
@@ -82,6 +84,39 @@ module Omnizip
               "#{self} must implement .metadata"
       end
 
+      # Class-level convenience: compress `data` and return the
+      # compressed bytes. Carries the algorithm's streaming contract
+      # over a single StringIO so callers with the whole payload in
+      # memory (parallel compressors, small files) don't need to
+      # instantiate and wire streams themselves.
+      #
+      # @param data [String, IO] Input to compress
+      # @param options [Hash] Forwarded to the algorithm
+      # @return [String] Compressed bytes
+      def compress(data, **options)
+        instance = new(options)
+        input = data.is_a?(::IO) ? data : ::StringIO.new(data.to_s.b)
+        output = ::StringIO.new
+        output.set_encoding(Encoding::BINARY)
+        instance.compress(input, output, options)
+        output.string
+      end
+
+      # Class-level convenience: decompress `data` and return the
+      # uncompressed bytes. Mirror of +.compress+.
+      #
+      # @param data [String, IO] Input to decompress
+      # @param options [Hash] Forwarded to the algorithm
+      # @return [String] Decompressed bytes
+      def decompress(data, **options)
+        instance = new(options)
+        input = data.is_a?(::IO) ? data : ::StringIO.new(data.to_s.b)
+        output = ::StringIO.new
+        output.set_encoding(Encoding::BINARY)
+        instance.decompress(input, output, options)
+        output.string
+      end
+
       # Automatically register algorithm when inherited.
       #
       # This hook is called whenever a class inherits from Algorithm,

data/lib/omnizip/algorithms/bzip2.rb

@@ -82,9 +82,7 @@ module Omnizip
       # @param options [Models::CompressionOptions] Decompression options
       # @return [void]
       def decompress(input_stream, output_stream, _options = nil)
-        if output_stream.respond_to?(:set_encoding)
-          output_stream.set_encoding(Encoding::BINARY)
-        end
+        output_stream.set_encoding(Encoding::BINARY)
         decoder = Decoder.new(input_stream)
         decompressed = decoder.decode_stream
         output_stream.write(decompressed)

data/lib/omnizip/algorithms/deflate.rb

@@ -77,9 +77,7 @@ module Omnizip
       # @param options [Models::CompressionOptions] Decompression options
       # @return [void]
       def decompress(input_stream, output_stream, _options = nil)
-        if output_stream.respond_to?(:set_encoding)
-          output_stream.set_encoding(Encoding::BINARY)
-        end
+        output_stream.set_encoding(Encoding::BINARY)
         decoder = Decoder.new(input_stream)
         decompressed = decoder.decode_stream
         output_stream.write(decompressed)

data/lib/omnizip/algorithms/deflate64.rb

@@ -73,8 +73,8 @@ module Omnizip
         return if compressed.nil? || compressed.empty?
 
         # Set output to binary mode if it's a StringIO
-        output.set_encoding(Encoding::BINARY) if output.respond_to?(:set_encoding)
-        output.binmode if output.respond_to?(:binmode)
+        output.set_encoding(Encoding::BINARY)
+        output.binmode
 
         # Use Zlib::Inflate with maximum window size
         inflater = Zlib::Inflate.new(Zlib::MAX_WBITS)

data/lib/omnizip/algorithms/lzma.rb

@@ -136,7 +136,7 @@ module Omnizip
       # @return [IO] The output_stream (for chaining)
       def decompress(input_stream, output_stream, options = nil)
         # Set binary encoding on output stream for proper byte handling
-        output_stream.set_encoding(Encoding::BINARY) if output_stream.respond_to?(:set_encoding)
+        output_stream.set_encoding(Encoding::BINARY)
 
         # Build decoder options, merging with instance variables as fallbacks
         decoder_opts = build_decoder_options(options)

data/lib/omnizip/algorithms/lzma2.rb

@@ -83,23 +83,19 @@ module Omnizip
         )
 
         # Read input
-        input_data = input.respond_to?(:read) ? input.read : input
+        input_data = Omnizip::IO::Source.for(input).read
 
         # Encode with LZMA2
         compressed = encoder.encode(input_data)
 
         # Write to output
-        if output.respond_to?(:write)
-          output.write(compressed)
-        else
-          output.replace(compressed)
-        end
+        Omnizip::IO::Sink.for(output).write(compressed)
       end
 
       # Decompress LZMA2 data
       def decompress(input, output, options = {})
         # Read input data
-        input_data = input.respond_to?(:read) ? input.read : input
+        input_data = Omnizip::IO::Source.for(input).read
         input_stream = StringIO.new(input_data)
         input_stream.set_encoding(Encoding::BINARY)
 
@@ -120,11 +116,7 @@ module Omnizip
         decompressed = decoder.decode_stream
 
         # Write to output
-        if output.respond_to?(:write)
-          output.write(decompressed)
-        else
-          output.replace(decompressed)
-        end
+        Omnizip::IO::Sink.for(output).write(decompressed)
       end
 
       # Encode dictionary size as single byte for LZMA2 properties

data/lib/omnizip/algorithms/zstandard.rb

@@ -96,9 +96,7 @@ module Omnizip
       # @param options [Models::CompressionOptions] Decompression options
       # @return [void]
       def decompress(input_stream, output_stream, _options = nil)
-        if output_stream.respond_to?(:set_encoding)
-          output_stream.set_encoding(Encoding::BINARY)
-        end
+        output_stream.set_encoding(Encoding::BINARY)
         decoder = Decoder.new(input_stream)
         decompressed = decoder.decode_stream
         output_stream.write(decompressed)

data/lib/omnizip/algorithms.rb

@@ -1,7 +1,20 @@
 # frozen_string_literal: true
 
 module Omnizip
-  # Compression algorithm implementations
+  # High-level compression algorithm facades.
+  #
+  # Each top-level constant in this namespace (Deflate, BZip2, LZMA,
+  # LZMA2, Zstandard, PPMd7, PPMd8) is a facade that exposes the
+  # streaming +.compress+/+.decompress+ contract from +Omnizip::Algorithm+.
+  # Format-specific readers/writers call these facades for the actual
+  # compression work; the facades themselves do not know about archive
+  # container structure.
+  #
+  # Bit-level SDK ports (state machines, range coders, match finders)
+  # live under +Omnizip::Implementations+, not here. This namespace
+  # owns the *streaming contract*; that namespace owns the *primitives*.
+  # The boundary is: +Algorithms+ calls +Implementations+, never the
+  # reverse.
   module Algorithms
     autoload :PPMdBase, "omnizip/algorithms/ppmd_base"
     autoload :LZMA2Const, "omnizip/algorithms/lzma2/constants"

data/lib/omnizip/commands/archive_list_command.rb

@@ -185,29 +185,31 @@ module Omnizip
         end
       end
 
-      # Helper methods to handle different entry types
+      # Helper methods to handle different entry types. Each delegates
+      # to the Omnizip::Entry contract; format-specific accessors stay
+      # on the format entry classes themselves.
       def entry_name(entry)
-        entry.respond_to?(:name) ? entry.name : entry.to_s
+        entry.entry_name || entry.to_s
       end
 
       def entry_directory?(entry)
-        entry.respond_to?(:directory?) && entry.directory?
+        entry.entry_directory?
       end
 
       def entry_size(entry)
-        entry.respond_to?(:size) ? entry.size : 0
+        entry.entry_size || 0
       end
 
       def entry_compressed_size(entry)
-        entry.respond_to?(:compressed_size) ? entry.compressed_size : nil
+        entry.compressed_size if entry.is_a?(Omnizip::Formats::SevenZip::Models::FileEntry)
       end
 
       def entry_has_stream?(entry)
-        entry.respond_to?(:has_stream?) ? entry.has_stream? : true
+        entry.has_stream? if entry.is_a?(Omnizip::Formats::SevenZip::Models::FileEntry)
       end
 
       def entry_mtime(entry)
-        entry.respond_to?(:mtime) ? entry.mtime : nil
+        entry.entry_mtime
       end
     end
   end

data/lib/omnizip/entry.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module Omnizip
+  # Mix-in that establishes the common contract every archive entry
+  # satisfies, regardless of format.
+  #
+  # Each format-specific entry class (Zip::Entry, Tar::Entry,
+  # SevenZip::Models::FileEntry, Iso::DirectoryRecord, Ole::Dirent,
+  # Xar::Entry, Rpm::Entry, Msi::Entry, Cpio::Entry, Rar3::Entry,
+  # Rar5::Entry, Xz::Entry, Metadata::EntryMetadata) includes this
+  # module so callers can rely on a single narrow interface instead
+  # of duck-typing through `respond_to?(:name) / :path / :filename`
+  # cascades.
+  #
+  # Including classes MUST override the four methods below. The
+  # defaults raise `NotImplementedError` to surface missing
+  # implementations at call time rather than at include time.
+  module Entry
+    # @return [String, nil] in-archive path, or nil if the entry has
+    #   no name (e.g. empty anti-entry)
+    def entry_name
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_name"
+    end
+
+    # @return [Boolean] true if the entry is a directory
+    def entry_directory?
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_directory?"
+    end
+
+    # @return [Integer, nil] uncompressed size in bytes
+    def entry_size
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_size"
+    end
+
+    # @return [Time, nil] modification time
+    def entry_mtime
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_mtime"
+    end
+  end
+end

data/lib/omnizip/extraction/selective_extractor.rb

@@ -184,18 +184,10 @@ module Omnizip
 
       # Get filename from entry
       #
-      # @param entry [Object] Entry
+      # @param entry [#entry_name] Entry (Omnizip::Entry or compatible)
       # @return [String] Filename
       def entry_filename(entry)
-        if entry.respond_to?(:name)
-          entry.name
-        elsif entry.respond_to?(:path)
-          entry.path
-        elsif entry.respond_to?(:filename)
-          entry.filename
-        else
-          entry.to_s
-        end
+        entry.entry_name || entry.to_s
       end
 
       # Update progress tracker

data/lib/omnizip/filter_pipeline.rb

@@ -36,7 +36,7 @@ module Omnizip
     # Filters are applied in the order they are added during encoding,
     # and in reverse order during decoding.
     #
-    # @param filter [Filters::FilterBase] Filter instance to add
+    # @param filter [Omnizip::Filter] Filter instance to add
     # @return [self] For method chaining
     def add_filter(filter)
       @filters << filter

data/lib/omnizip/filter_registry.rb

@@ -4,16 +4,18 @@ module Omnizip
   class FilterRegistry < Omnizip::Registry
     DEFAULT_FORMATS = %i[xz seven_zip].freeze
 
+    # Single source of truth for builtin filters. Each entry binds the
+    # filter name to its class and the formats that support it.
     BUILTIN_FILTERS = {
-      "bcj-x86": "Omnizip::Filters::BcjX86",
-      "bcj-arm": "Omnizip::Filters::BcjArm",
-      "bcj-arm64": "Omnizip::Filters::BcjArm64",
-      "bcj-ia64": "Omnizip::Filters::BcjIa64",
-      "bcj-ppc": "Omnizip::Filters::BcjPpc",
-      "bcj-sparc": "Omnizip::Filters::BcjSparc",
-      bcj: "Omnizip::Filters::BCJ",
-      bcj2: "Omnizip::Filters::Bcj2",
-      delta: "Omnizip::Filters::Delta",
+      "bcj-x86": { const: "Omnizip::Filters::BcjX86",   formats: %i[xz seven_zip] },
+      "bcj-arm": { const: "Omnizip::Filters::BcjArm",   formats: %i[xz seven_zip] },
+      "bcj-arm64": { const: "Omnizip::Filters::BcjArm64", formats: [:seven_zip] },
+      "bcj-ia64": { const: "Omnizip::Filters::BcjIa64", formats: %i[xz seven_zip] },
+      "bcj-ppc": { const: "Omnizip::Filters::BcjPpc", formats: %i[xz seven_zip] },
+      "bcj-sparc": { const: "Omnizip::Filters::BcjSparc", formats: %i[xz seven_zip] },
+      bcj: { const: "Omnizip::Filters::BCJ", formats: [:seven_zip] },
+      bcj2: { const: "Omnizip::Filters::Bcj2", formats: [:seven_zip] },
+      delta: { const: "Omnizip::Filters::Delta", formats: %i[xz seven_zip] },
     }.freeze
 
     class << self
@@ -86,7 +88,6 @@ module Omnizip
 
         trigger = lazy_triggers[normalized]
         if trigger
-          synchronize { lazy_triggers.delete(normalized) }
           trigger.call
           entry = storage[normalized]
           return entry if entry
@@ -98,12 +99,13 @@ module Omnizip
   end
 end
 
-# Lazy triggers — calling the Filters::Registry.register_all re-runs
-# the registration of every builtin filter. Idempotent: register just
-# overwrites the storage entry. Survives reset! because the trigger
-# proc is held by FilterRegistry, not the storage.
-Omnizip::FilterRegistry::BUILTIN_FILTERS.each_key do |name|
+# Lazy triggers — each trigger resolves the filter's class (firing
+# autoload) and registers it with its declared formats. Idempotent:
+# register overwrites the storage entry. Survives reset! because the
+# trigger proc is held by FilterRegistry, not the storage.
+Omnizip::FilterRegistry::BUILTIN_FILTERS.each do |name, spec|
   Omnizip::FilterRegistry.register_lazy(name) do
-    Omnizip::Filters::Registry.register_all
+    klass = Object.const_get(spec[:const])
+    Omnizip::FilterRegistry.register(name, klass, formats: spec[:formats])
   end
 end

data/lib/omnizip/filters/bcj2.rb

@@ -34,7 +34,7 @@ module Omnizip
     # NOTE: Currently only decoding (decompression) is implemented.
     # Encoding is extremely complex and will be added in a future version.
     # For compression use cases, the simpler BCJ-x86 filter is recommended.
-    class Bcj2 < FilterBase
+    class Bcj2 < Omnizip::Filter
       # Encode is not currently supported for BCJ2.
       #
       # @param _data [String] Binary data to encode

data/lib/omnizip/filters/bcj_arm.rb

@@ -28,7 +28,7 @@ module Omnizip
     # The filter improves compression by making branch targets
     # position-independent. The offset in ARM BL instructions is stored
     # as word offset (divided by 4), and is relative to PC+8.
-    class BcjArm < FilterBase
+    class BcjArm < Omnizip::Filter
       # ARM BL (Branch and Link) opcode
       OPCODE_BL = 0xEB
 

data/lib/omnizip/filters/bcj_arm64.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets and
     # page-aligned addresses position-independent.
-    class BcjArm64 < FilterBase
+    class BcjArm64 < Omnizip::Filter
       # B/BL instruction base opcode
       OPCODE_B_BL = 0x94000000
 

data/lib/omnizip/filters/bcj_ia64.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets
     # position-independent.
-    class BcjIa64 < FilterBase
+    class BcjIa64 < Omnizip::Filter
       # Size of IA-64 instruction bundle (16 bytes, little-endian)
       BUNDLE_SIZE = 16
 

data/lib/omnizip/filters/bcj_ppc.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets
     # position-independent.
-    class BcjPpc < FilterBase
+    class BcjPpc < Omnizip::Filter
       # PPC B/BL instruction base (0x48000000)
       OPCODE_BASE = 0x48000000
 

data/lib/omnizip/filters/bcj_sparc.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets
     # position-independent.
-    class BcjSparc < FilterBase
+    class BcjSparc < Omnizip::Filter
       # Size of SPARC instruction (4 bytes, big-endian)
       INSTRUCTION_SIZE = 4
 

data/lib/omnizip/filters/bcj_x86.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter is reversible and works on 5-byte boundaries (1-byte
     # opcode + 4-byte address).
-    class BcjX86 < FilterBase
+    class BcjX86 < Omnizip::Filter
       # x86 CALL opcode
       OPCODE_CALL = 0xE8
 

data/lib/omnizip/filters.rb

@@ -3,7 +3,6 @@
 module Omnizip
   # Filter implementations for preprocessing data before compression
   module Filters
-    autoload :FilterBase, "omnizip/filters/filter_base"
     autoload :BCJ, "omnizip/filters/bcj"
     autoload :BcjX86, "omnizip/filters/bcj_x86"
     autoload :Bcj2, "omnizip/filters/bcj2"
@@ -17,6 +16,5 @@ module Omnizip
     autoload :BcjSparc, "omnizip/filters/bcj_sparc"
     autoload :BcjIa64, "omnizip/filters/bcj_ia64"
     autoload :Delta, "omnizip/filters/delta"
-    autoload :Registry, "omnizip/filters/registry"
   end
 end

data/lib/omnizip/formats/bzip2_file.rb

@@ -112,12 +112,6 @@ module Omnizip
           output_io.write(decompressed_io.read)
         end
 
-        # Register BZIP2 format when loaded
-        def register!
-          FormatRegistry.register(".bz2", Omnizip::Formats::Bzip2File)
-          FormatRegistry.register(".bzip2", Omnizip::Formats::Bzip2File)
-        end
-
         private
 
         # Write BZIP2 header
@@ -163,4 +157,3 @@ module Omnizip
     end
   end
 end
-

data/lib/omnizip/formats/cpio/entry.rb

@@ -8,6 +8,7 @@ module Omnizip
       # Represents a file, directory, or special file in a CPIO archive.
       # Supports newc, CRC, and ODC formats.
       class Entry
+        include Omnizip::Entry
         include Omnizip::Formats::Cpio::Constants
 
         # @return [String] Magic number identifying format
@@ -58,6 +59,11 @@ module Omnizip
         # @return [String] File data
         attr_accessor :data
 
+        def entry_name = name
+        def entry_directory? = directory?
+        def entry_size = filesize
+        def entry_mtime = mtime
+
         # Initialize CPIO entry
         #
         # @param attributes [Hash] Entry attributes

data/lib/omnizip/formats/cpio.rb

@@ -126,13 +126,7 @@ module Omnizip
             }
           end
         end
-
-        # Auto-register CPIO format when loaded
-        def register!
-          FormatRegistry.register(".cpio", Reader)
-        end
       end
     end
   end
 end
-

data/lib/omnizip/formats/gzip.rb

@@ -127,12 +127,6 @@ module Omnizip
           metadata
         end
 
-        # Register GZIP format when loaded
-        def register!
-          FormatRegistry.register(".gz", Omnizip::Formats::Gzip)
-          FormatRegistry.register(".gzip", Omnizip::Formats::Gzip)
-        end
-
         private
 
         # Write GZIP header
@@ -231,4 +225,3 @@ module Omnizip
     end
   end
 end
-

data/lib/omnizip/formats/iso/directory_record.rb

@@ -6,9 +6,16 @@ module Omnizip
       # ISO 9660 Directory Record
       # Represents a file or directory entry
       class DirectoryRecord
+        include Omnizip::Entry
+
         attr_accessor :full_path
         attr_reader :length, :extended_attr_length, :location, :data_length, :recording_date, :flags, :file_unit_size, :interleave_gap_size, :volume_sequence_number, :name, :system_use
 
+        def entry_name = full_path || name
+        def entry_directory? = directory?
+        def entry_size = data_length
+        def entry_mtime = recording_date
+
         # Set the full path for this record, used by Reader when walking
         # a directory tree. Allows callers to look up records by path
         # without re-joining parent names.

data/lib/omnizip/formats/iso.rb

@@ -127,11 +127,6 @@ module Omnizip
 
         writer.write
       end
-
-      # Auto-register ISO format when loaded
-      def self.register!
-        FormatRegistry.register(".iso", Reader)
-      end
     end
   end
 end

data/lib/omnizip/formats/lzip.rb

@@ -132,12 +132,6 @@ module Omnizip
           }
         end
 
-        # Register LZIP format when loaded
-        def register!
-          FormatRegistry.register(".lz", Omnizip::Formats::Lzip)
-          FormatRegistry.register(".lzip", Omnizip::Formats::Lzip)
-        end
-
         # Detect if data is LZIP format
         #
         # @param data [String] Data to check
@@ -149,4 +143,3 @@ module Omnizip
     end
   end
 end
-

data/lib/omnizip/formats/lzma_alone.rb

@@ -138,13 +138,7 @@ module Omnizip
             uncompressed_size: decoder.uncompressed_size,
           }
         end
-
-        # Register LZMA_Alone format when loaded
-        def register!
-          FormatRegistry.register(".lzma", Omnizip::Formats::LzmaAlone)
-        end
       end
     end
   end
 end
-

data/lib/omnizip/formats/msi/entry.rb

@@ -8,8 +8,14 @@ module Omnizip
       # Represents a single file within an MSI package.
       # File information is assembled from File, Component, and Directory tables.
       class Entry
+        include Omnizip::Entry
         include Omnizip::Formats::Msi::Constants
 
+        def entry_name = path
+        def entry_directory? = false
+        def entry_size = size
+        def entry_mtime = nil
+
         # @return [String] Full installation path
         attr_accessor :path
 

data/lib/omnizip/formats/msi.rb

@@ -67,16 +67,7 @@ module Omnizip
         def info(path)
           open(path, &:info)
         end
-
-        # Register MSI format with format registry
-        #
-        # This overrides OLE's .msi registration.
-        def register!
-          FormatRegistry.register(".msi", self)
-          FormatRegistry.register(".msp", self)
-        end
       end
     end
   end
 end
-

data/lib/omnizip/formats/ole/dirent.rb

@@ -8,8 +8,14 @@ module Omnizip
       # Represents a file or directory entry in an OLE compound document.
       # Each dirent is 128 bytes and contains metadata about the entry.
       class Dirent
+        include Omnizip::Entry
         include Omnizip::Formats::Ole::Constants
 
+        def entry_name = name
+        def entry_directory? = dir?
+        def entry_size = nil
+        def entry_mtime = modify_time
+
         # Pack format for dirent structure
         PACK = "a64 v C C V3 a16 V a8 a8 V2 a4"
 

data/lib/omnizip/formats/ole.rb

@@ -117,14 +117,6 @@ module Omnizip
           end
         end
 
-        # Register OLE format in registry
-        def register!
-          FormatRegistry.register(".ole", Storage)
-          FormatRegistry.register(".doc", Storage)
-          FormatRegistry.register(".xls", Storage)
-          FormatRegistry.register(".ppt", Storage)
-        end
-
         private
 
         # Recursively extract dirent to directory
@@ -147,4 +139,3 @@ module Omnizip
     end
   end
 end
-