Re: [FFmpeg-devel] [PATCH 1/2] refine the doc of hlsenc

2016-07-17 Thread Steven Liu
2016-07-13 4:24 GMT+08:00 Michael Niedermayer : > On Tue, Jul 12, 2016 at 09:35:46PM +0800, Steven Liu wrote: > > resend the patch, delete the trailing whitespace and check patch by > > ./tools/patcheck > > > > add the hls_flags round_durations, discont_start, omit_endlist flags > > describe > > >

[FFmpeg-devel] [PATCH] avformat/matroskaenc: Write duration early during mkv_write_header

2016-07-17 Thread Soft Works
This commit addresses the following scenario: we are using ffmpeg to transcode or remux mkv (or something else) to mkv. The result is being streamed on-the-fly to an HTML5 client (streaming starts while ffmpeg is still running). The problem here is that the client is unable to detect the durati

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/ccaption_dec: default rollup to row 10

2016-07-17 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 11:57:45AM -0700, Aman Gupta wrote: > From: Aman Gupta > > This ensures that captions are written towards the bottom of the screen > when tuning into mid-stream. The row will be reset on the receipt of the > next PAC command. Row 10 was chosen as it corresponds to the valu

Re: [FFmpeg-devel] [PATCH 3/5] avcodec/ccaption_dec: implement positioning for closed captions

2016-07-17 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 11:57:43AM -0700, Aman Gupta wrote: > From: Aman Gupta > > Positioning math is based on the guidelines in > https://dvcs.w3.org/hg/text-tracks/raw-file/default/608toVTT/608toVTT.html#positioning-in-cea-608 > --- > libavcodec/ccaption_dec.c | 29 ++

[FFmpeg-devel] [PATCH] configure: add support for new CPUs

2016-07-17 Thread James Almer
Add new -march values for Intel and AMD CPUs introduced with GCC 5 and 6, and improve SunCC flags accordingly. Signed-off-by: James Almer --- configure | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 4956afc..cc51e67 100755 --- a/configure +

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/ccaption_dec: change write_char() to void as return value is unused

2016-07-17 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 11:57:41AM -0700, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/ccaption_dec.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay di

[FFmpeg-devel] Patch submission

2016-07-17 Thread Soft Works
Hi, I just prepared a patch and I've also written a quite comprehensive explanation (about 60 lines of text). Should I put all the text into the git commit or should I use a shortened version for the commit and post the long version to this list only? Thanks, softworkz ___

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add Aman Gupta for ccaption_dec

2016-07-17 Thread Michael Niedermayer
On Sat, Jul 16, 2016 at 02:58:20PM +0200, Michael Niedermayer wrote: > Anshul appears to have no time currently as he has not reacted to patches or > mails > and Aman agreed to take over maintaince or help > > Signed-off-by: Michael Niedermayer > --- > MAINTAINERS |2 +- > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH 0/4] libavcodec/dnxhdenc: add support for dnxhr encoding

2016-07-17 Thread Carl Eugen Hoyos
Mark Reid gmail.com> writes: > The following patch series adds support for dnxhr encoding. > I added dnxhr as a profile to the dnxhd encoder. Is there also a raw format that our demuxer currently does not auto-detect (and / or not demux)? Carl Eugen ___

[FFmpeg-devel] [PATCH] af_hdcd: Improve HDCD detection

2016-07-17 Thread Burt P
HDCD is only "detected" if a valid code is active in both channels simultaneously, as described here: https://hydrogenaud.io/index.php/topic,79427.msg900371.html#msg900371 Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 34 -- 1 file changed, 24 insertions(+), 1

Re: [FFmpeg-devel] [PATCH 3/3] libavformat/libopenmpt: Add "date" to metadata.

2016-07-17 Thread Josh de Kock
On Sun, Jul 17, 2016, at 02:37 PM, Jörn Heusipp wrote: > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c > index 7b1dd5b..997c13d 100644 > --- a/libavformat/libopenmpt.c > ++

Re: [FFmpeg-devel] [PATCH 2/2] doc/examples/muxing: Fix packet duration

