Re: [FFmpeg-devel] Patch for CUDA Scale Filter

2017-04-17 Thread Yogender Gupta
>> Please find attached a CUDA based scale filter. This filter will be a >> starting point to add other CUDA based filters. The filter supports 420-8, >> 420-10, 444-8, 444-10 formats and will help to do complete 10-bit HW >> accelerated transcoding when using >> CUVID. >> Currently the filter r

Re: [FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-17 Thread Clément Bœsch
On Sun, Apr 16, 2017 at 05:20:02PM -0700, Aaron Levinson wrote: > From 9e6a9e2b8d58f17c661a3f455e03c95587ec7b18 Mon Sep 17 00:00:00 2001 > From: Aaron Levinson > Date: Sun, 16 Apr 2017 17:13:31 -0700 > Subject: [PATCH] libavutil/thread.h: Fixed g++ build error when > ASSERT_LEVEL is greater than

Re: [FFmpeg-devel] Patch for CUDA Scale Filter

2017-04-17 Thread Timo Rothenpieler
Am 17.04.2017 um 08:59 schrieb Yogender Gupta: Please find attached a CUDA based scale filter. This filter will be a starting point to add other CUDA based filters. The filter supports 420-8, 420-10, 444-8, 444-10 formats and will help to do complete 10-bit HW accelerated transcoding when using

Re: [FFmpeg-devel] Patch for CUDA Scale Filter

2017-04-17 Thread Yogender Gupta
>> Having to run a .bat file prior to building isn't really nice, and not >> something I and some other people on IRC would like to see in ffmpeg. >> Instead, it would be nice if configure/make would learn how to handle .cu >> files, converting them straight to an object-file. >> My idea for thi

Re: [FFmpeg-devel] Patch for CUDA Scale Filter

2017-04-17 Thread Timo Rothenpieler
Having to run a .bat file prior to building isn't really nice, and not something I and some other people on IRC would like to see in ffmpeg. Instead, it would be nice if configure/make would learn how to handle .cu files, converting them straight to an object-file. My idea for this would be t

Re: [FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-17 Thread James Almer
On 4/17/2017 5:39 AM, Clément Bœsch wrote: > On Sun, Apr 16, 2017 at 05:20:02PM -0700, Aaron Levinson wrote: >> From 9e6a9e2b8d58f17c661a3f455e03c95587ec7b18 Mon Sep 17 00:00:00 2001 >> From: Aaron Levinson >> Date: Sun, 16 Apr 2017 17:13:31 -0700 >> Subject: [PATCH] libavutil/thread.h: Fixed g++

Re: [FFmpeg-devel] Patch for CUDA Scale Filter

2017-04-17 Thread wm4
On Mon, 17 Apr 2017 12:22:00 +0200 Timo Rothenpieler wrote: > Am 17.04.2017 um 08:59 schrieb Yogender Gupta: > >>> Please find attached a CUDA based scale filter. This filter will be a > >>> starting point to add other CUDA based filters. The filter supports > >>> 420-8, 420-10, 444-8, 444-10 f

Re: [FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-17 Thread wm4
On Mon, 17 Apr 2017 12:06:59 -0300 James Almer wrote: > On 4/17/2017 5:39 AM, Clément Bœsch wrote: > > On Sun, Apr 16, 2017 at 05:20:02PM -0700, Aaron Levinson wrote: > >> From 9e6a9e2b8d58f17c661a3f455e03c95587ec7b18 Mon Sep 17 00:00:00 2001 > >> From: Aaron Levinson > >> Date: Sun, 16 Apr 20

Re: [FFmpeg-devel] [PATCH] webm_dash_manifest: Add option to specify bandwidth

2017-04-17 Thread James Zern
On Thu, Apr 13, 2017 at 6:20 PM, James Zern wrote: > On Tue, Apr 11, 2017 at 9:33 PM, Vignesh Venkatasubramanian > wrote: >> Add an option to webm_dash_manifest demuxer to specify a value for >> "bandwidth" field in the DASH manifest. The value is then used by >> the muxer. Fixes an existing FIXM

Re: [FFmpeg-devel] Patch for CUDA Scale Filter

2017-04-17 Thread Timo Rothenpieler
I'm generally in favor of adding CUDA based filtering, there's a lot of nice stuff that could be done that way. But there is one big issue with this approach: Having to run a .bat file prior to building isn't really nice, and not something I and some other people on IRC would like to see in ffmp

Re: [FFmpeg-devel] [PATCH] avcodec/aac_adtstoasc: fix ASC passthrough on small frames

2017-04-17 Thread Michael Niedermayer
On Wed, Apr 12, 2017 at 09:52:41PM -0300, James Almer wrote: > ASC frames smaller than AAC_ADTS_HEADER_SIZE were being discarded. > > Signed-off-by: James Almer > --- > libavcodec/aac_adtstoasc_bsf.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) LGTM thx [...] -- Michael

Re: [FFmpeg-devel] [PATCH] doc/encoders.texi: document libvpxenc's -row-mt

2017-04-17 Thread Michael Niedermayer
On Thu, Apr 13, 2017 at 06:53:26PM -0700, James Zern wrote: > added in: > 734d760e2f lavc/libvpxenc: add -row-mt option > > Signed-off-by: James Zern > --- > doc/encoders.texi | 2 ++ > 1 file changed, 2 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF1336

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-17 Thread Michael Niedermayer
On Sat, Apr 15, 2017 at 08:33:07PM +0200, Marton Balint wrote: > Fixes Coverity CID 1404841. > > Signed-off-by: Marton Balint > --- > ffmpeg.c | 14 +++--- > 1 file changed, 11 insertions(+), 3 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index e4b94b2..dd179b0 100644 > --- a/ffm

Re: [FFmpeg-devel] Fwd: [PATCH] added expr evaluation to drawtext - fontsize

2017-04-17 Thread Michael Niedermayer
On Sun, Apr 16, 2017 at 10:01:01PM -0700, Brett Harrison wrote: > Any comments on this patch? > > -- Forwarded message -- > From: Brett Harrison > Date: Tue, Apr 11, 2017 at 1:37 PM > Subject: Re: [FFmpeg-devel] [PATCH] added expr evaluation to drawtext - > fontsize > To: FFmpeg d

Re: [FFmpeg-devel] [PATCH] doc/fftools-common-opts: Fixed a typo in the common arguments list

2017-04-17 Thread Michael Niedermayer
On Sun, Apr 16, 2017 at 04:29:48PM +0100, Mickael Maison wrote: > - Fixed a typo for the -sources argument > > Signed-off-by: Mickael Maison > --- > doc/fftools-common-opts.texi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thx [...] -- Michael GnuPG fingerprint: 9F

Re: [FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-17 Thread Aaron Levinson
On 4/17/2017 8:28 AM, wm4 wrote: On Mon, 17 Apr 2017 12:06:59 -0300 James Almer wrote: On 4/17/2017 5:39 AM, Clément Bœsch wrote: On Sun, Apr 16, 2017 at 05:20:02PM -0700, Aaron Levinson wrote: From 9e6a9e2b8d58f17c661a3f455e03c95587ec7b18 Mon Sep 17 00:00:00 2001 From: Aaron Levinson Date:

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-17 Thread Nicolas George
Le nonidi 29 germinal, an CCXXV, Michael Niedermayer a écrit : > > +while ((ret = avcodec_receive_packet(enc, &pkt)) == > > AVERROR(EAGAIN)) { > > +ret = avcodec_send_frame(enc, NULL); The doc says: # The functions will not return AVERROR(EAGAIN), unless you f

Re: [FFmpeg-devel] [PATCH] doc/encoders.texi: document libvpxenc's -row-mt

2017-04-17 Thread James Zern
On Mon, Apr 17, 2017 at 5:06 PM, Michael Niedermayer wrote: > On Thu, Apr 13, 2017 at 06:53:26PM -0700, James Zern wrote: >> added in: >> 734d760e2f lavc/libvpxenc: add -row-mt option >> >> Signed-off-by: James Zern >> --- >> doc/encoders.texi | 2 ++ >> 1 file changed, 2 insertions(+) > > LGTM

Re: [FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-17 Thread wm4
On Mon, 17 Apr 2017 18:14:45 -0700 Aaron Levinson wrote: > On 4/17/2017 8:28 AM, wm4 wrote: > > On Mon, 17 Apr 2017 12:06:59 -0300 > > James Almer wrote: > > > >> On 4/17/2017 5:39 AM, Clément Bœsch wrote: > >>> On Sun, Apr 16, 2017 at 05:20:02PM -0700, Aaron Levinson wrote: > From 9e