[FFmpeg-devel] [PATCH] libavfilter-soc: make ffplay.c:request_input_frame() check for got_frame

Stefano Sabatini stefano.sabatini-lala
Thu Dec 25 11:45:26 CET 2008


Hi,

get_video_frame() returns 0 if it didn't get any frame, 1 otherwise,
and input_get_frame() is *requested* to get a frame, patch add a
check for this.

Problem discovered when playing MPlayer sample
MPEG2/res_change_ffmpeg_aspect.ts.

I'm not sure this is the better way to do it from the intrface point of
view, maybe a get_video_frame() like this:

static int get_video_frame(VideoState *is, AVFrame *frame, uint64_t *pts, AVPacket *pkt, int *got_frame)

would be nicer/more consistent with the rest of libav*, opinions?

Regards.
-- 
FFmpeg = Free and Furious Mysterious Problematic Elastic Gadget
-------------- next part --------------
A non-text attachment was scrubbed...
Name: check-for-got-frame.patch
Type: text/x-diff
Size: 707 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081225/a773d5fc/attachment.patch>



More information about the ffmpeg-devel mailing list