On Thu, Dec 15, 2016 at 12:03:36PM +0100, Anton Khirnov wrote:
> Quoting Diego Biurrun (2016-12-15 10:58:52)
> > On Thu, Dec 15, 2016 at 10:32:09AM +0100, Anton Khirnov wrote:
> > > Partially based on a patch by Timo Rothenpieler <[email protected]>
> > > ---
> > >  Changelog                 |   2 +-
> > >  configure                 |   3 +
> > >  libavcodec/Makefile       |   1 +
> > >  libavcodec/allcodecs.c    |   1 +
> > >  libavcodec/hevcdec.c      |   6 +-
> > >  libavcodec/vaapi_decode.c |   1 +
> > >  libavcodec/vaapi_hevc.c   | 443 
> > > ++++++++++++++++++++++++++++++++++++++++++++++
> > >  7 files changed, 455 insertions(+), 2 deletions(-)
> > >  create mode 100644 libavcodec/vaapi_hevc.c
> > 
> > version bump?
> 
> for what purpose?

We've always bumped minor version for new components. If we are going to
stop, we should do so across the board, not randomly.

> > > --- /dev/null
> > > +++ b/libavcodec/vaapi_hevc.c
> > > @@ -0,0 +1,443 @@
> > > +static int vaapi_hevc_decode_slice(AVCodecContext *avctx,
> > > +                                   const uint8_t  *buffer,
> > > +                                   uint32_t        size)
> > 
> > size_t size ;-p
> 
> stab
> 
> You could look at the context in which this function is used first.

I know that it's assigned to a function pointer ;-p

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to