Re: [FFmpeg-devel] [PATCH] matroskadec: partly revert "demux relevant subtitle packets after a seek" c16582579b1c6f66a86615c5808cd5b2bf17be73

2016-11-30 Thread Rainer Hochecker
We already included this patch in Kodi v17. Dropping this incomplete feature is much better than living with the downsides. Having files on a network share is a common use case and in most cases there is no index anyway: start video and seek forward. ___

[FFmpeg-devel] [PATCH] examples: fix a typo

2016-11-30 Thread Александр Слободенюк
--- doc/examples/transcode_aac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/examples/transcode_aac.c b/doc/examples/transcode_aac.c index 9b3ee67..d7947b4 100644 --- a/doc/examples/transcode_aac.c +++ b/doc/examples/transcode_aac.c @@ -566,7 +566,7 @@ static int

Re: [FFmpeg-devel] [PATCH 4/4] fate: Add a monoscopic spherical mov test

2016-11-30 Thread Michael Niedermayer
On Wed, Nov 30, 2016 at 06:36:42PM -0500, Vittorio Giovara wrote: > --- > This needs https://www.dropbox.com/s/0lqxkzy16o7fihz/spherical.mov?dl=0 > to be placed in $SAMPLES/mov. uploaded thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB There will always be

Re: [FFmpeg-devel] [DECISION] Revoke the decision of dropping ffserver

2016-11-30 Thread Steven Liu
2016-11-30 23:29 GMT+08:00 Nicolas George : > Le nonidi 9 frimaire, an CCXXV, James Almer a écrit : > > Seeing Nicolas is apparently very invested in ffserver, can we expect > him to > > maintain it, improve and extend it if it were to remain in the tree? Or > is he > > just

Re: [FFmpeg-devel] [PATCH 3/3] configure: fail if autodetect-libraries are requested but not found

2016-11-30 Thread Michael Niedermayer
On Thu, Dec 01, 2016 at 12:22:50AM +0100, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > configure | 72 > ++- > 1 file changed, 53 insertions(+), 19 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH 3/4] mov: Export spherical information

2016-11-30 Thread James Almer
On 11/30/2016 8:36 PM, Vittorio Giovara wrote: > This implements Spherical Video V1 and V2, as described in the > spatial-media collection by Google. > > Signed-off-by: Vittorio Giovara > --- > This addresses all comments from James. > Vittorio LGTM, at least the V2

[FFmpeg-devel] [PATCH 2/4] lavc: Add spherical packet side data API

2016-11-30 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- No particular change. Vittorio ffprobe.c | 13 + libavcodec/avcodec.h | 8 +++- libavcodec/avpacket.c | 1 + libavcodec/utils.c| 1 + libavformat/dump.c| 30 ++ 5

[FFmpeg-devel] [PATCH] fate/mov: Rename a couple of entries to respect the file naming scheme

2016-11-30 Thread Vittorio Giovara
--- tests/fate/mov.mak | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/fate/mov.mak b/tests/fate/mov.mak index 71c9a1f..f642f1d 100644 --- a/tests/fate/mov.mak +++ b/tests/fate/mov.mak @@ -7,7 +7,7 @@ FATE_MOV = fate-mov-3elist \

[FFmpeg-devel] [PATCH 4/4] fate: Add a monoscopic spherical mov test

2016-11-30 Thread Vittorio Giovara
--- This needs https://www.dropbox.com/s/0lqxkzy16o7fihz/spherical.mov?dl=0 to be placed in $SAMPLES/mov. Vittorio tests/fate/mov.mak| 4 tests/ref/fate/mov-spherical-mono | 16 2 files changed, 20 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH 3/4] mov: Export spherical information

2016-11-30 Thread Vittorio Giovara
This implements Spherical Video V1 and V2, as described in the spatial-media collection by Google. Signed-off-by: Vittorio Giovara --- This addresses all comments from James. Vittorio libavformat/isom.h | 7 ++ libavformat/mov.c | 239

[FFmpeg-devel] [PATCH 1/4 v4] lavu: Add AVSphericalMapping type and frame side data

2016-11-30 Thread Vittorio Giovara
While no decoder currently exports spherical information, this type represents a frame property that has to be passed through from container to frames. Signed-off-by: Vittorio Giovara --- Revised documentation of the new type, removed bounding rectangle values, as I

Re: [FFmpeg-devel] [PATCH] lavfi/avfiltergraph: always reduce all_layouts to a single layout

2016-11-30 Thread Michael Niedermayer
On Wed, Nov 30, 2016 at 08:29:52PM +0100, Marton Balint wrote: > > On Wed, 30 Nov 2016, Nicolas George wrote: > > >Le nonidi 9 frimaire, an CCXXV, Marton Balint a écrit : > >>There can be filters where .query_format is defined and they still can refer > >>to all_formats and not all_counts, so i

