Re: [FFmpeg-devel] compile error caused by configure script's change

2017-03-22 Thread Dzung Hoang
I banged my head against this and finally figured out that opencv 3.2.0 does not support C language binding. You need to get opencv 2.4.13. From: Masaru Nomiya To: ffmpeg-devel@ffmpeg.org Sent: Wednesday, March 22, 2017 9:38 PM Subject: [FFmpeg-devel] compile

Re: [FFmpeg-devel] [PATCH 0/7] convert most of the remaining files to stdatomic

2017-03-22 Thread James Almer
On 3/22/2017 11:01 PM, James Almer wrote: > On 3/22/2017 10:01 PM, Michael Niedermayer wrote: >> On Wed, Mar 22, 2017 at 08:34:05PM -0300, James Almer wrote: >>> This set deals with most of the remaining cases of libavutil/atomic.h usage. >>> "make fate THREADS=4" passes here on mingw-w64, but I'm

[FFmpeg-devel] compile error caused by configure script's change

2017-03-22 Thread Masaru Nomiya
Hello, I got the very latest git head, and tried to compile. But, with ./configure's run, I got an error message as follows; [...] gcc -L/usr/local/cuda/lib64 -Wl,--as-needed -Wl,-z,noexecstack -o /tmp/ffconf.G0t8P470 /tmp/ffconf.3kiEFzc2.o -lopencv_core -lopencv_imgproc -lopencore-amrwb

Re: [FFmpeg-devel] [PATCH 0/7] convert most of the remaining files to stdatomic

2017-03-22 Thread James Almer
On 3/22/2017 10:01 PM, Michael Niedermayer wrote: > On Wed, Mar 22, 2017 at 08:34:05PM -0300, James Almer wrote: >> This set deals with most of the remaining cases of libavutil/atomic.h usage. >> "make fate THREADS=4" passes here on mingw-w64, but I'm not 100% sure fate >> has proper coverage of

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/ac3dsp: add special-case handling for the C downmix_fixed function

2017-03-22 Thread Michael Niedermayer
On Wed, Mar 22, 2017 at 12:41:26PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/ac3dsp.c | 61 > + > 1 file changed, 61 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/2] fate: add tests for ac3_fixed 5.1 downmix

2017-03-22 Thread Michael Niedermayer
On Wed, Mar 22, 2017 at 12:41:25PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > tests/fate/ac3.mak | 8 > 1 file changed, 8 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He who knows,

[FFmpeg-devel] [PATCH v8] avformat/dashdec: add dash demuxer base version

2017-03-22 Thread Steven Liu
resend, remove unnecessary file modify v2 fixed: 1. from autodetect to disabled 2. from camelCase code style to ffmpeg code style 3. from RepType to AVMediaType 4. fix variable typo 5. change time value from uint32_t to uint64_t 6. removed be used once API 7. change 'time(NULL)`, except it is not

[FFmpeg-devel] [PATCH v8] avformat/dashdec: add dash demuxer base version

2017-03-22 Thread Steven Liu
v2 fixed: 1. from autodetect to disabled 2. from camelCase code style to ffmpeg code style 3. from RepType to AVMediaType 4. fix variable typo 5. change time value from uint32_t to uint64_t 6. removed be used once API 7. change 'time(NULL)`, except it is not 2038-safe.' to av_gettime and av_timegm

Re: [FFmpeg-devel] [PATCH 0/7] convert most of the remaining files to stdatomic

2017-03-22 Thread Michael Niedermayer
On Wed, Mar 22, 2017 at 08:34:05PM -0300, James Almer wrote: > This set deals with most of the remaining cases of libavutil/atomic.h usage. > "make fate THREADS=4" passes here on mingw-w64, but I'm not 100% sure fate > has proper coverage of some of these functions. > > Only ones remaining are

Re: [FFmpeg-devel] [PATCH 0/7] convert most of the remaining files to stdatomic

2017-03-22 Thread Marton Balint
On Wed, 22 Mar 2017, James Almer wrote: This set deals with most of the remaining cases of libavutil/atomic.h usage. "make fate THREADS=4" passes here on mingw-w64, but I'm not 100% sure fate has proper coverage of some of these functions. Only ones remaining are Decklink (It's C++98 and I'm

