Re: [FFmpeg-devel] APNG encoder can work incorrectly

2016-04-01 Thread Dmitriy
Yes of course > On 3/31/16, Dmitriy wrote: >> I create a special video. You can download it using the link >> >> https://drive.google.com/file/d/0B8rMLdq9Vq55QmwwcVJDVkZoM1E/view?usp=sharing >> >> Convert it to .apng format with >> ffmpeg -i apng_test.avi apng_test.apng >> >> The result file will

Re: [FFmpeg-devel] [PATCH]Addition of MLP encoder

2016-04-01 Thread Disha Singh
Somehow I am not getting these warnings when I compile. (not all) Should I remove the functions which are defined but not used. They may be needed sometime later. I have removed the error in line 1888. I do not get any error in line 2462. What should I add in my code that the compiler you used does

Re: [FFmpeg-devel] [PATCH 1/2] lavc/audiotoolboxdec: fix a number of config and timestamp issues

2016-04-01 Thread pon pon
it seems you are correct. i can't playback the ac3 files with afplay and afconvert, though @constant kAudioFormatAC3 is described in CoreAudioType.h and i can playback them with quicktime player(not 7 and without perian). the results are attached. i did some tests. decoding test(gsm_ms_at and i

Re: [FFmpeg-devel] [PATCH 2/6] lavc/audiotoolboxenc: fix a number of config issues

2016-04-01 Thread pon pon
i did some tests inclluding profiles under osx1095. encoding test 51ch->2ch and 51ch->51ch, aac_at, alac_at, pcm_alaw_at, pcm_mulaw_at there is message "Trying to remove 2048 samples, but the queue is empty" in encoding aac_at with profile aac_he and 51ch->2ch and 51ch->51ch and encoding aac_at wi

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

2016-04-01 Thread wm4
On Wed, 23 Mar 2016 13:47:08 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > + * warning: if frame already has been allocated, calling this function will > > + * leak memory. In addition, undefined behavior can occur in certain > > + * cases. > > If this is co

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

2016-04-01 Thread wm4
On Wed, 23 Mar 2016 16:34:57 +0100 Michael Niedermayer wrote: > On Wed, Mar 23, 2016 at 02:02:09PM +0100, wm4 wrote: > > Remove the duplicated code for handling failure of apply_param_change(). > > --- > > libavcodec/utils.c | 34 +++--- > > 1 file changed, 19 inserti

[FFmpeg-devel] [PATCH 1/2] avcodec: properly initialize AVCodecParameters profile/level

2016-04-01 Thread Hendrik Leppkes
--- libavcodec/utils.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index f991135..df30d57 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -3713,6 +3713,8 @@ static void codec_parameters_reset(AVCodecParameters *par) par->color_spac

[FFmpeg-devel] [PATCH 2/2] exif: take a generic log context

2016-04-01 Thread Hendrik Leppkes
The AVCodecContext is only used for logging, so instead take any valid log context. This allows reusing the exif functions more easily in avformat. --- libavcodec/exif.c | 16 libavcodec/exif.h | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/libavcodec/exi

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: properly initialize AVCodecParameters profile/level

2016-04-01 Thread wm4
On Fri, 1 Apr 2016 13:13:02 +0200 Hendrik Leppkes wrote: > --- > libavcodec/utils.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index f991135..df30d57 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/utils.c > @@ -3713,6 +3713,8 @@ s

Re: [FFmpeg-devel] [PATCH 2/2] exif: take a generic log context

2016-04-01 Thread wm4
On Fri, 1 Apr 2016 13:13:03 +0200 Hendrik Leppkes wrote: > The AVCodecContext is only used for logging, so instead take any valid log > context. > This allows reusing the exif functions more easily in avformat. > --- > libavcodec/exif.c | 16 > libavcodec/exif.h | 2 +- > 2 f

Re: [FFmpeg-devel] [PATCH 2/2] exif: take a generic log context

2016-04-01 Thread Thilo Borgmann
Am 01.04.16 um 13:13 schrieb Hendrik Leppkes: > The AVCodecContext is only used for logging, so instead take any valid log > context. > This allows reusing the exif functions more easily in avformat. > --- > libavcodec/exif.c | 16 > libavcodec/exif.h | 2 +- > 2 files changed,

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: properly initialize AVCodecParameters profile/level

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 01:13:02PM +0200, Hendrik Leppkes wrote: > --- > libavcodec/utils.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index f991135..df30d57 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/utils.c > @@ -3713,6 +3713,8

