On Mon, Oct 24, 2011 at 09:12:43PM +0200, Nicolas George wrote:
> This patch includes some fixes by Carl Eugen Hoyos.
> 
> --- a/configure
> +++ b/configure
> @@ -3169,6 +3173,7 @@ echo "libva enabled             ${vaapi-no}"
>  echo "libvdpau enabled          ${vdpau-no}"
>  echo "AVISynth enabled          ${avisynth-no}"
>  echo "frei0r enabled            ${frei0r-no}"
> +echo "libcelt enabled           ${libcelt-no}"
>  echo "libcdio support           ${libcdio-no}"
>  echo "libdc1394 support         ${libdc1394-no}"
>  echo "libdirac enabled          ${libdirac-no}"

one line lower please

> --- /dev/null
> +++ b/libavcodec/libcelt_dec.c
> @@ -0,0 +1,138 @@
> +
> +static int ff_celt_error_to_averror(int err)
> +{
> +    switch (err) {
> +        case CELT_BAD_ARG:          return AVERROR(EINVAL);

Indent the case at the same level as the switch, sorry I did
not notice earlier.

> +AVCodec ff_libcelt_decoder = {
> +    .name           = "libcelt",
> +    .type           = AVMEDIA_TYPE_AUDIO,
> +    .id             = CODEC_ID_CELT,
> +    .priv_data_size = sizeof(struct libcelt_context),
> +    .init           = libcelt_dec_init,
> +    .close          = libcelt_dec_close,
> +    .decode         = libcelt_dec_decode,
> +    .capabilities   = 0,
> +    .long_name = NULL_IF_CONFIG_SMALL("Xiph CELT decoder using libcelt"),

Please don't worry about line length and align the '='.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to