[Bug 650695] Patch adds LATM/LOAS support to codecmap and make distinction between ADTS and LATM/LOAS

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=650695
  GStreamer | gst-ffmpeg | git

--- Comment #2 from Rafael Diniz <rafael at riseup.net> 2011-05-23 14:42:29 UTC ---
You're right.
How can I make an "or" with ( adts || adif || raw ) in
ext/ffmpeg/gstffmpegcodecmap.c? Any other information I could use to do that?
Or just place CODEC_ID_AAC_LATM before CODEC_ID_AAC, and leave CODEC_ID_AAC
without any stream-format?

I think I should also add in the patch this:


diff --git a/ext/ffmpeg/gstffmpegdec.c b/ext/ffmpeg/gstffmpegdec.c
index 305178c..3fba4ba 100644
--- a/ext/ffmpeg/gstffmpegdec.c
+++ b/ext/ffmpeg/gstffmpegdec.c
@@ -828,7 +828,7 @@ gst_ffmpegdec_setcaps (GstPad * pad, GstCaps * caps)
   }

   /* for AAC we only use av_parse if not on raw caps */
-  if (oclass->in_plugin->id == CODEC_ID_AAC) {
+  if (oclass->in_plugin->id == CODEC_ID_AAC || oclass->in_plugin->id ==
CODEC_ID_AAC_LATM) {
     const gchar *format = gst_structure_get_string (structure,
"stream-format");

     if (format == NULL || strcmp (format, "raw") == 0) {

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