[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 #9 from Rafael Diniz <rafael at riseup.net> 2011-05-26 13:04:14 UTC ---
Nice to see how you deal with the stream-format slomo!
I made a small mistake, could you revert this part of the patch (after testing
the code with TSs from different encoders, I realized it makes audio playback
choppy, with lots of errors):


diff --git a/ext/ffmpeg/gstffmpegdec.c b/ext/ffmpeg/gstffmpegdec.c
index a377a50..5fb52fb 100644
--- a/ext/ffmpeg/gstffmpegdec.c
+++ b/ext/ffmpeg/gstffmpegdec.c
@@ -832,9 +832,8 @@ gst_ffmpegdec_setcaps (GstPad * pad, GstCaps * caps)
       || 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 ||
-        oclass->in_plugin->id == CODEC_ID_AAC_LATM) {
-      ffmpegdec->turnoff_parser = TRUE;
+    if (format == NULL || strcmp (format, "raw") == 0) {
+      ffmpegdec->turnoff_parser = TRUE; 
     }
   }

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