Re: [FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-17 Thread Clément Bœsch
On Mon, Sep 18, 2017 at 03:55:12AM +0200, Carl Eugen Hoyos wrote: > 2017-09-18 3:47 GMT+02:00 James Almer : > > On 9/17/2017 10:37 PM, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch fixes several warnings when compiling libavfilter with > >> current gcc. > >> > >>

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-15 Thread Clément Bœsch
On Fri, Sep 15, 2017 at 04:53:45PM +0200, Clément Bœsch wrote: > On Fri, Sep 15, 2017 at 11:15:14AM -0300, James Almer wrote: > > On 9/15/2017 10:15 AM, Clément Bœsch wrote: > > > On Fri, Sep 15, 2017 at 02:44:59AM +0200, Michael Niedermayer wrote: > > >> On Tue, Se

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-15 Thread Clément Bœsch
On Fri, Sep 15, 2017 at 11:15:14AM -0300, James Almer wrote: > On 9/15/2017 10:15 AM, Clément Bœsch wrote: > > On Fri, Sep 15, 2017 at 02:44:59AM +0200, Michael Niedermayer wrote: > >> On Tue, Sep 12, 2017 at 06:45:57PM -0700, Thierry Foucu wrote: > >>> Fix ticket #2

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-15 Thread Clément Bœsch
On Fri, Sep 15, 2017 at 02:44:59AM +0200, Michael Niedermayer wrote: > On Tue, Sep 12, 2017 at 06:45:57PM -0700, Thierry Foucu wrote: > > Fix ticket #2674 > > Tested with examples from ticket 2674. > > --- > > Sorry Michael, I forgot to configure using --enable-gpl. > > Please find new patch. > >

Re: [FFmpeg-devel] [PATCH] vp9: fix explicit memory order for report_progress.

2017-09-12 Thread Clément Bœsch
On Tue, Sep 12, 2017 at 03:16:43PM -0400, Ronald S. Bultje wrote: > --- > libavcodec/vp9.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > index 66ccb6c..6b5de19 100644 > --- a/libavcodec/vp9.c > +++ b/libavcodec/vp9.c > @@ -73,7

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

2017-09-08 Thread Clément Bœsch
On Thu, Sep 07, 2017 at 04:07:41PM +0200, Clément Bœsch wrote: [...] > > Patch 1-5 ok! > > For patch 6-7 maybe someone else can have a closer look. > > I'll wait a little more and push the patchset. > Patchset applied, thanks -- Clément B. signature.asc Descr

Re: [FFmpeg-devel] [PATCH 1/3] checkasm: use perf API on Linux ARM*

2017-09-08 Thread Clément Bœsch
On Sat, Sep 02, 2017 at 08:17:38PM +0200, Clément Bœsch wrote: [...] patchset applied -- Clément B. signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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 <cboe...@gopro.com> > > > > $headers is a variable set in the context of other functions (we don't > > use the &

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 <cboe...@gopro.com> > > > > Refer to "checkasm: use perf API on Linux ARM*" commit for the >

[FFmpeg-devel] [PATCH 5/7] build: add --disable-coreimage autodetect switch

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/configure b/configure index a4c2d81dee..1e36cde447 100755 --- a/configure +++ b/configure @@ -205,6 +205,7 @@ External library support: --disable-avfoun

[FFmpeg-devel] [PATCH 7/7] build: cleanup videotoolbox

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> - there is no need for kCVImageBufferColorPrimaries_ITU_R_2020 checks, it's done at runtime - VideoToolbox presence is now checked with check_apple_framework() - link to CoreServices is only done when videotoolbox is enabled --- configur

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

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> $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). Currently, when checking for AVFoundation/AVFoundation.h, the actual enabled header is math.h.

[FFmpeg-devel] [PATCH 4/7] build: add --disable-avfoundation autodetect switch

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 158f9ba24b..a4c2d81dee 100755 --- a/configure +++ b/configure @@ -202,6 +202,7 @@ External library support: them

[FFmpeg-devel] [PATCH 2/7] lavfi/coreimage: reduce dependency scope from QuartzCore to CoreImage

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> What is required by the filter is CoreImage, there is no QuartzCore usage. QuartzCore/CoreImage.h is simply an include to CoreImage/CoreImage.h. --- configure | 8 libavfilter/vf_coreimage.m | 2 +- 2 files changed, 5 inse

[FFmpeg-devel] [PATCH 3/7] build: add check_apple_framework()

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> This will be used in the following commits. --- configure | 9 + 1 file changed, 9 insertions(+) diff --git a/configure b/configure index 442ef606e0..158f9ba24b 100755 --- a/configure +++ b/configure @@ -1120,6 +1120,15 @@ check_header

[FFmpeg-devel] [PATCH 6/7] build: cleanup audiotoolbox handling

2017-09-06 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> *_at codecs only need the AudioToolbox framework, which is now checked like the other Apple frameworks. --- configure | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/configure b/configure index 1e36cde447..a91bc9f58a

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

2017-09-05 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> Currently, when checking for AVFoundation/AVFoundation.h, the actual enabled header is math.h. Similarly, when testing for QuartzCore/CoreImage.h, the actual enabled header is CoreGraphics/CoreGraphics.h. This is completely broken and may be the

[FFmpeg-devel] [PATCH 2/2] lavfi/coreimage: reduce dependency scope from QuartzCore to CoreImage

2017-09-05 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> What is required by the filter is CoreImage, there is no QuartzCore usage. QuartzCore/CoreImage.h is simply an include to CoreImage/CoreImage.h. --- configure | 8 libavfilter/vf_coreimage.m | 2 +- 2 files changed, 5 inse

[FFmpeg-devel] [PATCH] RFC: drop VDA

2017-09-05 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> I'm reworking the Apple Framework dependencies in the build system, and unfortunately I can't test VDA (because OSX doesn't have it anymore). I'll either break it badly without being able to fix it, or won't be able to do any the rework in the build

Re: [FFmpeg-devel] [PATCH] avcodec/snowdec: Check intra block dc differences.

2017-09-05 Thread Clément Bœsch
On Mon, Sep 04, 2017 at 08:35:17PM +0200, wm4 wrote: [...] > > > > Can't we just remove this codec? It has no use other than causing > > > > potential security issues and maintenance. > > > > > > I agree about removing the encoder, but the decoder is needed for > > > existing streams. Unless

Re: [FFmpeg-devel] [PATCH] lavd: drop QTKit indev

2017-09-04 Thread Clément Bœsch
On Mon, Sep 04, 2017 at 03:53:28PM +0200, Thilo Borgmann wrote: > Hi, > > Am 04.09.17 um 15:46 schrieb Clément Bœsch: > > From: Clément Bœsch <cboe...@gopro.com> > > > > QTKit has been deprecated in favor of AVFoundation for years, and we > > have an avfou

[FFmpeg-devel] [PATCH] lavd: drop QTKit indev

2017-09-04 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> QTKit has been deprecated in favor of AVFoundation for years, and we have an avfoundation input device. See https://developer.apple.com/documentation/qtkit --- Changelog| 1 + MAINTAINERS | 1 - con

Re: [FFmpeg-devel] [PATCH] lavd: drop disabled v4l code

2017-09-03 Thread Clément Bœsch
On Sat, Sep 02, 2017 at 07:32:17PM -0300, James Almer wrote: > On 9/2/2017 7:26 PM, Clément Bœsch wrote: > > This code is disabled since 2012. V4L1 was dropped from the kernel more > > than 10 years ago. > > --- > > configure| 2 - > >

[FFmpeg-devel] [PATCH] lavd: drop disabled v4l code

2017-09-02 Thread Clément Bœsch
This code is disabled since 2012. V4L1 was dropped from the kernel more than 10 years ago. --- configure| 2 - libavdevice/Makefile | 1 - libavdevice/alldevices.c | 1 - libavdevice/v4l.c| 364 --- 4 files changed, 368

Re: [FFmpeg-devel] [PATCH] cpu: add a function for querying maximum required data alignment

2017-09-02 Thread Clément Bœsch
On Sat, Sep 02, 2017 at 02:07:01PM -0300, James Almer wrote: [...] > +size_t av_cpu_max_align(void) > +{ > +int av_unused flags = av_get_cpu_flags(); > + > +#if ARCH_ARM || ARCH_AARCH64 > +if (flags & AV_CPU_FLAG_NEON) > +return 16; > +#elif ARCH_PPC > +if (flags &

[FFmpeg-devel] [PATCH 2/3] lavu/timer.h: factor out timer report

2017-09-02 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> This commit is meant to reduce the diff in the following one. --- libavutil/timer.h | 27 +++ 1 file changed, 15 insertions(+), 12 deletions(-) diff --git a/libavutil/timer.h b/libavutil/timer.h index ed3b047870..da0761b607

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

2017-09-02 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> Refer to "checkasm: use perf API on Linux ARM*" commit for the rationale. The implementation is somehow duplicated with checkasm, but so is the current usage of AV_READ_TIME(). Until these implementations and heuristics are made consist

[FFmpeg-devel] [PATCH 1/3] checkasm: use perf API on Linux ARM*

2017-09-02 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> On ARM platforms, accessing the PMU registers requires special user access permissions. Since there is no other way to get accurate timers, the current implementation of timers in FFmpeg is using these registers. Unfortunately, enabling user

Re: [FFmpeg-devel] Switching of automatically detected libraries [v2]

2017-09-02 Thread Clément Bœsch
On Sat, Sep 02, 2017 at 04:30:53PM +0200, Carl Eugen Hoyos wrote: > > > > Am 02.09.2017 um 16:17 schrieb Clément Bœsch <u...@pkh.me>: > > > Patchset applied, thanks > > Please write a news entry. > Why? default behaviour doesn't change. The new

Re: [FFmpeg-devel] Switching of automatically detected libraries [v2]

2017-09-02 Thread Clément Bœsch
On Sat, Sep 02, 2017 at 03:57:22PM +0300, Jan Ekstrom wrote: > Hi, > > On Wed, Aug 30, 2017 at 3:08 PM, Clément Bœsch <u...@pkh.me> wrote: > > ... > > Thanks to people who review the previous patchset and tested it. If > > someone wants to test without

Re: [FFmpeg-devel] [PATCH] avfilter/vf_subtitles: enable processing of alpha channel

2017-08-31 Thread Clément Bœsch
On Wed, Aug 30, 2017 at 09:04:59PM +0200, Paul B Mahol wrote: > Fixes #6605. > > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_subtitles.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c

Re: [FFmpeg-devel] Switching of automatically detected libraries [v2]

2017-08-30 Thread Clément Bœsch
On Wed, Aug 30, 2017 at 02:08:59PM +0200, Clément Bœsch wrote: > This patchset includes the following changes from previously: > > - The combination of --disable-autodetect and --enable- with > an automatically detected library now works > - Linking against libc's iconv i

[FFmpeg-devel] [PATCH 07/16] build: treat sdl2 like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 94b95f2df7..911b9ae569 100755 --- a/configure +++ b/configure @@ -6026,20 +6026,22 @@ if enabled gcrypt; then

[FFmpeg-devel] [PATCH 06/16] build: isolate sdl-to-sdl2 aliasing

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> This simplifies incoming SDL related changes by removing potential mismatching states of sdl and sdl2 variables. Since a component can have all kind of states (such as unset, enabled, disabled or requested), keeping these variables in sync ma

[FFmpeg-devel] [PATCH 13/16] build: make alsa part of the autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> alsa libs are already autodetected, this commit makes sure --disable-autodetect actually disable it unless --enable-alsa is specified. --- configure| 6 +++--- libavdevice/Makefile | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-)

[FFmpeg-devel] [PATCH 15/16] build: make sndio part of the autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> sndio is already autodetected, this commit makes sure --disable-autodetect actually disable it unless --enable-sndio is specified. --- configure| 5 +++-- libavdevice/Makefile | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff

[FFmpeg-devel] [PATCH 16/16] build: drop unused sndio_h and asoundlib_h

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 2 -- 1 file changed, 2 deletions(-) diff --git a/configure b/configure index f2af0711e3..6a50f40517 100755 --- a/configure +++ b/configure @@ -1854,7 +1854,6 @@ HAVE_LIST_PUB=" " HEADERS_LIST=" -alsa_asoun

[FFmpeg-devel] [PATCH 12/16] build: make sure a disabled autodetect still pick the libc's iconv

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/configure b/configure index a6bcee9466..0a961842df 100755 --- a/configure +++ b/configure @@ -3592,6 +3592,11 @@ for e in $env; do done if disabled auto

[FFmpeg-devel] [PATCH 14/16] build: make jack part of the autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> jack is already autodetected, this commit makes sure --disable-autodetect actually disable it unless --enable-jack is specified. --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index d378

[FFmpeg-devel] [PATCH 11/16] build: add --disable-autodetect switch

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- Changelog | 1 + configure | 6 ++ 2 files changed, 7 insertions(+) diff --git a/Changelog b/Changelog index b064328c3f..8b6057d6ae 100644 --- a/Changelog +++ b/Changelog @@ -38,6 +38,7 @@ version : - Some video filters with several inpu

[FFmpeg-devel] [PATCH 10/16] build: simplify weak-enabling of autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 18 +++--- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 7e80074686..e376e9562b 100755 --- a/configure +++ b/configure @@ -3593,21 +3593,9 @@ done for lib in $AUTODETECT_LI

[FFmpeg-devel] [PATCH 09/16] build: remove vda_framework from enable_weak

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> vda_framework is already pulled by the weakly enabled vda and videotoolbox. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 42742c5842..7e80074686 100755 --- a/configure +++ b/con

[FFmpeg-devel] [PATCH 05/16] build: treat securetransport and schannel like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/configure b/configure index e73773fb74..5d416d97f4 100755 --- a/configure +++ b/configure @@ -3606,6 +3606,7 @@ enable_weak cuda cuvid nvenc vda_fra

[FFmpeg-devel] [PATCH 04/16] build: treat libxcb like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 24 +--- 1 file changed, 5 insertions(+), 19 deletions(-) diff --git a/configure b/configure index b4b1288722..e73773fb74 100755 --- a/configure +++ b/configure @@ -3598,7 +3598,7 @@ enable_weak audiotoolbox #

[FFmpeg-devel] [PATCH 08/16] build: replace use of HAVE_SDL2 with existing CONFIG_SDL2

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> There is no need for duplication. --- configure| 1 - libavdevice/opengl_enc.c | 22 +++--- 2 files changed, 11 insertions(+), 12 deletions(-) diff --git a/configure b/configure index 911b9ae569..42742c5842

[FFmpeg-devel] [PATCH 03/16] build: treat iconv like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index a9932aeba6..b4b1288722 100755 --- a/configure +++ b/configure @@ -3605,6 +3605,8 @@ enable_weak cuda cuvid nvenc vda_framework videot

[FFmpeg-devel] Switching of automatically detected libraries [v2]

2017-08-30 Thread Clément Bœsch
This patchset includes the following changes from previously: - The combination of --disable-autodetect and --enable- with an automatically detected library now works - Linking against libc's iconv is now supported with --disable-autodetect (issue raised by Nicolas) - SDL detection should be

[FFmpeg-devel] [PATCH 02/16] build: treat crystalhd like other hwaccels

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 44e45791c2..a9932aeba6 100755 --- a/configure +++ b/configure @@ -1532,7 +1532,6 @@ EXTERNAL_LIBRAR

[FFmpeg-devel] [PATCH 01/16] build: group z libs with other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 61d8160491..44e45791c2 100755 --- a/configure +++ b/configure @@ -3602,6 +3602,9 @@ enable_weak xlib enable_weak cuda cuvid

Re: [FFmpeg-devel] Vignette filter limitations...

2017-08-27 Thread Clément Bœsch
On Sun, Aug 27, 2017 at 08:33:30AM +0100, Evert Vorster wrote: > Unfortunately I have a real problem with vignetting. > The vignette filter in ffmpeg seems to have some room for improvement. Yeah, definitely: the vignette filter was written with the fancy vignetting effect in mind (think of it as

Re: [FFmpeg-devel] [PATCH 01/16] doc/filters: document the unstability of the shorthand options notation.

2017-08-11 Thread Clément Bœsch
On Fri, Aug 11, 2017 at 12:33:25PM +0200, Nicolas George wrote: > Le quartidi 24 thermidor, an CCXXV, Clement Boesch a écrit : > > I'm afraid of the situation where a developer will feel like the order of > > the options is not ideal, or an option could be renamed for consistency > > with other

Re: [FFmpeg-devel] [PATCH] JPEG2000: SSE optimisation of DWT decoding

2017-08-11 Thread Clément Bœsch
On Fri, Aug 11, 2017 at 06:32:37PM +0300, Ivan Kalvachev wrote: > On 8/10/17, maxime taisant wrote: > >> From: Ivan Kalvachev > >> On 8/8/17, maxime taisant wrote: > >> > From: Maxime Taisant >

Re: [FFmpeg-devel] [PATCH 01/16] doc/filters: document the unstability of the shorthand options notation.

2017-08-11 Thread Clément Bœsch
On Fri, Aug 11, 2017 at 11:12:32AM +0200, Nicolas George wrote: > Le quartidi 24 thermidor, an CCXXV, Clement Boesch a écrit : > > I'd rather make such changes justified and documented as exceptional in > > the Changelog (or in APIchanges) when we can't get around it cleanly, than > > documenting

Re: [FFmpeg-devel] [PATCH 01/16] doc/filters: document the unstability of the shorthand options notation.

2017-08-11 Thread Clément Bœsch
On Thu, Aug 10, 2017 at 01:46:27PM +0200, Nicolas George wrote: > It does not mean that we intend to break the order of options > at a whim, but it gives us more freedom to make necessary > changes without extra unnecessary burden while giving stability > to users that require it. [...] I'd

Re: [FFmpeg-devel] [PATCH 11/14] lavfi/vf_paletteuse: convert to framesync2.

2017-08-10 Thread Clément Bœsch
On Thu, Aug 10, 2017 at 11:33:31AM +0200, Nicolas George wrote: > Le tridi 23 thermidor, an CCXXV, Clement Boesch a écrit : > > But unless it's API documented, that's implementation specific. I'd prefer > > if you keep that as a safe guard. It also has a documentation purpose. > > I will do it if

Re: [FFmpeg-devel] [PATCH 10/14] lavfi/vf_lut3d: convert to framesync2.

2017-08-10 Thread Clément Bœsch
On Mon, Jul 31, 2017 at 02:02:23PM +0200, Nicolas George wrote: [...] > static const AVOption haldclut_options[] = { > -{ "shortest", "force termination when the shortest input terminates", > OFFSET(dinput.shortest), AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1, FLAGS }, > -{ "repeatlast",

Re: [FFmpeg-devel] [PATCH 11/14] lavfi/vf_paletteuse: convert to framesync2.

2017-08-10 Thread Clément Bœsch
On Thu, Aug 10, 2017 at 09:13:13AM +0200, Nicolas George wrote: > Le tridi 23 thermidor, an CCXXV, Clement Boesch a écrit : > > > -.needs_writable = 1, // for error diffusal dithering > > why? > > It was not needed in the first place: IIRC, dualinput always returns a > writable main

Re: [FFmpeg-devel] [PATCH 11/14] lavfi/vf_paletteuse: convert to framesync2.

2017-08-09 Thread Clément Bœsch
On Mon, Jul 31, 2017 at 02:02:24PM +0200, Nicolas George wrote: [...] > @@ -1052,13 +1070,10 @@ static const AVFilterPad paletteuse_inputs[] = { > { > .name = "default", > .type = AVMEDIA_TYPE_VIDEO, > -.filter_frame = filter_frame, > -

Re: [FFmpeg-devel] [PATCH] JPEG2000: SSE optimisation of DWT decoding

2017-08-08 Thread Clément Bœsch
On Tue, Aug 08, 2017 at 09:09:44AM +, maxime taisant wrote: > From: Maxime Taisant > > Hi, > > Here is some SSE optimisations for the dwt function used to decode JPEG2000. > I tested this code by using the time command while reading a JPEG2000 encoded > video with

Re: [FFmpeg-devel] [PATCH 1/5] lavc/htmlsubtitles: improve handling broken garbage

2017-08-01 Thread Clément Bœsch
On Sat, Jul 29, 2017 at 09:27:47PM +0200, Clément Bœsch wrote: > This commit switches off forced correct nesting of tags and only keeps > it for font tags. See long explanations in the code for the rationale. > > This results in various FATE changes which I'll explain here: &g

Re: [FFmpeg-devel] [DISCUSSION] Motion Estimation API/Library

2017-08-01 Thread Clément Bœsch
On Tue, Aug 01, 2017 at 09:18:33AM +, Davinder Singh wrote: [...] > > In particular, the main policy of FFmpeg is to not depend on external > > libraries for core features. Therefore, if your project is a separate > > > > Just to be clear, it won't be "external" library like OpenCV... > > >

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/framesync2: add dualinput helper functions.

2017-07-30 Thread Clément Bœsch
On Sun, Jul 30, 2017 at 03:32:40PM +0200, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavfilter/framesync2.c | 78 > +--- > libavfilter/framesync2.h | 20 + > 2 files changed, 80 insertions(+), 18

Re: [FFmpeg-devel] [PATCH 1/2] fate: add fate-sws-pixdesc-query

2017-07-30 Thread Clément Bœsch
On Mon, Jul 24, 2017 at 08:02:25PM +0200, Carl Eugen Hoyos wrote: > 2017-07-24 17:03 GMT+02:00 Clément Bœsch <u...@pkh.me>: > > On Mon, Jul 24, 2017 at 01:55:40AM +0200, Carl Eugen Hoyos wrote: > >> 2017-03-19 15:00 GMT+01:00 Clément Bœsch <u...@pkh.me>: > &g

Re: [FFmpeg-devel] [PATCH 1/5] lavc/htmlsubtitles: improve handling broken garbage

2017-07-30 Thread Clément Bœsch
On Sun, Jul 30, 2017 at 04:34:16AM +0200, Michael Niedermayer wrote: [...] > > +struct font_tag stack[16] = {0}; > > this seems to produce a compiler warning: > > ./libavcodec/htmlsubtitles.c: In function ‘ff_htmlmarkup_to_ass’: > ./libavcodec/htmlsubtitles.c:112:12: warning: missing braces

[FFmpeg-devel] [PATCH 4/5] lavc/htmlsubtitles: handle colors starting with many '#'

2017-07-29 Thread Clément Bœsch
--- libavcodec/htmlsubtitles.c | 4 tests/ref/fate/sub-srt-badsyntax | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/htmlsubtitles.c b/libavcodec/htmlsubtitles.c index 29f763bee1..2b47c426ee 100644 --- a/libavcodec/htmlsubtitles.c +++

[FFmpeg-devel] [PATCH 3/5] lavc/htmlsubtitles: improve line breaks handling

2017-07-29 Thread Clément Bœsch
--- libavcodec/htmlsubtitles.c | 5 +++-- libavcodec/tests/htmlsubtitles.c| 2 ++ tests/ref/fate/libavcodec-htmlsubtitles | 1 + 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/libavcodec/htmlsubtitles.c b/libavcodec/htmlsubtitles.c index 69d855df21..29f763bee1

[FFmpeg-devel] [PATCH 5/5] lavc/htmlsubtitles: reindent after previous commits

2017-07-29 Thread Clément Bœsch
--- libavcodec/htmlsubtitles.c | 76 ++ 1 file changed, 36 insertions(+), 40 deletions(-) diff --git a/libavcodec/htmlsubtitles.c b/libavcodec/htmlsubtitles.c index 2b47c426ee..f4501816e5 100644 --- a/libavcodec/htmlsubtitles.c +++

[FFmpeg-devel] [PATCH 2/5] lavc/tests: add htmlsubtitles

2017-07-29 Thread Clément Bœsch
--- libavcodec/Makefile | 1 + libavcodec/tests/.gitignore | 1 + libavcodec/tests/htmlsubtitles.c| 49 + tests/fate/libavcodec.mak | 4 +++ tests/ref/fate/libavcodec-htmlsubtitles | 7 + 5 files

[FFmpeg-devel] [PATCH 1/5] lavc/htmlsubtitles: improve handling broken garbage

2017-07-29 Thread Clément Bœsch
/htmlsubtitles.c +++ b/libavcodec/htmlsubtitles.c @@ -1,5 +1,6 @@ /* * Copyright (c) 2010 Aurelien Jacobs <au...@gnuage.org> + * Copyright (c) 2017 Clément Bœsch <u...@pkh.me> * * This file is part of FFmpeg. * @@ -18,6 +19,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth F

Re: [FFmpeg-devel] [PATCH 8/8 v2] build: add --disable-autodetect switch

2017-07-28 Thread Clément Bœsch
On Fri, Jul 28, 2017 at 01:51:45PM +0200, Clément Bœsch wrote: > From: Clément Bœsch <cboe...@gopro.com> > > --- > Changelog | 1 + > configure | 6 -- > 2 files changed, 5 insertions(+), 2 deletions(-) > new version attached, fixing link problems. I still have

Re: [FFmpeg-devel] [PATCH 3/8] build: treat iconv like other autodetected libraries

2017-07-28 Thread Clément Bœsch
On Fri, Jul 28, 2017 at 01:57:53PM +0200, Nicolas George wrote: > Le decadi 10 thermidor, an CCXXV, Clement Boesch a écrit : > > From: Clément Bœsch <cboe...@gopro.com> > > > > --- > > configure | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) &

[FFmpeg-devel] [PATCH 8/8] build: add --disable-autodetect switch

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- Changelog | 1 + configure | 6 -- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/Changelog b/Changelog index 187ae7950a..50409118e6 100644 --- a/Changelog +++ b/Changelog @@ -29,6 +29,7 @@ version : - limiter video

[FFmpeg-devel] [PATCH 5/8] build: treat securetransport and schannel like other autodetected libraries

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/configure b/configure index d5e05a0eca..296e83926d 100755 --- a/configure +++ b/configure @@ -3597,6 +3597,7 @@ enable_weak cuda cuvid nvenc vda_fra

[FFmpeg-devel] [PATCH 7/8] build: simplify weak-enabling of autodetected libraries

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 18 +++--- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 6ddda239de..fa1b1d334a 100755 --- a/configure +++ b/configure @@ -3583,21 +3583,9 @@ done for lib in $AUTODETECT_LI

[FFmpeg-devel] [PATCH 6/8] build: remove vda_framework from enable_weak

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> vda_framework is already pulled by the weakly enabled vda and videotoolbox. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 296e83926d..6ddda239de 100755 --- a/configure +++ b/con

[FFmpeg-devel] [PATCH 4/8] build: treat libxcb like other autodetected libraries

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 24 +--- 1 file changed, 5 insertions(+), 19 deletions(-) diff --git a/configure b/configure index 5ba171d459..d5e05a0eca 100755 --- a/configure +++ b/configure @@ -3589,7 +3589,7 @@ enable_weak audiotoolbox #

[FFmpeg-devel] [PATCH 2/8] build: treat crystalhd like other hwaccels

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index a80f9cb2eb..a2ad72f7f4 100755 --- a/configure +++ b/configure @@ -1529,7 +1529,6 @@ EXTERNAL_LIBRAR

[FFmpeg-devel] [PATCH 3/8] build: treat iconv like other autodetected libraries

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index a2ad72f7f4..5ba171d459 100755 --- a/configure +++ b/configure @@ -3596,6 +3596,8 @@ enable_weak cuda cuvid nvenc vda_framework videot

[FFmpeg-devel] [PATCH 1/8] build: group z libs with other autodetected libraries

2017-07-28 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> --- configure | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 5811ee149a..a80f9cb2eb 100755 --- a/configure +++ b/configure @@ -3593,6 +3593,9 @@ enable_weak xlib enable_weak cuda cuvid

[FFmpeg-devel] Switching of automatically detected libraries

2017-07-28 Thread Clément Bœsch
Creating predictible builds is currentl difficult due to automatically detected libraries. This patchset adds an option to turn this off. As a side effect, it regroups all the automatically detected libraries in the same place. Note: crystalhd, securetransport, schannel and vda changes mostly

Re: [FFmpeg-devel] [PATCH] Add FITS Demuxer

2017-07-28 Thread Clément Bœsch
On Fri, Jul 28, 2017 at 12:32:59AM +0530, Paras Chadha wrote: [...] > +static int fits_probe(AVProbeData *p) > +{ > +const uint8_t *b = p->buf; > + > +if (AV_RB64(b) == 0x53494d504c452020 && > +AV_RB64(b + 8) == 0x3D20202020202020 && > +AV_RB64(b + 16) == 0x2020202020202020

Re: [FFmpeg-devel] [PATCH] doc/libav-merge: remove the hls merge TODO

2017-07-26 Thread Clément Bœsch
On Thu, Jul 27, 2017 at 11:45:32AM +0800, Steven Liu wrote: > 2017-07-25 11:42 GMT+08:00 Steven Liu : > > From: Steven Liu > > > > because the commit id 3c7fa8cbb93dba96acb12aef6e00d929c1da1ae8 > > have been done for OPENSSL at hlsenc. > > I don't

Re: [FFmpeg-devel] 3dlut filter: some .cube LUTs not working

2017-07-25 Thread Clément Bœsch
On Tue, Jul 25, 2017 at 09:58:39AM +0200, Felix Müller wrote: > The popular DELUTS SLOG2 set is not working with ffmpeg's 3dlut filter. > It simply states that this filter can not be used with it. > > I have attached one of those LUTs for testing purposes. > Works for me. You probably have the

Re: [FFmpeg-devel] [PATCH 1/2] fate: add fate-sws-pixdesc-query

2017-07-24 Thread Clément Bœsch
On Mon, Jul 24, 2017 at 08:02:25PM +0200, Carl Eugen Hoyos wrote: > 2017-07-24 17:03 GMT+02:00 Clément Bœsch <u...@pkh.me>: > > On Mon, Jul 24, 2017 at 01:55:40AM +0200, Carl Eugen Hoyos wrote: > >> 2017-03-19 15:00 GMT+01:00 Clément Bœsch <u...@pkh.me>: > &g

Re: [FFmpeg-devel] [PATCH 1/2] fate: add fate-sws-pixdesc-query

2017-07-24 Thread Clément Bœsch
On Mon, Jul 24, 2017 at 01:55:40AM +0200, Carl Eugen Hoyos wrote: > 2017-03-19 15:00 GMT+01:00 Clément Bœsch <u...@pkh.me>: > > Test the pixel format querying within libswscale. > > Afaict, this broke fate on big-endian: > $ make fate-sws-pixdesc-query > TESTsws-

Re: [FFmpeg-devel] [PATCH] movenc: support GPMF track (gpmd) remuxing

2017-07-24 Thread Clément Bœsch
On Fri, Jul 21, 2017 at 03:18:26PM +0200, Michael Niedermayer wrote: > On Fri, Jul 21, 2017 at 10:30:19AM +0200, Clément Bœsch wrote: > > From: Clément Bœsch <cboe...@gopro.com> > > > > See https://github.com/gopro/gpmf-parser for more information on

Re: [FFmpeg-devel] [PATCH] movenc: support GPMF track (gpmd) remuxing

2017-07-21 Thread Clément Bœsch
On Fri, Jul 21, 2017 at 04:09:29PM +0200, Clément Bœsch wrote: > On Fri, Jul 21, 2017 at 03:03:19PM +0100, Derek Buitenhuis wrote: > > On 7/21/2017 2:55 PM, Clément Bœsch wrote: > > > It's basically the same layout for every stsd sample description. Check > > >

Re: [FFmpeg-devel] [PATCH] movenc: support GPMF track (gpmd) remuxing

2017-07-21 Thread Clément Bœsch
On Fri, Jul 21, 2017 at 03:03:19PM +0100, Derek Buitenhuis wrote: > On 7/21/2017 2:55 PM, Clément Bœsch wrote: > > It's basically the same layout for every stsd sample description. Check > > the other tags that extend that layout, the also have that data ref idx to > > 1.

Re: [FFmpeg-devel] [PATCH] movenc: support GPMF track (gpmd) remuxing

2017-07-21 Thread Clément Bœsch
On Fri, Jul 21, 2017 at 02:21:23PM +0100, Derek Buitenhuis wrote: > On 7/21/2017 9:30 AM, Clément Bœsch wrote: > > +avio_wb16(pb, 1); /* Data-reference index */ > > Why is this whole atom hardcoded (i.e. with is '1'). > It's basically the same layout for every stsd sample

Re: [FFmpeg-devel] [PATCH] ffplay: terminate program on EOF.

2017-07-21 Thread Clément Bœsch
On Fri, Jul 21, 2017 at 02:26:03PM +0200, Jorge Ramirez-Ortiz wrote: > ffplay always requires user intervention via the GUI to close the > video at the end of the file. > > This commit, stops playing and terminates the program when EOF is > received. -autoexit? -- Clément B. signature.asc

Re: [FFmpeg-devel] [PATCH] fate: add vf_overlay test for main source with alpha channel

2017-07-21 Thread Clément Bœsch
On Thu, Jul 20, 2017 at 05:42:51PM +0200, Nicolas George wrote: [...] > diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c > index c4a5ae3742..f94c3ed9dd 100644 > --- a/libavfilter/vsrc_testsrc.c > +++ b/libavfilter/vsrc_testsrc.c > @@ -66,6 +66,9 @@ typedef struct

[FFmpeg-devel] [PATCH] movenc: support GPMF track (gpmd) remuxing

2017-07-21 Thread Clément Bœsch
From: Clément Bœsch <cboe...@gopro.com> See https://github.com/gopro/gpmf-parser for more information on the data stream itself. --- libavformat/movenc.c | 25 + 1 file changed, 25 insertions(+) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 3989

Re: [FFmpeg-devel] [PATCH 1/3] ffmpeg: add -(no)find_stream_info expert option

2017-07-18 Thread Clément Bœsch
On Wed, Jul 12, 2017 at 05:30:04PM +0200, Clément Bœsch wrote: > On Wed, Jul 12, 2017 at 05:14:14PM +0200, Nicolas George wrote: > > Le quartidi 24 messidor, an CCXXV, Clement Boesch a écrit : > > > --- > > > ffmpeg_opt.c | 22 +- > > >

Re: [FFmpeg-devel] [PATCH 1/3] ffmpeg: add -(no)find_stream_info expert option

2017-07-12 Thread Clément Bœsch
On Wed, Jul 12, 2017 at 05:14:14PM +0200, Nicolas George wrote: > Le quartidi 24 messidor, an CCXXV, Clement Boesch a écrit : > > --- > > ffmpeg_opt.c | 22 +- > > 1 file changed, 13 insertions(+), 9 deletions(-) > > > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > > index

[FFmpeg-devel] [PATCH 1/3] ffmpeg: add -(no)find_stream_info expert option

2017-07-12 Thread Clément Bœsch
--- ffmpeg_opt.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index 9b7e8c74b9..10ca260648 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -145,6 +145,7 @@ static int override_ffserver = 0; static int

[FFmpeg-devel] [PATCH 3/3] ffplay: add -(no)find_stream_info expert option

2017-07-12 Thread Clément Bœsch
--- ffplay.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/ffplay.c b/ffplay.c index c0b326c8cc..2eaeebc826 100644 --- a/ffplay.c +++ b/ffplay.c @@ -349,6 +349,7 @@ static int nb_vfilters = 0; static char *afilters = NULL; #endif static int autorotate = 1;

[FFmpeg-devel] [PATCH 2/3] ffprobe: add -(no)find_stream_info expert option

2017-07-12 Thread Clément Bœsch
--- ffprobe.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/ffprobe.c b/ffprobe.c index e295c53def..72a06d983b 100644 --- a/ffprobe.c +++ b/ffprobe.c @@ -130,6 +130,8 @@ typedef struct ReadInterval { static ReadInterval *read_intervals; static int

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

2017-07-09 Thread Clément Bœsch
On Sun, Jul 09, 2017 at 02:28:28PM +0200, Clément Bœsch wrote: [...] > 544 > 00:53:43,780 --> 00:53:45,941 > <<That's OK, >> - he calmed himself. > Side note here: "<<" and ">>" are used for « and ». It's important to not break this c

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

2017-07-09 Thread Clément Bœsch
On Sat, Jul 08, 2017 at 11:07:48PM +0200, Michael Niedermayer wrote: [...] > 5 days passed with no comments and no suggestions on how to implement > this better. > does anyone object to the fix as in the patch to be applied ? misc patterns I found you may want to give a try (note that these are

Re: [FFmpeg-devel] [PATCH] libavcodec/htmlsubtitles.c: make tags case-insensitive

2017-07-07 Thread Clément Bœsch
On Fri, Jun 16, 2017 at 08:26:37PM +0900, DongHoon Kang wrote: > Signed-off-by: DongHoon Kang > --- > libavcodec/htmlsubtitles.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > Rebased, updated for recent change, and applied. Sorry for the

<    1   2   3   4   5   6   7   8   9   10   >