[FFmpeg-devel] [PATCH] lavf: add libopenmpt demuxer

TER_DEMUXER (LIBOPENMPT, libopenmpt); initialized = 1; } diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c new file mode 100644 index 000..8c95c0a --- /dev/null +++ b/libavformat/libopenmpt.c @@ -0,0 +1,185 @@ +/* + * Tracked MOD demuxer (libopenmpt) + * Copyrigh

[FFmpeg-devel] [PATCH] lavf: add libopenmpt demuxer

PT, libopenmpt); initialized = 1; } diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c new file mode 100644 index 000..b2dcb9b --- /dev/null +++ b/libavformat/libopenmpt.c @@ -0,0 +1,182 @@ +/* + * Tracked MOD demuxer (libopenmpt) + * Copyright (c) 2016 Josh de

[FFmpeg-devel] [PATCHv3] lavf: add libopenmpt demuxer

EGISTER_MUXDEMUX(LIBNUT, libnut); +REGISTER_DEMUXER (LIBOPENMPT, libopenmpt); initialized = 1; } diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c new file mode 100644 index 000..4db820d --- /dev/null +++ b/libavformat/libopenmpt.c @@ -0,0 +1,188 @@ +/* + * Tr

Re: [FFmpeg-devel] [PATCH] configure: only check dispatch header on darwin

On 25/02/2016 14:00, Matthieu Bouron wrote: From: Matthieu Bouron Fixes build of lavd/jack on linux if dispatch happens to be available on this platform. dispatch, as well as its dependencies kqueue and pwq are generally not installed / distribued on linux

Re: [FFmpeg-devel] [PATCH] configure: only check dispatch header on darwin

On 25/02/2016 15:44, Clément Bœsch wrote: On Thu, Feb 25, 2016 at 03:34:37PM +, Josh de Kock wrote: On 25/02/2016 14:00, Matthieu Bouron wrote: From: Matthieu Bouron <matthieu.bou...@stupeflix.com> Fixes build of lavd/jack on linux if dispatch happens to be available on this pl

Re: [FFmpeg-devel] [PATCH] configure: only check dispatch header on darwin

On 25/02/2016 15:53, Nicolas George wrote: Le septidi 7 ventôse, an CCXXIV, Josh de Kock a écrit : Just thought it wouldn't hurt to fix the real problem--that libdispatch isn't being checked for correctly, rather than a quick fix. Either is fine, I guess, although the quick fix would just

[FFmpeg-devel] [PATCH] Fixed issue #43 JACK indev support on OSX

--- configure | 14 +++--- libavdevice/jack.c | 12 2 files changed, 23 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 287896f..f2f7c3d 100755 --- a/configure +++ b/configure @@ -1732,6 +1732,7 @@ BUILTIN_LIST=" mm_empty rdtsc

[FFmpeg-devel] [PATCH] Fixed issue #43 JACK indev support on OSX

--- configure | 12 +--- libavdevice/jack.c | 10 ++ 2 files changed, 19 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 287896f..887384c 100755 --- a/configure +++ b/configure @@ -1732,6 +1732,7 @@ BUILTIN_LIST=" mm_empty rdtsc

Re: [FFmpeg-devel] [PATCH] Fixed issue #43 JACK indev support on OSX

--- configure | 13 ++--- libavdevice/jack.c | 10 ++ 2 files changed, 20 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 287896f..78f9618 100755 --- a/configure +++ b/configure @@ -1732,6 +1732,7 @@ BUILTIN_LIST=" mm_empty rdtsc

[FFmpeg-devel] [PATCHv4] Fixed issue #43 JACK indev support on OSX

--- configure | 9 +++-- libavdevice/jack.c | 10 ++ 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/configure b/configure index a78e228..6b3ee5f 100755 --- a/configure +++ b/configure @@ -1732,6 +1732,7 @@ BUILTIN_LIST=" mm_empty rdtsc

Re: [FFmpeg-devel] [PATCH] lavc/indeo2: Fix banding artifacts and washed-out display

On 16/03/2016 08:31, Mats Peterson wrote: I can't really use Barbato as the author, since I was the one who included his changes, so I'm the author. But perhaps there is some way to attribute him properly in spite of that? Patches' authors are generally based on who originally wrote the patch,

Re: [FFmpeg-devel] Support master branch of OpenJPEG and Grok J2K codecs

On 04/04/2016 14:21, Aaron Boxer wrote: > All of your arguments could be applied equally to GPL v3 components. And > yet, > FFmpeg is happy to allow distribution under GPL v3. So, I don't see what > the problem is here. The problem is AGPL can make certain builds of FFmpeg _unusable_, it's a pain

[FFmpeg-devel] [PATCH WIPv2 2/2] avdev: add sdl2 device

R_INOUTDEV(V4L2, v4l2); //REGISTER_INDEV (V4L, v4l diff --git a/libavdevice/sdl2.c b/libavdevice/sdl2.c new file mode 100644 index 000..8c3d49e --- /dev/null +++ b/libavdevice/sdl2.c @@ -0,0 +1,458 @@ +/* + * Copyright (c) 2011 Stefano Sabatini + * Copyright (c) 20

[FFmpeg-devel] [PATCH WIP 0/2] Update usage of SDL1 library to SDL2

Hi, I'm trying to update the usages of SDL1 to SDL2 so that they can follow the new library, as the last version was released in 2013, and is probably becoming a bit stale. While this should be a fairly simple patch (for others), I'm having a little trouble with it, and it was suggested that I

[FFmpeg-devel] [PATCH WIP 1/2] ffplay: convert ffplay to use SDL2

From: Marton Balint This patch looks mostly fine, there's just one thing though, which is the ALSA fix using envvars. There doesn't seem to be a replacement for these functions in SDL2, so I was wondering how else it could be done. Maybe through putenv()? --- configure | 27

[FFmpeg-devel] [PATCH WIP 2/2] avdev: add sdl2 device

dl2.c @@ -0,0 +1,437 @@ +/* + * Copyright (c) 2011 Stefano Sabatini + * Copyright (c) 2016 Josh de Kock + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Softwar

Re: [FFmpeg-devel] [PATCH WIP 1/2] ffplay: convert ffplay to use SDL2

On 17/05/2016 21:47, Marton Balint wrote: > > On Tue, 17 May 2016, Josh de Kock wrote: > >> From: Marton Balint <c...@passwd.hu> >> >> This patch looks mostly fine, there's just one thing though, which is >> the ALSA fix using envvars.

[FFmpeg-devel] [PATCH] avdev/jack: remove duplicated dispatch macros

The macros were moved to compat/dispatch_semaphore/semaphore.h after a libav merge, and were never removed from jack.c --- libavdevice/jack.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/libavdevice/jack.c b/libavdevice/jack.c index 34e2152..076078c 100644 ---

[FFmpeg-devel] [PATCHv6] lavf: add libopenmpt demuxer

DPLUG, libmodplug); REGISTER_MUXDEMUX(LIBNUT, libnut); +REGISTER_DEMUXER (LIBOPENMPT, libopenmpt); initialized = 1; } diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c new file mode 100644 index 0000000..fd38828 --- /dev/null +++ b/libavfo

Re: [FFmpeg-devel] [PATCH] avdev/libdc1394: distinguish between enumeration errors, and no cameras found

On Mon, Jul 25, 2016, at 10:23 AM, Michael Niedermayer wrote: > > applied > > will push once git.videolan.org is up again > > thx if you haven't pushed already, could you change the following: if (dc1394_camera_enumerate(dc1394->d, ) != DC1394_SUCCESS || !list){ should be: if

Re: [FFmpeg-devel] [PATCH 1/2] avformat: read id3v2 comment tags

On Sat, Jul 23, 2016, at 01:59 AM, kod...@gmail.com wrote: > [...] > > /** > + * Parse a comment tag. > + */ > +static void read_comm(AVFormatContext *s, AVIOContext *pb, int taglen, > + AVDictionary **metadata) This should probably be `read_comment`, it's not too long of a

