[FFmpeg-cvslog] bink: no need to increase width twice

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Mar 21 07:58:25 CET 2012



On 21 Mar 2012, at 04:33, git at videolan.org (Paul B Mahol) wrote:

> ffmpeg | branch: master | Paul B Mahol <onemda at gmail.com> | Wed Mar 21 01:41:50 2012 +0000| [594a3d631583a2f086fac25db6fb3542fd0fb187] | committer: Michael Niedermayer
> 
> bink: no need to increase width twice
> 
> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> 
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=594a3d631583a2f086fac25db6fb3542fd0fb187
> ---
> 
> libavcodec/bink.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/libavcodec/bink.c b/libavcodec/bink.c
> index c8daab3..4ca96e5 100644
> --- a/libavcodec/bink.c
> +++ b/libavcodec/bink.c
> @@ -150,7 +150,7 @@ static void init_lengths(BinkContext *c, int width, int bw)
> 
>     c->bundle[BINK_SRC_BLOCK_TYPES].len = av_log2((width >> 3) + 511) + 1;
> 
> -    c->bundle[BINK_SRC_SUB_BLOCK_TYPES].len = av_log2(((width + 7) >> 4) + 511) + 1;
> +    c->bundle[BINK_SRC_SUB_BLOCK_TYPES].len = av_log2((width >> 4) + 511) + 1;
> 

Huh? Those don't give the same result when width is e.g. 9.
At least the commit message is really unclear.


More information about the ffmpeg-cvslog mailing list