[FFmpeg-devel] [PATCH] avformat/movenc: Fix muxing EAC3 with delay_moov

Niki Bowe nbowe at google.com
Wed Feb 6 02:36:09 EET 2019


On Tue, Feb 5, 2019 at 4:33 PM Niki Bowe <nbowe at google.com> wrote:

> Hi Carl.
> You are of course correct.
> Attached replacement patch.
>
>
> On Mon, Feb 4, 2019 at 6:24 PM Carl Eugen Hoyos <ceffmpeg at gmail.com>
> wrote:
>
>> 2019-02-05 3:11 GMT+01:00, Nikolas Bowe <nbowe-at-google.com at ffmpeg.org>:
>>
>> > +        if (mov->tracks[i].eac3_priv)
>> > +            av_freep(&mov->tracks[i].eac3_priv);
>>
>> I would have expected that the if() is unneeded, no?
>>
>> Carl Eugen
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
>
> --
>
> Nikolas Bowe |  SWE |  nbowe at google.com |  408-565-5137
>


-- 

Nikolas Bowe |  SWE |  nbowe at google.com |  408-565-5137
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avformat-movenc-Fix-muxing-EAC3-with-delay_moov.patch
Type: text/x-patch
Size: 1086 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190205/2215ecd1/attachment.bin>


More information about the ffmpeg-devel mailing list