[libav-devel] [PATCH] mxfdec: replace right-shift by av_log2(foo) with division by foo

From: "Ronald S. Bultje" Since all operands involved are constants, the division can be computed at compile time whereas av_log2() cannot. --- libavformat/mxfdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index dd10240

Re: [libav-devel] [PATCH] mxfdec: replace right-shift by av_log2(foo) with division by foo

On 06/18/2012 09:18 PM, Mans Rullgard wrote: > From: "Ronald S. Bultje" > > Since all operands involved are constants, the division can be > computed at compile time whereas av_log2() cannot. > --- > libavformat/mxfdec.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Ok. -- L

Re: [libav-devel] [PATCH] mxfdec: replace right-shift by av_log2(foo) with division by foo

On Mon, Jun 18, 2012 at 08:18:38PM +0100, Mans Rullgard wrote: > From: "Ronald S. Bultje" > > Since all operands involved are constants, the division can be > computed at compile time whereas av_log2() cannot. > --- > libavformat/mxfdec.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletio

Re: [libav-devel] [PATCH] mxfdec: replace right-shift by av_log2(foo) with division by foo

Kostya Shishkov writes: > On Mon, Jun 18, 2012 at 08:18:38PM +0100, Mans Rullgard wrote: >> From: "Ronald S. Bultje" >> >> Since all operands involved are constants, the division can be >> computed at compile time whereas av_log2() cannot. >> --- >> libavformat/mxfdec.c |4 ++-- >> 1 file

Re: [libav-devel] [PATCH] mxfdec: replace right-shift by av_log2(foo) with division by foo

On Mon, Jun 18, 2012 at 08:48:09PM +0100, Måns Rullgård wrote: > Kostya Shishkov writes: > > > On Mon, Jun 18, 2012 at 08:18:38PM +0100, Mans Rullgard wrote: > >> From: "Ronald S. Bultje" > >> > >> Since all operands involved are constants, the division can be > >> computed at compile time wher