On Sun, Jul 08, 2012 at 06:30:05PM +0200, Tomas Härdin wrote:
> On Sun, 2012-07-08 at 18:06 +0200, Diego Biurrun wrote:
> > On Sun, Jul 08, 2012 at 05:18:23PM +0200, Tomas Härdin wrote:
> > > --- a/libavutil/mem.h
> > > +++ b/libavutil/mem.h
> > > @@ -113,6 +113,18 @@ void av_free(void *ptr);
> > >  
> > >  /**
> > > + * Allocate a block of nmemb * size bytes with alignment suitable for all
> > > + * memory accesses (including vectors if available on the CPU) and
> > > + * zero all the bytes of the block.
> > > + * The allocation will fail if nmemb * size is greater than or equal
> > > + * to INT_MAX.
> > > + * @param nmemb
> > > + * @param size
> > > + * @return Pointer to the allocated block, NULL if it cannot be 
> > > allocated.
> > > + */
> > > +void *av_calloc(size_t nmemb, size_t size) av_malloc_attrib;
> > 
> > Please fix the @param descriptions and squash patch 2/2 into this one.
> > Otherwise LGTM.
> 
> Done. I kept the original author since the changes to the patch are
> trivial.

I'm queueing this.  Will push with the next batch after running fate
and giving people a chance to comment.

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

Reply via email to