Re: [FFmpeg-devel] [PATCH 1/4] opencl: Print error string when compilation fails

2015-10-16 Thread Timothy Gu
On Wed, Oct 14, 2015 at 2:21 PM Michael Niedermayer wrote: > On Tue, Oct 13, 2015 at 09:48:35PM -0700, Timothy Gu wrote: > > --- > > libavutil/opencl.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > LGTM > > thanks > Set pushed. Timothy

Re: [FFmpeg-devel] [PATCH] avfilter/avfiltergraph: fix -Wunused-result warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 8:34 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 7:58 PM, Ganesh Ajjanagadde > wrote: >> >> On Wed, Oct 14, 2015 at 10:29 PM, Ganesh Ajjanagadde >> wrote: >> > Commit bf0d2d6030c239f91e0368a20fb2dc0705bfec99 introduced >> > av_warn_unused_result to avfil

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 8:34 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 8:18 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> On Fri,

Re: [FFmpeg-devel] [PATCH] avfilter/avfiltergraph: fix -Wunused-result warnings

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 7:58 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 10:29 PM, Ganesh Ajjanagadde > wrote: > > Commit bf0d2d6030c239f91e0368a20fb2dc0705bfec99 introduced > > av_warn_unused_result to avfilter/formats, whose associated warnings > > were mostly fixed in 6aaac24

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 8:18 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes > >> wrote: > >> > On Fri, Oct 1

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes >> wrote: >> > On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde >> > wrote: >> >> On Wed, Oct 14, 2015 at 10:

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes > wrote: > > On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde > > wrote: > >> On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde > >> wrote: > >>> This patch results in iden

Re: [FFmpeg-devel] [PATCH] avfilter/avfiltergraph: fix -Wunused-result warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Wed, Oct 14, 2015 at 10:29 PM, Ganesh Ajjanagadde wrote: > Commit bf0d2d6030c239f91e0368a20fb2dc0705bfec99 introduced > av_warn_unused_result to avfilter/formats, whose associated warnings > were mostly fixed in 6aaac24d72a7da631173209841a3944fcb4a3309. This > fixes the issues in avfilter/avfil

Re: [FFmpeg-devel] [PATCH] avutil/mem: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 6:39 PM, Michael Niedermayer wrote: > On Fri, Oct 16, 2015 at 05:21:57PM -0400, Ganesh Ajjanagadde wrote: >> On Fri, Oct 16, 2015 at 5:11 PM, Ganesh Ajjanagadde wrote: >> > On Fri, Oct 16, 2015 at 11:37 AM, Michael Niedermayer >> > wrote: >> >> On Thu, Oct 15, 2015 at 05:

Re: [FFmpeg-devel] [PATCH] vp9_parser: fix endless loop w/0-sized frame

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 6:46 PM, James Zern wrote: > treat this the same as an over-sized superframe packet to break out of > the parser loop and allow the decoder to fail. > > Signed-off-by: James Zern > --- > libavcodec/vp9_parser.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletion

[FFmpeg-devel] [PATCH] vp9_parser: fix endless loop w/0-sized frame

2015-10-16 Thread James Zern
treat this the same as an over-sized superframe packet to break out of the parser loop and allow the decoder to fail. Signed-off-by: James Zern --- libavcodec/vp9_parser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/vp9_parser.c b/libavcodec/vp9_parser.c in

Re: [FFmpeg-devel] [PATCH] avutil/mem: add av_warn_unused_result

2015-10-16 Thread Michael Niedermayer
On Fri, Oct 16, 2015 at 05:21:57PM -0400, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 5:11 PM, Ganesh Ajjanagadde wrote: > > On Fri, Oct 16, 2015 at 11:37 AM, Michael Niedermayer > > wrote: > >> On Thu, Oct 15, 2015 at 05:38:55PM -0400, Ganesh Ajjanagadde wrote: > >>> This adds av_warn_un

Re: [FFmpeg-devel] [PATCH] avutil: undo FF_API_CRYPTO_CONTEXT deprecation