2016-07-17 Thread Gonzalo
El 16/07/16 a las 23:10, Michael Niedermayer escribió: Set needed fields in st->codec Fixes Ticket5684 Signed-off-by: Michael Niedermayer --- @@ -434,6 +434,8 @@ static void open_video(AVFormatContext *oc, AVCodec *codec, OutputStream *ost, A fprintf(stderr, "Could not copy the st

[FFmpeg-devel] ffprobe missing string in code

2016-07-17 Thread Dmitry Vagin
Hello, FFmpeg-devel! ffprobe not show tags when set only '-show_entries programs' Patch in attachment. /Dmitry --- ffprobe.c.orig 2016-04-29 17:08:17.0 +0500 +++ ffprobe.c 2016-07-15 10:30:37.701155000 +0500 @@ -3178,6 +3178,7 @@ SET_DO_SHOW(FRAME_TAGS, frame_tags); SET_DO_SHOW

Re: [FFmpeg-devel] CII Best Practices badge for FFMPEG

2016-07-17 Thread Wheeler, David A
>>> “The project SHOULD implement perfect forward secrecy for key agreement >>> protocols so a session key derived from a set of long-term keys cannot be >>> compromised if one of the long-term keys is compromised in the future. >>> [crypto_pfs]   >> This might be okay to state as “unmet” or “N/

[FFmpeg-devel] [PATCH] MAINTAINERS: add myself for libopenmpt

2016-07-17 Thread Josh de Kock
--- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index e5b466c..cb3675a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -402,6 +402,7 @@ Muxers/Demuxers: jvdec.c Peter Ross libmodplug.c Clément

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/libopenmpt: Fix memory leak in error path in read_header_openmpt().

2016-07-17 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 03:37:12PM +0200, Jörn Heusipp wrote: > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) applied btw, there is noone listed in MAINTAINERs for libavformat/libopenmpt.c can one of the authors/contrib

Re: [FFmpeg-devel] [PATCH] lavfi/curves: do not automatically insert points at x=0 and x=1

2016-07-17 Thread Clément Bœsch
On Sun, Jul 17, 2016 at 05:03:35PM +0200, Clément Bœsch wrote: > There is actually a need for the origin and end point not to be defined. > We can not automatically insert them with the y value of the first and > last point as it will influence the curves in a wrong way. > > Fixes #5397 > --- > C

[FFmpeg-devel] [PATCH] lavfi/curves: do not automatically insert points at x=0 and x=1

2016-07-17 Thread Clément Bœsch
There is actually a need for the origin and end point not to be defined. We can not automatically insert them with the y value of the first and last point as it will influence the curves in a wrong way. Fixes #5397 --- Changelog | 1 + doc/filters.texi| 10 ++ libavfilt

Re: [FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-07-17 Thread Thilo Borgmann
Am 17.07.16 um 16:22 schrieb Michael Niedermayer: > On Sun, Jul 17, 2016 at 07:25:31PM +0530, Umair Khan wrote: >> On Sun, Jul 17, 2016 at 1:50 PM, Thilo Borgmann >> wrote: >>> Hi, >>> From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 From: Umair Khan Date: Su

Re: [FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-07-17 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 07:25:31PM +0530, Umair Khan wrote: > On Sun, Jul 17, 2016 at 1:50 PM, Thilo Borgmann > wrote: > > Hi, > > > >> From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 > >> From: Umair Khan > >> Date: Sun, 17 Jul 2016 13:05:49 +0530 > >> Subject: [PATCH 1/1

Re: [FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-07-17 Thread Umair Khan
On Sun, Jul 17, 2016 at 1:50 PM, Thilo Borgmann wrote: > Hi, > >> From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 >> From: Umair Khan >> Date: Sun, 17 Jul 2016 13:05:49 +0530 >> Subject: [PATCH 1/1] fate/als: add floating point decoding test >> >> Signed-off-by: Umair Khan

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/libopenmpt: Fix memory leak in error path in read_header_openmpt().

2016-07-17 Thread Nicolas George
Le decadi 30 messidor, an CCXXIV, Jörn Heusipp a écrit : > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) I am not objecting to the patch, but I think it shows demuxers would benefit from a deinit() method just like Rodge

[FFmpeg-devel] [PATCH 2/3] libavformat/libopenmpt: Actually set stream duration and fix the confusion around the used time base.

2016-07-17 Thread Jörn Heusipp
Signed-off-by: Jörn Heusipp --- libavformat/libopenmpt.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 58a02be..7b1dd5b 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -94,9 +94,8 @@ stati

[FFmpeg-devel] [PATCH 1/3] libavformat/libopenmpt: Fix memory leak in error path in read_header_openmpt().

2016-07-17 Thread Jörn Heusipp
Signed-off-by: Jörn Heusipp --- libavformat/libopenmpt.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index fd38828..58a02be 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -89,8 +89,11 @@ stat

[FFmpeg-devel] [PATCH 3/3] libavformat/libopenmpt: Add "date" to metadata.

2016-07-17 Thread Jörn Heusipp
Signed-off-by: Jörn Heusipp --- libavformat/libopenmpt.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 7b1dd5b..997c13d 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -87,6 +87,7 @@ static int read_header_ope

Re: [FFmpeg-devel] [PATCH 1/4] libavformat/movenc: add dnxhr compatibility for apple players

2016-07-17 Thread Michael Niedermayer
On Sat, Jul 16, 2016 at 07:37:36PM -0700, Mark Reid wrote: > --- > libavcodec/dnxhddec.c | 3 +++ > libavformat/movenc.c | 19 ++- > 2 files changed, 17 insertions(+), 5 deletions(-) this should be split a single patch changing both libs is a bit suspicous API wise > > diff -

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: implement floating point decoding

2016-07-17 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 11:54:49AM +0200, Michael Niedermayer wrote: > On Sun, Jul 17, 2016 at 09:00:48AM +0530, Umair Khan wrote: > > Hi, > > > > On Sun, Jul 17, 2016 at 12:25 AM, Thilo Borgmann > > wrote: > > > Hi, > > > > > >> From 70e65b26cc3f84c9c664c30808b43a5e1cf16eaa Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH 1/3] configure: require libvpx-1.4.0 for vp9 support

2016-07-17 Thread Nicolas George
L'octidi 28 messidor, an CCXXIV, James Zern a écrit : > this will simplify libvpxenc/dec.c and ensure a more stable vp9 encoder > is present. > > Signed-off-by: James Zern > --- > configure | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) Does that mean no VP9 with Git head on De

Re: [FFmpeg-devel] [PATCH v2 3/7] avformat/tee: Rescale ts using av_packet_rescale_ts

2016-07-17 Thread Nicolas George
Le nonidi 29 messidor, an CCXXIV, sebechlebsky...@gmail.com a écrit : > From: Jan Sebechlebsky > > This ensures that AV_NOPTS_VALUE value is handled > correctly. > > Signed-off-by: Jan Sebechlebsky > --- > libavformat/tee.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) LGTM, good

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: implement floating point decoding

2016-07-17 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 09:00:48AM +0530, Umair Khan wrote: > Hi, > > On Sun, Jul 17, 2016 at 12:25 AM, Thilo Borgmann > wrote: > > Hi, > > > >> From 70e65b26cc3f84c9c664c30808b43a5e1cf16eaa Mon Sep 17 00:00:00 2001 > >> From: Umair Khan > >> Date: Sat, 16 Jul 2016 23:52:39 +0530 > >> Subject:

Re: [FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-07-17 Thread Thilo Borgmann
Hi, > From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 > From: Umair Khan > Date: Sun, 17 Jul 2016 13:05:49 +0530 > Subject: [PATCH 1/1] fate/als: add floating point decoding test > > Signed-off-by: Umair Khan > --- > tests/fate/als.mak | 3 +++ > 1 file changed, 3 insert

[FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-07-17 Thread Umair Khan
Hi, Patch attached. - Umair From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 From: Umair Khan Date: Sun, 17 Jul 2016 13:05:49 +0530 Subject: [PATCH 1/1] fate/als: add floating point decoding test Signed-off-by: Umair Khan --- tests/fate/als.mak | 3 +++ 1 file changed, 3