deinterlace: The return value of gst_pad_set_caps() is not relevant anymore
authorSebastian Dröge <slomo@circular-chaos.org>
Wed, 22 May 2013 15:32:33 +0000 (17:32 +0200)
committerSebastian Dröge <slomo@circular-chaos.org>
Wed, 22 May 2013 15:34:07 +0000 (17:34 +0200)
Caps can fail to be set because the pad is not linked yet for example.

gst/deinterlace/gstdeinterlace.c

index b93b5b7..3210990 100644 (file)
@@ -2462,8 +2462,7 @@ gst_deinterlace_setcaps (GstDeinterlace * self, GstPad * pad, GstCaps * caps)
         "progressive", NULL);
   }
 
-  if (!gst_pad_set_caps (self->srcpad, srccaps))
-    goto caps_not_accepted;
+  gst_pad_set_caps (self->srcpad, srccaps);
 
   if (fps_n != 0) {
     self->field_duration = gst_util_uint64_scale (GST_SECOND, fps_d, fps_n);
@@ -2489,12 +2488,6 @@ invalid_caps:
     GST_ERROR_OBJECT (pad, "Invalid caps: %" GST_PTR_FORMAT, caps);
     return FALSE;
   }
-caps_not_accepted:
-  {
-    GST_ERROR_OBJECT (pad, "Caps not accepted: %" GST_PTR_FORMAT, srccaps);
-    gst_caps_unref (srccaps);
-    return FALSE;
-  }
 no_bufferpool:
   {
     GST_ERROR_OBJECT (pad, "could not negotiate bufferpool");