Re: [FFmpeg-devel] [PATCH] mov: fix decode of fragments that overlap in time

2017-10-11 Thread John Stebbins
On 10/11/2017 06:25 PM, Michael Niedermayer wrote: On Tue, Oct 10, 2017 at 06:11:08PM -0700, John Stebbins wrote: When keyframe intervals of dash segments are not perfectly aligned, fragments in the stream can overlap in time. The previous sorting by timestamp causes packets to be read out of de

Re: [FFmpeg-devel] [PATCH] mov: fix decode of fragments that overlap in time

2017-10-11 Thread John Stebbins
On 10/11/2017 06:25 PM, Michael Niedermayer wrote: > On Tue, Oct 10, 2017 at 06:11:08PM -0700, John Stebbins wrote: >> When keyframe intervals of dash segments are not perfectly aligned, >> fragments in the stream can overlap in time. The previous sorting by >> timestamp causes packets to be read o

Re: [FFmpeg-devel] [PATCH] avformat/srt: add Haivision Open SRT protocol

2017-10-11 Thread nablet developer
On 12-Oct-17 01:01, Derek Buitenhuis wrote: Is it possible to deduplicate / share code with udp.c? It seems really iffy to duplicate a large swath of code. For example, a bunch of my review comments may also correspond to udp.c (in fact they may originate from udp.c). that's true - 90% of your c

Re: [FFmpeg-devel] [PATCH] avformat/srt: add Haivision Open SRT protocol

2017-10-11 Thread nablet developer
On 12-Oct-17 01:11, Derek Buitenhuis wrote: So user options are overridden by URL params...? exactly. at least in case of udp and few other protocols. it's documented https://ffmpeg.org/ffmpeg-protocols.html#udp The required syntax for an UDP URL is: udp://hostname:port[?options] options con

Re: [FFmpeg-devel] warning: unknown warning option '-Wno-bool-operation'; did you mean '-Wno-bool-conversion'? [-Wunknown-warning-option]

2017-10-11 Thread Helmut K. C. Tessarek
On 2017-10-11 21:59, James Almer wrote: > Clang seems to warn instead of fail when an option does not exist, so > configure things it exists. > > A solution could be to somehow make clang fail instead of warn (-Werror > only during the test maybe?). Or just revert this again and ignore the > two o

Re: [FFmpeg-devel] warning: unknown warning option '-Wno-bool-operation'; did you mean '-Wno-bool-conversion'? [-Wunknown-warning-option]

2017-10-11 Thread James Almer
On 10/10/2017 4:29 PM, Derek Buitenhuis wrote: > On 10/10/2017 8:26 PM, Helmut K. C. Tessarek wrote: >>> And then reapplied in a different form: >>> >> Yep, and there they are again: >> warning: unknown warn

Re: [FFmpeg-devel] warning: unknown warning option '-Wno-bool-operation'; did you mean '-Wno-bool-conversion'? [-Wunknown-warning-option]

2017-10-11 Thread Helmut K. C. Tessarek
On 2017-10-10 15:29, Derek Buitenhuis wrote: > This is weird. it's using check_disable_warning which should only add it if > it exists... So, what's the plan now? This warnings are truly annoying. -- regards Helmut K. C. Tessarek KeyID 0xF7832007C11F128D Key fingerprint = 28A3 166

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63'

2017-10-11 Thread James Almer
On 10/11/2017 9:22 PM, Michael Niedermayer wrote: > On Wed, Oct 11, 2017 at 09:26:18PM +, James Almer wrote: >> ffmpeg | branch: master | James Almer | Wed Oct 11 >> 17:55:25 2017 -0300| [6dfcbd80ad446ff163b47f2bf432bbf706436ea8] | committer: >> James Almer >> >> Merge commit '7cb1d9e2dbbe5b

Re: [FFmpeg-devel] [PATCH] mov: fix decode of fragments that overlap in time

2017-10-11 Thread Michael Niedermayer
On Tue, Oct 10, 2017 at 06:11:08PM -0700, John Stebbins wrote: > When keyframe intervals of dash segments are not perfectly aligned, > fragments in the stream can overlap in time. The previous sorting by > timestamp causes packets to be read out of decode order and results > in decode errors. > >

Re: [FFmpeg-devel] lavfi: add ProcAmp(color balance) vaapi video filter

