Re: [FFmpeg-devel] [PATCH] vf_lut: Add support for RGB48 and RGBA64

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 09:41:21PM +0100, Paul B Mahol wrote: > On 10/25/15, Steven Robertson wrote: > > Done, thanks. > > > > probably ok applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster algorithms should always be preferr

[FFmpeg-devel] [PATCH] ffmpeg.c can not process invalid PTS

2015-10-29 Thread 朱世奇
diff --git a/ffmpeg.c b/ffmpeg.c index f91fb7b..22766b3 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -3864,6 +3864,9 @@ static int process_input(int file_index) if (pkt.dts != AV_NOPTS_VALUE) pkt.dts *= ist->ts_scale; +if(pkt.pts == AV_NOPTS_VALUE) +pkt.pts = ist->next_pts; +

Re: [FFmpeg-devel] [PATCH 1/5] ffprobe: add support for printing packet strings metadata as packet tags

2015-10-29 Thread Stefano Sabatini
On date Saturday 2015-10-24 22:42:12 +0200, Marton Balint encoded: > ffprobe.xsd already contains the tag element. > > Signed-off-by: Marton Balint > --- > ffprobe.c | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) > > diff --git a/ffprobe.c b/ffprobe.c > index ac03689..

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:31 PM, wm4 wrote: > On Thu, 29 Oct 2015 20:05:28 -0400 > Ganesh Ajjanagadde wrote: > >> More generally, how is this problem "easy to verify"? It may be >> included indirectly, etc. Since you seem to think it is easy, go ahead > > Indirect inclusion is IMHO not fine for s

Re: [FFmpeg-devel] Idea for a better filter that reduces noise

2015-10-29 Thread P. van Gaans
On 10/29/2015 09:42 AM, Paul B Mahol wrote: On 10/29/15, P. van Gaans wrote: You all know the CSI episodes where they read a license plate by "enhancing" some super-grainy security footage. Nonsense, right? Well, maybe not.. If the car was parked. And it seems what I found doesn't exist yet. (b

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:31 PM, wm4 wrote: > On Thu, 29 Oct 2015 20:05:28 -0400 > Ganesh Ajjanagadde wrote: > >> More generally, how is this problem "easy to verify"? It may be >> included indirectly, etc. Since you seem to think it is easy, go ahead > > Indirect inclusion is IMHO not fine for s

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:13 PM, Hendrik Leppkes wrote: > On Fri, Oct 30, 2015 at 1:05 AM, Ganesh Ajjanagadde wrote: >> On Thu, Oct 29, 2015 at 7:50 PM, Hendrik Leppkes wrote: >>> On Fri, Oct 30, 2015 at 12:08 AM, Ganesh Ajjanagadde >>> wrote: On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bu

Re: [FFmpeg-devel] [PATCH]lavc/proresdec2: Fix slice_count for very high resolutions

2015-10-29 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > +// QT ignores the written value > > +// slice_count = AV_RB16(buf + 5); > > +slice_count = ctx->mb_height * > > ((ctx->mb_width >> log2_slice_mb_width) + > > + > > av_popcount(ctx->mb_width & (1 << l

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread wm4
On Thu, 29 Oct 2015 20:05:28 -0400 Ganesh Ajjanagadde wrote: > More generally, how is this problem "easy to verify"? It may be > included indirectly, etc. Since you seem to think it is easy, go ahead Indirect inclusion is IMHO not fine for such compat headers which define standard functions on s

Re: [FFmpeg-devel] [PATCH]lavc/cdg: Add transparency

2015-10-29 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > libavcodec/cdgraphics.c |8 > > tests/fate/video.mak |2 > > tests/ref/fate/cdgraphics | 424 > i can confirm that the outputted alpha looks correct after the patch Patch applied. Carl Eugen ___

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Michael Niedermayer
On Fri, Oct 30, 2015 at 12:50:46AM +0100, Hendrik Leppkes wrote: > On Fri, Oct 30, 2015 at 12:08 AM, Ganesh Ajjanagadde wrote: > > On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje > > wrote: > >> Hi, > >> > >> On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer > >> >>> wrote: > >> > >>> On

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Hendrik Leppkes
On Fri, Oct 30, 2015 at 1:05 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 7:50 PM, Hendrik Leppkes wrote: >> On Fri, Oct 30, 2015 at 12:08 AM, Ganesh Ajjanagadde >> wrote: >>> On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje >>> wrote: Hi, On Thu, Oct 29, 2015 at 8:

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 7:50 PM, Hendrik Leppkes wrote: > On Fri, Oct 30, 2015 at 12:08 AM, Ganesh Ajjanagadde wrote: >> On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer >>> wrote: >>> On Thu, Oct 29, 2015 at 12:19

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Hendrik Leppkes
On Fri, Oct 30, 2015 at 12:08 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje wrote: >> Hi, >> >> On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer >> wrote: >> >>> On Thu, Oct 29, 2015 at 12:19:59AM -0400, Ganesh Ajjanagadde wrote: >>> > This is more concise

