Re: [FFmpeg-devel] [PATCH] avcodec/allcodecs: Re-order nvenc encoders below x264/5.

2015-06-06 Thread Philip Langdale
On Sun, 7 Jun 2015 00:12:18 +0200 Michael Niedermayer wrote: > On Sat, Jun 06, 2015 at 01:22:45PM -0700, Philip Langdale wrote: > > We shouldn't pick nvenc by default. > > > > Signed-off-by: Philip Langdale > > --- > > libavcodec/allcodecs.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 d

Re: [FFmpeg-devel] [PATCH] ffserver: Check allocations (likely not all)

2015-06-06 Thread Reynaldo H. Verdejo Pinochet
Hi On 06/06/2015 10:43 AM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > ffserver.c | 20 > 1 file changed, 20 insertions(+) > > [..] Looks good. Thanks! Bests, -- Reynaldo H. Verdejo Pinochet Open Source Group Samsung Research America / Sil

Re: [FFmpeg-devel] [PATCH] vf_colormatrix: calculate coefficients only once

2015-06-06 Thread Michael Niedermayer
On Fri, Jun 05, 2015 at 04:05:18PM -0400, Ganesh Ajjanagadde wrote: > calc_coefficients is no longer being called every frame > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/vf_colormatrix.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) applied thanks [...] -- Michae

Re: [FFmpeg-devel] [PATCH] avcodec/allcodecs: Re-order nvenc encoders below x264/5.

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 01:22:45PM -0700, Philip Langdale wrote: > We shouldn't pick nvenc by default. > > Signed-off-by: Philip Langdale > --- > libavcodec/allcodecs.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B1

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Marcus Johnson
I like the John Nash idea. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Helmut K. C. Tessarek
On 06.06.15 8:19 , Michael Niedermayer wrote: > also if someone wants a specific name for it say/suggest it now, > if theres no name suggested or no consensus > then ill pick a random scientists/mathematicans name that was suggested > previously To honor the beautiful mind who made game theory a w

[FFmpeg-devel] [PATCH] avcodec/allcodecs: Re-order nvenc encoders below x264/5.

2015-06-06 Thread Philip Langdale
We shouldn't pick nvenc by default. Signed-off-by: Philip Langdale --- libavcodec/allcodecs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c index 159d842..6a91d64 100644 --- a/libavcodec/allcodecs.c +++ b/libavcodec/allc

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/nvenc: Add 'nvenc_h264' as an alternative name for 'nvenc'

2015-06-06 Thread Timo Rothenpieler
> not really related to this patch but > are the nvenc encoders intended to be used by default for > "h264/hevc" even when libx264 / libx265 is enabled ? > they are registered before them so they would be used if no encoder > is explicitly specified I was under the impression that the list is alp

Re: [FFmpeg-devel] [PATCH 0/5] Remove nvenc_b

2015-06-06 Thread Timo Rothenpieler
> We've decided to drop the libav reimplementation of nvenc support and retain > ours as is. This patch series removes the files, along with also reverting > the various fixes we made on top of them. The series LGTM for me. signature.asc Description: OpenPGP digital signature _

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Rename nvenc_h265 to nvenc_hevc

2015-06-06 Thread Timo Rothenpieler
> For the sake of compatibility, and because pretty much everything else in the > codebase calls it HEVC. Both patches LGTM for me. signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpe

Re: [FFmpeg-devel] [PATCH 0/5] Remove nvenc_b

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 10:51:22AM -0700, Philip Langdale wrote: > We've decided to drop the libav reimplementation of nvenc support and retain > ours as is. This patch series removes the files, along with also reverting > the various fixes we made on top of them. > > In subsequent changes, I'll r

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/nvenc: Add 'nvenc_h264' as an alternative name for 'nvenc'

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 11:12:13AM -0700, Philip Langdale wrote: > This allows us to offer the same codec name that libav uses. We don't have > a special way to do aliases, so it's all a bit more verbose than you'd want > but such is life. > > Signed-off-by: Philip Langdale > --- > configure

Re: [FFmpeg-devel] [PATCH 1/5] Revert "Changelog: Drop duplicate NVENC entry"

