gst/mxf/: Coding style fixes.

Original commit message from CVS:
* gst/mxf/mxfdemux.c:
(gst_mxf_demux_handle_header_metadata_resolve_references):
* gst/mxf/mxfdv-dif.c: (mxf_dv_dif_create_caps):
* gst/mxf/mxfjpeg2000.c: (mxf_jpeg2000_create_caps):
Coding style fixes.
* gst/mxf/mxfmpeg.c: (mxf_mpeg_audio_handle_essence_element),
(mxf_mpeg_es_create_caps), (mxf_mpeg_create_caps):
Add support for MPEG, AC3, DTS audio.

ChangeLog

@@ -1,3 +1,15 @@
+2008-12-04  Sebastian Dröge  <sebastian.droege@collabora.co.uk>
+
+	* gst/mxf/mxfdemux.c:
+	(gst_mxf_demux_handle_header_metadata_resolve_references):
+	* gst/mxf/mxfdv-dif.c: (mxf_dv_dif_create_caps):
+	* gst/mxf/mxfjpeg2000.c: (mxf_jpeg2000_create_caps):
+	Coding style fixes.
+
+	* gst/mxf/mxfmpeg.c: (mxf_mpeg_audio_handle_essence_element),
+	(mxf_mpeg_es_create_caps), (mxf_mpeg_create_caps):
+	Add support for MPEG, AC3, DTS audio.
+
 2008-12-04  Sebastian Dröge  <sebastian.droege@collabora.co.uk>
 
 	* gst/mxf/mxfdemux.c:

gst/mxf/mxfmpeg.c

@@ -237,6 +237,24 @@ mxf_mpeg_video_handle_essence_element (const MXFUL * key, GstBuffer * buffer,
   return GST_FLOW_OK;
 }
 
