[FFmpeg-devel] [PATCH 1/2] jpeg2000dec: parse CDEF

[Michael Niedermayer]

On Sat, Jul 13, 2013 at 12:48:25AM +0200, Michael Niedermayer wrote:
> On Fri, Jul 12, 2013 at 10:10:10PM +0000, Paul B Mahol wrote:
> > On 7/12/13, Nicolas BERTRAND <nicoinattendu at gmail.com> wrote:
> > > Le 12/07/2013 23:47, Paul B Mahol a ecrit :
> > >> On 7/12/13, Nicolas BERTRAND <nicoinattendu at gmail.com> wrote:
> > >>>
> > >>> I'm not sure the parse of CDEF is in the right place. as it is defined
> > >>> in the JPEG2000 file format syntax. It will probably best fit with a JP2
> > >>> container in avformat?
> > >>
> > >> Does jp2 may have more than single jpeg2000 image, if not, such
> > >> separation
> > >> is pointless.
> > > JP2 can be a file format including a Jpeg2000 codestream. Like jp2 files
> > > Or have only a peg2000 codestream ( like jpeg2000 for Dcinema or MJPEG2000)
> > > This why i differentiate both. And in file format there are a lot of
> > > color-space related stuff.
> > 
> > well you are telling that jp2 have only single image. Thus having own
> > demuxer for it is suboptimal.
> 

> also for jpeg the fileformat (JFIF is also handled at codec level)
> and these fileformat JFIF headers are stored per frame in some mjpeg
> in avis (i have such files)

note, this is not meant as an objection against handling jp2 at the
avformat level. More meant as related side information.

Iam also quite curious about if mov/avi ever store jp2 "files" per
frame


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130713/8aeb522a/attachment.asc>