2015-06-06 Thread Philip Langdale
On Sat, 06 Jun 2015 15:10:40 -0300 James Almer wrote: > On 06/06/15 2:51 PM, Philip Langdale wrote: > > This reverts commit 440fa7758b687bbb0007f85497eed8bb9aec96bd. > > > > Signed-off-by: Philip Langdale > > --- > > Changelog | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Cha

Re: [FFmpeg-devel] [PATCH 1/5] Revert "Changelog: Drop duplicate NVENC entry"

2015-06-06 Thread James Almer
On 06/06/15 3:10 PM, James Almer wrote: > On 06/06/15 2:51 PM, Philip Langdale wrote: >> This reverts commit 440fa7758b687bbb0007f85497eed8bb9aec96bd. >> >> Signed-off-by: Philip Langdale >> --- >> Changelog | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/Changelog b/Changelog >> inde

[FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Rename nvenc_h265 to nvenc_hevc

2015-06-06 Thread Philip Langdale
For the sake of compatibility, and because pretty much everything else in the codebase calls it HEVC. Signed-off-by: Philip Langdale --- Changelog | 2 +- configure | 2 +- libavcodec/allcodecs.c | 2 +- libavcodec/nvenc.c | 18 +- 4 files changed

[FFmpeg-devel] [PATCH 2/2] avcodec/nvenc: Add 'nvenc_h264' as an alternative name for 'nvenc'

2015-06-06 Thread Philip Langdale
This allows us to offer the same codec name that libav uses. We don't have a special way to do aliases, so it's all a bit more verbose than you'd want but such is life. Signed-off-by: Philip Langdale --- configure | 1 + libavcodec/allcodecs.c | 1 + libavcodec/nvenc.c | 25 ++

Re: [FFmpeg-devel] [PATCH 5/5] Revert "avcodec: Rename nvenc.c to nvenc_a.c, to avoid conflict with the other implementation"

2015-06-06 Thread James Almer
On 06/06/15 2:51 PM, Philip Langdale wrote: > This reverts commit d0d0913702358c61dfb9c5ac8b141cd203524869. > > Signed-off-by: Philip Langdale > --- > libavcodec/Makefile |2 +- > libavcodec/nvenc.c | 1452 > ++ > libavcodec/nvenc_a.c | 145

Re: [FFmpeg-devel] [PATCH 1/5] Revert "Changelog: Drop duplicate NVENC entry"

2015-06-06 Thread James Almer
On 06/06/15 2:51 PM, Philip Langdale wrote: > This reverts commit 440fa7758b687bbb0007f85497eed8bb9aec96bd. > > Signed-off-by: Philip Langdale > --- > Changelog | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Changelog b/Changelog > index c73f1dd..193ca43 100644 > --- a/Changelog > +++

[FFmpeg-devel] [PATCH 5/5] Revert "avcodec: Rename nvenc.c to nvenc_a.c, to avoid conflict with the other implementation"

2015-06-06 Thread Philip Langdale
This reverts commit d0d0913702358c61dfb9c5ac8b141cd203524869. Signed-off-by: Philip Langdale --- libavcodec/Makefile |2 +- libavcodec/nvenc.c | 1452 ++ libavcodec/nvenc_a.c | 1452 -- 3 files

[FFmpeg-devel] [PATCH 3/5] Revert "nvenc: remove cuda.h requirement for nvenc_a"

2015-06-06 Thread Philip Langdale
This reverts commit f10497042f092d56f82f536276d5998d7fd1e4c4. Signed-off-by: Philip Langdale --- configure | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/configure b/configure index 8168edc..6fdf816 100755 --- a/configure +++ b/configure @@ -1673,7 +1673,6 @@ HEADERS_

[FFmpeg-devel] [PATCH 4/5] Revert "Merge commit 'b08caa87c35a768ec0abb16b1e99c3a85f1df28e'"

2015-06-06 Thread Philip Langdale
This reverts commit d8bbb99c51af08c76dd0a66588fecc5c0106844d, reversing changes made to d0d0913702358c61dfb9c5ac8b141cd203524869. Signed-off-by: Philip Langdale --- Changelog |1 - configure | 21 +- libavcodec/Makefile |3 - libavcodec/allcodecs.c

[FFmpeg-devel] [PATCH 1/5] Revert "Changelog: Drop duplicate NVENC entry"

2015-06-06 Thread Philip Langdale
This reverts commit 440fa7758b687bbb0007f85497eed8bb9aec96bd. Signed-off-by: Philip Langdale --- Changelog | 1 + 1 file changed, 1 insertion(+) diff --git a/Changelog b/Changelog index c73f1dd..193ca43 100644 --- a/Changelog +++ b/Changelog @@ -24,6 +24,7 @@ version : - WebPAnimEncoder API wh

[FFmpeg-devel] [PATCH 2/5] Revert "avcodec/Makefile: fix checkheaders for nvenc_b"

2015-06-06 Thread Philip Langdale
This reverts commit 3be811a528461816063059377fcb2ac60eabfa77. Signed-off-by: Philip Langdale --- libavcodec/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index b018a67..f89ed36 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@

[FFmpeg-devel] [PATCH 0/5] Remove nvenc_b

2015-06-06 Thread Philip Langdale
We've decided to drop the libav reimplementation of nvenc support and retain ours as is. This patch series removes the files, along with also reverting the various fixes we made on top of them. In subsequent changes, I'll rename our nvenc_h265 to nvenc_hevc and add an alias for nvenc_h264. Philip

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread James Almer
On 06/06/15 2:00 PM, Philip Langdale wrote: > On Sat, 6 Jun 2015 14:34:11 +0200 > Michael Niedermayer wrote: > >> On Sat, Jun 06, 2015 at 02:16:14PM +0200, Timo Rothenpieler wrote: Hi all Its time again for another release (its about 3 months since 2.6) so if theres anything y

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Philip Langdale
On Sat, 6 Jun 2015 14:34:11 +0200 Michael Niedermayer wrote: > On Sat, Jun 06, 2015 at 02:16:14PM +0200, Timo Rothenpieler wrote: > > > Hi all > > > > > > Its time again for another release (its about 3 months since 2.6) > > > so if theres anything you want in it, or you want something > > > spe

Re: [FFmpeg-devel] [PATCH] avutil/colorspace: Remove RGB_TO_Y/U/V

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 03:05:59PM +0200, Clément Bœsch wrote: > On Sat, Jun 06, 2015 at 02:57:15PM +0200, Michael Niedermayer wrote: > > They are unused and colorspace.h is not a installed header > > > > Found-by: Timo Rothenpieler > > --- > > libavutil/colorspace.h | 12 > > 1 f

Re: [FFmpeg-devel] [PATCH] configure: improve the checks for gmtime_r and localtime_r

2015-06-06 Thread James Almer
On 06/06/15 8:42 AM, Michael Niedermayer wrote: > On Sat, Jun 06, 2015 at 02:00:28AM -0300, James Almer wrote: >> They are inlined wrapper functions inside the time.h header on MinGW-w64, so >> neither check_func() or check_func_headers() work with them. >> >> Signed-off-by: James Almer >> --- >>

[FFmpeg-devel] [PATCH] ffserver: Check allocations (likely not all)

2015-06-06 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- ffserver.c | 20 1 file changed, 20 insertions(+) diff --git a/ffserver.c b/ffserver.c index 06218e0..9d0cb0b 100644 --- a/ffserver.c +++ b/ffserver.c @@ -1204,6 +1204,10 @@ static FFServerIPAddressACL* parse_dynamic_acl(FFServerStre

Re: [FFmpeg-devel] [PATCH] avutil/colorspace: Remove RGB_TO_Y/U/V

2015-06-06 Thread Clément Bœsch
On Sat, Jun 06, 2015 at 02:57:15PM +0200, Michael Niedermayer wrote: > They are unused and colorspace.h is not a installed header > > Found-by: Timo Rothenpieler > --- > libavutil/colorspace.h | 12 > 1 file changed, 12 deletions(-) > > diff --git a/libavutil/colorspace.h b/libav

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Add simple autodetection for client HTTP method, based on AVIO_FLAG_READ.

2015-06-06 Thread Nicolas George
L'octidi 18 prairial, an CCXXIII, Stephan Holljes a écrit : > No, should I? I'm still learning the ropes with git. I haven't used it > with more than 3 people at once. It may be convenient, but it absolutely not mandatory. Regards, -- Nicolas George signature.asc Description: Digital signat

[FFmpeg-devel] [PATCH] avutil/colorspace: Remove RGB_TO_Y/U/V

2015-06-06 Thread Michael Niedermayer
They are unused and colorspace.h is not a installed header Found-by: Timo Rothenpieler --- libavutil/colorspace.h | 12 1 file changed, 12 deletions(-) diff --git a/libavutil/colorspace.h b/libavutil/colorspace.h index f438159..826ffd5 100644 --- a/libavutil/colorspace.h +++ b/li

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Add simple autodetection for client HTTP method, based on AVIO_FLAG_READ.

2015-06-06 Thread Stephan Holljes
On Sat, Jun 6, 2015 at 9:47 AM, Nicolas George wrote: > Le septidi 17 prairial, an CCXXIII, Stephan Holljes a écrit : >> Signed-off-by: Stephan Holljes >> --- >> libavformat/http.c | 9 + >> 1 file changed, 9 insertions(+) > > LGTM. > > Michael, I have pushed these to my tree, can you pu

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 02:16:14PM +0200, Timo Rothenpieler wrote: > > Hi all > > > > Its time again for another release (its about 3 months since 2.6) > > so if theres anything you want in it, or you want something specific > > written in some releas notes / changelog, write it now or its too lat

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 02:10:20PM +0200, Michael Niedermayer wrote: > Hi all > > Its time again for another release (its about 3 months since 2.6) > so if theres anything you want in it, or you want something specific > written in some releas notes / changelog, write it now or its too late also

Re: [FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Timo Rothenpieler
> Hi all > > Its time again for another release (its about 3 months since 2.6) > so if theres anything you want in it, or you want something specific > written in some releas notes / changelog, write it now or its too late > > Thanks Something I just noticed, due to the merge of the libav nvenc

Re: [FFmpeg-devel] colorspace.h

2015-06-06 Thread Timo Rothenpieler
> The specific code I'm referring to is at the end of colorspace.h where it > looks like they're trying to encode RGB to BT.601. > > #define RGB_TO_Y(r1, g1, b1, shift) > #define RGB_TO_U(r1, g1, b1, shift) > #define RGB_TO_V(r1, g1, b1, shift) A quick search over the ffmpeg codebase tells me th

[FFmpeg-devel] FFmpeg 2.7

2015-06-06 Thread Michael Niedermayer
Hi all Its time again for another release (its about 3 months since 2.6) so if theres anything you want in it, or you want something specific written in some releas notes / changelog, write it now or its too late Thanks -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH] configure: improve the checks for gmtime_r and localtime_r

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 02:00:28AM -0300, James Almer wrote: > They are inlined wrapper functions inside the time.h header on MinGW-w64, so > neither check_func() or check_func_headers() work with them. > > Signed-off-by: James Almer > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+),

Re: [FFmpeg-devel] [PATCH] hevc: fix typo (mpv -> mvp).

2015-06-06 Thread Michael Niedermayer
On Fri, Jun 05, 2015 at 10:08:45PM -0400, Ronald S. Bultje wrote: > --- > libavcodec/hevc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB There will always be a question for which you

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Add simple autodetection for client HTTP method, based on AVIO_FLAG_READ.

2015-06-06 Thread Michael Niedermayer
On Sat, Jun 06, 2015 at 09:47:36AM +0200, Nicolas George wrote: > Le septidi 17 prairial, an CCXXIII, Stephan Holljes a écrit : > > Signed-off-by: Stephan Holljes > > --- > > libavformat/http.c | 9 + > > 1 file changed, 9 insertions(+) > > LGTM. > > Michael, I have pushed these to my t

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Add simple autodetection for client HTTP method, based on AVIO_FLAG_READ.

2015-06-06 Thread Nicolas George
Le septidi 17 prairial, an CCXXIII, Stephan Holljes a écrit : > Signed-off-by: Stephan Holljes > --- > libavformat/http.c | 9 + > 1 file changed, 9 insertions(+) LGTM. Michael, I have pushed these to my tree, can you pull please: aa74401 lavf/http: Document method option. bbcee92 lavf