Re: [FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference

On 10/16/2016 9:30 PM, James Almer wrote: > On 10/16/2016 5:11 PM, Andreas Cadhalpun wrote: >> The problem was introduced in commit 1273bc6. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/matroskadec.c | 2 +- >> 1 file changed, 1 insertion(+), 1

Re: [FFmpeg-devel] [PATCH] astdec: fix division by zero

On Sun, Oct 16, 2016 at 10:41:32PM +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavformat/astdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/astdec.c b/libavformat/astdec.c > index

Re: [FFmpeg-devel] [PATCH] aiffdec: fix division by zero

On Sun, Oct 16, 2016 at 10:38:42PM +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavformat/aiffdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c > index

Re: [FFmpeg-devel] comma at the end of enumerator lists

On Mon, Oct 17, 2016 at 12:11:08AM +0200, Michael Behrisch wrote: > Am 16.10.2016 um 23:24 schrieb Clément Bœsch: > > On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: > >> Hi all, > >> first of all thanks for providing this great library. > >> > >> Today my pull request

Re: [FFmpeg-devel] FFmpeg 3.2

On Sun, Oct 16, 2016 at 06:01:54PM -0400, Helmut K. C. Tessarek wrote: > On 2016-09-27 09:30, Michael Niedermayer wrote: > > Its long since FFmpeg 3.1, so its time to make 3.2 > > ill branch release/3.2 off master and make 3.2 in maybe about a week or > > 2 unless something delays it > > What

[FFmpeg-devel] [PATCH 1/2] avformat/mov: zero initialize codec_name in mov_parse_stsd_video()

Fixes valgrind warning about "Conditional jump or move depends on uninitialised value(s)" Signed-off-by: James Almer --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index add1812..7462ecf 100644

[FFmpeg-devel] [PATCH 2/2] avformat/mov: pass the demuxer's AVFormatContext to avpriv_request_sample()

Signed-off-by: James Almer --- libavformat/mov.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 7462ecf..5189706 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2762,7 +2762,8 @@ static int

Re: [FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference

On 10/16/2016 5:11 PM, Andreas Cadhalpun wrote: > The problem was introduced in commit 1273bc6. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/matroskadec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [FFmpeg-devel] comma at the end of enumerator lists

Hi, On Sun, Oct 16, 2016 at 6:11 PM, Michael Behrisch wrote: > Am 16.10.2016 um 23:24 schrieb Clément Bœsch: > > On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: > >> Hi all, > >> first of all thanks for providing this great library. > >> > >> Today my pull

[FFmpeg-devel] [PATCH] avfilter/firequalizer: optimize gain/gain_entry command

do not rebuild when gain/gain_entry command are equal with old gain/gain_entry Signed-off-by: Muhammad Faiz --- libavfilter/af_firequalizer.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/libavfilter/af_firequalizer.c

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

this is fontconfig pattern Signed-off-by: Muhammad Faiz --- doc/filters.texi | 10 ++ libavfilter/avf_showcqt.c | 85 +++ libavfilter/avf_showcqt.h | 1 + 3 files changed, 89 insertions(+), 7 deletions(-) diff --git

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

2016-10-16 22:54 GMT+08:00 Michael Niedermayer : > On Sun, Oct 16, 2016 at 11:32:37AM +0800, Steven Liu wrote: > > update > > you didnt attach any new patch > Just change the help message from disable to enable > > [...] > -- > Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] fate: add swr-convertaudio test

test for flt to s16 should pass on correct rounding to nearest Signed-off-by: Muhammad Faiz --- tests/fate/libswresample.mak | 8 1 file changed, 8 insertions(+) diff --git a/tests/fate/libswresample.mak b/tests/fate/libswresample.mak index 93b8d6e..7cb3bdb 100644

Re: [FFmpeg-devel] comma at the end of enumerator lists

Am 16.10.2016 um 23:24 schrieb Clément Bœsch: > On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: >> Hi all, >> first of all thanks for providing this great library. >> >> Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been >> rejected which tried to remove the

Re: [FFmpeg-devel] FFmpeg 3.2

On 2016-09-27 09:30, Michael Niedermayer wrote: > Its long since FFmpeg 3.1, so its time to make 3.2 > ill branch release/3.2 off master and make 3.2 in maybe about a week or > 2 unless something delays it What happened to 3.2? Cheers, K. C. -- regards Helmut K. C. Tessarek lookup

Re: [FFmpeg-devel] FFmpeg 3.2

binp92P_YWvzj.bin Description: PGP/MIME version identification encrypted.asc Description: OpenPGP encrypted message ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] comma at the end of enumerator lists

