ogginfo-rb 0.0.3 → 0.0.4

data/lib/ogginfo-rb.rb

@@ -56,22 +56,8 @@ module Ogg
 
     private
     def calculate_duration file
-      if file.stat.size > 5000
-        # This seek is a dirty performance hack (saves reading whole file)
-        file.seek(-5000, IO::SEEK_END)
-      else
-        file.rewind
-      end
-      
-      # Find last page
-      pg = page = Page.find_page(file)
-      while pg
-        page = pg
-        pg = pg.next_page
-      end
-      
       # Duration is last granule position divided by sample rate
-      return page.granule_position.to_f / @sample_rate
+      return Page.find_last_page(file).granule_position.to_f / @sample_rate
     end
   end
 
@@ -131,6 +117,28 @@ module Ogg
       file.pos -= CAPTURE.size
       return Page.new(file)
     end
+    
+    def self.find_last_page file
+			buffer_size = 1024
+			pos = file.stat.size - buffer_size
+			while pos > 0
+				file.seek pos, IO::SEEK_SET
+				sio = StringIO.new file.read(buffer_size)
+				
+				# Find last page in buffer
+        pg = page = Page.find_page(sio)
+        while pg
+          page = pg
+          pg = pg.next_page
+        end
+				return page if page
+        
+				pos -= buffer_size * 2 - (CAPTURE.size - 1)
+			end
+			
+			# This means that the Ogg file is not valid
+			return nil
+    end
   end
 
   class Packet
@@ -205,7 +213,7 @@ module Ogg
           length = @data.read(4).unpack('L').first
           comment = @data.read(length)
           key, value = comment.split('=', 2)
-          (@comments[key.downcase] ||= []) << value
+          (@comments[key.upcase] ||= []) << value
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: ogginfo-rb
 version: !ruby/object:Gem::Version 
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors: 
 - Aiden Nibali
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-10-09 00:00:00 +11:00
+date: 2009-10-10 00:00:00 +11:00
 default_executable: 
 dependencies: []