2015-10-16 Thread James Almer
On 10/16/2015 6:36 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 4:34 PM, James Almer wrote: > >> It's been argued that the benefits of the current implementation far >> outweight >> those of making the structs opaque. >> >> This deprecation is not present in any release, so it c

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes wrote: > On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde > wrote: >> On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde >> wrote: >>> This patch results in identical behavior of movenc, and suppresses >>> -Wstrict-overflow >>> warnings obs

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Hendrik Leppkes
On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde > wrote: >> This patch results in identical behavior of movenc, and suppresses >> -Wstrict-overflow >> warnings observed in GCC 5.2: >> http://fate.ffmpeg.org/log.cgi?time=201509262

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde wrote: > This patch results in identical behavior of movenc, and suppresses > -Wstrict-overflow > warnings observed in GCC 5.2: > http://fate.ffmpeg.org/log.cgi?time=20150926231053&log=compile&slot=x86_64-archlinux-gcc-threads-misc, > "warning:

Re: [FFmpeg-devel] [PATCH] avutil/sha512: add av_warn_unused_result to av_sha512_init

2015-10-16 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 7:59 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 7:37 PM, Ganesh Ajjanagadde > wrote: >> This will trigger some useful warnings in avutil/hash that need to be fixed. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/sha512.h | 1 + >> 1 file changed

Re: [FFmpeg-devel] [PATCH] avutil: undo FF_API_CRYPTO_CONTEXT deprecation

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 4:34 PM, James Almer wrote: > It's been argued that the benefits of the current implementation far > outweight > those of making the structs opaque. > > This deprecation is not present in any release, so it can be safely > removed. > > Signed-off-by: James Almer > --

Re: [FFmpeg-devel] [PATCH 03/11] avdevice/sndio: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:46 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 10:22:17PM -0400, Ganesh Ajjanagadde wrote: >> This does not trigger any warnings, but adds robustness. >> Untested, as my configure does not compile this file. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >>

Re: [FFmpeg-devel] [PATCH 01/11] avdevice/internal: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:33 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 10:22:15PM -0400, Ganesh Ajjanagadde wrote: >> This does not trigger any warnings but adds robustness. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavdevice/internal.h | 1 + >> 1 file changed, 1 inser

Re: [FFmpeg-devel] [PATCH] avutil/file: add av_warn_unused_result to av_file_map

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:30 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 06:19:08PM -0400, Ganesh Ajjanagadde wrote: >> The return code here should be checked. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/file.h | 1 + >> 1 file changed, 1 insertion(+) > > LGTM > > t

Re: [FFmpeg-devel] [PATCH] avutil/internal: add av_warn_unused_result to avpriv_open

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:29 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 05:50:53PM -0400, Ganesh Ajjanagadde wrote: >> The open syscall can obviously fail, and its return code needs to be >> checked. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/internal.h | 1 + >>

Re: [FFmpeg-devel] [PATCH 2/2] swscale/swscale: add av_warn_unused_result to sws_init_context

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:07 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 09:46:28PM -0400, Ganesh Ajjanagadde wrote: >> This should not trigger any warnings; but adds robustness. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libswscale/swscale.h | 1 + >> 1 file changed, 1 inse

Re: [FFmpeg-devel] [PATCH] avutil/mem: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 5:11 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 11:37 AM, Michael Niedermayer > wrote: >> On Thu, Oct 15, 2015 at 05:38:55PM -0400, Ganesh Ajjanagadde wrote: >>> This adds av_warn_unused_result to functions whose return codes need to >>> be checked. >>> >>> Si

Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh Ajjanagadde wrote: >> This will trigger a bunch of warnings (rightfully so). This API has been >> abused, see the Doxygen comment above for what to do if the error code >> is not meant t

Re: [FFmpeg-devel] [PATCH] avutil/audio_fifo: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:44 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 07:26:04PM -0400, Ganesh Ajjanagadde wrote: >> This one should not trigger any warnings, but will be useful for future >> robustness. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/audio_fifo.h |

Re: [FFmpeg-devel] [PATCH] avutil/mem: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:37 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 05:38:55PM -0400, Ganesh Ajjanagadde wrote: >> This adds av_warn_unused_result to functions whose return codes need to >> be checked. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/mem.h | 3 +++ >

Re: [FFmpeg-devel] [PATCH] avfilter/transform: add av_warn_unused_result to avfilter_transform

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:32 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 05:27:44PM -0400, Ganesh Ajjanagadde wrote: >> This adds av_warn_unused_result to avfilter_transform to trigger >> warnings for non-checked return codes, currently none. >> >> Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] avutil: undo FF_API_CRYPTO_CONTEXT deprecation

2015-10-16 Thread James Almer
It's been argued that the benefits of the current implementation far outweight those of making the structs opaque. This deprecation is not present in any release, so it can be safely removed. Signed-off-by: James Almer --- libavutil/blowfish.c | 9 - libavutil/blowfish.h | 5 - liba

