Re: [FFmpeg-devel] [PATCH] interplayacm: increase bitstream buffer size by AV_INPUT_BUFFER_PADDING_SIZE

2016-10-31 Thread Andreas Cadhalpun
On 31.10.2016 08:33, Paul B Mahol wrote: > On 10/30/16, Andreas Cadhalpun wrote: >> On 30.10.2016 22:18, Paul B Mahol wrote: >>> On 10/30/16, Andreas Cadhalpun wrote: This fixes out-of-bounds reads by the bitstream reader.

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-10-31 Thread Zhenni Huang
On Mon, Oct 24, 2016 at 4:48 PM, Carl Eugen Hoyos wrote: > 2016-10-25 1:23 GMT+02:00 Zhenni Huang peg.org>: > > > Thanks for your reply. Setting strict_std_compliance to 2 could > > help in this case. However, as it is a global flag, it could

[FFmpeg-devel] [PATCH] pixblockdsp: disable altivec optimizations on ppc64be

2016-10-31 Thread Andreas Cadhalpun
The checkasm test fails, see trac ticket 5508. Also, the following tests fail due to this: fate-vsynth1-dnxhd-2k-hr-hq fate-vsynth1-dnxhd-edge1-hr fate-vsynth1-dnxhd-edge2-hr fate-vsynth1-dnxhd-edge3-hr fate-vsynth1-dnxhd-hr-sq-mov fate-vsynth1-dnxhd-hr-hq-mov fate-vsynth2-dnxhd-2k-hr-hq

Re: [FFmpeg-devel] [PATCH] pixblockdsp: disable altivec optimizations on ppc64be

2016-10-31 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 12:27:00AM +0100, Andreas Cadhalpun wrote: > The checkasm test fails, see trac ticket 5508. > > Also, the following tests fail due to this: > fate-vsynth1-dnxhd-2k-hr-hq fate-vsynth1-dnxhd-edge1-hr > fate-vsynth1-dnxhd-edge2-hr fate-vsynth1-dnxhd-edge3-hr >

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Set duration to zero if the duration is UINT_MAX

2016-10-31 Thread Sasi Inguva
For MP4F files , stream duration is updated by the sum of duration of moof's http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/mov.c;h=414007e7aa128aa66e07395b42c0cd4b369b3146;hb=HEAD#l4193 , only if the duration in the header is less than that. Normally duration field in header is 0, so

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Set duration to zero if the duration is UINT_MAX

2016-10-31 Thread Yusuke Nakamura
2016-10-27 4:40 GMT+09:00 Sasi Inguva : > Fixes some MP4F files which have duration in mdhd set to UINT_MAX instead > of zero. > What does this patch fix? > > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c | 5 + > 1 file changed, 5

Re: [FFmpeg-devel] [PATCH] mov: only read e_old if there were any old streams

2016-10-31 Thread Andreas Cadhalpun
On 31.10.2016 19:20, Sasi Inguva wrote: > First of all, if nb_old == 0 i.e. there are no entries in AVIndex, then > there is no point in calling mov_fix_index function at all. So instead of > doing the above , you can directly check for st->nb_index_entries > 0 at > the top of mov_fix_index and

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-31 Thread Steven Liu
2016-11-01 3:16 GMT+08:00 Jan Ekstrom : > On Mon, Oct 31, 2016 at 5:30 PM, Nicolas George wrote: > > Le nonidi 9 brumaire, an CCXXV, Steven Liu a écrit : > >> I saw ffmpeg have no HDS and DASH demuxer, and all of them's > format is > >> use xml, maybe this

Re: [FFmpeg-devel] [PATCH 1/2] Add experimental muxing support for FLAC in ISO BMFF (MP4).