[FFmpeg-devel] [PATCH] hapdec: remove unused memory.h include

Most systems have this, so it isn't really a problem to include it even if it's not used, but some do not have memory.h as it is non-standard. Since it's unused just remove it anyway. --- libavcodec/hapdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/hapdec.c

Re: [FFmpeg-devel] [PATCH] hapdec: remove unused memory.h include

On Fri, Jul 29, 2016, at 03:49 AM, Timothy Gu wrote: > Looks good to me. For what it's worth, the include was added in > commit 3ee217853a6741b829a2683f49c590618891b1ab, and looks like a stray > change. > > Timothy Thanks, applied. Josh ___

Re: [FFmpeg-devel] [PATCH 2/2] docs/demuxers: add libopenmpt section

On Tue, Jul 19, 2016, at 04:39 PM, Josh de Kock wrote: > [...] Set applied -- Josh ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] libavformat/libopenmpt: Add "date" to metadata.

The rest of the set was applied, thanks. -- Josh ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] MAINTAINERS: add myself for libopenmpt

Bœsch libnut.c Oded Shimon + libopenmpt.c Josh de Kock lmlm4.c Ivo van Poorten lvfdec.c Paul B Mahol lxfdec.c Tomas Härdin -- 2.7.4 (Apple Git

Re: [FFmpeg-devel] [PATCH 3/3] libavformat/libopenmpt: Add "date" to metadata.

On Sun, Jul 17, 2016, at 02:37 PM, Jörn Heusipp wrote: > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c > index 7b1dd5b..997c13d 100644 >

[FFmpeg-devel] [PATCH] libopenmpt: add subsong support

--- libavformat/libopenmpt.c | 8 1 file changed, 8 insertions(+) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 58a02be..4190510 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -36,6 +36,7 @@ typedef struct OpenMPTContext { /*

[FFmpeg-devel] [PATCHv2] libopenmpt: add subsong support

--- libavformat/libopenmpt.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 58a02be..286a908 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -36,6 +36,7 @@ typedef struct OpenMPTContext {

[FFmpeg-devel] [PATCH 2/2] docs/demuxers: add libopenmpt section

--- doc/demuxers.texi | 26 ++ 1 file changed, 26 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index e34f8b3..3646e29 100644 --- a/doc/demuxers.texi +++ b/doc/demuxers.texi @@ -267,6 +267,32 @@ track. Track indexes start at 0. The demuxer exports the

[FFmpeg-devel] [PATCH 1/2 v3] libopenmpt: add subsong support

--- libavformat/libopenmpt.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 58a02be..670866f 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -36,14 +36,19 @@ typedef

Re: [FFmpeg-devel] [PATCH 1/3] libopenmpt: change layout option to use string

On Tue, Jul 19, 2016, at 10:39 PM, Hendrik Leppkes wrote: > > We do have this option, so I would prefer if we keep using it. > Sure, that makes sense. I'll drop this patch from the set. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH 2/3 v3] libopenmpt: add subsong support

--- libavformat/libopenmpt.c | 26 +++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 17eb611..d52ebd6 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -43,9 +43,12 @@ typedef

[FFmpeg-devel] [PATCH 1/3] libopenmpt: change layout option to use string

Libav doesn't have AV_OPT_TYPE_CHANNEL_LAYOUT so use strings instead to maintain consistency. --- libavformat/libopenmpt.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 58a02be..17eb611 100644 ---

[FFmpeg-devel] [PATCH 3/3 v2] docs/demuxers: add libopenmpt section

--- doc/demuxers.texi | 27 +++ 1 file changed, 27 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index e34f8b3..69890c8 100644 --- a/doc/demuxers.texi +++ b/doc/demuxers.texi @@ -267,6 +267,33 @@ track. Track indexes start at 0. The demuxer exports the

[FFmpeg-devel] [PATCH 1/2 v4] libopenmpt: add subsong support

--- libavformat/libopenmpt.c | 30 +++--- 1 file changed, 27 insertions(+), 3 deletions(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index 58a02be..ac67d28 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -36,15 +36,19 @@

[FFmpeg-devel] [PATCH] aviobuf: cap len to orig_buffer_size

Note: I don't know much about the issue itself, just converting the suggested changes into patch form --- libavformat/aviobuf.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c index 31e7202..b0a6208 100644 ---

Re: [FFmpeg-devel] [PATCH 2/2] avformat/libmodplug: Remove code that has become redundant

On Sat, Jul 16, 2016, at 02:31 PM, Jörn Heusipp wrote: > [...] > Unless one of libopenmpt and libmodplug demuxers lowers its own file > extension probing score, this will not work, as they both handle pretty > much the same file formats. This could be solved by making the demuxers mutually

[FFmpeg-devel] [PATCH] lavf: remove libmodplug demuxer

replaced by the libopenmpt demuxer --- Changelog| 1 + configure| 4 - libavformat/Makefile | 1 - libavformat/allformats.c | 1 - libavformat/libmodplug.c | 382 --- 5 files changed, 1 insertion(+), 388

Re: [FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

Hi, On Thu, Jul 7, 2016, at 10:47 AM, Carl Eugen Hoyos wrote: > Josh de Kock itanimul.li> writes: > > > +ret = openmpt_could_open_propability( > > I don't know anything about openmpt but what was > wrong with the original probe function sent by you? It probed one f

[FFmpeg-devel] [PATCHv5] lavf: add libopenmpt demuxer

DPLUG, libmodplug); REGISTER_MUXDEMUX(LIBNUT, libnut); +REGISTER_DEMUXER (LIBOPENMPT, libopenmpt); initialized = 1; } diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c new file mode 100644 index 0000000..a237ad6 --- /dev/null +++ b/libavfo

[FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

DPLUG, libmodplug); REGISTER_MUXDEMUX(LIBNUT, libnut); +REGISTER_DEMUXER (LIBOPENMPT, libopenmpt); initialized = 1; } diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c new file mode 100644 index 0000000..c9e0a8e --- /dev/null +++ b/libavfo

[FFmpeg-devel] [PATCH] lavc/libzvbi: base support for multiple levels

Also add support for level 1.0 explicitly. Signed-off-by: Josh de Kock <j...@itanimul.li> --- libavcodec/libzvbi-teletextdec.c | 60 1 file changed, 42 insertions(+), 18 deletions(-) diff --git a/libavcodec/libzvbi-teletextdec.c b/libavcodec/l

[FFmpeg-devel] [PATCH] lavf/mpegts: document DVB teletext PMT fields

Signed-off-by: Josh de Kock <j...@itanimul.li> --- libavformat/mpegts.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 590abb0..bcf2118 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -1718,6 +1718,11

Re: [FFmpeg-devel] [PATCH] lavc/libzvbi: base support for multiple levels

On 18/02/2017 00:51, Marton Balint wrote: > > On Fri, 17 Feb 2017, Josh de Kock wrote: > >> Also add support for level 1.0 explicitly. >> > > What is the use case for this? Reducing the number of colors? > Pretty much. > I don't think you can reduce the n

[FFmpeg-devel] [PATCH] lavc/libzvbi: remove deprecated API usage

Signed-off-by: Josh de Kock <j...@itanimul.li> --- libavcodec/libzvbi-teletextdec.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavcodec/libzvbi-teletextdec.c b/libavcodec/libzvbi-teletextdec.c index d1f0a9f..2ed4a82 100644 --- a/libavcodec/l

[FFmpeg-devel] [PATCH v2] lavc/libzvbi: remove deprecated API usage

Hi Carl, I'm not sure, 0.2.28 doesn't compile on my system. It has been tested with the latest version 0.2.38. Does the updated patch look better? Josh Signed-off-by: Josh de Kock <j...@itanimul.li> --- libavcodec/libzvbi-teletextdec.c | 15 +-- 1 file changed, 13 insertions

Re: [FFmpeg-devel] [PATCH v2] lavc/libzvbi: remove deprecated API usage

-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 12/02/2017 15:28, Marton Balint wrote: > > On Sun, 12 Feb 2017, Josh de Kock wrote: > >> Hi Carl, >> >> I'm not sure, 0.2.28 doesn't compile on my system. It has been >> tested with the latest version 0.2.3

Re: [FFmpeg-devel] [RFC] marking non developer tickets

On 07/09/2016 18:48, Lou Logan wrote: If you want to use "task" for this then that would be fine with me. Alternatively, a wiki page could be setup similar to: Are all these on the trac? It might be worth adding a 'small' keyword

[FFmpeg-devel] [PATCH v4 1/3] lavd: Add SDL2 output device

Integrated comments from Moritz Barsnick. Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure| 25 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c

[FFmpeg-devel] [PATCH v5 1/3] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure| 25 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c | 373 +++ 4 files changed, 399 inse

[FFmpeg-devel] [PATCH v6 1/3] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure| 28 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c | 373 +++ 4 files changed, 402 inse

Re: [FFmpeg-devel] [PATCH] avdev: Add SDL2 output device

On 11/09/2016 19:04, Clément Bœsch wrote: On Sun, Sep 11, 2016 at 08:02:12PM +0200, Michael Niedermayer wrote: On Sun, Sep 11, 2016 at 04:38:12PM +0100, Josh de Kock wrote: This actually seems to work now, it didn't in the past, so maybe SDL2 was patched or something changed in FFmpeg. I

[FFmpeg-devel] [PATCH] avdev: Add SDL2 output device

This actually seems to work now, it didn't in the past, so maybe SDL2 was patched or something changed in FFmpeg. I will make an avfilter of this as well as the outdev. Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure| 24 ++- libavdevice/Makefile

[FFmpeg-devel] [PATCH v3 1/3] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure| 23 +++ libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c | 373 +++ 4 files changed, 398 inse

[FFmpeg-devel] [PATCH v3 3/3] lavd: deprecate opengl outdev

This device depends on SDL which is deprecated. Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog| 2 +- doc/outdevs.texi | 2 ++ libavdevice/opengl_enc.c | 2 ++ 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/Changelog b/Changelog index f

[FFmpeg-devel] [PATCH v3 2/3] lavd: deprecate SDL device

to switch to SDL2 as there'd no longer be a dependency on the SDL avdevice. Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog | 1 + doc/outdevs.texi | 2 ++ libavdevice/sdl.c | 2 ++ 3 files changed, 5 insertions(+) diff --git a/Changelog b/Changelog index 64695c8..f

Re: [FFmpeg-devel] [PATCH] avdev: Add SDL2 output device

On 11/09/2016 20:00, Moritz Barsnick wrote: Nice, seeing how often this has been pitched in the past. On Sun, Sep 11, 2016 at 16:38:12 +0100, Josh de Kock wrote: This actually seems to work now, it didn't in the past, so maybe SDL2 was patched or something changed in FFmpeg. Interesting

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

On Wed, Sep 14, 2016, at 10:12 AM, Michael Niedermayer wrote: > On Tue, Sep 13, 2016 at 01:45:12PM -0300, James Almer wrote: > > > > Tested on Windows (compiled with mingw-w64) and works fine. > > patchwork uses "^Tested-by:" to detect patches having been tested > should the regexp be

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

On 14/09/2016 16:34, Carl Eugen Hoyos wrote: [...] If a supported Linux distribution does not ship a sufficiently recent sdl2, I am not sure if this is such a good idea. Does the patch offer new functionality? It doesn't add any extra functionality, but SDL1 has been unmaintained for more than

Re: [FFmpeg-devel] [PATCH v6 1/3] lavd: Add SDL2 output device

On 14/09/2016 17:17, Carl Eugen Hoyos wrote: 2016-09-11 23:14 GMT+02:00 Josh de Kock <j...@itanimul.li>: +{ AV_PIX_FMT_RGB24, SDL_PIXELFORMAT_RGB888 }, +{ AV_PIX_FMT_BGR24, SDL_PIXELFORMAT_BGR888 }, Did you test them / why did you remove the comments that said they do no

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

On 14/09/2016 20:07, James Almer wrote: [...] I'm fine with that if others are (Especially Marton and Josh, since they develop/maintain ffplay). It's a good compromise. But much like ffserver it should similarly be marked as deprecated and essentially unmaintained somewhere. I'm fine with

Re: [FFmpeg-devel] [PATCH v3 3/3] lavd: deprecate opengl outdev

On 14/09/2016 18:53, Nicolas George wrote: Le sextidi 26 fructidor, an CCXXIV, Josh de Kock a écrit : This device depends on SDL which is deprecated. Signed-off-by: Josh de Kock <j...@itanimul.li> Although your port of the code to SDL2 address my concern about deprecating w

Re: [FFmpeg-devel] [PATCH] configure: don't build ffserver unless explicitly enabled

On 10/09/2016 22:50, Timo Rothenpieler wrote: On 9/10/2016 11:40 PM, Josh de Kock wrote: On 10/09/2016 22:25, Timo Rothenpieler wrote: [...] +DEPRECATED_PROGRAM_LIST=" +ffserver +" [...] I don't really see the point of this, the other programs are unlikely to be depre

Re: [FFmpeg-devel] [PATCH] configure: don't build ffserver unless explicitly enabled

On 10/09/2016 22:25, Timo Rothenpieler wrote: [...] +DEPRECATED_PROGRAM_LIST=" +ffserver +" > [...] I don't really see the point of this, the other programs are unlikely to be deprecated soon, and this list will be removed after ffserver is. I think it'd just be best to leave it in

[FFmpeg-devel] [PATCH v3 2/2] lavd: deprecate opengl outdev

This device depends on SDL which is deprecated. Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog| 2 +- doc/outdevs.texi | 2 ++ libavdevice/opengl_enc.c | 2 ++ 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/Changelog b/Changelog index f

[FFmpeg-devel] [PATCH] lavd: deprecate SDL device

to switch to SDL2 as there'd no longer be a dependency on the SDL avdevice. Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure | 3 +++ libavdevice/sdl.c | 2 ++ 2 files changed, 5 insertions(+) diff --git a/configure b/configure index b11ca7f..48cce34 100755 --- a/configure

[FFmpeg-devel] lavd API discussion

Hi all, So I know that not as many people use lavd, but I just wanted to ask a couple questions about the lavd/lavf APIs (but mostly about lavd), maybe fuel some thought about potentially developing/evolving the lavd API further. 1. What is the real difference between lavf and lavd? Both

[FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

From: Marton Balint <c...@passwd.hu> Depends on the other SDL patches. Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure | 6 +- ffplay.c | 594 ++ 2 files changed, 249 insertions(+), 351 deletions(-)

[FFmpeg-devel] [PATCH v2 2/4] ffplay: make copy for SDL1

Signed-off-by: Josh de Kock <j...@itanimul.li> --- Makefile | 1 + ffplay.c | 5 + ffplay.c => ffplay_sdl1.c | 0 3 files changed, 6 insertions(+) copy ffplay.c => ffplay_sdl1.c (100%) diff --git a/Makefile b/Makefile index 8aa72fd..15

[FFmpeg-devel] [PATCH v2 3/4] ffplay: add SDL2 support

From: Marton Balint <c...@passwd.hu> Tested-by: James Almer <jamr...@gmail.com> (Windows, mingw-w64) Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog | 1 + configure | 7 +- ffplay.c | 594 ++ 3

[FFmpeg-devel] [PATCH 3/4] ffplay: add SDL2 support

From: Marton Balint <c...@passwd.hu> Tested-by: James Almer <jamr...@gmail.com> (Windows, mingw-w64) Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure | 7 +- ffplay.c | 594 -- ffplay_sdl1.c | 1

[FFmpeg-devel] [PATCH 2/4] ffplay: make copy for SDL1

Signed-off-by: Josh de Kock <j...@itanimul.li> --- Makefile | 1 + ffplay.c | 5 + ffplay.c => ffplay_sdl1.c | 0 3 files changed, 6 insertions(+) copy ffplay.c => ffplay_sdl1.c (100%) diff --git a/Makefile b/Makefile index 8aa72fd..15

[FFmpeg-devel] [PATCH 1/4] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- configure| 28 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c | 377 +++ 4 files changed, 406 inse

[FFmpeg-devel] [PATCH 4/4] MAINTAINERS: update my entries

Signed-off-by: Josh de Kock <j...@itanimul.li> --- MAINTAINERS | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index f7e8298..52d8eed 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -24,7 +24,8 @@ ffmpeg: ff

[FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

Hi, Resending this set with ffplay now having two versions, a SDL2 and a SDL1 version. I've integrated all comments up until now (hopefully). Josh Josh de Kock (3): lavd: Add SDL2 output device ffplay: make copy for SDL1 MAINTAINERS: update my entries Marton Balint (1): ffplay: add

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

On 15/09/2016 23:00, Lukasz Marek wrote: On 15.09.2016 23:36, Josh de Kock wrote: On 15/09/2016 22:28, Lukasz Marek wrote: On 15.09.2016 23:19, Moritz Barsnick wrote: On Thu, Sep 15, 2016 at 14:36:32 -0300, James Almer wrote: * SDL1 is old and effectively unmaintained. I understand

Re: [FFmpeg-devel] [PATCH v3 2/2] lavd: deprecate opengl outdev

On 11/09/2016 08:36, Nicolas George wrote: Le sextidi 26 fructidor, an CCXXIV, Josh de Kock a écrit : This device depends on SDL which is deprecated. Are we in the business of deprecating things that do not have a replacement, now? Before deprecating the best video output device, making some

[FFmpeg-devel] [PATCH] news: add sdl entry

--- src/index | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/index b/src/index index f10f8a6..1d797fe 100644 --- a/src/index +++ b/src/index @@ -37,6 +37,13 @@ News + September 24th, 2016, SDL1 support dropped. + +Support for the SDL1 library has been dropped,

[FFmpeg-devel] [PATCH] lavd/sdl2: remove unused code

Signed-off-by: Josh de Kock <j...@itanimul.li> --- libavdevice/sdl2.c | 27 +-- 1 file changed, 5 insertions(+), 22 deletions(-) diff --git a/libavdevice/sdl2.c b/libavdevice/sdl2.c index e8252f0..217ccc0 100644 --- a/libavdevice/sdl2.c +++ b/libavdevice/sdl2.c @@

Re: [FFmpeg-devel] [PATCH 0/5] Add SDL2 support & drop SDL1 support

On 22/09/2016 20:28, Josh de Kock wrote: Hopefully the final iteration of this set--it drops SDL1 support entirely rather than adding another ffplay version and keeping the SDL1 output device. All of it has been reviewed before, so there shouldn't be too much (or anything to change). Josh de

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

On 29/09/2016 00:21, Stefano Sabatini wrote: On date Monday 2016-09-26 18:55:47 +0200, wm4 encoded: On Sun, 25 Sep 2016 19:32:37 +0200 Stefano Sabatini wrote: [...] My use case: I need to build a data stream with scripting/manual editing. Since I don't want to have to

Re: [FFmpeg-devel] [PATCH 0/4] V12 - SCTE-35 support

On 01/10/2016 18:27, Carlos Fernandez Sanz wrote: - Addresses new comments such as like idx value checking and filename checking Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 2 +

Re: [FFmpeg-devel] [PATCH v2 1/2] doc/developer: reword some of the policies

On 03/10/2016 00:05, Michael Niedermayer wrote: On Sun, Oct 02, 2016 at 11:16:49PM +0100, Josh de Kock wrote: On 02/10/2016 22:47, Michael Niedermayer wrote: On Sun, Oct 02, 2016 at 01:51:41AM +0100, Josh de Kock wrote: Explicitly state that FATE should pass, and code should work for all

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/ivi_dsp.c: fix warning due to misleading indentation

On 03/10/2016 15:40, Josh de Kock wrote: On 02/10/2016 19:46, Adriano Pallavicino wrote: LGTM. Will apply both patches squashed in a day if there are no further comments. Applied. -- Josh ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http

[FFmpeg-devel] [PATCH] doc/developer: remove duplicate policies and fix error

Fixes regression as of ee72b6d1 Signed-off-by: Josh de Kock <j...@itanimul.li> --- The irony of this patch is terrible, but oh well. Maybe we're missing a 'everyone makes mistakes' policy. doc/developer.texi | 18 ++ 1 file changed, 2 insertions(+), 16 deletions(-) diff

Re: [FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

On 14/09/2016 23:44, Thomas Volkert wrote: On 15.09.2016 00:27, Josh de Kock wrote: Hi, Resending this set with ffplay now having two versions, a SDL2 and a SDL1 version. I've integrated all comments up until now (hopefully). Josh Josh de Kock (3): lavd: Add SDL2 output device ffplay

[FFmpeg-devel] [PATCH] lavd/opengl: use SDL2

I did this in about 5 minutes, and only tested it with one sample (on OSX using the cocoa opengl renderer). Seems to work, but probably won't for all cases. Would like some feedback where it doesnt work etc Signed-off-by: Josh de Kock <j...@itanimul.li> --- libavdevice/opengl_enc.

Re: [FFmpeg-devel] [PATCH] lavd/opengl: use SDL2

On 18/09/2016 05:51, Lukasz Marek wrote: W dniu niedziela, 18 września 2016 Lou Logan <l...@lrcd.com <javascript:_e(%7B%7D,'cvml','l...@lrcd.com');>> napisał(a): On Sat, Sep 17, 2016, at 05:21 PM, Lukasz Marek wrote: On 18 September 2016 at 00:30, Josh de Kock <j...@ita

Re: [FFmpeg-devel] [PATCH] Add CONTRIBUTING.md

On 18/09/2016 01:29, James Almer wrote: On 9/17/2016 8:42 PM, Josh de Kock wrote: TimothyGu (https://github.com/FFmpeg/FFmpeg/pull/153#issuecomment-143274708) suggested doing this a while ago, but it was never done. As pull requests seem to be still coming through, I thought I'd send a patch

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

On 15/09/2016 22:28, Lukasz Marek wrote: On 15.09.2016 23:19, Moritz Barsnick wrote: On Thu, Sep 15, 2016 at 14:36:32 -0300, James Almer wrote: * SDL1 is old and effectively unmaintained. I understand this verbatim, but what is it supposed to mean? Has SDL1 gotten rotten? Or more precisely:

Re: [FFmpeg-devel] [PATCH v3 3/3] lavd: deprecate opengl outdev

On 15/09/2016 21:56, Lukasz Marek wrote: On 14.09.2016 19:53, Nicolas George wrote: Le sextidi 26 fructidor, an CCXXIV, Josh de Kock a écrit : This device depends on SDL which is deprecated. Signed-off-by: Josh de Kock <j...@itanimul.li> If I understand correctly, SDL is onl

Re: [FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

On 15/09/2016 11:11, Thomas Volkert wrote: [...] Yes, it is quite a bit of unnecessary overhead, but it's much cleaner than #ifdeffery in a single source file. In general, I don't like the idea of having 2 source files with almost the same content. This usually indicates to me that the code

Re: [FFmpeg-devel] [PATCH v2 2/4] ffplay: make copy for SDL1

On 15/09/2016 10:58, Carl Eugen Hoyos wrote: 2016-09-15 10:39 GMT+02:00 Josh de Kock <j...@itanimul.li>: Signed-off-by: Josh de Kock <j...@itanimul.li> --- Makefile | 1 + ffplay.c | 5 + ffplay.c => ffplay_sdl1.c | 0 3 files change

[FFmpeg-devel] [PATCH v2 1/4] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog| 1 + configure| 28 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c

Re: [FFmpeg-devel] Resurrection of ffserver

On 17/09/2016 22:22, Michael Fritscher wrote: Hello, The other programs will probably be exported elsewhere sometime in the future, for now since ffserver is getting removed having it put someplace else is a good starting point. My idea was to have the 2 repositories "ffmpeg-main" with the

[FFmpeg-devel] [PATCH] Add CONTRIBUTING.md

TimothyGu (https://github.com/FFmpeg/FFmpeg/pull/153#issuecomment-143274708) suggested doing this a while ago, but it was never done. As pull requests seem to be still coming through, I thought I'd send a patch to add the notice. Signed-off-by: Josh de Kock <j...@itanimul

Re: [FFmpeg-devel] [PATCH] lavd/opengl: use SDL2

From c8988099c8535c77382b6f05d23326a0270bb2f4 Mon Sep 17 00:00:00 2001 From: Lukasz Marek Date: Sun, 18 Sep 2016 19:13:12 +0200 Subject: [PATCH] lavd/opengl: use SDL2 Signed-off-by: Lukasz Marek --- libavdevice/opengl_enc.c | 109

[FFmpeg-devel] [PATCH v2 1/5] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog| 1 + configure| 28 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c

[FFmpeg-devel] [PATCH v3 1/5] lavd: Add SDL2 output device

Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock <j...@itanimul.li> --- Changelog| 1 + configure| 28 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c

  1   2   3   4   >