[gstreamer-bugs] [Bug 612313] qtdemux: Post AAC profile/level in caps

[GStreamer (bugzilla.gnome.org)]

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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.26

--- Comment #12 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-10-05 18:51:28 UTC ---
I took the liberty to change your patch a little, I don't think it does what
you meant to do (we want to keep the data_len == 2 check for the part where we
override the stream parameters):

 commit 4a244e0d552f01d17b04de2cfcfb270ecd848521
 Author: Arun Raghavan <arun.raghavan at collabora.co.uk>
 Date:   Tue Oct 5 19:40:50 2010 +0100

    qtdemux: AAC codec_data can be > 2 bytes long

    This fixes the assumption that DecoderSpecificInfo must be 2 bytes long
    for AAC files. The specification allows HE-AAC to be explicitly
    signalled in a backward compatible way. This is done by means of an
    additional information after the regular AAC header. It is expected that
    decoders that can play AAC but not HE-AAC will parse the header normally
    and ignore extended bits, much as they do for the HE-AAC specific payload
    in the actual stream.

    https://bugzilla.gnome.org/show_bug.cgi?id=612313

Now we just need to fix up the codec utils a little for HE-AAC. I'll file a new
bug for that.

-- 
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.