Re: [FFmpeg-devel] [PATCH] ffmpeg: remove unused and errorneous AVFrame timestamp check

2016-10-02 Thread wm4
On Sat, 1 Oct 2016 16:15:45 +0200 Hendrik Leppkes wrote: > Decoders have previously not used AVFrame.pts, and with the upcoming > deprecation of pkt_pts (in favor of pts), this would lead to an errorneous > interpration of timestamps. > --- > ffmpeg.c | 7 +-- > 1 file changed, 1 insertion(

[FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-02 Thread wm4
Some API users (in particular ffmpeg.c) check the best effort timestamp only. --- Still undecided if this is the right approach. --- libavcodec/utils.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index ef3da65..1875a69 100644 --

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread wm4
On Mon, 3 Oct 2016 01:51:25 +0200 Carl Eugen Hoyos wrote: > From 35bf44b909658864c9e8cca8dbc855dbde1e1ef5 Mon Sep 17 00:00:00 2001 > From: Carl Eugen Hoyos > Date: Mon, 3 Oct 2016 01:41:37 +0200 > Subject: [PATCH] doc/platform: Mention musl where x86_32 is not supported. > > --- > doc/platform

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread wm4
On Mon, 3 Oct 2016 17:01:12 +0200 u-9...@aetey.se wrote: > Ronald, > > I sincerely appreciate that you are taking the effort to talk to me > and explain your position. > > Unfortunately in your messages you consistently ignored a crucial point > and this is the last time I try to retell it: > >

Re: [FFmpeg-devel] [PATCH 3/3] lavf/mp3dec: read encoder delay/padding from Info tag

2016-10-04 Thread wm4
On Mon, 3 Oct 2016 17:45:08 -0700 Jon Toohill wrote: > Muxers can check AVCodecParameters.initial_padding for the > encoder+decoder delay, and read the AV_PKT_DATA_SKIP_SAMPLES > side data from the last packet for the encoder padding. > > This change also fixes the first_discard_sample calculat

Re: [FFmpeg-devel] [PATCH] ffmpeg: remove unused and errorneous AVFrame timestamp check

2016-10-04 Thread wm4
On Tue, 4 Oct 2016 14:15:03 +0200 Michael Niedermayer wrote: > On Tue, Oct 04, 2016 at 01:52:02PM +0200, Hendrik Leppkes wrote: > > On Tue, Oct 4, 2016 at 1:44 PM, Hendrik Leppkes > > wrote: > > > On Tue, Oct 4, 2016 at 1:23 PM, Michael Niedermayer > > > wrote: > > >> On Tue, Oct 04, 2016

Re: [FFmpeg-devel] [PATCH] ffmpeg: remove unused and errorneous AVFrame timestamp check

2016-10-04 Thread wm4
On Tue, 4 Oct 2016 16:35:02 +0200 Hendrik Leppkes wrote: > On Tue, Oct 4, 2016 at 4:32 PM, wm4 wrote: > > On Tue, 4 Oct 2016 14:15:03 +0200 > > Michael Niedermayer wrote: > > > >> On Tue, Oct 04, 2016 at 01:52:02PM +0200, Hendrik Leppkes wrote: > &g

[FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-04 Thread wm4
Some API users (in particular ffmpeg.c) check the best effort timestamp only. --- Using guess_correct_pts() - not sure what I was thinking. --- libavcodec/utils.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index ef3da65..9f8f

Re: [FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-05 Thread wm4
On Tue, 4 Oct 2016 20:26:49 +0200 Michael Niedermayer wrote: > On Tue, Oct 04, 2016 at 06:33:48PM +0200, wm4 wrote: > > Some API users (in particular ffmpeg.c) check the best effort timestamp > > only. > > --- > > Using guess_correct_pts() - not sure what I was thinki

Re: [FFmpeg-devel] [PATCH 3/3] lavf/mp3dec: read encoder delay/padding from Info tag

2016-10-05 Thread wm4
On Wed, 5 Oct 2016 10:42:13 -0700 Jon Toohill wrote: > On Wed, Oct 5, 2016 at 10:40 AM, Jon Toohill wrote: > > > On Tue, Oct 4, 2016 at 7:19 AM, wm4 wrote: > > > >> On Mon, 3 Oct 2016 17:45:08 -0700 > >> Jon Toohill wrote: > >> > >>

Re: [FFmpeg-devel] [PATCH]lavu/hwaccel_vaapi: Add a fixme for the missing byte_order check

2016-10-06 Thread wm4
On Thu, 6 Oct 2016 10:34:35 +0100 Mark Thompson wrote: > On 06/10/16 09:05, Carl Eugen Hoyos wrote: > > 2016-10-05 21:55 GMT+02:00 Mark Thompson : > > > > What I meant is: > > How can "LE" make sense for an 8bit planar format? > > It does not. So, another reason to ignore that field. > > >

Re: [FFmpeg-devel] [PATCH] avutil: add AV_PIX_FMT_GBR(A)P32F?E & AV_PIX_FMT_YUV(A)P32?E

2016-02-10 Thread wm4
On Wed, 10 Feb 2016 21:56:07 +0100 Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavutil/pixdesc.c | 105 > +++ > libavutil/pixfmt.h | 13 +++ > 2 files changed, 118 insertions(+) > > diff --git a/libavutil/pi

Re: [FFmpeg-devel] [PATCH] avutil: add AV_PIX_FMT_GBR(A)P32F?E & AV_PIX_FMT_YUV(A)P32?E

2016-02-10 Thread wm4
On Wed, 10 Feb 2016 23:00:00 +0100 Michael Niedermayer wrote: > On Wed, Feb 10, 2016 at 10:21:34PM +0100, wm4 wrote: > > On Wed, 10 Feb 2016 21:56:07 +0100 > > Michael Niedermayer wrote: > > > > > Signed-off-by: Michael Niedermayer > > >

Re: [FFmpeg-devel] [PATCH] doc/protocols: document protocol_whitelist

2016-02-11 Thread wm4
On Thu, 11 Feb 2016 13:12:14 +0100 Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/protocols.texi | 16 > 1 file changed, 16 insertions(+) > > diff --git a/doc/protocols.texi b/doc/protocols.texi > index 05c4bdb..f790351 100644 > --- a/doc/protoco

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-02-11 Thread wm4
On Wed, 10 Feb 2016 15:47:52 + Derek Buitenhuis wrote: > * Upstream libutvideo doesn't build as a lib, and Linux is not supported. > It has no API. > * The fork that was used as an argument to keep support is *also* broken and > unmaintained now, and it's maintainer/author is fine with re

Re: [FFmpeg-devel] [PATCH v9] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

2016-02-12 Thread wm4
On Fri, 12 Feb 2016 13:23:02 +0100 Mats Peterson wrote: > On 02/12/2016 12:49 PM, Michael Niedermayer wrote: > >> +av_log(avctx, AV_LOG_DEBUG, "PACKET SIZE: %d\n", avpkt->size); > >> +av_log(avctx, AV_LOG_DEBUG, "STRIDE: %d\n", stride); > > > > this looks like some private debug code, w

Re: [FFmpeg-devel] [PATCH v9] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

2016-02-13 Thread wm4
On Sat, 13 Feb 2016 13:09:48 +0100 Mats Peterson wrote: > On 02/13/2016 01:03 PM, Mats Peterson wrote: > > On 02/13/2016 01:00 PM, Mats Peterson wrote: > >> On 02/13/2016 12:58 PM, Mats Peterson wrote: > >>> On 02/13/2016 12:57 PM, Mats Peterson wrote: > On 02/13/2016 12:54 PM, Mats Pe

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: av_image_get_buffer_size(): do not insert padding between stride*height and palette

2016-02-13 Thread wm4
On Sat, 13 Feb 2016 19:38:01 +0100 Michael Niedermayer wrote: > This fixes the layout that is stored in pal8 nut with odd width*height > > Signed-off-by: Michael Niedermayer > --- > libavcodec/avpicture.c |2 ++ > libavutil/imgutils.c |4 > 2 files changed, 6 insertions(+) > >

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: av_image_get_buffer_size(): do not insert padding between stride*height and palette

2016-02-13 Thread wm4
On Sat, 13 Feb 2016 21:51:48 +0100 Michael Niedermayer wrote: > On Sat, Feb 13, 2016 at 08:46:34PM +0100, wm4 wrote: > > On Sat, 13 Feb 2016 19:38:01 +0100 > > Michael Niedermayer wrote: > > > > > This fixes the layout that is stored in pal8 nut with odd width*

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread wm4
On Sun, 14 Feb 2016 13:07:04 -0300 James Almer wrote: > On 2/14/2016 1:04 PM, Carl Eugen Hoyos wrote: > > Henrik Gramner gramner.com> writes: > > > >> The Ubuntu 16.04 LTS feature freeze is coming up next week, > >> so it'd be nice to have a release before that. > > > > (I am all for it,

Re: [FFmpeg-devel] [PATCH] alac: allow extra data without the ATOM header

2016-02-15 Thread wm4
On Mon, 15 Feb 2016 10:21:19 +0100 Steve Lhomme wrote: > On Mon, Feb 15, 2016 at 10:16 AM, Carl Eugen Hoyos wrote: > > Steve Lhomme videolabs.io> writes: > > > >> I implemented the same change in VLC. > > > > From a quick look, you added code to the decoder > > wrapper but how is remuxing s

Re: [FFmpeg-devel] [FFmpeg-mentors] New task ideas for GSoC and Outreachy

2016-02-15 Thread wm4
On Mon, 15 Feb 2016 10:17:39 +0100 Stefano Sabatini wrote: > On date Monday 2016-02-15 09:21:15 +0100, Paul B Mahol encoded: > > On 2/15/16, Stefano Sabatini wrote: > > > On date Wednesday 2016-02-10 16:21:09 +, Kieran Kunhya encoded: > [...] > > >> So, in the spirit of open source, I'd

Re: [FFmpeg-devel] [PATCH v5] Added VideoToolbox H.264 encoder.

2016-02-16 Thread wm4
On Tue, 16 Feb 2016 08:32:40 + (UTC) Carl Eugen Hoyos wrote: > Rick Kern gmail.com> writes: > > > +enabled vtenc && > > require VideoToolbox VideoToolbox/VTCompressionSession.h > > On which osx systems is this header not available? The same where videotoolbox decoding is no

Re: [FFmpeg-devel] [PATCH v5] Added VideoToolbox H.264 encoder.

2016-02-16 Thread wm4
On Tue, 16 Feb 2016 09:58:30 +0800 Rick Kern wrote: > Enable with configure --enable-vtenc and encode using -codec:v vtenc_h264. > > Signed-off-by: Rick Kern > --- > MAINTAINERS|1 + > configure |3 + > libavcodec/Makefile|1 + > libavcodec/allcodecs.c

Re: [FFmpeg-devel] MediaCodec support

2016-02-16 Thread wm4
On Mon, 15 Feb 2016 18:52:25 +0100 Matthieu Bouron wrote: > Hello, > > The following patchset adds basic MediaCodec support to libavcodec, ie: only > h264 is supported and the HWAccel part (Surface output) is missing. > > JNI comes as a dependency. The JNI support is based on the same patchset

Re: [FFmpeg-devel] [PATCH v2] avfilter/avf_showcqt: improve pts handling

2016-02-16 Thread wm4
On Tue, 16 Feb 2016 18:01:16 +0700 Muhammad Faiz wrote: > From bc59d4a7636e2f199b3dbda06e8e3bc53e260cae Mon Sep 17 00:00:00 2001 > From: Muhammad Faiz > Date: Tue, 16 Feb 2016 07:03:37 +0700 > Subject: [PATCH v2] avfilter/avf_showcqt: improve pts handling > > correct output pts based on input p

Re: [FFmpeg-devel] MediaCodec support

2016-02-16 Thread wm4
On Tue, 16 Feb 2016 12:09:58 +0100 Matthieu Bouron wrote: > On Tue, Feb 16, 2016 at 10:41 AM, wm4 wrote: > > > On Mon, 15 Feb 2016 18:52:25 +0100 > > Matthieu Bouron wrote: > > > > > Hello, > > > > > > The following patchset adds bas

Re: [FFmpeg-devel] [PATCH v6] Added VideoToolbox H.264 encoder.

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 18:30:04 +0800 Rick Kern wrote: > Autodetected by default. Encode using -codec:v vtenc. > > Signed-off-by: Rick Kern > --- > MAINTAINERS|1 + > configure |5 + > libavcodec/Makefile|1 + > libavcodec/allcodecs.c |1 + > libavcode

Re: [FFmpeg-devel] [PATCH v2] avfilter/avf_showcqt: improve pts handling

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 01:13:54 +0700 Muhammad Faiz wrote: > On Tue, Feb 16, 2016 at 6:22 PM, wm4 wrote: > > On Tue, 16 Feb 2016 18:01:16 +0700 > > Muhammad Faiz wrote: > ... > >> remaining -= s->remaining_fill; > >> +if (o

Re: [FFmpeg-devel] [PATCH v2] avfilter/avf_showcqt: improve pts handling

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 10:39:30 +0100 Paul B Mahol wrote: > On 2/16/16, Muhammad Faiz wrote: > > correct output pts based on input pts > > make seeking possible > > output frame one by one on eof > > tested with showinfo filter > > > > patch attached > > > > thank's > > > > Tested with mpv, work

Re: [FFmpeg-devel] [PATCH 2/2] lavc/rawdec: Retrieve nut palette from packets

2016-02-17 Thread wm4
r others to decide, my oppinion is that the > > muxer bug must be fixed first to prevent the generation of invalid > > files. I wanted to look into it as you dont look into it but i dont > > have time now > > > > Please be patient, also its not going to be done sooner b

Re: [FFmpeg-devel] [PATCH v2] avfilter/avf_showcqt: improve pts handling

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 12:33:49 +0100 Paul B Mahol wrote: > On 2/17/16, wm4 wrote: > > On Wed, 17 Feb 2016 10:39:30 +0100 > > Paul B Mahol wrote: > > > >> On 2/16/16, Muhammad Faiz wrote: > >> > correct output pts based on input pts > >> >

Re: [FFmpeg-devel] [PATCH] avfilter/src_movie: add various commands

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 21:30:20 +0700 Muhammad Faiz wrote: > From 6291c06dacbe2aa48a97bf36835ec8a3bce8a40a Mon Sep 17 00:00:00 2001 > From: Muhammad Faiz > Date: Wed, 17 Feb 2016 21:21:12 +0700 > Subject: [PATCH] avfilter/src_movie: add various commands > > add seek command > add get_duration comm

Re: [FFmpeg-devel] [PATCH] avfilter/src_movie: add various commands

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 22:55:47 +0700 Muhammad Faiz wrote: > On Wed, Feb 17, 2016 at 10:20 PM, wm4 wrote: > > On Wed, 17 Feb 2016 21:30:20 +0700 > > Muhammad Faiz wrote: > > > >> From 6291c06dacbe2aa48a97bf36835ec8a3bce8a40a Mon Sep 17 00:00:00 2001 > >>

Re: [FFmpeg-devel] [PATCH] avfilter/src_movie: add various commands

2016-02-17 Thread wm4
On Wed, 17 Feb 2016 17:05:18 +0100 Michael Niedermayer wrote: > On Wed, Feb 17, 2016 at 04:58:31PM +0100, wm4 wrote: > > On Wed, 17 Feb 2016 22:55:47 +0700 > > Muhammad Faiz wrote: > > > > > On Wed, Feb 17, 2016 at 10:20 PM, wm4 wrote: > > &g

Re: [FFmpeg-devel] GSoC 2016 own ideas

2016-02-18 Thread wm4
On Thu, 18 Feb 2016 18:27:45 +0100 Gerion Entrup wrote: > Good day, > > I'm a master student and long term FFmpeg-user. I want to participate in the > GSoC 2016 for FFmpeg. The reason, I write this, is that I want to suggest > some own ideas. It could be, that some of the mentioned things are

Re: [FFmpeg-devel] GSoC 2016 own ideas

2016-02-18 Thread wm4
On Thu, 18 Feb 2016 20:41:29 +0100 Gerion Entrup wrote: > On Donnerstag, 18. Februar 2016 20:10:47 CET wm4 wrote: > > On Thu, 18 Feb 2016 18:27:45 +0100 > > Gerion Entrup wrote: > > > > > Good day, > > > > > > I'm a master studen

Re: [FFmpeg-devel] [PATCH] vp9: add superframe merging bitstream filter.

2016-02-20 Thread wm4
On Fri, 19 Feb 2016 16:28:44 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Fri, Feb 19, 2016 at 3:48 PM, Michael Niedermayer > wrote: > > > On Fri, Feb 19, 2016 at 02:43:48PM -0500, Ronald S. Bultje wrote: > > > Fixes ticket 4313. > > > --- > > > ffmpeg.c| 3 + > >

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-20 Thread wm4
On Sun, 14 Feb 2016 16:25:02 +0100 Michael Niedermayer wrote: > This was suggested by wm4 and stefano. > After this patch using align=1 the size used by various functions would not > contain padding, while the palette would be aligned at align>1 > > This patch makes it requir

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

2016-02-20 Thread wm4
On Sat, 20 Feb 2016 17:41:40 + Derek Buitenhuis wrote: > > ffmpeg | branch: master | Michael Niedermayer | > > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] > > | committer: Michael Niedermayer > > [...] > > > +if (align < 4) { > > +av_l

Re: [FFmpeg-devel] Killing ASS timings in AVSubtitle ass text field

2016-02-21 Thread wm4
On Sun, 21 Feb 2016 13:08:27 +0100 Clément Bœsch wrote: > [PATCH 1/6] Kill timed SSA > > This one is an old patch, which kills the usage of the old and unused > AV_CODEC_ID_SSA > > [PATCH 2/6] lavc: allow subtitle text format to be ASS without timing > > The big commit. Text based decoders do

Re: [FFmpeg-devel] [PATCH v6] Added VideoToolbox H.264 encoder.

2016-02-22 Thread wm4
On Wed, 17 Feb 2016 18:30:04 +0800 Rick Kern wrote: > Autodetected by default. Encode using -codec:v vtenc. > > Signed-off-by: Rick Kern > --- The configure check doesn't work. It tries to enable it on non-OSX, which breaks building ffmpeg out of the box on anything but OSX. Also, I noticed t

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

2016-02-22 Thread wm4
t; everyone happy. > > I'm truly not sure. In my view, the technically correct thing to do > is to introduce a new API function (according to Semantic Versioning). > This introduces yet another some_function2, which is more API churn... > > I am not sure how many people ou

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-02-22 Thread wm4
On Mon, 22 Feb 2016 12:20:36 +0100 Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > ... Some remarks: - The qcom stuff should probably be moved into its own source file, because it's a lot of code, but self-contained. - Do you really need h264_extradata_to_annexb_sps_pps? The BSF a

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-02-23 Thread wm4
On Tue, 23 Feb 2016 09:53:43 +0100 Matthieu Bouron wrote: > On Mon, Feb 22, 2016 at 01:08:49PM +0100, Michael Niedermayer wrote: > > On Mon, Feb 22, 2016 at 12:20:36PM +0100, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > [...] > > > +codec = (*env)->NewObject(env, jfields.

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-02-23 Thread wm4
On Mon, 22 Feb 2016 12:20:36 +0100 Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > +FFAMediaFormat *ff_AMediaFormat_new() > +{ > +int attached = 0; That's C++, not C. In C it needs to be *ff_AMediaFormat_new(void). An empty argument list means that the argument list is unknown a

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-02-23 Thread wm4
On Tue, 23 Feb 2016 10:21:10 +0100 Matthieu Bouron wrote: > On Mon, Feb 22, 2016 at 02:55:06PM +0100, wm4 wrote: > > On Mon, 22 Feb 2016 12:20:36 +0100 > > Matthieu Bouron wrote: > > > > > From: Matthieu Bouron > > > > > > --- > > &g

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-02-24 Thread wm4
On Wed, 24 Feb 2016 10:32:24 + (UTC) Carl Eugen Hoyos wrote: > Ronald S. Bultje gmail.com> writes: > > > > As requested on ffmpeg-user. > > > > I'm a little ambivalent to this. Let me explain. You can > > easily fix this with a shell script that creates links > > from img-{1000...1}.jp

Re: [FFmpeg-devel] [PATCH 1/3] lavu/attributes: introduce av_likely, av_unlikely

2016-02-24 Thread wm4
On Thu, 25 Feb 2016 15:48:17 +1100 Matt Oliver wrote: > On 25 February 2016 at 13:20, Ganesh Ajjanagadde wrote: > > > From: Ganesh Ajjanagadde > > > > These use __builtin_expect, and may be useful for optimizing nearly > > certain branches, to yield size and/or speed improvements. > > > > Note

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mux: add a format flag which ensure parsed and standardized creation time

2016-02-25 Thread wm4
On Thu, 25 Feb 2016 02:11:00 +0100 Marton Balint wrote: > This can be used for formats which write all format metadata as string to > files, therefore non-standard creation times such as 'now' will be parsed. > > The standardized creation time is UTC ISO 8601 with microsecond precision. > > Sig

Re: [FFmpeg-devel] [PATCH] mss2: Fix buffer overflow.

2016-02-25 Thread wm4
On Thu, 25 Feb 2016 21:06:46 +0100 Reimar Döffinger wrote: > Reported as https://trac.mplayerhq.hu/ticket/2264 but have > not been able to reproduce with FFmpeg-only. > I have no idea what coded_height is used for here exactly, > so this might not be the best fix. > Fixes the following chain of e

Re: [FFmpeg-devel] [PATCH] mss2: Fix buffer overflow.

2016-02-26 Thread wm4
On Thu, 25 Feb 2016 22:39:51 +0100 Reimar Döffinger wrote: > On Thu, Feb 25, 2016 at 09:25:08PM +0100, wm4 wrote: > > On Thu, 25 Feb 2016 21:06:46 +0100 > > Reimar Döffinger wrote: > > > > > Reported as https://trac.mplayerhq.hu/ticket/2264 but have > &g

Re: [FFmpeg-devel] [PATCH] mjpegdec: Do not assume unused plane pointer are NULL.

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 02:38:13 +0100 Michael Niedermayer wrote: > On Fri, Feb 26, 2016 at 12:15:19AM +0100, Reimar Döffinger wrote: > > We do neither document nor check such a requirement > > and for application-provided get_buffer2 they could > > contain the result of a malloc(0) or whatever value

Re: [FFmpeg-devel] [PATCH] Always read frame progress values under progress_mutex.

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 05:14:47 +0100 Michael Niedermayer wrote: > On Thu, Feb 25, 2016 at 04:42:22PM -0800, Wan-Teh Chang wrote: > > On Thu, Feb 25, 2016 at 4:15 PM, Hendrik Leppkes > > wrote: > > > > > > Similar to the other patch, please explain what this actually fixes, > > > those sanitizer

Re: [FFmpeg-devel] [PATCH] mjpegdec: Do not assume unused plane pointer are NULL.

2016-02-26 Thread wm4
t; >> > >>> also, is it really a good idea to leave stale pointers in the array? > >> > >> No, of course not. The question from my point of view is rather: should > >> stale pointers or malloc(0) lead directly to an exploitable buffer > >&g

Re: [FFmpeg-devel] [PATCH] mjpegdec: Do not assume unused plane pointer are NULL.

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 13:17:29 +0100 Reimar Döffinger wrote: > Well, there I go, doing my own idle discussions > I just complained about ;) > > On Fri, Feb 26, 2016 at 01:00:23PM +0100, wm4 wrote: > > On Fri, 26 Feb 2016 12:19:18 +0100 > > Reimar Döffinger wrote: >

Re: [FFmpeg-devel] [PATCH] lavf/mp3: Properly check return values of seeks and reads while reading the header

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 17:14:24 + Derek Buitenhuis wrote: > Fixes large amounts of seeking past EOF, which could be extremely > slow over a network. > > Signed-off-by: Derek Buitenhuis > --- > Example of the problem: > ffmpeg -f mp3 -i http://chromashift.org/skyfire.ico > > This problem w

Re: [FFmpeg-devel] [PATCH] lavf/mp3: Properly check return values of seeks and reads while reading the header

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 17:32:36 + Derek Buitenhuis wrote: > On 2/26/2016 5:30 PM, wm4 wrote: > > The ffio_ensure_seekback() checks are probably unnecessary. > > I can remove them if you prefer. Should I? I'd vote yet, but that's just my personal opinion. > &

Re: [FFmpeg-devel] [PATCH] Document and validate AVFrame plane pointers.

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 22:25:21 +0100 Reimar Döffinger wrote: > Check that the required plane pointers and only > those are set up. > > Signed-off-by: Reimar Döffinger > --- > libavcodec/utils.c | 14 ++ > libavutil/frame.h | 3 +++ > 2 files changed, 17 insertions(+) > > diff --gi

[FFmpeg-devel] [PATCH] vf_copy: exclude hwaccel formats

2016-02-26 Thread wm4
This does obviously not work. (Additions based on vf_crop.c.) --- libavfilter/vf_copy.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/libavfilter/vf_copy.c b/libavfilter/vf_copy.c index fb9a906..b0159cf 100644 --- a/libavfilter/vf_copy.c +++ b/libavfilter/vf_copy.c @@ -2

Re: [FFmpeg-devel] [PATCH] Document and validate AVFrame plane pointers.

2016-02-26 Thread wm4
On Fri, 26 Feb 2016 22:59:17 +0100 Reimar Döffinger wrote: > On Fri, Feb 26, 2016 at 10:39:41PM +0100, wm4 wrote: > > On Fri, 26 Feb 2016 22:25:21 +0100 > > Reimar Döffinger wrote: > > > > > Check that the required plane pointers and only > > > those

Re: [FFmpeg-devel] [PATCH] Document and validate AVFrame plane pointers.

2016-02-27 Thread wm4
On Sat, 27 Feb 2016 12:54:50 +0100 Reimar Döffinger wrote: > On Fri, Feb 26, 2016 at 11:41:41PM +0100, wm4 wrote: > > On Fri, 26 Feb 2016 22:59:17 +0100 > > Reimar Döffinger wrote: > > > > Can you move it to a separate function? Say, ff_verify_frame(). This

Re: [FFmpeg-devel] [PATCH] Document and validate AVFrame plane pointers.

2016-02-27 Thread wm4
On Sat, 27 Feb 2016 13:01:40 +0100 Reimar Döffinger wrote: > Check that the required plane pointers and only > those are set up. > Currently does not enforce anything for the palette > pointer of pseudopal formats as I am unsure about the > requirements. > > Signed-off-by: Reimar Döffinger > --

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/utils: Warn if a encoder is missing the pix_fmts or sample_fmts list

2016-02-27 Thread wm4
On Sun, 28 Feb 2016 02:14:20 +0100 Michael Niedermayer wrote: > On Sun, Feb 28, 2016 at 01:26:16AM +0100, Hendrik Leppkes wrote: > > On Sun, Feb 28, 2016 at 12:17 AM, Michael Niedermayer > > wrote: > > > This would require listing supported formats for rawvideo and > > > wrapped_avframe > > >

Re: [FFmpeg-devel] [PATCH 2/3] avformat: use ff_standardize_creation_time for formats writing all format string metadata

2016-02-28 Thread wm4
On Sun, 28 Feb 2016 14:28:20 +0100 Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavformat/apetag.c| 1 + > libavformat/cafenc.c| 1 + > libavformat/flvenc.c| 1 + > libavformat/id3v2enc.c | 1 + > libavformat/lrcenc.c| 1 + > libavformat/nutenc.c| 1 + > li

Re: [FFmpeg-devel] [PATCH v7] VideoToolbox H.264 Encoder

2016-02-29 Thread wm4
On Mon, 29 Feb 2016 23:14:17 +0800 Rick Kern wrote: > Autodetected by default. Encode using -codec:v vtenc. > > Signed-off-by: Rick Kern > --- OK, autodetection seems to work properly on non-OSX too. I'll apply in 18 hours or so. ___ ffmpeg-devel mai

Re: [FFmpeg-devel] [PATCH v8] VideoToolbox H.264 Encoder

2016-02-29 Thread wm4
On Tue, 1 Mar 2016 01:30:58 +0800 Rick Kern wrote: > Autodetected by default. Encode using -codec:v vtenc. > > Signed-off-by: Rick Kern > --- > MAINTAINERS|1 + > configure | 19 + > libavcodec/Makefile|1 + > libavcodec/allcodecs.c |1 + > libavcode

Re: [FFmpeg-devel] [PATCH v8] VideoToolbox H.264 Encoder

2016-02-29 Thread wm4
On Mon, 29 Feb 2016 20:07:15 +0100 Clément Bœsch wrote: > On Mon, Feb 29, 2016 at 06:57:15PM +0100, wm4 wrote: > [...] > > > +static bool get_vt_profile_level(AVCodecContext *avctx, > > > + CFStringRef*profile_level_val) > > >

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 11:31:18 +0100 Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 01:40:53PM +0800, Rick Kern wrote: > > Autodetected by default. Encode using -codec:v vtenc. > [...] > > +static void set_async_error(VTEncContext *vtctx, int err) > > +{ > > +BufNode *info; > > + > > +

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 11:31:18 +0100 Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 01:40:53PM +0800, Rick Kern wrote: > > Autodetected by default. Encode using -codec:v vtenc. > [...] > > +static void set_async_error(VTEncContext *vtctx, int err) > > +{ > > +BufNode *info; > > + > > +

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread wm4
On Tue, 01 Mar 2016 14:57:29 + Timothy Gu wrote: > Hi, > > On Mon, Feb 29, 2016 at 9:42 PM Rick Kern wrote: > > > Autodetected by default. Encode using -codec:v vtenc. > > > > Signed-off-by: Rick Kern > > --- > > MAINTAINERS|1 + > > configure | 19 + > > l

[FFmpeg-devel] [PATCH 0/6] Add decoding/encoding API with decoupled input/output

2016-03-01 Thread wm4
This has been often discussed, and everyone agreed it was a good idea. Here are patches which add such an API, but without making proper use of it yet. wm4 (6): lavu: improve documentation of some AVFrame functions lavc: factor apply_param_change() AV_EF_EXPLODE handling lavc: introduce a

[FFmpeg-devel] [PATCH 3/6] lavc: introduce a new decoding/encoding API with decoupled input/output

2016-03-01 Thread wm4
Until now, the decoding API was restricted to outputting 0 or 1 frames per input packet. It also enforces a somewhat rigid dataflow in general. This new API seeks to relax these restrictions by decoupling input and output. Instead of doing a single call on each decode step, which may consume the p

[FFmpeg-devel] [PATCH 1/6] lavu: improve documentation of some AVFrame functions

2016-03-01 Thread wm4
--- libavutil/frame.h | 12 1 file changed, 12 insertions(+) diff --git a/libavutil/frame.h b/libavutil/frame.h index 76a8123..2d6299b 100644 --- a/libavutil/frame.h +++ b/libavutil/frame.h @@ -591,6 +591,10 @@ void av_frame_free(AVFrame **frame); * If src is not reference counted,

[FFmpeg-devel] [PATCH 2/6] lavc: factor apply_param_change() AV_EF_EXPLODE handling

2016-03-01 Thread wm4
Remove the duplicated code for handling failure of apply_param_change(). --- libavcodec/utils.c | 34 +++--- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index f435588..86236f8 100644 --- a/libavcodec/utils.c ++

[FFmpeg-devel] [PATCH 5/6] ffmpeg: use new decode API

2016-03-01 Thread wm4
--- Unfortunately requires a hack to make FATE pass: ffmpeg.c will pass a flush AVPacket (data/size are null) with the dts field set, and will expect to get the dts back with the flushed frame. Should probably be fixed somehow. I'm not even sure if this is valid API usage (with the old API). --- f

[FFmpeg-devel] [PATCH 4/6] ffmpeg: remove sub-frame warning

2016-03-01 Thread wm4
It's not practical to keep this with the new decode API. --- ffmpeg.c | 7 --- ffmpeg.h | 1 - 2 files changed, 8 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d148588..f24ee41 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -2313,13 +2313,6 @@ static int process_input_packet(InputStream *ist,

[FFmpeg-devel] [PATCH 6/6] ffmpeg: use new encode API

2016-03-01 Thread wm4
--- I'm wondering if the ost->sync_opts++; and ost->frame_number++; lines in do_video_out() should be moved into the loop. --- ffmpeg.c | 71 +--- 1 file changed, 46 insertions(+), 25 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 5

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 19:52:08 +0100 Matthieu Bouron wrote: > On Fri, Feb 26, 2016 at 05:36:40PM +0100, Matthieu Bouron wrote: > > On Fri, Feb 26, 2016 at 4:41 PM, Matthieu Bouron > > wrote: > > > > > On Mon, Feb 22, 2016 at 12:20:35PM +0100, Matthieu Bouron wrote: > > > > From: Matthieu Bouro

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 20:10:50 +0100 Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 07:56:30PM +0100, wm4 wrote: > > On Tue, 1 Mar 2016 19:52:08 +0100 > > Matthieu Bouron wrote: > > > > > On Fri, Feb 26, 2016 at 05:36:40PM +0100, Matthieu Bouron wrote: > &

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 20:33:14 +0100 Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 08:24:27PM +0100, Matthieu Bouron wrote: > > On Tue, Mar 01, 2016 at 08:13:48PM +0100, wm4 wrote: > > > On Tue, 1 Mar 2016 20:10:50 +0100 > > > Matthieu Bouron wrote: > > >

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 20:57:12 +0100 Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 08:55:23PM +0100, Matthieu Bouron wrote: > > On Tue, Mar 01, 2016 at 08:38:04PM +0100, wm4 wrote: > > > On Tue, 1 Mar 2016 20:33:14 +0100 > > > Matthieu Bouron wrote: > > >

Re: [FFmpeg-devel] [PATCH v10] VideoToolbox H.264 Encoder

2016-03-02 Thread wm4
On Wed, 2 Mar 2016 20:11:40 +0800 Rick Kern wrote: > Autodetected by default. Encode using -codec:v h264_videotoolbox. > > Signed-off-by: Rick Kern > --- Applied. Thanks, and sorry for the delays/revisions. ___ ffmpeg-devel mailing list ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] lavf: allow BSFs to drop packets.

2016-03-02 Thread wm4
On Wed, 2 Mar 2016 15:48:11 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Mon, Feb 29, 2016 at 10:32 AM, Ronald S. Bultje > wrote: > > > If pkt->size == 0 && pkt->side_data_elems == 0 after bsf->filter() > > returns, the packet is considered dropped. > > > Ping? > (Personally I'm not so f

Re: [FFmpeg-devel] [PATCH] Use matroska TrackNumber for populating AVStream::id

2016-03-03 Thread wm4
On Wed, 2 Mar 2016 15:29:21 -0800 Sergey Volk wrote: > As far as I can see FFmpeg currently doesn't set AVStream::id for > matroska/webm streams. I think we could use either MatroskaTrack::num > (TrackNumber) or MatroskaTrack::uid (TrackUID) for that. > I have found a few discussions claiming tha

Re: [FFmpeg-devel] [PATCH] version.sh: Always use latest tag for generated version number

2016-03-03 Thread wm4
On Fri, 4 Mar 2016 00:55:35 + (UTC) Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > > > Of course, this argument operates on the premise that > > making things easier for users is of utmost concern > > for us. Please inspire me if this is not the case. > > On the contrary, I

Re: [FFmpeg-devel] [PATCH] version.sh: Always use latest tag for generated version number

2016-03-03 Thread wm4
On Fri, 4 Mar 2016 08:47:14 +0100 Thilo Borgmann wrote: > Am 04.03.16 um 08:23 schrieb wm4: > > On Fri, 4 Mar 2016 00:55:35 + (UTC) > > Carl Eugen Hoyos wrote: > > > >> Timothy Gu gmail.com> writes: > >> > >>> Of course, this argu

Re: [FFmpeg-devel] [PATCH] avcodec: Extend fft to size 2^17

2016-03-04 Thread wm4
On Fri, 4 Mar 2016 13:12:45 +0100 Paul B Mahol wrote: > On 3/4/16, Michael Niedermayer wrote: > > On Fri, Mar 04, 2016 at 11:21:23AM +0100, Paul B Mahol wrote: > >> On 3/4/16, Michael Niedermayer wrote: > >> > Hi > >> > > >> > patch to extend fft is attached (my git-send email atm doesnt wo

Re: [FFmpeg-devel] [PATCH 1/2] lavf: allow BSFs to drop packets.

2016-03-04 Thread wm4
On Fri, 4 Mar 2016 11:24:00 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Wed, Mar 2, 2016 at 4:11 PM, Ronald S. Bultje wrote: > > > Hi, > > > > On Wed, Mar 2, 2016 at 3:51 PM, wm4 wrote: > > > >> On Wed, 2 Mar 2016 15:48:11

Re: [FFmpeg-devel] [PATCH] lavf/dump.c: Print mastering display metadata

2016-03-04 Thread wm4
On Fri, 4 Mar 2016 23:15:52 +0100 Michael Niedermayer wrote: > On Mon, Feb 29, 2016 at 05:20:17PM -0800, Neil Birkbeck wrote: > > Signed-off-by: Neil Birkbeck > > --- > > libavformat/dump.c | 21 + > > 1 file changed, 21 insertions(+) > > applied > > thanks > > [...] T

Re: [FFmpeg-devel] [PATCH] lavf/mp3dec: avoid printing useless message in default log level

2016-03-08 Thread wm4
On Tue, 8 Mar 2016 15:36:06 +0100 Moritz Barsnick wrote: > "Skipping 0 bytes of junk" is useless to the user, and essentially > indicates a NOP. At 0 bytes, this message is now pushed back to > the debug log level. > > Signed-off-by: Moritz Barsnick > --- > libavformat/mp3dec.c | 2 +- > 1 fi

Re: [FFmpeg-devel] [PATCH v2] lavf/mp3dec: avoid printing useless message in default log level

2016-03-08 Thread wm4
On Tue, 8 Mar 2016 16:54:42 +0100 Moritz Barsnick wrote: > "Skipping 0 bytes of junk" is useless to the user, and essentially > indicates a NOP. At 0 bytes, this message is now pushed back to > the verbose log level. > > Signed-off-by: Moritz Barsnick > --- > libavformat/mp3dec.c | 2 +- > 1

Re: [FFmpeg-devel] [PATCHv2 2/3] avformat/utils: increase detected start_time with skip_samples

2016-03-08 Thread wm4
On Tue, 8 Mar 2016 21:29:52 +0100 Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavformat/utils.c | 10 -- > tests/ref/fate/gapless2-ipod-aac1 | 72 > +++ > tests/ref/fate/gapless2-ipod-aac2 | 72 > +++-

Re: [FFmpeg-devel] [PATCHv2 2/3] avformat/utils: increase detected start_time with skip_samples

2016-03-09 Thread wm4
On Tue, 8 Mar 2016 23:44:13 +0100 (CET) Marton Balint wrote: > On Tue, 8 Mar 2016, Hendrik Leppkes wrote: > > > On Tue, Mar 8, 2016 at 10:54 PM, Marton Balint wrote: > >> > >> On Tue, 8 Mar 2016, wm4 wrote: > >> > >>> On Tue, 8

Re: [FFmpeg-devel] [PATCHv2 2/3] avformat/utils: increase detected start_time with skip_samples

2016-03-10 Thread wm4
On Wed, 9 Mar 2016 23:02:14 +0100 (CET) Marton Balint wrote: > On Wed, 9 Mar 2016, wm4 wrote: > > On Tue, 8 Mar 2016 23:44:13 +0100 (CET) > > Marton Balint wrote: > > > >> On Tue, 8 Mar 2016, Hendrik Leppkes wrote: > >> > >> > On

Re: [FFmpeg-devel] [PATCH 0/3] showcqt x86 optimization using intrinsic

2016-03-10 Thread wm4
On Thu, 10 Mar 2016 16:53:12 +0700 Muhammad Faiz wrote: > I use intrinsic because writing asm using nasm or inline asm > is difficult task for me. > > [PATCH 1/3] configure: add x86 intrinsic support > [PATCH 2/3] avfilter/avf_showcqt: cqt_calc x86 optimization > [PATCH 3/3] avfilter/avf_showcqt

Re: [FFmpeg-devel] [PATCHv2 2/3] avformat/utils: increase detected start_time with skip_samples

2016-03-10 Thread wm4
On Thu, 10 Mar 2016 20:34:57 +0100 (CET) Marton Balint wrote: > On Thu, 10 Mar 2016, wm4 wrote: > > > On Wed, 9 Mar 2016 23:02:14 +0100 (CET) > > Marton Balint wrote: > > > > [...] > > > Anyway, with just an audio track, adjusting start_time is rathe

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-10 Thread wm4
On Thu, 10 Mar 2016 18:12:57 +0100 Gerion Entrup wrote: > On Dienstag, 8. März 2016 20:42:39 CET Clément Bœsch wrote: > > On Tue, Mar 08, 2016 at 06:21:12PM +0100, Gerion Entrup wrote: > > > Hello, > > > > > > > Hi, > > > > > my own ideas seems not to be suitable for GSoC, so I looked ag

Re: [FFmpeg-devel] [PATCH]lavf/mkvenc: Set bit_depth for lossless audio codecs

2016-03-12 Thread wm4
On Sat, 12 Mar 2016 16:51:18 +0100 Carl Eugen Hoyos wrote: > Hi! > > Attached patch makes the output of the mkv muxer more similar to what > mkvmerge does. > May fix ticket #5332. > > Please comment, Carl Eugen Are you sure this is correct, and _why_? _

Re: [FFmpeg-devel] [PATCH 1/2] lavu/dict: Add new flag to allow multiple equal keys.

2016-03-13 Thread wm4
On Sat, 12 Mar 2016 15:13:21 +0100 Thilo Borgmann wrote: > From a1d9ce388c69eabb256e6b351c2acd36d7f4076e Mon Sep 17 00:00:00 2001 > From: Thilo Borgmann > Date: Sat, 12 Mar 2016 14:52:17 +0100 > Subject: [PATCH 1/2] lavu/dict: Add new flag to allow multiple equal keys. > > --- > libavutil/dict

<    4   5   6   7   8   9   10   11   12   13   >