[FFmpeg-devel] [PATCH] Add FITS Decoder

Made the changes suggested above Signed-off-by: Paras Chadha --- Changelog | 1 + doc/general.texi| 2 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 8 +

[FFmpeg-devel] [PATCH] libavformat/avformat.h: Move docs inside of #if

Otherwise AVTimebaseSource gets av_apply_bitstream_filters' documentation in doxygen. Signed-off-by: Max Weber --- libavformat/avformat.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index

Re: [FFmpeg-devel] [PATCH v11] - Added Turing codec interface for ffmpeg

Dear all, >> A build failure seems a bit like a deal breaker Sorry for this. I have just pushed a fix to the Turing repository to remove bool type from the libturing APIs. The patch builds and runs fine now in our tests. Thanks, Saverio ___

Re: [FFmpeg-devel] [PATCH 2/2] libavfilter/scale2ref: Maintain main input's DAR

Ping :) On Mon, Jun 12, 2017 at 1:08 AM, Kevin Mark wrote: > I've been using this patch for the past week now and I believe it's > good to go. Does someone want to take a second look before merging? > > On Mon, Jun 5, 2017 at 6:55 AM, Kevin Mark wrote: >>

Re: [FFmpeg-devel] [PATCH] x86/utvideodsp: make restore_rgb_planes functions work on x86_32

On 6/30/17, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/x86/utvideodsp.asm| 20 ++-- > libavcodec/x86/utvideodsp_init.c | 2 +- > 2 files changed, 15 insertions(+), 7 deletions(-) > ok if tested.

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: add sbrdsp tests

On Thu, Jun 29, 2017 at 10:53:06PM -0300, James Almer wrote: > On 6/29/2017 10:14 PM, Henrik Gramner wrote: > > On Fri, Jun 30, 2017 at 1:58 AM, Michael Niedermayer > > wrote: > >> Program received signal SIGSEGV, Segmentation fault. > >> 0x00684919 in

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/proresenc: switch default prores encoder to prores_ks

On Mon, Jun 26, 2017 at 11:55:35PM +0200, Paul B Mahol wrote: > Rationale: > prores_ks have more features and is faster for qscale > 0 > and gives better quality output. > > Signed-off-by: Paul B Mahol > --- > libavcodec/Makefile | 2 +- >

Re: [FFmpeg-devel] [PATCH] Add FITS Decoder

Hi. A few technical / cosmetic remarks below. I do not know the FITS format itself more than in passing. Le duodi 12 messidor, an CCXXV, Paras Chadha a écrit : > Made the changes suggested above > > Signed-off-by: Paras Chadha > --- > Changelog | 1 + >

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

On 6/30/17, Derek Buitenhuis wrote: > On 6/30/2017 4:44 PM, Thilo Borgmann wrote: >> Yes but I cannot share, unfortunately. > > There are two backtraces in the mail thread from before that could be > helpful. Thanks, but I do not see important variables.

Re: [FFmpeg-devel] [PATCH] hevcdec: properly export some side data with frame threading

On Thu, 29 Jun 2017 15:37:16 +0200 wm4 wrote: > I noticed this with mastering display data. If frame threading is > enabled, this side data is exported only for some frames. It turns out > it's not properly propagated to the worker threads. > > I didn't touch A53

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

On 6/30/2017 4:31 PM, Paul B Mahol wrote: > This reverts commit 18f94df8af04f2c02a25a7dec512289feff6517f. > > Signed-off-by: Paul B Mahol > --- > libavcodec/alsdec.c | 6 +- http://ffmpeg.org/pipermail/ffmpeg-devel/2016-November/202513.html - Derek

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: use correct variable when checking for overwrite

On 6/30/17, Thilo Borgmann wrote: > Am 30.06.17 um 18:43 schrieb Paul B Mahol: >> Fixes #5297. >> >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/alsdec.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

On 6/30/17, Ricardo Constantino wrote: > On 30 June 2017 at 16:55, Paul B Mahol wrote: >> On 6/28/17, Paul B Mahol wrote: >>> On 6/28/17, Ricardo Constantino wrote: On 24 June 2017 at 13:47, Ricardo Constantino

Re: [FFmpeg-devel] [PATCH] libavformat/avformat.h: Move docs inside of #if

On Thu, Jun 29, 2017 at 11:58:41PM -0600, Max Weber wrote: > Otherwise AVTimebaseSource gets av_apply_bitstream_filters' documentation in > doxygen. > > Signed-off-by: Max Weber > --- > libavformat/avformat.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

