On Tue, Sep 04, 2012 at 03:25:02PM +0300, Martin Storsjö wrote: > On Tue, 4 Sep 2012, Kostya Shishkov wrote: > > >On Tue, Sep 04, 2012 at 02:21:25PM +0300, Martin Storsjö wrote: > >>From: Michael Niedermayer <[email protected]> > >> > >>Previously, the value given to put_bits was 10 bits long for positive > >>predictors, even though 9 bits were to be written. The extra bit could > >>in some cases overwrite existing bits in the bitstream writer cache. > >> > >>This fixes a failed assert in put_bits.h, when running a version > >>built with -DDEBUG. > >> > >>The fate test result gets slightly improved, thanks to getting rid > >>of the overwritten bits in the bitstream writer cache. > >>--- > >> libavcodec/adpcmenc.c | 2 +- > >> tests/ref/fate/acodec-adpcm-ima_qt | 6 +++--- > >> 2 files changed, 4 insertions(+), 4 deletions(-) > > > >does the output differ if you use put_sbits() instead of put_bits()? > > Yes, that gives you the exact same output as with this patch.
The patch is OK then _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
