Re: [FFmpeg-devel] [PATCH] avformat/assenc: mux all extradata at once

2014-09-21 Thread Nicolas George
Le jour de la Raison, an CCXXII, Clément Bœsch a écrit : > Before this commit, the code was muxing up to the 2nd line after > "[Event]" (assuming it to be the "Format:" line). The remaining are > generally "Comment:" directives which can stay in that place. mkvextract > behaves that way so it seems

Re: [FFmpeg-devel] [PATCH] avformat/assenc: mux all extradata at once

2014-09-20 Thread wm4
On Sat, 20 Sep 2014 18:40:57 +0200 Clément Bœsch wrote: > Before this commit, the code was muxing up to the 2nd line after > "[Event]" (assuming it to be the "Format:" line). The remaining are > generally "Comment:" directives which can stay in that place. mkvextract > behaves that way so it seem

Re: [FFmpeg-devel] [PATCH] avformat/assenc: mux all extradata at once

2014-09-20 Thread Clément Bœsch
On Sat, Sep 20, 2014 at 06:40:57PM +0200, Clément Bœsch wrote: > Before this commit, the code was muxing up to the 2nd line after > "[Event]" (assuming it to be the "Format:" line). The remaining are EventS* typo fixed locally > generally "Comment:" directives which can stay in that place. mkvext

[FFmpeg-devel] [PATCH] avformat/assenc: mux all extradata at once

2014-09-20 Thread Clément Bœsch
Before this commit, the code was muxing up to the 2nd line after "[Event]" (assuming it to be the "Format:" line). The remaining are generally "Comment:" directives which can stay in that place. mkvextract behaves that way so it seems there is no reason for that extra complexity. --- libavformat/a