added func gst_play_set_data_src removed pipeline GST_PLAY_PIPE_VIDEO_THREADSAFE...

Original commit message from CVS:
added func gst_play_set_data_src
removed pipeline GST_PLAY_PIPE_VIDEO_THREADSAFE - it never worked and its just taking up space

diff --git a/gst-libs/gst/play/play.old.c b/gst-libs/gst/play/play.old.c
index e375979..4d3daec 100644 (file)
--- a/gst-libs/gst/play/play.old.c
+++ b/gst-libs/gst/play/play.old.c
@@ -312,21 +312,16 @@ gst_play_new (GstPlayPipeType pipe_type, GError **error)
                case GST_PLAY_PIPE_VIDEO:
                        play->setup_pipeline = gst_play_video_setup;
                        play->teardown_pipeline = NULL;
+                       play->set_data_src = gst_play_video_set_data_src;
                        play->set_autoplugger = gst_play_video_set_auto;
                        play->set_video_sink = gst_play_video_set_video;
                        play->set_audio_sink = gst_play_video_set_audio;
                        break;
-               case GST_PLAY_PIPE_VIDEO_THREADSAFE:
-                       play->setup_pipeline = gst_play_videots_setup;
-                       play->teardown_pipeline = NULL;
-                       play->set_autoplugger = gst_play_videots_set_auto;
-                       play->set_video_sink = gst_play_videots_set_video;
-                       play->set_audio_sink = gst_play_videots_set_audio;
-                       break;
                case GST_PLAY_PIPE_AUDIO:
                        /* we can reuse the threaded set functions */
                        play->setup_pipeline = gst_play_audio_setup;
                        play->teardown_pipeline = NULL;
+                       play->set_data_src = gst_play_simple_set_data_src;
                        play->set_autoplugger = gst_play_audiot_set_auto;
                        play->set_video_sink = NULL;
                        play->set_audio_sink = gst_play_audiot_set_audio;
@@ -334,6 +329,7 @@ gst_play_new (GstPlayPipeType pipe_type, GError **error)
                case GST_PLAY_PIPE_AUDIO_THREADED:
                        play->setup_pipeline = gst_play_audiot_setup;
                        play->teardown_pipeline = NULL;
+                       play->set_data_src = gst_play_simple_set_data_src;
                        play->set_autoplugger = gst_play_audiot_set_auto;
                        play->set_video_sink = NULL;
                        play->set_audio_sink = gst_play_audiot_set_audio;
@@ -341,6 +337,7 @@ gst_play_new (GstPlayPipeType pipe_type, GError **error)
                case GST_PLAY_PIPE_AUDIO_HYPER_THREADED:
                        play->setup_pipeline = gst_play_audioht_setup;
                        play->teardown_pipeline = NULL;
+                       play->set_data_src = gst_play_simple_set_data_src;
                        play->set_autoplugger = gst_play_audioht_set_auto;
                        play->set_video_sink = NULL;
                        play->set_audio_sink = gst_play_audioht_set_audio;
@@ -808,6 +805,24 @@ gst_play_get_sink_element (GstPlay *play, GstElement *element){
 }
 
 gboolean
