Re: [FFmpeg-devel] [PATCH] avcodec/encode: remove usage of av_dup_packet()

2017-10-02 Thread wm4
On Sun, 1 Oct 2017 23:17:59 -0300 James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/encode.c | 20 > 1 file changed, 12 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/encode.c b/libavcodec/encode.c > index 525ee1f5d6..dd50486bcf 100644 > --- a/

Re: [FFmpeg-devel] [PATCH] build: don't strip binaries during compilation

2017-10-03 Thread wm4
On Mon, 2 Oct 2017 21:34:18 +0200 Michael Niedermayer wrote: > On Sun, Oct 01, 2017 at 07:55:29PM -0300, James Almer wrote: > > Do it during install instead, like with the libraries. > > > > There's no benefit making a stripped copy of the CLI tools in the > > build folder. Doing it during insta

Re: [FFmpeg-devel] [PATCH] avcodec/encode: don't return immediately on failure

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 01:55:44 -0300 James Almer wrote: > Fixes memleaks introduced by skipping cleanup at the end of the > functions. > > Regression since a22c6a4796ca1f2cbee6784262515da876fbec22. > > Signed-off-by: James Almer > --- > libavcodec/encode.c | 16 > 1 file change

[FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread wm4
dynamic loading of the cuda/cuvid libraries. (I wouldn't be able to test with the fixed SDK anyway, because installing the CUDA SDK on Linux is hell.) Signed-off-by: wm4 --- Changelog | 1 + configure | 9 +- fftools/ffmpeg.h| 1 + fftools/ffmpeg_

[FFmpeg-devel] [PATCH 3/7] decode: add a mechanism for performing delayed processing on the decoded frames

2017-10-03 Thread wm4
From: Anton Khirnov This will be useful in the CUVID hwaccel. Merges Libav commit badf0951f54c1332e77455dc40398f3512540c1b. --- libavcodec/decode.c | 11 +++ libavcodec/decode.h | 15 +++ 2 files changed, 26 insertions(+) diff --git a/libavcodec/decode.c b/libavcodec/decode

[FFmpeg-devel] [PATCH 0/7] Merge Libav cuvid hwaccel

2017-10-03 Thread wm4
delayed processing on the decoded frames decode: add a per-frame private data for hwaccel use h264dec: add a CUVID hwaccel wm4 (2): avcodec/cuvid: rename cuvid.c to cuviddec.c avcodec: allow multiple hwaccels for the same codec/pixfmt Changelog |1 + configure

[FFmpeg-devel] [PATCH 6/7] avcodec: allow multiple hwaccels for the same codec/pixfmt

2017-10-03 Thread wm4
Currently, AVHWAccels are looked up using a (codec_id, pixfmt) tuple. This means it's impossible to have 2 decoders for the same codec and using the same opaque hardware pixel format. This breaks merging Libav's CUVID hwaccel. FFmpeg has its own CUVID support, but it's a full stream decoder, using

[FFmpeg-devel] [PATCH 1/7] decode: avoid leaks on failure in ff_get_buffer()

2017-10-03 Thread wm4
From: Anton Khirnov If the get_buffer() call fails, the frame might have some side data already set. Make sure it gets freed. CC: libav-sta...@libav.org Merges Libav commit de77671438c24ffea93398c8dc885d4dd04477de. --- libavcodec/decode.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/

[FFmpeg-devel] [PATCH 5/7] avcodec/cuvid: rename cuvid.c to cuviddec.c

2017-10-03 Thread wm4
cuvid.c is used by Libav's CUVID hwaccel. Resolve the conflict and avoid future merge problems by renaming our decoder. --- libavcodec/Makefile| 10 +- libavcodec/{cuvid.c => cuviddec.c} | 0 2 files changed, 5 insertions(+), 5 deletions(-) rename libavcodec/{cuvid.c => c

[FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-03 Thread wm4
From: Anton Khirnov Use the AVFrame.opaque_ref field. The original user's opaque_ref is wrapped in the lavc struct and then unwrapped before the frame is returned to the caller. This new struct will be useful in the following commits. Merges Libav commit 359a8a3e2d1194b52b6c386f94fd0929567dfb67

[FFmpeg-devel] [PATCH 4/7] decode: add a per-frame private data for hwaccel use

2017-10-03 Thread wm4
From: Anton Khirnov This will be useful in the CUVID hwaccel. It should also eventually replace current decoder-specific mechanisms used by various other hwaccels. Merges Libav commit 704311b2946d74a80f65906961cd9baaa18683a3. --- libavcodec/decode.c | 3 +++ libavcodec/decode.h | 6 ++ 2 fi

Re: [FFmpeg-devel] [PATCH] build: don't strip binaries during compilation

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 10:23:08 -0300 James Almer wrote: > On 10/3/2017 5:18 AM, wm4 wrote: > > On Mon, 2 Oct 2017 21:34:18 +0200 > > Michael Niedermayer wrote: > > > >> On Sun, Oct 01, 2017 at 07:55:29PM -0300, James Almer wrote: > >>> Do it durin

Re: [FFmpeg-devel] [PATCH 6/7] avcodec: allow multiple hwaccels for the same codec/pixfmt

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 15:58:32 +0200 Timo Rothenpieler wrote: > Am 03.10.2017 um 15:15 schrieb wm4: > > Currently, AVHWAccels are looked up using a (codec_id, pixfmt) tuple. > > This means it's impossible to have 2 decoders for the same codec and > > using the same op

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 16:08:32 +0200 Timo Rothenpieler wrote: > I'm not a fan of there being cuvid and cuvid_hwaccel now, meaning > potentially multiple things. It seems super confusing to me. Yes, that's a pretty annoying situation. > I'd propose to use this as a chance to get in line with nvidi

Re: [FFmpeg-devel] [PATCH 6/7] avcodec: allow multiple hwaccels for the same codec/pixfmt

2017-10-03 Thread wm4
e actually wants to try the patches, here's an updated set: https://github.com/wm4/FFmpeg/tree/libav-cuvid ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] build: don't strip binaries during compilation

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 18:01:14 +0200 Michael Niedermayer wrote: > differently. Bugs in the past occuring with ffmpeg did sometimes not > reproduce with ffmpeg_g. But if one talked about "ffmpeg" or "ffmpeg_g" > together with its command line output which contains revission and > build flags it was c

Re: [FFmpeg-devel] [PATCH] build: don't strip binaries during compilation

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 13:27:21 -0300 James Almer wrote: > I don't really agree with this, seeing almost every other project out > there does pretty much what this patch is trying to achieve and don't > seem to have issues handling bug reports, but if both you and Carl > consider this behavior as imp

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-04 Thread wm4
On Tue, 3 Oct 2017 21:40:58 +0200 Michael Niedermayer wrote: > On Tue, Oct 03, 2017 at 03:15:13PM +0200, wm4 wrote: > > From: Anton Khirnov > > > > > Use the AVFrame.opaque_ref field. The original user's opaque_ref is > > wrapped in the lavc struct and

Re: [FFmpeg-devel] [PATCH 1/2] lavc/v4l2: Remove use of lfind()

2017-10-04 Thread wm4
On Tue, 3 Oct 2017 21:08:13 +0100 Mark Thompson wrote: > This is not present in older bionic and therefore fails to build there, > and the code is much simpler without it anyway. > --- +1 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ff

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-04 Thread wm4
On Tue, 3 Oct 2017 07:17:25 -0700 Philip Langdale wrote: > > I'd propose to use this as a chance to get in line with nvidias new > > naming, and call the new cuvid decoder/hwaccel nvdec. This is quite a > > deviation from libav, but we need to rename it anyways, so might as > > well pick an ent

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2: set sizeimage param for non-raw buffers [fixes #6716]

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 11:17:24 +0200 Jorge Ramirez-Ortiz wrote: > Some V4L2 drivers fail to allocate buffers when sizeimage is not set > to a max value. This is indeed the case for sp5-mfc [1] > > Most drivers should be able to calculate this value from the frame > dimensions and format - or at le

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 11:22:37 +0200 Michael Niedermayer wrote: > On Wed, Oct 04, 2017 at 09:12:29AM +0200, wm4 wrote: > > On Tue, 3 Oct 2017 21:40:58 +0200 > > Michael Niedermayer wrote: > > > > > On Tue, Oct 03, 2017 at 03:15:13PM +0200, wm4 wrote:

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 11:47:39 +0200 Michael Niedermayer wrote: > On Wed, Oct 04, 2017 at 11:34:18AM +0200, wm4 wrote: > > On Wed, 4 Oct 2017 11:22:37 +0200 > > Michael Niedermayer wrote: > > > > > On Wed, Oct 04, 2017 at 09:12:29AM +0200, wm4 wrote: > > &

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_context: Replace "0 in case of success" by "non negative in case of success"

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 12:51:42 +0200 Michael Niedermayer wrote: > This makes the return code consistent with most of FFmpeg > > Signed-off-by: Michael Niedermayer > --- That's a bit redundant because the FFmpeg convention is guaranteed to work on the caller site anyway. _

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 13:37:31 +0200 Tobias Rapp wrote: > On 04.10.2017 11:34, wm4 wrote: > > On Wed, 4 Oct 2017 11:22:37 +0200 > > Michael Niedermayer wrote: > > > >> On Wed, Oct 04, 2017 at 09:12:29AM +0200, wm4 wrote: > >>> On Tue, 3 Oct 2017 21:40

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2: set sizeimage param for non-raw buffers [fixes #6716]

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 17:48:25 +0200 Jorge Ramirez-Ortiz wrote: > On 10/04/2017 11:28 AM, Jorge Ramirez-Ortiz wrote: > > On 10/04/2017 11:23 AM, wm4 wrote: > >> On Wed,  4 Oct 2017 11:17:24 +0200 > >> Jorge Ramirez-Ortiz wrote: > >> > >>>

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2: set sizeimage param for non-raw buffers [fixes #6716]

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 18:13:24 +0200 Jorge Ramirez-Ortiz wrote: > On 10/04/2017 05:59 PM, wm4 wrote: > >>>>> diff --git a/libavcodec/v4l2_context.c b/libavcodec/v4l2_context.c > >>>>> index 297792f..2707ef5 100644 > >>>>> --- a/libavcod

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2: set sizeimage param for non-raw buffers [fixes #6716]

2017-10-04 Thread wm4
On Wed, 4 Oct 2017 19:03:12 +0200 Jorge Ramirez-Ortiz wrote: > On 10/04/2017 06:20 PM, wm4 wrote: > >>>>>> Isn't this something that should be fixed in the driver? > >>>>> yes but it might take forever and I dont know how ma

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-05 Thread wm4
On Thu, 5 Oct 2017 09:02:10 +0200 Carl Eugen Hoyos wrote: > 2017-10-04 14:04 GMT+02:00 wm4 : > > > if you care about what gets merged, discuss it with the > > original author on the libav-devel mailing list before it > > gets committed. > > You do realize tha

Re: [FFmpeg-devel] [PATCH] avformat: fix id3 chapters

2017-10-05 Thread wm4
On Thu, 5 Oct 2017 03:34:19 +0200 Lukas Stabe wrote: > These changes store id3 chapter data in ID3v2ExtraMeta and introduce > ff_id3v2_parse_chapters to parse them into the format context if needed. > > Encoders using ff_id3v2_read, which previously parsed chapters into the > format context a

Re: [FFmpeg-devel] [PATCH 3/3] avformat/hls: add http_keepalive option

2017-10-05 Thread wm4
On Wed, 4 Oct 2017 15:03:34 -0700 Aman Gupta wrote: > From: Aman Gupta > > This teaches the HLS demuxer to use the HTTP protocols > multiple_requests=1 option, to take advantage of "Connection: > Keep-Alive" when downloading playlists and segments from the HLS server. > > With the new option,

Re: [FFmpeg-devel] [PATCH 3/3] avformat/hls: add http_keepalive option

2017-10-05 Thread wm4
On Thu, 5 Oct 2017 16:39:25 +0200 Moritz Barsnick wrote: > On Thu, Oct 05, 2017 at 13:41:07 +0200, wm4 wrote: > > I don't really see where this patch checks whether the host changes? > > The minimal requirements for reusing the connection would be using > > the same

Re: [FFmpeg-devel] [PATCH] avformat: fix id3 chapters

2017-10-05 Thread wm4
On Thu, 5 Oct 2017 03:34:19 +0200 Lukas Stabe wrote: > These changes store id3 chapter data in ID3v2ExtraMeta and introduce > ff_id3v2_parse_chapters to parse them into the format context if needed. > > Encoders using ff_id3v2_read, which previously parsed chapters into the > format context a

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-05 Thread wm4
On Thu, 5 Oct 2017 18:47:01 +0200 Michael Niedermayer wrote: > On Wed, Oct 04, 2017 at 02:04:54PM +0200, wm4 wrote: > > On Wed, 4 Oct 2017 13:37:31 +0200 > > Tobias Rapp wrote: > > > > > On 04.10.2017 11:34, wm4 wrote: > > > > On Wed, 4 Oct 2017

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-05 Thread wm4
On Thu, 5 Oct 2017 22:01:56 +0200 Carl Eugen Hoyos wrote: > 2017-10-05 21:22 GMT+02:00 wm4 : > > >> And this is just one example ... > > > > Your one example was invalid, next one please. > > One example is more than enough, we can all see that this >

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-05 Thread wm4
On Fri, 6 Oct 2017 00:01:30 +0200 Michael Niedermayer wrote: > The opaque_ref wraping is a really bad design. Iam not sure why > people defend it. FFmpeg is full of this design. There are plenty of structs with opaque/priv fields that change meaning depending on the context (basically how the st

Re: [FFmpeg-devel] [PATCH 3/3] avformat/hls: add http_keepalive option

2017-10-05 Thread wm4
On Fri, 6 Oct 2017 00:19:33 +0200 Hendrik Leppkes wrote: > On Thu, Oct 5, 2017 at 10:21 PM, Carl Eugen Hoyos wrote: > > 2017-10-05 16:32 GMT+02:00 Moritz Barsnick : > > > >> Issue: The hls demuxer now inconsistently no longer reports > >> each opened URL, only the first one. (Reporting of thes

Re: [FFmpeg-devel] FFmpeg 3.4

2017-10-06 Thread wm4
On Fri, 6 Oct 2017 16:53:17 +0200 Michael Niedermayer wrote: > Hi all > > if there are no objections i will branch release/3.4 in the next days > and make the 3.4 release a few days after that > > If people prefer a specific name, suggest one now, otherwise i will > pick a random one from past

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

2017-10-06 Thread wm4
On Fri, 6 Oct 2017 18:02:44 -0300 James Almer wrote: > On 10/6/2017 5:44 PM, Paul B Mahol wrote: > > On 10/6/17, Michael Niedermayer wrote: > >> On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote: > >>> Hi, > >>> > >>> On Thu, Oct 5, 2017 at 7:52 PM, Michael Niedermayer > >>>

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

2017-10-09 Thread wm4
On Sun, 8 Oct 2017 16:49:58 +0100 Mark Thompson wrote: > This has been deprecated in libva2 because hardware does not and will not > support it. Therefore never consider it for decode, and for encode assume > the user meant constrained baseline profile instead. > --- > On 08/10/17 16:44, Derek B

Re: [FFmpeg-devel] FFmpeg 3.4

2017-10-09 Thread wm4
On Sun, 8 Oct 2017 13:53:13 +0200 Michael Niedermayer wrote: > On Sat, Oct 07, 2017 at 12:06:23AM +0200, wm4 wrote: > > On Fri, 6 Oct 2017 16:53:17 +0200 > > Michael Niedermayer wrote: > > > > > Hi all > > > > > > if there are no objec

Re: [FFmpeg-devel] [PATCH 6/6] hwcontext_vaapi: Add support for mapping to DRM objects

2017-10-09 Thread wm4
On Sun, 8 Oct 2017 17:13:24 +0100 Derek Buitenhuis wrote: > On 10/8/2017 5:11 PM, Mark Thompson wrote: > > This is just how hardware surfaces are stored in AVFrames - they have their > > own API-specific handles in the data[] pointers because that's the only > > place to put them. > > > > See

Re: [FFmpeg-devel] [PATCH 00/20] Coded bitstream editing (v3)

2017-10-09 Thread wm4
On Sun, 8 Oct 2017 21:01:34 +0100 Mark Thompson wrote: > Incorporating all review comments from last time: > * Change all CBS users to hold a pointer rather than the whole structure. > * Rearrange the MPEG-2 framerate stuff so that it doesn't add code and then > remove it in the series. > * Add

Re: [FFmpeg-devel] [PATCH] build: prevent SDL2 from polluting global cflags and extralibs

2017-10-09 Thread wm4
On Sun, 8 Oct 2017 17:16:21 -0300 James Almer wrote: > Remove the SDL_main define from the global cflags but not from the > ffplay cflags, and the -mwindows linker option from extralibs instead > of overriding it with the addition of -mconsole. > > Signed-off-by: James Almer > --- > configure

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

2017-10-09 Thread wm4
On Mon, 9 Oct 2017 03:04:53 +0300 Ivan Kalvachev wrote: > The public functions av_alloc_vdpaucontext() and > av_vdpau_alloc_context() are allocating AVVDPAUContext > structure that is supposed to be placed in avctx->hwaccel_context. > > However the rest of libavcodec/vdpau.c uses avctx->hwaccel_

Re: [FFmpeg-devel] [V3 1/4] ffmpeg: remove hwaccel_lax_profile_check opt.

2017-10-09 Thread wm4
On Mon, 9 Oct 2017 12:40:41 +0100 Mark Thompson wrote: > On 09/10/17 08:49, Jun Zhao wrote: > > V3: Remove hwaccel_lax_profile_check opt, and add new pre-stream > > hwaccel_flags option > > > > > From 2b1585fd6e6e68c81761ace0a8503385067086e0 Mon Sep 17 00:00:00 2001 > > From: Jun Zhao > > Da

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

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

Re: [FFmpeg-devel] FFmpeg 3.4

2017-10-10 Thread wm4
On Tue, 10 Oct 2017 12:45:59 -0300 James Almer wrote: > On 10/9/2017 8:17 AM, wm4 wrote: > > On Sun, 8 Oct 2017 13:53:13 +0200 > > Michael Niedermayer wrote: > > > >> On Sat, Oct 07, 2017 at 12:06:23AM +0200, wm4 wrote: > >>> On Fri, 6 Oct 201

Re: [FFmpeg-devel] FFmpeg 3.4

2017-10-10 Thread wm4
On Tue, 10 Oct 2017 13:25:53 -0300 James Almer wrote: > On 10/10/2017 1:01 PM, wm4 wrote: > > On Tue, 10 Oct 2017 12:45:59 -0300 > > James Almer wrote: > > > >> On 10/9/2017 8:17 AM, wm4 wrote: > >>> On Sun, 8 Oct 2017 13:53:13 +0200 > >>

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

2017-10-10 Thread wm4
On Tue, 10 Oct 2017 10:36:58 -0700 Sasi Inguva wrote: > This is required for FLV files, for which duration_pts comes out to be zero. > > Signed-off-by: Sasi Inguva > --- > fftools/ffmpeg.c | 9 +++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/fftools/ffmpeg.c b/fft

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

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

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

2017-10-12 Thread wm4
On Thu, 12 Oct 2017 01:28:01 +0200 Michael Niedermayer wrote: > On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > > 2017-10-10 19:36 GMT+02:00 Sasi Inguva : > > > > > This is required for FLV files, for which duration_pts comes out to be > > > zero. > > > > > > Signed-off-by: Sas

Re: [FFmpeg-devel] [PATCH] udp: added option to ignore empty UDP packets

2017-10-12 Thread wm4
On Thu, 12 Oct 2017 16:16:31 +0200 Nicolas George wrote: > Le quintidi 15 vendémiaire, an CCXXVI, Daniel Kučera a écrit : > > I'm not sure if you mean this patch is unacceptable but if so, I want > > to note, that this patch is not the same as I submitted before: this > > one adds cmdlne option t

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

2017-10-12 Thread wm4
On Thu, 12 Oct 2017 14:28:01 -0400 "Helmut K. C. Tessarek" wrote: > I'm rather busy, and creating my compile script is an ongoing process > which I have been maintaining for the last 6 years. Probably better not to use bleeding edge git master, then.

Re: [FFmpeg-devel] [PATCH] libavformat/wtvdec: return AVERROR_EOF on EOF

2017-10-13 Thread wm4
On Fri, 13 Oct 2017 16:46:31 +0200 Daniel Kučera wrote: > bump. LGTM too. The maintainer is relatively inactive, and it's a small fix that doesn't even require the maintainer's approval, so I pushed the patch. ___ ffmpeg-devel mailing list ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] configure: force erroring out in check_disable_warning() if an option doesn't exists

2017-10-13 Thread wm4
On Fri, 13 Oct 2017 16:30:59 +0200 Hendrik Leppkes wrote: > On Fri, Oct 13, 2017 at 4:14 PM, James Almer wrote: > > On 10/12/2017 6:30 PM, James Almer wrote: > >> Should prevent some options from being added to cflags when they > >> don't exist and the compiler only warns about it. > >> > >> S

[FFmpeg-devel] [PATCH 0/5] Cuvid/videotoolbox preparations

2017-10-13 Thread wm4
m for performing delayed processing on the decoded frames decode: add a per-frame private data for hwaccel use wm4 (1): lavc/avrndec: remove AV_CODEC_CAP_DR1, as it's broken libavcodec/avrndec.c | 1 - libavcodec/decode.c

[FFmpeg-devel] [PATCH 3/5] decode: add a method for attaching lavc-internal data to frames

2017-10-13 Thread wm4
From: Anton Khirnov Use the AVFrame.opaque_ref field. The original user's opaque_ref is wrapped in the lavc struct and then unwrapped before the frame is returned to the caller. This new struct will be useful in the following commits. Merges Libav commit 359a8a3e2d1194b52b6c386f94fd0929567dfb67

[FFmpeg-devel] [PATCH 1/5] lavc/avrndec: remove AV_CODEC_CAP_DR1, as it's broken

2017-10-13 Thread wm4
In the is_mjpeg case, the user's get_buffer2 callback is not called, thus completely breaking the API. --- libavcodec/avrndec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/avrndec.c b/libavcodec/avrndec.c index c37f99661b..104ff2d904 100644 --- a/libavcodec/avrndec.c +++ b/libavc

[FFmpeg-devel] [PATCH 2/5] decode: avoid leaks on failure in ff_get_buffer()

2017-10-13 Thread wm4
From: Anton Khirnov If the get_buffer() call fails, the frame might have some side data already set. Make sure it gets freed. CC: libav-sta...@libav.org Merges Libav commit de77671438c24ffea93398c8dc885d4dd04477de. --- libavcodec/decode.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/

[FFmpeg-devel] [PATCH 4/5] decode: add a mechanism for performing delayed processing on the decoded frames

2017-10-13 Thread wm4
From: Anton Khirnov This will be useful in the CUVID hwaccel. Merges Libav commit badf0951f54c1332e77455dc40398f3512540c1b. --- libavcodec/decode.c | 11 +++ libavcodec/decode.h | 15 +++ 2 files changed, 26 insertions(+) diff --git a/libavcodec/decode.c b/libavcodec/decode

[FFmpeg-devel] [PATCH 5/5] decode: add a per-frame private data for hwaccel use

2017-10-13 Thread wm4
From: Anton Khirnov This will be useful in the CUVID hwaccel. It should also eventually replace current decoder-specific mechanisms used by various other hwaccels. Merges Libav commit 704311b2946d74a80f65906961cd9baaa18683a3. --- libavcodec/decode.c | 3 +++ libavcodec/decode.h | 6 ++ 2 fi

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-13 Thread wm4
On Fri, 13 Oct 2017 20:03:12 +0200 Michael Niedermayer wrote: > I dont really know and as a maintainer of some of this code, i dont > really want to have to keep track of how exactly AVFrames can pass > through the code. You have to do that anyway. > And as the one taking care of a lot of secur

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-13 Thread wm4
On Fri, 13 Oct 2017 19:41:28 +0200 Michael Niedermayer wrote: > On Fri, Oct 06, 2017 at 01:48:14AM +0200, wm4 wrote: > > On Fri, 6 Oct 2017 00:01:30 +0200 > > Michael Niedermayer wrote: > > > > > The opaque_ref wraping is a really bad design. Iam not s

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-16 Thread wm4
On Sat, 14 Oct 2017 23:01:41 +0200 Michael Niedermayer wrote: > On Fri, Oct 13, 2017 at 09:19:04PM +0200, wm4 wrote: > > On Fri, 13 Oct 2017 19:41:28 +0200 > > Michael Niedermayer wrote: > > > > > On Fri, Oct 06, 2017 at 01:48:14AM +0200, wm4 wrote: >

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-16 Thread wm4
On Mon, 16 Oct 2017 19:28:27 +0800 Xiaolei Yu wrote: > On 10/03/2017 09:15 PM, wm4 wrote: > > From: Anton Khirnov > > > > Use the AVFrame.opaque_ref field. The original user's opaque_ref is > > wrapped in the lavc struct and then unwrapped before the fra

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-16 Thread wm4
On Mon, 16 Oct 2017 20:32:03 +0800 Xiaolei Yu wrote: > On 10/16/2017 07:36 PM, wm4 wrote: > > On Mon, 16 Oct 2017 19:28:27 +0800 > > Xiaolei Yu wrote: > > > >> On 10/03/2017 09:15 PM, wm4 wrote: > >>> From: Anton Khirnov > >>> >

Re: [FFmpeg-devel] [PATCH] frame: add an av_frame_new_side_data_from_buf function

2017-10-16 Thread wm4
On Mon, 16 Oct 2017 16:02:19 +0100 Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > libavutil/frame.c | 16 ++-- > libavutil/frame.h | 13 + > 2 files changed, 19 insertions(+), 10 deletions(-) > > diff --git a/libavutil/frame.c b/libavutil/fram

Re: [FFmpeg-devel] [PATCH] frame: add an av_frame_new_side_data_from_buf function

2017-10-16 Thread wm4
On Mon, 16 Oct 2017 12:27:17 -0300 James Almer wrote: > On 10/16/2017 12:02 PM, Rostislav Pehlivanov wrote: > > Signed-off-by: Rostislav Pehlivanov > > --- > > libavutil/frame.c | 16 ++-- > > libavutil/frame.h | 13 + > > 2 files changed, 19 insertions(+), 10 deletions(

Re: [FFmpeg-devel] [PATCH] frame: add an av_frame_new_side_data_from_buf function

2017-10-16 Thread wm4
On Mon, 16 Oct 2017 16:24:56 +0100 Rostislav Pehlivanov wrote: > On 16 October 2017 at 16:14, wm4 wrote: > > > On Mon, 16 Oct 2017 16:02:19 +0100 > > Rostislav Pehlivanov wrote: > > > > > Signed-off-by: Rostislav Pehlivanov > &

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-16 Thread wm4
Tue, 17 Oct 2017 00:58:58 +0200 Michael Niedermayer wrote: > On Mon, Oct 16, 2017 at 09:40:26AM +0200, wm4 wrote: > > On Sat, 14 Oct 2017 23:01:41 +0200 > > Michael Niedermayer wrote: > > > > > On Fri, Oct 13, 2017 at 09:19:04PM +0200, wm4 wrote: > &g

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-16 Thread wm4
Correction: will push as part of cuvid/videotoolbox patches whenever they're ready. On Tue, 17 Oct 2017 00:58:58 +0200 Michael Niedermayer wrote: > > It doesn't - not the user's. We use only the field for internal > > purposes (as AVFrame users), and we never do anything with the user's > > valu

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2: fix access to priv_data after codec close.

2017-10-17 Thread wm4
On Tue, 17 Oct 2017 19:20:50 +0200 Jorge Ramirez-Ortiz wrote: > A user can close the codec while keeping references to some of the > capture buffers. When the codec is closed, the structure that keeps > the contexts, state and the driver file descriptor is freed. > > Since access to some of the

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-17 Thread wm4
On Tue, 17 Oct 2017 20:23:25 +0200 Thilo Borgmann wrote: > Am 17.10.17 um 06:49 schrieb wm4: > > I have realized that your veto is actually not valid: > > - it's a Libav merge > > - it has been for months in the Libav repo and you didn't specifically > >

Re: [FFmpeg-devel] [PATCH] openssl: Support version 1.1.0.

2016-10-14 Thread wm4
On Mon, 10 Oct 2016 02:39:51 +1100 Matt Oliver wrote: > --- > configure | 3 +- > libavformat/tls_openssl.c | 159 > -- > 2 files changed, 98 insertions(+), 64 deletions(-) > > diff --git a/configure b/configure > index df6ffa2..7506

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

2016-10-14 Thread wm4
On Thu, 13 Oct 2016 11:27:12 +0200 Nicolas George wrote: > Le duodi 22 vendémiaire, an CCXXV, Steven Liu a écrit : > > init add hds demuxer > > > > Based-on: patch by CORY MCCARTHY > > Based-on: patch by Gorilla Maguila > > Signed-off-by: Steven Liu > > Thanks for the update. > > I think

Re: [FFmpeg-devel] [PATCH] d3d11va: use the proper slice index

2016-10-14 Thread wm4
On Tue, 11 Oct 2016 10:18:06 +0200 Hendrik Leppkes wrote: > On Mon, Oct 10, 2016 at 11:34 PM, compn wrote: > > On Mon, 10 Oct 2016 17:07:06 +0200 > > Hendrik Leppkes wrote: > > > >> On Mon, Oct 10, 2016 at 4:01 PM, Michael Niedermayer > >> > > >> > maybe our dxva2 maintainer wants a co main

Re: [FFmpeg-devel] Developer "cehoyos" closed my bug without any explanation, and without solving it

2016-10-14 Thread wm4
On Tue, 11 Oct 2016 00:23:20 +0200 Alexey Eromenko wrote: > Hello all, > > Bug #5882; Certain x264 videos do not play on Apple decoders > (Quicktime/iTunes/iPad) > > Today was closed by developer "cehoyos" without any explanation, and > without any resolution. > He treats all bugs like that ? W

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

2016-10-14 Thread wm4
On Fri, 14 Oct 2016 16:31:58 +0200 Nicolas George wrote: > Le tridi 23 vendémiaire, an CCXXV, Steven Liu a écrit : > > > Rather than NIHing it, I think using libexpat would be a good choice > > > (even if its API is terrible). > > ok, i'll learn how to use it, and use it next step. :-) > > W

Re: [FFmpeg-devel] [PATCH] avcodec/vda: define av_vda_default_init2 when CONFIG_H264_VDA_HWACCEL equ 0

2016-10-14 Thread wm4
On Thu, 13 Oct 2016 20:57:07 +0800 Steven Liu wrote: > ping > > 2016-10-12 17:36 GMT+08:00 Steven Liu : > > > on OSX: > > ../configure --disable-everything --enable-demuxer=hls make > > error message: Undefined symbols for architecture x86_64: > > "_av_vda_default_init2", referenced from:_video

Re: [FFmpeg-devel] [PATCH] avformat/udp: deprecate local_port option

2016-10-14 Thread wm4
On Sat, 8 Oct 2016 15:35:02 +0800 Steven Liu wrote: > If you send a patch like this, make sure you do the following things: - explain in the commit message WHY - document the deprecation in the, you know, documentation - document the replacement It should be natural that this should be required

Re: [FFmpeg-devel] [PATCH] avformat/rtsp: support rtsps

2016-10-16 Thread wm4
On Sun, 16 Oct 2016 12:24:37 -0400 jayri...@gmail.com wrote: > From: Jay Ridgeway > > Pass TLS args to support RTSPS. This patch requires TLS patch. > --- > libavformat/rtsp.c | 19 --- > libavformat/rtsp.h | 8 > 2 files changed, 24 insertions(+), 3 deletions(-) > >

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-16 Thread wm4
On Sun, 16 Oct 2016 21:51:32 +0200 Nicolas George wrote: > Le quintidi 25 vendémiaire, an CCXXV, Michael Behrisch a écrit : > > Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been > > rejected which tried to remove the comma at the end of enumerator lists > > which triggers a

Re: [FFmpeg-devel] FFmpeg 3.2

2016-10-17 Thread wm4
On Mon, 17 Oct 2016 04:24:40 +0200 Michael Niedermayer wrote: > On Sun, Oct 16, 2016 at 06:01:54PM -0400, Helmut K. C. Tessarek wrote: > > On 2016-09-27 09:30, Michael Niedermayer wrote: > > > Its long since FFmpeg 3.1, so its time to make 3.2 > > > ill branch release/3.2 off master and make 3.

Re: [FFmpeg-devel] [PATCH] avformat/rtsp: support rtsps

2016-10-17 Thread wm4
On Sun, 16 Oct 2016 21:20:04 + Jay wrote: > This is the approach present in avformat/rtsp.c, but I can modify. What is > preferred? > > For clarity, this patch does not copy url params from the `rtsps://` input. > It passes tls params provided as cmd line options to the `tls://` protocol > h

Re: [FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference

2016-10-17 Thread wm4
On Sun, 16 Oct 2016 22:11:03 +0200 Andreas Cadhalpun wrote: > The problem was introduced in commit 1273bc6. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/matroskadec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/matroskadec.c b/libavformat/m

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-17 Thread wm4
On Mon, 17 Oct 2016 13:09:36 +0200 Michael Niedermayer wrote: > On Mon, Oct 17, 2016 at 10:07:42AM +0200, Nicolas George wrote: > > Le sextidi 26 vendémiaire, an CCXXV, Michael Niedermayer a écrit : > > > probably, yes > > > > I would have said exactly the opposite. It is nothing but a waste

Re: [FFmpeg-devel] FFmpeg 3.2

2016-10-17 Thread wm4
On Mon, 17 Oct 2016 15:47:06 +0200 Michael Niedermayer wrote: > On Mon, Oct 17, 2016 at 12:01:56PM +0200, wm4 wrote: > > On Mon, 17 Oct 2016 04:24:40 +0200 > > Michael Niedermayer wrote: > > > > > On Sun, Oct 16, 2016 at 06:01:54PM -0400, Helmut K. C. Tessarek

Re: [FFmpeg-devel] [PATCH 2/3] lavfi/loudnorm: add an internal libebur128 library

2016-10-17 Thread wm4
On Sun, 16 Oct 2016 21:32:03 +0200 Marton Balint wrote: > Also contains the following changes to the library: > - add ff_ prefix to functions > - remove cplusplus defines. > - add FF_ prefix to contants and some structs > - remove true peak calculation feature, since it uses its own resampler, an

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-18 Thread wm4
On Fri, 5 Aug 2016 17:18:39 -0700 Sasi Inguva wrote: > In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit lists which are not just used to offset > video start from audio start) and multiple edit lists. Recently the uploads > of such f

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-19 Thread wm4
successfully, but we have generated a file with AVSync problems, and > which plays wrong. > > On Tue, Oct 18, 2016 at 7:31 AM, wm4 wrote: > > > On Fri, 5 Aug 2016 17:18:39 -0700 > > Sasi Inguva wrote: > > > > > In YouTube we have long been receiving MOV

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-21 Thread wm4
On Fri, 21 Oct 2016 16:59:14 +0100 Derek Buitenhuis wrote: > On 10/21/2016 4:32 PM, Derek Buitenhuis wrote: > > * Audio packets. Especially audio packets with a large number of > > samples. > > It's extremely likely that edits will not fall on packet boundaries, > > and > > depe

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

2016-10-24 Thread wm4
On Sat, 15 Oct 2016 15:52:48 +0200 Nicolas George wrote: > Le tridi 23 vendémiaire, an CCXXV, wm4 a écrit : > > XML is very complex > > This is the usual, and often only, argument raised in this kind of > situation. And unfortunately, I already addressed it in this ver

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-24 Thread wm4
On Sun, 23 Oct 2016 13:02:01 -0400 "Ronald S. Bultje" wrote: > Hi, > > On Sat, Oct 22, 2016 at 11:36 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > > > On Sat, Oct 22, 2016 at 10:10:01PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > general comment about all other dec p

Re: [FFmpeg-devel] [PATCH] avutil/x86/emms: Document the emms_c() vs alloc/free relation.

2016-10-24 Thread wm4
On Sun, 23 Oct 2016 05:37:25 +0200 Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavutil/x86/emms.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavutil/x86/emms.h b/libavutil/x86/emms.h > index 6fda6e2..42c18e2 100644 > --- a/libavutil/x86/emms.h >

Re: [FFmpeg-devel] [PATCH 02/13] avcodec: Clear MMX state in ff_thread_report_progress(INT_MAX)

2016-10-24 Thread wm4
On Sat, 22 Oct 2016 22:06:22 -0400 "Ronald S. Bultje" wrote: > Hi, > > On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer > wrote: > > > This decreases the number of FPU state violations in fate on x86-64 from > > 309 to 79 > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcode

Re: [FFmpeg-devel] [PATCH 3/4] lavfi: add FFFrameQueue API.

2016-10-24 Thread wm4
On Sun, 23 Oct 2016 12:27:41 +0200 Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavfilter/Makefile | 1 + > libavfilter/framequeue.c | 123 + > libavfilter/framequeue.h | 173 > +++ > 3 files c

Re: [FFmpeg-devel] [PATCH 02/12] bfi: validate sample_rate

2016-10-24 Thread wm4
On Sun, 23 Oct 2016 18:27:02 +0200 Andreas Cadhalpun wrote: > A negative sample rate doesn't make sense and triggers assertions in > av_rescale_rnd. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/bfi.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/bfi.c b/

Re: [FFmpeg-devel] ogg kate text subtitles support (patch)

2016-10-24 Thread wm4
On Mon, 24 Oct 2016 09:31:25 +0200 u-9...@aetey.se wrote: > Hello, > > Given the practical constraints I can not thoroughly fulfill all the > requirements for submitting a patch. I hope it can make it at least to > the list archive, for possible future perusal by someone. > > The patch addresses

Re: [FFmpeg-devel] ogg kate text subtitles support (patch)

2016-10-24 Thread wm4
On Mon, 24 Oct 2016 10:52:14 +0200 u-9...@aetey.se wrote: > worth to mention: > > On Mon, Oct 24, 2016 at 10:25:44AM +0200, u-9...@aetey.se wrote: > > > >https://trac.ffmpeg.org/ticket/3039 > > The ticket refers to a sample with graphical subtitles. This is _not_ > being solved by

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