Re: [FFmpeg-devel] [PATCH] x11grab: fixed next frame capture time calculation

2016-02-03 Thread Carl Eugen Hoyos
Trevor \\ Higgins gmail.com> writes: > -if (delay <= 0) { The patch will get more readable and easier to review if you don't change this line. > +do{ > +}while((s->time_frame * av_q2d(s->time_base) - curtime) <= 0); Please make this: do { ... } while ((... to keep the style

[FFmpeg-devel] [PATCH] x11grab: fixed next frame capture time calculation

2016-02-03 Thread Trevor \\ Higgins
The next frame time could slip, causing the frame rate to drop until frames were dropped. Now will capture at the next correct moment instead. --- libavdevice/x11grab.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/libavdevice/x11grab.c b/libavdevice/x11grab.c