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

Feng Duan digamma2function at gmail.com
Fri Jun 28 10:46:45 CEST 2013


On Thu, Jun 27, 2013 at 9:45 PM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:

> 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
>

Thank for your review.

Yes, I meant that the copy right as you said. I will change the pix_fmt in
next patch.

Thanks.

Feng

>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list