[Ffmpeg-devel] CamStudio support

[Diego Biurrun]

On Sun, Jan 08, 2006 at 05:55:13PM +0100, Reimar D?ffinger wrote:
> the attached patch adds support for camstudio video:
> Please tell me what you think about it, especially the configure part for
> lzo.

Colors are off on my PPC, looks like an endianness issue.  Let me know
if you need further info to reproduce or if I should test something for
you.

> --- configure	3 Jan 2006 15:40:39 -0000	1.227
> +++ configure	8 Jan 2006 16:50:21 -0000
> @@ -67,6 +67,7 @@
>  echo "  --disable-dv1394         disable DV1394 grabbing [default=no]"
>  echo "  --disable-network        disable network support [default=no]"
>  echo "  --disable-zlib           disable zlib [default=no]"
> +echo "  --disable-lzo            disable lzo [default=no], requires --enable-gpl"

I think you can leave out the "requires --enable-gpl" part, it's nowhere
else.

> @@ -985,6 +989,25 @@
>  extralibs="$extralibs -lz"
>  fi
>  
> +if test "$lzo" = "yes"; then
> +# check for liblzo
> +if test "$gpl" = "yes"; then

Simplify this:

  if test "$lzo" = "yes" -a "$gpl" = "yes"; then

> +            av_log(avctx, AV_LOG_ERROR, "compiled without lzo (GPL) support\n");

This may be a more general problem, but is there a way to avoid this
getting printed for every frame if lzo is not available?

Diego