[FFmpeg-devel] [PATCH 7/7] avcodec/videotoolboxenc: remove unused atomic header

2017-03-22 Thread James Almer
Signed-off-by: James Almer --- libavcodec/videotoolboxenc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 4b8718c9aa..f1c1670dd1 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec/videotoolboxenc.c

[FFmpeg-devel] [PATCH 0/7] convert most of the remaining files to stdatomic

2017-03-22 Thread James Almer
This set deals with most of the remaining cases of libavutil/atomic.h usage. "make fate THREADS=4" passes here on mingw-w64, but I'm not 100% sure fate has proper coverage of some of these functions. Only ones remaining are Decklink (It's C++98 and I'm not sure how wise would it be to include a

[FFmpeg-devel] [PATCH 1/7] avcodec/mediacodec: convert to stdatomic

2017-03-22 Thread James Almer
Signed-off-by: James Almer --- Untested. libavcodec/mediacodec.c | 5 ++--- libavcodec/mediacodecdec.c| 1 - libavcodec/mediacodecdec_common.c | 14 ++ libavcodec/mediacodecdec_common.h | 5 +++-- 4 files changed, 11 insertions(+), 14

Re: [FFmpeg-devel] [PATCH] clear s->error in avio_read

2017-03-22 Thread Fredrik Hubinette
ping? On Mon, Mar 20, 2017 at 2:49 PM, Fredrik Hubinette wrote: > Hopefully valid patch attached. > > > On Mon, Mar 20, 2017 at 2:39 PM, Fredrik Hubinette > wrote: > >> It looks like the value in s->error also comes from an earlier call to >> avio_read().

Re: [FFmpeg-devel] [PATCHv2 2/3] ffplay: convert to new decode API

2017-03-22 Thread Philip Langdale
On Tue, 21 Mar 2017 07:34:32 -0700 Philip Langdale wrote: > On Mon, 20 Mar 2017 21:51:49 +0100 > Marton Balint wrote: > > > Since subtitles are not yet supported with the new API, > > CODEC_CAP_DELAY subtitle codecs (only libzvbi so far) may loose the > >

[FFmpeg-devel] [PATCH] ffmpeg: Remove hw_device_ctx output filter reinit hack

2017-03-22 Thread Mark Thompson
This was skipped in c17563c5d3c974a69709ebae0171534763b3051c because it depended on the filter setup merge, but was forgotten after that actually happened. Fixes hwaccel fate for stream size change tests. --- ffmpeg_filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] [PATCH 2/2] lavc: vdpau: add support for new hw_frames_ctx and hw_device_ctx API

2017-03-22 Thread Mark Thompson
On 22/03/17 06:47, wm4 wrote: > This supports retrieving the device from a provided hw_frames_ctx, and > automatically creating a hw_frames_ctx if hw_device_ctx is set. > > The old API is not deprecated yet. The user can still use > av_vdpau_bind_context() (with or without setting hw_frames_ctx),

[FFmpeg-devel] [PATCH] avcodec/qdrw: support 16bpp files with bppcnt == 2 && bpp == 8

2017-03-22 Thread Piotr Bandurski
fixes ticket #6254 0001-avcodec-qdrw-support-16bpp-files-with-bppcnt-2-bpp-8.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat, ffmpeg: deprecate old rotation API

2017-03-22 Thread Michael Niedermayer
On Tue, Mar 21, 2017 at 08:42:58AM +0100, wm4 wrote: > The old "API" that signaled rotation as a metadata value has been > replaced by DISPLAYMATRIX side data quite a while ago. > > There is no reason to make muxers/demuxers/API users support both. In > addition, the metadata API is dangerous, as

Re: [FFmpeg-devel] [PATCH] avfilter/vf_fps: fix duration

