[FFmpeg-devel] [PATCH] adding xavs encoding support

[Måns Rullgård]

jianwen chen <jianwen.chen.video at gmail.com> writes:

> Dears,
>
> Attached is the latest patch to support xavs encoding in ffmpeg.
>
> And please put the libxavs.c to  ffmpeg source code under ffmpeg/libavcodec/
>
> Index: configure
> ===================================================================
> --- configure	(revision 24248)
> +++ configure	(working copy)
> @@ -2660,6 +2664,9 @@
>  enabled libx264    && require  libx264 x264.h x264_encoder_encode -lx264 &&
>                        { check_cpp_condition x264.h "X264_BUILD >= 98" ||
>                          die "ERROR: libx264 version must be >= 0.98."; }
> +enabled libxavs    && require  libxavs xavs.h xavs_encoder_encode -lxavs -lm &&
> +		      { check_cpp_condition xavs.h "XAVS_BUILD >= 1" ||
> +                        die "ERROR: libxavs version should at least be 1"; }

Until some specific version is required, the additional check is unnecessary.

> Index: libavcodec/avcodec.h
> ===================================================================
> --- libavcodec/avcodec.h	(revision 24248)
> +++ libavcodec/avcodec.h	(working copy)
> @@ -2382,6 +2382,12 @@
>  #define X264_PART_P4X4 0x020  /* Analyze p8x4, p4x8, p4x4 */
>  #define X264_PART_B8X8 0x100  /* Analyze b16x8, b8x16 and b8x8 */
>  
> +#define XAVS_PART_I4X4 0x001  /* Analyze i4x4 */
> +#define XAVS_PART_P4X4 0x020  /* Analyze p8x4, p4x8, p4x4 */
> +#define XAVS_PART_I8X8 0x002  /* Analyze i8x8 (requires 8x8 transform) */
> +#define XAVS_PART_P8X8 0x010  /* Analyze p16x8, p8x16 and p8x8 */
> +#define XAVS_PART_B8X8 0x100  /* Analyze b16x8, b8x16 and b8x8 */

I see no reason for adding additional defines for the same values with
the same meanings as the x264 ones.

-- 
M?ns Rullg?rd
mans at mansr.com