[MPlayer-dev-eng] delete draw_slice stub in vo_corevideo.m
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Tue Jan 21 21:02:56 CET 2014
On Tue, Jan 21, 2014 at 01:01:45PM +0900, Zongyao Qu wrote:
> when deleting draw_frame , mplayer will crash at filter_video
>
> I don't have time to look into it. I just thought there must at least one
> draw_ xx function there.
There should be no such requirement, however a crash is possible
in cases that would have caused incorrect output already before.
Basically not implementing draw_slice is incorrect for all VOs that
support YUV formats, unless they use VOCAP_NOSLICES.
This is the main reason why I decided to go with NULL instead of a stub
function, it should make these kind of bugs fairly obvious.
Except that I see this one does not support YUV formats, so something
else is wrong here.
I suspect the reason is that draw_image incorrect returns 0 when it
should return VO_TRUE on success.
This causes MPlayer to try to draw the frame via the other functions.
More information about the MPlayer-dev-eng
mailing list