On Fri, Oct 14, 2016 at 01:31:12AM +0200, Michael Niedermayer wrote: > On Thu, Oct 13, 2016 at 04:41:00PM +0200, Ståle Kristoffersen wrote: > > On 2016-10-12 at 13:57, Carl Eugen Hoyos wrote: > > > 2016-10-11 16:27 GMT+02:00 Michael Niedermayer <mich...@niedermayer.cc>: > > > > > > > also the img2 demuxer for mjpeg2000 from img2dec.c does not work > > > > for this ? > > > > > > We separated the jpg from the mjpeg probe, jpg makes sure that the > > > sample starts like a jpg image and contains an end marker and can > > > be decoded, mjpeg probe is less strict but requires more than one > > > frame. > > > Same could be argued for j2k or was our reasoning bad? > > > > Since nobody chimed in on this, I have created two new patches, one > > containing a separate mjpeg 2000 probe > > (0001-libavf-Auto-detect-mjpeg-2000-in-mpeg-ts2.patch), and one that > > incorporates it into the img2dec jpeg 2000 probe > > (0001-libavf-Make-jpeg-2000-probe-work-for-mjpeg-2000.patch). > > > > > I think it is best to have them separated, as that matches the mjpeg > > behaviour. > > have you tested the demuxer ? > i dont think it works outside probing for mpegts > i think for j2k codestream support a jpeg2000 parser is needed first > same as we have for mjpeg
theres actually a j2k parser patch on the ML so applied this one here thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Everything should be made as simple as possible, but not simpler. -- Albert Einstein
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel