Re: [FFmpeg-devel] [PATCH] lavc/lpc: exploit even symmetry of window function

2016-03-08 Thread Reimar Döffinger
On 09.03.2016, at 04:16, Ganesh Ajjanagadde wrote: > Yields 2x improvement in function performance, and boosts aac encoding > speed by ~ 4% overall. Sample benchmark (Haswell+GCC under -march=native): > after: > ffmpeg -i sin.flac -acodec aac -y sin_new.aac 5.22s user 0.03s system 105% > cpu 4.

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-08 Thread Reimar Döffinger
On 08.03.2016, at 04:48, Ganesh Ajjanagadde wrote: > +nzl += expf(logf(s / ethresh) * nzslope); Shouldn't log2f/exp2f be faster? log2f at least has CPU support on x86 AFAICT. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

[FFmpeg-devel] [PATCH] lavc/lpc: exploit even symmetry of window function

2016-03-08 Thread Ganesh Ajjanagadde
Yields 2x improvement in function performance, and boosts aac encoding speed by ~ 4% overall. Sample benchmark (Haswell+GCC under -march=native): after: ffmpeg -i sin.flac -acodec aac -y sin_new.aac 5.22s user 0.03s system 105% cpu 4.970 total before: ffmpeg -i sin.flac -acodec aac -y sin_new.aa

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-08 Thread Ganesh Ajjanagadde
On Tue, Mar 8, 2016 at 8:02 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Mar 7, 2016 at 10:48 PM, Ganesh Ajjanagadde > wrote: >> >> This is ~2x faster for y not an integer on Haswell+GCC, and should >> generally be faster due to the fact that anyway powf essentially does >> this under the hood.

Re: [FFmpeg-devel] [PATCH] aacenc_utils: unroll loops to allow compiler to use SIMD.

2016-03-08 Thread Ganesh Ajjanagadde
On Tue, Mar 8, 2016 at 2:30 PM, Reimar Döffinger wrote: > On Mon, Mar 07, 2016 at 10:50:53PM -0500, Ganesh Ajjanagadde wrote: >> On Mon, Mar 7, 2016 at 2:54 AM, Reimar Döffinger >> wrote: >> >> Can you be more specific, and are you sure about this? >> > >> > Just run your favourite performance an

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-08 Thread Ganesh Ajjanagadde
On Tue, Mar 8, 2016 at 9:10 PM, Rostislav Pehlivanov wrote: > On 9 March 2016 at 01:02, Ronald S. Bultje wrote: > >> Hi, >> >> On Mon, Mar 7, 2016 at 10:48 PM, Ganesh Ajjanagadde >> wrote: >> >> > This is ~2x faster for y not an integer on Haswell+GCC, and should >> > generally be faster due to

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-08 Thread Rostislav Pehlivanov
On 9 March 2016 at 01:02, Ronald S. Bultje wrote: > Hi, > > On Mon, Mar 7, 2016 at 10:48 PM, Ganesh Ajjanagadde > wrote: > > > This is ~2x faster for y not an integer on Haswell+GCC, and should > > generally be faster due to the fact that anyway powf essentially does > > this under the hood. > T

Re: [FFmpeg-devel] [PATCH] lavf/avidec: Skip xxpc entries in index

2016-03-08 Thread Mats Peterson
On 03/09/2016 02:54 AM, Mats Peterson wrote: Interim fix of the parsing of idx1 indexes with 'xxpc' (palette change) entries. Indices, not indexes. Mats ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-

Re: [FFmpeg-devel] [PATCH] avutil/frame: Assert that width/height/channels is 0 for the destination of av_frame*_ref()

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 10:24:55PM -0300, Gonzalo wrote: > > > El 08/03/16 a las 17:14, Michael Niedermayer escribió: > >diff --git a/libavutil/frame.c b/libavutil/frame.c > >index 5607206..dde32b0 100644 > >--- a/libavutil/frame.c > >+++ b/libavutil/frame.c > >@@ -375,6 +375,9 @@ int av_frame_re

[FFmpeg-devel] [PATCH] lavf/avidec: Skip xxpc entries in index

2016-03-08 Thread Mats Peterson
Interim fix of the parsing of idx1 indexes with 'xxpc' (palette change) entries. Implementation of 'xxpc' index entry storage for seeking will come in the future (provided I can manage it). Test file TOON.AVI from King's Quest VI with indexed 'xxpc' chunks which would previously cause "strea

Re: [FFmpeg-devel] [PATCH] Add tests for functions in hash.c

2016-03-08 Thread James Almer
On 3/8/2016 2:21 AM, NagaChaitanya Vellanki wrote: > --- > libavutil/Makefile | 1 + > libavutil/hash.c | 42 ++ > tests/fate/libavutil.mak | 4 > tests/ref/fate/hash | 45 + > 4 files cha

Re: [FFmpeg-devel] Support seek in encrypted MP4

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 09:13:30PM +, Eran Kornblau wrote: > > > > check tests/fate/seek.mak > > also you can pass parameters to seek-test see fate-seek-cache-pipe > > i dont know if there is anything else special needed for this > > > Thanks Michael, I looked at this some more, and there's on

Re: [FFmpeg-devel] [PATCH] avutil/frame: Assert that width/height/channels is 0 for the destination of av_frame*_ref()

2016-03-08 Thread Gonzalo
El 08/03/16 a las 17:14, Michael Niedermayer escribió: diff --git a/libavutil/frame.c b/libavutil/frame.c index 5607206..dde32b0 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -375,6 +375,9 @@ int av_frame_ref(AVFrame *dst, const AVFrame *src) { int i, ret = 0; +av_ass

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-08 Thread Ronald S. Bultje
Hi, On Mon, Mar 7, 2016 at 10:48 PM, Ganesh Ajjanagadde wrote: > This is ~2x faster for y not an integer on Haswell+GCC, and should > generally be faster due to the fact that anyway powf essentially does > this under the hood. > > Note that there are some accuracy differences, that should genera

Re: [FFmpeg-devel] [PATCH] Add tests for functions in hash.c

2016-03-08 Thread NagaChaitanya Vellanki
Please review the latest patch. the DST_BUF_SIZE is not AV_HASH_MAX_SIZE * 8 since AV_HASH_MAX_SIZE is 64. Thank you, Naga On Sun, Mar 6, 2016 at 9:45 PM, James Almer wrote: > On 3/7/2016 12:26 AM, NagaChaitanya Vellanki wrote: > > --- > > libavutil/Makefile | 1 + > > libavutil/hash.c

Re: [FFmpeg-devel] [PATCH] aacenc_utils: Use temporary variable.

2016-03-08 Thread Reimar Döffinger
On Tue, Mar 08, 2016 at 10:45:41PM +, Rostislav Pehlivanov wrote: > On 6 March 2016 at 20:26, Reimar Döffinger wrote: > > > This ensures gcc does not create unnecessary > > loads or stores and possibly even does not vectorize > > the negation. > > Speeds up mp3 to aac transcoding with default

Re: [FFmpeg-devel] [PATCH] aacenc: use generational cache instead of resetting.

2016-03-08 Thread Reimar Döffinger
On Tue, Mar 08, 2016 at 10:40:18PM +, Rostislav Pehlivanov wrote: > On 6 March 2016 at 16:29, Reimar Döffinger wrote: > > > Approximately 11% faster transcoding from mp3 with > > default settings. > > > > Signed-off-by: Reimar Döffinger > > > > Very nice, thanks. > LGTM, feel free to apply

Re: [FFmpeg-devel] [PATCH] aacenc_utils: Use temporary variable.

2016-03-08 Thread Rostislav Pehlivanov
On 6 March 2016 at 20:26, Reimar Döffinger wrote: > This ensures gcc does not create unnecessary > loads or stores and possibly even does not vectorize > the negation. > Speeds up mp3 to aac transcoding with default settings > by 10% when using "gcc (Debian 5.3.1-10) 5.3.1 20160224". > > Signed-o

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

2016-03-08 Thread Marton Balint
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 Mar 2016 21:29:52 +0100 Marton Balint wrote: Signed-off-by: Marton Balint --- libavformat/utils.c | 10 -- tests/ref/fate/gaples

Re: [FFmpeg-devel] [PATCH] aacenc: use generational cache instead of resetting.

2016-03-08 Thread Rostislav Pehlivanov
On 6 March 2016 at 16:29, Reimar Döffinger wrote: > Approximately 11% faster transcoding from mp3 with > default settings. > > Signed-off-by: Reimar Döffinger > Very nice, thanks. LGTM, feel free to apply whenever you can. Can confirm I get a speedup and the output is still bit-identical befor

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-08 Thread Rostislav Pehlivanov
On 8 March 2016 at 03:48, Ganesh Ajjanagadde wrote: > This is ~2x faster for y not an integer on Haswell+GCC, and should > generally be faster due to the fact that anyway powf essentially does > this under the hood. > > Note that there are some accuracy differences, that should generally be > neg

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

2016-03-08 Thread Hendrik Leppkes
On Tue, Mar 8, 2016 at 10:54 PM, Marton Balint wrote: > > On Tue, 8 Mar 2016, wm4 wrote: > >> 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 >>>

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-08 Thread Clément Bœsch
On Tue, Mar 08, 2016 at 10:33:44PM +0100, Gerion Entrup wrote: [...] > > For text subtitles, the ass field contains the text in ASS markup: indeed, > > we consider the ASS markup to be the best/least worst superset supporting > > almost every style of every other subtitles formats have, so it's use

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

2016-03-08 Thread Marton Balint
On Tue, 8 Mar 2016, wm4 wrote: 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] Motion interpolation in libavfilter

