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

Michael Niedermayer michael at niedermayer.cc
Thu Oct 6 21:28:16 EEST 2016


On Thu, Oct 06, 2016 at 11:27:08AM -0300, James Almer wrote:
> On 10/6/2016 7:29 AM, Michael Niedermayer wrote:
> > On Mon, Oct 03, 2016 at 08:36:56PM -0300, James Almer 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.
> > 
> > some questions
> > Does this reduce writing speed ? for example when remuxing high
> > bitrate data like rawvideo ?
> 
> The bulk of the CRC calculation is done once per level 1 element, so it
> shouldn't affect remuxing speed in a considerable way. But if preferred,
> i could see about adding an option to disable it.

i guess such a option cant hurt

thanks

no furter comments from me about this patchset

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161006/1603bad4/attachment.sig>


More information about the ffmpeg-devel mailing list