Am 30.06.17 um 17:42 schrieb Paul B Mahol: > On 6/30/17, Thilo Borgmann wrote: >> Hi, >> >> Am 30.06.17 um 17:31 schrieb Paul B Mahol: >>> This reverts commit 18f94df8af04f2c02a25a7dec512289feff6517f. >>> >>> Signed-off-by: Paul B Mahol >>> --- >>>

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: use correct variable when checking for overwrite

Am 30.06.17 um 19:34 schrieb Paul B Mahol: > On 6/30/17, Thilo Borgmann wrote: >> Am 30.06.17 um 18:43 schrieb Paul B Mahol: >>> Fixes #5297. >>> >>> Signed-off-by: Paul B Mahol >>> --- >>> libavcodec/alsdec.c | 4 ++-- >>> 1 file changed, 2

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

On 6/30/17, Thilo Borgmann wrote: > Hi, > > Am 30.06.17 um 17:31 schrieb Paul B Mahol: >> This reverts commit 18f94df8af04f2c02a25a7dec512289feff6517f. >> >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/alsdec.c | 6 +- >> 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

On 30 June 2017 at 16:55, Paul B Mahol wrote: > On 6/28/17, Paul B Mahol wrote: >> On 6/28/17, Ricardo Constantino wrote: >>> On 24 June 2017 at 13:47, Ricardo Constantino wrote: libvorbis comes with pkg-config

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

On 6/30/17, Thilo Borgmann wrote: > Am 30.06.17 um 17:42 schrieb Paul B Mahol: >> On 6/30/17, Thilo Borgmann wrote: >>> Hi, >>> >>> Am 30.06.17 um 17:31 schrieb Paul B Mahol: This reverts commit 18f94df8af04f2c02a25a7dec512289feff6517f.

[FFmpeg-devel] [PATCH] avcodec/alsdec: use correct variable when checking for overwrite

Fixes #5297. Signed-off-by: Paul B Mahol --- libavcodec/alsdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index d95e30d..a925502 100644 --- a/libavcodec/alsdec.c +++ b/libavcodec/alsdec.c @@ -705,8 +705,8

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: add sbrdsp tests

On Fri, Jun 30, 2017 at 03:55:52PM +0200, Michael Niedermayer wrote: > On Thu, Jun 29, 2017 at 10:53:06PM -0300, James Almer wrote: > > On 6/29/2017 10:14 PM, Henrik Gramner wrote: > > > On Fri, Jun 30, 2017 at 1:58 AM, Michael Niedermayer > > > wrote: > > >> Program

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: use correct variable when checking for overwrite

Am 30.06.17 um 18:43 schrieb Paul B Mahol: > Fixes #5297. > > Signed-off-by: Paul B Mahol > --- > libavcodec/alsdec.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c > index d95e30d..a925502 100644 > ---

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: add sbrdsp tests

On 6/30/2017 10:55 AM, Michael Niedermayer wrote: > On Thu, Jun 29, 2017 at 10:53:06PM -0300, James Almer wrote: >> On 6/29/2017 10:14 PM, Henrik Gramner wrote: >>> On Fri, Jun 30, 2017 at 1:58 AM, Michael Niedermayer >>> wrote: Program received signal SIGSEGV,

Re: [FFmpeg-devel] [PATCH 1/5 v3] movenc: use correct tag list for AVOutputFormat.codec_tag

On 6/29/2017 10:31 PM, James Almer wrote: >> dashenc is also using ff_mp4_obj_type. Guess it should be a matter of >> changing that as well. It would require making the new codec_mp4_tags[] >> shared in isom.c > For the record, this also affects libav. dashenc is currently broken > there because

[FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

This reverts commit 18f94df8af04f2c02a25a7dec512289feff6517f. Signed-off-by: Paul B Mahol --- libavcodec/alsdec.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index d95e30d..209feb8 100644 ---

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

Hi, Am 30.06.17 um 17:31 schrieb Paul B Mahol: > This reverts commit 18f94df8af04f2c02a25a7dec512289feff6517f. > > Signed-off-by: Paul B Mahol > --- > libavcodec/alsdec.c | 6 +- > 1 file changed, 1 insertion(+), 5 deletions(-) this has not yet been reverted for a reason

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

On 6/30/2017 4:44 PM, Thilo Borgmann wrote: > Yes but I cannot share, unfortunately. There are two backtraces in the mail thread from before that could be helpful. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

On 6/28/17, Paul B Mahol wrote: > On 6/28/17, Ricardo Constantino wrote: >> On 24 June 2017 at 13:47, Ricardo Constantino wrote: >>> libvorbis comes with pkg-config files since at least v1.0.1, way back >>> in 2003. >>> --- >>> The

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/alsdec: check predictor order against block length"

On 6/30/2017 4:58 PM, Paul B Mahol wrote: > Thanks, but I do not see important variables. There are line numbers and dates, though, pointing at: 931:*raw_samples++ -= y >> 20; Better than nothing. - Derek ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] patch 1. Correction of loop behaviour in ffpeg.c

On Wed, Jun 28, 2017 at 15:02:42 +0300, ffm...@a.legko.ru wrote: In addition to what Michael wrote: > +if (nb_input_files == 1) > + return 0; Indentation is not only inconsistent, but also wrong. Your first level is four spaces, your second level is seven, should be eight. Please get

[FFmpeg-devel] [PATCH] configure: require pkg-config for libvorbis

libvorbis comes with pkg-config files since at least v1.0.1, way back in 2003. The extra check is needed for shared builds, as the pkg-config file for vorbisenc doesn't include vorbis and ogg if --static isn't used. The previous check could be also used as an extra fallback in case pkg-config

Re: [FFmpeg-devel] [PATCH] Add FITS Decoder

On Fri, Jun 30, 2017 at 15:08:17 +0200, Nicolas George wrote: > > + * function calculates the data_min and data_max values from the data. > > The style for doxygen comments is impersonal verbs: "Calculate the ...". I don't want to nitpick, and I *may* be wrong, but the grammar freak in me thinks

Re: [FFmpeg-devel] [WIP][PATCH]v3 Opus Pyramid Vector Quantization Search in x86 SIMD asm

First, I've removed the hacked formula. While it seems to improve precision in the synthetic test, it is not enough to avoid assigning pulses in padded area. Worse, it also interferes with the special case with subtraction of pulses from y[i]==0 output. Handling these cases needs combining two

[FFmpeg-devel] [PATCH 1/2] avdevice/decklink_dec: add support for decoding teletext from 10bit ancillary data

This also add supports for 4K DeckLink cards because they always output the ancillary data in 10-bit. Signed-off-by: Marton Balint --- doc/indevs.texi | 4 ++-- libavdevice/decklink_dec.cpp | 56 +++- 2 files changed, 42

[FFmpeg-devel] [PATCH 2/2] avdevice/decklink_dec: add support for receiving op47 teletext

Signed-off-by: Marton Balint --- doc/indevs.texi | 18 --- libavdevice/decklink_dec.cpp | 125 +++ 2 files changed, 125 insertions(+), 18 deletions(-) diff --git a/doc/indevs.texi b/doc/indevs.texi index

Re: [FFmpeg-devel] [WIP][PATCH]v2 Opus Pyramid Vector Quantization Search in x86 SIMD asm

On 6/26/17, Henrik Gramner wrote: > On Sat, Jun 24, 2017 at 10:39 PM, Ivan Kalvachev > wrote: >> +%define HADDPS_IS_FAST 0 >> +%define PHADDD_IS_FAST 0 > [...] >> +haddps %1, %1 >> +haddps %1, %1 > [...] >> + phaddd

[FFmpeg-devel] [PATCH] avcodec/alsdec: properly allocate raw_buffer

This also reverts 18f94df8. Fixes #5297. Signed-off-by: Paul B Mahol --- libavcodec/alsdec.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index d95e30d..4a8f13d 100644 --- a/libavcodec/alsdec.c +++

Re: [FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

Hmm, finally got around to looking into this again and this still isn't fixed. Just seeking a few times in ffplay can trigger this issue with the clip linked in my original message: http://storage.googleapis.com/dalecurtis-shared/buck480p30_na.mp4 ./ffplay -v debug -drp 1

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: use correct variable when checking for overwrite

On Fri, Jun 30, 2017 at 07:34:32PM +0200, Paul B Mahol wrote: > On 6/30/17, Thilo Borgmann wrote: > > Am 30.06.17 um 18:43 schrieb Paul B Mahol: > >> Fixes #5297. > >> > >> Signed-off-by: Paul B Mahol > >> --- > >> libavcodec/alsdec.c | 4 ++-- > >> 1