[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

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] [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

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),

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

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

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

[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

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

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:

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. >> >>

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,

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

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

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] 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

[FFmpeg-devel] [PATCH 2/2 v2] avformat/matroskaenc: fix targets for attachment tags

2016-10-07 Thread James Almer
Attachment tags were being written targetting non-existent streams in the output file. Also filter filename and mimetype entries, as they are standard elements in the Attachment master. Signed-off-by: James Almer --- Now rebased after the CRC32 patchset. The fileuid is