[FFmpeg-devel] [PATCH] libopenjpeg wrapper for jpeg2k decoding

[Diego Biurrun]

On Mon, Feb 02, 2009 at 05:05:14PM +0530, Jai Menon wrote:
> 
> On Mon, Feb 2, 2009 at 4:57 PM, Diego Biurrun <diego at biurrun.de> wrote:
> > On Mon, Feb 02, 2009 at 04:48:17PM +0530, Jai Menon wrote:
> >>
> >> i also reworked most of the code to use get/release_buffer
> >> revised patch attached.
> >>
> >> --- libavcodec/libopenjpeg.c  (revision 0)
> >> +++ libavcodec/libopenjpeg.c  (revision 0)
> >> @@ -0,0 +1,184 @@
> >> +
> >> +AVCodec libopenjpeg_decoder = {
> >> +    "libopenjpeg",
> >> +    .long_name = NULL_IF_CONFIG_SMALL("OpenJPEG based JPEG 2000 decoder"),
> >
> > libopenjpeg JPEG 2000
> 
> Is it just me or does that sound incorrect?
> I'll change it though if you want...

That's what we use for all other external libs, "libtheora Theora", etc.

> >> --- doc/general.texi  (revision 16912)
> >> +++ doc/general.texi  (working copy)
> >> @@ -162,6 +162,7 @@
> >>  @item JPEG         @tab X @tab X @tab Progressive JPEG is not supported.
> >> + at item JPEG2000     @tab   @tab E @tab decoding supported through external library libopenjpeg
> >>  @item PAM          @tab X @tab X @tab PAM is a PNM extension with alpha support.
> >
> > You need to add the explanation for the 'E' below the table.  Copy it
> > from one of the other tables.
> 
> I'm sorry but didn't quite understand what explanation is required?
> and where does it have to be added?

Look at the explanations below the video and audio codec tables.

Diego