[FFmpeg-devel] [PATCH] opus/matroska: Adding support for End Trimming in demuxer/decoder
vigneshv at google.com
Tue Sep 3 19:46:34 CEST 2013
On Mon, Sep 2, 2013 at 4:30 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Fri, Aug 30, 2013 at 11:25:12AM -0700, Vignesh Venkatasubramanian wrote:
>> Implementing support for end trimming Opus in Matroska by making
>> use of the DiscardPadding value from the container and discarding
>> the samples accordingly. With this patch and its companion muxer
>> patch, we can ensure number of samples in == number of samples out
>> for Opus in Matroska.
>> Signed-off-by: Vignesh Venkatasubramanian <vigneshv at google.com>
>> libavcodec/utils.c | 21 +++++++++++++++++++++
>> libavformat/matroskadec.c | 30 +++++++++++++++++++++++++-----
>> 2 files changed, 46 insertions(+), 5 deletions(-)
> this (with the other patched) causes fate to fail
when you say other patched, do you mean the muxer and encoder patches? or the
memset patch? because i applied all the 4 patches and fate passes for me.
just to be clear, by 4 patches i mean the patches with following titles:
* avpacket: Fixing side data copy when src == dst
* opus/matroska: Adding support for DiscardPadding in muxer
* lavcodec: Adding support for End Trimming in Opus encoder
* opus/matroska: Adding support for End Trimming in demuxer/decoder
could you please check once again? thanks.
> --- ./tests/ref/fate/exif-image-embedded 2013-09-02 12:10:30.045471774 +0200
> +++ tests/data/fate/exif-image-embedded 2013-09-03 01:29:07.774481269 +0200
> @@ -34,7 +34,7 @@
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> The real ebay dictionary, page 2
> "100% positive feedback" - "All either got their money back or didnt complain"
> "Best seller ever, very honest" - "Seller refunded buyer after failed scam"
Vignesh Venkat | Software Engineer | vigneshv at google.com | 650-861-1330
More information about the ffmpeg-devel