Re: [FFmpeg-devel] [PATCH] configure: fail if sdl2 was requested but not found

2016-11-30 Thread Andreas Cadhalpun
On 23.11.2016 00:23, Josh de Kock wrote: > On 22/11/2016 23:11, Andreas Cadhalpun wrote: >> On 22.11.2016 23:43, Josh de Kock wrote: >>> On 22/11/2016 22:33, Andreas Cadhalpun wrote: Because it avoids confusing users. And the checks for other system libraries should do this, too.

[FFmpeg-devel] [PATCH 3/3] configure: fail if autodetect-libraries are requested but not found

2016-11-30 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- configure | 72 ++- 1 file changed, 53 insertions(+), 19 deletions(-) diff --git a/configure b/configure index 28c78ef..0e9695b 100755 --- a/configure +++

[FFmpeg-devel] [PATCH 2/3] configure: enable autodetected libraries after option parsing

2016-11-30 Thread Andreas Cadhalpun
This is needed for the following patch. Signed-off-by: Andreas Cadhalpun --- configure | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/configure b/configure index 2ce7555..28c78ef 100755 --- a/configure +++ b/configure @@

[FFmpeg-devel] [PATCH 1/3] configure: enable sdl if sdl2 is enabled

2016-11-30 Thread Andreas Cadhalpun
sdl is supposed to be an alias, so it should have the same value. Signed-off-by: Andreas Cadhalpun --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 6345fc2..2ce7555 100755 --- a/configure +++

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Ronald S. Bultje
Hi, On Wed, Nov 30, 2016 at 5:26 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 30.11.2016 22:55, Ronald S. Bultje wrote: > > On Wed, Nov 30, 2016 at 4:51 PM, Andreas Cadhalpun < > > andreas.cadhal...@googlemail.com> wrote: > > > >> On 30.11.2016 19:16, Vittorio Giovara

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread James Almer
On 11/30/2016 7:26 PM, Andreas Cadhalpun wrote: > On 30.11.2016 22:55, Ronald S. Bultje wrote: >> On Wed, Nov 30, 2016 at 4:51 PM, Andreas Cadhalpun < >> andreas.cadhal...@googlemail.com> wrote: >> >>> On 30.11.2016 19:16, Vittorio Giovara wrote: >>> You can't just add a gap like that. >>> The

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Andreas Cadhalpun
On 30.11.2016 22:55, Ronald S. Bultje wrote: > On Wed, Nov 30, 2016 at 4:51 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: > >> On 30.11.2016 19:16, Vittorio Giovara wrote: >> You can't just add a gap like that. >> The current code assumes that the numbers are consecutive,

Re: [FFmpeg-devel] [PATCH] Define ETIMEDOUT in fifo_muxer.c for MinGW/Windows fate build.

2016-11-30 Thread James Almer
On 11/30/2016 7:02 PM, Gregory J. Wolfe wrote: > Fate failed to build in the MinGW/Windows environment because > macro ETIMEDOUT was undefined. When this condition is detected, > the code now defines this symbol the same way it's done in > libavformat/network.h. > > Signed-off-by: Gregory J.

[FFmpeg-devel] [PATCH] Define ETIMEDOUT in fifo_muxer.c for MinGW/Windows fate build.

2016-11-30 Thread Gregory J. Wolfe
Fate failed to build in the MinGW/Windows environment because macro ETIMEDOUT was undefined. When this condition is detected, the code now defines this symbol the same way it's done in libavformat/network.h. Signed-off-by: Gregory J. Wolfe ---

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Andreas Cadhalpun
On 30.11.2016 19:16, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > Please CC. > Vittorio > > libavutil/pixdesc.c | 1 + > libavutil/pixfmt.h | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c >

[FFmpeg-devel] [PATCH] swscale: add gbr(a)p16 output support

2016-11-30 Thread Paul B Mahol
Hi, patch attached, needs fate update. From 1a734f3da2f5da690f5261fb41bd60012790a1a3 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Wed, 30 Nov 2016 22:45:08 +0100 Subject: [PATCH] swscale: add gbr(a)p16 output support Signed-off-by: Paul B Mahol ---

Re: [FFmpeg-devel] [PATCH 1/3] tests: print errors from ffserver

2016-11-30 Thread Andreas Cadhalpun
On 30.11.2016 08:22, Reynaldo H. Verdejo Pinochet wrote: > The entire set looks good to go. Please feel free to push. Pushed. > Thank you! You're welcome. ;) Best regards, Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread James Almer
On 11/30/2016 5:16 PM, Nicolas George wrote: > Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : >> I am sorry, I don't have any public link I can share. > > Ok. Thanks for James for his gratis-but-sell-your-soul link. > >>> But I am pretty sure this document does not specify the

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/h264: sse2 and avx 4:2:2 idct add8 10-bit functions

