Re: [FFmpeg-devel] [PATCH] fate: add test for asetnsamples filter with padding disabled

2017-09-07 Thread Michael Niedermayer
On Tue, Sep 05, 2017 at 03:13:15PM +0200, Tobias Rapp wrote: > Adds another test for asetnsamples filter where padding of the last > frame is switched off. Renames the existing test to make the difference > obvious. > > Signed-off-by: Tobias Rapp > --- > tests/fate/filter-audio.mak

Re: [FFmpeg-devel] [FFmpeg-cvslog] FATE: Add FITS tests

2017-09-07 Thread Michael Niedermayer
On Wed, Aug 30, 2017 at 10:20:32AM +, Paras Chadha wrote: > ffmpeg | branch: master | Paras Chadha | Tue Aug 29 > 23:24:42 2017 +0530| [9d99f0afbeedf3e170478c3e8c05789bf2eef48f] | committer: > Paul B Mahol > > FATE: Add FITS tests > > Signed-off-by: Paras Chadha > > > http://git.videolan

Re: [FFmpeg-devel] [PATCH] configure: Fix DEF file post-processing with LTO enabled.

2017-09-07 Thread Michael Niedermayer
On Wed, Sep 06, 2017 at 08:03:18PM +0200, Kacper Michajlow wrote: > 2017-08-22 21:26 GMT+02:00 Kacper Michajłow : > > > With LTO enabled exported symbol entry looks like: > > av_audio_convert @3 DATA > > > > In order to maintain valid format we need to strip everything after @. > > > > This patch

Re: [FFmpeg-devel] [PATCH] libavcodec/h264_parse: don't use uninitialized value when chroma_format_idc==0

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 09:42:07AM -0400, Mark Wachsler wrote: > When parsing a monochrome file, chroma_log2_weight_denom was used without > being initialized, which could lead to a bogus error message being printed, > e.g. > [h264 @ 0x61a26480] chroma_log2_weight_denom 24576 is out of range

[FFmpeg-devel] [PATCH] avcodec/hevc_ps: improve check for missing default display window bitstream

2017-09-07 Thread James Almer
Fixes ticket #6644 Signed-off-by: James Almer --- libavcodec/hevc_ps.c | 33 +++-- 1 file changed, 27 insertions(+), 6 deletions(-) diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c index ee31cc093c..eb104ca1b9 100644 --- a/libavcodec/hevc_ps.c +++ b/libavcode

[FFmpeg-devel] [PATCHv3] avformat/mpegts: opus muxing & demuxing for mapping family 255

2017-09-07 Thread pkv.stream
Hi, I've removed inline declarations in the patch since they're not accepted in ffmpeg. Thanks for any more comments or reviews. Regards From d05c62039b3edfd151b8c3824cc8f509f9ea8053 Mon Sep 17 00:00:00 2001 From: pkviet Date: Tue, 29 Aug 2017 01:24:12 +0200 Subject: [PATCH] avformat/mpegt

Re: [FFmpeg-devel] [PATCH 5/5] ffmpeg: send EOF pts to filters.

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 10:59:39AM +0200, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > ffmpeg.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) should be ok thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Rewriting c

Re: [FFmpeg-devel] [PATCH 3/5] ffmpeg: use reordered duration for stream PTS.

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 10:59:37AM +0200, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > ffmpeg.c | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) probably ok thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I am the wisest

Re: [FFmpeg-devel] [PATCH 2/5] ffmpeg: rename a variable.

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 10:59:36AM +0200, Nicolas George wrote: > Makes the reason of the "FIXME" comment more obvious. > Avoid name conflicts for the next commit. > > Signed-off-by: Nicolas George > --- > ffmpeg.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) LGTM thx

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mxfdec: use the common packet pts setter function for opatom files

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 05:11:40PM +0200, Marton Balint wrote: > Fixes ticket #6631. > > Signed-off-by: Marton Balint > --- > libavformat/mxfdec.c | 12 +++- > 1 file changed, 3 insertions(+), 9 deletions(-) please add a fate this if you push this no more comments from me thx [...]

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfdec: factorize packet pts setter function

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 05:11:39PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavformat/mxfdec.c | 52 > +--- > 1 file changed, 29 insertions(+), 23 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128

