On 03/01/2017 11:23 AM, Luca Barbato wrote: > On 28/02/2017 20:11, John Stebbins wrote: >> and fix fate-lavf-gif: sync_put_bits before writing each chunk. >> Cached bits from the previous chunk get written in the next chunk. >> This causes inconsistent output if put_bits caches 64 bits instead >> of 32. sync_put_bits make sure < 8 bits are still cached. >> --- >> libavcodec/put_bits.h | 89 >> +++++++++++++++++++++++++++++++++++++++------------ >> libavformat/gif.c | 1 + >> tests/ref/lavf/gif | 2 +- >> 3 files changed, 70 insertions(+), 22 deletions(-) >> > Maybe you could split the fix from the feature. Looks file, is fate > happy after it? > >
sure, I can do sync_put_bits and gif change first in a separate patch. that will stand on it's own. then do 64bit put_bits after that. -- John GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01 83F0 49F1 D7B2 60D4 D0F7
signature.asc
Description: OpenPGP digital signature
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
