On Thu, Dec 31, 2020 at 12:31:56AM +0100, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com>
> ---
> libavcodec/svq1dec.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c
> index 7aea138060..e9c068de19 100644
> --- a/libavcodec/svq1dec.c
> +++ b/libavcodec/svq1dec.c
> @@ -456,7 +456,7 @@ static int svq1_decode_delta_block(AVCodecContext *avctx,
> HpelDSPContext *hdsp,
> int result = 0;
>
> /* get block type */
> - block_type = get_vlc2(bitbuf, svq1_block_type.table, 2, 2);
> + block_type = get_vlc2(bitbuf, svq1_block_type.table, 3, 1);
>
> /* reset motion vectors */
> if (block_type == SVQ1_BLOCK_SKIP || block_type == SVQ1_BLOCK_INTRA) {
> @@ -765,9 +765,9 @@ err:
>
> static av_cold void svq1_static_init(void)
> {
> - INIT_VLC_STATIC(&svq1_block_type, 2, 4,
> + INIT_VLC_STATIC(&svq1_block_type, 3, 4,
> &ff_svq1_block_type_vlc[0][1], 2, 1,
> - &ff_svq1_block_type_vlc[0][0], 2, 1, 6);
> + &ff_svq1_block_type_vlc[0][0], 2, 1, 8);
> please use some named constant (of course can be in a seperate patch) duplicating these numbers risks havung then get out of sync patch LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us carefully observe those good qualities wherein our enemies excel us and endeavor to excel them, by avoiding what is faulty, and imitating what is excellent in them. -- Plutarch
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
