FaSTTUBe/GST-Tensordecoder-ov_ep
35
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

dvdspu: Always push a frame at the start of a still frame, and fix a leak.

Browse Source 
Make sure to push the frame for a still frame, with discont = true and
timestamp=none, so that it gets displayed by the sink. Also, don't leak
each rendered video frame during still menus.
This commit is contained in:
Jan Schmidt 2009-05-08 17:42:12 +01:00
parent 571a7746fc
commit 513367a88c
1 changed files with 24 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
gst/dvdspu/gstdvdspu.c
View File

@ -94,7 +94,7 @@ static GstFlowReturn gst_dvd_spu_video_chain (GstPad * pad, GstBuffer * buf);
static gboolean gst_dvd_spu_video_event (GstPad * pad, GstEvent * event); static gboolean gst_dvd_spu_video_event (GstPad * pad, GstEvent * event);
static GstFlowReturn gst_dvd_spu_buffer_alloc (GstPad * sinkpad, guint64 offset, static GstFlowReturn gst_dvd_spu_buffer_alloc (GstPad * sinkpad, guint64 offset,
guint size, GstCaps * caps, GstBuffer ** buf); guint size, GstCaps * caps, GstBuffer ** buf);
static void gst_dvd_spu_redraw_still (GstDVDSpu * dvdspu); static void gst_dvd_spu_redraw_still (GstDVDSpu * dvdspu, gboolean force);
static void gst_dvd_spu_check_still_updates (GstDVDSpu * dvdspu); static void gst_dvd_spu_check_still_updates (GstDVDSpu * dvdspu);
static GstFlowReturn gst_dvd_spu_subpic_chain (GstPad * pad, GstBuffer * buf); static GstFlowReturn gst_dvd_spu_subpic_chain (GstPad * pad, GstBuffer * buf);
@ -429,7 +429,7 @@ gst_dvd_spu_video_event (GstPad * pad, GstEvent * event)
/* And re-draw the still frame to make sure it appears on /* And re-draw the still frame to make sure it appears on
* screen, otherwise the last frame might have been discarded * screen, otherwise the last frame might have been discarded
* by QoS */ * by QoS */
gst_dvd_spu_redraw_still (dvdspu); gst_dvd_spu_redraw_still (dvdspu, TRUE);
} else } else
state->flags &= ~(SPU_STATE_STILL_FRAME); state->flags &= ~(SPU_STATE_STILL_FRAME);
DVD_SPU_UNLOCK (dvdspu); DVD_SPU_UNLOCK (dvdspu);
@ -633,15 +633,17 @@ no_ref_frame:
/* With SPU LOCK */ /* With SPU LOCK */
static void static void
gst_dvd_spu_redraw_still (GstDVDSpu * dvdspu) gst_dvd_spu_redraw_still (GstDVDSpu * dvdspu, gboolean force)
{ {
/* If we have an active SPU command set and a reference frame, copy the /* If we have an active SPU command set and a reference frame, copy the
* frame, redraw the SPU and store it as the pending frame for output */ * frame, redraw the SPU and store it as the pending frame for output */
if (dvdspu->ref_frame) { if (dvdspu->ref_frame) {
if ((dvdspu->spu_state.flags & SPU_STATE_FORCED_DSP) || gboolean redraw = (dvdspu->spu_state.flags & SPU_STATE_FORCED_DSP);
((dvdspu->spu_state.flags & SPU_STATE_FORCED_ONLY) == 0 && redraw |= (dvdspu->spu_state.flags & SPU_STATE_FORCED_ONLY) == 0 &&
(dvdspu->spu_state.flags & SPU_STATE_DISPLAY))) { (dvdspu->spu_state.flags & SPU_STATE_DISPLAY);
GstBuffer *buf = gst_buffer_copy (dvdspu->ref_frame);
if (redraw) {
GstBuffer *buf = gst_buffer_ref (dvdspu->ref_frame);
buf = gst_buffer_make_writable (buf); buf = gst_buffer_make_writable (buf);
@ -654,9 +656,21 @@ gst_dvd_spu_redraw_still (GstDVDSpu * dvdspu)
/* Render the SPU overlay onto the buffer */ /* Render the SPU overlay onto the buffer */
gst_dvd_spu_render_spu (dvdspu, buf); gst_dvd_spu_render_spu (dvdspu, buf);
gst_buffer_replace (&dvdspu->pending_frame, buf); gst_buffer_replace (&dvdspu->pending_frame, buf);
gst_buffer_unref (buf);
} else if (force) {
/* Simply output the reference frame */
GstBuffer *buf = gst_buffer_ref (dvdspu->ref_frame);
buf = gst_buffer_make_metadata_writable (buf);
GST_BUFFER_FLAG_SET (buf, GST_BUFFER_FLAG_DISCONT);
GST_BUFFER_TIMESTAMP (buf) = GST_CLOCK_TIME_NONE;
GST_BUFFER_DURATION (buf) = GST_CLOCK_TIME_NONE;
GST_DEBUG_OBJECT (dvdspu, "Pushing reference frame at start of still");
gst_buffer_replace (&dvdspu->pending_frame, buf);
gst_buffer_unref (buf);
} else { } else {
GST_LOG_OBJECT (dvdspu, GST_LOG_OBJECT (dvdspu, "Redraw due to Still Frame skipped");
"Redraw due to Still Frame skipped - no SPU to draw");
} }
} else { } else {
GST_LOG_OBJECT (dvdspu, "Not redrawing still frame - no ref frame"); GST_LOG_OBJECT (dvdspu, "Not redrawing still frame - no ref frame");
@ -1000,7 +1014,7 @@ gst_dvd_spu_handle_dvd_event (GstDVDSpu * dvdspu, GstEvent * event)
} }
if (hl_change && (state->flags & SPU_STATE_STILL_FRAME)) { if (hl_change && (state->flags & SPU_STATE_STILL_FRAME)) {
gst_dvd_spu_redraw_still (dvdspu); gst_dvd_spu_redraw_still (dvdspu, FALSE);
} }
gst_event_unref (event); gst_event_unref (event);