Re: [FFmpeg-devel] [PATCH]Addition of MLP encoder

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 02:27:43PM +0530, Disha Singh wrote: > Somehow I am not getting these warnings when I compile. (not all) > Should I remove the functions which are defined but not used. They may be > needed sometime later. > I have removed the error in line 1888. theres now an error in line

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread wm4
On Wed, 23 Mar 2016 22:15:12 +0100 Michael Niedermayer wrote: > On Wed, Mar 23, 2016 at 09:51:46PM +0100, wm4 wrote: > > On Wed, 23 Mar 2016 21:36:35 +0100 > > Michael Niedermayer wrote: > > > > > On Wed, Mar 23, 2016 at 08:44:41PM +0100, wm4 wrote: > > > > On Wed, 23 Mar 2016 18:37:25 +010

Re: [FFmpeg-devel] [PATCH]Addition of MLP encoder

2016-04-01 Thread Ronald S. Bultje
Hi, On Fri, Apr 1, 2016 at 4:57 AM, Disha Singh wrote: > Somehow I am not getting these warnings when I compile. (not all) OK, so we should stop here and fix this. Why is the encoder not being compiled for you? This back-and-forth on compiler warnings/errors is ridiculous. Please figure out ho

Re: [FFmpeg-devel] swscale/arm/yuv2rgb: make the code bitexact with its aarch64 counter part

2016-04-01 Thread Matthieu Bouron
On Mon, Mar 28, 2016 at 9:12 PM, Matthieu Bouron wrote: > > > On Sun, Mar 27, 2016 at 5:58 PM, Matthieu Bouron < > matthieu.bou...@gmail.com> wrote: > >> >> >> On Fri, Mar 25, 2016 at 11:45 PM, Matthieu Bouron < >> matthieu.bou...@gmail.com> wrote: >> >>> The following patchset aims to make bitex

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread Paul B Mahol
Dana 1. 4. 2016. 15:43 osoba "wm4" napisala je: > > On Wed, 23 Mar 2016 22:15:12 +0100 > Michael Niedermayer wrote: > > > On Wed, Mar 23, 2016 at 09:51:46PM +0100, wm4 wrote: > > > On Wed, 23 Mar 2016 21:36:35 +0100 > > > Michael Niedermayer wrote: > > > > > > > On Wed, Mar 23, 2016 at 08:44:41P

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread wm4
On Fri, 1 Apr 2016 16:44:42 +0200 Paul B Mahol wrote: > It should set it because it is not sane format. I mean, why do we want to keep this message, if it obviously didn't help with anything? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http:/

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 04:53:55PM +0200, wm4 wrote: > On Fri, 1 Apr 2016 16:44:42 +0200 > Paul B Mahol wrote: > > > It should set it because it is not sane format. > > I mean, why do we want to keep this message, if it obviously didn't > help with anything? it is useful for debuging and to pre

Re: [FFmpeg-devel] swscale/arm/yuv2rgb: make the code bitexact with its aarch64 counter part

2016-04-01 Thread Matthieu Bouron
On Fri, Apr 1, 2016 at 4:15 PM, Matthieu Bouron wrote: > > > On Mon, Mar 28, 2016 at 9:12 PM, Matthieu Bouron < > matthieu.bou...@gmail.com> wrote: > >> >> >> On Sun, Mar 27, 2016 at 5:58 PM, Matthieu Bouron < >> matthieu.bou...@gmail.com> wrote: >> >>> >>> >>> On Fri, Mar 25, 2016 at 11:45 PM, M

Re: [FFmpeg-devel] [PATCH] add attribution to OpenJPEG code in FFmpeg encoder j2kenc.c file

2016-04-01 Thread Michael Niedermayer
On Thu, Mar 31, 2016 at 04:06:01PM -0400, Aaron Boxer wrote: > Hi Folks, > > Here is a patch to reference the OpenJPEG license from j2kenc.c, > as per earlier discussion. > > Cheers, > Aaron > j2kenc.c | 38 ++ > 1 file changed, 38 insertions(+) > 67a21d362

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 05:27:11PM +0200, Michael Niedermayer wrote: > On Fri, Apr 01, 2016 at 04:53:55PM +0200, wm4 wrote: > > On Fri, 1 Apr 2016 16:44:42 +0200 > > Paul B Mahol wrote: > > > > > It should set it because it is not sane format. > > > > I mean, why do we want to keep this message,

