Re: [FFmpeg-devel] [PATCH 3/4] V13 - SCTE-35 support in hlsenc

2016-10-15 Thread Anssi Hannula
04.10.2016, 02:14, Carlos Fernandez Sanz kirjoitti: > From: Carlos Fernandez > > Signed-off-by: Carlos Fernandez > --- > libavformat/Makefile | 2 +- > libavformat/hlsenc.c | 108 +-- > libavformat/scte_35.c | 527 > ++ > libavformat/

Re: [FFmpeg-devel] [PATCH] mov: move stsd finalization to an appropriate place

2016-10-15 Thread Hendrik Leppkes
On Fri, Oct 14, 2016 at 3:21 PM, Carl Eugen Hoyos wrote: > 2016-10-14 13:56 GMT+02:00 Hendrik Leppkes : >> mov_finalize_stsd_codec parses stream information from the ALAC extradata, >> so run it after the extradata processing is completed in mov_read_stsd. > > Please mention ticket #5826 if your p

Re: [FFmpeg-devel] [PATCH 2/2] lavf/mp3enc: write encoder delay/padding upon closing

2016-10-15 Thread Andreas Cadhalpun
On 05.10.2016 21:37, Jon Toohill wrote: > --- > libavformat/mp3enc.c | 34 +++--- > 1 file changed, 27 insertions(+), 7 deletions(-) > > diff --git a/libavformat/mp3enc.c b/libavformat/mp3enc.c > index de63401..ddf4b93 100644 > --- a/libavformat/mp3enc.c > +++ b/libavf

Re: [FFmpeg-devel] [PATCH 1/2] lavc/libmp3lame: send encoder delay/padding in packet side data

2016-10-15 Thread Andreas Cadhalpun
On 05.10.2016 21:37, Jon Toohill wrote: > --- > libavcodec/libmp3lame.c | 26 +- > 1 file changed, 25 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/libmp3lame.c b/libavcodec/libmp3lame.c > index 5642264..b3ba0d8 100644 > --- a/libavcodec/libmp3lame.c > +++ b/lib

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-15 Thread Nicolas George
Le tridi 23 vendémiaire, an CCXXV, wm4 a écrit : > XML is very complex This is the usual, and often only, argument raised in this kind of situation. And unfortunately, I already addressed it in this very thread. XML is very complex, yes: processing instructions, entity definitions, DTSs and

[FFmpeg-devel] [PATCH 3/3] avcodec/utils: print Chroma Location string in verbose log level

2016-10-15 Thread James Almer
It's container level information on some formats (Matroska, MXF, yuv4mpeg), so it should be printed at higher log levels than debug. Signed-off-by: James Almer --- libavcodec/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index

[FFmpeg-devel] [PATCH 2/3] avformat/matroskadec: support parsing Chroma Location elements

2016-10-15 Thread James Almer
Signed-off-by: James Almer --- libavformat/matroskadec.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index acf1ccb..cfe4692 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -1826,6 +1826,10 @@ static int mk

[FFmpeg-devel] [PATCH 1/3] avformat/matroskaenc: support writting Chroma Location elements

2016-10-15 Thread James Almer
Signed-off-by: James Almer --- libavformat/matroskaenc.c | 8 1 file changed, 8 insertions(+) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index a48f5bb..d6557a1 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -829,6 +829,14 @@ static int

[FFmpeg-devel] [PATCH] avformat/tests/gitignore: add fifo_muxer entry

2016-10-15 Thread Zhao Zhili
From a3ee9afc37331315e4ec57f1ebf881779b62bf60 Mon Sep 17 00:00:00 2001 From: Zhao Zhili Date: Sun, 16 Oct 2016 00:09:25 +0800 Subject: [PATCH] avformat/tests/gitignore: add fifo_muxer entry --- libavformat/tests/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/tests/.gi

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

