Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-13 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > New patch attached that fixes ticket #5151. > > LGTM Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Carl Eugen Hoyos
Hi! I guess that attached patch fixes ticket #5151. It is the user's responsibility to know if the input stream is suitable for the bitstream filter or not. Please comment, Carl Eugen diff --git a/libavcodec/mjpeg2jpeg_bsf.c b/libavcodec/mjpeg2jpeg_bsf.c index 68640db..b29039e 100644 ---

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Carl Eugen Hoyos
On Tuesday 12 January 2016 02:16:52 pm Michael Niedermayer wrote: > On Tue, Jan 12, 2016 at 09:58:53AM +0100, Carl Eugen Hoyos wrote: > > -if (memcmp("AVI1", buf + 6, 4)) { > > -av_log(avctx, AV_LOG_ERROR, "input is not MJPEG/AVI1\n"); > > -return AVERROR_INVALIDDATA; > > +

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Michael Niedermayer
On Tue, Jan 12, 2016 at 02:41:12PM +0100, Carl Eugen Hoyos wrote: > On Tuesday 12 January 2016 02:28:28 pm Michael Niedermayer wrote: > > On Tue, Jan 12, 2016 at 02:19:53PM +0100, Carl Eugen Hoyos wrote: > > > On Tuesday 12 January 2016 02:16:52 pm Michael Niedermayer wrote: > > > > On Tue, Jan

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Michael Niedermayer
On Tue, Jan 12, 2016 at 02:19:53PM +0100, Carl Eugen Hoyos wrote: > On Tuesday 12 January 2016 02:16:52 pm Michael Niedermayer wrote: > > On Tue, Jan 12, 2016 at 09:58:53AM +0100, Carl Eugen Hoyos wrote: > > > > -if (memcmp("AVI1", buf + 6, 4)) { > > > -av_log(avctx, AV_LOG_ERROR,

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Michael Niedermayer
On Tue, Jan 12, 2016 at 09:58:53AM +0100, Carl Eugen Hoyos wrote: > Hi! > > I guess that attached patch fixes ticket #5151. > It is the user's responsibility to know if the input stream > is suitable for the bitstream filter or not. > > Please comment, Carl Eugen > mjpeg2jpeg_bsf.c |9

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Carl Eugen Hoyos
On Tuesday 12 January 2016 02:28:28 pm Michael Niedermayer wrote: > On Tue, Jan 12, 2016 at 02:19:53PM +0100, Carl Eugen Hoyos wrote: > > On Tuesday 12 January 2016 02:16:52 pm Michael Niedermayer wrote: > > > On Tue, Jan 12, 2016 at 09:58:53AM +0100, Carl Eugen Hoyos wrote: > > > > -if

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Carl Eugen Hoyos
On Tuesday 12 January 2016 03:19:24 pm Michael Niedermayer wrote: > > mjpeg2jpeg_bsf.c |5 + > > 1 file changed, 5 insertions(+) > > a261f4350cbfeefc9c011cfc93fc39e5c4f7fe7c patchmjpeg2jpgffd8.diff > > diff --git a/libavcodec/mjpeg2jpeg_bsf.c b/libavcodec/mjpeg2jpeg_bsf.c > > LGTM Patch

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-12 Thread Michael Niedermayer
On Tue, Jan 12, 2016 at 04:18:35PM +0100, Carl Eugen Hoyos wrote: > On Tuesday 12 January 2016 03:19:24 pm Michael Niedermayer wrote: > > > mjpeg2jpeg_bsf.c |5 + > > > 1 file changed, 5 insertions(+) > > > a261f4350cbfeefc9c011cfc93fc39e5c4f7fe7c patchmjpeg2jpgffd8.diff > > > diff --git