summaryrefslogtreecommitdiffstats
path: root/gst/mxf/mxfdemux.c
diff options
context:
space:
mode:
authorSebastian Dröge <slomo@circular-chaos.org>2008-12-16 12:30:34 +0000
committerSebastian Dröge <slomo@circular-chaos.org>2008-12-16 12:30:34 +0000
commit5587f912104a8b4b7d3fbc71c9b7e39981235b43 (patch)
treec9b1ceb88f30576f7f065d93b14013c5c425067c /gst/mxf/mxfdemux.c
parent2bdd84b48d1ee8a64130efca7a953711614431b4 (diff)
downloadgst-plugins-bad-5587f912104a8b4b7d3fbc71c9b7e39981235b43.tar.gz
gst-plugins-bad-5587f912104a8b4b7d3fbc71c9b7e39981235b43.tar.bz2
gst-plugins-bad-5587f912104a8b4b7d3fbc71c9b7e39981235b43.zip
gst/mxf/: Implement an essence element handling registry and use this instead of an hardcoded, large if-then-else block.
Original commit message from CVS: * gst/mxf/mxfaes-bwf.c: (mxf_is_aes_bwf_essence_track), (mxf_bwf_handle_essence_element), (mxf_aes3_handle_essence_element), (mxf_bwf_create_caps), (mxf_aes3_create_caps), (mxf_aes_bwf_create_caps), (mxf_aes_bwf_init): * gst/mxf/mxfaes-bwf.h: * gst/mxf/mxfalaw.c: (mxf_is_alaw_essence_track), (mxf_alaw_handle_essence_element), (mxf_alaw_create_caps), (mxf_alaw_init): * gst/mxf/mxfalaw.h: * gst/mxf/mxfd10.c: (mxf_is_d10_essence_track), (mxf_d10_picture_handle_essence_element), (mxf_d10_sound_handle_essence_element), (mxf_d10_create_caps), (mxf_d10_init): * gst/mxf/mxfd10.h: * gst/mxf/mxfdemux.c: (gst_mxf_demux_reset_metadata), (gst_mxf_demux_handle_header_metadata_update_streams), (gst_mxf_demux_handle_generic_container_essence_element): * gst/mxf/mxfdv-dif.c: (mxf_is_dv_dif_essence_track), (mxf_dv_dif_handle_essence_element), (mxf_dv_dif_create_caps), (mxf_dv_dif_init): * gst/mxf/mxfdv-dif.h: * gst/mxf/mxfjpeg2000.c: (mxf_is_jpeg2000_essence_track), (mxf_jpeg2000_handle_essence_element), (mxf_jpeg2000_create_caps), (mxf_jpeg2000_init): * gst/mxf/mxfjpeg2000.h: * gst/mxf/mxfmpeg.c: (mxf_is_mpeg_essence_track), (mxf_mpeg_video_handle_essence_element), (mxf_mpeg_audio_handle_essence_element), (mxf_mpeg_es_create_caps), (mxf_mpeg_create_caps), (mxf_mpeg_init): * gst/mxf/mxfmpeg.h: * gst/mxf/mxfparse.c: (mxf_essence_element_handler_register), (mxf_essence_element_handler_find): * gst/mxf/mxfparse.h: * gst/mxf/mxfup.c: (mxf_is_up_essence_track), (mxf_up_handle_essence_element), (mxf_up_rgba_create_caps), (mxf_up_create_caps), (mxf_up_init): * gst/mxf/mxfup.h: Implement an essence element handling registry and use this instead of an hardcoded, large if-then-else block.
Diffstat (limited to 'gst/mxf/mxfdemux.c')
-rw-r--r--gst/mxf/mxfdemux.c122
1 files changed, 24 insertions, 98 deletions
diff --git a/gst/mxf/mxfdemux.c b/gst/mxf/mxfdemux.c
index b2f43ed8..4b1a3ba1 100644
--- a/gst/mxf/mxfdemux.c
+++ b/gst/mxf/mxfdemux.c
@@ -43,8 +43,6 @@
* timestamp calculation, etc.
* - Handle timecode tracks correctly (where is this documented?)
* - Handle Generic container system items
- * - Use an "essence element/track handling" registry instead of the large if-then-else
- * block when detecting the codec
* - Force synchronization of tracks. Packets that have the timestamp are not required
* to be stored at the same position in the essence stream, especially if tracks
* with different source packages (body sid) are used.
@@ -69,13 +67,6 @@
#include "mxfdemux.h"
#include "mxfparse.h"
#include "mxfmetadata.h"
-#include "mxfaes-bwf.h"
-#include "mxfmpeg.h"
-#include "mxfdv-dif.h"
-#include "mxfalaw.h"
-#include "mxfjpeg2000.h"
-#include "mxfd10.h"
-#include "mxfup.h"
#include <string.h>
@@ -109,8 +100,9 @@ typedef struct
GstFlowReturn last_flow;
guint64 essence_element_count;
- MXFEssenceElementHandler handle_essence_element;
gpointer mapping_data;
+ const MXFEssenceElementHandler *handler;
+ MXFEssenceElementHandleFunc handle_func;
GstTagList *tags;
@@ -235,6 +227,9 @@ gst_mxf_demux_reset_metadata (GstMXFDemux * demux)
for (i = 0; i < demux->src->len; i++) {
GstMXFDemuxPad *pad = g_ptr_array_index (demux->src, i);
+ pad->handler = NULL;
+ pad->handle_func = NULL;
+
pad->material_track = NULL;
pad->material_package = NULL;
pad->component = NULL;
@@ -559,11 +554,11 @@ gst_mxf_demux_choose_package (GstMXFDemux * demux)
for (i = 0; i < demux->preface->content_storage->n_packages; i++) {
if (demux->preface->content_storage->packages[i] &&
- MXF_IS_METADATA_MATERIAL_PACKAGE (demux->preface->content_storage->
- packages[i])) {
+ MXF_IS_METADATA_MATERIAL_PACKAGE (demux->preface->
+ content_storage->packages[i])) {
ret =
- MXF_METADATA_GENERIC_PACKAGE (demux->preface->content_storage->
- packages[i]);
+ MXF_METADATA_GENERIC_PACKAGE (demux->preface->
+ content_storage->packages[i]);
break;
}
}
@@ -736,90 +731,23 @@ gst_mxf_demux_handle_header_metadata_update_streams (GstMXFDemux * demux)
pad->source_package = source_package;
pad->source_track = source_track;
- pad->handle_essence_element = NULL;
+ pad->handler = NULL;
g_free (pad->mapping_data);
+ pad->handle_func = NULL;
pad->mapping_data = NULL;
- switch (track->parent.type) {
- case MXF_METADATA_TRACK_PICTURE_ESSENCE:
- if (mxf_is_mpeg_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_mpeg_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_dv_dif_essence_track ((MXFMetadataTrack *)
- source_track))
- caps =
- mxf_dv_dif_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_jpeg2000_essence_track ((MXFMetadataTrack *)
- source_track))
- caps =
- mxf_jpeg2000_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_d10_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_d10_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_up_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_up_create_caps (MXF_METADATA_GENERIC_PACKAGE (source_package),
- (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- break;
- case MXF_METADATA_TRACK_SOUND_ESSENCE:
- if (mxf_is_aes_bwf_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_aes_bwf_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_dv_dif_essence_track ((MXFMetadataTrack *)
- source_track))
- caps =
- mxf_dv_dif_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_alaw_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_alaw_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_mpeg_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_mpeg_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_d10_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_d10_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- break;
- case MXF_METADATA_TRACK_DATA_ESSENCE:
- if (mxf_is_dv_dif_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_dv_dif_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_mpeg_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_mpeg_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- else if (mxf_is_d10_essence_track ((MXFMetadataTrack *) source_track))
- caps =
- mxf_d10_create_caps (MXF_METADATA_GENERIC_PACKAGE
- (source_package), (MXFMetadataTrack *) source_track, &pad->tags,
- &pad->handle_essence_element, &pad->mapping_data);
- break;
- default:
- g_assert_not_reached ();
- break;
+ pad->handler = mxf_essence_element_handler_find (source_track);
+
+ if (!pad->handler) {
+ GST_WARNING_OBJECT (demux, "No essence element handler for track found");
+ gst_object_unref (pad);
+ continue;
}
+ caps =
+ pad->handler->create_caps (source_track, &pad->tags, &pad->handle_func,
+ &pad->mapping_data);
+
if (!caps) {
GST_WARNING_OBJECT (demux, "No caps created, ignoring stream");
gst_object_unref (pad);
@@ -1081,13 +1009,11 @@ gst_mxf_demux_handle_generic_container_essence_element (GstMXFDemux * demux,
GST_BUFFER_OFFSET_END (inbuf) = GST_BUFFER_OFFSET_NONE;
gst_buffer_set_caps (inbuf, pad->caps);
- if (pad->handle_essence_element) {
+ if (pad->handle_func) {
/* Takes ownership of inbuf */
ret =
- pad->handle_essence_element (key, inbuf, pad->caps,
- (MXFMetadataGenericPackage *) pad->source_package,
- (MXFMetadataTrack *) pad->source_track, pad->component,
- pad->mapping_data, &outbuf);
+ pad->handle_func (key, inbuf, pad->caps,
+ pad->source_track, pad->component, pad->mapping_data, &outbuf);
inbuf = NULL;
} else {
outbuf = inbuf;
generated by cgit v1.2.1 (git 2.18.0) at 2025-03-09 14:58:24 +0000