From: Sebastian Dröge <slomo@circular-chaos.org>
Date: Sat, 29 Sep 2007 07:01:55 +0000 (+0000)
Subject: ext/gio/: Some minor cleanup and allow setting the location only when the element... 
X-Git-Tag: 1.19.3~511^2~10915
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d4e3e0e75fff918c53e577c7dadf5e59fa0485c5;p=platform%2Fupstream%2Fgstreamer.git

ext/gio/: Some minor cleanup and allow setting the location only when the element is not playing or paused.

Original commit message from CVS:
* ext/gio/gstgiosink.c: (gst_gio_sink_base_init),
(gst_gio_sink_set_property), (gst_gio_sink_render):
* ext/gio/gstgiosrc.c: (gst_gio_src_base_init),
(gst_gio_src_set_property):
Some minor cleanup and allow setting the location only when the
element is not playing or paused.
---

diff --git a/ext/gio/gstgiosink.c b/ext/gio/gstgiosink.c
index 0b2f65a..2d9677b 100644
--- a/ext/gio/gstgiosink.c
+++ b/ext/gio/gstgiosink.c
@@ -55,8 +55,7 @@ enum
 static GstStaticPadTemplate sink_factory = GST_STATIC_PAD_TEMPLATE ("sink",
     GST_PAD_SINK,
     GST_PAD_ALWAYS,
-    GST_STATIC_CAPS ("ANY")
-    );
+    GST_STATIC_CAPS_ANY);
 
 GST_BOILERPLATE_FULL (GstGioSink, gst_gio_sink, GstBaseSink, GST_TYPE_BASE_SINK,
     gst_gio_uri_handler_do_init);
@@ -81,7 +80,7 @@ gst_gio_sink_base_init (gpointer gclass)
   static GstElementDetails element_details = {
     "GIO sink",
     "Sink/File",
-    "Write to any GVFS-supported location",
+    "Write to any GIO-supported location",
     "Ren\xc3\xa9 Stadler <mail@renestadler.de>"
   };
   GstElementClass *element_class = GST_ELEMENT_CLASS (gclass);
@@ -154,6 +153,10 @@ gst_gio_sink_set_property (GObject * object, guint prop_id,
 
   switch (prop_id) {
     case ARG_LOCATION:
+      if (GST_STATE (sink) == GST_STATE_PLAYING ||
+          GST_STATE (sink) == GST_STATE_PAUSED)
+        break;
+
       g_free (sink->location);
       sink->location = g_strdup (g_value_get_string (value));
       break;
@@ -370,7 +373,9 @@ gst_gio_sink_render (GstBaseSink * base_sink, GstBuffer * buffer)
     /* FIXME: Can this happen?  Should we handle it gracefully?  gnomevfssink
      * doesn't... */
     GST_ELEMENT_ERROR (sink, RESOURCE, WRITE, (NULL),
-        ("Could not write to location %s: (short write)", sink->location));
+        ("Could not write to location %s: (short write, only %"
+            G_GUINT64_FORMAT " bytes of %d bytes written)",
+            sink->location, written, GST_BUFFER_SIZE (buffer)));
     return GST_FLOW_ERROR;
   }
 
diff --git a/ext/gio/gstgiosrc.c b/ext/gio/gstgiosrc.c
index 6f41e44..15e0b9d 100644
--- a/ext/gio/gstgiosrc.c
+++ b/ext/gio/gstgiosrc.c
@@ -49,8 +49,7 @@ enum
 static GstStaticPadTemplate src_factory = GST_STATIC_PAD_TEMPLATE ("src",
     GST_PAD_SRC,
     GST_PAD_ALWAYS,
-    GST_STATIC_CAPS ("ANY")
-    );
+    GST_STATIC_CAPS_ANY);
 
 GST_BOILERPLATE_FULL (GstGioSrc, gst_gio_src, GstBaseSrc, GST_TYPE_BASE_SRC,
     gst_gio_uri_handler_do_init);
@@ -76,7 +75,7 @@ gst_gio_src_base_init (gpointer gclass)
   static GstElementDetails element_details = {
     "GIO source",
     "Source/File",
-    "Read from any GVFS-supported location",
+    "Read from any GIO-supported location",
     "Ren\xc3\xa9 Stadler <mail@renestadler.de>"
   };
   GstElementClass *element_class = GST_ELEMENT_CLASS (gclass);
@@ -147,6 +146,10 @@ gst_gio_src_set_property (GObject * object, guint prop_id,
 
   switch (prop_id) {
     case ARG_LOCATION:
+      if (GST_STATE (src) == GST_STATE_PLAYING ||
+          GST_STATE (src) == GST_STATE_PAUSED)
+        break;
+
       g_free (src->location);
       src->location = g_strdup (g_value_get_string (value));
       break;