Andreas Rheinhardt: > Hello, > > this patchset is mostly concerned with reducing the unnecessary overhead > of the Matroska muxer; this overhead mostly comes from not using the > minimal amount of bytes when writing the length field of master > elements. The length fields of most level 1 elements are currently > written on eight bytes; after this patchset, they are always written > with the minimal amount of bytes. > > Currently, writing level 1 elements always uses dynamic buffers > (regardless of whether one is in seekable or non-seekable mode), but > strangely, there are nevertheless two different codepaths for writing > for the seekable and the non-seekable modes. These have been unified; as > a consequence, CRC-32 elements are now also written in > non-seekable/live-mode. > > Given that this patchset changes the output, I had to update some fate > tests. This is the first time I did this, so I hope I did everything > right. But doublechecking wouldn't hurt. > > - Andreas > > Andreas Rheinhardt (15): > avformat/matroskaenc: Fix relative timestamp check > avformat/matroskaenc: Fix BlockGroup size calculation > avformat/matroskaenc: Slightly improve size bounds for cues > avformat/matroskaenc: Don't waste bytes in EBML Header > avformat/matroskaenc: Remove traces of secondary seek head > avformat/matroskaenc: Simplify check for writing CRCs > avformat/matroskaenc: Improve log message > avformat/matroskaenc: Remove redundant check > avformat/matroskaenc: Change variable types > avformat/matroskaenc: Avoid seeking when writing level 1 elements > avformat/matroskaenc: Write CRC-32 in non-seekable mode > avformat/matroskaenc: Cosmetics and typo > avformat/matroskaenc: Don't waste bytes writing level 1 elements > avformat/matroskaenc: Improve log messages for blocks > avformat/matroskaenc: Reduce usage of ebml_master > > libavformat/matroskaenc.c | 319 +++++++++++++-------------- > tests/fate/matroska.mak | 2 +- > tests/fate/wavpack.mak | 4 +- > tests/ref/fate/aac-autobsf-adtstoasc | 4 +- > tests/ref/fate/binsub-mksenc | 2 +- > tests/ref/fate/rgb24-mkv | 4 +- > tests/ref/lavf/mka | 4 +- > tests/ref/lavf/mkv | 4 +- > tests/ref/lavf/mkv_attachment | 4 +- > tests/ref/seek/lavf-mkv | 44 ++-- > 10 files changed, 185 insertions(+), 206 deletions(-) >
I'd like to ping this patchset. And given that my original submissions was munged (the sender was changed to "Andreas Rheinhardt via ffmpeg-devel") I want to know if it's appropriate to resend this whole patchset or not. - Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