2016-03-08 Thread Andy Furniss
compn wrote: On Tue, 8 Mar 2016 09:26:22 +0100 Paul B Mahol wrote: On 3/8/16, Subhashish Pradhan wrote: Hello, I am Subhashish Pradhan, a CS undergrad from India. I want to take up the project "Motion interpolation in libavfilter" for GSoC 2016 and I'd like to understand the goals for this

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-08 Thread Gerion Entrup
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, Thank you for your answer. > > > my own ideas seems not to be suitable for GSoC, so I looked again on the > > ideas page, > > because I have high i

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] Support seek in encrypted MP4

2016-03-08 Thread Eran Kornblau
> > check tests/fate/seek.mak > also you can pass parameters to seek-test see fate-seek-cache-pipe > i dont know if there is anything else special needed for this > Thanks Michael, I looked at this some more, and there's one thing that I'm still missing - where do I get the sample encrypted file f

[FFmpeg-devel] Looking for FFmpeg automation specialist

2016-03-08 Thread Dennis P
Looking to fill a 6 month contract position (Toronto, ON CANADA), to assist with design and building of FFmpeg based media transcoding system - thorough understanding of media processing and transcoding optimizations - integrate with Java based automation framework (expand API, errors/status handl

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

2016-03-08 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/utils.c | 10 -- tests/ref/fate/gapless2-ipod-aac1 | 72 +++ tests/ref/fate/gapless2-ipod-aac2 | 72 +++ 3 files changed, 80 insertions(+), 74 deletions(-) diff

[FFmpeg-devel] [PATCHv2 3/3] avformat/mov: read start_pad from edit list start time if codec is aac

2016-03-08 Thread Marton Balint
Related to ticket #2324, #2325. Stream duration still need to be fixed... Signed-off-by: Marton Balint --- libavformat/mov.c | 3 +++ tests/ref/fate/gapless2-ipod-aac1 | 36 ++-- tests/ref/fate/gapless2-ipod-aac2 | 36 ++--

[FFmpeg-devel] [PATCHv2 1/3] tests/gapless: add gapless aac tests

2016-03-08 Thread Marton Balint
Signed-off-by: Marton Balint --- tests/fate-run.sh | 30 ++ tests/fate/gapless.mak| 20 - tests/ref/fate/gapless2-ipod-aac1 | 86 +++ tests/ref/fate/gapless2-ipod-aac2 | 86 +++

Re: [FFmpeg-devel] [PATCH 1/3] tests/gapless: add gapless aac tests

2016-03-08 Thread Marton Balint
On Tue, 8 Mar 2016, Michael Niedermayer wrote: On Tue, Mar 08, 2016 at 01:49:43AM +0100, Marton Balint wrote: Signed-off-by: Marton Balint --- tests/fate-run.sh | 30 + tests/fate/gapless.mak| 20 - tests/ref/fate/gapless2-ipod-aac1 | 88 ++

[FFmpeg-devel] [PATCH] avutil/frame: Assert that width/height/channels is 0 for the destination of av_frame*_ref()

2016-03-08 Thread Michael Niedermayer
This should detect caes where these functions are called in unclean destinations --- libavutil/frame.c |6 ++ 1 file changed, 6 insertions(+) diff --git a/libavutil/frame.c b/libavutil/frame.c index 5607206..dde32b0 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -375,6 +375,9 @

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Ronald S. Bultje
Hi, On Tue, Mar 8, 2016 at 2:03 PM, Carl Eugen Hoyos wrote: > On Tuesday 08 March 2016 07:26:31 pm Ronald S. Bultje wrote: > > On Tue, Mar 8, 2016 at 12:56 PM, Carl Eugen Hoyos wrote: > > > Attached patch fixes ticket #4980 for me. > > > Debugged by Kurosu and Hendrik. > > > > That fixes 444, bu

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-08 Thread Clément Bœsch
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 again on the > ideas page, > because I have high interest to do something for FFmpeg this summer. > > The project, that I find most interesting, unfortuna

Re: [FFmpeg-devel] [PATCH] aacenc_utils: unroll loops to allow compiler to use SIMD.

2016-03-08 Thread Reimar Döffinger
On Mon, Mar 07, 2016 at 10:50:53PM -0500, Ganesh Ajjanagadde wrote: > On Mon, Mar 7, 2016 at 2:54 AM, Reimar Döffinger > wrote: > >> Can you be more specific, and are you sure about this? > > > > Just run your favourite performance analysis tool and you'll see. > > As it is non-inlined libc code I

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Carl Eugen Hoyos
On Tuesday 08 March 2016 07:26:31 pm Ronald S. Bultje wrote: > On Tue, Mar 8, 2016 at 12:56 PM, Carl Eugen Hoyos wrote: > > Attached patch fixes ticket #4980 for me. > > Debugged by Kurosu and Hendrik. > > That fixes 444, but not 422. For 422 (idc_fmt=2), horiz_mult (for l/r) > should be 2, but ver

Re: [FFmpeg-devel] Fwd: Fwd: libavformat/segment : add option to increment timecode

2016-03-08 Thread Stefano Sabatini
On date Sunday 2016-02-28 21:41:32 +0100, Martin Vignali encoded: > 2016-02-23 20:08 GMT+01:00 Martin Vignali : [...] > Hello, > > New patch attached, with the timecode incrementation inside segment_end() > Use this time start_time and end_time (from SegmentListEntry) to calculate > the duration o

Re: [FFmpeg-devel] Motion interpolation in libavfilter

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 07:26:23PM +0100, Michael Niedermayer wrote: > On Tue, Mar 08, 2016 at 01:21:16PM -0500, compn wrote: > > On Tue, 8 Mar 2016 09:26:22 +0100 > > Paul B Mahol wrote: > > > > > On 3/8/16, Subhashish Pradhan wrote: > > > > Hello, > > > > > > > > I am Subhashish Pradhan, a CS

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Ronald S. Bultje
Hi, On Tue, Mar 8, 2016 at 12:56 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4980 for me. > Debugged by Kurosu and Hendrik. That fixes 444, but not 422. For 422 (idc_fmt=2), horiz_mult (for l/r) should be 2, but vert_mult (for t/b) should be 1. Ronald __

Re: [FFmpeg-devel] Motion interpolation in libavfilter

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 01:21:16PM -0500, compn wrote: > On Tue, 8 Mar 2016 09:26:22 +0100 > Paul B Mahol wrote: > > > On 3/8/16, Subhashish Pradhan wrote: > > > Hello, > > > > > > I am Subhashish Pradhan, a CS undergrad from India. I want to take > > > up the project "Motion interpolation in li

Re: [FFmpeg-devel] Motion interpolation in libavfilter

2016-03-08 Thread compn
On Tue, 8 Mar 2016 09:26:22 +0100 Paul B Mahol wrote: > On 3/8/16, Subhashish Pradhan wrote: > > Hello, > > > > I am Subhashish Pradhan, a CS undergrad from India. I want to take > > up the project "Motion interpolation in libavfilter" for GSoC 2016 > > and I'd like to understand the goals for t

[FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4980 for me. Debugged by Kurosu and Hendrik. Please comment, Carl Eugen diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c index 64d6e2f..b4e5c5b 100644 --- a/libavcodec/hevc_ps.c +++ b/libavcodec/hevc_ps.c @@ -856,11 +856,11 @@ int ff_hevc_parse_sps(HEVCSPS

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

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 07:21:36PM +0100, wm4 wrote: > --- > 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 @@ vo

[FFmpeg-devel] "OPAtom misinterpreted as OP1a"

2016-03-08 Thread Rolf Howarth
Hi, I have an MXF file (originally generated from Avid Media Composer I believe) on which ffmpeg fails to correctly read the second audio track. The file is called test_Stereo.mxf and has two audio channels: on 0:a:0 there are 5s of tone, then 5s silence, then 5s tone, then 5s silence, while

[FFmpeg-devel] Subtitles for GSoC

2016-03-08 Thread Gerion Entrup
Hello, my own ideas seems not to be suitable for GSoC, so I looked again on the ideas page, because I have high interest to do something for FFmpeg this summer. The project, that I find most interesting, unfortunately is a unmentored one, the subtitle support. Is someone willing to mentor this?

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Set sar for multiscope videos

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 10:41:03AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes the samples in > samples.ffmpeg.org/ffmpeg-bugs/trac/ticket4535 for me. > > Please comment, Carl Eugen > mjpegdec.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > 88ce4aaf4530ae044

Re: [FFmpeg-devel] [PATCH] configure: build fix for P5600 with mips code restructuring

2016-03-08 Thread Michael Niedermayer
On Mon, Mar 07, 2016 at 07:49:10PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Signed-off-by: Shivraj Patil > --- > configure | 312 > +++-- > 1 file changed, 117 insertions(+), 195 deletions(-) applied thanks [..

Re: [FFmpeg-devel] TOON.AVI not playing correctly with FFplay

2016-03-08 Thread Ronald S. Bultje
Hi, On Tue, Mar 8, 2016 at 10:21 AM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: > On 03/08/2016 03:55 PM, Mats Peterson wrote: > >> Perhaps you could fix the index parsing of this file, Michael? It's >> clearly over my head. It's a file with xxpc chunks from Sierra's King's >> Quest

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] [PATCH v2] lavc/mjpegdec: avoid printing useless message in default log level

2016-03-08 Thread Moritz Barsnick
On Tue, Mar 08, 2016 at 16:55:02 +0100, Moritz Barsnick wrote: > The change of bps from 0 doesn't contain any useful information. > This message is now at info log level only if the original value > is !=0, otherwise pushed back to verbose log level. The original > value is displayed additionally.

[FFmpeg-devel] [PATCH v2] lavc/mjpegdec: avoid printing useless message in default log level

2016-03-08 Thread Moritz Barsnick
The change of bps from 0 doesn't contain any useful information. This message is now at info log level only if the original value is !=0, otherwise pushed back to verbose log level. The original value is displayed additionally. Signed-off-by: Moritz Barsnick --- libavcodec/mjpegdec.c | 2 +- 1 f

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

2016-03-08 Thread Moritz Barsnick
"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 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mp3dec

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

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 03:36:27PM +0100, Moritz Barsnick wrote: > The change of bps from 0 doesn't contain any info useful to the > user. This message is now at info log level only if the original > value is !=0, otherwise pushed back to debug log level. The > original value is displayed additiona

Re: [FFmpeg-devel] GSOC

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 05:07:04PM +0530, Arth Patel wrote: > Hey, > > I'd like to work on the Project to implement missing AAC decoder > features as a part > of GSOC'16. This project has been listed under unmentored projects. So > I'm looking for somebody to mentor me if possible. > > Also, can

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

2016-03-08 Thread Moritz Barsnick
On Tue, Mar 08, 2016 at 15:53:15 +0100, wm4 wrote: > AV_LOG_INFO is effectively a verbose level anyway, because FFmpeg > prints a lot of stuff at level INFO, so I'm not sure how useful it is. It is verbose, but it's the default, and has recently been cluttered with this confusing information. I ju

Re: [FFmpeg-devel] TOON.AVI not playing correctly with FFplay

2016-03-08 Thread Mats Peterson
On 03/08/2016 03:55 PM, Mats Peterson wrote: Perhaps you could fix the index parsing of this file, Michael? It's clearly over my head. It's a file with xxpc chunks from Sierra's King's Quest VI, and it plays fine with Windows Media Player. It has the xxpc chunks indexed, and somehow the demuxer w

[FFmpeg-devel] TOON.AVI not playing correctly with FFplay

2016-03-08 Thread Mats Peterson
Perhaps you could fix the index parsing of this file, Michael? It's clearly over my head. It's a file with xxpc chunks from Sierra's King's Quest VI, and it plays fine with Windows Media Player. It has the xxpc chunks indexed, and somehow the demuxer will barf on them. File: https://drive.goog

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 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 03:28 PM, Michael Niedermayer wrote: On Tue, Mar 08, 2016 at 11:42:48AM +0100, Mats Peterson wrote: Forget the old one. Refactored some variables to more appropriate locations as well. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ avienc.c | 15 +-- 1

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 11:42:48AM +0100, Mats Peterson wrote: > Forget the old one. Refactored some variables to more appropriate > locations as well. > > Mats > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > avienc.c | 15 +-- > 1 file changed, 5 insertions(+), 10 de

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

2016-03-08 Thread Moritz Barsnick
"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 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mp3dec.c

[FFmpeg-devel] [PATCH] lavc/mjpegdec: avoid printing useless message in default log level

2016-03-08 Thread Moritz Barsnick
The change of bps from 0 doesn't contain any info useful to the user. This message is now at info log level only if the original value is !=0, otherwise pushed back to debug log level. The original value is displayed additionally. Signed-off-by: Moritz Barsnick --- libavcodec/mjpegdec.c | 2 +-

Re: [FFmpeg-devel] [PATCH] avfilter/avf_showcqt: add performance debugging log

2016-03-08 Thread Muhammad Faiz
On Sun, Mar 6, 2016 at 5:01 PM, Muhammad Faiz wrote: > for easier development > > Signed-off-by: Muhammad Faiz > --- > libavfilter/avf_showcqt.c | 64 > +++ > libavfilter/avf_showcqt.h | 9 +++ > 2 files changed, 68 insertions(+), 5 deletions(-)

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 11:42 AM, Mats Peterson wrote: Forget the old one. Refactored some variables to more appropriate locations as well. Even if you won't apply patch 2/2, please apply this one. It's a no-brainer. The same logic checking for AV_PIX_FMT_PAL8 only is already used in movenc.c. Mats

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Mats Peterson
On 03/08/2016 02:06 PM, Ronald S. Bultje wrote: Hi, On Tue, Mar 8, 2016 at 7:48 AM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: On 03/08/2016 01:44 PM, Mats Peterson wrote: On 03/08/2016 01:43 PM, Ronald S. Bultje wrote: Hi Mats, On Mon, Mar 7, 2016 at 8:59 PM, Mats Peterson

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Ronald S. Bultje
Hi, On Tue, Mar 8, 2016 at 7:48 AM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: > On 03/08/2016 01:44 PM, Mats Peterson wrote: > >> On 03/08/2016 01:43 PM, Ronald S. Bultje wrote: >> >>> Hi Mats, >>> >>> On Mon, Mar 7, 2016 at 8:59 PM, Mats Peterson < >>> matsp888-at-yahoo@ffmpeg

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Mats Peterson
On 03/08/2016 01:44 PM, Mats Peterson wrote: On 03/08/2016 01:43 PM, Ronald S. Bultje wrote: Hi Mats, On Mon, Mar 7, 2016 at 8:59 PM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: On 03/07/2016 07:48 PM, Mats Peterson wrote: On 03/07/2016 07:43 PM, Mats Peterson wrote: Here's a

[FFmpeg-devel] [PATCH] Use lowercase includes and library names for schannel check

2016-03-08 Thread Ismail Donmez
This fixes cross-build on Linux with mingw-w64. Patch attached. This will not break MSVC because Windows is case-insensitive unlike Linux. Please review. Regards, ismail From e30470da6c21bb57c2a067660c613356cdbf6b92 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C4=B0smail=20D=C3=B6nmez?= Date: Tue,

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Mats Peterson
On 03/08/2016 01:43 PM, Ronald S. Bultje wrote: Hi Mats, On Mon, Mar 7, 2016 at 8:59 PM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: On 03/07/2016 07:48 PM, Mats Peterson wrote: On 03/07/2016 07:43 PM, Mats Peterson wrote: Here's a somewhat experimental patch with a new AVOpti

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Ronald S. Bultje
Hi Mats, On Mon, Mar 7, 2016 at 8:59 PM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: > On 03/07/2016 07:48 PM, Mats Peterson wrote: > >> On 03/07/2016 07:43 PM, Mats Peterson wrote: >> >>> Here's a somewhat experimental patch with a new AVOption >>> write_keyframe_palette that allows

Re: [FFmpeg-devel] [PATCH 2/2] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Mats Peterson
On 03/08/2016 11:47 AM, Mats Peterson wrote: Here's a somewhat experimental patch with a new AVOption write_keyframe_palette that allows you to write the palette to every keyframe. It is disabled by default, but it is needed for any file that contains palette changes in order to switch palette pr

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 01:01 PM, Mats Peterson wrote: On 03/08/2016 12:58 PM, Mats Peterson wrote: On 03/08/2016 12:56 PM, Mats Peterson wrote: I'm only writing 1 << bits_per_coded_sample entries to the file, although it will currently be exclusively 8. The bit depth, that is. No, that's probably

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 12:58 PM, Mats Peterson wrote: On 03/08/2016 12:56 PM, Mats Peterson wrote: I'm only writing 1 << bits_per_coded_sample entries to the file, although it will currently be exclusively 8. The bit depth, that is. No, that's probably wrong of me. On stream copy, the bit depth wi

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 12:56 PM, Mats Peterson wrote: I'm only writing 1 << bits_per_coded_sample entries to the file, although it will currently be exclusively 8. The bit depth, that is. Mats ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ff

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 12:54 PM, Mats Peterson wrote: On 03/08/2016 12:51 PM, Michael Niedermayer wrote: On Tue, Mar 08, 2016 at 11:42:48AM +0100, Mats Peterson wrote: Forget the old one. Refactored some variables to more appropriate locations as well. Mats -- Mats Peterson http://matsp888.no-ip.org/~

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 12:51 PM, Michael Niedermayer wrote: On Tue, Mar 08, 2016 at 11:42:48AM +0100, Mats Peterson wrote: Forget the old one. Refactored some variables to more appropriate locations as well. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ avienc.c | 15 +-- 1

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Michael Niedermayer
On Tue, Mar 08, 2016 at 11:42:48AM +0100, Mats Peterson wrote: > Forget the old one. Refactored some variables to more appropriate > locations as well. > > Mats > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > avienc.c | 15 +-- > 1 file changed, 5 insertions(+), 10 de

[FFmpeg-devel] GSOC

2016-03-08 Thread Arth Patel
Hey, I'd like to work on the Project to implement missing AAC decoder features as a part of GSOC'16. This project has been listed under unmentored projects. So I'm looking for somebody to mentor me if possible. Also, can I directly start working to fix a bug on AAC issues or will I be assigned a

[FFmpeg-devel] [PATCH 2/2] lavf/avienc: New AVOption write_keyframe_palette

2016-03-08 Thread Mats Peterson
Here's a somewhat experimental patch with a new AVOption write_keyframe_palette that allows you to write the palette to every keyframe. It is disabled by default, but it is needed for any file that contains palette changes in order to switch palette properly in FFplay (and, in the future, proba

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
On 03/08/2016 11:42 AM, Mats Peterson wrote: Forget the old one. Refactored some variables to more appropriate locations as well. And forget the old 2-part patch set with the avi_write_xxpc() function. It would only be used once, so it was superfluous. Mats

[FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette changing code only concerns AV_PIX_FMT_PAL8

2016-03-08 Thread Mats Peterson
Forget the old one. Refactored some variables to more appropriate locations as well. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 766dc9c46ed9d528d41505f8b67e18ef261f8f0e Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Tue, 8 Mar 2016 11:26:32 +0100 Subject: [PATCH 1/2] lavf

[FFmpeg-devel] [PATCH]lavc/mjpegdec: Set sar for multiscope videos

2016-03-08 Thread Carl Eugen Hoyos
Hi! Attached patch fixes the samples in samples.ffmpeg.org/ffmpeg-bugs/trac/ticket4535 for me. Please comment, Carl Eugen diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index 1e83e88..e24db44 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -1878,8 +1878,10 @@ static

Re: [FFmpeg-devel] Motion interpolation in libavfilter

2016-03-08 Thread Paul B Mahol
On 3/8/16, Subhashish Pradhan wrote: > Hello, > > I am Subhashish Pradhan, a CS undergrad from India. I want to take up the > project "Motion interpolation in libavfilter" for GSoC 2016 and I'd like to > understand the goals for this project. Goals are already written on wiki, if you have more sp