Re: [FFmpeg-devel] [PATCH v2 7/8] lavf: use new decode API

2016-04-01 Thread Michael Niedermayer
On Wed, Mar 23, 2016 at 02:02:14PM +0100, wm4 wrote: > --- > libavformat/utils.c | 29 - > 1 file changed, 12 insertions(+), 17 deletions(-) btw, ill test 7 & 8 when 1-6 are working/updated. Or should i try to test these already ? [...] -- Michael GnuPG fingerpri

Re: [FFmpeg-devel] [PATCH 1/2] swscale: fix bt709 yuv2rgb coefficients.

2016-04-01 Thread Michael Niedermayer
On Thu, Mar 31, 2016 at 04:01:16PM -0400, Ronald S. Bultje wrote: > They were derived from incorrect values cr=0.2125 (should be cr=0.2126) > and cb=0.0721 (should be cb=0.0722). > --- > libswscale/yuv2rgb.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libswscale/y

Re: [FFmpeg-devel] [PATCH v2 7/8] lavf: use new decode API

2016-04-01 Thread wm4
On Fri, 1 Apr 2016 18:18:17 +0200 Michael Niedermayer wrote: > On Wed, Mar 23, 2016 at 02:02:14PM +0100, wm4 wrote: > > --- > > libavformat/utils.c | 29 - > > 1 file changed, 12 insertions(+), 17 deletions(-) > > btw, ill test 7 & 8 when 1-6 are working/updated. O

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread wm4
On Fri, 1 Apr 2016 17:47:45 +0200 Michael Niedermayer wrote: > On Fri, Apr 01, 2016 at 05:27:11PM +0200, Michael Niedermayer wrote: > > On Fri, Apr 01, 2016 at 04:53:55PM +0200, wm4 wrote: > > > On Fri, 1 Apr 2016 16:44:42 +0200 > > > Paul B Mahol wrote: > > > > > > > It should set it becau

[FFmpeg-devel] [PATCH] According to SMPTE 370M (rev.2002), page 15 : "FS: First/second field flag",

2016-04-01 Thread Александр Слободенюк
quoting: FS indicates a field which is delivered during the field one period (see table 16) now quoting table 16: FF FS Output field --- 1 1Field 1 and field 2 are output in this order (1,2 sequence). --- 1

Re: [FFmpeg-devel] [PATCH] fate: Test for Ticket4560

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 01:02:56AM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate/filter-video.mak |9 ++ > tests/ref/fate/filter-meta-4560-rotate0 | 259 > +++ > 2 files changed, 268 insertions(+) > create

Re: [FFmpeg-devel] [PATCH] fate: Add regression test for Ticket 1239

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 12:34:04AM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate/real.mak |3 ++ > tests/ref/fate/rv20-1239 | 91 > ++ > 2 files changed, 94 insertions(+) > create mode 100644 test

Re: [FFmpeg-devel] [PATCH] fate: add demux test for TS with AC3 (Ticket 4864)

2016-04-01 Thread Michael Niedermayer
On Wed, Mar 30, 2016 at 09:41:10PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate/demux.mak|3 +++ > tests/ref/fate/ts-demux | 23 +++ > 2 files changed, 26 insertions(+) > create mode 100644 tests/ref/fate/ts-demux applie

Re: [FFmpeg-devel] [PATCH v2 7/8] lavf: use new decode API

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 06:51:39PM +0200, wm4 wrote: > On Fri, 1 Apr 2016 18:18:17 +0200 > Michael Niedermayer wrote: > > > On Wed, Mar 23, 2016 at 02:02:14PM +0100, wm4 wrote: > > > --- > > > libavformat/utils.c | 29 - > > > 1 file changed, 12 insertions(+), 17 dele

