Justin Ruggles <[email protected]> writes:

> It may have improved cross-platform stability, but wasn't the only place in
> the encoder with bitexact issues. It is no longer needed because we have FATE
> tests for float encoders using fuzzy comparison.
> ---
>  libavcodec/wmaenc.c |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/wmaenc.c b/libavcodec/wmaenc.c
> index 6fd3494..b9c40d8 100644
> --- a/libavcodec/wmaenc.c
> +++ b/libavcodec/wmaenc.c
> @@ -306,10 +306,6 @@ static int encode_block(WMACodecContext *s, float 
> (*src_coefs)[BLOCK_MAX_SIZE],
>                          if(1<<coef_nb_bits <= abs_level)
>                              return -1;
>
> -
> -                        //Workaround minor rounding differences for the 
> regression tests, FIXME we should find and replace the problematic float by 
> fixpoint for reg tests
> -                        if(abs_level == 0x71B && (s->avctx->flags & 
> CODEC_FLAG_BITEXACT)) abs_level=0x71A;
> -
>                          put_bits(&s->pb, coef_nb_bits, abs_level);
>                          put_bits(&s->pb, s->frame_len_bits, run);
>                      }
> -- 

OK

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to