Menu
▾
▴
[gst-cvs] gst-plugins-base: playbin2: actually use configured audio/video sinks
|
From: <tp...@ke...> - 2009-05-28 14:40:54
|
Module: gst-plugins-base Branch: master Commit: 7620d8800de108a686c61f64787e27e98bfaf3e2 URL: http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=7620d8800de108a686c61f64787e27e98bfaf3e2 Author: Tim-Philipp Müller <tim...@co...> Date: Thu May 28 15:21:42 2009 +0100 playbin2: actually use configured audio/video sinks playbin2 inadvertently used autoaudiosink and autovideosink up to now, since it would overwrite the sinks configured via the "audio-sink" and "video-sink" properties with the stream-specific group sinks when configuring the outputs. Those are usually NULL however, so that would overwrite the configured sinks with NULL which makes playbin2 then default to the auto sinks. Fix this by keeping a reference to each configured sink in playbin2 and setting up the right sinks depending on whether there is a stream-specific sink or not. Fixes #584020. --- gst/playback/gstplaybin2.c | 103 +++++++++++++++++++++++++++++++++++++------ gst/playback/gstplaysink.c | 9 ++-- 2 files changed, 93 insertions(+), 19 deletions(-) diff --git a/gst/playback/gstplaybin2.c b/gst/playback/gstplaybin2.c index 1be6c2e..640285c 100644 --- a/gst/playback/gstplaybin2.c +++ b/gst/playback/gstplaybin2.c @@ -376,6 +376,11 @@ struct _GstPlayBin gboolean have_selector; /* set to FALSE when we fail to create an * input-selector, so that we only post a * warning once */ + + GstElement *audio_sink; /* configured audio sink, or NULL */ + GstElement *video_sink; /* configured video sink, or NULL */ + GstElement *subpic_sink; /* configured subpicture sink, or NULL */ + GstElement *text_sink; /* configured text sink, or NULL */ }; struct _GstPlayBinClass @@ -1049,6 +1054,14 @@ gst_play_bin_finalize (GObject * object) if (playbin->source) gst_object_unref (playbin->source); + if (playbin->video_sink) + gst_object_unref (playbin->video_sink); + if (playbin->audio_sink) + gst_object_unref (playbin->audio_sink); + if (playbin->text_sink) + gst_object_unref (playbin->text_sink); + if (playbin->subpic_sink) + gst_object_unref (playbin->subpic_sink); g_value_array_free (playbin->elements); g_free (playbin->encoding); @@ -1416,6 +1429,29 @@ gst_play_bin_set_encoding (GstPlayBin * playbin, const gchar * encoding) } static void +gst_play_bin_set_sink (GstPlayBin * playbin, GstElement ** elem, + const gchar * dbg, GstElement * sink) +{ + GST_INFO_OBJECT (playbin, "Setting %s sink to %" GST_PTR_FORMAT, dbg, sink); + + GST_PLAY_BIN_LOCK (playbin); + if (*elem != sink) { + GstElement *old; + + old = *elem; + if (sink) { + gst_object_ref (sink); + gst_object_sink (sink); + } + *elem = sink; + if (old) + gst_object_unref (old); + } + GST_LOG_OBJECT (playbin, "%s sink now %" GST_PTR_FORMAT, dbg, *elem); + GST_PLAY_BIN_UNLOCK (playbin); +} + +static void gst_play_bin_set_property (GObject * object, guint prop_id, const GValue * value, GParamSpec * pspec) { @@ -1447,11 +1483,11 @@ gst_play_bin_set_property (GObject * object, guint prop_id, gst_play_bin_set_encoding (playbin, g_value_get_string (value)); break; case PROP_VIDEO_SINK: - gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_VIDEO, + gst_play_bin_set_sink (playbin, &playbin->video_sink, "video", g_value_get_object (value)); break; case PROP_AUDIO_SINK: - gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_AUDIO, + gst_play_bin_set_sink (playbin, &playbin->audio_sink, "audio", g_value_get_object (value)); break; case PROP_VIS_PLUGIN: @@ -1459,11 +1495,11 @@ gst_play_bin_set_property (GObject * object, guint prop_id, g_value_get_object (value)); break; case PROP_TEXT_SINK: - gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_TEXT, + gst_play_bin_set_sink (playbin, &playbin->text_sink, "text", g_value_get_object (value)); break; case PROP_SUBPIC_SINK: - gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_SUBPIC, + gst_play_bin_set_sink (playbin, &playbin->subpic_sink, "subpicture", g_value_get_object (value)); break; case PROP_VOLUME: @@ -1493,6 +1529,24 @@ gst_play_bin_set_property (GObject * object, guint prop_id, } } +static GstElement * +gst_play_bin_get_current_sink (GstPlayBin * playbin, GstElement ** elem, + const gchar * dbg, GstPlaySinkType type) +{ + GstElement *sink; + + sink = gst_play_sink_get_sink (playbin->playsink, type); + + GST_LOG_OBJECT (playbin, "play_sink_get_sink() returned %s sink %" + GST_PTR_FORMAT ", the originally set %s sink is %" GST_PTR_FORMAT, + dbg, sink, dbg, *elem); + + if (sink == NULL) + sink = *elem; + + return sink; +} + static void gst_play_bin_get_property (GObject * object, guint prop_id, GValue * value, GParamSpec * pspec) @@ -1590,11 +1644,13 @@ gst_play_bin_get_property (GObject * object, guint prop_id, GValue * value, break; case PROP_VIDEO_SINK: g_value_set_object (value, - gst_play_sink_get_sink (playbin->playsink, GST_PLAY_SINK_TYPE_VIDEO)); + gst_play_bin_get_current_sink (playbin, &playbin->video_sink, + "video", GST_PLAY_SINK_TYPE_VIDEO)); break; case PROP_AUDIO_SINK: g_value_set_object (value, - gst_play_sink_get_sink (playbin->playsink, GST_PLAY_SINK_TYPE_AUDIO)); + gst_play_bin_get_current_sink (playbin, &playbin->audio_sink, + "audio", GST_PLAY_SINK_TYPE_AUDIO)); break; case PROP_VIS_PLUGIN: g_value_set_object (value, @@ -1602,12 +1658,13 @@ gst_play_bin_get_property (GObject * object, guint prop_id, GValue * value, break; case PROP_TEXT_SINK: g_value_set_object (value, - gst_play_sink_get_sink (playbin->playsink, GST_PLAY_SINK_TYPE_TEXT)); + gst_play_bin_get_current_sink (playbin, &playbin->text_sink, + "text", GST_PLAY_SINK_TYPE_TEXT)); break; case PROP_SUBPIC_SINK: g_value_set_object (value, - gst_play_sink_get_sink (playbin->playsink, - GST_PLAY_SINK_TYPE_SUBPIC)); + gst_play_bin_get_current_sink (playbin, &playbin->subpic_sink, + "subpicture", GST_PLAY_SINK_TYPE_SUBPIC)); break; case PROP_VOLUME: g_value_set_double (value, gst_play_sink_get_volume (playbin->playsink)); @@ -2012,12 +2069,28 @@ no_more_pads_cb (GstElement * decodebin, GstSourceGroup * group) if (configure) { /* if we have custom sinks, configure them now */ GST_SOURCE_GROUP_LOCK (group); - GST_LOG_OBJECT (playbin, "setting custom audio sink %p", group->audio_sink); - gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_AUDIO, - group->audio_sink); - GST_LOG_OBJECT (playbin, "setting custom video sink %p", group->video_sink); - gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_VIDEO, - group->video_sink); + if (group->audio_sink) { + GST_INFO_OBJECT (playbin, "setting custom audio sink %" GST_PTR_FORMAT, + group->audio_sink); + gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_AUDIO, + group->audio_sink); + } else { + GST_INFO_OBJECT (playbin, "setting default audio sink %" GST_PTR_FORMAT, + playbin->audio_sink); + gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_AUDIO, + playbin->audio_sink); + } + if (group->video_sink) { + GST_INFO_OBJECT (playbin, "setting custom video sink %" GST_PTR_FORMAT, + group->video_sink); + gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_VIDEO, + group->video_sink); + } else { + GST_INFO_OBJECT (playbin, "setting default video sink %" GST_PTR_FORMAT, + playbin->video_sink); + gst_play_sink_set_sink (playbin->playsink, GST_PLAY_SINK_TYPE_VIDEO, + playbin->video_sink); + } GST_SOURCE_GROUP_UNLOCK (group); GST_LOG_OBJECT (playbin, "reconfigure sink"); diff --git a/gst/playback/gstplaysink.c b/gst/playback/gstplaysink.c index ca77180..0f0322f 100644 --- a/gst/playback/gstplaysink.c +++ b/gst/playback/gstplaysink.c @@ -339,6 +339,8 @@ gst_play_sink_set_sink (GstPlaySink * playsink, GstPlaySinkType type, { GstElement **elem = NULL, *old = NULL; + GST_LOG ("Setting sink %" GST_PTR_FORMAT " as sink type %d", sink, type); + GST_PLAY_SINK_LOCK (playsink); switch (type) { case GST_PLAY_SINK_TYPE_AUDIO: @@ -360,10 +362,8 @@ gst_play_sink_set_sink (GstPlaySink * playsink, GstPlaySinkType type, } if (elem) { old = *elem; - if (sink) { + if (sink) gst_object_ref (sink); - gst_object_sink (sink); - } *elem = sink; } GST_PLAY_SINK_UNLOCK (playsink); @@ -1327,7 +1327,8 @@ gen_audio_chain (GstPlaySink * playsink, gboolean raw, gboolean queue) GST_DEBUG_OBJECT (playsink, "making audio chain %p", chain); if (playsink->audio_sink) { - GST_DEBUG_OBJECT (playsink, "trying configured audiosink"); + GST_DEBUG_OBJECT (playsink, "trying configured audiosink %" GST_PTR_FORMAT, + playsink->audio_sink); chain->sink = try_element (playsink, playsink->audio_sink, FALSE); } if (chain->sink == NULL) { |
