Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: fix cue relative position values when CRC32 is enabled

2016-10-26 Thread James Almer
On 10/25/2016 12:38 PM, James Almer wrote: > The dynamic buffer does not contain the CRC32 element so calls to avio_tell() > don't take it into account. This resulted in CueRelativePosition values being > six bytes short. > This is a regression since 6724525a1576ca334d2ffdc085620bb44aea7394

[FFmpeg-devel] [PATCH] avformat/matroskaenc: fix cue relative position values when CRC32 is enabled

2016-10-25 Thread James Almer
The dynamic buffer does not contain the CRC32 element so calls to avio_tell() don't take it into account. This resulted in CueRelativePosition values being six bytes short. This is a regression since 6724525a1576ca334d2ffdc085620bb44aea7394 Instead of adding yet another custom check for CRC32 to