2016-10-15 Thread Pallavi Kumari
I will makes the necessary changes. On Sat, Oct 15, 2016 at 5:45 AM, Michael Niedermayer wrote: > On Sat, Oct 15, 2016 at 03:33:27AM +0530, Pallavi Kumari wrote: > > Hi Michael, > > > > Please find the task at https://github.com/atana1/audio_stream > > The code should be a avfilter in libavfilte

[FFmpeg-devel] [PATCH] Improved Selftest Coverage for libavcodec

2016-10-15 Thread Thomas Turner
Added test for libavcodec/avpacket.c Function(s) tested: av_packet_clone() Signed-off-by: Thomas Turner --- libavcodec/Makefile | 3 +- libavcodec/tests/avpacket.c | 115 tests/fate/libavcodec.mak | 5 ++ 3 files changed, 122 insertions(

Re: [FFmpeg-devel] [PATCH] RTSP: pass TLS args for RTSPS

2016-10-15 Thread Michael Niedermayer
On Sat, Oct 01, 2016 at 04:20:39PM -0400, jayri...@gmail.com wrote: > From: Jay Ridgeway > > > This patch enables TLS args for RTSPS. This is necessary for client > certificates and cert validation. > > Squash changes from feedback into one patch. > > --- > libavformat/rtsp.c|

Re: [FFmpeg-devel] [PATCH] Improved Selftest Coverage for libavcodec

2016-10-15 Thread Michael Niedermayer
On Sat, Oct 15, 2016 at 03:05:11AM -0700, Thomas Turner wrote: > Added test for libavcodec/avpacket.c > Function(s) tested: av_packet_clone() > > Signed-off-by: Thomas Turner > --- > libavcodec/Makefile | 3 +- > libavcodec/tests/avpacket.c | 115 >

Re: [FFmpeg-devel] [PATCH] Improved Selftest Coverage for libavcodec

2016-10-15 Thread Michael Niedermayer
On Sat, Oct 15, 2016 at 03:05:11AM -0700, Thomas Turner wrote: > Added test for libavcodec/avpacket.c > Function(s) tested: av_packet_clone() The commit message should also have a first line that is "topic: short description" see the "Development Policy" for more information http://ffmpeg.org/dev

Re: [FFmpeg-devel] [PATCH] RTSP: pass TLS args for RTSPS

2016-10-15 Thread Jay
Thank you for the feedback. I have been trying to get RTSPS cert validation incorporated for several weeks. I would greatly appreciate someone on the core team helping me guide this to completion. Please find your questions answered below. > the get_file_handle extensions should be in a spererate

[FFmpeg-devel] Refund request for FFmpeg at LinuxCon 2016

2016-10-15 Thread Thomas Volkert
Hi, I'd like to ask for refund of my expenses for LinuxCon (see thread "FFmpeg at LinuxCon 2016"): - 191 € for 2 * 675km by car I will contact Stefano if there are no objections. Best regards, Thomas. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

[FFmpeg-devel] [PATCH 1/2] avformat/matroskadec: set aspect ratio only when DisplayWidth and DisplayHeight are in pixels

2016-10-15 Thread James Almer
A missing DisplayUnit element or one with the default value of 0 means DisplayWidth and DisplayHeight should be interpreted as pixels. The current code setting st->sample_aspect_ratio is wrong when DisplayUnit is anything else. Signed-off-by: James Almer --- libavformat/matroska.h| 8 +

[FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: write a DisplayUnit element when aspect ratio is unknown

2016-10-15 Thread James Almer
We don't currently support values 1 (centimeters), 2 (inches) or 3 (DAR), only the default value 0 (pixels) which doesn't need to be written. The fate refs are updated as unknown SAR is now correctly propagated by the matroska demuxer. Signed-off-by: James Almer --- See the relevant discussion i

Re: [FFmpeg-devel] [PATCH] avfilter/showcqt: add csp option

2016-10-15 Thread Muhammad Faiz
On Fri, Oct 14, 2016 at 8:47 AM, Michael Niedermayer wrote: > On Fri, Oct 14, 2016 at 08:05:15AM +0700, Muhammad Faiz wrote: >> On Fri, Oct 14, 2016 at 3:20 AM, Michael Niedermayer >> wrote: >> > On Fri, Oct 14, 2016 at 02:50:52AM +0700, Muhammad Faiz wrote: >> >> On Thu, Oct 13, 2016 at 10:38 PM

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskadec: set aspect ratio only when DisplayWidth and DisplayHeight are in pixels

2016-10-15 Thread Carl Eugen Hoyos
2016-10-16 0:09 GMT+02:00 James Almer : > +if (track->video.display_unit == > MATROSKA_VIDEO_DISPLAYUNIT_PIXELS) If you don't mind please add braces here. Both patches make sense afaict. Thank you, Carl Eugen ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] avfilter/firequalizer: add scale option

2016-10-15 Thread Muhammad Faiz
Signed-off-by: Muhammad Faiz --- doc/filters.texi | 13 + libavfilter/af_firequalizer.c | 33 ++--- 2 files changed, 43 insertions(+), 3 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index f5cc9b7..d4eec70 100644 --- a/doc/filt

Re: [FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: write a DisplayUnit element when aspect ratio is unknown

2016-10-15 Thread Kieran O Leary
Hi, On Sat, Oct 15, 2016 at 11:09 PM, James Almer wrote: > > -#sar 0: 1/1 > +#sar 0: 0/1 > This commit looks very useful. A few archivists (including myself) encountered this issue, so this is a wonderful contribution! Thanks James! -Kieran. ___ ffmp

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskadec: set aspect ratio only when DisplayWidth and DisplayHeight are in pixels

2016-10-15 Thread James Almer
On 10/15/2016 8:07 PM, Carl Eugen Hoyos wrote: > 2016-10-16 0:09 GMT+02:00 James Almer : > >> +if (track->video.display_unit == >> MATROSKA_VIDEO_DISPLAYUNIT_PIXELS) > > If you don't mind please add braces here. Added brackets, which i assume is what you meant. > > Both patches ma

Re: [FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: write a DisplayUnit element when aspect ratio is unknown

2016-10-15 Thread Dave Rice
> On Oct 15, 2016, at 7:54 PM, Kieran O Leary wrote: > > Hi, > > On Sat, Oct 15, 2016 at 11:09 PM, James Almer wrote: > >> >> -#sar 0: 1/1 >> +#sar 0: 0/1 >> > > This commit looks very useful. A few archivists (including myself) > encountered this issue, so this is a wonderful contribution

Re: [FFmpeg-devel] [PATCH] RTSP: pass TLS args for RTSPS

2016-10-15 Thread Michael Niedermayer
On Sat, Oct 15, 2016 at 08:31:23PM +, Jay wrote: > Thank you for the feedback. I have been trying to get RTSPS cert validation > incorporated for several weeks. I would greatly appreciate someone on the > core team helping me guide this to completion. Please find your questions > answered below

Re: [FFmpeg-devel] [PATCH 1/2] img2: added support for %t output pattern

2016-10-15 Thread Roger Pack
On 10/10/16, Roger Pack wrote: > On 9/22/16, Roger Pack wrote: >> On 1/4/12, Yuval Adam wrote: >>> From: Yuval Adam >>> >>> The image2 muxer now supports timestamps in output filenames. >>> When used in an output patterm '%t' will be replaced with the frames >>> timestamp in hours, minutes and

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-15 Thread Steven Liu
update 2016-10-15 0:08 GMT+08:00 Steven Liu : > patch update. > > test passed: > Linux: > ../configure > MingW: > ../configure --cc='ccache x86_64-w64-mingw32-gcc' --arch=x86_64 > --target-os=mingw32 --cross-prefix=x86_64-w64-mingw32- --disable-yasm > > test passed with hds demux > Linux: > ../