Re: [FFmpeg-devel] [PATCH] avutil/mathematics: correct documentation for av_gcd

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 4:51 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 10:49:08PM -0400, Ganesh Ajjanagadde wrote: >> On Wed, Oct 28, 2015 at 10:09 AM, Ganesh Ajjanagadde >> wrote: >> > On Tue, Oct 27, 2015 at 8:18 PM, Ganesh Ajjanagadde >> > wrote: >> >> av_gcd is now always defi

Re: [FFmpeg-devel] [PATCH 1/2] swscale/swscale_internal: add av_warn_unused_result

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 5:50 PM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 09:46:08PM -0400, Ganesh Ajjanagadde wrote: >> This triggers a few warnings that need to be fixed. > > do you have a list of warnings that adds ? > Iam asking as it would make it easier to judge if they are all i

Re: [FFmpeg-devel] [PATCH] avutil/mathematics: make av_gcd more robust

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 4:46 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 10:48:14PM -0400, Ganesh Ajjanagadde wrote: >> This ensures that no undefined behavior is invoked, while retaining >> identical return values in all cases and at no loss of performance >> (identical asm on clang

Re: [FFmpeg-devel] [PATCH 08/11] avfilter/avf_showvolume: use log10 instead of log()/M_LN10

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 9:35 AM, Ganesh Ajjanagadde wrote: > > On Oct 29, 2015 9:26 AM, "Ronald S. Bultje" wrote: >> >> Hi, >> >> On Thu, Oct 29, 2015 at 8:47 AM, Ganesh Ajjanagadde >> wrote: >> >> > On Thu, Oct 29, 2015 at 8:22 AM, Paul B Mahol wrote: >> > > On 10/29/15, Ganesh Ajjanagadde wr

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer > wrote: > >> On Thu, Oct 29, 2015 at 12:19:59AM -0400, Ganesh Ajjanagadde wrote: >> > This is more concise and conveys the intent better. >> > Furthermore, it is likely more pre

Re: [FFmpeg-devel] [PATCH 10/11] avfilter/vf_ssim: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 2:07 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 1:37 PM, Michael Niedermayer > wrote: >> On Thu, Oct 29, 2015 at 12:20:08AM -0400, Ganesh Ajjanagadde wrote: >>> This is likely more precise and conveys the intent better. >>> >>> Signed-off-by: Ganesh Ajjanagadd

Re: [FFmpeg-devel] [PATCH 07/11] avfilter/avf_showspectrum: use log10 instead of log()/...

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 5:47 PM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:05AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/avf_showspectrum.c | 2 +- >> 1 file chang

Re: [FFmpeg-devel] [PATCH 3/3] swscale: rename sws_rgb2rgb_init to ff_sws_rgb2rgb_init

2015-10-29 Thread Andreas Cadhalpun
On 28.10.2015 00:22, Michael Niedermayer wrote: > On Tue, Oct 27, 2015 at 11:11:09PM +0100, Andreas Cadhalpun wrote: >> It is an internal swscale function and thus should not be exported. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libswscale/colorspace-test.c | 2 +- >> libswscale/rgb2rgb.c

Re: [FFmpeg-devel] [PATCH 09/11] avfilter/vf_psnr: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 1:34 PM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:07AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/vf_psnr.c | 2 +- >> 1 file changed, 1 ins

Re: [FFmpeg-devel] [PATCH] avformat: stop exporting ffurl_read_complete, ffurl_seek and ffurl_size

2015-10-29 Thread Andreas Cadhalpun
On 27.10.2015 22:52, Andreas Cadhalpun wrote: > On 27.10.2015 22:04, Hendrik Leppkes wrote: >> On Tue, Oct 27, 2015 at 9:58 PM, Andreas Cadhalpun >> wrote: >>> They are not in public headers and not used outside of libavformat. >>> >>> Signed-off-by: Andreas Cadhalpun >>> --- >>> >>> It's been a

Re: [FFmpeg-devel] [PATCH 2/3] swscale: rename sws_context_class to ff_sws_context_class

2015-10-29 Thread Andreas Cadhalpun
On 28.10.2015 00:21, Michael Niedermayer wrote: > On Tue, Oct 27, 2015 at 11:10:58PM +0100, Andreas Cadhalpun wrote: >> It is an internal swscale symbol and thus should not be exported. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libswscale/options.c | 4 ++-- >> libswscale/swscale_

Re: [FFmpeg-devel] [PATCH 03/11] avcodec/snowenc: use log2 instead of log() / log(2...)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:20 AM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:01AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> The expression has also been accordingly simplified. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >>

Re: [FFmpeg-devel] [PATCH 02/11] avcodec/nellymoserenc: use log2 instead of log()/M_LN2

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:23 AM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:00AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/nellymoserenc.c | 2 +- >> 1 file changed,

Re: [FFmpeg-devel] [PATCH 04/11] avcodec/zmbvenc: use log2 instead of log()/M_LN2

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 9:25 AM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:02AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/zmbvenc.c | 2 +- >> 1 file changed, 1 inse

Re: [FFmpeg-devel] [PATCH 05/11] avfilter/af_volume: use log10 instead of log()/M_LN10

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 5:45 PM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:03AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/af_volume.c | 2 +- >> 1 file changed, 1 i

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

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 7:52 PM, Ganesh Ajjanagadde wrote: > This triggers a few warnings that will need to be fixed - not that bad, > the current code is mostly fine. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/parseutils.h | 5 + > 1 file changed, 5 insertions(+) > > diff --git

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

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 6:13 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 11:03:33PM -0400, Ganesh Ajjanagadde wrote: >> On Fri, Oct 16, 2015 at 5:17 PM, Ganesh Ajjanagadde wrote: >> > On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer >> > wrote: >> >> On Thu, Oct 15, 2015 at 07:

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

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 11:04:24PM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 10:24 PM, Ganesh Ajjanagadde wrote: > > On Thu, Oct 15, 2015 at 10:22 PM, Ganesh Ajjanagadde > > wrote: > >> On Thu, Oct 15, 2015 at 10:16 PM, James Almer wrote: > >>> On 10/15/2015 10:23 PM, Ganesh Aj

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

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 11:03:33PM -0400, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 5:17 PM, Ganesh Ajjanagadde wrote: > > 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

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

2015-10-29 Thread Michael Niedermayer
Hi Brad do you have a oppinion on the change below ? you are the author of the code 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 Aj

Re: [FFmpeg-devel] [PATCH 1/2] swscale/swscale_internal: add av_warn_unused_result

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 09:46:08PM -0400, Ganesh Ajjanagadde wrote: > This triggers a few warnings that need to be fixed. do you have a list of warnings that adds ? Iam asking as it would make it easier to judge if they are all issues that need fixing or if any are valid cases that do not need a r

Re: [FFmpeg-devel] [PATCH 07/11] avfilter/avf_showspectrum: use log10 instead of log()/...

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:05AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/avf_showspectrum.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/avf_

Re: [FFmpeg-devel] [PATCH 05/11] avfilter/af_volume: use log10 instead of log()/M_LN10

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:03AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/af_volume.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG f

Re: [FFmpeg-devel] [PATCH] avutil/mathematics: correct documentation for av_gcd

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 10:49:08PM -0400, Ganesh Ajjanagadde wrote: > On Wed, Oct 28, 2015 at 10:09 AM, Ganesh Ajjanagadde > wrote: > > On Tue, Oct 27, 2015 at 8:18 PM, Ganesh Ajjanagadde > > wrote: > >> av_gcd is now always defined regardless of input. This documents this > >> change in the "doc

Re: [FFmpeg-devel] [PATCH] avutil/mathematics: make av_gcd more robust

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 10:48:14PM -0400, Ganesh Ajjanagadde wrote: > This ensures that no undefined behavior is invoked, while retaining > identical return values in all cases and at no loss of performance > (identical asm on clang and gcc). > Essentially, this patch exchanges undefined behavior w

Re: [FFmpeg-devel] [PATCH] vf_lut: Add support for RGB48 and RGBA64

2015-10-29 Thread Paul B Mahol
On 10/25/15, Steven Robertson wrote: > Done, thanks. > probably ok > > On Tue, Oct 13, 2015 at 12:51 AM, Paul B Mahol wrote: > >> On 10/11/15, Steven Robertson wrote: >> > Thanks for taking a look! >> > >> > Steve >> > >> >> lgtm, do you need to update fate? >>

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

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 10:53:59PM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 6:24 PM, Ganesh Ajjanagadde > wrote: > > Signed-off-by: Ganesh Ajjanagadde > > --- > > libavutil/opencl_internal.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavutil/opencl_interna

Re: [FFmpeg-devel] [PATCH][RFC] avcodec: disallow hwaccel with frame threads

2015-10-29 Thread Andy Furniss
Hendrik Leppkes wrote: On Thu, Oct 22, 2015 at 11:27 AM, Wang Bin wrote: VLC is using frame threading with hwaccel Then they should fix their usage, its broken by design, as explained in detail my post earlier in this thread. Seems ffmpeg cli does the same by default - OK so users can speci

Re: [FFmpeg-devel] [PATCH 10/11] avfilter/vf_ssim: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 2:33 PM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 29, 2015 at 2:07 PM, Ganesh Ajjanagadde > wrote: > >> On Thu, Oct 29, 2015 at 1:37 PM, Michael Niedermayer >> wrote: >> > On Thu, Oct 29, 2015 at 12:20:08AM -0400, Ganesh Ajjanagadde wrote: >> >> This is likely more p

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add ADPCM AICA decoder

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 10:12:02PM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/Makefile | 1 + > libavcodec/adpcm.c | 13 + > libavcodec/allcodecs.c | 1 + > libavcodec/avcodec.h| 1 + > libavcodec/codec_desc.c | 7 +++ > 5 files c

Re: [FFmpeg-devel] [PATCH 2/2] avformat: add DC STR demuxer

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 10:12:03PM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/dcstr.c | 79 > > 3 files changed, 81 insertions(+) > creat

Re: [FFmpeg-devel] [PATCH 10/11] avfilter/vf_ssim: use log10 instead of log()/log(10)

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 2:07 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 1:37 PM, Michael Niedermayer > wrote: > > On Thu, Oct 29, 2015 at 12:20:08AM -0400, Ganesh Ajjanagadde wrote: > >> This is likely more precise and conveys the intent better. > >> > >> Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCH 10/11] avfilter/vf_ssim: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 1:37 PM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:08AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/vf_ssim.c | 2 +- >> 1 file changed, 1 ins

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter - second try