On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: > Hi all, > first of all thanks for providing this great library. > > Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been > rejected which tried to remove the comma at the end of enumerator lists > which

Re: [FFmpeg-devel] [PATCH] avformat/rtsp: support rtsps

This is the approach present in avformat/rtsp.c, but I can modify. What is preferred? For clarity, this patch does not copy url params from the `rtsps://` input. It passes tls params provided as cmd line options to the `tls://` protocol handler. Personally, I would like tls negotiation to be

[FFmpeg-devel] [PATCH] westwood_aud: prevent division by zero

Signed-off-by: Andreas Cadhalpun --- libavformat/westwood_aud.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/westwood_aud.c b/libavformat/westwood_aud.c index 4750167..9c2d35c 100644 --- a/libavformat/westwood_aud.c +++

[FFmpeg-devel] [PATCH] astdec: fix division by zero

Signed-off-by: Andreas Cadhalpun --- libavformat/astdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/astdec.c b/libavformat/astdec.c index f3ca721..7a53d0b 100644 --- a/libavformat/astdec.c +++ b/libavformat/astdec.c @@ -90,7

[FFmpeg-devel] [PATCH] aiffdec: fix division by zero

Signed-off-by: Andreas Cadhalpun --- libavformat/aiffdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c index cd916f9..de82787 100644 --- a/libavformat/aiffdec.c +++ b/libavformat/aiffdec.c @@

[FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference

The problem was introduced in commit 1273bc6. Signed-off-by: Andreas Cadhalpun --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 8847c62..a5d3c0e 100644

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

On Sun, Oct 16, 2016 at 6:10 AM, Muhammad Faiz wrote: > Signed-off-by: Muhammad Faiz > --- > doc/filters.texi | 13 + > libavfilter/af_firequalizer.c | 33 ++--- > 2 files changed, 43 insertions(+), 3

Re: [FFmpeg-devel] [PATCH 3/3] fate: add loudnorm filter test

On Sun, 16 Oct 2016, Marton Balint wrote: Signed-off-by: Marton Balint --- tests/fate/filter-audio.mak | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tests/fate/filter-audio.mak b/tests/fate/filter-audio.mak index 9c6f7cd..d376f25 100644 ---

Re: [FFmpeg-devel] comma at the end of enumerator lists

On Sun, 16 Oct 2016 21:51:32 +0200 Nicolas George wrote: > Le quintidi 25 vendémiaire, an CCXXV, Michael Behrisch a écrit : > > Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been > > rejected which tried to remove the comma at the end of enumerator lists >

Re: [FFmpeg-devel] comma at the end of enumerator lists

Le quintidi 25 vendémiaire, an CCXXV, Michael Behrisch a écrit : > Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been > rejected which tried to remove the comma at the end of enumerator lists > which triggers a warning when compiling with gcc and -Wpedantic. I still > think

[FFmpeg-devel] comma at the end of enumerator lists

Hi all, first of all thanks for providing this great library. Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been rejected which tried to remove the comma at the end of enumerator lists which triggers a warning when compiling with gcc and -Wpedantic. I still think it could be

Re: [FFmpeg-devel] [PATCH 1/3] tools: add loudnorm script example to use loudnorm

Le quintidi 25 vendémiaire, an CCXXV, Marton Balint a écrit : > From: Kyle Swanson > > Signed-off-by: Kyle Swanson > Signed-off-by: Marton Balint > --- > tools/loudnorm.rb | 60 > +++ > 1 file

[FFmpeg-devel] [PATCH 1/3] tools: add loudnorm script example to use loudnorm

From: Kyle Swanson Signed-off-by: Kyle Swanson Signed-off-by: Marton Balint --- tools/loudnorm.rb | 60 +++ 1 file changed, 60 insertions(+) create mode 100755 tools/loudnorm.rb diff --git

[FFmpeg-devel] [PATCH 2/3] lavfi/loudnorm: add an internal libebur128 library

Also contains the following changes to the library: - add ff_ prefix to functions - remove cplusplus defines. - add FF_ prefix to contants and some structs - remove true peak calculation feature, since it uses its own resampler, and af_audnorm does not need it. - remove version info and some

[FFmpeg-devel] [PATCH 3/3] fate: add loudnorm filter test

Signed-off-by: Marton Balint --- tests/fate/filter-audio.mak | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tests/fate/filter-audio.mak b/tests/fate/filter-audio.mak index 9c6f7cd..d376f25 100644 --- a/tests/fate/filter-audio.mak +++ b/tests/fate/filter-audio.mak @@

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

On Sat, Oct 15, 2016 at 1:44 AM, Anssi Hannula wrote: > > Function pointers are not useful here as they always point to the same > functions. Function pointer are kept there so that scte_35.c function are not exposed to users using ffmpeg library. > > Also, the members of

Re: [FFmpeg-devel] replacing -r for -framerate in StreamingGuide

On Sun, Oct 16, 2016, at 03:39 AM, Sven C. Dack wrote: > Hello, > > I've just noticed how the StreamingGuide > https://trac.ffmpeg.org/wiki/StreamingGuide uses -r for the framerate in > its > examples. I believe the correct command line argument here now is > -framerate. Thanks, but it is a

Re: [FFmpeg-devel] [PATCH] avformat/rtsp: support rtsps

On Sun, 16 Oct 2016 12:24:37 -0400 jayri...@gmail.com wrote: > From: Jay Ridgeway > > Pass TLS args to support RTSPS. This patch requires TLS patch. > --- > libavformat/rtsp.c | 19 --- > libavformat/rtsp.h | 8 > 2 files changed, 24 insertions(+),

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

On 10/16/2016 2:01 PM, Nicolas George wrote: > Le quintidi 25 vendémiaire, an CCXXV, James Almer a écrit : >>> Nothing probably, just me not giving it much thought after i couldn't find >>> any sample using cm, in or dar instead of pixels, and assuming the >>> calculations were tailored

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

Le quintidi 25 vendémiaire, an CCXXV, James Almer a écrit : > > Nothing probably, just me not giving it much thought after i couldn't find > > any sample using cm, in or dar instead of pixels, and assuming the > > calculations were tailored specifically for sizes in pixels. > > I manually created

[FFmpeg-devel] [PATCH] avformat/rtsp: support rtsps

From: Jay Ridgeway Pass TLS args to support RTSPS. This patch requires TLS patch. --- libavformat/rtsp.c | 19 --- libavformat/rtsp.h | 8 2 files changed, 24 insertions(+), 3 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index

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

OK. I understand. I do not see anything like that in tls. On Sun, Oct 16, 2016 at 11:06 AM Michael Niedermayer wrote: > On Sun, Oct 16, 2016 at 01:43:24PM +, Jay wrote: > > On Sat, Oct 15, 2016 at 10:44 PM Michael Niedermayer > > > wrote: > >

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

On Mon, Oct 10, 2016 at 02:56:24PM -0600, 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

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

On 10/16/2016 11:32 AM, Michael Niedermayer wrote: > On Sat, Oct 15, 2016 at 12:40:56PM -0300, James Almer wrote: >> 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

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

On Sun, Oct 16, 2016 at 01:43:24PM +, Jay wrote: > On Sat, Oct 15, 2016 at 10:44 PM Michael Niedermayer > wrote: > > > 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 > > >

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

On Sat, Oct 15, 2016 at 12:40:56PM -0300, James Almer wrote: > 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

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

On Sat, Oct 15, 2016 at 10:44 PM Michael Niedermayer wrote: > 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

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

On 10/16/2016 10:28 AM, James Almer wrote: > On 10/16/2016 5:58 AM, Nicolas George wrote: >> > Le quartidi 24 vendémiaire, an CCXXV, James Almer a écrit : >>> >> A missing DisplayUnit element or one with the default value of 0 means >>> >> DisplayWidth and DisplayHeight should be interpreted as

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

On 10/16/2016 5:58 AM, Nicolas George wrote: > Le quartidi 24 vendémiaire, an CCXXV, James Almer a écrit : >> 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

[FFmpeg-devel] replacing -r for -framerate in StreamingGuide

Hello, I've just noticed how the StreamingGuide https://trac.ffmpeg.org/wiki/StreamingGuide uses -r for the framerate in its examples. I believe the correct command line argument here now is -framerate. ___ ffmpeg-devel mailing list

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

Le quartidi 24 vendémiaire, an CCXXV, James Almer a écrit : > 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.

Re: [FFmpeg-devel] FFmpeg at the Embedded Linux Conference Europe (ELCE) 2016

Hi, > FFmpeg will have a booth at the ELCE 2016 in Berlin, Germany from Oct. 11th to > Oct 13th! > > The booth will be manned with me, everyone interested is welcome to visit us > and > the rest of the ELCE. I'm looking for a fellow developer to share the booth > work > with me and have some