2016-10-31 Thread Matthew Gregan
At 2016-10-31T11:04:16-0300, James Almer wrote: > > +static int mov_write_dfla_tag(AVIOContext *pb, MOVTrack *track) > > +{ > > +const size_t FLAC_STREAMINFO_SIZE = 34; > > +int64_t pos = avio_tell(pb); > > +avio_wb32(pb, 0); > > +ffio_wfourcc(pb, "dfLa"); > > +avio_w8(pb, 0);

[FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-10-31 Thread James Almer
Fixes remuxing apng streams coming from the apng demuxer. This is a regression since 97792e85c338d129342f5812e2a52048373e57d6. Signed-off-by: James Almer --- libavformat/apngenc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/apngenc.c

Re: [FFmpeg-devel] [PATCH 1/2] Add experimental muxing support for FLAC in ISO BMFF (MP4).

2016-10-31 Thread James Almer
On 10/31/2016 10:11 PM, Matthew Gregan wrote: > At 2016-10-31T11:04:16-0300, James Almer wrote: >>> +static int mov_write_dfla_tag(AVIOContext *pb, MOVTrack *track) >>> +{ >>> +const size_t FLAC_STREAMINFO_SIZE = 34; >>> +int64_t pos = avio_tell(pb); >>> +avio_wb32(pb, 0); >>> +

[FFmpeg-devel] [PATCH] avcodec/dnxhdenc: Fix alignment of edge_buf*

2016-10-31 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/dnxhdenc.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/dnxhdenc.h b/libavcodec/dnxhdenc.h index 91ef6d0..eb9da12 100644 --- a/libavcodec/dnxhdenc.h +++ b/libavcodec/dnxhdenc.h @@

Re: [FFmpeg-devel] [PATCH] pixblockdsp: disable altivec optimizations on ppc64be

2016-10-31 Thread Andreas Cadhalpun
On 01.11.2016 02:04, Michael Niedermayer wrote: > On Tue, Nov 01, 2016 at 12:27:00AM +0100, Andreas Cadhalpun wrote: >> The checkasm test fails, see trac ticket 5508. >> >> Also, the following tests fail due to this: >> fate-vsynth1-dnxhd-2k-hr-hq fate-vsynth1-dnxhd-edge1-hr >>

Re: [FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-10-31 Thread James Almer
On 10/31/2016 11:32 PM, James Almer wrote: > Fixes remuxing apng streams coming from the apng demuxer. > This is a regression since 97792e85c338d129342f5812e2a52048373e57d6. Should be 940b8908b94404a65f9f55e33efb4ccc6c81383c, sorry. > > Signed-off-by: James Almer > --- >

Re: [FFmpeg-devel] [PATCH] mov: only read e_old if there were any old streams

2016-10-31 Thread Sasi Inguva
patch looks good to me. Thanks for the fix. On Mon, Oct 31, 2016 at 5:17 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 31.10.2016 19:20, Sasi Inguva wrote: > > First of all, if nb_old == 0 i.e. there are no entries in AVIndex, then > > there is no point in calling

Re: [FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-10-31 Thread James Almer
On 10/31/2016 11:32 PM, James Almer wrote: > Fixes remuxing apng streams coming from the apng demuxer. > This is a regression since 97792e85c338d129342f5812e2a52048373e57d6. > > Signed-off-by: James Almer > --- > libavformat/apngenc.c | 7 +++ > 1 file changed, 7

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 3:30 PM, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : >> It's an scenario that could happen, like it or not. I'd rather not open the >> doors for it. > > "Opening doors" sounds like a different wording for the slippery slope > fallacy. Would you

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara > > wrote: > >> > >> On

Re: [FFmpeg-devel] [PATCH 1/3] vf_colorspace: Add support for iec61966-2.4 (xvYCC) transfer

2016-10-31 Thread Michael Niedermayer
On Sun, Oct 30, 2016 at 09:14:47AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > > Signed-off-by: Vittorio Giovara > > --- > > As described in

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Kevin Wheatley
On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje wrote: > Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 refers > to the two whitepoints here as DCI-P3 D65 and DCI-P3 Theater. Calling one > D65 and the other DCI seems confusing in that light (assuming the

Re: [FFmpeg-devel] [PATCH] interplayacm: increase bitstream buffer size by AV_INPUT_BUFFER_PADDING_SIZE

2016-10-31 Thread Paul B Mahol
On 10/30/16, Andreas Cadhalpun wrote: > On 30.10.2016 22:18, Paul B Mahol wrote: >> On 10/30/16, Andreas Cadhalpun wrote: >>> This fixes out-of-bounds reads by the bitstream reader. >>> >>> Signed-off-by: Andreas Cadhalpun

[FFmpeg-devel] [PATCH 1/2] Add experimental muxing support for FLAC in ISO BMFF (MP4).

2016-10-31 Thread Matthew Gregan
Based on the draft spec at https://git.xiph.org/?p=flac.git;a=blob;f=doc/isoflac.txt '-strict -2' is required to create files in this format. Signed-off-by: Matthew Gregan --- libavformat/isom.c | 2 ++ libavformat/movenc.c | 43 +--

[FFmpeg-devel] [PATCH 2/2] Add experimental demuxing support for FLAC in ISO BMFF (MP4).

2016-10-31 Thread Matthew Gregan
Based on the draft spec at https://git.xiph.org/?p=flac.git;a=blob;f=doc/isoflac.txt Signed-off-by: Matthew Gregan --- libavformat/mov.c | 41 + 1 file changed, 41 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley wrote: > On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje > wrote: > > Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 > refers > > to the two whitepoints here as DCI-P3 D65

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

2016-10-31 Thread Derek Buitenhuis
On 10/27/2016 10:48 PM, Sasi Inguva wrote: > gentle ping. > > Thanks! A ping from me, too. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

2016-10-31 Thread Rostislav Pehlivanov
On 31 October 2016 at 13:13, Derek Buitenhuis wrote: > On 10/27/2016 10:48 PM, Sasi Inguva wrote: > > gentle ping. > > > > Thanks! > > A ping from me, too. > > - Derek > ___ > ffmpeg-devel mailing list >

Re: [FFmpeg-devel] [PATCH 1/2] Add experimental muxing support for FLAC in ISO BMFF (MP4).

2016-10-31 Thread James Almer
On 10/31/2016 3:27 AM, Matthew Gregan wrote: > Based on the draft spec at > https://git.xiph.org/?p=flac.git;a=blob;f=doc/isoflac.txt > > '-strict -2' is required to create files in this format. > > Signed-off-by: Matthew Gregan > --- > libavformat/isom.c | 2 ++ >

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Jean-Baptiste Kempf
Hi, On Mon, 31 Oct 2016, at 07:03, wm4 wrote: > I very strongly disagree with this patch, but consider myself > over-voted. I strongly disagree with this patch too. -- Jean-Baptiste Kempf - President +33 672 704 734 ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, Jean-Baptiste Kempf a écrit : > I strongly disagree with this patch too. We do not care about the number of people who disagree, we only care about practical arguments. IIRC, the last argument on the matter was mine:

Re: [FFmpeg-devel] [PATCH 4/4] qsv: Merge libav implementation

2016-10-31 Thread Hendrik Leppkes
On Wed, Oct 26, 2016 at 9:50 PM, Mark Thompson wrote: > Merged as-at libav 398f015, and therefore includes outstanding > skipped merges 04b17ff and 130e1f1. > > All features not in libav are preserved, and no options change. > --- LGTM, this should make further work on this much

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 11:20 AM, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, Jean-Baptiste Kempf a écrit : >> I strongly disagree with this patch too. > > We do not care about the number of people who disagree, we only care > about practical arguments. That's a curious thing to say when we

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, James Almer a écrit : > That's a curious thing to say when we have a voting committee to > deal with polarizing subjects that went beyond practical arguments. The voting committee is there for when arguments are failed: when there are significant pros and cons on

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread wm4
On Mon, 31 Oct 2016 09:51:10 +0100 Stefano Sabatini wrote: > On date Thursday 2016-10-27 17:08:50 +0200, Stefano Sabatini encoded: > > On date Wednesday 2016-10-26 01:46:35 +0200, Michael Niedermayer encoded: > > > On Tue, Oct 25, 2016 at 05:07:34PM +0200, Stefano Sabatini

[FFmpeg-devel] why the option 'timeout' in native rtmp plugin doesn't work

2016-10-31 Thread qw
Hi, 1) I want to open and read some rtmp live stream, and don't like wait indefinite time for some rtmp stream when there is no rtmp stream to come. Therefore, the option of 'timeout' is used to set the longest waiting time. But it didn't work, and avformat_open_input() reported as follows:

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Stefano Sabatini
On date Thursday 2016-10-27 17:08:50 +0200, Stefano Sabatini encoded: > On date Wednesday 2016-10-26 01:46:35 +0200, Michael Niedermayer encoded: > > On Tue, Oct 25, 2016 at 05:07:34PM +0200, Stefano Sabatini wrote: > [...] > > > 267580c51d49daf94e73a33175c63ecfed6a0bed > > >

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-31 Thread Greg Rowe
On 2016-10-18 2:20 PM, Michael Niedermayer wrote: On Tue, Oct 18, 2016 at 12:46:56PM -0400, Greg Rowe wrote: see libavfilter/asrc_sine.c this code should probably be reused / factored (note, any code moving/factoring of existing code should be in a seperate patch) Since silenceremove works

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-31 Thread Nicolas George
Le nonidi 9 brumaire, an CCXXV, Steven Liu a écrit : > I saw ffmpeg have no HDS and DASH demuxer, and all of them's format is > use xml, maybe this parser is a very useful parser, what about the basic > xml :-D The Timed Text Markup Language, a subtitle format used by Youtube and possibly a

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, Stefano Sabatini a écrit : > So, in practical terms, do you want me to start a formal vote about > the inclusion of the format? NO! Until ten minutes ago, there was no actual arguments against. The technical discussion just started. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Jean-Baptiste Kempf: > ffprobe is not a video/audio format. > It has no public specification, is made up and completely arbitrary > format. > It will not be used by the large majority of applications. Indeed. The same goes for a lot of obscure formats used to decode game cutscene data. At least,

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 11:30 AM, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : >> That's a curious thing to say when we have a voting committee to >> deal with polarizing subjects that went beyond practical arguments. > > The voting committee is there for when arguments

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Vittorio Giovara
On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley > wrote: >> >> On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje >> wrote: >> > Hmm... So, the wikipedia page

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Jean-Baptiste Kempf
Hi, On Mon, 31 Oct 2016, at 07:20, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, Jean-Baptiste Kempf a écrit : > > I strongly disagree with this patch too. > > If you, or anyone else opposing it, has counter-arguments, please give > them. Otherwise, the patch should go in once it is

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Stefano Sabatini
On date Monday 2016-10-31 11:27:17 -0300, James Almer encoded: > On 10/31/2016 11:20 AM, Nicolas George wrote: > > Le decadi 10 brumaire, an CCXXV, Jean-Baptiste Kempf a écrit : > >> I strongly disagree with this patch too. > > > > We do not care about the number of people who disagree, we only

Re: [FFmpeg-devel] [PATCH v2] libavformat/tee: Add fifo support for tee

2016-10-31 Thread Nicolas George
Le sextidi 26 vendémiaire, an CCXXV, sebechlebsky...@gmail.com a écrit : > From: Jan Sebechlebsky > > Signed-off-by: Jan Sebechlebsky > --- > Thanks for noticing, I've fixed the patch > (also some minor formatting issues I've noticed). >

Re: [FFmpeg-devel] [PATCH] news: add final report for summer of code 2016

2016-10-31 Thread Thomas Volkert
On 31.10.2016 04:47, Reynaldo H. Verdejo Pinochet wrote: > Thanks for the comments and corrections. Fixed the typos and pushed as: > > commit f06598a8e1fcccef8c38a657162db309773d1515 > Author: Reynaldo H. Verdejo Pinochet > Date: Sun Oct 30 01:37:26 2016 -0700 > >

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, James Almer a écrit : > So you're saying no matter how many people veto a patch it means shit and > they will be committed anyway because the writer of the patch and one other > person want to? > > One author, one positive review and seven billion Nack means the

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, James Almer a écrit : > Someone will come up with an hexdump demuxer next and call hexdump output > a multimedia file, if we follow your views. Or an XML parser for a similarly > arbitrary custom format they happened to devise. Or maybe oggz-dump's output. > >

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread wm4
On Mon, 31 Oct 2016 15:20:15 +0100 Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, Jean-Baptiste Kempf a écrit : > > I strongly disagree with this patch too. > > We do not care about the number of people who disagree, we only care > about practical arguments. > >

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 1:27 PM, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, wm4 a écrit : >> I don't have the energy for such long-winded technological discussions >> which (predictably) lead to nothing. I just want my disagreement noted. > > Well, you noted it, and I personally do not care

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, wm4 a écrit : > I don't have the energy for such long-winded technological discussions > which (predictably) lead to nothing. I just want my disagreement noted. Well, you noted it, and I personally do not care at all without arguments. > If we have Reviewed-by

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 1:36 PM, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : >> So you're saying no matter how many people veto a patch it means shit and >> they will be committed anyway because the writer of the patch and one other >> person want to? >> >> One author, one

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-10-31 Thread Sasi Inguva
ping. Thanks! On Wed, Oct 26, 2016 at 11:31 AM, Sasi Inguva wrote: > According to spec ISO_IEC_15444_12 "For any media stream for which no > segment index is present, referred to as non‐indexed stream, the media > stream associated with the first Segment Index box in the

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 11:57 AM, Nicolas George wrote: > Jean-Baptiste Kempf: >> ffprobe is not a video/audio format. >> It has no public specification, is made up and completely arbitrary >> format. >> It will not be used by the large majority of applications. > > Indeed. The same goes for a lot of

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, James Almer a écrit : > You not liking an argument doesn't mean the argument doesn't exist. Indeed. On the other hand, someone saying they do not intend to give arguments means the argument does not exist. That is exactly the short and long version of wm4's

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Set duration to zero if the duration is UINT_MAX

2016-10-31 Thread Sasi Inguva
ping. Thanks! On Wed, Oct 26, 2016 at 12:42 PM, Sasi Inguva wrote: > Attaching the file that it fixes. > > > On Wed, Oct 26, 2016 at 12:40 PM, Sasi Inguva wrote: > >> Fixes some MP4F files which have duration in mdhd set to UINT_MAX instead >> of zero. >>

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Kevin Wheatley
I would really strongly suggest including DCI in the name at least - though nobody else would choose to use it for anything other than the reference calibration - most titles use a creative white different to that of the encoding reference (one that is less green). Kevin

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread James Almer
On 10/31/2016 1:54 PM, Nicolas George wrote: > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : >> Someone will come up with an hexdump demuxer next and call hexdump output >> a multimedia file, if we follow your views. Or an XML parser for a similarly >> arbitrary custom format they

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Michael Niedermayer
On Mon, Oct 31, 2016 at 02:41:05PM -0300, James Almer wrote: > On 10/31/2016 1:54 PM, Nicolas George wrote: > > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : [...] > > > >> Which so happens to be what every libav* user has to do for their projects. > >> Write a program using the

Re: [FFmpeg-devel] [PATCH] mov: only read e_old if there were any old streams

2016-10-31 Thread Sasi Inguva
First of all, if nb_old == 0 i.e. there are no entries in AVIndex, then there is no point in calling mov_fix_index function at all. So instead of doing the above , you can directly check for st->nb_index_entries > 0 at the top of mov_fix_index and return otherwise. Also, I don't understand how

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, James Almer a écrit : > It's an scenario that could happen, like it or not. I'd rather not open the > doors for it. "Opening doors" sounds like a different wording for the slippery slope fallacy. > "why this format and not this other" is coincidentally the

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXV, Josh de Kock a écrit : > I am extremely against the inclusion of this patch as well. It's something > which has really limited use-cases and should be a standalone tool. Duplicate argument. Regards, -- Nicolas George signature.asc Description: Digital

Re: [FFmpeg-devel] [PATCH 4/4] qsv: Merge libav implementation

2016-10-31 Thread Mark Thompson
On 31/10/16 13:50, Hendrik Leppkes wrote: > On Wed, Oct 26, 2016 at 9:50 PM, Mark Thompson wrote: >> Merged as-at libav 398f015, and therefore includes outstanding >> skipped merges 04b17ff and 130e1f1. >> >> All features not in libav are preserved, and no options change. >> --- >

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: remove MFX_EXTBUFF_CODING_OPTION3

2016-10-31 Thread Mark Thompson
On 18/06/16 05:33, zera...@gmail.com wrote: > From: Kyle Schwarz > > 4th generation Intel CPUs don't support MFX_EXTBUFF_CODING_OPTION3. > > This patch fixes bug #5324. Applied with the qsv merge from libav. Thank you! - Mark

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

2016-10-31 Thread Michael Niedermayer
On Mon, Oct 31, 2016 at 01:48:26PM +, Rostislav Pehlivanov wrote: > On 31 October 2016 at 13:13, Derek Buitenhuis > wrote: > > > On 10/27/2016 10:48 PM, Sasi Inguva wrote: > > > gentle ping. > > > > > > Thanks! > > > > A ping from me, too. > > > > - Derek > >

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley < > kevin.j.wheat...@gmail.com> > > wrote: > >> > >> On

Re: [FFmpeg-devel] [PATCH 3/4] V14 - SCTE-35 support in hlsenc

2016-10-31 Thread Carlos Fernandez Sanz
Hi, Just a ping in case this can be revised and/or applied soonish - thanks. CC'ing Carl Eugen and Thilo since I could discuss this a bit in person at the GSoC summit. Thanks both for your time by the way. Carlos On Tue, Oct 18, 2016 at 5:36 PM, Carlos Fernandez Sanz

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Vittorio Giovara
On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara > wrote: >> >> On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Mon, Oct 31,

[FFmpeg-devel] [PATCH] vf_colorspace: Add support for film primaries

2016-10-31 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- This is the last easy picking, the remaining trc/prm require more convoluted changes to the filter that will need more discussion. Please CC. Vittorio libavfilter/vf_colorspace.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-31 Thread Michael Niedermayer
On Mon, Oct 31, 2016 at 07:22:58PM +0100, Michael Niedermayer wrote: > On Mon, Oct 31, 2016 at 02:41:05PM -0300, James Almer wrote: > > On 10/31/2016 1:54 PM, Nicolas George wrote: > > > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : > [...] > > > > > >> Which so happens to be what every

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for film primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 2:43 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > This is the last easy picking, the remaining trc/prm require more > convoluted > changes to the filter that will need more discussion.

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-31 Thread Jan Ekstrom
On Mon, Oct 31, 2016 at 5:30 PM, Nicolas George wrote: > Le nonidi 9 brumaire, an CCXXV, Steven Liu a écrit : >> I saw ffmpeg have no HDS and DASH demuxer, and all of them's format is >> use xml, maybe this parser is a very useful parser, what about the basic >> xml :-D > >