2015-10-29 Thread Carl Eugen Hoyos
Sven Dueking nablet.com> writes: > Please find attached my second attempt to add Intel´s > VPP to FFMpeg. As requested, the patch includes a > documentation file I don't know much about how the documentation works but why isn't this part of the filter documentation and how are users supposed

Re: [FFmpeg-devel] [PATCH 10/11] avfilter/vf_ssim: use log10 instead of log()/log(10)

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:08AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/vf_ssim.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/vf_ssim.c b/l

Re: [FFmpeg-devel] [PATCH 09/11] avfilter/vf_psnr: use log10 instead of log()/log(10)

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:07AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/vf_psnr.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fin

Re: [FFmpeg-devel] [PATCH] avfilter/tremolo: fix wavetable buffer size

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:48:05PM +0100, Paul B Mahol wrote: > On 10/29/15, Kyle Swanson wrote: > > Signed-off-by: Kyle Swanson > > --- > > libavfilter/af_tremolo.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tre

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

2015-10-29 Thread Lucas Andrade
Em ter, 27 de out de 2015 às 09:28, Lucas Andrade escreveu: > Here is the patch to backport it to release/2.8. To be fixed on 2.8.2. > Anything else I should do? > > ps. Sorry, that's my first time on opensource project, I'm a little > confused that I needed to add it to the release branch. > Do