2017-03-22 Thread Thomas Mundt
>>Michael Niedermayer schrieb am Mi, 22.3.2017: >>>On Tue, Mar 21, 2017 at 11:25:21PM +, Thomas Mundt wrote: >> Hi, >> >> attached patch will fix ticket #2674. >> It makes the fps filter duplicate or drop the last frame depending on the >> input and output frame

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264: Check weight values to be within the specs limits.

2017-03-22 Thread Michael Niedermayer
On Wed, Mar 22, 2017 at 09:06:09AM -0400, Ronald S. Bultje wrote: > Hi, > > On Tue, Mar 21, 2017 at 9:59 PM, Michael Niedermayer > wrote: > > > @@ -59,6 +59,9 @@ int ff_h264_pred_weight_table(GetBitContext *gb, const > > SPS *sps, > > if (luma_weight_flag)

[FFmpeg-devel] [PATCH 2/2] avcodec/ac3dsp: add special-case handling for the C downmix_fixed function

2017-03-22 Thread James Almer
Signed-off-by: James Almer --- libavcodec/ac3dsp.c | 61 + 1 file changed, 61 insertions(+) diff --git a/libavcodec/ac3dsp.c b/libavcodec/ac3dsp.c index 74f9e3caae..43438da131 100644 --- a/libavcodec/ac3dsp.c +++

[FFmpeg-devel] [PATCH 1/2] fate: add tests for ac3_fixed 5.1 downmix

2017-03-22 Thread James Almer
Signed-off-by: James Almer --- tests/fate/ac3.mak | 8 1 file changed, 8 insertions(+) diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak index e6362718ea..76be2e869a 100644 --- a/tests/fate/ac3.mak +++ b/tests/fate/ac3.mak @@ -36,6 +36,14 @@ FATE_AC3 +=

Re: [FFmpeg-devel] [PATCH v6] avformat/dashdec: add dash demuxer base version

2017-03-22 Thread Steven Liu
2017-03-22 22:36 GMT+08:00 Rodger Combs : > > > On Mar 22, 2017, at 09:31, Steven Liu wrote: > > > > 2017-03-22 22:01 GMT+08:00 Rodger Combs >: > > > >> > >>> On Mar 21, 2017, at 08:16, Steven Liu

Re: [FFmpeg-devel] [PATCH v6] avformat/dashdec: add dash demuxer base version

2017-03-22 Thread Rodger Combs
> On Mar 22, 2017, at 09:31, Steven Liu wrote: > > 2017-03-22 22:01 GMT+08:00 Rodger Combs >: > >> >>> On Mar 21, 2017, at 08:16, Steven Liu wrote: >>> >>> v2 fixed: >>> 1. from

Re: [FFmpeg-devel] [PATCH v6] avformat/dashdec: add dash demuxer base version

2017-03-22 Thread Steven Liu
2017-03-22 22:01 GMT+08:00 Rodger Combs : > > > On Mar 21, 2017, at 08:16, Steven Liu wrote: > > > > v2 fixed: > > 1. from autodetect to disabled > > 2. from camelCase code style to ffmpeg code style > > 3. from RepType to AVMediaType > > 4. fix

Re: [FFmpeg-devel] [PATCH v6] avformat/dashdec: add dash demuxer base version

2017-03-22 Thread Rodger Combs
> On Mar 21, 2017, at 08:16, Steven Liu wrote: > > v2 fixed: > 1. from autodetect to disabled > 2. from camelCase code style to ffmpeg code style > 3. from RepType to AVMediaType > 4. fix variable typo > 5. change time value from uint32_t to uint64_t > 6. removed be used

Re: [FFmpeg-devel] [PATCH 4/4] vaapi_vp9: Mark as async-safe

2017-03-22 Thread Ronald S. Bultje
Hi, On Tue, Mar 21, 2017 at 6:43 PM, Mark Thompson wrote: > --- > This was missed in e0cd598bc4684654d63942e9ff4872c0b48a7dc2 because it > isn't in libav. > > > libavcodec/vaapi_vp9.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavcodec/vaapi_vp9.c

Re: [FFmpeg-devel] [PATCH 3/4] vaapi_hevc: Mark as async-safe

2017-03-22 Thread Ronald S. Bultje
Hi, On Tue, Mar 21, 2017 at 6:43 PM, Mark Thompson wrote: > --- > This was missed in e0cd598bc4684654d63942e9ff4872c0b48a7dc2 because it > wasn't in libav at the time. > > > libavcodec/vaapi_hevc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264: Check weight values to be within the specs limits.

2017-03-22 Thread Ronald S. Bultje
Hi, On Tue, Mar 21, 2017 at 9:59 PM, Michael Niedermayer wrote: > @@ -59,6 +59,9 @@ int ff_h264_pred_weight_table(GetBitContext *gb, const > SPS *sps, > if (luma_weight_flag) { > pwt->luma_weight[i][list][0] = get_se_golomb(gb); >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/h264_ps: Fix runtime error: signed integer overflow: 2147483647 + 26 cannot be represented in type 'int'

2017-03-22 Thread Ronald S. Bultje
Hi, On Tue, Mar 21, 2017 at 9:59 PM, Michael Niedermayer wrote: > Fixes: 902/clusterfuzz-testcase-4561155144024064 > > Found-by: continuous fuzzing process https://github.com/google/oss- > fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 2/2] avformat/apng: set max_fps to no limit by default

2017-03-22 Thread Benoit Fouet
Hi, On 21/03/2017 14:03, James Almer wrote: > On 3/21/2017 9:52 AM, Michael Niedermayer wrote: >> On Mon, Mar 20, 2017 at 11:03:23PM -0300, James Almer wrote: >>> Should fix ticket #6252 >>> >>> Signed-off-by: James Almer >>> --- >>> libavformat/apngdec.c | 2 +- >>> 1 file

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/hevc: add the dimension constraints when attempting to use DXVA

2017-03-22 Thread Hendrik Leppkes
On Wed, Mar 22, 2017 at 10:41 AM, Steve Lhomme wrote: > --- > libavcodec/hevc.c | 12 > 1 file changed, 12 insertions(+) > Just to mention it on this ML as well so it doesn't get applied due to lack of comments - we're still discussing better alternatives for a

Re: [FFmpeg-devel] [PATCH v3] avformat/rtmpproto: change rtmp_open from url_open to url_open2

2017-03-22 Thread Steven Liu
2017-03-17 11:53 GMT+08:00 Steven Liu : > use the option set by user > > Reported-by: Lancelot Lai > Signed-off-by: Steven Liu > --- > libavformat/rtmpproto.c | 13 ++--- > 1 file changed, 6 insertions(+), 7 deletions(-) >

[FFmpeg-devel] [PATCH 2/2] avcodec/hevc: add the dimension constraints when attempting to use DXVA

2017-03-22 Thread Steve Lhomme
--- libavcodec/hevc.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index 505249e6b5..35e9a273fa 100644 --- a/libavcodec/hevc.c +++ b/libavcodec/hevc.c @@ -328,6 +328,14 @@ static void export_stream_params(AVCodecContext *avctx, const

[FFmpeg-devel] [PATCH 1/2] avcodec/mpeg12dec: set the coded dimensions with proper alignement

2017-03-22 Thread Steve Lhomme
--- libavcodec/mpeg12dec.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index e49167f89e..55fcac0cb2 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -1291,12 +1291,11 @@ static int

[FFmpeg-devel] [PATCH 2/2] lavc: vdpau: add support for new hw_frames_ctx and hw_device_ctx API

2017-03-22 Thread wm4
This supports retrieving the device from a provided hw_frames_ctx, and automatically creating a hw_frames_ctx if hw_device_ctx is set. The old API is not deprecated yet. The user can still use av_vdpau_bind_context() (with or without setting hw_frames_ctx), or use the API before that by

[FFmpeg-devel] [PATCH 1/2] lavc: Add hwaccel_flags field to AVCodecContext

2017-03-22 Thread wm4
This "reuses" the flags introduced for the av_vdpau_bind_context() API function, and makes them available to all hwaccels. This does not affect the current vdpau API, as av_vdpau_bind_context() should obviously override the AVCodecContext.hwaccel_flags flags for the sake of compatibility. ---