Re: [FFmpeg-devel] [PATCH v2 4/8] ffmpeg: remove sub-frame warning

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 06:52:53PM +0200, wm4 wrote: > On Fri, 1 Apr 2016 17:47:45 +0200 > Michael Niedermayer wrote: > > > On Fri, Apr 01, 2016 at 05:27:11PM +0200, Michael Niedermayer wrote: > > > On Fri, Apr 01, 2016 at 04:53:55PM +0200, wm4 wrote: > > > > On Fri, 1 Apr 2016 16:44:42 +0200 >

Re: [FFmpeg-devel] [PATCH] lavfi: new colorspace conversion filter.

2016-04-01 Thread Michael Niedermayer
On Thu, Mar 31, 2016 at 08:29:37PM -0400, Ronald S. Bultje wrote: > The intent here is similar to colormatrix, but it's LGPLv2.1-or-later > (instead of GPLv2.0) and supports gamma/chromaticity correction. > --- > doc/filters.texi | 183 +++ > libavfilter/Makefile

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Fix crash when closing codec after error

2016-04-01 Thread Richard Kern
> On Mar 24, 2016, at 12:46 PM, Richard Kern wrote: > >> >> On Mar 25, 2016, at 12:37 AM, wm4 wrote: >> >> On Mon, 21 Mar 2016 00:55:56 +0800 >> Rick Kern wrote: >> >>> Fixes crash in #5352. VTCompressionSessionInvalidate() crashes if the >>> internal encoder hasn't completed, but hasn't e

Re: [FFmpeg-devel] [PATCH v2] lavc/videotoolboxenc: Workaround encoder error

2016-04-01 Thread Richard Kern
> On Mar 24, 2016, at 1:56 PM, Rick Kern wrote: > > CMVideoFormatDescriptionGetH264ParameterSetAtIndex() fails on some > hardware/OS versions when retrieving the parameter set count alone. > > Signed-off-by: Rick Kern > --- > libavcodec/videotoolboxenc.c | 43 -

Re: [FFmpeg-devel] codecpar regressions

2016-04-01 Thread Michael Niedermayer
On Fri, Apr 01, 2016 at 01:53:45AM +0200, Michael Niedermayer wrote: > Hi > > from #ffmpeg-devel > im not doing any fixes today though, im beat. > ... > michaelni, if im not around mail me them, or putthem on github or > something > > i found another regression, thought "here" is a good "somet

[FFmpeg-devel] [PATCH] fate: Add test for Ticket 2397

2016-04-01 Thread Michael Niedermayer
If someone can create a smaller test file (its 2.5Mb) that still has the same coverage that is welcome. otherwise ill upload the sample attached to the ticket Signed-off-by: Michael Niedermayer --- tests/fate/filter-video.mak|4 + tests/filtergraphs/overlay-2397|2 + tests/r

Re: [FFmpeg-devel] [PATCH] avformat/framehash: Add more information to the output

2016-04-01 Thread James Almer
On 3/30/2016 6:16 PM, wm4 wrote: > Please wait with this after TEP2 is merged. The TEP2 wip branch needs to be rebased to current master before it can be merged, and the amount of ref files it modified was relatively small, so this commit shouldn't generate many conflicts. This adds a bunch of new

Re: [FFmpeg-devel] [PATCH] avformat/framehash: Add more information to the output

2016-04-01 Thread James Almer
On 3/30/2016 5:54 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/framehash.c| 13 + Since you're pretty much rewriting every ref file, this may be a good chance to turn framecrc into an alias for framemd5, hardcodi

Re: [FFmpeg-devel] [PATCH] According to SMPTE 370M (rev.2002), page 15 : "FS: First/second field flag",

2016-04-01 Thread Carl Eugen Hoyos
Александр Слободенюк bramtech.ru> writes: > I've found this case from practice, where DVCProHD coded > video (by Matrox device) with image, recorded as "top_field_first" > got AVFrame->top_field_first == 0 Please upload a sample that shows this issue. Carl Eugen ___

[FFmpeg-devel] Seeking dev for simple loudness app

2016-04-01 Thread Daniel J. Lewis
Hi! I'm Daniel J. Lewis, host of The Audacity to Podcast. I'm interested in hiring a developer to create a simple cross-platform GUI that uses ffmpeg for measuring integrated loudness, loudness range, and true peak. The command is simple: ffmpeg -nostats -i SOURCE-FILE.wav -filter_complex ebur12