2016-11-30 Thread James Darnley
On 2016-11-30 13:57, Ronald S. Bultje wrote: > On Wed, Nov 30, 2016 at 7:10 AM, James Darnley wrote: >>> Nehalem: >>> - sse2: >>>- complex: 4.13x faster (1514 vs. 367 cycles) >>>- simple: 4.38x faster (1836 vs. 419 cycles) >>> >>> Haswell: >>> -

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Nicolas George
Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : > I am sorry, I don't have any public link I can share. Ok. Thanks for James for his gratis-but-sell-your-soul link. > > But I am pretty sure this document does not specify the values of enums > > in FFmpeg's API. If the rest of the

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread James Almer
On 11/30/2016 4:51 PM, Nicolas George wrote: > Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : >> This is the value specified in the 23001-8_2013 document. > > This looks paywalled. Please give links to public versions of specs in > that kind of case.

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Vittorio Giovara
On Wed, Nov 30, 2016 at 2:51 PM, Nicolas George wrote: > Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : >> This is the value specified in the 23001-8_2013 document. > > This looks paywalled. Please give links to public versions of specs in > that kind of case. I am

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Nicolas George
Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : > This is the value specified in the 23001-8_2013 document. This looks paywalled. Please give links to public versions of specs in that kind of case. But I am pretty sure this document does not specify the values of enums in FFmpeg's

Re: [FFmpeg-devel] [PATCH] lavfi/avfiltergraph: always reduce all_layouts to a single layout

2016-11-30 Thread Marton Balint
On Wed, 30 Nov 2016, Nicolas George wrote: Le nonidi 9 frimaire, an CCXXV, Marton Balint a écrit : There can be filters where .query_format is defined and they still can refer to all_formats and not all_counts, so i am not sure we can remove the all_formats/all_counts logic so easily. There

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Vittorio Giovara
On Wed, Nov 30, 2016 at 1:21 PM, Nicolas George wrote: > Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : >> Signed-off-by: Vittorio Giovara >> --- > >> Please CC. > > You can achieve the same result more reliably by setting the reply-to >

[FFmpeg-devel] [PATCH 2/2] vf_colorspace: Add support for jedec p22 primaries

