[gstreamer-bugs] [Bug 631200] flacparse: major performance improvements

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=631200
  GStreamer | gst-plugins-bad | git

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.21

--- Comment #28 from Sebastian Dröge <slomo at circular-chaos.org> 2010-10-07 21:39:12 UTC ---
commit 812075dc5db7c479a367815e6f386bf63d84504d
Author: Sebastian Dröge <sebastian.droege at collabora.co.uk>
Date:   Thu Oct 7 23:37:36 2010 +0200

    flacparse: Simplify frame header parsing by using lookup tables

    Based on a patch by Felipe Contreras.

    See bug #631200.

commit 64407ca94b3295bf88dd491d79eeb3aae9b4686a
Author: Sebastian Dröge <sebastian.droege at collabora.co.uk>
Date:   Thu Oct 7 23:28:08 2010 +0200

    flacparse: Don't parse the complete FLAC frames but only look for valid
fram

    Thanks to Felipe Contreras for the suggestion. This is partially
    based on his patches and makes flacparse more than 3.5 times faster.

    Looking for valid frame headers is unlikely to give false positives
    because every frame header is at least 9 bytes long, contains a
    14 bit sync code and a 8 bit checksum over the first 8 bytes.

    Fixes bug #631200.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.