+static GstFlowReturn
+mxf_mpeg_audio_handle_essence_element (const MXFUL * key, GstBuffer * buffer,
+    GstCaps * caps, MXFMetadataGenericPackage * package,
+    MXFMetadataTrack * track, MXFMetadataStructuralComponent * component,
+    gpointer mapping_data, GstBuffer ** outbuf)
+{
+  *outbuf = buffer;
+
+  /* SMPTE 381M 6.2 */
+  if (key->u[12] != 0x16 || (key->u[14] != 0x05 && key->u[14] != 0x06
+          && key->u[14] != 0x07)) {
+    GST_ERROR ("Invalid MPEG audio essence element");
+    return GST_FLOW_ERROR;
+  }
+
+  return GST_FLOW_OK;
+}
+
 /* Private uid used by SONY C0023S01.mxf,
  * taken from the ffmpeg mxf demuxer */
 static const guint8 sony_mpeg4_extradata[] = {
@@ -244,22 +262,52 @@ static const guint8 sony_mpeg4_extradata[] = {
   0x01, 0x00, 0x00
 };
 
+/* RP224 */
+
+static const MXFUL sound_essence_compression_ac3 = { {
+        0x06, 0x0E, 0x2B, 0x34, 0x04, 0x01, 0x01, 0x01, 0x04, 0x02, 0x02, 0x02,
+    0x03, 0x02, 0x01, 0x00}
+};
+
+static const MXFUL sound_essence_compression_mpeg1_layer1 = { {
+        0x06, 0x0E, 0x2B, 0x34, 0x04, 0x01, 0x01, 0x01, 0x04, 0x02, 0x02, 0x02,
+    0x03, 0x02, 0x04, 0x00}
+};
+
+static const MXFUL sound_essence_compression_mpeg1_layer12 = { {
+        0x06, 0x0E, 0x2B, 0x34, 0x04, 0x01, 0x01, 0x01, 0x04, 0x02, 0x02, 0x02,
+    0x03, 0x02, 0x05, 0x00}
+};
+
+static const MXFUL sound_essence_compression_mpeg1_layer2 = { {
+        0x06, 0x0E, 0x2B, 0x34, 0x04, 0x01, 0x01, 0x08, 0x04, 0x02, 0x02, 0x02,
+    0x03, 0x02, 0x05, 0x01}
+};
+
+static const MXFUL sound_essence_compression_mpeg2_layer1 = { {
+        0x06, 0x0E, 0x2B, 0x34, 0x04, 0x01, 0x01, 0x08, 0x04, 0x02, 0x02, 0x02,
+    0x03, 0x02, 0x06, 0x00}
+};
+
+static const MXFUL sound_essence_compression_dts = { {
+        0x06, 0x0E, 0x2B, 0x34, 0x04, 0x01, 0x01, 0x08, 0x04, 0x02, 0x02, 0x02,
+    0x03, 0x02, 0x1c, 0x00}
+};
+
 static GstCaps *
 mxf_mpeg_es_create_caps (MXFMetadataGenericPackage * package,
     MXFMetadataTrack * track, GstTagList ** tags,
     MXFEssenceElementHandler * handler, gpointer * mapping_data,
-    MXFMetadataFileDescriptor * f,
     MXFMetadataGenericPictureEssenceDescriptor * p,
-    MXFMetadataGenericSoundEssenceDescriptor * s,
+    MXFMetadataGenericSoundEssenceDescriptor * s)
-    MXFMetadataMPEGVideoDescriptor * d)
 {
   GstCaps *caps = NULL;
   const gchar *codec_name = NULL;
 
   /* SMPTE RP224 */
-  if (!p || mxf_ul_is_zero (&p->picture_essence_coding)) {
+  if (p) {
-    GST_WARNING
+    if (mxf_ul_is_zero (&p->picture_essence_coding)) {
-        ("No picture essence descriptor or no picture essence coding defined");
+      GST_WARNING ("No picture essence coding defined, assuming MPEG2");
       caps =
           gst_caps_new_simple ("video/mpeg", "mpegversion", G_TYPE_INT, 2,
           "systemstream", G_TYPE_BOOLEAN, FALSE, NULL);
@@ -289,9 +337,9 @@ mxf_mpeg_es_create_caps (MXFMetadataGenericPackage * package,
       codec_name = "MPEG-1 Video";
     } else if (p->picture_essence_coding.u[13] == 0x20) {
       MXFLocalTag *local_tag =
-        (((MXFMetadataGenericDescriptor *) f)->other_tags) ?
+          (((MXFMetadataGenericDescriptor *) p)->other_tags) ?
           g_hash_table_lookup (((MXFMetadataGenericDescriptor *)
-            f)->other_tags, &sony_mpeg4_extradata) : NULL;
+              p)->other_tags, &sony_mpeg4_extradata) : NULL;
 
       caps = gst_caps_new_simple ("video/mpeg", "mpegversion", G_TYPE_INT, 4,
           "systemstream", G_TYPE_BOOLEAN, FALSE, NULL);
@@ -310,9 +358,56 @@ mxf_mpeg_es_create_caps (MXFMetadataGenericPackage * package,
           p->picture_essence_coding.u[13]);
       caps = NULL;
     }
+    if (caps)
+      *handler = mxf_mpeg_video_handle_essence_element;
+  } else if (s) {
+    if (mxf_ul_is_zero (&s->sound_essence_compression)) {
+      GST_WARNING ("Zero sound essence compression, assuming MPEG1 audio");
+      caps =
+          gst_caps_new_simple ("audio/mpeg", "mpegversion", G_TYPE_INT, 1,
+          NULL);
+    } else if (mxf_ul_is_equal (&s->sound_essence_compression,
+            &sound_essence_compression_ac3)) {
+      caps = gst_caps_new_simple ("audio/ac3", NULL);
+    } else if (mxf_ul_is_equal (&s->sound_essence_compression,
+            &sound_essence_compression_mpeg1_layer1)) {
+      caps =
+          gst_caps_new_simple ("audio/mpeg", "mpegversion", G_TYPE_INT, 1,
+          "layer", G_TYPE_INT, 1, NULL);
+    } else if (mxf_ul_is_equal (&s->sound_essence_compression,
+            &sound_essence_compression_mpeg1_layer12)) {
+      caps =
+          gst_caps_new_simple ("audio/mpeg", "mpegversion", G_TYPE_INT, 1,
+          NULL);
+    } else if (mxf_ul_is_equal (&s->sound_essence_compression,
+            &sound_essence_compression_mpeg1_layer2)) {
+      caps =
+          gst_caps_new_simple ("audio/mpeg", "mpegversion", G_TYPE_INT, 1,
+          "layer", G_TYPE_INT, 2, NULL);
+    } else if (mxf_ul_is_equal (&s->sound_essence_compression,
+            &sound_essence_compression_mpeg2_layer1)) {
+      caps =
+          gst_caps_new_simple ("audio/mpeg", "mpegversion", G_TYPE_INT, 1,
+          "layer", G_TYPE_INT, 1, "mpegaudioversion", G_TYPE_INT, 2, NULL);
+    } else if (mxf_ul_is_equal (&s->sound_essence_compression,
+            &sound_essence_compression_dts)) {
+      caps = gst_caps_new_simple ("audio/x-dts", NULL);
+    }
+
+    if (caps) {
+      if (s->audio_sampling_rate.n != 0 && s->audio_sampling_rate.d != 0)
+        gst_caps_set_simple (caps, "rate", G_TYPE_INT,
+            (gint) (((gdouble) s->audio_sampling_rate.n) /
+                ((gdouble) s->audio_sampling_rate.d) + 0.5), NULL);
+      if (s->channel_count != 0)
+        gst_caps_set_simple (caps, "channels", G_TYPE_INT, s->channel_count,
+            NULL);
+
+      *handler = mxf_mpeg_audio_handle_essence_element;
+    }
+  }
 
   if (caps) {
-    *handler = mxf_mpeg_video_handle_essence_element;
     if (!*tags)
       *tags = gst_tag_list_new ();
     if (codec_name)
@@ -328,7 +423,6 @@ mxf_mpeg_create_caps (MXFMetadataGenericPackage * package,
     MXFMetadataTrack * track, GstTagList ** tags,
     MXFEssenceElementHandler * handler, gpointer * mapping_data)
 {
-  MXFMetadataMPEGVideoDescriptor *d = NULL;
   MXFMetadataFileDescriptor *f = NULL;
   MXFMetadataGenericPictureEssenceDescriptor *p = NULL;
   MXFMetadataGenericSoundEssenceDescriptor *s = NULL;
@@ -345,12 +439,8 @@ mxf_mpeg_create_caps (MXFMetadataGenericPackage * package,
 
   for (i = 0; i < track->n_descriptor; i++) {
     if (((MXFMetadataGenericDescriptor *) track->descriptor[i])->type ==
-        MXF_METADATA_MPEG_VIDEO_DESCRIPTOR) {
+        MXF_METADATA_MPEG_VIDEO_DESCRIPTOR ||
-      d = (MXFMetadataMPEGVideoDescriptor *) track->descriptor[i];
+        ((MXFMetadataGenericDescriptor *) track->descriptor[i])->type ==
-      f = track->descriptor[i];
-      p = (MXFMetadataGenericPictureEssenceDescriptor *) track->descriptor[i];
-      break;
-    } else if (((MXFMetadataGenericDescriptor *) track->descriptor[i])->type ==
         MXF_METADATA_CDCI_PICTURE_ESSENCE_DESCRIPTOR ||
         ((MXFMetadataGenericDescriptor *) track->descriptor[i])->type ==
         MXF_METADATA_GENERIC_PICTURE_ESSENCE_DESCRIPTOR) {
@@ -362,9 +452,6 @@ mxf_mpeg_create_caps (MXFMetadataGenericPackage * package,
       f = track->descriptor[i];
       s = (MXFMetadataGenericSoundEssenceDescriptor *) track->descriptor[i];
       break;
-    } else if (((MXFMetadataGenericDescriptor *) track->descriptor[i])->type ==
-        MXF_METADATA_FILE_DESCRIPTOR) {
-      f = track->descriptor[i];
     }
   }
 
@@ -378,8 +465,8 @@ mxf_mpeg_create_caps (MXFMetadataGenericPackage * package,
     GST_DEBUG ("Found MPEG ES stream");
 
     caps =
-        mxf_mpeg_es_create_caps (package, track, tags, handler, mapping_data, f,
+        mxf_mpeg_es_create_caps (package, track, tags, handler, mapping_data, p,
-        p, s, d);
+        s);
   } else if (f->essence_container.u[13] == 0x07) {
     GST_ERROR ("MPEG PES streams not supported yet");
     return NULL;