Re: [FFmpeg-devel] [PATCH] avfilter/asrc_sine: fix options typos

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 10:29:39AM -0500, Kyle Swanson wrote: > On Wed, Oct 28, 2015 at 3:02 PM, Michael Niedermayer > wrote: > > On Wed, Oct 28, 2015 at 07:10:24PM +0100, Nicolas George wrote: > >> Le septidi 7 brumaire, an CCXXIV, Kyle Swanson a écrit : > >> > Signed-off-by: Kyle Swanson > >> >

[FFmpeg-devel] [PATCH] Added QSV based VPP filter - second try

2015-10-29 Thread Sven Dueking
Hi all, Please find attached my second attempt to add Intel´s VPP to FFMpeg. As requested, the patch includes a documentation file (patch checker shows some warnings here, but I am not sure if they needs to be fixed). Again, looking forward to get feedback … and I am nervous ;). Thanks, Sven

Re: [FFmpeg-devel] [PATCH] avfilter/asrc_sine: fix options typos

2015-10-29 Thread Kyle Swanson
On Wed, Oct 28, 2015 at 3:02 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 07:10:24PM +0100, Nicolas George wrote: >> Le septidi 7 brumaire, an CCXXIV, Kyle Swanson a écrit : >> > Signed-off-by: Kyle Swanson >> > --- >> > libavfilter/asrc_sine.c | 4 ++-- >> > 1 file changed, 2 insert

