[FFmpeg-devel] [PATCH] ffmpeg: explicitly write headers for files with no streams

2016-10-07 Thread Hendrik Leppkes
Recent changes to ffmpeg.c tied output file init to stream init, which broke stream-less files, specifically ffmetadata output. --- ffmpeg.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffmpeg.c b/ffmpeg.c index 454e193..b3e23ef 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -3460,6 +3

Re: [FFmpeg-devel] [PATCH] ffmpeg: explicitly write headers for files with no streams

2016-10-07 Thread Hendrik Leppkes
On Fri, Oct 7, 2016 at 3:46 PM, Hendrik Leppkes wrote: > Recent changes to ffmpeg.c tied output file init to stream init, which broke > stream-less files, specifically ffmetadata output. > --- > ffmpeg.c | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/ffmpeg.c b/ffmpeg.c > in

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Carl Eugen Hoyos
2016-10-06 22:40 GMT+02:00 Vittorio Giovara : >>> https://www.dropbox.com/s/w2uu37o11rvoz1q/moviedispmat.mp4?dl=1 >> what is the intended / correct SAR / DAR for this sample ? > > without the patch Stream #0:0(und): Video: h264 (High) (avc1 / > 0x31637661), yuv420p, 540x576 [SAR 1:1 DAR 15:16]

Re: [FFmpeg-devel] [PATCH] ffmpeg: explicitly write headers for files with no streams

2016-10-07 Thread Michael Niedermayer
On Fri, Oct 07, 2016 at 03:46:14PM +0200, Hendrik Leppkes wrote: > Recent changes to ffmpeg.c tied output file init to stream init, which broke > stream-less files, specifically ffmetadata output. > --- > ffmpeg.c | 10 ++ > 1 file changed, 10 insertions(+) LGTM thx [...] -- Michael

Re: [FFmpeg-devel] [PATCH] ffmpeg: explicitly write headers for files with no streams

2016-10-07 Thread Hendrik Leppkes
On Fri, Oct 7, 2016 at 4:00 PM, Michael Niedermayer wrote: > On Fri, Oct 07, 2016 at 03:46:14PM +0200, Hendrik Leppkes wrote: >> Recent changes to ffmpeg.c tied output file init to stream init, which broke >> stream-less files, specifically ffmetadata output. >> --- >> ffmpeg.c | 10 ++ >>

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: fix Tags master on seekable output if there are tags after the last stream duration

2016-10-07 Thread James Almer
On 10/7/2016 3:05 AM, James Almer wrote: > The dynamic AVIOContext would get closed pointing to the wrong position > in the buffer. > This is a regression since 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd. > > Signed-off-by: James Almer > --- > Example: > ./ffmpeg -i fate-samples/vorbis/vorbis_chapt

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-10-07 Thread Pallavi Kumari
>> iam interrested, i intend to mentor only one applicant though >> iam also the mentor for "Improve Selftest coverage" >> (hint if someone wants to take mentoring that over for example ...) Thank you for your interest :) Is there any applicant for ''Improve Selftest coverage'' ? >> Do you h

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: fix Tags master on seekable output if there are tags after the last stream duration

2016-10-07 Thread Dave Rice
> On Oct 7, 2016, at 10:30 AM, James Almer wrote: > > On 10/7/2016 3:05 AM, James Almer wrote: >> The dynamic AVIOContext would get closed pointing to the wrong position >> in the buffer. >> This is a regression since 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd. >> >> Signed-off-by: James Almer >

[FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Vittorio Giovara
This matrix needs to be applied after all others have (currently only display matrix from trak), but cannot be handled in movie box, since streams are not allocated yet. So store it in main context and if not identity, apply it when appropriate, handling the case when trak display matrix is identi

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: fix Tags master on seekable output if there are tags after the last stream duration

2016-10-07 Thread James Almer
On 10/7/2016 3:29 PM, Dave Rice wrote: > >> On Oct 7, 2016, at 10:30 AM, James Almer wrote: >> >> On 10/7/2016 3:05 AM, James Almer wrote: >>> The dynamic AVIOContext would get closed pointing to the wrong position >>> in the buffer. >>> This is a regression since 650e17d88b63b5aca6e0a43483e89e64

Re: [FFmpeg-devel] [PATCH] lavfi/metadata: fix metadata deletion if comparison returns false

2016-10-07 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 11:49:06PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavfilter/f_metadata.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) should be ok if paul has no comment/time thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF1

Re: [FFmpeg-devel] [PATCH] libavformat/tee: Add fifo support for tee

2016-10-07 Thread Michael Niedermayer
On Fri, Oct 07, 2016 at 01:03:03AM +0200, sebechlebsky...@gmail.com wrote: > From: Jan Sebechlebsky > > Signed-off-by: Jan Sebechlebsky > --- > This commit makes use of fifo muxer together with tee muxer > easier, fifo muxer does not have to be explicitly specified > for each slave. For the most

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Michael Niedermayer
On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. > > So store it in main context and if not identity, app

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Michael Niedermayer
On Sat, Oct 08, 2016 at 02:38:27AM +0200, Michael Niedermayer wrote: > On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: > > This matrix needs to be applied after all others have (currently only > > display matrix from trak), but cannot be handled in movie box, since > > streams are

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-10-07 Thread Michael Niedermayer
On Fri, Oct 07, 2016 at 10:49:42PM +0530, Pallavi Kumari wrote: > >> iam interrested, i intend to mentor only one applicant though > >> iam also the mentor for "Improve Selftest coverage" > >> (hint if someone wants to take mentoring that over for example ...) > > Thank you for your interest

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Vittorio Giovara
On Fri, Oct 7, 2016 at 8:38 PM, Michael Niedermayer wrote: > On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: >> This matrix needs to be applied after all others have (currently only >> display matrix from trak), but cannot be handled in movie box, since >> streams are not allocat

Re: [FFmpeg-devel] [PATCH] ffmpeg: remove unused and errorneous AVFrame timestamp check

2016-10-07 Thread Hendrik Leppkes
On Thu, Oct 6, 2016 at 4:08 PM, Hendrik Leppkes wrote: > On Tue, Oct 4, 2016 at 4:44 PM, James Almer wrote: >> On 10/4/2016 11:35 AM, Hendrik Leppkes wrote: >>> On Tue, Oct 4, 2016 at 4:32 PM, wm4 wrote: On Tue, 4 Oct 2016 14:15:03 +0200 Michael Niedermayer wrote: > On Tue, O