Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

That would probably be considered a broken WebVTT file, since "&" need to be encoded as "". On 8 October 2015 at 20:46, Clément Bœsch <u...@pkh.me> wrote: > On Thu, Oct 08, 2015 at 05:20:52PM +0100, Ricardo Constantino wrote: > > Also fixes a

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

Even if not valid WebVTT, it should now work with something like "Ben". Sample: http://trac.ffmpeg.org/attachment/ticket/4915/htmlescapes.vtt Only issue left is CR-only endings not working, but since Apple stopped using that, it's probably not that important.

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

015 at 10:50:49PM +0100, Ricardo wrote: > > That would probably be considered a broken WebVTT file, since "&" need to > > be encoded as "". > > > > What about "Clment" or any unsupported escape? > > [...] > > -- > Clément

Re: [FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

Attached is the sample mentioned. WebVTT_extended_tester.vtt Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] Fwd: [PATCH] avfilter/vf_subtitles: allow setting fonts directory

This is mostly useful if libass was compiled with a font provider other than fontconfig, but is still useful in that case. Signed-off-by: Ricardo Constantino wiia...@gmail.com --- doc/filters.texi | 4 libavfilter/vf_subtitles.c | 4 2 files changed, 8 insertions(+) diff

Re: [FFmpeg-devel] Fwd: [PATCH] avfilter/vf_subtitles: allow setting fonts directory

On 1 September 2015 at 22:44, wm4 <nfx...@googlemail.com> wrote: > On Tue, 1 Sep 2015 23:20:27 +0200 > Michael Niedermayer <michae...@gmx.at> wrote: > > > On Mon, Aug 31, 2015 at 12:41:24AM +0100, Ricardo wrote: > > > This is mostly useful if libass was compi

[FFmpeg-devel] [PATCH] configure: Add option to use gmp

Enables RTMP(T)E support with SChannel (Not sure if this is the right way to enable this) Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- configure | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 9a736ce..6c658f2

[FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

Also fixes adjacent tags not being parsed correctly. Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavcodec/webvttdec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/libavcodec/webvttdec.c b/libavcodec/webvttdec.c index 1284a17..dec4105

[FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

Also fixes adjacent tags not being parsed correctly. Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavcodec/webvttdec.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/libavcodec/webvttdec.c b/libavcodec/webvttdec.c index 1284a17..a

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

On 11 October 2015 at 15:37, Clément Bœsch wrote: > nbsp → \h in ASS nevcairiel suggested not relying on \h being parsed as nbsp in .srt with VSFilter and Libass, but maybe using the unicode char itself. Unless FFmpeg converts \h to the char code in srtenc or something? Aegisub

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

Ok, nevermind. I see the srt and webvttenc samples use \h already, so that's what you'll want. On 11 October 2015 at 16:53, Ricardo Constantino <wiia...@gmail.com> wrote: > > On 11 October 2015 at 15:37, Clément Bœsch <u...@pkh.me> wrote: > >> nbsp → \h in ASS

Re: [FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

Attached is a new version of the sample with the missing non-breaking spaces tests. WebVTT_extended_tester.vtt Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

to validate WebVTT. Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- tests/fate/subtitles.mak | 3 +++ tests/ref/fate/sub-webvtt2 | 24 2 files changed, 27 insertions(+) create mode 100644 tests/ref/fate/sub-webvtt2 diff --git a/tests/fate/subtitles

[FFmpeg-devel] [PATCH 1/3] avcodec/webvttdec: Deal with WebVTT escapes

Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavcodec/webvttdec.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff

[FFmpeg-devel] [PATCH 2/3] avformat/webvttdec: Don't stop parsing on comments

Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavformat/webvttdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c index 43c2a63..47a3255 100644 --- a/libavformat/webvttdec.c +++ b/libavformat/webvt

Re: [FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

I had git misconfigured to convert CRLF to LF, so I'll probably reupload the patches (if it makes a difference?), but the sample should be OK as is. On 10 October 2015 at 01:26, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Fri, Oct 09, 2015 at 07:21:04PM +0100, Rica

[FFmpeg-devel] [PATCH 1/3] avcodec/webvttdec: Deal with WebVTT escapes

Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavcodec/webvttdec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff

[FFmpeg-devel] [PATCH 2/3] avformat/webvttdec: Don't stop parsing on comments

Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavformat/webvttdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c index 43c2a63..47a3255 100644 --- a/libavformat/webvttdec.c +++ b/libavformat/webvt

[FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

to validate WebVTT. Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- tests/fate/subtitles.mak | 3 +++ tests/ref/fate/sub-webvtt2 | 24 2 files changed, 27 insertions(+) create mode 100644 tests/ref/fate/sub-webvtt2 diff --git a/tests/fate/subtitles

[FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavcodec/webvttdec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff

[FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

Includes escapes that should now be supported and a few features not yet supported, like comments, regions, classes, ruby, lang. All were tested with https://quuz.org/webvtt/ for validation, except regions because the validator doesn't support them yet. Signed-off-by: Ricardo Constantino <w

[FFmpeg-devel] [PATCH] avformat/webvttdec: Don't stop parsing on comments

Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- libavformat/webvttdec.c| 3 ++- tests/ref/fate/sub-webvtt2 | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c index 43c2a63..47a3255 100644 --- a/libav

[FFmpeg-devel] [PATCH] configure: add libsoxr to swresample's pkgconfig

Fixes linking in FFMS and f265 at least, when ffmpeg is compiled with libsoxr. Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index e77c58d..d85ee5b 100755 --- a/configure

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

On 3 January 2016 at 23:13, Jan Ekstrom wrote: > Hi, > > In general looks good, although it might look a bit weird for someone > as usually libraries have initialization functions called like that. > That said, this is what > >

[FFmpeg-devel] [PATCH] configure: Use libgcrypt-config if available

Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- configure | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/configure b/configure index c986aba..d639b8d 100755 --- a/configure +++ b/configure @@ -5396,7 +5396,6 @@ enabled av

[FFmpeg-devel] [PATCH] configure: Add missing extralib for gcrypt

Signed-off-by: Ricardo Constantino <wiia...@gmail.com> --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index e3b7904..6d41343 100755 --- a/configure +++ b/configure @@ -5396,7 +5396,7 @@ enabled avisynth && { { check_l

Re: [FFmpeg-devel] [PATCH] configure: Add missing extralib for gcrypt

On 29 December 2015 at 22:17, Nicolas George wrote: > > This should be using libgcrypt-config, as per documentation: > > https://gnupg.org/documentation/manuals/gcrypt/Building-sources.html > > Otherwise, we have to duplicate the effort each time a dependency changes. > >

[FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

Trying to use gcrypt-enabled FFmpeg with rtmpe returns a few errors: Fatal: failed to create the RNG lock: Invalid argument FATAL: failed to acquire the FSM lock in libgrypt: Invalid argument This is probably not how or where this should be done, but it does fix the problem for me. You can test

Re: [FFmpeg-devel] [PATCH] configure: Use libgcrypt-config if available

On 8 January 2016 at 20:40, James Almer wrote: > Shouldn't this also check libgcrypt-config --cflags? Assuming > the library is in an unusual path, the above will look for > the header in whatever include path configure was using by > then and either fail or include the wrong

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

Ping ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

On 10 January 2016 at 00:17, Ricardo Constantino <wiia...@gmail.com> wrote: > On 9 January 2016 at 23:32, Michael Niedermayer <mich...@niedermayer.cc> > wrote: > >> >> we should pass the version number we need i think >> see >> >>

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

On 9 January 2016 at 23:32, Michael Niedermayer wrote: > > we should pass the version number we need i think > see > > https://quickgit.kde.org/?p=libktorrent.git=commit=a23bafe9191e0dcbd5ae75335fd5d8e49bfda9ed > > I wouldn't know the minimum version FFmpeg needs.

Re: [FFmpeg-devel] [PATCH] configure: Use libgcrypt-config if available

Like this? 0001-configure-Use-libgcrypt-config-s-cflags.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Include more developers in the voting committee [#3]

On 24 November 2015 at 23:40, compn wrote: > old developers have no right? is baptiste bad because he works on ffmbc? > is roberto bad because he works on mplayer? are you bad because you work > on mpv ? what makes a developer have rights or have no rights? > Most of those

Re: [FFmpeg-devel] [PATCH] configure: Add option to use gmp

I previously had an option for gcrypt too but it seemed to not work (FFmpeg crashed, iirc), so I just gave up on it. On 18 November 2015 at 05:41, Matt Oliver <protogo...@gmail.com> wrote: > On 15 November 2015 at 01:09, Ricardo Constantino <wiia...@gmail.com> > wrote: >

Re: [FFmpeg-devel] [PATCH] configure: Add user options to enable gcrypt/gmp for rtmp(t)e support.

>+ --enable-gcrypt enable gmp, needed for rtmp(t)e support >+ if openssl, librtmp or gmp is not used [no] Probably meant "enable gcrypt" there? On 21 November 2015 at 15:28, Matt Oliver wrote: > Adding rtmp(t)e support when not using

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

On 12 January 2016 at 05:05, Timothy Gu wrote: > Are you sure you want to initialize libgcrypt unconditionally as you are > doing here? > I don't think it's unconditional. It only tries to initialize gcrypt if it's not been initialized already. >+if

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

This patch seems to break build with decklink in MinGW: In file included from C:/builds/ab-full/build/ffmpeg-git/libavformat/internal.h:28:0, from C:/builds/ab-full/build/ffmpeg-git/libavdevice/decklink_common.cpp:34: C:/builds/ab-full/build/ffmpeg-git/libavformat/os_support.h:

Re: [FFmpeg-devel] [PATCH] configure: Use libgcrypt-config if available

Ping? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

Tried with 1.5.4 with the link on the first email and another one from Crunchyroll and it worked. Tried compiling 1.5.0 but didn't work (on MinGW). 0001-rtmpdh-Initialize-gcrypt-before-using-it.patch Description: Binary data ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it

On 10 January 2016 at 19:54, Michael Niedermayer wrote: > > please explain in the commit message why secmem is disabled > > 0001-rtmpdh-Initialize-gcrypt-before-using-it.patch Description: Binary data ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

On 6 April 2016 at 12:04, Rostislav Pehlivanov wrote: > On 6 April 2016 at 06:45, Clément Bœsch wrote: > >> >> Mmh. That's nice and all but... why not use/adjust the native ebur128 >> filter we have instead of relying on an external library? >> >> > What's worse

Re: [FFmpeg-devel] FFmpeg code Attribution

On 23 March 2016 at 22:35, Aaron Boxer wrote: > Back to my original point, what is the reasoning not to just switch to > OpenJPEG? Both OpenJPEG 1 and 2 are supported to add as external libraries in FFmpeg. What do you mean by switching to OpenJPEG?

Re: [FFmpeg-devel] [PATCH] configure: Don't require nonfree for nvenc

On 27 April 2016 at 13:50, Derek Buitenhuis wrote: > I presume there is no way to check the version? > configure already checks for minimum major version 6 which is the first one using MIT license and fails if lower. ___

Re: [FFmpeg-devel] [PATCH] lavc/libopenh264enc: update to openh264 1.6

On 2 April 2016 at 11:29, Stefano Sabatini wrote: > On date Monday 2016-03-07 18:05:30 +0100, Stefano Sabatini encoded: >> In particular, the slice mode API was changed in commit: >> >> commit 33c378f7b791310e4cb64b53e2bb8f3f3bded105 >> Author: sijchen >>

Re: [FFmpeg-devel] [PATCH] lavc/libopenh264enc: update to openh264 1.6

On 20 May 2016 at 11:37, Michael Niedermayer wrote: > as one testing ffmpeg with openh264 (building at least) > > i have mixed feelings about this, it would cause me to drop further > testing with openh264 in all releases OR in git master > because releases wont build with

Re: [FFmpeg-devel] how to use two rtmp plugins in ffmpeg

If librtmp is enabled FFmpeg will only use it. You can't select one or the other on runtime. On 28 June 2016 at 12:14, qw wrote: > Hi, > > ffmpeg has its own built-in rtmp plugin, and also support to use rtmpdump as > another rtmp plugin. > > How to use specified rtmp

Re: [FFmpeg-devel] [PATCH] lavc/libopenh264enc: update to openh264 1.6

On 23 May 2016 at 20:11, Michael Niedermayer wrote: > > it would be great to have support for both 1.6 and older versions > There's a patch by Martin Störjo in libav-devel that seems to allow support for both releases and git master. Tested just now in MinGW with git

Re: [FFmpeg-devel] [PATCH] configure/rtmpdh: Fix OpenSSL 1.1.0 support

On 22 February 2017 at 15:17, Stefan _ wrote: > 1) configure only looks for OPENSSL_init_ssl using pkg-config, this > breaks in case pkg-config is not available (cross-compiling) > > > 2) The rtmpdh code uses the DH struct from OpenSSL which was made > private in the 1.1 series >

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Remove pthread dependency

On 22 February 2017 at 22:55, Kyle Schwarz wrote: > DeckLink depends on pthread, and is silently disabled if pthread is > missing. > > Also fixes w32pthreads to support C++. > --- > compat/w32pthreads.h| 31 +++ > configure

Re: [FFmpeg-devel] Hardware Encode H264

Wrong mailing list. You're looking for http://ffmpeg.org/mailman/listinfo/ffmpeg-user. Also, your e-mail client sending HTML e-mails as plain text for some reason which makes them quite unreadable. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] add hds demuxer

On 2016-10-14 14:53, Steven Liu wrote: > @@ -295,6 +295,7 @@ External library support: > on OSX if openssl and gnutls are not used > [autodetect] >--enable-x11grab enable X11 grabbing (legacy) [no] >--disable-xlib disable xlib [autodetect] >

Re: [FFmpeg-devel] [PATCH] rtmpproto: send swfverify value as swfurl if latter is unused

Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] rtmpproto: send swfverify value as swfurl if latter is unused

On 27 March 2017 at 00:15, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Mon, Mar 20, 2017 at 08:22:51PM +, Ricardo Constantino wrote: > > Replicates lavf/librtmp.c behavior in L149-156 and rtmpdump's > > behavior with "--swfVfy " passing the ur

[FFmpeg-devel] [PATCH] rtmpproto: send swfverify value as swfurl if latter is unused

Replicates lavf/librtmp.c behavior in L149-156 and rtmpdump's behavior with "--swfVfy " passing the url to swfUrl. Fixes trac ticket #5549. --- libavformat/rtmpproto.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c

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

Also, 'manifest' seems to be typo'd through the patch to 'mainifest'. Not sure if intended. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

On 21 March 2017 at 22:50, Steven Liu wrote: > I don't know how to replay to an old thread with the patch by git > send-email :( > > Find the email you want to reply to in http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2017-March/thread.html and check the link in the

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

On 3 April 2017 at 10:56, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2017-04-01 16:37 GMT+02:00 Ricardo Constantino <wiia...@gmail.com>: > > On 1 April 2017 at 14:03, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > > > >> > >> This is the

[FFmpeg-devel] [PATCH] vf_pad: center image on padded area if negative x/y

or if x/y go beyond padded area. This is mostly useful when paired with the aspect option. Defaults aren't changed. Idea for this was taken from mpv's soon-to-be-removed expand vf. --- doc/filters.texi | 3 +++ libavfilter/vf_pad.c | 12 +++- 2 files changed, 10 insertions(+), 5

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

On 2 April 2017 at 22:13, Paul B Mahol wrote: > On 4/2/17, Paul B Mahol wrote: > > Signed-off-by: Paul B Mahol > > --- > > doc/filters.texi | 3 +++ > > libavfilter/vf_pad.c | 14 ++ > > 2 files changed, 17 insertions(+) >

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

On 3 April 2017 at 19:18, Paul B Mahol <one...@gmail.com> wrote: > On 4/3/17, Ricardo Constantino <wiia...@gmail.com> wrote: > > On 2 April 2017 at 22:13, Paul B Mahol <one...@gmail.com> wrote: > > > >> On 4/2/17, Paul B Mahol <one...@gmail.com>

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

On 1 April 2017 at 14:03, Hendrik Leppkes wrote: > > This is the Boost Software license, very similar to BSD/MIT, so if > that license is all that matters, its obviously fine. > > Just downloaded the latest 10.8.5 SDK and checked every file inside "Blackmagic DeckLink SDK

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

On 1 April 2017 at 00:20, Carl Eugen Hoyos wrote: > Hi! > > To the best of my knowledge, decklink is a non-free dependency. > > Please comment, Carl Eugen > > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org >

Re: [FFmpeg-devel] Added require fallback for libmfx in the case that pkg-config cannot find libmfx

On 15 April 2017 at 02:51, Aaron Levinson wrote: > From e0c73c054add0137901d0bf7a7893e42e7e566c8 Mon Sep 17 00:00:00 2001 > From: Aaron Levinson > Date: Fri, 14 Apr 2017 18:38:37 -0700 > Subject: [PATCH] Added require fallback for libmfx in the case

Re: [FFmpeg-devel] Added require fallback for libmfx in the case that pkg-config cannot find libmfx

On 15 April 2017 at 13:41, Aaron Levinson wrote: > Doesn't require() already do that? Here is the contents of require(): > > require(){ > log require "$@" > name_version="$1" > headers="$2" > func="$3" > shift 3 > check_lib "$headers" $func "$@" ||

Re: [FFmpeg-devel] [PATCH] rtmpproto: send swfverify value as swfurl if latter is unused

Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

On 3 April 2017 at 20:04, Paul B Mahol <one...@gmail.com> wrote: > On 4/3/17, Ricardo Constantino <wiia...@gmail.com> wrote: > > On 3 April 2017 at 19:58, Paul B Mahol <one...@gmail.com> wrote: > > > >> > >> Yes, but that conflict with expan

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

On 3 April 2017 at 19:58, Paul B Mahol wrote: > > Yes, but that conflict with expand behaviour. > > How so? Still works fine with squared pixels: $ ffmpeg -f lavfi -i "color=s=hd720" -vf pad=aspect=4/3 -vframes 1 -f null - -v verbose 2>&1 | grep Parsed_pad [Parsed_pad_0 @

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

On 2017-03-21 13:16, Steven Liu wrote: > @@ -5920,6 +5923,7 @@ enabled openssl && { use_pkg_config openssl > openssl/ssl.h OPENSSL_init > check_lib openssl/ssl.h SSL_library_init > -lssl -lcrypto -lws2_32 -lgdi32 || > die

Re: [FFmpeg-devel] [PATCH] configure: use pkg-config for libgme, if available

On 12 July 2017 at 08:43, wm4 <nfx...@googlemail.com> wrote: > On Tue, 11 Jul 2017 21:56:21 +0100 > Ricardo Constantino <wiia...@gmail.com> wrote: > >> On 6 July 2017 at 23:16, Ricardo Constantino <wiia...@gmail.com> wrote: >> > On 23 June 2017 a

Re: [FFmpeg-devel] [PATCH] configure: use pkg-config for libgme, if available

On 6 July 2017 at 23:16, Ricardo Constantino <wiia...@gmail.com> wrote: > On 23 June 2017 at 17:08, wm4 <nfx...@googlemail.com> wrote: >> On Fri, 23 Jun 2017 01:53:37 +0100 >> Ricardo Constantino <wiia...@gmail.com> wrote: >> >>> The pkg-config file

Re: [FFmpeg-devel] [PATCH] configure: use pkg-config for libgme, if available

On 23 June 2017 at 17:08, wm4 <nfx...@googlemail.com> wrote: > On Fri, 23 Jun 2017 01:53:37 +0100 > Ricardo Constantino <wiia...@gmail.com> wrote: > >> The pkg-config file is relatively new (2013), so some distros might >> not have it yet. And the -lstdc++

Re: [FFmpeg-devel] [PATCH] doc: Add initial documentation explaining undefined behavior and SUINT

On 15 July 2017 at 18:57, Michael Niedermayer wrote: > Requested-by: Kieran Kunhya > > Signed-off-by: Michael Niedermayer > --- > doc/undefined.txt | 47 +++ > 1 file changed, 47

Re: [FFmpeg-devel] SSL certificate for ffmpeg.org website is not valid anymore

On 18 July 2017 at 02:12, Gerion Entrup wrote: > Am Dienstag, 18. Juli 2017, 01:52:53 CEST schrieb Reimar Döffinger: >> On 18.07.2017, at 00:59, James Almer wrote: >> >> > On 7/17/2017 7:49 PM, Moritz Barsnick wrote: >> >> On Mon, Jul 10, 2017 at

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

On 30 June 2017 at 16:55, Paul B Mahol <one...@gmail.com> wrote: > On 6/28/17, Paul B Mahol <one...@gmail.com> wrote: >> On 6/28/17, Ricardo Constantino <wiia...@gmail.com> wrote: >>> On 24 June 2017 at 13:47, Ricardo Constantino <wiia...@gmail.com> w

[FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

libvorbis comes with pkg-config files since at least v1.0.1, way back in 2003. The extra check is needed for shared builds, as the pkg-config file for vorbisenc doesn't include vorbis and ogg if --static isn't used. The previous check could be also used as an extra fallback in case pkg-config

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

On 4 July 2017 at 07:54, Reimar Döffinger <reimar.doeffin...@gmx.de> wrote: > On 30.06.2017, at 20:06, Ricardo Constantino <wiia...@gmail.com> wrote: > >> libvorbis comes with pkg-config files since at least v1.0.1, way back >> in 2003. >> >> The extra che

[FFmpeg-devel] [PATCH v3] configure: require pkg-config for libvorbis

libvorbis comes with pkg-config files since at least v1.0.1, way back in 2003. We need the two checks for vorbis and vorbisenc because we use functions from both and Xiph considers them separate libraries. The check is inverted (vorbis first then vorbisenc) because add_extralibs() prepends to

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

On 24 June 2017 at 13:47, Ricardo Constantino <wiia...@gmail.com> wrote: > libvorbis comes with pkg-config files since at least v1.0.1, way back > in 2003. > --- > The previous patches weren't parsed by patchwork, and it makes less > sense to have a fallback for libvorb

Re: [FFmpeg-devel] [PATCH] avfilter/af_pan: ignore named channels not in current layout

On 11 May 2017 at 00:01, Marton Balint <c...@passwd.hu> wrote: > > On Wed, 10 May 2017, Ricardo Constantino wrote: > > instead of erroring out. >> >> af pan=stereo|FL=FL|FR=FR|FC=FC would error because there's no >> Front Center in stereo layout. >> >&

Re: [FFmpeg-devel] [PATCH] avfilter/af_pan: ignore named channels not in current layout

On 11 May 2017 at 08:49, Nicolas George wrote: > Le duodi 22 floréal, an CCXXV, Marton Balint a écrit : > > You still have to use a different command line for stereo and 7.1, so I > > don't quite see how this simplifies your use case. Even if your use case > > were justified,

[FFmpeg-devel] [PATCH] avfilter/af_pan: ignore named channels not in current layout

instead of erroring out. af pan=stereo|FL=FL|FR=FR|FC=FC would error because there's no Front Center in stereo layout. This allows just changing the output channel layout and keeping the same channel configuration across more than one output. Example usecase * changing balance to the right-side

[FFmpeg-devel] Fwd: [PATCH] configure: use or require pkg-config for libvorbis

I wasn't sure if adding use_pkg_config or completely changing to pkg-config-only would be more appropriate so I submit two patches doing one or the other. Cleanest would be changing to require pkg-config, but I leave the decision to the committer, if either are accepted.

[FFmpeg-devel] [PATCH] configure: require pkg-config for libgme

--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index dd9608540e..08bd6c0ae8 100755 --- a/configure +++ b/configure @@ -5826,7 +5826,7 @@ enabled fontconfig&& enable libfontconfig enabled libfontconfig && require_pkg_config

[FFmpeg-devel] [PATCH] configure: use pkg-config for libgme, if available

The pkg-config file is relatively new (2013), so some distros might not have it yet. And the -lstdc++ being required for the static lib is only present since the last release in December 2016. --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure

[FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

libvorbis comes with pkg-config files since at least v1.0.1, way back in 2003. --- The previous patches weren't parsed by patchwork, and it makes less sense to have a fallback for libvorbis since the .pc file is available since 2003. configure | 2 +- 1 file changed, 1 insertion(+), 1

Re: [FFmpeg-devel] [PATCH] Fix static linking openssl library

On 10 June 2017 at 09:13, wrote: > From: topilski > > --- > configure | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/configure b/configure > index 4ec8f21..c92dd1f 100755 > --- a/configure > +++ b/configure > @@ -5942,6 +5942,7 @@ enabled omx

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

I've actually tested the patch with FFmpeg/mpv now with a few .mpd from Youtube (non-segmented), Vimeo and Facebook and it seems to work mostly fine. You can obtain URLs to these by using youtube-dl json output together with jq cli tool, ex: working: ffmpeg -i $(youtube-dl -J

Re: [FFmpeg-devel] [PATCH] rtmpproto: send swfverify value as swfurl if latter is unused

Third ping. This patch is trivial, and makes behavior of librtmp and ffrtmp consistent when using rtmp_swfverify. What's blocking this? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] rtmpproto: send swfverify value as swfurl if latter is unused

On 6 May 2017 at 23:05, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Sat, May 06, 2017 at 01:13:41PM +0100, Ricardo Constantino wrote: > > Third ping. > > > > This patch is trivial, and makes behavior of librtmp and ffrtmp > consistent > > when

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

On 18 September 2017 at 14:27, Werner Robitza wrote: > Replaces French "Mo" with "Bytes". > > Signed-off-by: Werner Robitza > --- > doc/faq.texi | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/doc/faq.texi

Re: [FFmpeg-devel] [PATCH 2/2] opusenc: (WIP) add a new psychoacoustic system for the native Opus encoder

On 23 September 2017 at 00:46, Rostislav Pehlivanov wrote: > On 12 April 2017 at 23:26, Rostislav Pehlivanov > wrote: > > > > > > Here's the latest version, which I consider to be non-WIP now. > I plan to push it tomorrow morning if there are no

[FFmpeg-devel] [PATCH] compat/cuda/ptx2c: strip CR from each line

Windows nvcc + cl.exe produce a .ctx file with CR+LF newlines which need to be stripped to work with gcc. --- compat/cuda/ptx2c.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/compat/cuda/ptx2c.sh b/compat/cuda/ptx2c.sh index 1f37023290..44b08b6ea2 100755 ---

Re: [FFmpeg-devel] [PATCH] configure: add support for libnpp* from cuda sdk 9

On 29 August 2017 at 13:38, Timo Rothenpieler wrote: > Signed-off-by: Timo Rothenpieler > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 61d8160491..172ff3fc07 100755 > ---

Re: [FFmpeg-devel] [PATCH] lavc: drop support for OpenJPEG 1.3-2.0

On 19 October 2017 at 00:14, James Almer <jamr...@gmail.com> wrote: > On 10/18/2017 7:49 PM, Ricardo Constantino wrote: > > > There's a few other libs that require static-only Cflags (libxml2, at > > least), so openjpeg shouldn't be different. pkgconf does support > &

Re: [FFmpeg-devel] [PATCH] lavc: drop support for OpenJPEG 1.3-2.0

On 18 October 2017 at 23:22, James Almer wrote: > On 10/18/2017 6:56 PM, Michael Bradshaw wrote: > > -DOPJ_STATIC was originally added to ffmpeg's configure script for > Windows. > > Unconditionally adding -DOPJ_STATIC would conflict with people who are > > dynamically

[FFmpeg-devel] [PATCH] avdevice: remove dangling parenthesis from 2245476e5c45

--- libavdevice/decklink_dec.cpp| 2 +- libavdevice/libndi_newtek_dec.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index e786dd85dc..191547ff10 100644 --- a/libavdevice/decklink_dec.cpp +++

Re: [FFmpeg-devel] [PATCH] libavformat: add mbedTLS based TLS

On 26 May 2018 at 19:42, Reino Wijnsma wrote: > On 23-5-2018 2:11, Reino Wijnsma wrote: > > On 22-4-2018 14:47, Thomas Volkert wrote: > >> +enabled mbedtls && { check_pkg_config mbedtls mbedtls > mbedtls/x509_crt.h

[FFmpeg-devel] [PATCH] Remove battleforthenet widget

--- src/template_head2 | 23 --- 1 file changed, 23 deletions(-) diff --git a/src/template_head2 b/src/template_head2 index 71daf07..a0b11ab 100644 --- a/src/template_head2 +++ b/src/template_head2 @@ -3,29 +3,6 @@

Re: [FFmpeg-devel] [PATCH] configure: Change license required for NewTek SDK

On 14 February 2018 at 12:56, Tomas Härdin wrote: > On 2018-02-14 13:50, Kyle Schwarz wrote: > >> On Wed, Feb 14, 2018 at 7:45 AM, Hendrik Leppkes >> wrote: >> >>> On Wed, Feb 14, 2018 at 1:32 PM, Kyle Schwarz wrote: >>> On Wed,

  1   2   >