On 07/06/2012 07:31 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Jul 6, 2012 at 10:29 AM, Luca Barbato <lu_z...@gentoo.org> wrote: >> On 07/06/2012 07:12 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Thu, Jul 5, 2012 at 4:17 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: >>>> On Wed, Jul 4, 2012 at 8:14 AM, Ronald S. Bultje <rsbul...@gmail.com> >>>> wrote: >>>>> From: "Ronald S. Bultje" <rsbul...@gmail.com> >>>>> >>>>> Also replace x>>av_log2(sizeof(..)) + 1 by x/sizeof(..). The +1 is >>>>> probably meant to emulate av_log2_ceil(sizeof(..)) in cases where ".." >>>>> is not a power of two. >>>>> --- >>>>> libavformat/mxfdec.c | 8 ++++---- >>>>> 1 file changed, 4 insertions(+), 4 deletions(-) >>>> >>>> Ping. >>> >>> Ping2. >>> >> >> Wasn't it pending an inline av_calloc from Tomas? > > That can be done on top of this patch.
Patch looks fine to me, feel free to commit. lu -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel