audio: Trivial comment for unhandled MPEG-2 payloading case
authorArun Raghavan <arun@accosted.net>
Wed, 24 Sep 2014 15:08:31 +0000 (20:38 +0530)
committerArun Raghavan <arun@accosted.net>
Sat, 27 Sep 2014 05:10:27 +0000 (10:40 +0530)
The spec mentions a version of the MPEG-2 frame with a base frame and
extension frame. I don't have IEC 13818-3 to figure out what that is,
and don't see any references in search results, so it's a FIXME for now.

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

gst-libs/gst/audio/gstaudioiec61937.c

index c67ec68..6ae854b 100644 (file)
@@ -276,7 +276,8 @@ gst_audio_iec61937_payload (const guint8 * src, guint src_n, guint8 * dst,
        *                            06 = MPEG 2, with extension
        *                            08 - MPEG 2 LSF, Layer 1
        *                            09 - MPEG 2 LSF, Layer 2
-       *                            10 - MPEG 2 LSF, Layer 3 */
+       *                            10 - MPEG 2 LSF, Layer 3
+       *                 FIXME: we don't handle type 06 at the moment */
       if (version == 1 && layer == 1)
         dst[five] = 0x04;
       else if ((version == 1 && (layer == 2 || layer == 3)) ||