[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