[FFmpeg-devel] [PATCH] libavf: Auto-detect mjpeg 2000 in mpeg-ts

[Michael Niedermayer]

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 <michael at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161218/969c71fe/attachment.sig>