[FFmpeg-devel] [PATCH 3/3] doc/muxers: Document range for mpegts periods

2015-10-16 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- doc/muxers.texi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 06483fa..cef04e1 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -802,9 +802,9 @@ Set a constant muxrate (default VBR). Overri

[FFmpeg-devel] [PATCH 2/3] mpegts: Make the sdt_period a double

2015-10-16 Thread Derek Buitenhuis
Having it as a float didn't even allow enough precision to check for INT_MAX/2. Signed-off-by: Derek Buitenhuis --- libavformat/mpegtsenc.c | 4 ++-- libavformat/version.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c i

[FFmpeg-devel] [PATCH 1/3] mpegts: Make the pat_period a double

2015-10-16 Thread Derek Buitenhuis
Having it as a float didn't even allow enough precision to check for INT_MAX/2. Signed-off-by: Derek Buitenhuis --- libavformat/mpegtsenc.c | 4 ++-- libavformat/version.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c i

[FFmpeg-devel] [PATCH 0/3] mpegts: Fix range and check for pat and sdt periods

2015-10-16 Thread Derek Buitenhuis
float rounding was causing some values to not work, and there was an undocumented upper limit. Derek Buitenhuis (3): mpegts: Make the pat_period a double mpegts: Make the sdt_period a double doc/muxers: Document range for mpegts periods doc/muxers.texi | 4 ++-- libavformat/mpegts

[FFmpeg-devel] [PATCH] avformat/mp3dec: improve junk skipping heuristic

2015-10-16 Thread wm4
Commit 2b3e9bbfb529e6bde238aeb511b55ebe461664c8 caused problems for a certain API user: https://code.google.com/p/chromium/issues/detail?id=537725 https://code.google.com/p/chromium/issues/detail?id=542032 The problem seems rather arbitrary, because if there's junk, anything can happen. In this c

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: uses a queue to buffer audio and video samples

2015-10-16 Thread Michael Niedermayer
On Fri, Oct 16, 2015 at 11:14:58AM +0200, Matthieu Bouron wrote: > > > On 09/21/2015 08:05 AM, Matthieu Bouron wrote: > >On 08/25/2015 10:45 AM, Matthieu Bouron wrote: > >>From: Matthieu Bouron > >> > >>Tries to avoid losing frames when frames are not consumed > >>quickly enough. > >> > >>Lockin

Re: [FFmpeg-devel] [PATCH 03/11] avdevice/sndio: add av_warn_unused_result

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 10:22:17PM -0400, Ganesh Ajjanagadde wrote: > This does not trigger any warnings, but adds robustness. > Untested, as my configure does not compile this file. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavdevice/sndio.h | 1 + > 1 file changed, 1 insertion(+) iam no

Re: [FFmpeg-devel] [PATCH] avutil/audio_fifo: add av_warn_unused_result

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 07:26:04PM -0400, Ganesh Ajjanagadde wrote: > This one should not trigger any warnings, but will be useful for future > robustness. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/audio_fifo.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavutil/aud

Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh Ajjanagadde wrote: > This will trigger a bunch of warnings (rightfully so). This API has been > abused, see the Doxygen comment above for what to do if the error code > is not meant to be checked. > > Signed-off-by: Ganesh Ajjanagadde > --- > liba

Re: [FFmpeg-devel] [PATCH] avutil/mem: add av_warn_unused_result

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 05:38:55PM -0400, Ganesh Ajjanagadde wrote: > This adds av_warn_unused_result to functions whose return codes need to > be checked. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/mem.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavutil/mem.h b

