Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-29 Thread Matthieu Bouron
On Wed, Oct 28, 2015 at 5:36 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 03:10:00PM +0100, Matthieu Bouron wrote: > > On Wed, Oct 28, 2015 at 12:58:53PM +0100, Michael Niedermayer wrote: > > > On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote: > > > > On Tue, Oct 27

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-28 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 03:10:00PM +0100, Matthieu Bouron wrote: > On Wed, Oct 28, 2015 at 12:58:53PM +0100, Michael Niedermayer wrote: > > On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote: > > > On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote: > > > > On Sun, Oct

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-28 Thread Matthieu Bouron
On Wed, Oct 28, 2015 at 12:58:53PM +0100, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote: > > On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote: > > > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote: > > > [...] > > > >

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-28 Thread Matthieu Bouron
On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote: > On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote: > > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote: > > [...] > > > > > > Patch updated, the markers are now properly skipped (which also fixes a

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-28 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote: > On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote: > > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote: > > [...] > > > > > > Patch updated, the markers are now properly skipped (which also fixes a

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-27 Thread Michael Niedermayer
On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote: > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote: > [...] > > > > Patch updated, the markers are now properly skipped (which also fixes a > > crash). > > Patch updated. It fixes an issue with mjpeg streams (and in p

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-27 Thread Matthieu Bouron
On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote: [...] > > Patch updated, the markers are now properly skipped (which also fixes a > crash). Patch updated. It fixes an issue with mjpeg streams (and in particular with the fate sample ffmpeg-issue-897.avi) due to the EOI marker not

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-18 Thread Matthieu Bouron
On Sat, Oct 17, 2015 at 10:34:23PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavcodec/mjpegdec.c | 20 > 1 file changed, 20 insertions(+) > > diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c > index 1a86b7b..8a90e94 100644 > --- a/libavcode

[FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-17 Thread Matthieu Bouron
From: Matthieu Bouron --- libavcodec/mjpegdec.c | 20 1 file changed, 20 insertions(+) diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index 1a86b7b..8a90e94 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -2038,6 +2038,21 @@ int ff_mjpeg_decode