[FFmpeg-devel] [PATCH] lavc add XVBA decoder and hardware accelerator

[Carl Eugen Hoyos]

Feng <digamma2function <at> gmail.com> writes:

> + * Copyright (C) 2005-2011 Team XBMC
> + * 
> + * Modified by Feng Duan <fengduan <at> multicorewareinc.com>

I wonder what "Modified by" means in the context 
of a license header.

Maybe you meant:
* Copyright (C) 2005-2011 Team XBMC
* Copyright (C) 2013 Feng Duan <fengduan <at> multicorewareinc.com>

?

>      AV_PIX_FMT_VDA_VLD,    ///< hardware decoding through VDA
> +    AV_PIX_FMT_XVBA_VLD,   ///< HW decoding through XVBA, 
> Picture.data[3] contains a XVBA surface.

You cannot add a pix_fmt in the middle of the list, 
this breaks ABI.

Thank you for working on this, Carl Eugen