[FFmpeg-devel] [PATCH] libavformat/matroskaenc.c: Write Tags element for WebM

2017-09-07 Thread Ivan Janatra
This is already supported per https://www.webmproject.org/docs/container/#Tags and https://github.com/nbirkbeck/matroska-specification/commit/28a54f991f118fff31fe6bfe256c2dfab46d00e5 --- libavformat/matroskaenc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavfo

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tonemap: don't use NAN constant as an initializer

2017-09-07 Thread Michael Niedermayer
On Thu, Sep 07, 2017 at 06:26:50PM -0300, James Almer wrote: > On 9/7/2017 6:16 PM, Michael Niedermayer wrote: > > On Wed, Sep 06, 2017 at 10:59:13PM -0300, James Almer wrote: > >> Netbsd: > >> src/libavfilter/vf_tonemap.c:314: error: initializer element is not > >> constant > >> src/libavfilter/v

[FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-07 Thread Mark Thompson
--- Now sets the trusted packet flag; otherwise unchanged. configure| 1 + libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/kmsgrab.c| 455 +++ 4 files changed, 458 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH 4/5] lavc: Add wrapped_avframe decoder

2017-09-07 Thread Mark Thompson
Intended for use with hardware frames for which rawvideo is not sufficient. Requires the trusted packet flag to be set - decoding fails if not to avoid security issues (the wrapped AVFrame can contain pointers to arbitrary data). --- Now uses the trusted packet flag, rejects the packet if not set.

[FFmpeg-devel] [PATCH 1/5] lavu: Add DRM hwcontext

2017-09-07 Thread Mark Thompson
--- Unchanged. configure | 3 + libavutil/Makefile | 2 + libavutil/hwcontext.c | 4 + libavutil/hwcontext.h | 1 + libavutil/hwcontext_drm.c | 291 + libavutil/hwcontext_drm.h | 166

[FFmpeg-devel] [PATCH 3/5] lavc: Add flag to mark packets containing trusted input

2017-09-07 Thread Mark Thompson
--- Also version bump. As suggested by previous discussion around a wrapped_avframe decoder. doc/APIchanges | 3 +++ libavcodec/avcodec.h | 7 +++ 2 files changed, 10 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index b98a3419c4..26aa0b6481 100644 --- a/doc/APIchanges +

[FFmpeg-devel] [PATCH 2/5] hwcontext_vaapi: Add DRM to VAAPI mapping

2017-09-07 Thread Mark Thompson
--- Unchanged. libavutil/hwcontext_vaapi.c | 215 +++- 1 file changed, 214 insertions(+), 1 deletion(-) diff --git a/libavutil/hwcontext_vaapi.c b/libavutil/hwcontext_vaapi.c index 3970726d30..c69db39238 100644 --- a/libavutil/hwcontext_vaapi.c +++ b/liba

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tonemap: don't use NAN constant as an initializer