2016-11-30 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- (no I don't have samples, but the numbers should be correct) Please CC. Vittorio doc/filters.texi| 3 +++ libavfilter/vf_colorspace.c | 2 ++ 2 files changed, 5 insertions(+) diff --git a/doc/filters.texi

Re: [FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Nicolas George
Le decadi 10 frimaire, an CCXXV, Vittorio Giovara a écrit : > Signed-off-by: Vittorio Giovara > --- > Please CC. You can achieve the same result more reliably by setting the reply-to header. > Vittorio > > libavutil/pixdesc.c | 1 + > libavutil/pixfmt.h | 1 + >

[FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

2016-11-30 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- Please CC. Vittorio libavutil/pixdesc.c | 1 + libavutil/pixfmt.h | 1 + 2 files changed, 2 insertions(+) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index 3b9c45d..04eab0b 100644 --- a/libavutil/pixdesc.c +++

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_h26[45]: add crop info support in vaapi_h26[4, 5]

2016-11-30 Thread Michael Niedermayer
On Wed, Nov 30, 2016 at 04:56:01PM +0100, wm4 wrote: > On Wed, 30 Nov 2016 16:40:53 +0100 > Michael Niedermayer wrote: > > > On Wed, Nov 30, 2016 at 02:18:36PM +0100, wm4 wrote: > > > On Wed, 30 Nov 2016 11:08:10 + > > > Mark Thompson wrote: > > >

Re: [FFmpeg-devel] [PATCH] Allow client to enable/disable openh264 load balancing.

2016-11-30 Thread Gregory J Wolfe
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On > Behalf Of Moritz Barsnick > Sent: Tuesday, November 29, 2016 7:15 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] Allow client to

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_h26[45]: add crop info support in vaapi_h26[4, 5]

2016-11-30 Thread Michael Niedermayer
On Wed, Nov 30, 2016 at 02:18:36PM +0100, wm4 wrote: > On Wed, 30 Nov 2016 11:08:10 + > Mark Thompson wrote: > > > On 30/11/16 02:20, Jun Zhao wrote: > > > From 20bedd18213420c77d5e8a26fbe741d8d204ac10 Mon Sep 17 00:00:00 2001 > > > From: Jun Zhao > > >

Re: [FFmpeg-devel] [DECISION] Revoke the decision of dropping ffserver

2016-11-30 Thread Nicolas George
Le nonidi 9 frimaire, an CCXXV, James Almer a écrit : > Seeing Nicolas is apparently very invested in ffserver, can we expect him to > maintain it, improve and extend it if it were to remain in the tree? Or is he > just fighting this fight to not remove code for the sake of not removing code, >

Re: [FFmpeg-devel] [PATCH] lavfi/avfiltergraph: always reduce all_layouts to a single layout

2016-11-30 Thread Nicolas George
Le nonidi 9 frimaire, an CCXXV, Marton Balint a écrit : > There can be filters where .query_format is defined and they still can refer > to all_formats and not all_counts, so i am not sure we can remove the > all_formats/all_counts logic so easily. There is no difference between these filters and

Re: [FFmpeg-devel] [PATCH v2] ffmpeg: fix uninitialized return value

2016-11-30 Thread Tobias Rapp
On 22.11.2016 15:34, Tobias Rapp wrote: On 22.11.2016 15:06, Michael Niedermayer wrote: On Tue, Nov 22, 2016 at 02:43:57PM +0100, Tobias Rapp wrote: On 22.11.2016 14:34, Michael Niedermayer wrote: On Tue, Nov 22, 2016 at 09:16:26AM +0100, Tobias Rapp wrote: Signed-off-by: Tobias Rapp

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_h26[45]: add crop info support in vaapi_h26[4, 5]

2016-11-30 Thread wm4
On Wed, 30 Nov 2016 11:08:10 + Mark Thompson wrote: > On 30/11/16 02:20, Jun Zhao wrote: > > From 20bedd18213420c77d5e8a26fbe741d8d204ac10 Mon Sep 17 00:00:00 2001 > > From: Jun Zhao > > Date: Tue, 29 Nov 2016 14:14:25 +0800 > > Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/h264: sse2 and avx 4:2:2 idct add8 10-bit functions

2016-11-30 Thread Ronald S. Bultje
Hi, On Wed, Nov 30, 2016 at 7:10 AM, James Darnley wrote: > On 2016-11-29 21:09, Carl Eugen Hoyos wrote: > > 2016-11-29 17:14 GMT+01:00 James Darnley : > >> On 2016-11-29 15:30, Carl Eugen Hoyos wrote: > >>> 2016-11-29 12:52 GMT+01:00 James Darnley

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/h264: sse2 and avx 4:2:2 idct add8 10-bit functions

2016-11-30 Thread James Darnley
On 2016-11-29 21:09, Carl Eugen Hoyos wrote: > 2016-11-29 17:14 GMT+01:00 James Darnley : >> On 2016-11-29 15:30, Carl Eugen Hoyos wrote: >>> 2016-11-29 12:52 GMT+01:00 James Darnley : sse2: complex: 4.13x faster (1514 vs. 367 cycles) simple: 4.38x

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_h26[45]: add crop info support in vaapi_h26[4, 5]

2016-11-30 Thread Mark Thompson
On 30/11/16 02:20, Jun Zhao wrote: > From 20bedd18213420c77d5e8a26fbe741d8d204ac10 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Tue, 29 Nov 2016 14:14:25 +0800 > Subject: [PATCH] lavc/vaapi_h26[45]: add crop info support in vaapi_h26[4,5] > > add crop information support

Re: [FFmpeg-devel] [PATCH] matroskadec: partly revert "demux relevant subtitle packets after a seek" c16582579b1c6f66a86615c5808cd5b2bf17be73

2016-11-30 Thread wm4
On Sat, 26 Nov 2016 08:27:44 +0100 Rainer Hochecker wrote: > From: Rainer Hochecker > > > Alternative patch. Revert the original code because it does more harm than > any good. > > > Signed-off-by: Rainer Hochecker > --- >

Re: [FFmpeg-devel] [PATCH] ffserver: Add client requested urls to the status page

2016-11-30 Thread Michael Niedermayer
On Tue, Nov 29, 2016 at 11:58:06PM -0800, Reynaldo H. Verdejo Pinochet wrote: > Looks good to go. Thank you! applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Old school: Use the lowest level language in which you can solve the problem

Re: [FFmpeg-devel] [PATCH] doc/filters: drawtext: add example of printing texts on same baseline

2016-11-30 Thread Andrey Utkin
On Tue, Nov 29, 2016 at 10:19:04AM -0900, l...@lrcd.com wrote: > On Tue, Nov 29, 2016, at 09:55 AM, Andrey Utkin wrote: > > > > The point is that with more naive approach to printing parts of text, > > using just same vertical offset (code given below), it will look fine in > > some cases (e.g.

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_h26[45]: add crop info support in vaapi_h26[4, 5]

2016-11-30 Thread wm4
On Wed, 30 Nov 2016 10:20:54 +0800 Jun Zhao wrote: > From 20bedd18213420c77d5e8a26fbe741d8d204ac10 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Tue, 29 Nov 2016 14:14:25 +0800 > Subject: [PATCH] lavc/vaapi_h26[45]: add crop info support in