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

2017-07-03 Thread Paras Chadha
On Mon, Jul 3, 2017 at 3:56 AM, Moritz Barsnick wrote: > On Sun, Jul 02, 2017 at 20:48:17 +0530, Paras Chadha wrote: > > +int64_t header_size = 0, data_size=0, ret, pcount=0, gcount=1, d; > [...] > > +header_size += 80; > [...] > > +header_size += 80; > [...] > > +header_size += 8

[FFmpeg-devel] [PATCH] avfilter: add VMAF filter

2017-07-03 Thread Ashish Singh
Hi, my pull request for vmaf library interface has been merged. First install vmaf library (libvmaf) from https://github.com/Netflix/vmaf It can be installed by first doing 'make', then 'make lib' (for building libvmaf.a), then 'make install' (for installing) from root directory of vmaf. Fixed the

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Paul B Mahol
On 7/2/17, Hein-Pieter van Braam wrote: > Fixes: 6503 crash with fuzzed file > > Signed-off-by: Hein-Pieter van Braam > --- > libavcodec/interplayvideo.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/libavcodec/interplayvideo.c b/libavcodec/interplayvideo.c > index d6f484a..86

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

2017-07-03 Thread Paul B Mahol
On 7/2/17, Michael Niedermayer wrote: > On Sun, Jul 02, 2017 at 01:14:00PM +0200, wm4 wrote: >> On Sun, 2 Jul 2017 00:09:42 +0200 >> Michael Niedermayer wrote: >> >> > This reduces the number of strstr() calls per byte >> > This diasalows empty tags like '< >' as well as '<' in tags like >> > ''

[FFmpeg-devel] [PATCH] avcodec/alsdec: fix some undefined shifts

2017-07-03 Thread Paul B Mahol
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 37d6b38..df5e2bb 100644 --- a/libavcodec/alsdec.c +++ b/libavcodec/alsdec.c @@ -756,7 +756,7 @@ static int read_var_block_dat

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Hein-Pieter van Braam
On Mon, 2017-07-03 at 11:43 +0200, Paul B Mahol wrote: > On 7/2/17, Hein-Pieter van Braam wrote: > > Fixes: 6503 crash with fuzzed file > > > > Signed-off-by: Hein-Pieter van Braam > > --- > >  libavcodec/interplayvideo.c | 8 > >  1 file changed, 8 insertions(+) > > > > diff --git a/li

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Paul B Mahol
On 7/3/17, Hein-Pieter van Braam wrote: > On Mon, 2017-07-03 at 11:43 +0200, Paul B Mahol wrote: >> On 7/2/17, Hein-Pieter van Braam wrote: >> > Fixes: 6503 crash with fuzzed file >> > >> > Signed-off-by: Hein-Pieter van Braam >> > --- >> > libavcodec/interplayvideo.c | 8 >> > 1 file

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Hein-Pieter van Braam
On Mon, 2017-07-03 at 11:59 +0200, Paul B Mahol wrote: > > > You are comparing pointer with same pointer increased by some value. > > This is wrong way to fix it. Urgh, you're right, of course. I don't know what I was thinking. ___ ffmpeg-devel mailin

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

2017-07-03 Thread wm4
On Sun, 2 Jul 2017 13:43:57 +0200 Michael Niedermayer wrote: > On Sun, Jul 02, 2017 at 01:14:00PM +0200, wm4 wrote: > > On Sun, 2 Jul 2017 00:09:42 +0200 > > Michael Niedermayer wrote: > > > > > This reduces the number of strstr() calls per byte > > > This diasalows empty tags like '< >' as

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

2017-07-03 Thread Michael Niedermayer
On Mon, Jul 03, 2017 at 11:44:26AM +0200, Paul B Mahol wrote: > On 7/2/17, Michael Niedermayer wrote: > > On Sun, Jul 02, 2017 at 01:14:00PM +0200, wm4 wrote: > >> On Sun, 2 Jul 2017 00:09:42 +0200 > >> Michael Niedermayer wrote: > >> > >> > This reduces the number of strstr() calls per byte > >

Re: [FFmpeg-devel] [PATCH] vdpau: do not use buggy HEVC support by default

2017-07-03 Thread wm4
On Sat, 1 Jul 2017 07:10:47 -0700 Philip Langdale wrote: > On Sat, 1 Jul 2017 11:40:38 +0200 > wm4 wrote: > > > NVIDIA broke its own API when using VDPAU decoding. If you retrieve > > the decoded YUV data, or if you map the surfaces with GL interop, the > > result are interlacing artifacts. Th

