gst/alpha/gstalpha.c: Don't ignore return value of the parent class's state

Original commit message from CVS:
* gst/alpha/gstalpha.c: (gst_alpha_change_state):
Don't ignore return value of the parent class's state
change function (#331385, patch by: Wouter Paesen).
This commit is contained in:
Wouter Paesen 2006-02-16 09:17:58 +00:00 committed by Tim-Philipp Müller
parent 5e5386b430
commit 2d150478bb
2 changed files with 18 additions and 8 deletions

View file

@ -1,3 +1,9 @@
2006-02-16 Tim-Philipp Müller <tim at centricular dot net>
* gst/alpha/gstalpha.c: (gst_alpha_change_state):
Don't ignore return value of the parent class's state
change function (#331385, patch by: Wouter Paesen).
2006-02-15 Wim Taymans <wim@fluendo.com> 2006-02-15 Wim Taymans <wim@fluendo.com>
* configure.ac: * configure.ac:

View file

@ -857,6 +857,7 @@ gst_alpha_chain (GstPad * pad, GstBuffer * buffer)
static GstStateChangeReturn static GstStateChangeReturn
gst_alpha_change_state (GstElement * element, GstStateChange transition) gst_alpha_change_state (GstElement * element, GstStateChange transition)
{ {
GstStateChangeReturn res;
GstAlpha *alpha; GstAlpha *alpha;
alpha = GST_ALPHA (element); alpha = GST_ALPHA (element);
@ -868,18 +869,21 @@ gst_alpha_change_state (GstElement * element, GstStateChange transition)
gst_alpha_init_params (alpha); gst_alpha_init_params (alpha);
break; break;
case GST_STATE_CHANGE_PAUSED_TO_PLAYING: case GST_STATE_CHANGE_PAUSED_TO_PLAYING:
break; default:
case GST_STATE_CHANGE_PLAYING_TO_PAUSED:
break;
case GST_STATE_CHANGE_PAUSED_TO_READY:
break;
case GST_STATE_CHANGE_READY_TO_NULL:
break; break;
} }
parent_class->change_state (element, transition); res = parent_class->change_state (element, transition);
return GST_STATE_CHANGE_SUCCESS; switch (transition) {
case GST_STATE_CHANGE_PLAYING_TO_PAUSED:
case GST_STATE_CHANGE_PAUSED_TO_READY:
case GST_STATE_CHANGE_READY_TO_NULL:
default:
break;
}
return res;
} }
static gboolean static gboolean