Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Regression fix for invalid MKV headers

2017-01-04 Thread Soft Works
==> Revised patch in plain text (the attachment from my previous reply wasn't recognized by patchwork) The following three commits created a regression by writing initially invalid mkv headers: 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd avformat/matroskaenc: write a CRC32 element on Tags

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Regression fix for invalid MKV headers

2017-01-04 Thread Soft Works
> From: ffmpeg-devel on behalf of Michael > Niedermayer > > this patch breaks fate > make fate I apologize, I had run fate but for some reason I hadn't got an error. But now I ran again, found and fixed the error. Please find attached

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Regression fix for invalid MKV headers

2017-01-04 Thread Michael Niedermayer
On Tue, Jan 03, 2017 at 11:49:07PM +, Soft Works wrote: > The following three commits created a regression by writing initially > invalid mkv headers: > > 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd avformat/matroskaenc: write a > CRC32 element on Tags > 3bcadf822711720ff0f8d14db71ae47cdf97e652