Re: [FFmpeg-devel] [PATCH 08/11] avfilter/avf_showvolume: use log10 instead of log()/M_LN10

2015-10-29 Thread Ganesh Ajjanagadde
On Oct 29, 2015 9:26 AM, "Ronald S. Bultje" wrote: > > Hi, > > On Thu, Oct 29, 2015 at 8:47 AM, Ganesh Ajjanagadde > wrote: > > > On Thu, Oct 29, 2015 at 8:22 AM, Paul B Mahol wrote: > > > On 10/29/15, Ganesh Ajjanagadde wrote: > > >> This is likely more precise and conveys the intent better. >

Re: [FFmpeg-devel] [PATCH 04/11] avcodec/zmbvenc: use log2 instead of log()/M_LN2

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:02AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/zmbvenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/zmbvenc.c b/lib

Re: [FFmpeg-devel] [PATCH 08/11] avfilter/avf_showvolume: use log10 instead of log()/M_LN10

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 8:47 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 8:22 AM, Paul B Mahol wrote: > > On 10/29/15, Ganesh Ajjanagadde wrote: > >> This is likely more precise and conveys the intent better. > >> > >> Signed-off-by: Ganesh Ajjanagadde > >> --- > >> libavfilt

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:44 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 29, 2015 at 8:32 AM, Ganesh Ajjanagadde > wrote: > >> On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer >> > >> wrote: >> > >> >> On Th

Re: [FFmpeg-devel] [PATCH 08/11] avfilter/avf_showvolume: use log10 instead of log()/M_LN10

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:22 AM, Paul B Mahol wrote: > On 10/29/15, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/avf_showvolume.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >>

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 8:32 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer > >> wrote: > > > >> On Thu, Oct 29, 2015 at 12:19:59AM -0400, Ganesh Ajjanagadde wrote: > >> > This

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 8:27 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer > wrote: > >> On Thu, Oct 29, 2015 at 12:19:59AM -0400, Ganesh Ajjanagadde wrote: >> > This is more concise and conveys the intent better. >> > Furthermore, it is likely more pre

Re: [FFmpeg-devel] [PATCH] vp9: update timestamps in ref files using multiple invisible frames.

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 7:42 AM, Hendrik Leppkes wrote: > On Thu, Oct 29, 2015 at 12:40 PM, Ronald S. Bultje > wrote: > > For the 10-show-existing-frame, the source file indeed has a timestamp > > of 3 (or 100/33) for the second visible frame, so the fix appears to > > work correctly. For t

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 8:04 AM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:19:59AM -0400, Ganesh Ajjanagadde wrote: > > This is more concise and conveys the intent better. > > Furthermore, it is likely more precise as well due to lack of floating > > point division. > > > > Sig

Re: [FFmpeg-devel] [PATCH 02/11] avcodec/nellymoserenc: use log2 instead of log()/M_LN2

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:00AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/nellymoserenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thanks [...] -- Michael G

Re: [FFmpeg-devel] [PATCH 08/11] avfilter/avf_showvolume: use log10 instead of log()/M_LN10

2015-10-29 Thread Paul B Mahol
On 10/29/15, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/avf_showvolume.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/avf_showvolume.c b/libavfilter/avf_s

Re: [FFmpeg-devel] [PATCH 03/11] avcodec/snowenc: use log2 instead of log() / log(2...)

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:01AM -0400, Ganesh Ajjanagadde wrote: > This is likely more precise and conveys the intent better. > The expression has also been accordingly simplified. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/snowenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 d

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:19:59AM -0400, Ganesh Ajjanagadde wrote: > This is more concise and conveys the intent better. > Furthermore, it is likely more precise as well due to lack of floating > point division. > > Signed-off-by: Ganesh Ajjanagadde patchset tested on linux32, inux64, mingw32,

Re: [FFmpeg-devel] [PATCHv2] all: fix enum definition for large values

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 7:55 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 12:29 AM, Michael Niedermayer > wrote: > > On Wed, Oct 28, 2015 at 07:02:42PM -0400, Ganesh Ajjanagadde wrote: > >> On Wed, Oct 28, 2015 at 3:05 PM, Ronald S. Bultje > wrote: > >> > Hi, > >> > > >> > On We

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: add testsrc2 test source.

