On Mon, Jul 16, 2012 at 05:44:34PM +0200, Luca Barbato wrote:
> From: Michael Bradshaw <[email protected]>
>
> --- a/configure
> +++ b/configure
> @@ -178,7 +178,7 @@ External library support:
> --enable-libopencore-amrnb enable AMR-NB de/encoding via libopencore-amrnb
> [no]
> --enable-libopencore-amrwb enable AMR-WB decoding via libopencore-amrwb
> [no]
> --enable-libopencv enable video filtering via libopencv [no]
> - --enable-libopenjpeg enable JPEG 2000 decoding via OpenJPEG [no]
> + --enable-libopenjpeg enable JPEG 2000 support via OpenJPEG [no]
Please say "de/encoding" instead of "support" - I'll send a patch to clean
up the rest of the help output in a moment.
> --- /dev/null
> +++ b/libavcodec/libopenjpegenc.c
> @@ -0,0 +1,483 @@
> +static int libopenjpeg_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
> + const AVFrame *frame, int *got_packet)
> +{
> + LibOpenJPEGContext *ctx = avctx->priv_data;
> + opj_cinfo_t *compress = ctx->compress;
> + opj_image_t *image = ctx->image;
> + opj_cio_t *stream;
> + int cpyresult = 0;
Let's add the missing 'o' to this poor variable name.
Maybe wait for comments from other people, otherwise LGTM if it works.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