Re: [FFmpeg-devel] [PATCH 01/11] avdevice/internal: add av_warn_unused_result

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 10:22:15PM -0400, Ganesh Ajjanagadde wrote: > This does not trigger any warnings but adds robustness. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavdevice/internal.h | 1 + > 1 file changed, 1 insertion(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B

Re: [FFmpeg-devel] [PATCH] avfilter/transform: add av_warn_unused_result to avfilter_transform

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 05:27:44PM -0400, Ganesh Ajjanagadde wrote: > This adds av_warn_unused_result to avfilter_transform to trigger > warnings for non-checked return codes, currently none. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/transform.h | 1 + > 1 file changed, 1 inserti

Re: [FFmpeg-devel] [PATCH] avutil/file: add av_warn_unused_result to av_file_map

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 06:19:08PM -0400, Ganesh Ajjanagadde wrote: > The return code here should be checked. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/file.h | 1 + > 1 file changed, 1 insertion(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC7

Re: [FFmpeg-devel] [PATCH] avutil/internal: add av_warn_unused_result to avpriv_open

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 05:50:53PM -0400, Ganesh Ajjanagadde wrote: > The open syscall can obviously fail, and its return code needs to be > checked. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/internal.h | 1 + > 1 file changed, 1 insertion(+) LGTM thx [...] -- Michael GnuPG

[FFmpeg-devel] QSV dynamically linked on linux

2015-10-16 Thread Julien Fond
On Fri, Oct 16, 2015 at 12:10 PM, Ivan Uskov wrote: > Hello Julien, > > Thursday, October 15, 2015, 7:08:10 PM, you wrote: > > JF> Hi all, > > JF> I'm using QuickSync and FFmpeg on linux. > JF> It works well for encoding and decoding when I build it with > JF> --enable-static. > > JF> But when I

Re: [FFmpeg-devel] [PATCH 2/2] swscale/swscale: add av_warn_unused_result to sws_init_context

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 09:46:28PM -0400, Ganesh Ajjanagadde wrote: > This should not trigger any warnings; but adds robustness. > > Signed-off-by: Ganesh Ajjanagadde > --- > libswscale/swscale.h | 1 + > 1 file changed, 1 insertion(+) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147E

Re: [FFmpeg-devel] [PATCH] avfilter/avfilter: Error out if audio parameters change instead of failing an assert

2015-10-16 Thread Michael Niedermayer
On Fri, Oct 16, 2015 at 10:19:11AM +0200, Paul B Mahol wrote: > On 10/16/15, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Filters which support such changes should be excluded from these checks > > > > Fixes Ticket4884 > > > > Signed-off-by: Michael Niedermayer > > --- > > l

Re: [FFmpeg-devel] [PATCH] avcodec: loongson optimize h264dsp idct and loop filter with mmi

2015-10-16 Thread Michael Niedermayer
On Fri, Oct 16, 2015 at 12:49:31PM +0800, 周晓勇 wrote: > Hi, > i am making rpm package on fedora21-loongson > but find out the master contain this patch, but neither in release/2.8 or > n2.8.1 backported to release/2.8, will be in 2.8.2 [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BAD

Re: [FFmpeg-devel] [PATCH] vp9: use AVFrame.buf[0] to check if a frame is valid

2015-10-16 Thread Hendrik Leppkes
On Thu, Oct 15, 2015 at 2:52 PM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 15, 2015 at 8:02 AM, Hendrik Leppkes > wrote: > >> AVFrame.data[0] is not guaranteed to be set with a HWAccel >> --- >> libavcodec/vp9.c | 46 +++--- >> 1 file changed, 23 inser

Re: [FFmpeg-devel] [PATCH] w32pthreads: fix mingw build on x86 with -msse2 or higher

2015-10-16 Thread Hendrik Leppkes
On Wed, Oct 14, 2015 at 11:20 PM, Hendrik Leppkes wrote: > When SSE2 or higher compiler optimizations are used, mingw uses > the _mm_mfence intrinsic for MemoryBarrier, however it doesn't include > the appropriate headers automatically. > --- > compat/w32pthreads.h | 5 + > 1 file changed, 5

Re: [FFmpeg-devel] usage of AV_GLUE?

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 8:44 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 8:03 AM, Ganesh Ajjanagadde > wrote: > >> On Thu, Oct 15, 2015 at 11:21 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Thu, Oct 15, 2015 at 8:37 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> Hi all,

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: uses a queue to buffer audio and video samples

2015-10-16 Thread Matthieu Bouron
On 09/21/2015 08:05 AM, Matthieu Bouron wrote: On 08/25/2015 10:45 AM, Matthieu Bouron wrote: From: Matthieu Bouron Tries to avoid losing frames when frames are not consumed quickly enough. Locking/Condition waiting is now performed with a NSConditionLock instead of a pthread mutex/cond

Re: [FFmpeg-devel] usage of AV_GLUE?

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 8:03 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 11:21 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Thu, Oct 15, 2015 at 8:37 PM, Ganesh Ajjanagadde > > wrote: > > > >> Hi all, > >> > >> Examining libavutil/macros, there is AV_GLUE/AV_JOIN (why are t

Re: [FFmpeg-devel] [PATCH] avutil/opencl: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 11:17 PM, highgod0401 wrote: > > From: Ganesh Ajjanagadde > Date: 2015-10-16 08:08 > To: ffmpeg-devel > CC: Ganesh Ajjanagadde > Subject: [FFmpeg-devel] [PATCH] avutil/opencl: add av_warn_unused_result > This will trigger a few warnings. My config does not compile the openc

Re: [FFmpeg-devel] [PATCH] fate: use PROGSSUF

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 02:45:45AM +0200, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 01:31:11AM +0200, Michael Niedermayer wrote: > > On Wed, Oct 14, 2015 at 07:18:27PM +0200, Christophe Gisquet wrote: > > > May require exporting in the shell var PROGSUF when invoking a > > > shell script

Re: [FFmpeg-devel] usage of AV_GLUE?

2015-10-16 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 11:21 PM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 15, 2015 at 8:37 PM, Ganesh Ajjanagadde > wrote: > >> Hi all, >> >> Examining libavutil/macros, there is AV_GLUE/AV_JOIN (why are there >> two?) defined for preprocessor token pasting. However, it is not being >> used

Re: [FFmpeg-devel] [PATCH 04/11] avdevice/alsa: add av_warn_unused_result

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 7:30 AM, Nicolas George wrote: > Le quartidi 24 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> This does not trigger any warnings, but adds robustness. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavdevice/alsa.h | 4 >> 1 file changed, 4 insertions(

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 7:30 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 06:38:10AM -0400, Ganesh Ajjanagadde wrote: >> On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes wrote: >> > On Wed, Oct 14, 2015 at 12:49 PM, Carl Eugen Hoyos >> > wrote: >> >> Ganesh Ajjanagadde mit.edu> wri

Re: [FFmpeg-devel] [PATCH 04/11] avdevice/alsa: add av_warn_unused_result

2015-10-16 Thread Nicolas George
Le quartidi 24 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > This does not trigger any warnings, but adds robustness. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavdevice/alsa.h | 4 > 1 file changed, 4 insertions(+) Ok for me. (I suppose I de-facto maintain ALSA for our sid

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 06:38:10AM -0400, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes wrote: > > On Wed, Oct 14, 2015 at 12:49 PM, Carl Eugen Hoyos wrote: > >> Ganesh Ajjanagadde mit.edu> writes: > >> > >>> What? My numbers actually show that the new code may be

Re: [FFmpeg-devel] QSV dynamically linked on linux

2015-10-16 Thread Ivan Uskov
Hello Julien, Thursday, October 15, 2015, 7:08:10 PM, you wrote: JF> Hi all, JF> I'm using QuickSync and FFmpeg on linux. JF> It works well for encoding and decoding when I build it with JF> --enable-static. JF> But when I build it with --enable-shared, it hangs. JF> So my question is: is it su

[FFmpeg-devel] [PATCH 2/2] libavcodec/mpegvideo_enc.c: Fix encoding videos with less frames than the delay of the encoder.

2015-10-16 Thread Alexis Ballier
When the encoder is fed with less frames than its delay, the picture list looks like { NULL, NULL, ..., frame, frame, frame }. When flushing the encoder (input frame == NULL), we need to ensure the picture list is shifted enough so that we do not return an empty packet, which would mean the enco

[FFmpeg-devel] [PATCH 1/2] libavformat/mxfenc.c: Fix segfault when writing an audio packet when there has not been a video one.

2015-10-16 Thread Alexis Ballier
This happens when writing the trailer of a file containing audio but that has not muxed any video packet. Fixes ticket #4817. This ticket also highlights the fact that mpeg2 video encoder produces no output when it has received less frames than its delay. --- libavformat/mxfenc.c | 2 +-

Re: [FFmpeg-devel] [PATCH] avfilter/avfilter: Error out if audio parameters change instead of failing an assert

2015-10-16 Thread Paul B Mahol
On 10/16/15, Michael Niedermayer wrote: > From: Michael Niedermayer > > Filters which support such changes should be excluded from these checks > > Fixes Ticket4884 > > Signed-off-by: Michael Niedermayer > --- > libavfilter/avfilter.c | 23 +++ > 1 file changed, 19 inserti

Re: [FFmpeg-devel] [PATCH 2/2] lavu/intmath.h: Move x86 only msvc/icl functions to x86 specific header.

2015-10-16 Thread James Almer
On 10/16/2015 3:04 AM, Matt Oliver wrote: > New patch attached. The HAVE_FAST_CLZ check is back and the intel functions > now apply to both icl and icc. Should be ok. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listin