2015-10-29 Thread Nicolas George
Le sextidi 6 brumaire, an CCXXIV, Clement Boesch a écrit : > We have testsrc and mptestsrc so... fftestsrc? or ffmire, miresrc, > colorfuzz, rainbowlicornsrc, ... or it could also just be a "ffmpeg" mode > option for testsrc so we could have all sort of mire templates (but that's > going to be inco

Re: [FFmpeg-devel] [PATCH] vp9_parser: don't overwrite cached timestamps with nopts.

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 6:39 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 29, 2015 at 12:45 AM, James Almer wrote: > >> On 10/28/2015 11:16 PM, Ronald S. Bultje wrote: >> > Hi, >> > >> > On Wed, Oct 28, 2015 at 5:51 PM, wm4 wrote: >> > >> >> On Wed, 28 Oct 2015 16:05:49 -0400 >> >> "R

Re: [FFmpeg-devel] [PATCHv2] all: fix enum definition for large values

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 12:29 AM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 07:02:42PM -0400, Ganesh Ajjanagadde wrote: >> On Wed, Oct 28, 2015 at 3:05 PM, Ronald S. Bultje wrote: >> > Hi, >> > >> > On Wed, Oct 28, 2015 at 2:46 PM, Ganesh Ajjanagadde >> > >> > wrote: >> >> >> >> On W

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 7:36 AM, wm4 wrote: > On Thu, 29 Oct 2015 07:27:20 -0400 > Ganesh Ajjanagadde wrote: > >> On Thu, Oct 29, 2015 at 5:43 AM, wm4 wrote: >> > On Thu, 29 Oct 2015 00:19:59 -0400 >> > Ganesh Ajjanagadde wrote: >> > >> >> This is more concise and conveys the intent better. >>

Re: [FFmpeg-devel] [PATCH] avfilter/tremolo: fix wavetable buffer size

2015-10-29 Thread Paul B Mahol
On 10/29/15, Kyle Swanson wrote: > Signed-off-by: Kyle Swanson > --- > libavfilter/af_tremolo.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tremolo.c > index 50df2e4..572e9e3 100644 > --- a/libavfilter/af_tremolo.c > ++

Re: [FFmpeg-devel] [PATCH] vp9: update timestamps in ref files using multiple invisible frames.

2015-10-29 Thread Hendrik Leppkes
On Thu, Oct 29, 2015 at 12:40 PM, Ronald S. Bultje wrote: > For the 10-show-existing-frame, the source file indeed has a timestamp > of 3 (or 100/33) for the second visible frame, so the fix appears to > work correctly. For the other, only the timebase is fixed, but again > appears to be correct n

[FFmpeg-devel] [PATCH] vp9: update timestamps in ref files using multiple invisible frames.

2015-10-29 Thread Ronald S. Bultje
For the 10-show-existing-frame, the source file indeed has a timestamp of 3 (or 100/33) for the second visible frame, so the fix appears to work correctly. For the other, only the timebase is fixed, but again appears to be correct now. --- tests/ref/fate/vp9-10-show-existing-frame | 2 +- tests/re

Re: [FFmpeg-devel] [PATCH 5/8] tools/crypto_bench: add AES-CBC modes

2015-10-29 Thread Nicolas George
L'octidi 8 brumaire, an CCXXIV, Rodger Combs a écrit : > Take a look at some results; CBC is significantly slower than CTR in both > libcrypto's and our AES code. There's some hardware limitation it hits. Yes, that is to be expected. But I would expect something like this: time(AES-X-CBC) - tim

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread wm4
On Thu, 29 Oct 2015 07:27:20 -0400 Ganesh Ajjanagadde wrote: > On Thu, Oct 29, 2015 at 5:43 AM, wm4 wrote: > > On Thu, 29 Oct 2015 00:19:59 -0400 > > Ganesh Ajjanagadde wrote: > > > >> This is more concise and conveys the intent better. > >> Furthermore, it is likely more precise as well due

Re: [FFmpeg-devel] [PATCH] fate: add decimate test.

2015-10-29 Thread Nicolas George
Le quintidi 5 brumaire, an CCXXIV, Michael Niedermayer a écrit : > seems failing as 32bit on 64bit x86 linux (works on 64bit) > > --- ffmpeg/tests/ref/fate/filter-decimate-simple 2015-10-25 > 23:19:29.618103426 +0100 > +++ tests/data/fate/filter-decimate-simple 2015-10-26 00:02:08.8381

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

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 5:23 AM, wm4 wrote: > On Thu, 29 Oct 2015 09:17:16 +0100 > Hendrik Leppkes wrote: > >> On Thu, Oct 29, 2015 at 3:51 AM, Ganesh Ajjanagadde >> wrote: >> > On Thu, Oct 15, 2015 at 6:12 PM, Ganesh Ajjanagadde >> > wrote: >> >> Some warnings do get triggered that will need t

Re: [FFmpeg-devel] [PATCH 5/8] tools/crypto_bench: add AES-CBC modes

2015-10-29 Thread Rodger Combs
Take a look at some results; CBC is significantly slower than CTR in both libcrypto's and our AES code. There's some hardware limitation it hits. > On Oct 29, 2015, at 06:13, Nicolas George wrote: > > Le septidi 7 brumaire, an CCXXIV, Rodger Combs a écrit : >> --- >> tools/crypto_bench.c | 140

Re: [FFmpeg-devel] [PATCH 11/11] avutil/rational: use frexp rather than ad-hoc log to get floating point exponent

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 12:20 AM, Ganesh Ajjanagadde wrote: > This simplifies and cleans up the code. > Furthermore, it is much faster due to absence of the slow log computation. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/rational.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 del

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 5:43 AM, wm4 wrote: > On Thu, 29 Oct 2015 00:19:59 -0400 > Ganesh Ajjanagadde wrote: > >> This is more concise and conveys the intent better. >> Furthermore, it is likely more precise as well due to lack of floating >> point division. >> >> Signed-off-by: Ganesh Ajjanagadd

Re: [FFmpeg-devel] [PATCH 4/8] tools/crypto_bench: add AES-192 and AES-256

2015-10-29 Thread Nicolas George
Le septidi 7 brumaire, an CCXXIV, Rodger Combs a écrit : > --- > tools/crypto_bench.c | 82 > > 1 file changed, 82 insertions(+) Just a suggestion: maybe use macros to factor the code between the three key sizes. That would probably reduce the

Re: [FFmpeg-devel] [PATCH 7/8] tools/crypto_bench: add support for multiple lavu versions by cpuflag

2015-10-29 Thread Nicolas George
Le septidi 7 brumaire, an CCXXIV, Rodger Combs a écrit : > --- > tools/crypto_bench.c | 50 ++ > 1 file changed, 42 insertions(+), 8 deletions(-) > > diff --git a/tools/crypto_bench.c b/tools/crypto_bench.c > index 8a468ba..b513c55 100644 > --- a/to

Re: [FFmpeg-devel] [PATCH 6/8] tools/crypto_bench: switch to OpenSSL's faster AES API

2015-10-29 Thread Nicolas George
Le septidi 7 brumaire, an CCXXIV, Rodger Combs a écrit : > --- > tools/crypto_bench.c | 85 > +--- > 1 file changed, 47 insertions(+), 38 deletions(-) I do not know enough of libcrypto API to comment on the actual code. No objection if it works. Co

Re: [FFmpeg-devel] [PATCH 5/8] tools/crypto_bench: add AES-CBC modes

2015-10-29 Thread Nicolas George
Le septidi 7 brumaire, an CCXXIV, Rodger Combs a écrit : > --- > tools/crypto_bench.c | 140 > +-- > 1 file changed, 137 insertions(+), 3 deletions(-) Should be ok. Not sure all the modes are really needed since the operation mode is quite orthogon

Re: [FFmpeg-devel] [PATCH 1/3] lavc/pngdec: honor skip_frame option

2015-10-29 Thread Matthieu Bouron
On Wed, Oct 28, 2015 at 5:35 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 03:09:02PM +0100, Matthieu Bouron wrote: > > On Wed, Oct 28, 2015 at 12:59:54PM +0100, Michael Niedermayer wrote: > > > On Mon, Oct 19, 2015 at 10:28:18AM +0200, Matthieu Bouron wrote: > > > > On Sat, Oct 17, 20

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-29 Thread Matthieu Bouron
On Wed, Oct 28, 2015 at 5:36 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 03:10:00PM +0100, Matthieu Bouron wrote: > > On Wed, Oct 28, 2015 at 12:58:53PM +0100, Michael Niedermayer wrote: > > > On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote: > > > > On Tue, Oct 27

Re: [FFmpeg-devel] [PATCH] vp9_parser: don't overwrite cached timestamps with nopts.

2015-10-29 Thread Ronald S. Bultje
Hi, On Thu, Oct 29, 2015 at 12:45 AM, James Almer wrote: > On 10/28/2015 11:16 PM, Ronald S. Bultje wrote: > > Hi, > > > > On Wed, Oct 28, 2015 at 5:51 PM, wm4 wrote: > > > >> On Wed, 28 Oct 2015 16:05:49 -0400 > >> "Ronald S. Bultje" wrote: > >> > >>> Hi, > >>> > >>> On Wed, Oct 28, 2015 at 3

Re: [FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

2015-10-29 Thread wm4
On Thu, 29 Oct 2015 00:19:59 -0400 Ganesh Ajjanagadde wrote: > This is more concise and conveys the intent better. > Furthermore, it is likely more precise as well due to lack of floating > point division. > > Signed-off-by: Ganesh Ajjanagadde > --- These patches are all pretty similar. And li

Re: [FFmpeg-devel] [PATCH 3/3] WIP: lavf/utils: try to avoid decoding a frame to get the codec parameters

2015-10-29 Thread wm4
On Thu, 29 Oct 2015 08:34:42 +0100 Clément Bœsch wrote: > On Wed, Oct 28, 2015 at 06:34:06PM +0100, Hendrik Leppkes wrote: > > On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron > > wrote: > > > From: Matthieu Bouron > > > > > > Avoid decoding twice images such as jpeg and png, once in the > >

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

2015-10-29 Thread wm4
On Thu, 29 Oct 2015 09:17:16 +0100 Hendrik Leppkes wrote: > On Thu, Oct 29, 2015 at 3:51 AM, Ganesh Ajjanagadde > wrote: > > On Thu, Oct 15, 2015 at 6:12 PM, Ganesh Ajjanagadde > > wrote: > >> Some warnings do get triggered that will need to be fixed. > >> > >> Signed-off-by: Ganesh Ajjanagad

Re: [FFmpeg-devel] Idea for a better filter that reduces noise

2015-10-29 Thread Paul B Mahol
On 10/29/15, P. van Gaans wrote: > You all know the CSI episodes where they read a license plate by > "enhancing" some super-grainy security footage. Nonsense, right? Well, > maybe not.. If the car was parked. And it seems what I found doesn't > exist yet. (but perhaps I overlooked it) > > If you

[FFmpeg-devel] [PATCH] avutil/file_open: avoid file handle inheritance on Windows

2015-10-29 Thread Tobias Rapp
Attached patch fixes file lock issues in my Windows application when a child process is started with handle inheritance enabled (standard input/output redirection) while a FFmpeg transcoding is running in the parent process. BTW: It would be great if the patch could also be applied to the 2.7/

Re: [FFmpeg-devel] [PATCH 3/3] WIP: lavf/utils: try to avoid decoding a frame to get the codec parameters

2015-10-29 Thread Hendrik Leppkes
On Thu, Oct 29, 2015 at 8:34 AM, Clément Bœsch wrote: > On Wed, Oct 28, 2015 at 06:34:06PM +0100, Hendrik Leppkes wrote: >> On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron >> wrote: >> > From: Matthieu Bouron >> > >> > Avoid decoding twice images such as jpeg and png, once in the >> > avformat

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

2015-10-29 Thread Hendrik Leppkes
On Thu, Oct 29, 2015 at 3:51 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 6:12 PM, Ganesh Ajjanagadde > wrote: >> Some warnings do get triggered that will need to be fixed. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/opt.h | 2 ++ >> 1 file changed, 2 insertions(+) >>

Re: [FFmpeg-devel] [PATCH 3/3] WIP: lavf/utils: try to avoid decoding a frame to get the codec parameters

2015-10-29 Thread Clément Bœsch
On Thu, Oct 29, 2015 at 08:34:42AM +0100, Clément Bœsch wrote: > On Wed, Oct 28, 2015 at 06:34:06PM +0100, Hendrik Leppkes wrote: > > On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron > > wrote: > > > From: Matthieu Bouron > > > > > > Avoid decoding twice images such as jpeg and png, once in the

Re: [FFmpeg-devel] [PATCH 3/3] WIP: lavf/utils: try to avoid decoding a frame to get the codec parameters

2015-10-29 Thread Clément Bœsch
On Wed, Oct 28, 2015 at 06:34:06PM +0100, Hendrik Leppkes wrote: > On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron > wrote: > > From: Matthieu Bouron > > > > Avoid decoding twice images such as jpeg and png, once in the > > avformat_find_stream_info and once when the actual decode is made. > >