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

2019-02-05 Thread Niki Bowe
On Tue, Feb 5, 2019 at 4:33 PM Niki Bowe wrote: > Hi Carl. > You are of course correct. > Attached replacement patch. > > > On Mon, Feb 4, 2019 at 6:24 PM Carl Eugen Hoyos > wrote: > >> 2019-02-05 3:11 GMT+01:00, Nikolas Bowe : >> >> > +if (mov->tracks[i].eac3_priv) >> > +

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

2019-02-05 Thread Niki Bowe
Hi Carl. You are of course correct. Attached replacement patch. On Mon, Feb 4, 2019 at 6:24 PM Carl Eugen Hoyos wrote: > 2019-02-05 3:11 GMT+01:00, Nikolas Bowe : > > > +if (mov->tracks[i].eac3_priv) > > +av_freep(>tracks[i].eac3_priv); > > I would have expected that the

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

2019-02-04 Thread Carl Eugen Hoyos
2019-02-05 3:11 GMT+01:00, Nikolas Bowe : > +if (mov->tracks[i].eac3_priv) > +av_freep(>tracks[i].eac3_priv); I would have expected that the if() is unneeded, no? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

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

2019-02-04 Thread Nikolas Bowe
When using delay_moov mov_write_moov_tag gets called multiple times. Therefore we need to keep eac3_priv around for subsequent calls. Signed-off-by: Nikolas Bowe --- libavformat/movenc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/movenc.c