[FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

Dave Rice dave at dericed.com
Wed Oct 5 18:07:44 EEST 2016


Hi,

> On Oct 3, 2016, at 7:36 PM, James Almer <jamrial at gmail.com> wrote:
> 
> This patchsets implements the feature requested on ticket #4347.
> The first three patches are preparation work. The first one isn't
> strictly related to the implementation, but comes in handy
> nonetheless.
> 
> Patches 4 to 11 can be squashed into a single commit before pushing
> if that's prefered, but for review purposes i split things as one
> patch per Level 1 element being adapted to write a CRC32 element.
> 
> Fate tests are updated as needed.

I tested the patches and it works well and resolves ticket #4347. The CRC-32 Elements are in the right positions according to the latest CELLAR drafts on Matroska and the resulting files pass mkvalidator. Thanks much for this.
Dave Rice


More information about the ffmpeg-devel mailing list