[FFmpeg-devel] [PATCH] avcodec/vp9: AVX2 ipred_dl_32x32 improvement

2017-07-03 Thread Ilia Valiakhmetov
Use symmetry properties of the ipred_dl function for better performance. vp9_diag_downleft_32x32_12bpp_c: 1534.2 vp9_diag_downleft_32x32_12bpp_sse2: 145.9 vp9_diag_downleft_32x32_12bpp_ssse3: 140.0 vp9_diag_downleft_32x32_12bpp_avx: 134.8 vp9_diag_downleft_32x32_12bpp_avx2: 78.9 ~40% faster than

[FFmpeg-devel] [PATCH] avcodec/vp9: AVX2 ipred_vl_16x16

2017-07-03 Thread Ilia Valiakhmetov
vp9_vert_left_16x16_12bpp_c: 273.8 vp9_vert_left_16x16_12bpp_sse2: 69.4 vp9_vert_left_16x16_12bpp_ssse3: 35.3 vp9_vert_left_16x16_12bpp_avx: 34.6 vp9_vert_left_16x16_12bpp_avx2: 22.4 ~35% faster than avx Signed-off-by: Ilia Valiakhmetov --- libavcodec/x86/vp9dsp_init_16bpp.c| 2 ++ libavco

[FFmpeg-devel] [PATCH] htmlsubtitles: support tag

2017-07-03 Thread wm4
Some .srt files use this tag. (An alternative implementation would be correctly ignoring unknown tags, and treating them as whitespace. libass can do automatic line wrapping.) --- libavcodec/htmlsubtitles.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/htmlsubtitles.c b/libavco

Re: [FFmpeg-devel] [PATCH] htmlsubtitles: support tag

2017-07-03 Thread Clément Bœsch
On Mon, Jul 03, 2017 at 01:43:43PM +0200, wm4 wrote: > Some .srt files use this tag. > > (An alternative implementation would be correctly ignoring unknown tags, > and treating them as whitespace. libass can do automatic line wrapping.) > --- > libavcodec/htmlsubtitles.c | 2 ++ > 1 file changed,

Re: [FFmpeg-devel] [PATCH] htmlsubtitles: support tag

