gst-plugins-bad: intervideosrc: Fix GAP flag setting on black frames correctly

Sebastian Dröge sebastian at centricular.com
Thu Jan 22 12:06:05 PST 2015


On Mi, 2015-01-21 at 21:39 -0800, Arun Raghavan wrote:
> Module: gst-plugins-bad
> Branch: master
> Commit: 825239b45063e41f68771917ca047a25c852fbd3
> URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=825239b45063e41f68771917ca047a25c852fbd3
> 
> Author: Arun Raghavan <arun at centricular.com>
> Date:   Thu Jan 22 11:08:23 2015 +0530
> 
> intervideosrc: Fix GAP flag setting on black frames correctly
> 
> This did not actually work since the video_buffer was set to NULL after
> the first black frame.
> [...]
> -    /* Repeated buffer? */
> -    if (intervideosrc->surface->video_buffer_count > 0)
> -      is_gap = TRUE;

If I'm not mistaken, this part is missing now. It should mark repeated
non-black frames as GAP too. The logic was to mark every frame as GAP,
that is the same as the previous one.

-- 
Sebastian Dröge, Centricular Ltd · http://www.centricular.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 949 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/gstreamer-devel/attachments/20150122/bbe18020/attachment.sig>


More information about the gstreamer-devel mailing list