2017-09-07 Thread James Almer
On 9/7/2017 6:16 PM, Michael Niedermayer wrote: > On Wed, Sep 06, 2017 at 10:59:13PM -0300, James Almer wrote: >> Netbsd: >> src/libavfilter/vf_tonemap.c:314: error: initializer element is not constant >> src/libavfilter/vf_tonemap.c:314: error: (near initialization for >> 'tonemap_options[8].defa

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tonemap: don't use NAN constant as an initializer

2017-09-07 Thread Michael Niedermayer
On Wed, Sep 06, 2017 at 10:59:13PM -0300, James Almer wrote: > Netbsd: > src/libavfilter/vf_tonemap.c:314: error: initializer element is not constant > src/libavfilter/vf_tonemap.c:314: error: (near initialization for > 'tonemap_options[8].default_val.dbl') > > DJGPP > src/libavfilter/vf_tonemap.

[FFmpeg-devel] [PATCH 2/2] avcodec/vp9: change avctx->execute3 in favor of ff_slice_thread_execute_with_mainfunc()

2017-09-07 Thread Ilia Valiakhmetov
Signed-off-by: Ilia Valiakhmetov v8: --- libavcodec/vp9.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index b780262..a71045e 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -1628,7 +1628,7 @@ FF_ENABLE_DEPRECATION_WARNINGS

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/pthread_slice: add main function support for avpriv_slicethread_create()

2017-09-07 Thread gh0st
Fixed in v2. On Fri, Sep 8, 2017 at 3:40 AM, Michael Niedermayer wrote: > On Fri, Sep 08, 2017 at 12:55:29AM +0700, Ilia Valiakhmetov wrote: > > --- > > libavcodec/internal.h | 4 > > libavcodec/pthread_slice.c | 33 ++--- > > libavcodec/thread.h|

[FFmpeg-devel] [PATCHv2 2/2] avcodec/pthread_slice: add ff_slice_thread_execute_with_mainfunc()

2017-09-07 Thread Ilia Valiakhmetov
Signed-off-by: Ilia Valiakhmetov v2: --- libavcodec/internal.h | 4 libavcodec/pthread_slice.c | 22 -- libavcodec/thread.h| 4 +++- 3 files changed, 27 insertions(+), 3 deletions(-) diff --git a/libavcodec/internal.h b/libavcodec/internal.h index 64120ea

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/pthread_slice: add main function support for avpriv_slicethread_create()

2017-09-07 Thread Michael Niedermayer
On Fri, Sep 08, 2017 at 12:55:29AM +0700, Ilia Valiakhmetov wrote: > --- > libavcodec/internal.h | 4 > libavcodec/pthread_slice.c | 33 ++--- > libavcodec/thread.h| 1 + > libavutil/slicethread.h| 18 ++ > 4 files changed, 37 ins

[FFmpeg-devel] [PATCHv11] libavcodec: v4l2: add support for v4l2 mem2mem codecs

2017-09-07 Thread Jorge Ramirez-Ortiz
This patchset enhances Alexis Ballier's original patch and validates it using Qualcomm's Venus hardware (driver recently landed upstream [1]). This has been tested on Qualcomm's DragonBoard 410c and 820c Configure/make scripts have been validated on Ubuntu 10.04 and 16.04.

[FFmpeg-devel] [PATCH] avfilter: add starfield video source

2017-09-07 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vsrc_starfield.c | 191 +++ 3 files changed, 193 insertions(+) create mode 100644 libavfilter/vsrc_starfield.c diff --git a/libavfilter

Re: [FFmpeg-devel] [PATCH 3/4] avdevice/decklink_dec: Added Closed caption decode from VANC

2017-09-07 Thread Marton Balint
On Tue, 5 Sep 2017, Jeyapal, Karthick wrote: On 9/3/17, 8:19 PM, "Marton Balint" mailto:c...@passwd.hu>> wrote: Why don't use you simply use uint16_t here as well? I think we typically try to avoid unsigned as a type. Done. Changed it to uint16_t. This VANC parser does not seem right.

Re: [FFmpeg-devel] [PATCH] vf_fps: uses the last frame duration to duplicate it if needed

2017-09-07 Thread Thierry Foucu
On Thu, Sep 7, 2017 at 11:37 AM, Nicolas George wrote: > [ Ccing you, but no need to Cc me. ] > > Le primidi 21 fructidor, an CCXXV, Thierry Foucu a écrit : > > Sorry I did not know and I did not see any documentation in the header > > explaining which field of the AVFrame can/cannot be used by f

[FFmpeg-devel] [PATCH] avformat/mux: stop delaying writing the header

2017-09-07 Thread James Almer
There's no need to wait for the first packet of every stream now that every bitstream filter behaves as intended. Signed-off-by: James Almer --- What should we do with the AVFMT_FLAG_AUTO_BSF flag? Do we deprecate it and force the automatic insertion of muxer-required bitstream filters now that t

Re: [FFmpeg-devel] [PATCH] vf_fps: uses the last frame duration to duplicate it if needed

2017-09-07 Thread Nicolas George
[ Ccing you, but no need to Cc me. ] Le primidi 21 fructidor, an CCXXV, Thierry Foucu a écrit : > Sorry I did not know and I did not see any documentation in the header > explaining which field of the AVFrame can/cannot be used by filter. Yes, it lacks documentation. > If we cannot used the dura

Re: [FFmpeg-devel] [PATCH] vf_fps: uses the last frame duration to duplicate it if needed

2017-09-07 Thread Thierry Foucu
Hi Nicolas, On Wed, Sep 6, 2017 at 2:07 PM, Nicolas George wrote: > Le decadi 20 fructidor, an CCXXV, Thierry Foucu a écrit : > > --- > > libavfilter/vf_fps.c| 40 ++ > +- > > tests/ref/fate/filter-fps | 6 ++ > > tests/ref/fate/filter-fps-r |

Re: [FFmpeg-devel] [PATCH] avformat/mux: stop delaying writing the header

2017-09-07 Thread Michael Niedermayer
On Fri, May 26, 2017 at 05:08:51PM -0300, James Almer wrote: > There's no need to wait for the first packet of every stream now that > every bitstream filter behaves as intended. > > Signed-off-by: James Almer > --- > What should we do with the AVFMT_FLAG_AUTO_BSF flag? Do we deprecate > it and f

[FFmpeg-devel] [PATCH 2/2] avcodec/pthread_slice: add main function support for avpriv_slicethread_create()

2017-09-07 Thread Ilia Valiakhmetov
--- libavcodec/internal.h | 4 libavcodec/pthread_slice.c | 33 ++--- libavcodec/thread.h| 1 + libavutil/slicethread.h| 18 ++ 4 files changed, 37 insertions(+), 19 deletions(-) diff --git a/libavcodec/internal.h b/libavcodec/in

Re: [FFmpeg-devel] [PATCH 1/4] avdevice/decklink_dec: Added VANC search for all resolutions

2017-09-07 Thread Marton Balint
On Tue, 5 Sep 2017, Jeyapal, Karthick wrote: Overlooked a “signed vs unsigned comparison” compilation warning in the previous patch. Sorry about that. Please use this updated patch(attached) for review. Thanks, I think it is better if we parse all lines after the line affected by switchin

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

2017-09-07 Thread Steven Liu
2017-09-05 17:36 GMT+08:00 Steven Liu : > 2017-09-01 18:26 GMT+08:00 Steven Liu : >> ffmpeg need a dash demuxer for demux the dash formats base on >> https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch >> >> TODO: >> 1. support multi

Re: [FFmpeg-devel] [PATCH] lavd: implement threaded NewTek NDI output

2017-09-07 Thread Maksym Veremeyenko
06.09.2017 11:42, Marton Balint пише: [...] I've given this some more thought, and this only makes sense if NDI is throttling the output (e.g via clock_audio or clock_video). If not, then using threads should not make any difference, so something different might be going on. Do you have an idea

Re: [FFmpeg-devel] [PATCH] lavd: implement threaded NewTek NDI output

2017-09-07 Thread Maksym Veremeyenko
05.09.2017 23:50, Marton Balint пише: [...] If I get this correctly, this patch is needed because you can only schedule 1 frame to the NDI API, therefore especially for shorter audio frames the buffer may underrun, right?. If that is the case, then I'd describe this in a bit more detail in the

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > ffmpeg -lavfi > "testsrc2,trim=end_frame=1,zoompan=z='min(zoom+0.0015,1.5)':d=700:x='iw/2-(iw/zoom/2)':y='ih/2-(ih/zoom/2)'" > -f framecrc - > > Should output 700 frames and not 1 frame. It does. So my patch is valid. Will push soon unl

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Paul B Mahol
On 9/7/17, Nicolas George wrote: > Le primidi 21 fructidor, an CCXXV, Paul B Mahol a ecrit : >> Take single image as input, and example as listed in documentation of >> zoompan: >> >> zoompan=z='min(zoom+0.0015,1.5)':d=700:x='iw/2-(iw/zoom/2)':y='ih/2-(ih/zoom/2)' > > I am sorry but I have better

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > Take single image as input, and example as listed in documentation of zoompan: > > zoompan=z='min(zoom+0.0015,1.5)':d=700:x='iw/2-(iw/zoom/2)':y='ih/2-(ih/zoom/2)' I am sorry but I have better use of my time than debugging this filter. S

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Paul B Mahol
On 9/7/17, Nicolas George wrote: > Le primidi 21 fructidor, an CCXXV, Paul B Mahol a ecrit : >> What it should be? > > It should look like it does look after this patch. Maybe the change > breaks something I did not notice, but the general logic should be what > I wrote. > > The things that are wr

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > What it should be? It should look like it does look after this patch. Maybe the change breaks something I did not notice, but the general logic should be what I wrote. The things that are wrong in the current state of code: checking ff_o

[FFmpeg-devel] [PATCH 1/2] avformat/mxfdec: factorize packet pts setter function

2017-09-07 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 52 +--- 1 file changed, 29 insertions(+), 23 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 91731a7533..3e2f5011ee 100644 --- a/libavformat/mxfdec.c +++ b/libavfo

[FFmpeg-devel] [PATCH 2/2] avformat/mxfdec: use the common packet pts setter function for opatom files

2017-09-07 Thread Marton Balint
Fixes ticket #6631. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 3e2f5011ee..476d284c96 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -321

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Paul B Mahol
On 9/7/17, Nicolas George wrote: > Le primidi 21 fructidor, an CCXXV, Paul B Mahol a ecrit : >> ffmpeg | branch: master | Paul B Mahol | Thu Sep 7 >> 16:49:46 2017 +0200| [e1524de4546beab75cbf600fdde6c14204a66059] | >> committer: Paul B Mahol >> >> avfilter/vf_zoompan: fix specific corner case w

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_zoompan: fix specific corner case when no frame was ever requested from input

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > ffmpeg | branch: master | Paul B Mahol | Thu Sep 7 > 16:49:46 2017 +0200| [e1524de4546beab75cbf600fdde6c14204a66059] | committer: > Paul B Mahol > > avfilter/vf_zoompan: fix specific corner case when no frame was ever > requested fro

Re: [FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > > ./ffmpeg_g -lavfi testsrc2,zoompan -f framecrc - > Lavd bug. Read again, this is not using lavd at all. And it should work. The fact that it does not is definitely a zoompan bug. Regards, -- Nicolas George signature.asc Descripti

Re: [FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Paul B Mahol
On 9/7/17, Nicolas George wrote: > Le primidi 21 fructidor, an CCXXV, Paul B Mahol a ecrit : >> Works here, you will need to be more verbose. > > ./ffmpeg_g -lavfi testsrc2,zoompan -f framecrc - Lavd bug. Normal -vf approach works. ___ ffmpeg-devel mail

Re: [FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > Works here, you will need to be more verbose. ./ffmpeg_g -lavfi testsrc2,zoompan -f framecrc - begs to differ. Regards, -- Nicolas George signature.asc Description: Digital signature ___

Re: [FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Paul B Mahol
On 9/7/17, Nicolas George wrote: > Le primidi 21 fructidor, an CCXXV, Paul B Mahol a ecrit : >> > I do not have the faintest idea what the output of this filter is >> > supposed >> > to look like, so testing is limited. But at least now it outputs >> > something. >> Breaks duration expression. > >

Re: [FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Paul B Mahol a écrit : > > I do not have the faintest idea what the output of this filter is supposed > > to look like, so testing is limited. But at least now it outputs something. > Breaks duration expression. I do not see how it could be, as the filter current

Re: [FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Paul B Mahol
On 9/7/17, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavfilter/vf_zoompan.c | 36 +--- > 1 file changed, 9 insertions(+), 27 deletions(-) > > > I do not have the faintest idea what the output of this filter is supposed > to look like, so test

Re: [FFmpeg-devel] [PATCH]lavf/supenc: Add a raw PGS muxer

2017-09-07 Thread Carl Eugen Hoyos
2017-09-07 16:14 GMT+02:00 wm4 : > On Thu, 7 Sep 2017 15:32:58 +0200 > Carl Eugen Hoyos wrote: > >> Hi! >> >> Attached patch implements ticket #2208. >> >> Please comment, Carl Eugen > > Probably worse than: Looks similar except for working with mkv input and a useless cast. > From: phint...@gm

Re: [FFmpeg-devel] [PATCH]lavf/supenc: Add a raw PGS muxer

2017-09-07 Thread wm4
On Thu, 7 Sep 2017 15:32:58 +0200 Carl Eugen Hoyos wrote: > Hi! > > Attached patch implements ticket #2208. > > Please comment, Carl Eugen Probably worse than: From: phint...@gmail.com To: ffmpeg-devel@ffmpeg.org Cc: Petri Hintukainen Subject: [FFmpeg-devel] [PATCH v2] Add SUP/PGS subtit

Re: [FFmpeg-devel] [PATCH 1/7] build: fix objcc header check

2017-09-07 Thread Clément Bœsch
On Wed, Sep 06, 2017 at 10:05:40PM +0200, Thilo Borgmann wrote: > Hi, > > Am 06.09.17 um 11:58 schrieb Clément Bœsch: > > From: Clément Bœsch > > > > $headers is a variable set in the context of other functions (we don't > > use the "local" keyword in our scripts, so those variables are global).

Re: [FFmpeg-devel] [PATCH 3/3] lavu/timer.h: add Linux Perf API support

2017-09-07 Thread Clément Bœsch
On Sun, Sep 03, 2017 at 08:30:24PM +0200, Michael Niedermayer wrote: > On Sat, Sep 02, 2017 at 08:17:40PM +0200, Clément Bœsch wrote: > > From: Clément Bœsch > > > > Refer to "checkasm: use perf API on Linux ARM*" commit for the > > rationale. > > > > The implementation is somehow duplicated wit

Re: [FFmpeg-devel] [PATCH] libavcodec/h264_parse: don't use uninitialized value when chroma_format_idc==0

2017-09-07 Thread Mark Wachsler
I have updated my patch to also fix the references to chroma_weight in the MBAFF case, as suggested by Mark Thompson, but without erroring out on invalid values. On Thu, Sep 7, 2017 at 9:42 AM, Mark Wachsler wrote: > When parsing a monochrome file, chroma_log2_weight_denom was used without > bei

[FFmpeg-devel] [PATCH] libavcodec/h264_parse: don't use uninitialized value when chroma_format_idc==0

2017-09-07 Thread Mark Wachsler
When parsing a monochrome file, chroma_log2_weight_denom was used without being initialized, which could lead to a bogus error message being printed, e.g. [h264 @ 0x61a26480] chroma_log2_weight_denom 24576 is out of range It also could led to warnings using AddressSanitizer. --- libavcodec/h

Re: [FFmpeg-devel] [PATCH]lavf/gdv: Improve palette saturation

2017-09-07 Thread Ronald S. Bultje
Hi Carl, On Mon, Aug 28, 2017 at 10:49 AM, Carl Eugen Hoyos wrote: > 2017-08-27 9:03 GMT+02:00 Paul B Mahol : > > On 8/26/17, Carl Eugen Hoyos wrote: > > >> Attached patch slightly improves the saturation of the > >> gdv palette. > >> > >> Please comment, Carl Eugen > > > > Does this match how

[FFmpeg-devel] [PATCH]lavf/supenc: Add a raw PGS muxer

2017-09-07 Thread Carl Eugen Hoyos
Hi! Attached patch implements ticket #2208. Please comment, Carl Eugen From 919cd9728e73a9570a11f0d575b23c0212ca52e4 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Thu, 7 Sep 2017 15:28:55 +0200 Subject: [PATCH] lavf/supenc: Add a raw PGS muxer. Fixes ticket #2208. --- Changelog

Re: [FFmpeg-devel] [PATCH]lavf/gdv: Improve palette saturation

2017-09-07 Thread Carl Eugen Hoyos
2017-08-26 13:00 GMT+02:00 Carl Eugen Hoyos : > Hi! > > Attached patch slightly improves the saturation of the gdv palette. I'll push this if there are no objections. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mai

[FFmpeg-devel] rtmps connect block

2017-09-07 Thread Qiming Jin (qimijin)
Hi all Please help check an issue: When I sent date to rtmps server(facebook live). Connection will block after sent 10 packet. Below is the situation: 1 build: ./configure –enable-openssl && make && make install 2 connet to rtmp is ok. But rtmps is fail, BLOCK after sent out 10 packet. 3 if dele

[FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-07 Thread Gildas Fargeas
Ok, I changed the texi, added the bitrate and turned rgb48 value to rgb10. --- doc/indevs.texi | 22 ++-- libavdevice/decklink_common.cpp | 2 +- libavdevice/decklink_common_c.h | 1 + libavdevice/decklink_dec.cpp| 46 ++--

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-07 Thread Gildas Fargeas
Sorry I completely forgot about the texi doc ! I agree with you regarding rgb48. It feels misleading. r210 is fine with me. I'll send the changes in a few minutes. 2017-09-06 18:47 GMT+02:00 Marton Balint : > > > On Wed, 6 Sep 2017, Gildas Fargeas wrote: > > Alright, this should do the trick: >>

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve vp9 mc msa functions

2017-09-07 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Monday, September 4, 2017 6:03 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve vp9 mc msa functions Fro

Re: [FFmpeg-devel] [PATCH v2] ffprobe: use consistent string for unspecified color_range value

2017-09-07 Thread Tobias Rapp
On 05.09.2017 15:28, Paul B Mahol wrote: On 9/5/17, Tobias Rapp wrote: On 29.08.2017 16:07, Tobias Rapp wrote: Makes the handling of unspecified/unknown color_range values on stream level consistent to the value used on frame level. Signed-off-by: Tobias Rapp --- ffprobe.c

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: Fix DoS in read_tfra()

2017-09-07 Thread Michael Niedermayer
On Tue, Sep 05, 2017 at 02:55:25AM +0200, Michael Niedermayer wrote: > Fixes: Missing EOF check in loop > No testcase > > Found-by: Xiaohei and Wangchu from Alibaba Security Team > Signed-off-by: Michael Niedermayer > --- > libavformat/mov.c | 7 +++ > 1 file changed, 7 insertions(+) patchs

Re: [FFmpeg-devel] [PATCH] avformat/gdv: Make FixedSize static

2017-09-07 Thread Michael Niedermayer
On Mon, Sep 04, 2017 at 07:36:09PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/gdv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many that live d

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: migrate to new encode API

2017-09-07 Thread Timo Rothenpieler
Am 06.09.2017 um 23:07 schrieb Philip Langdale: On Wed, 6 Sep 2017 18:14:17 +0200 Timo Rothenpieler wrote: +int ff_nvenc_receive_packet(AVCodecContext *avctx, AVPacket *pkt) +{ +CUresult cu_res; +CUcontext dummy; +NvencSurface *tmpoutsurf; You can remove tmpoutsurf in ff_nvenc_se

Re: [FFmpeg-devel] [PATCH 8/8] doc: update filter_design.txt.

2017-09-07 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Reto Kromer a écrit : > tiny typo: first Thanks, locally fixed. Also fixed inut -> input in the same paragraph. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing li

Re: [FFmpeg-devel] [PATCH 8/8] doc: update filter_design.txt.

2017-09-07 Thread Reto Kromer
Nicolas George wrote: >Signed-off-by: Nicolas George >--- > doc/filter_design.txt | 251 >+++--- > 1 file changed, 135 insertions(+), 116 deletions(-) > >diff --git a/doc/filter_design.txt b/doc/filter_design.txt >index e8a7c53ee9..90fa53367b 100644 >--

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve vp9 idct msa functions

2017-09-07 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Monday, September 4, 2017 6:02 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve vp9 idct msa functions F

[FFmpeg-devel] [PATCH 4/5] lavfi/buffersrc: add av_buffersrc_close().

2017-09-07 Thread Nicolas George
TODO APIChanges and minor bump. Signed-off-by: Nicolas George --- libavfilter/buffersrc.c | 22 -- libavfilter/buffersrc.h | 8 2 files changed, 20 insertions(+), 10 deletions(-) diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c index e8f59c2de7..ad5ae

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve vp9 lpf msa functions

2017-09-07 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Monday, September 4, 2017 6:02 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve vp9 lpf msa functions Fr

[FFmpeg-devel] [PATCH 3/5] ffmpeg: use reordered duration for stream PTS.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- ffmpeg.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index c8ee64621c..b95addd277 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -2368,7 +2368,7 @@ static int decode_audio(InputStream *ist, AVPacket *pkt, int *got_out

[FFmpeg-devel] [PATCH 5/5] ffmpeg: send EOF pts to filters.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- ffmpeg.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index b95addd277..1d248bc269 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -2223,14 +2223,14 @@ static int ifilter_send_frame(InputFilter *ifilter, AVFrame *fra

[FFmpeg-devel] [PATCH 2/5] ffmpeg: rename a variable.

2017-09-07 Thread Nicolas George
Makes the reason of the "FIXME" comment more obvious. Avoid name conflicts for the next commit. Signed-off-by: Nicolas George --- ffmpeg.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) This patch and the following have already been discussed. The changes are tested and wor

[FFmpeg-devel] [PATCH 1/5] lavfi: guess a timestamp for compat status change.

2017-09-07 Thread Nicolas George
Use the earliest input with the same status. If that fails, print a warning and use the earliest source. With this change, simple filter forward correctly the timestamp of EOF. Filters that are supposed to change it should be updated to actually forward it. Signed-off-by: Nicolas George --- liba

[FFmpeg-devel] [PATCH 8/8] doc: update filter_design.txt.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/filter_design.txt | 251 +++--- 1 file changed, 135 insertions(+), 116 deletions(-) diff --git a/doc/filter_design.txt b/doc/filter_design.txt index e8a7c53ee9..90fa53367b 100644 --- a/doc/filter_design.txt +++ b/d

[FFmpeg-devel] [PATCH 6/8] lavfi: remove framesync.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/framesync.c | 343 libavfilter/framesync.h | 297 - 2 files changed, 640 deletions(-) delete mode 100644 libavfilter/framesync.c delete mode 100644 libavfilter/

[FFmpeg-devel] [PATCH 7/8] lavfi: rename framesync2 to framesync.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/Makefile | 48 +++ libavfilter/f_streamselect.c | 12 libavfilter/{framesync2.c => framesync.c} | 32 ++--- libavfilter/{framesync2.h => framesync.h} | 42 +

[FFmpeg-devel] [PATCH 4/8] lavfi/vf_zoompan: fix scheduling logic.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/vf_zoompan.c | 36 +--- 1 file changed, 9 insertions(+), 27 deletions(-) I do not have the faintest idea what the output of this filter is supposed to look like, so testing is limited. But at least now it outputs some

[FFmpeg-devel] [PATCH 5/8] lavfi/vf_zoompan: make some function arguments simple.

2017-09-07 Thread Nicolas George
The modified value is never used after the function call. Signed-off-by: Nicolas George --- libavfilter/vf_zoompan.c | 39 +++ 1 file changed, 19 insertions(+), 20 deletions(-) diff --git a/libavfilter/vf_zoompan.c b/libavfilter/vf_zoompan.c index b1ade33b1f.

[FFmpeg-devel] [PATCH 3/8] lavfi/af_sidechaincompress: use helper macros.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/af_sidechaincompress.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/libavfilter/af_sidechaincompress.c b/libavfilter/af_sidechaincompress.c index f174b70a5b..55bed43844 100644 --- a/libavfilter/af_sidechaincomp

[FFmpeg-devel] [PATCH 2/8] lavfi/af_agate: use helper macros.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/af_agate.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/libavfilter/af_agate.c b/libavfilter/af_agate.c index f4fcabef93..20905ccb19 100644 --- a/libavfilter/af_agate.c +++ b/libavfilter/af_agate.c @@ -270,10 +2

[FFmpeg-devel] [PATCH 1/8] lavfi: add helper functions and macros for activate.

2017-09-07 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/avfilter.c | 20 + libavfilter/filters.h | 78 ++ 2 files changed, 98 insertions(+) diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c index 6a97456054..91939eab8a 100644 --- a/li