2017-07-03 Thread wm4
On Mon, 3 Jul 2017 13:54:50 +0200 Clément Bœsch wrote: > On Mon, Jul 03, 2017 at 01:43:43PM +0200, wm4 wrote: > > Some .srt files use this tag. > > > > (An alternative implementation would be correctly ignoring unknown tags, > > and treating them as whitespace. libass can do automatic line wrapp

[FFmpeg-devel] [PATCH] lavf/utils: update stream duration only if it is not set or 0

2017-07-03 Thread Matthieu Bouron
--- The following patch makes lavf/utils only update stream duration only if it is not set or 0 in fill_all_stream_timings (which is called by avformat_find_stream_info). In the context of mov demuxing, the patch makes the last packet duration the same as the one declared in the stts table for ex

Re: [FFmpeg-devel] [PATCH] avfomat/hlsenc: support fmp4 format in hls

2017-07-03 Thread Steven Liu
2017-07-01 20:05 GMT+08:00 Steven Liu : > add the fmp4 format into hlsenc > because the fmp4 format add into hls from version 7. > the spec link is: > https://tools.ietf.org/html/draft-pantos-http-live-streaming-20 > and the describe on WWDC > https://developer.apple.com/videos/play/wwdc2017/515/ >

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

2017-07-03 Thread Matthieu Bouron
On Fri, Jun 30, 2017 at 05:16:37PM +0200, Matthieu Bouron wrote: > 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, Michae

[FFmpeg-devel] [PATCH] avfilter: add ANSNR filter

2017-07-03 Thread Ashish Singh
This is ANSNR filter, one of the component filters of VMAF. Run it using: ffmpeg -i main -i ref -lavfi ansnr -f null - Currently it outputs the average ansnr score over all frames. --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/ansnr.h | 32 libavfi

Re: [FFmpeg-devel] [PATCH] avfilter: add ANSNR filter

2017-07-03 Thread Moritz Barsnick
On Mon, Jul 03, 2017 at 18:39:38 +0530, Ashish Singh wrote: > This is ANSNR filter, one of the component filters of VMAF. Would you please expand the abbreviation in at least one place? Optimally in doc/filters.texi, the content of which is missing (as it still is in your VMAF patch). (And perhaps

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

2017-07-03 Thread Derek Buitenhuis
On 7/3/2017 3:00 AM, James Almer wrote: >> +{ AV_CODEC_ID_EAC3, MKTAG('a', 'c', '-', '3') }, > Should be ec-3. Changing it fixes fate-copy-trac3074 as pointed by > Michael in a reply to patch 2/5. OK, this was my bug; Libav does't have that entry at all. > >> +{ AV_CODEC_ID_DTS

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread Derek Buitenhuis
On 7/3/2017 2:18 AM, Michael Niedermayer wrote: > breaks fate I'll look into it tonight; busy today. . . . Aside: I'll just add, though, that these two word 'breaks fate' emails are kind of obnoxious when the test in question was added days after I sent the set, so I couldn't have possibly test

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread wm4
On Mon, 3 Jul 2017 16:17:42 +0100 Derek Buitenhuis wrote: > On 7/3/2017 2:18 AM, Michael Niedermayer wrote: > > breaks fate > > I'll look into it tonight; busy today. > > . > . > . > > Aside: > > I'll just add, though, that these two word 'breaks fate' emails > are kind of obnoxious when th

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

2017-07-03 Thread Michael Niedermayer
On Mon, Jul 03, 2017 at 12:40:13PM +0200, wm4 wrote: > On Sun, 2 Jul 2017 13:43:57 +0200 > Michael Niedermayer wrote: > > > On Sun, Jul 02, 2017 at 01:14:00PM +0200, wm4 wrote: > > > On Sun, 2 Jul 2017 00:09:42 +0200 > > > Michael Niedermayer wrote: > > > > > > > This reduces the number of s

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

2017-07-03 Thread Aaron Levinson
On 6/30/2017 5:05 PM, Marton Balint wrote: 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 +++-

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: relax opt_order limit

2017-07-03 Thread Thilo Borgmann
Am 02.07.17 um 19:45 schrieb Paul B Mahol: > On 6/11/16, Thilo Borgmann wrote: >> >> Hi, >> >>> Am 09.06.2016 um 03:33 schrieb Michael Niedermayer >>> : >>> >>> Fixes: Ticket5297 >>> >>> Needs review by maintainer / author to check that this is ok and >>> sufficient >>> --- >>> libavcodec/alsdec.c

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: remove unused header

2017-07-03 Thread Thilo Borgmann
Am 03.07.17 um 08:01 schrieb Steven Liu: > 2017-07-02 4:24 GMT+08:00 Paul B Mahol : >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/alsdec.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c >> index 31e95e2..ac59885 100644 >> --- a/libavcode

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread Hendrik Leppkes
On Mon, Jul 3, 2017 at 5:39 PM, wm4 wrote: > On Mon, 3 Jul 2017 16:17:42 +0100 > Derek Buitenhuis wrote: > >> On 7/3/2017 2:18 AM, Michael Niedermayer wrote: >> > breaks fate >> >> I'll look into it tonight; busy today. >> >> . >> . >> . >> >> Aside: >> >> I'll just add, though, that these two wo

Re: [FFmpeg-devel] [PATCH] htmlsubtitles: support tag

2017-07-03 Thread Michael Niedermayer
On Mon, Jul 03, 2017 at 01:54:50PM +0200, Clément Bœsch wrote: > On Mon, Jul 03, 2017 at 01:43:43PM +0200, wm4 wrote: > > Some .srt files use this tag. > > > > (An alternative implementation would be correctly ignoring unknown tags, > > and treating them as whitespace. libass can do automatic line

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: fix some undefined shifts

2017-07-03 Thread Thilo Borgmann
Am 03.07.17 um 11:49 schrieb Paul B Mahol: > Signed-off-by: Paul B Mahol > --- > libavcodec/alsdec.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Ok if you really feel we are anywhere near a sign bit here. -Thilo ___ ffmpeg-devel maili

[FFmpeg-devel] [PATCH] avfilter: add ANSNR filter

2017-07-03 Thread Ashish Singh
Added ansnr section in doc/filters.texi and changelog and fixed issues. --- Changelog| 1 + doc/filters.texi | 24 +++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/ansnr.h | 29 libavfilter/vf_ansnr.c | 403 +++

[FFmpeg-devel] [PATCH] lavc/mediacodec: rescale pts before decoding for both hw and sw buffers

2017-07-03 Thread Aman Gupta
From: Aman Gupta Replicates the logic used in the wrap_hw_buffer path to wrap_sw_buffer as well. This fixes decoding issues observed on AMLogic devices with OMX.amlogic.mpeg2.decoder.awesome, where the decoder would spit out a constant stream of "mPtsRecoveryCount" errors and decoded frames were

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

2017-07-03 Thread Aaron Levinson
On 6/30/2017 5:05 PM, Marton Balint wrote: 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/inde

[FFmpeg-devel] [PATCH] avfilter: add VMAF filter

2017-07-03 Thread Ashish Singh
Added vmaf section in doc/filters.texi and Changelog. --- Changelog| 1 + configure| 5 + doc/filters.texi | 33 libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_vmaf.c| 402 +++

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread Marton Balint
On Mon, 3 Jul 2017, Hendrik Leppkes wrote: On Mon, Jul 3, 2017 at 5:39 PM, wm4 wrote: On Mon, 3 Jul 2017 16:17:42 +0100 Derek Buitenhuis wrote: On 7/3/2017 2:18 AM, Michael Niedermayer wrote: > breaks fate I'll look into it tonight; busy today. . . . Aside: I'll just add, though, that

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread Paul B Mahol
On 7/3/17, Marton Balint wrote: > > > On Mon, 3 Jul 2017, Hendrik Leppkes wrote: > >> On Mon, Jul 3, 2017 at 5:39 PM, wm4 wrote: >>> On Mon, 3 Jul 2017 16:17:42 +0100 >>> Derek Buitenhuis wrote: >>> On 7/3/2017 2:18 AM, Michael Niedermayer wrote: > breaks fate I'll look into

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread Nicolas George
Le quintidi 15 messidor, an CCXXV, Paul B Mahol a écrit : > What would then remain for Carl and folks? Do you show such despise for other people's work on purpose or is it your real opinion seeping through? Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread James Almer
On 7/3/2017 12:45 PM, Hendrik Leppkes wrote: > On Mon, Jul 3, 2017 at 5:39 PM, wm4 wrote: >> On Mon, 3 Jul 2017 16:17:42 +0100 >> Derek Buitenhuis wrote: >> >>> On 7/3/2017 2:18 AM, Michael Niedermayer wrote: breaks fate >>> >>> I'll look into it tonight; busy today. >>> >>> . >>> . >>> . >>

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

2017-07-03 Thread Marton Balint
On Mon, 3 Jul 2017, Aaron Levinson wrote: +static uint8_t* teletext_data_unit_from_vbi_data_10bit(int line, uint8_t *src, uint8_t *tgt) +{ +uint8_t y[720]; +uint8_t *py = y; +uint8_t *pend = y + 720; +while (py < pend) { +*py++ = (src[1] >> 4) + ((src[2] & 15) << 4);

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

2017-07-03 Thread Marton Balint
On Mon, 3 Jul 2017, Aaron Levinson wrote: + +This option is a bitmask of the SD VBI lines captured, specifically lines 6 to Probably should be "SD PAL VBI lines" to make it clear that NTSC is not supported. ok. +22, and lines 318 to 335. Line 6 is the LSB in the mask. Selected lines

[FFmpeg-devel] [PATCH] lavc/mediacodec: add missing newline on warning

2017-07-03 Thread Aman Gupta
From: Aman Gupta --- libavcodec/mediacodecdec_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mediacodecdec_common.c b/libavcodec/mediacodecdec_common.c index 1263188d34..f88b2cde54 100644 --- a/libavcodec/mediacodecdec_common.c +++ b/libavcodec/mediacode

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

2017-07-03 Thread Aaron Levinson
On 7/3/2017 1:06 PM, Marton Balint wrote: On Mon, 3 Jul 2017, Aaron Levinson wrote: +22, and lines 318 to 335. Line 6 is the LSB in the mask. Selected lines which +do not contain teletext information will be ignored. You can use the special +@option{all} constant to select all possible line

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

2017-07-03 Thread Marton Balint
On Mon, 3 Jul 2017, Aaron Levinson wrote: On 7/3/2017 1:06 PM, Marton Balint wrote: On Mon, 3 Jul 2017, Aaron Levinson wrote: +22, and lines 318 to 335. Line 6 is the LSB in the mask. Selected lines which +do not contain teletext information will be ignored. You can use the special +@op

[FFmpeg-devel] [PATCH] decode.c: Handle 0-size packets in compat_decode

2017-07-03 Thread Reimar Döffinger
The old API did that just fine, and if we provide a compatibility layer it should at least be compatible. For the test-case (feeding AVParser output directly to decoder, failing to discard 0-size packets) just discarding 0-size packets at the start works, but not sure if in some cases we might want

Re: [FFmpeg-devel] [PATCH] decode.c: Handle 0-size packets in compat_decode

2017-07-03 Thread wm4
On Mon, 3 Jul 2017 20:57:21 +0200 Reimar Döffinger wrote: > The old API did that just fine, and if we provide > a compatibility layer it should at least be compatible. > For the test-case (feeding AVParser output directly to > decoder, failing to discard 0-size packets) just discarding > 0-size

[FFmpeg-devel] [PATCH] avfilter: add limiter filter

2017-07-03 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 17 +++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/limiter.h | 33 ++ libavfilter/vf_limiter.c | 228 ++ libavf

Re: [FFmpeg-devel] [PATCH] avfilter: add limiter filter

2017-07-03 Thread James Almer
On 7/3/2017 4:45 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 17 +++ > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/limiter.h | 33 ++ > libavfilter/vf_limiter.c

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: add missing newline on warning

2017-07-03 Thread Matthieu Bouron
On Mon, Jul 03, 2017 at 11:07:36AM -0700, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/mediacodecdec_common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/mediacodecdec_common.c > b/libavcodec/mediacodecdec_common.c > index 1263188d34..f88b2c

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: rescale pts before decoding for both hw and sw buffers

2017-07-03 Thread Matthieu Bouron
On Mon, Jul 03, 2017 at 09:21:50AM -0700, Aman Gupta wrote: > From: Aman Gupta > > Replicates the logic used in the wrap_hw_buffer path to wrap_sw_buffer > as well. > > This fixes decoding issues observed on AMLogic devices with > OMX.amlogic.mpeg2.decoder.awesome, where the decoder would spit o

Re: [FFmpeg-devel] [PATCH] decode.c: Handle 0-size packets in compat_decode

2017-07-03 Thread Reimar Döffinger
On 03.07.2017, at 21:07, wm4 wrote: > On Mon, 3 Jul 2017 20:57:21 +0200 > Reimar Döffinger wrote: > >> The old API did that just fine, and if we provide >> a compatibility layer it should at least be compatible. >> For the test-case (feeding AVParser output directly to >> decoder, failing to d

Re: [FFmpeg-devel] [PATCH 2/5] movenc: simplify codec_tag lookup

2017-07-03 Thread Reimar Döffinger
On 03.07.2017, at 17:17, Derek Buitenhuis wrote: > On 7/3/2017 2:18 AM, Michael Niedermayer wrote: >> breaks fate > > I'll look into it tonight; busy today. > > . > . > . > > Aside: > > I'll just add, though, that these two word 'breaks fate' emails > are kind of obnoxious when the test in qu

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

2017-07-03 Thread Reimar Döffinger
> +static int64_t find_size(AVIOContext * pb, FITSContext * fits) > +{ > +int bitpix, naxis, dim_no, i, naxisn[999], groups=0; > +int64_t header_size = 0, data_size=0, ret, pcount=0, gcount=1, d; > +char buf[81], c; This is more than 4kB of data on the stack. Large stack arrays have a

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

2017-07-03 Thread Nicolas George
Hi. Nice to see you back. Le sextidi 16 messidor, an CCXXV, Reimar Döffinger a écrit : > This is more than 4kB of data on the stack. > Large stack arrays have a huge amount of security implications, please > put such buffers (if really needed) into the context. 4 ko is not large, and neither is w

Re: [FFmpeg-devel] [PATCH] Add support for RockChip Media Process Platform

2017-07-03 Thread Steven Liu
2017-06-13 19:02 GMT+08:00 wm4 : > On Tue, 13 Jun 2017 11:17:35 +0100 > Mark Thompson wrote: > >> > +vp8_rkmpp_hwaccel_deps="rkmpp" >> > vp9_d3d11va_hwaccel_deps="d3d11va DXVA_PicParams_VP9" >> > vp9_d3d11va_hwaccel_select="vp9_decoder" >> > vp9_dxva2_hwaccel_deps="dxva2 DXVA_PicParams_VP9" >>

[FFmpeg-devel] [PATCH] avformat/hlsenc: copy codec_tag when stream copy

2017-07-03 Thread Steven Liu
when use fmp4 segment type in hls and use codec copy, there have an error message. error message: [mp4 @ 0x25df020] Tag avc1 incompatible with output codec id '28' ([33][0][0][0]) [hls @ 0x2615c80] Some of the provided format options in '(null)' are not recognized Could not write header

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

2017-07-03 Thread Reimar Döffinger
On 04.07.2017, at 00:51, Nicolas George wrote: > Hi. Nice to see you back. > > Le sextidi 16 messidor, an CCXXV, Reimar Döffinger a écrit : >> This is more than 4kB of data on the stack. >> Large stack arrays have a huge amount of security implications, please >> put such buffers (if really need

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

2017-07-03 Thread Reimar Döffinger
On 30.06.2017, at 20:06, Ricardo Constantino wrote: > 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. If you still manu