2017-10-11 Thread Jun Zhao
On 2017/10/11 19:25, wm4 wrote: > On Wed, 11 Oct 2017 14:58:22 +0800 > Jun Zhao wrote: > > Most of this seems to be copy&pasted from the vaapi deint filter. Yes, VA-API VPP have the same programming flow, so vaapi scale/deint/procamp have similar file/function structure, then I think we can spli

Re: [FFmpeg-devel] lavfi: add ProcAmp(color balance) vaapi video filter

2017-10-11 Thread Jun Zhao
On 2017/10/12 6:42, Michael Niedermayer wrote: > On Wed, Oct 11, 2017 at 02:58:22PM +0800, Jun Zhao wrote: >> Makefile |1 >> allfilters.c |1 >> vf_procamp_vaapi.c | 543 >> + >> 3 files changed, 545 insertions(+) >

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63'

2017-10-11 Thread Michael Niedermayer
On Wed, Oct 11, 2017 at 09:26:18PM +, James Almer wrote: > ffmpeg | branch: master | James Almer | Wed Oct 11 > 17:55:25 2017 -0300| [6dfcbd80ad446ff163b47f2bf432bbf706436ea8] | committer: > James Almer > > Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63' > > * commit '7cb1d9e2dbbe5

Re: [FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-10-11 Thread Mark Thompson
On 11/10/17 23:34, Michael Niedermayer wrote: > Hi > > On Mon, Oct 09, 2017 at 08:04:47PM +0100, Mark Thompson wrote: > [...] > >> diff --git a/libavcodec/cbs.h b/libavcodec/cbs.h >> new file mode 100644 >> index 00..e35175fc74 >> --- /dev/null >> +++ b/libavcodec/cbs.h >> @@ -0,0 +1,283

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-11 Thread Michael Niedermayer
On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > 2017-10-10 19:36 GMT+02:00 Sasi Inguva : > > > This is required for FLV files, for which duration_pts comes out to be > > zero. > > > > Signed-off-by: Sasi Inguva > > --- > > fftools/ffmpeg.c | 9 +++-- > > 1 file changed, 7 ins

Re: [FFmpeg-devel] lavfi: add ProcAmp(color balance) vaapi video filter

2017-10-11 Thread Michael Niedermayer
On Wed, Oct 11, 2017 at 02:58:22PM +0800, Jun Zhao wrote: > > Makefile |1 > allfilters.c |1 > vf_procamp_vaapi.c | 543 > + > 3 files changed, 545 insertions(+) > 4664c5a9c7bc6e6fb86b920c6dcf1cae02d8a92f > 0001-l

Re: [FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-10-11 Thread Michael Niedermayer
Hi On Mon, Oct 09, 2017 at 08:04:47PM +0100, Mark Thompson wrote: [...] > diff --git a/libavcodec/cbs.h b/libavcodec/cbs.h > new file mode 100644 > index 00..e35175fc74 > --- /dev/null > +++ b/libavcodec/cbs.h > @@ -0,0 +1,283 @@ > +/* > + * This file is part of FFmpeg. > + * > + * FFmpeg

Re: [FFmpeg-devel] [PATCH] configure: remove libdl dependency from libndi_newtek

2017-10-11 Thread Marton Balint
On Tue, 10 Oct 2017, James Almer wrote: On 10/10/2017 6:34 PM, Marton Balint wrote: We are not using dynamic loading for libndi. Signed-off-by: Marton Balint --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index e2ef54fb3b..259619c

[FFmpeg-devel] [PATCH] Fix for paletteuse to support transparency

2017-10-11 Thread Bjorn Roche
The pallettes generated by palettegen include one transparent color. This patch enables paletteuse to identify the transparency in incoming video and tag transparent pixels on outgoing video. This requires tracking the transparency index in the palette, establishing an alpha threshold below which

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: increase max tile size

2017-10-11 Thread Nicolas George
Le decadi 20 vendémiaire, an CCXXVI, Derek Buitenhuis a écrit : > Seems slightly more reasonable to me, I think... I am fine with any solution that is safe but does not increase the complexity of the code too much. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] avformat/srt: add Haivision Open SRT protocol

2017-10-11 Thread Derek Buitenhuis
On 10/11/2017 12:08 PM, Nablet Developer wrote: > sure, what do you suggest? use "libopensrt" or just "libsrt"? the second > one wasn't recommended, because it introduces confusion with subtitle > format library. Probably the former, but I have no real strong opinion on changing it. > it's docu

Re: [FFmpeg-devel] [PATCH] avformat/srt: add Haivision Open SRT protocol

2017-10-11 Thread Derek Buitenhuis
On 10/11/2017 4:57 AM, nablet developer wrote: > you're absolutely right - this patch is based on udp.c (actually I > initially copied udp.c and replaced bsd sockets to corresponding SRT > calls, with some additional modifications). Is it possible to deduplicate / share code with udp.c? It seems

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: increase max tile size

2017-10-11 Thread Derek Buitenhuis
On 10/11/2017 8:46 AM, Paul B Mahol wrote: > Check could be removed, and add check for overflows. Seems slightly more reasonable to me, I think... - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-de

Re: [FFmpeg-devel] [PATCH] Fix for paletteuse to support transparency

2017-10-11 Thread Derek Buitenhuis
On 10/11/2017 3:04 PM, Bjorn Roche wrote: > Can you explain where I went wrong? Is "Fix for paletteuse to support > transparency" incorrect or not sufficient? Do you require that I repeat the > message in the body as well? Or perhaps I invoked my git commands > incorrectly? That tells me what the

Re: [FFmpeg-devel] [PATCH] Fix crash if av_vdpau_bind_context() is not used.

2017-10-11 Thread Ivan Kalvachev
On 10/10/17, wm4 wrote: > On Tue, 10 Oct 2017 03:24:56 +0300 > Ivan Kalvachev wrote: > >> On 10/9/17, wm4 wrote: >> > On Mon, 9 Oct 2017 03:04:53 +0300 >> > Ivan Kalvachev wrote: >> > >> >> The public functions av_alloc_vdpaucontext() and >> >> av_vdpau_alloc_context() are allocating AVVDPAUCon

Re: [FFmpeg-devel] libavcodec/blockdsp : add clear_blocks_prores func (SSE, AVX) for prores decoding

2017-10-11 Thread Ronald S. Bultje
Hi Martin, (as you've probably noticed, I'm generally particularly interested in optimization patches, not just the SIMD, but also the general algorithmic thought behind it.) On Thu, Oct 5, 2017 at 10:58 AM, Martin Vignali wrote: > In attach patchs to add a dedicated func for clear_block inside

Re: [FFmpeg-devel] [PATCH] Fix for paletteuse to support transparency

2017-10-11 Thread Bjorn Roche
On Tue, Oct 10, 2017 at 5:56 PM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 10/10/2017 10:41 PM, Bjorn Roche wrote: > > Partial fix for https://trac.ffmpeg.org/ticket/4443 > > (Animated GIFs are incorrectly optimized, so they still won’t be output > > correctly). > > --- > > libav

[FFmpeg-devel] [PATCH] avfilter:vf_libvmaf: fix errors while running with psnr=1 and improve docs

2017-10-11 Thread Ashish Pratap Singh
From: ashk43712 Hi, this patch fixes the seg fault which ocuured while running libvmaf filter with option psnr=1. This also improves libvmaf doc a bit. Signed-off-by: ashk43712 --- doc/filters.texi | 19 ++- libavfilter/vf_libvmaf.c | 7 --- 2 files changed, 10 ins

Re: [FFmpeg-devel] [PATCH] avformat/srt: add Haivision Open SRT protocol

2017-10-11 Thread Nicolas George
Le nonidi 19 vendémiaire, an CCXXVI, Nablet Developer a écrit : > protocol requires libsrt (https://github.com/Haivision/srt) to be > installed > > Signed-off-by: Nablet Developer > --- > configure |5 + > libavformat/Makefile|1 + > libavformat/opensrt.c | 1105 > +

Re: [FFmpeg-devel] lavfi: add ProcAmp(color balance) vaapi video filter

2017-10-11 Thread wm4
On Wed, 11 Oct 2017 14:58:22 +0800 Jun Zhao wrote: > Most of this seems to be copy&pasted from the vaapi deint filter. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni weighted vert mc msa functions

2017-10-11 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Wednesday, October 11, 2017 2:21 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni weighted vert

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni horiz mc msa functions

2017-10-11 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Wednesday, October 11, 2017 2:20 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni horiz mc msa

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi copy mc msa functions

2017-10-11 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Wednesday, October 11, 2017 2:19 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi copy mc msa fu

Re: [FFmpeg-devel] [PATCH] avformat/srt: add Haivision Open SRT protocol

2017-10-11 Thread Nablet Developer
On 10/10/2017 22:47, Derek Buitenhuis wrote: On 10/10/2017 7:29 AM, Nablet Developer wrote: @@ -293,6 +293,7 @@ External library support: --enable-opengl enable OpenGL rendering [no] --enable-openssl enable openssl, needed for https support

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc chroma avg horiz mc msa functions

2017-10-11 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Wednesday, October 11, 2017 1:44 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc chroma avg horiz mc

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc put mc 12, 32 and 22 msa functions

2017-10-11 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Wednesday, October 11, 2017 2:17 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc put mc 12, 32 and 2

Re: [FFmpeg-devel] [PATCH 2/6] vaapi: Remove H.264 baseline profile

2017-10-11 Thread Mark Thompson
On 11/10/17 09:10, Li, Zhong wrote: >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >> Of Mark Thompson >> Sent: Tuesday, October 10, 2017 6:36 AM >> To: ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] [PATCH 2/6] vaapi: Remove H.264 bas

[FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni weighted vert mc msa functions

2017-10-11 Thread kaustubh.raste
From: Kaustubh Raste Pack the data to half word before clipping. Use immediate unsigned saturation for clip to max saving one vector register. Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_mc_uniw_msa.c | 441 ++-- 1 file changed, 222 insertions(+), 21

[FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni horiz mc msa functions

2017-10-11 Thread kaustubh.raste
From: Kaustubh Raste Update macros to remove adds. Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_mc_uni_msa.c | 612 +++-- 1 file changed, 323 insertions(+), 289 deletions(-) diff --git a/libavcodec/mips/hevc_mc_uni_msa.c b/libavcodec/mips/hevc_mc_un

[FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi copy mc msa functions

2017-10-11 Thread kaustubh.raste
From: Kaustubh Raste Load the specific destination bytes instead of MSA load and pack. Use immediate unsigned saturation for clip to max saving one vector register. Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_mc_bi_msa.c | 652 +- 1 file changed,

[FFmpeg-devel] [PATCH] avcodec/mips: Improve avc put mc 12, 32 and 22 msa functions

2017-10-11 Thread kaustubh.raste
From: Kaustubh Raste Remove loops and unroll as block sizes are known. Removed unused functions. Signed-off-by: Kaustubh Raste --- libavcodec/mips/h264qpel_msa.c | 1013 +--- 1 file changed, 747 insertions(+), 266 deletions(-) diff --git a/libavcodec/mips/h

[FFmpeg-devel] [PATCH] avcodec/mips: Improve avc chroma avg horiz mc msa functions

2017-10-11 Thread kaustubh.raste
From: Kaustubh Raste Replace generic with block size specific function. Load the specific destination bytes instead of MSA load and pack. Signed-off-by: Kaustubh Raste --- libavcodec/mips/h264chroma_msa.c | 329 -- 1 file changed, 169 insertions(+), 160 del

Re: [FFmpeg-devel] lavfi: add ProcAmp(color balance) vaapi video filter

2017-10-11 Thread Jun Zhao
On 2017/10/11 15:18, 刘歧 wrote: >> 在 2017年10月11日,14:58,Jun Zhao 写道: >> >> >> <0001-lavfi-add-ProcAmp-color-balance-vaapi-video-filter.patch> > about the options doc need add into this patch. Doc will send with the other patch >> ___ >> ffmpeg-devel mail

Re: [FFmpeg-devel] [PATCH 2/6] vaapi: Remove H.264 baseline profile

2017-10-11 Thread Li, Zhong
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: Tuesday, October 10, 2017 6:36 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH 2/6] vaapi: Remove H.264 baseline > profile > > On 09/10/17 22:48, Ca

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: increase max tile size

2017-10-11 Thread Paul B Mahol
On 10/10/17, Derek Buitenhuis wrote: > On 10/10/2017 8:59 PM, Paul B Mahol wrote: >> Not possible, there is option used from start. > > It's possible to deprecate and add a new one. > >> Beside its small change. Why do you care at all about lavfi? > > Cool it. > > I get it's a simple change, and I

Re: [FFmpeg-devel] lavfi: add ProcAmp(color balance) vaapi video filter

2017-10-11 Thread 刘歧
> 在 2017年10月11日,14:58,Jun Zhao 写道: > > > <0001-lavfi-add-ProcAmp-color-balance-vaapi-video-filter.patch> about the options doc need add into this patch. > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/f