+gst_play_set_data_src (GstPlay *play, GstElement *data_src)
+{
+       g_return_val_if_fail (GST_IS_PLAY (play), FALSE);
+       g_return_val_if_fail (GST_IS_ELEMENT (data_src), FALSE);
+
+       if (gst_play_get_state (play) != GST_STATE_READY){
+               gst_play_set_state (play, GST_STATE_READY);
+       }
+
+       if (play->set_data_src){
+               return play->set_data_src(play, data_src);
+       }
+
+       /* if there is no set_data_src func, fail quietly */
+       return FALSE;
+}
+
+gboolean
 gst_play_set_video_sink (GstPlay *play, GstElement *video_sink)
 {
        g_return_val_if_fail (GST_IS_PLAY (play), FALSE);

diff --git a/gst-libs/gst/play/play.old.h b/gst-libs/gst/play/play.old.h
index 02d55e0..acf777c 100644 (file)
--- a/gst-libs/gst/play/play.old.h
+++ b/gst-libs/gst/play/play.old.h
@@ -50,7 +50,6 @@ typedef enum {
        GST_PLAY_PIPE_AUDIO,
        GST_PLAY_PIPE_AUDIO_THREADED,
        GST_PLAY_PIPE_AUDIO_HYPER_THREADED,
-       GST_PLAY_PIPE_VIDEO_THREADSAFE,
        GST_PLAY_PIPE_VIDEO,
 } GstPlayPipeType;
 
@@ -85,8 +84,9 @@ struct _GstPlay
        GObject parent;
        
        gboolean (*setup_pipeline)     (GstPlay *play, GError **error);
-       void (*teardown_pipeline)  (GstPlay *play);
-       gboolean (*set_autoplugger)  (GstPlay *play, GstElement *autoplugger);
+       void (*teardown_pipeline)      (GstPlay *play);
+       gboolean (*set_data_src)       (GstPlay *play, GstElement *datasrc);
+       gboolean (*set_autoplugger)    (GstPlay *play, GstElement *autoplugger);
        gboolean (*set_video_sink)     (GstPlay *play, GstElement *videosink);
        gboolean (*set_audio_sink)     (GstPlay *play, GstElement *audiosink);
        
@@ -155,6 +155,7 @@ void      gst_play_seek_to_time (GstPlay *play, gint64 time_nanos);
 
 GstElement*       gst_play_get_sink_element (GstPlay *play, GstElement *element);
 
+gboolean         gst_play_set_data_src (GstPlay *play, GstElement *data_src);
 gboolean         gst_play_set_video_sink  (GstPlay *play, GstElement *element);
 gboolean         gst_play_set_audio_sink  (GstPlay *play, GstElement *element);
 void             gst_play_need_new_video_window  (GstPlay *play);

diff --git a/gst-libs/gst/play/playpipelines.c b/gst-libs/gst/play/playpipelines.c
index 67e4d5a..527ea43 100644 (file)
--- a/gst-libs/gst/play/playpipelines.c
+++ b/gst-libs/gst/play/playpipelines.c
@@ -21,6 +21,30 @@
  * Boston, MA 02111-1307, USA.
  */
 
+static gboolean
+gst_play_default_set_data_src (GstPlay *play, GstElement *datasrc, GstElement* parent)
+{
+       g_return_val_if_fail (GST_IS_PLAY (play), FALSE);
+       g_return_val_if_fail (GST_IS_ELEMENT (datasrc), FALSE);
+       
+       if (GST_IS_ELEMENT(play->source)) {
+               /* we need to remove the existing data source before creating a new one */
+               if (GST_IS_ELEMENT(play->autoplugger)){
+                       gst_element_unlink (play->autoplugger, play->source);
+               }
+               gst_bin_remove (GST_BIN(parent), play->source);
+       }
+       
+       play->source = datasrc;
+       g_return_val_if_fail (play->source != NULL, FALSE);
+
+       gst_bin_add (GST_BIN (parent), play->source);
+       if (GST_IS_ELEMENT(play->autoplugger)){
+               gst_element_link (play->autoplugger, play->source);
+       }
+       return TRUE;
+}
+
 /*  
  *  GST_PLAY_PIPE_AUDIO
  *  gnomevfssrc ! spider ! volume ! osssink
@@ -80,6 +104,12 @@ gst_play_audio_setup (GstPlay *play, GError **error)
        return TRUE;
 }
 
+static gboolean
+gst_play_simple_set_data_src (GstPlay *play, GstElement *datasrc)
+{
+       return gst_play_default_set_data_src(play, datasrc, play->pipeline);
+}
+
 /*
  *  GST_PLAY_PIPE_AUDIO_THREADED
  *  { gnomevfssrc ! spider ! volume ! osssink }
@@ -492,6 +522,15 @@ gst_play_video_setup (GstPlay *play, GError **error)
        return TRUE;
 }
 
+static gboolean
+gst_play_video_set_data_src (GstPlay *play, GstElement *datasrc)
+{
+       GstElement *work_thread;
+       g_return_val_if_fail (GST_IS_PLAY(play), FALSE);
+
+       work_thread = g_hash_table_lookup(play->other_elements, "work_thread");
+       return gst_play_default_set_data_src(play, datasrc, work_thread);
+}
 
 static gboolean
 gst_play_video_set_auto (GstPlay *play, GstElement *autoplugger)
@@ -589,235 +628,5 @@ gst_play_video_set_audio (GstPlay *play, GstElement *audio_sink)
        return TRUE;
 }
 
-/*
- * GST_PLAY_PIPE_VIDEO_THREADSAFE
- * { gnomevfssrc ! spider ! { queue ! volume ! osssink } } 
- * spider0.src2 ! queue ! videosink
- * (note that the xvideosink is not contained by a thread)
- */
-
-static gboolean 
-gst_play_videots_setup (GstPlay *play, GError **error)
-{
-       GstElement *audio_bin, *audio_queue, *video_queue, *auto_identity, *work_thread;
-
-       g_return_val_if_fail (GST_IS_PLAY(play), FALSE);
-       
-       /* creating pipeline */
-       play->pipeline = gst_pipeline_new ("main_pipeline");
-       g_return_val_if_fail (GST_IS_PIPELINE (play->pipeline), FALSE);
-
-       /* creating work thread */      
-       work_thread = gst_thread_new ("work_thread");
-       g_return_val_if_fail (GST_IS_THREAD (work_thread), FALSE);
-       g_hash_table_insert(play->other_elements, "work_thread", work_thread);
-       
-       gst_bin_add (GST_BIN (play->pipeline), work_thread);
-
-       /* create source element */
-       play->source = gst_element_factory_make ("gnomevfssrc", "source");
-       if (!play->source)
-       {
-               gst_play_error_plugin (GST_PLAY_ERROR_GNOMEVFSSRC, error);
-               return FALSE;
-       }
-       gst_bin_add (GST_BIN (work_thread), play->source);
-       
-       auto_identity = gst_element_factory_make ("identity", "auto_identity");
-       g_return_val_if_fail (auto_identity != NULL, FALSE);
-       g_hash_table_insert(play->other_elements, "auto_identity", auto_identity);
-
-       gst_bin_add (GST_BIN (work_thread), auto_identity);
-       gst_element_add_ghost_pad (work_thread, 
-                                  gst_element_get_pad (auto_identity, "src"),
-                                  "src");
-       
-       /* create volume elements */
-       play->volume = gst_element_factory_make ("volume", "volume");
-       if (!play->volume)
-       {
-               gst_play_error_plugin (GST_PLAY_ERROR_VOLUME, error);
-               return FALSE;
-       }
-
-       /* creating audio_sink element */
-       play->audio_sink = gst_element_factory_make ("fakesink", "fake_audio");
-       if (!play->audio_sink)
-       {
-               gst_play_error_plugin (GST_PLAY_ERROR_FAKESINK, error);
-               return FALSE;
-       }
-       play->audio_sink_element = NULL;
-       
-       audio_queue = gst_element_factory_make ("queue", "audio_queue");
-       if (!audio_queue)
-       {
-               gst_play_error_plugin (GST_PLAY_ERROR_QUEUE, error);
-               return FALSE;
-       }
-       g_hash_table_insert(play->other_elements, "audio_queue", audio_queue);
-       
-       audio_bin = gst_thread_new ("audio_bin");
-       if (!audio_bin)
-       {
-               gst_play_error_plugin (GST_PLAY_ERROR_THREAD, error);
-               return FALSE;
-       }
-       g_hash_table_insert(play->other_elements, "audio_bin", audio_bin);
-
-       gst_bin_set_pre_iterate_function(
-                       GST_BIN (audio_bin), 
-                       (GstBinPrePostIterateFunction) callback_bin_pre_iterate,
-                       play->audio_bin_mutex);
-       
-       gst_bin_set_post_iterate_function(
-                       GST_BIN (audio_bin), 
-                       (GstBinPrePostIterateFunction) callback_bin_post_iterate,
-                       play->audio_bin_mutex);
-
-       gst_bin_add_many (
-                       GST_BIN (audio_bin), audio_queue,
-                       play->volume, play->audio_sink, NULL);
-       
-       gst_element_link_many (
-                       audio_queue, play->volume,
-                       play->audio_sink, NULL);
-       
-       gst_element_add_ghost_pad (
-                       audio_bin, 
-                       gst_element_get_pad (audio_queue, "sink"),
-                       "sink");
-
-       gst_bin_add (GST_BIN (work_thread), audio_bin);
-
-       /* create video elements */
-       play->video_sink = gst_element_factory_make ("xvideosink", "show");
-       
-       g_object_set (G_OBJECT (play->video_sink), "toplevel", FALSE, NULL);
-       
-       g_signal_connect (
-                       G_OBJECT (play->video_sink), "have_xid",
-                       G_CALLBACK (callback_video_have_xid), play);
-       
-       g_signal_connect (
-                       G_OBJECT (play->video_sink), "have_size",
-                       G_CALLBACK (callback_video_have_size), play);
-
-       g_return_val_if_fail (play->video_sink != NULL, FALSE);
-       
-       video_queue = gst_element_factory_make ("queue", "video_queue");
-       g_return_val_if_fail (video_queue != NULL, FALSE);
-       g_hash_table_insert(play->other_elements, "video_queue", video_queue);
-       g_object_set (G_OBJECT (video_queue), "block_timeout", 1000, NULL);
-
-       gst_bin_add_many (
-                       GST_BIN (play->pipeline), video_queue,
-                       play->video_sink, NULL);
-                       
-       gst_element_link (video_queue, play->video_sink);
-       
-       gst_bin_set_pre_iterate_function(
-                       GST_BIN (play->pipeline), 
-                       (GstBinPrePostIterateFunction) callback_bin_pre_iterate,
-                       play->video_bin_mutex);
-                       
-       gst_bin_set_post_iterate_function(
-                       GST_BIN (play->pipeline), 
-                       (GstBinPrePostIterateFunction) callback_bin_post_iterate,
-                       play->video_bin_mutex);
-       
-       gst_element_link (work_thread, video_queue);
-
-       return TRUE;
-}
-
-
-static gboolean
-gst_play_videots_set_auto (GstPlay *play, GstElement *autoplugger)
-{
-
-       GstElement *audio_bin, *auto_identity, *work_thread;
-
-       g_return_val_if_fail (GST_IS_PLAY(play), FALSE);
-       g_return_val_if_fail (GST_IS_ELEMENT (autoplugger), FALSE);
-       
-       audio_bin = g_hash_table_lookup(play->other_elements, "audio_bin");
-       auto_identity = g_hash_table_lookup(play->other_elements, "auto_identity");
-       work_thread = g_hash_table_lookup(play->other_elements, "work_thread");
-
-       if (play->autoplugger) {
-               /* we need to remove the existing autoplugger 
-                * before creating a new one */
-               gst_element_unlink (play->autoplugger, audio_bin);
-               gst_element_unlink (play->autoplugger, play->source);
-               gst_element_unlink (play->autoplugger, auto_identity);
-
-               gst_bin_remove (GST_BIN (work_thread), play->autoplugger);
-       }
-       
-       play->autoplugger = autoplugger;
-       g_return_val_if_fail (play->autoplugger != NULL, FALSE);
-
-       gst_bin_add (GST_BIN (work_thread), play->autoplugger);
-       gst_element_link (play->source, play->autoplugger);
-       gst_element_link (play->autoplugger, audio_bin);
-       gst_element_link (play->autoplugger, auto_identity);
-
-       return TRUE;
-}
-
-
-static gboolean
-gst_play_videots_set_video (GstPlay *play, GstElement *video_sink)
-{
-       GstElement *video_mate;
-       
-       g_return_val_if_fail (GST_IS_PLAY (play), FALSE);
-       g_return_val_if_fail (GST_IS_ELEMENT (video_sink), FALSE);
-       
-       video_mate = g_hash_table_lookup (play->other_elements, "video_queue");
-
-       if (play->video_sink) {
-               gst_element_unlink (video_mate, play->video_sink);
-               gst_bin_remove (GST_BIN (play->pipeline), play->video_sink);
-       }
-       play->video_sink = video_sink;
-       gst_bin_add (GST_BIN (play->pipeline), play->video_sink);
-       gst_element_link (video_mate, play->video_sink);
-
-       return TRUE;
-}
-
-
-static gboolean
-gst_play_videots_set_audio (GstPlay *play, GstElement *audio_sink)
-{
-       GstElement *audio_bin;
-       
-       g_return_val_if_fail (GST_IS_PLAY (play), FALSE);
-       g_return_val_if_fail (GST_IS_ELEMENT (audio_sink), FALSE);
-       
-       audio_bin = g_hash_table_lookup(play->other_elements, "audio_bin");
-       
-       if (play->audio_sink)
-       {
-               gst_element_unlink (play->volume, play->audio_sink);
-               gst_bin_remove (GST_BIN (audio_bin), play->audio_sink);
-       }
-
-       play->audio_sink = audio_sink;
-       gst_bin_add (GST_BIN (audio_bin), play->audio_sink);
-       gst_element_link (play->volume, play->audio_sink);
-
-       play->audio_sink_element = gst_play_get_sink_element (play, audio_sink);
-       
-       if (play->audio_sink_element != NULL) {
-               g_signal_connect (G_OBJECT (play->audio_sink_element), "eos",
-                                 G_CALLBACK (callback_audio_sink_eos), play);
-       }
-
-       return TRUE;
-}
-
 /* modelines */
 /* vim:set ts=8:sw=8:noet */