[FFmpeg-devel] compile ffmpeg errors

2015-12-10 Thread ????????
I use below commands to compile ffmpeg, but make command reports some errors. git clone https://github.com/FFmpeg/FFmpeg.git cd FFmpeg git checkout origin/release/2.8 PKG_CONFIG_PATH="${DEST}/build/lib/pkgconfig" ./configure --prefix="${DEST}/build" \ --extra-cflags="-I${DEST}/build/include" --

Re: [FFmpeg-devel] [PATCH] nvenc set slice number to 1 to improve encoding quality and clamp initial qp value to [1, 51]

2015-12-10 Thread Agatha Hu
On 2015/12/11 11:10, Agatha Hu wrote: --- libavcodec/nvenc.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 43b8e78..6365434 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -762,6 +762,17 @@ st

[FFmpeg-devel] [PATCH] nvenc set slice number to 1 to improve encoding quality and clamp initial qp value to [1, 51]

2015-12-10 Thread Agatha Hu
--- libavcodec/nvenc.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 43b8e78..6365434 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -762,6 +762,17 @@ static av_cold int nvenc_encode_init(AVCode

[FFmpeg-devel] [PATCH] nvenc set slice number to 1 to improve encoding quality and clamp initial qp value to [1, 51]

2015-12-10 Thread Agatha Hu
--- libavcodec/nvenc.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 43b8e78..6365434 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -762,6 +762,17 @@ static av_cold int nvenc_encode_init(AVCodecC

[FFmpeg-devel] Fw: Fw: [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Mats Peterson
I do agree that calling ff_mov_read_stsd_entries() in mov.c is an ugly hack, though. But don't blame me, I didn't write it ;) It would be better to manage the palette inside of matroskadec.c, of course, even if it seems to work just fine as it is right now. Mats -- Mats Peterson http://matsp88

Re: [FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

2015-12-10 Thread Michael Niedermayer
On Thu, Dec 10, 2015 at 11:50:45PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch is definitely a good idea imo, the mov demuxer also > sets codec_tag reading the same atom, for "A_MS/ACM" codec_tag is > already set. > > Please comment, Carl Eugen > matroskadec.c |1 + > 1 file c

[FFmpeg-devel] Fw: [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Mats Peterson
I should say "QuickTime video in a Matroska file", not "QuickTime file". -- Mats Peterson http://matsp888.no-ip.org/~mats/ - Forwarded Message - From: Mats Peterson To: FFmpeg development discussions and patches Sent: Friday, December 11, 2015 2:00 AM Subject: Re: [FFmpeg-devel

Re: [FFmpeg-devel] [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Mats Peterson
It's not replacing the private data at all.It allocates extradata separately from it. No program I know of uses this extradata *in Quicktime video in Matroska using a palette* for anything but tacking the palette onto a BITMAPINFOHEADER, like MPlayer. This is by far the best solution to make MP

Re: [FFmpeg-devel] [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Hendrik Leppkes
On Thu, Dec 10, 2015 at 12:06 PM, Mats Peterson wrote: > I've attached a unified diff of the latest Git version of matroskadec.c that > does two things: > 1. It allows FFmpeg to recognize QuickTime version 0 sound sample > descriptions by using 36 instead of 86 as the minimum private data size f

Re: [FFmpeg-devel] [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Mats Peterson
I'm not very versed at (or planning to become) using Git and its utilites, that's why I provide a patch here for you to examine. I hope this approach is accepted. And although it's about two different issues, they are both concerning the same file, i.e. matroskadec.c. Mats  -- Mats Peterson htt

Re: [FFmpeg-devel] [PATCH] avfilter: add SOFAlizer audio filter

2015-12-10 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 12:03:34AM +0100, Paul B Mahol wrote: [...] > +static int query_formats(AVFilterContext *ctx) > +{ > +struct SOFAlizerContext *s = ctx->priv; > +AVFilterFormats *formats = NULL; > +AVFilterChannelLayouts *layouts = NULL; > +static int sample_rates[] = { 4800

Re: [FFmpeg-devel] [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Michael Niedermayer
On Thu, Dec 10, 2015 at 11:06:58AM +, Mats Peterson wrote: > I've attached a unified diff of the latest Git version of matroskadec.c that > does two things: > 1. It allows FFmpeg to recognize QuickTime version 0 sound sample > descriptions by using 36 instead of 86 as the minimum private data

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Philip Langdale
On 2015-12-08 02:53, Timo Rothenpieler wrote: Nvidia finaly decided to put a propper MIT license on their nvenc header, so it can be included, removing any external dependencies for nvenc and making it no longer require the non-free flag. nvenc.h is the original nvEncodeApi.h from the NVENC SDK

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Hendrik Leppkes
On Fri, Dec 11, 2015 at 12:03 AM, Carl Eugen Hoyos wrote: > On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: >> @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab >> >> die_license_disabled nonfree libaacplus >> die_license_disabled nonfree libfaac >> -die_license_disabled non

Re: [FFmpeg-devel] [PATCH] avutil/softfloat: Assert that the exponent did not overflow the legal range in av_normalize1_sf()

2015-12-10 Thread Michael Niedermayer
On Mon, Nov 09, 2015 at 09:17:34PM +0100, Andreas Cadhalpun wrote: > On 08.11.2015 21:51, Andreas Cadhalpun wrote: > > On 08.11.2015 13:41, Michael Niedermayer wrote: > >> From: Michael Niedermayer > >> > >> Signed-off-by: Michael Niedermayer > >> --- > >> libavutil/softfloat.h |1 + > >> 1

Re: [FFmpeg-devel] [PATCH]lavc/libvo-aac: Mark the encoder as experimental

2015-12-10 Thread Hendrik Leppkes
On Fri, Dec 11, 2015 at 12:07 AM, Carl Eugen Hoyos wrote: > On Wednesday 09 December 2015 09:24:06 pm Hendrik Leppkes wrote: >> On Wed, Dec 9, 2015 at 7:09 PM, James Almer wrote: >> > On 12/5/2015 9:31 PM, Carl Eugen Hoyos wrote: >> >> Hi! >> >> >> >> I prefer attached patch over removing the enc

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 11.12.2015 00:03, Carl Eugen Hoyos wrote: > On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: >> @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab >> >> die_license_disabled nonfree libaacplus >> die_license_disabled nonfree libfaac >> -die_license_disabled nonfree nvenc >

Re: [FFmpeg-devel] [PATCH] aac_fixed: fix overflow in sbr_sum_square_c

2015-12-10 Thread Andreas Cadhalpun
On 02.12.2015 20:58, Andreas Cadhalpun wrote: > On 19.11.2015 01:01, Andreas Cadhalpun wrote: >> Subject: [PATCH] sbrdsp_fixed: assert that input values for sbr_sum_square_c >> are valid >> >> Larger values can cause overflows, leading to this function returning a >> negative value. >> >> Signed-o

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Carl Eugen Hoyos
On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: > @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab > > die_license_disabled nonfree libaacplus > die_license_disabled nonfree libfaac > -die_license_disabled nonfree nvenc Sorry, but this makes absolutely no sense imo: I asked

Re: [FFmpeg-devel] [PATCH] sbr_qmf_analysis: sanitize input for 32-bit imdct

2015-12-10 Thread Andreas Cadhalpun
On 02.12.2015 20:58, Andreas Cadhalpun wrote: > On 19.11.2015 01:02, Andreas Cadhalpun wrote: >> If the input contains too many too large values, the imdct can overflow. >> Even if it didn't, the output would be larger than the valid range of 29 >> bits. >> >> Note that this is a very delicate limi

Re: [FFmpeg-devel] [PATCH 2/3] aacsbr: ensure strictly monotone time borders

2015-12-10 Thread Andreas Cadhalpun
On 02.12.2015 20:57, Andreas Cadhalpun wrote: > On 08.11.2015 22:04, Andreas Cadhalpun wrote: >> This fixes a SIGFPE crash in the aac_fixed decoder. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/aacsbr_template.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> dif

Re: [FFmpeg-devel] [PATCH]lavc/libvo-aac: Mark the encoder as experimental

2015-12-10 Thread Carl Eugen Hoyos
On Wednesday 09 December 2015 09:24:06 pm Hendrik Leppkes wrote: > On Wed, Dec 9, 2015 at 7:09 PM, James Almer wrote: > > On 12/5/2015 9:31 PM, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> I prefer attached patch over removing the encoder immediately. > And its quality is horrible. > > No idea why

[FFmpeg-devel] [PATCH 1/3] x86/hevc_sao: simplify sao_band_filter 10/12bit

2015-12-10 Thread James Almer
Signed-off-by: James Almer --- libavcodec/x86/hevc_sao_10bit.asm | 142 +++--- 1 file changed, 57 insertions(+), 85 deletions(-) diff --git a/libavcodec/x86/hevc_sao_10bit.asm b/libavcodec/x86/hevc_sao_10bit.asm index f45fc56..3a7048a 100644 --- a/libavcodec/x86/

[FFmpeg-devel] [PATCH] avfilter: add SOFAlizer audio filter

2015-12-10 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- Lite version of one sent to VLC mailing list with only slow but high quality mode present. To use you need recent netCDF library, SOFA file(s), multichannel audio and headphones. --- configure | 4 + libavfilter/Makefile | 1 + libavfil

[FFmpeg-devel] [PATCH 3/3] x86/hevc_sao: add ff_hevc_sao_edge_filter_{8, 16}_{10, 12}

2015-12-10 Thread James Almer
Signed-off-by: James Almer --- libavcodec/x86/hevc_sao_10bit.asm | 9 - libavcodec/x86/hevcdsp_init.c | 8 ++-- 2 files changed, 10 insertions(+), 7 deletions(-) diff --git a/libavcodec/x86/hevc_sao_10bit.asm b/libavcodec/x86/hevc_sao_10bit.asm index 79776ac..f81e2d5 100644 ---

[FFmpeg-devel] [PATCH 2/3] x86/hevc_sao: simplify sao_edge_filter 10/12bit

2015-12-10 Thread James Almer
Signed-off-by: James Almer --- libavcodec/x86/hevc_sao_10bit.asm | 150 ++ 1 file changed, 54 insertions(+), 96 deletions(-) diff --git a/libavcodec/x86/hevc_sao_10bit.asm b/libavcodec/x86/hevc_sao_10bit.asm index 3a7048a..79776ac 100644 --- a/libavcodec/x86/

[FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

2015-12-10 Thread Carl Eugen Hoyos
Hi! Attached patch is definitely a good idea imo, the mov demuxer also sets codec_tag reading the same atom, for "A_MS/ACM" codec_tag is already set. Please comment, Carl Eugen diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index aad567a..95cebdd 100644 --- a/libavformat/mat

[FFmpeg-devel] [PATCH 3/3] avformat/mpegtsenc: Add basic multi program support

2015-12-10 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavformat/mpegtsenc.c | 57 ++- 1 file changed, 42 insertions(+), 15 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 055e441..19032f3 100644 --- a/

Re: [FFmpeg-devel] [PATCH] diracdec: Fix FPE on invalid low_delay data

2015-12-10 Thread Michael Niedermayer
On Wed, Dec 09, 2015 at 12:56:02AM +, Kieran Kunhya wrote: > --- > libavcodec/diracdec.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavcodec/diracdec.c b/libavcodec/diracdec.c > index 6a53f38..0542ad7 100644 > --- a/libavcodec/diracdec.c > +++ b/libavcodec/diracdec.c > @@

Re: [FFmpeg-devel] [PATCH 10/11] diracdec: Read picture types by using parse_code

2015-12-10 Thread Michael Niedermayer
On Wed, Dec 09, 2015 at 12:05:36AM +, Kieran Kunhya wrote: > --- > libavcodec/diracdec.c | 57 > +-- > 1 file changed, 42 insertions(+), 15 deletions(-) should be ok thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611E

Re: [FFmpeg-devel] [PATCH 08/11] diracdec: Support new extended quantiser range

2015-12-10 Thread Michael Niedermayer
On Wed, Dec 09, 2015 at 12:05:34AM +, Kieran Kunhya wrote: > --- > libavcodec/diracdec.c | 58 > --- > 1 file changed, 37 insertions(+), 21 deletions(-) LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B

[FFmpeg-devel] [PATCH 1/3] avformat: Add av_program_add_stream_index()

2015-12-10 Thread Michael Niedermayer
From: Michael Niedermayer This will be used by the subsequent commit(s) Signed-off-by: Michael Niedermayer --- doc/APIchanges |3 +++ libavformat/avformat.h |2 ++ libavformat/hls.c |2 +- libavformat/internal.h |2 -- libavformat/mpegts.c |4 ++-- libavforma

[FFmpeg-devel] [PATCH 2/3] ffmpeg: Add basic support to mux multiple programs

2015-12-10 Thread Michael Niedermayer
From: Michael Niedermayer TODO: add docs Signed-off-by: Michael Niedermayer --- ffmpeg.h |2 ++ ffmpeg_opt.c | 38 ++ 2 files changed, 40 insertions(+) diff --git a/ffmpeg.h b/ffmpeg.h index 82ab1ee..fa24910 100644 --- a/ffmpeg.h +++ b/ffmpeg.h @@

Re: [FFmpeg-devel] [PATCH 1/2] avutil/random_seed: use arc4random() when available

2015-12-10 Thread Ganesh Ajjanagadde
On Wed, Dec 9, 2015 at 8:08 AM, Ganesh Ajjanagadde wrote: > On Sun, Dec 6, 2015 at 10:56 PM, Ganesh Ajjanagadde > wrote: >> arc4random() was designed as a superior interface for system random >> number generation, designed for OpenBSD. It is thus an improvement to >> use it whenever available. >>

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 20:37, Nicolas George wrote: > Le decadi 20 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> Using the header, one could create a dummy libfoo.so containing only >> stub functions. > > Exactly. > >> I don't think the FSF would agree [1]. > > They do not make the law. Neither do

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Nicolas George
Le decadi 20 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > Using the header, one could create a dummy libfoo.so containing only > stub functions. Exactly. > I don't think the FSF would agree [1]. They do not make the law. Claiming that the GPL enforces more than it can is obviously their ga

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 19:48, Matt Oliver wrote: > On 11 December 2015 at 04:23, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: > >> On 10.12.2015 17:42, Matt Oliver wrote: >>> Im not sure that Debian not including libcuda is a valid argument for it >>> not being a system library as thats j

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 19:03, Nicolas George wrote: > Le decadi 20 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> I'm not sure it's that easy. If that were correct, it would become >> GPL-compatible to distribute a GPL-licensed program linked with a >> proprietary library by simply inserting a MIT lice

Re: [FFmpeg-devel] [PATCH 1/8] avfilter/vf_overlay: fix memory leaks

2015-12-10 Thread Ganesh Ajjanagadde
On Thu, Dec 10, 2015 at 7:54 AM, Ganesh Ajjanagadde wrote: > On Thu, Dec 10, 2015 at 3:47 AM, Paul B Mahol wrote: >> On 12/10/15, Ganesh Ajjanagadde wrote: >>> On Wed, Dec 9, 2015 at 6:09 PM, Paul B Mahol wrote: On 12/9/15, Ganesh Ajjanagadde wrote: > On Fri, Dec 4, 2015 at 9:39 AM, G

[FFmpeg-devel] [PATCH] ffmpeg: change command line option -dump to work without -loglevel debug

2015-12-10 Thread Tom Marecek
-hex and -dump command line options do nothing unless -loglevel debug is set. -dump by itself is useful for monitoring live streams (to get the current PTS for example) however when it is used with -loglevel debug for an RTMP stream, librtmp also dumps the packet data which makes the output too

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Matt Oliver
On 11 December 2015 at 04:23, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 10.12.2015 17:42, Matt Oliver wrote: > >> > >> What is a system library depends on the system. > >> For example, Debian (main) does not even include libcuda or > >> libnvidia-encode, so they

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Nicolas George
Le decadi 20 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > I'm not sure it's that easy. If that were correct, it would become > GPL-compatible to distribute a GPL-licensed program linked with a > proprietary library by simply inserting a MIT licensed header in the middle. But yes, it is that

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 17:42, Matt Oliver wrote: >> >> What is a system library depends on the system. >> For example, Debian (main) does not even include libcuda or >> libnvidia-encode, so they certainly cannot be system libraries there. > >> > > Im not sure that Debian not including libcu

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 17:34, Hendrik Leppkes wrote: > Am 10.12.2015 17:10 schrieb "Andreas Cadhalpun" < > andreas.cadhal...@googlemail.com>: >> If that's the only thing that allows distribution of compiled nvenc.c, >> it shouldn't be enabled by default. >> > > I don't mind disabling it by default, if that

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Matt Oliver
> > > I don't mind disabling it by default, if that alleviates some concerns. > One caveat to my previous comments is that just like avisynth I think that nvenc should be disabled by default aswell. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org h

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Hendrik Leppkes
Am 10.12.2015 17:10 schrieb "Andreas Cadhalpun" < andreas.cadhal...@googlemail.com>: > > On 10.12.2015 16:49, Hendrik Leppkes wrote: > > On Thu, Dec 10, 2015 at 4:25 PM, Andreas Cadhalpun > > wrote: > >> The GPL-3 requires that the 'Corresponding Source' of the distributed object code > >> is also

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Matt Oliver
> > > >>> What is a system library depends on the system. > > >>> For example, Debian (main) does not even include libcuda or > > >>> libnvidia-encode, so they certainly cannot be system libraries there. > > >> > Im not sure that Debian not including libcuda is a valid argument for it not being a

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 16:49, Hendrik Leppkes wrote: > On Thu, Dec 10, 2015 at 4:25 PM, Andreas Cadhalpun > wrote: >> The GPL-3 requires that the 'Corresponding Source' of the distributed object >> code >> is also distributed. This is defined as [1]: >> The “Corresponding Source” for a work in object code

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Hendrik Leppkes
On Thu, Dec 10, 2015 at 4:25 PM, Andreas Cadhalpun wrote: > On 10.12.2015 15:18, Philip Langdale wrote: >> On 2015-12-10 06:05, Andreas Cadhalpun wrote: >>> On 09.12.2015 17:38, Hendrik Leppkes wrote: On Wed, Dec 9, 2015 at 5:33 PM, Timo Rothenpieler wrote: > If I understand carl r

Re: [FFmpeg-devel] [PATCH] avcodec/nellymoserdec: replace pow by exp2

2015-12-10 Thread Michael Niedermayer
On Wed, Dec 09, 2015 at 07:02:02PM -0500, Ganesh Ajjanagadde wrote: > exp2 suffices here. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/nellymoserdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 10.12.2015 15:18, Philip Langdale wrote: > On 2015-12-10 06:05, Andreas Cadhalpun wrote: >> On 09.12.2015 17:38, Hendrik Leppkes wrote: >>> On Wed, Dec 9, 2015 at 5:33 PM, Timo Rothenpieler >>> wrote: If I understand carl right, the question is not about the header, but about if dlop

[FFmpeg-devel] [PATCH RFC] avfilter/vf_delogo: Use AVPixFmtDescriptor.nb_components

2015-12-10 Thread Jean Delvare
Relying on AVPixFmtDescriptor.nb_components is cleaner and faster than checking data and linesize for every possible plane. Signed-off-by: Jean Delvare --- I see that most filters use AVPixFmtDescriptor.nb_components while others still check data and linesize. Am I correct assuming that the forme

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Philip Langdale
On 2015-12-09 21:34, wm4 wrote: On Mon, 7 Dec 2015 19:34:20 +0100 Timo Rothenpieler wrote: > I don't remember if this was discussed when avisynth and other headers > where included, but what's the advantage of directly including the > header and burden the FFmpeg sources, rather than asking th

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Philip Langdale
On 2015-12-10 06:05, Andreas Cadhalpun wrote: On 09.12.2015 17:38, Hendrik Leppkes wrote: On Wed, Dec 9, 2015 at 5:33 PM, Timo Rothenpieler wrote: If I understand carl right, the question is not about the header, but about if dlopen'ing a non-free library, the CUDA and NVENC ones in this cas

Re: [FFmpeg-devel] [PATCH 7/9] avcodec/mdct_template: use lrint instead of floor hack

2015-12-10 Thread Ganesh Ajjanagadde
On Sun, Dec 6, 2015 at 8:20 AM, Ganesh Ajjanagadde wrote: > On Tue, Dec 1, 2015 at 7:27 PM, Ganesh Ajjanagadde > wrote: >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/mdct_template.c | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/mdct_temp

Re: [FFmpeg-devel] [RFC][WIP][PATCH] avfilter: add SOFAlizer audio filter

2015-12-10 Thread Michael Niedermayer
On Thu, Dec 10, 2015 at 12:43:58AM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/Makefile | 1 + > libavfilter/af_sofalizer.c | 816 > + > libavfilter/allfilters.c | 1 + > 3 files changed, 818 insertions(+) >

Re: [FFmpeg-devel] [PATCH 1/8] avfilter/vf_overlay: fix memory leaks

2015-12-10 Thread Ganesh Ajjanagadde
On Thu, Dec 10, 2015 at 3:47 AM, Paul B Mahol wrote: > On 12/10/15, Ganesh Ajjanagadde wrote: >> On Wed, Dec 9, 2015 at 6:09 PM, Paul B Mahol wrote: >>> On 12/9/15, Ganesh Ajjanagadde wrote: On Fri, Dec 4, 2015 at 9:39 AM, Ganesh Ajjanagadde wrote: > Recent commits 6aaac24d72a7da

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable parser for mp3s by old HandBrake

2015-12-10 Thread Michael Niedermayer
On Thu, Dec 10, 2015 at 10:37:42AM +0100, Moritz Barsnick wrote: > On Wed, Dec 09, 2015 at 22:24:08 +0100, Michael Niedermayer wrote: > > +if (sscanf(str, "HandBrake %d.%d.%d", &major, &minor, µ) > > == 3) { > > +c->handbrake_version = 100*major + 1000*minor + micro

Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Ivan Uskov
Hello Hendrik, Thursday, December 10, 2015, 2:00:51 PM, you wrote: >> What is wrong then? HL> The public qsv.h uses config.h, thats not allowed. ah, got it, thank. -- Best regards, Ivanmailto:ivan.us...@nablet.com ___ ffm

[FFmpeg-devel] [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

2015-12-10 Thread Mats Peterson
I've attached a unified diff of the latest Git version of matroskadec.c that does two things: 1. It allows FFmpeg to recognize QuickTime version 0 sound sample descriptions by using 36 instead of 86 as the minimum private data size for A_QUICKTIME. 2. The palette, in QuickTime video that has one,

Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Hendrik Leppkes
On Thu, Dec 10, 2015 at 11:20 AM, Ivan Uskov wrote: > Hello Hendrik, > > Thursday, December 10, 2015, 1:10:59 PM, you wrote: > > This patch expose 3 QSV functions as public. > This is needed because the VPP needs access to these functions too. > > Please review. > >>> >>> HL> p

Re: [FFmpeg-devel] Ideas to replace the options system

2015-12-10 Thread Nicolas George
Replying to everybody at once: To Ronald, regarding "rewrite EVERYTHING!!!" versus "added as extensions of the current API": My gut feeling is that starting from scratch would prove less efforts in total. The current code is already quite convoluted, even adding a single type requires adding cod

Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Ivan Uskov
Hello Hendrik, Thursday, December 10, 2015, 1:10:59 PM, you wrote: This patch expose 3 QSV functions as public. This is needed because the VPP needs access to these functions too. Please review. >> >> HL> public API is not allowed to use config.h, it needs to be config an

Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Hendrik Leppkes
On Thu, Dec 10, 2015 at 11:01 AM, Ivan Uskov wrote: > Hello Hendrik, > > Thursday, December 10, 2015, 12:19:23 PM, you wrote: > > HL> On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote: >>> This patch expose 3 QSV functions as public. >>> This is needed because the VPP needs access to these fun

Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Ivan Uskov
Hello Hendrik, Thursday, December 10, 2015, 12:19:23 PM, you wrote: HL> On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote: >> This patch expose 3 QSV functions as public. >> This is needed because the VPP needs access to these functions too. >> >> Please review. >> HL> public API is not allo

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable parser for mp3s by old HandBrake

2015-12-10 Thread Moritz Barsnick
On Wed, Dec 09, 2015 at 22:24:08 +0100, Michael Niedermayer wrote: > +if (sscanf(str, "HandBrake %d.%d.%d", &major, &minor, µ) == > 3) { > +c->handbrake_version = 100*major + 1000*minor + micro; [...] > +if (mov->handbrake_version && > +mov->hand

Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Hendrik Leppkes
On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote: > This patch expose 3 QSV functions as public. > This is needed because the VPP needs access to these functions too. > > Please review. > public API is not allowed to use config.h, it needs to be config and OS agnostic. - Hendrik

[FFmpeg-devel] [PATCH] QSV : making three qsv routines public

2015-12-10 Thread Sven Dueking
This patch expose 3 QSV functions as public. This is needed because the VPP needs access to these functions too. Please review. Thanks, Sven 0001-make-some-internal-QSV-routines-public.patch Description: Binary data ___ ffmpeg-devel mailing list ffmp

Re: [FFmpeg-devel] [PATCH 1/8] avfilter/vf_overlay: fix memory leaks

2015-12-10 Thread Paul B Mahol
On 12/10/15, Ganesh Ajjanagadde wrote: > On Wed, Dec 9, 2015 at 6:09 PM, Paul B Mahol wrote: >> On 12/9/15, Ganesh Ajjanagadde wrote: >>> On Fri, Dec 4, 2015 at 9:39 AM, Ganesh Ajjanagadde >>> wrote: Recent commits 6aaac24d72a7da631173209841a3944fcb4a3309 and 3835554bf8ed78539a3492c23

Re: [FFmpeg-devel] Detecting invalid filter parameters

2015-12-10 Thread Jean Delvare
On Thu, 10 Dec 2015 08:58:05 +0100, Nicolas George wrote: > Le decadi 20 frimaire, an CCXXIV, Jean Delvare a écrit : > > This brings two additional questions: > > > > 1* Can a filter declare which midstream changes it does or does not > > support? > > > > 2* Can a filter be called back on midstre