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

[Diego Biurrun]

On Thu, Jan 29, 2009 at 11:35:31AM +0530, Jai Menon wrote:
> 
> revised patch attached.
> 
> --- configure	(revision 16716)
> +++ configure	(working copy)
> @@ -105,6 +105,7 @@
>    echo "                           native (de)muxer exists [no]"
> +  echo "  --enable-libopenjpeg     enable JPEG2000 decoding via OpenJPEG [no]"

The correct spelling is "JPEG 2000" (notice the space).  Please fix this
everywhere.

> --- Changelog	(revision 16716)
> +++ Changelog	(working copy)
> @@ -145,6 +145,7 @@
>  - hybrid WavPack support
>  - R3D REDCODE demuxer
> +- OpenJPEG based JPEG2000 decoder

libopenjpeg JPEG 2000 decoder

This will not apply, update to HEAD.

> --- libavcodec/libopenjpeg.c	(revision 0)
> +++ libavcodec/libopenjpeg.c	(revision 0)
> @@ -0,0 +1,185 @@
> +/*
> + * JPEG2000 decoding support via OpenJPEG
> + */
> +
> +/**
> +* @file libopenjpeg.c
> +* JPEG2000 decoder using libopenjpeg
> +*/

OpenJPEG or libopenjpeg?  I'd settle for the latter.

> +    .long_name = NULL_IF_CONFIG_SMALL("OpenJPEG based JPEG2000 decoder"),

libopenjpeg JPEG 2000

> --- doc/general.texi	(revision 16716)
> +++ doc/general.texi	(working copy)
> @@ -162,6 +162,7 @@
>  @item animated GIF @tab X @tab X @tab Only uncompressed GIFs are generated.
>  @item JPEG         @tab X @tab X @tab Progressive JPEG is not supported.
> + at item JPEG2000     @tab   @tab X @tab Using the external libopenjpeg library

Use an 'E' to mark external libs as such and reword as

    @tab decoding supported through external library libopenjpeg

Diego