diff options
| author | Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com> | 2015-10-17 01:51:24 -0700 |
|---|---|---|
| committer | Sebastian Dröge <sebastian@centricular.com> | 2015-10-20 10:12:32 +0300 |
| commit | 9fb73a18e2565c5b7310260a697156293b6760a2 (patch) | |
| tree | 3149db3dc5a5a753515a8529f177e0af56b4477c /gst/pnm/gstpnmdec.c | |
| parent | f6c66f2a7842b3ec195f5311d76ce8ec0d448721 (diff) | |
| download | gstreamer-plugins-bad-9fb73a18e2565c5b7310260a697156293b6760a2.tar.gz | |
pnmdec: propagate input state after parsing
Store and copy input state fields when setting the
output state of the decoder. Avoids problems like
the framerate set by an upstream element being ignored
Related to:
https://bugzilla.gnome.org/show_bug.cgi?id=756563
Diffstat (limited to 'gst/pnm/gstpnmdec.c')
| -rw-r--r-- | gst/pnm/gstpnmdec.c | 32 |
1 files changed, 31 insertions, 1 deletions
diff --git a/gst/pnm/gstpnmdec.c b/gst/pnm/gstpnmdec.c index d99374a10..2f7e774fe 100644 --- a/gst/pnm/gstpnmdec.c +++ b/gst/pnm/gstpnmdec.c @@ -44,6 +44,9 @@ #include <stdio.h> static gboolean gst_pnmdec_start (GstVideoDecoder * decoder); +static gboolean gst_pnmdec_set_format (GstVideoDecoder * decoder, + GstVideoCodecState * state); +static gboolean gst_pnmdec_stop (GstVideoDecoder * decoder); static GstFlowReturn gst_pnmdec_parse (GstVideoDecoder * decoder, GstVideoCodecFrame * frame, GstAdapter * adapter, gboolean at_eos); static GstFlowReturn gst_pnmdec_handle_frame (GstVideoDecoder * decoder, @@ -83,8 +86,10 @@ gst_pnmdec_class_init (GstPnmdecClass * klass) "Lutz Mueller <lutz@users.sourceforge.net>"); vdec_class->start = gst_pnmdec_start; + vdec_class->stop = gst_pnmdec_stop; vdec_class->parse = gst_pnmdec_parse; vdec_class->handle_frame = gst_pnmdec_handle_frame; + vdec_class->set_format = gst_pnmdec_set_format; } static void @@ -111,6 +116,31 @@ gst_pnmdec_init (GstPnmdec * s) GST_PAD_SET_ACCEPT_TEMPLATE (GST_VIDEO_DECODER_SINK_PAD (s)); } +static gboolean +gst_pnmdec_set_format (GstVideoDecoder * decoder, GstVideoCodecState * state) +{ + GstPnmdec *pnmdec = (GstPnmdec *) decoder; + + if (pnmdec->input_state) + gst_video_codec_state_unref (pnmdec->input_state); + pnmdec->input_state = gst_video_codec_state_ref (state); + + return TRUE; +} + +static gboolean +gst_pnmdec_stop (GstVideoDecoder * decoder) +{ + GstPnmdec *pnmdec = (GstPnmdec *) decoder; + + if (pnmdec->input_state) { + gst_video_codec_state_unref (pnmdec->input_state); + pnmdec->input_state = NULL; + } + + return TRUE; +} + static GstFlowReturn gst_pnmdec_parse_ascii (GstPnmdec * s, const guint8 * b, guint bs) { @@ -330,7 +360,7 @@ gst_pnmdec_parse (GstVideoDecoder * decoder, GstVideoCodecFrame * frame, } output_state = gst_video_decoder_set_output_state (GST_VIDEO_DECODER (s), format, - s->mngr.info.width, s->mngr.info.height, NULL); + s->mngr.info.width, s->mngr.info.height, s->input_state); gst_video_codec_state_unref (output_state); if (gst_video_decoder_negotiate (GST_VIDEO_DECODER (s)) == FALSE) { r = GST_FLOW_NOT_NEGOTIATED; |