[FFmpeg-devel] [PATCH] Add hwaccel configury (take 2)

Diego Biurrun diego
Thu Apr 2 09:44:48 CEST 2009


On Mon, Mar 23, 2009 at 11:29:05PM +0100, Gwenole Beauchesne wrote:
> 
> Hmm, my sent-box does have it... Trying again:
> 
> --- configure	(revision 18172)
> +++ configure	(working copy)
> @@ -104,6 +104,9 @@
>    echo "  --enable-memalign-hack   emulate memalign, interferes with memory debuggers"
>    echo "  --enable-beos-netserver  enable BeOS netserver"
>    echo "  --disable-encoder=NAME   disable encoder NAME"
> +  echo "  --disable-hwaccel=NAME   disable hwaccel NAME"
> +  echo "  --enable-hwaccel=NAME    enable hwaccel NAME"
> +  echo "  --disable-hwaccels       disable all hwaccels"
>    echo "  --enable-encoder=NAME    enable encoder NAME"
>    echo "  --disable-encoders       disable all encoders"
>    echo "  --disable-decoder=NAME   disable decoder NAME"

Bad, you are placing yourself in the middle of two related options.

> @@ -130,6 +133,7 @@
>    echo "  --enable-filter=NAME     enable filter NAME"
>    echo "  --disable-filter=NAME    disable filter NAME"
>    echo "  --disable-filters        disable all filters"
> +  echo "  --list-hwaccels          show all available hardware accelerators"
>    echo "  --list-decoders          show all available decoders"
>    echo "  --list-encoders          show all available encoders"
>    echo "  --list-muxers            show all available muxers"

Put hwaccels after decoders/encoders everywhere.  This applies
everywhere else.

Also, respect alphabetical order.

Diego



More information about the ffmpeg-devel mailing list