format_parser 2.9.0 → 2.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04b14799c012037cf53b949b0920e0f8e2d5a42e30182363fee0d283d41eade4
-  data.tar.gz: 78ff4a354e05fb0e67a7b8dd8368768fceb313afc4dd736d5fa41592dfbc7b35
+  metadata.gz: 922909c34fc304a86a75f6396bd9627cdacf009aba5191da863064811ca2906f
+  data.tar.gz: 864b73aa70b7ec650dab326f677b17f7ed1905d05560deb669a564bb9fdc3d37
 SHA512:
-  metadata.gz: e5792c23191b2d4b4d63a5d9dfbfe5a3b28862f9e8ba977c49e39f3ebaf6bd222a31c296f224743a00fc877f8dbbfb6f97cdd19a0b6d712db6e3a00d29dd30d8
-  data.tar.gz: 6fd2ac968375090928b407afa3234c623c933150a36a00da965e75bb97d01ee2dcb502124e71fa100f0f2321b16a0f268c8a8fa94691cd195fbeb65a1fbab177
+  metadata.gz: 5f2527d385f3270f9b11976d478030108787a165e4440825c4ffa8c6a56054e67e5a7fb1a88d18d25dcb945ab6c2befc97156e7a2a5d5ad98f2e11e7c558ce2a
+  data.tar.gz: 788254d0f2e40625a6f2fd17c3c7f8959456ad1018fceb62305c5a877a3a8fd52ad5587915b0d3711a8ab68c6dc0ca6d283d9556f68a3234bbd3ecb7a50ece95

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 2.10.0
+* Improve WAV parser by focusing on performance rather than on attempting a best-effort when extracting metadata from files that do not strictly follow the format spec.
+
 ## 2.9.0
 * Improve WAV parser by performing a best-effort when extracting metadata from files that do not strictly follow the format spec.
 

data/lib/format_parser/version.rb

@@ -1,3 +1,3 @@
 module FormatParser
-  VERSION = '2.9.0'
+  VERSION = '2.10.0'
 end

data/lib/parsers/wav_parser.rb

@@ -21,30 +21,27 @@ class FormatParser::WAVParser
     # The specification does not require the Format chunk to be the first chunk
     # after the RIFF header.
     # https://www.mmsp.ece.mcgill.ca/Documents/AudioFormats/WAVE/WAVE.html
+    fmt_processed = false
+    data_processed = false
     fmt_data = {}
     data_size = 0
-    total_sample_frames = nil
     loop do
       chunk_type, chunk_size = safe_read(io, 8).unpack('a4l')
       case chunk_type
       when 'fmt ' # watch out: the chunk ID of the format chunk ends with a space
         fmt_data = unpack_fmt_chunk(io, chunk_size)
+        fmt_processed = true
       when 'data'
         data_size = chunk_size
-      when 'fact'
-        total_sample_frames = safe_read(io, 4).unpack('l').first
-        safe_skip(io, chunk_size - 4)
+        data_processed = true
       else
         # Skip this chunk until a known chunk is encountered
         safe_skip(io, chunk_size)
       end
-    rescue FormatParser::IOUtils::InvalidRead
-      # We've reached EOF, so it's time to make the most out of the metadata we
-      # managed to parse
-      break
+      break if fmt_processed && data_processed
     end
 
-    file_info(fmt_data, data_size, total_sample_frames)
+    file_info(fmt_data, data_size)
   end
 
   def unpack_fmt_chunk(io, chunk_size)
@@ -70,10 +67,10 @@ class FormatParser::WAVParser
     }
   end
 
-  def file_info(fmt_data, data_size, sample_frames)
+  def file_info(fmt_data, data_size)
     # NOTE: Each sample includes information for each channel
-    sample_frames ||= data_size / (fmt_data[:channels] * fmt_data[:bits_per_sample] / 8) if fmt_data[:channels] > 0 && fmt_data[:bits_per_sample] > 0
-    duration_in_seconds = sample_frames / fmt_data[:sample_rate].to_f if fmt_data[:sample_rate] > 0
+    sample_frames = data_size / (fmt_data[:channels] * fmt_data[:bits_per_sample] / 8) if fmt_data[:channels] > 0 && fmt_data[:bits_per_sample] > 0
+    duration_in_seconds = sample_frames / fmt_data[:sample_rate].to_f if sample_frames && fmt_data[:byte_rate] > 0
     FormatParser::Audio.new(
       format: :wav,
       num_audio_channels: fmt_data[:channels],

data/spec/parsers/wav_parser_spec.rb

@@ -20,8 +20,9 @@ describe FormatParser::WAVParser do
     expect(parse_result.format).to eq(:wav)
     expect(parse_result.num_audio_channels).to eq(1)
     expect(parse_result.audio_sample_rate_hz).to eq(8000)
-    expect(parse_result.media_duration_frames).to eq(110488)
-    expect(parse_result.media_duration_seconds).to be_within(0.01).of(13.81)
+    # Fixture does not define bits_per_sample in the fmt chunk
+    expect(parse_result.media_duration_frames).to be_nil
+    expect(parse_result.media_duration_seconds).to be_nil
   end
 
   it 'returns correct info about pcm files with more channels' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: format_parser
 version: !ruby/object:Gem::Version
-  version: 2.9.0
+  version: 2.10.0
 platform: ruby
 authors:
 - Noah Berman
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-09 00:00:00.000000000 Z
+date: 2024-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: exifr