Re: [FFmpeg-devel] Add support for 24 bit RTSP audio playback

2016-11-05 Thread Michael Niedermayer
On Sat, Nov 05, 2016 at 01:15:44AM +0300, Timur Aydin wrote: > Hello guys, > > When ffplay is used to play from the RTSP URL that serves 24 bit > audio content, ffplay fails to recognize the audio codec format. The > attached patch adds support for playing 24 bit audio content over > RTSP by defin

Re: [FFmpeg-devel] [PATCH v1] avformat/flvenc: add add_keyframe_index option

2016-11-05 Thread Michael Niedermayer
On Sat, Nov 05, 2016 at 10:47:29AM +0800, Steven Liu wrote: > 2016-11-04 23:04 GMT+08:00 Michael Niedermayer : > > > On Fri, Nov 04, 2016 at 10:47:35PM +0800, Steven Liu wrote: > > > 2016-11-04 22:38 GMT+08:00 Michael Niedermayer : > > > > > > > On Fri, Nov 04, 2016 at 08:58:56PM +0800, Steven Liu

[FFmpeg-devel] Fwd: [devroom-managers] Open Media devroom FOSDEM 2017 - Call For Participation

2016-11-05 Thread Kieran Kunhya
Hi all, Following a successful Open Media room at FOSDEM 2015 and 2016, we are organizing another Open Media room at FOSDEM 2017, and we are looking for propositions of talks and panels. FOSDEM is one of the largest (5,000+ hackers!) gatherings of Free Software contributors in the world and happen

[FFmpeg-devel] [PATCH] avformat/flvenc: add add_keyframe_index option

2016-11-05 Thread Steven Liu
Add keyframe index metadata Used to facilitate seeking; particularly for HTTP pseudo streaming. Reviewed-by: Lou Logan Signed-off-by: Steven Liu Signed-off-by: Steven Liu --- doc/muxers.texi |3 + libavformat/flvenc.c | 331 -- 2 files

[FFmpeg-devel] [PATCH v2] avformat/flvenc: add add_keyframe_index option

2016-11-05 Thread Steven Liu
Add keyframe index metadata Used to facilitate seeking; particularly for HTTP pseudo streaming. 1. read live streaming or file by sequence 2. if use add_keyframe_index option, add a mark flag at the position, use to insert new context at the last step. 3. add the keyframes *offset* and *timesta

Re: [FFmpeg-devel] [PATCH] avformat/flvenc: add add_keyframe_index option

2016-11-05 Thread Steven Liu
Ignore this please! i have send a new version patch. [PATCH v2] avformat/flvenc: add add_keyframe_index option 2016-11-05 20:42 GMT+08:00 Steven Liu : > Add keyframe index metadata > Used to facilitate seeking; particularly for HTTP pseudo streaming. > > Reviewed-by: Lou Logan > > Signed-off-by

Re: [FFmpeg-devel] [PATCH v2] avformat/flvenc: add add_keyframe_index option

2016-11-05 Thread Michael Niedermayer
On Sat, Nov 05, 2016 at 08:44:01PM +0800, Steven Liu wrote: > Add keyframe index metadata > Used to facilitate seeking; particularly for HTTP pseudo streaming. > 1. read live streaming or file by sequence > 2. if use add_keyframe_index option, add a mark flag at the position, use to > insert new

Re: [FFmpeg-devel] [PATCH 2/2] hevc: Support extradata changes

2016-11-05 Thread Michael Niedermayer
On Wed, Nov 02, 2016 at 11:48:58AM -0400, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > Please CC. > Vittorio > > libavcodec/hevc.c | 18 ++ > libavformat/mov.c | 4 > 2 files changed, 18 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/hevc.c

Re: [FFmpeg-devel] Add support for 24 bit RTSP audio playback

2016-11-05 Thread compn
On Sat, 5 Nov 2016 01:15:44 +0300 Timur Aydin wrote: > ff_register_rtp_dynamic_payload_handlers(void) > ff_register_dynamic_payload_handler(&ff_vorbis_dynamic_handler); > ff_register_dynamic_payload_handler(&ff_vp8_dynamic_handler); > ff_register_dynamic_payload_handler(&ff_vp9_dynamic_handler);

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-11-05 Thread Michael Niedermayer
On Fri, Nov 04, 2016 at 11:37:24PM +0530, Pallavi Kumari wrote: > >> ok, do you know how to compute that or should i explain it ? > > Explanation will help. Is there any function in FFmpeg to get absolute > value of FFT? hypot() of the 2 real components of each complex value should work complex

Re: [FFmpeg-devel] Add support for 24 bit RTSP audio playback

2016-11-05 Thread Timur Aydin
Here is a simple test case: 1) Use the live555 RTSP server to serve a 24 bit wav file. 2) Play this file using ffplay as follows: $ ffplay rtsp:///audio.wav In this case, ffplay fails as follows: [ta@bonsai ~]$ ffplay rtsp://10.2.1.21:8554/wave ffplay version 3.2 Copyright (c) 2003-2016 the

[FFmpeg-devel] [PATCH 1/2] avformat/hls: Factor copying stream info to a separate function

2016-11-05 Thread Anssi Hannula
Signed-off-by: Anssi Hannula --- libavformat/hls.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 3c09dd8..6fb652c 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -1528,6 +1528,16 @@ static void add_str

Re: [FFmpeg-devel] [PATCH] hls: call avformat_find_stream_info for mpegts subdemuxer

2016-11-05 Thread Anssi Hannula
03.11.2016, 20:34, Andreas Cadhalpun kirjoitti: > On 03.11.2016 11:18, Anssi Hannula wrote: >> Andreas Cadhalpun kirjoitti 2016-11-03 02:12: >>> This fixes probing dts/eac3/mp2 in hls. >>> >>> The problem was introduced in commit >>> 04964ac311abe670fb3b60290a330f2067544b13. >>> >>> Also update th

[FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-05 Thread Anssi Hannula
Commit 04964ac311abe670f ("avformat/hls: Fix missing streams in some cases with MPEG TS") caused a regression where subdemuxer streams that use probing (e.g. dts/eac3/mp2 in mpegts) no longer get probed properly. This is because the codec parameters from the subdemuxer stream, once probed, are not

Re: [FFmpeg-devel] [PATCH v3] ffmpeg: parameters for filter thread counts

2016-11-05 Thread DeHackEd
On 10/30/2016 09:25 AM, DeHackEd wrote: > Enables specifying how many threads are available to each filtergraph. > --- > v2->v3: > - Typos in docs fixed > > v1->v2: > - Reworded documentation > - Removed hunk from ffmpeg_filter.c not directly applicable to patch > > doc/ffmpeg.texi | 10

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Fallback to finding non-keyframe in fix_index, if keyframe search fails.

2016-11-05 Thread Derek Buitenhuis
On 11/4/2016 9:53 PM, Sasi Inguva wrote: > So, if I am not wrong, the only case when that would happen is, if edit > list media time was < 0 . And that would be a corrupt MOV file, unless edit > list media time = -1. > I tried to create such a file with AtomInspector but it doesn't let me. FWIW, t

Re: [FFmpeg-devel] Add support for 24 bit RTSP audio playback

2016-11-05 Thread Michael Niedermayer
On Sat, Nov 05, 2016 at 09:21:46AM -0400, compn wrote: > On Sat, 5 Nov 2016 01:15:44 +0300 > Timur Aydin wrote: > > > ff_register_rtp_dynamic_payload_handlers(void) > > ff_register_dynamic_payload_handler(&ff_vorbis_dynamic_handler); > > ff_register_dynamic_payload_handler(&ff_vp8_dynamic_handler

Re: [FFmpeg-devel] [PATCH] workaround for IOS9 getaddrinfo in IPv6 only network use hardcode IPv4 address can not resolve port number.

2016-11-05 Thread Michael Niedermayer
On Wed, Aug 24, 2016 at 10:46:41AM +, liu jc wrote: > Signed-off-by: liujingchao > --- > libavformat/tcp.c | 8 > 1 file changed, 8 insertions(+) > mode change 100644 => 100755 libavformat/tcp.c applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-05 Thread Hendrik Leppkes
On Sat, Nov 5, 2016 at 5:39 PM, Anssi Hannula wrote: > Commit 04964ac311abe670f ("avformat/hls: Fix missing streams in some > cases with MPEG TS") caused a regression where subdemuxer streams that > use probing (e.g. dts/eac3/mp2 in mpegts) no longer get probed properly. > > This is because the co

Re: [FFmpeg-devel] [PATCH 1/2] avformat/hls: Factor copying stream info to a separate function

2016-11-05 Thread Andreas Cadhalpun
On 05.11.2016 17:38, Anssi Hannula wrote: > Signed-off-by: Anssi Hannula > --- > libavformat/hls.c | 18 +++--- > 1 file changed, 11 insertions(+), 7 deletions(-) > > diff --git a/libavformat/hls.c b/libavformat/hls.c > index 3c09dd8..6fb652c 100644 > --- a/libavformat/hls.c > +++ b/

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-05 Thread Andreas Cadhalpun
On 05.11.2016 17:39, Anssi Hannula wrote: > Commit 04964ac311abe670f ("avformat/hls: Fix missing streams in some > cases with MPEG TS") caused a regression where subdemuxer streams that > use probing (e.g. dts/eac3/mp2 in mpegts) no longer get probed properly. > > This is because the codec paramet

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-05 Thread Andreas Cadhalpun
On 05.11.2016 18:47, Andreas Cadhalpun wrote: > On 05.11.2016 17:39, Anssi Hannula wrote: >> @@ -1981,6 +1986,11 @@ static int hls_read_packet(AVFormatContext *s, >> AVPacket *pkt) >> >> pls->ctx->streams[pls->pkt.stream_index]->time_base, >>

Re: [FFmpeg-devel] [libav-devel] [PATCH] lzf: update pointer p after realloc

2016-11-05 Thread Andreas Cadhalpun
On 05.11.2016 01:36, Luca Barbato wrote: > On 04/11/2016 23:33, Andreas Cadhalpun wrote: >> This fixes heap-use-after-free detected by AddressSanitizer. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/lzf.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/libavcodec/lzf.c

Re: [FFmpeg-devel] [PATCH] mxfdec: fix NULL pointer dereference

2016-11-05 Thread Andreas Cadhalpun
On 05.11.2016 02:09, Michael Niedermayer wrote: > On Sat, Nov 05, 2016 at 12:20:21AM +0100, Andreas Cadhalpun wrote: >> Metadata streams have priv_data set to NULL. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/mxfdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > >

Re: [FFmpeg-devel] [PATCH] lavf/mpegtsenc: write metadata descriptor for timed ID3 packets

2016-11-05 Thread Michael Niedermayer
On Sun, Aug 21, 2016 at 06:23:21PM +0200, Michael Niedermayer wrote: > On Sun, Aug 21, 2016 at 10:44:27AM +0200, Stefano Sabatini wrote: > > On date Saturday 2016-08-20 18:57:52 +0200, Michael Niedermayer encoded: > > > On Sat, Aug 20, 2016 at 03:48:35PM +0200, Stefano Sabatini wrote: > > > > On da

Re: [FFmpeg-devel] [PATCH 1/6] lavc/qsv(hevc): Change default plugin from hevc_sw to hevc_default, which will load hevc_hw first, due to newly released MSDK.

2016-11-05 Thread Michael Niedermayer
On Sun, Oct 09, 2016 at 12:51:42PM +0200, Michael Niedermayer wrote: > Hi > > Patches 1,5,6 of this patchset have no comments, are they rejected > need changes or something else ? patches dont apply anymore ill mark them as not applicable and consider them droped this lack of interrest in qsv is

Re: [FFmpeg-devel] [PATCH 1/6] lavc/qsv(hevc): Change default plugin from hevc_sw to hevc_default, which will load hevc_hw first, due to newly released MSDK.

2016-11-05 Thread James Almer
On 11/5/2016 3:21 PM, Michael Niedermayer wrote: > On Sun, Oct 09, 2016 at 12:51:42PM +0200, Michael Niedermayer wrote: >> Hi >> >> Patches 1,5,6 of this patchset have no comments, are they rejected >> need changes or something else ? > > patches dont apply anymore > ill mark them as not applicabl

Re: [FFmpeg-devel] [PATCH v3] ffmpeg: parameters for filter thread counts

2016-11-05 Thread Michael Niedermayer
On Sun, Oct 30, 2016 at 09:25:39AM -0400, DeHackEd wrote: > Enables specifying how many threads are available to each filtergraph. > --- > v2->v3: > - Typos in docs fixed > > v1->v2: > - Reworded documentation > - Removed hunk from ffmpeg_filter.c not directly applicable to patch > > doc/ffmpeg.

Re: [FFmpeg-devel] [PATCH] avutil/softfloat_ieee754: make all functions inline

2016-11-05 Thread James Almer
On 11/5/2016 5:50 PM, Michael Niedermayer wrote: > On Fri, Sep 16, 2016 at 11:44:48AM +0200, Michael Niedermayer wrote: >> On Thu, Sep 15, 2016 at 11:08:47PM -0300, James Almer wrote: >>> Removes "defined but not used" warnings >>> >>> Signed-off-by: James Almer >>> --- >>> libavutil/softfloat_ie

[FFmpeg-devel] [PATCH] avformat/matroskadec: fix DiscardPadding element parsing

2016-11-05 Thread James Almer
If the value is negative then it means padding at the start of the packet instead of at the end. Based on a patch by Hendrik Leppkes. Signed-off-by: James Almer --- libavformat/matroskadec.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/libavformat/matroskadec

Re: [FFmpeg-devel] [PATCH] avutil/softfloat_ieee754: make all functions inline

2016-11-05 Thread Michael Niedermayer
On Fri, Sep 16, 2016 at 11:44:48AM +0200, Michael Niedermayer wrote: > On Thu, Sep 15, 2016 at 11:08:47PM -0300, James Almer wrote: > > Removes "defined but not used" warnings > > > > Signed-off-by: James Almer > > --- > > libavutil/softfloat_ieee754.h | 14 +++--- > > 1 file changed, 7

Re: [FFmpeg-devel] [PATCH v3 01/25] avformat/movenc: mov_write_ftyp_tag: write the major brand as a compatible brand

2016-11-05 Thread Michael Niedermayer
On Mon, Sep 19, 2016 at 04:25:24PM +0300, Erkki Seppälä wrote: > when the option "brand" is used. This allows custom brands to end up in > the compatible brands as well. > > Signed-off-by: Erkki Seppälä > Signed-off-by: OZOPlayer > --- > libavformat/movenc.c | 2 ++ > 1 file changed, 2 insertio

Re: [FFmpeg-devel] [PATCH v3 03/25] avformat/movenc: support for multiple and client-provided track references

2016-11-05 Thread Michael Niedermayer
On Mon, Sep 19, 2016 at 04:25:26PM +0300, Erkki Seppälä wrote: > Instead of one track reference, allow many, and instead of > of track reference type (ie. 'cdsc'), allow many. > > In addition this patch allows client to explicitly add track references > with side packet AV_PKT_DATA_TRACK_REFERENCE

Re: [FFmpeg-devel] [PATCH v3 04/25] avformat/mov, isom: read (multiple) track references (tag and multiple ids)

2016-11-05 Thread Michael Niedermayer
On Mon, Sep 19, 2016 at 04:25:27PM +0300, Erkki Seppälä wrote: > This can be useful in particular with timed meta data tracks related > to multiple tracks. > > Signed-off-by: Erkki Seppälä > Signed-off-by: OZOPlayer > --- > libavformat/isom.h | 3 ++ > libavformat/mov.c | 119 >

Re: [FFmpeg-devel] [PATCH v3 10/25] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

2016-11-05 Thread Michael Niedermayer
On Mon, Sep 19, 2016 at 04:25:33PM +0300, Erkki Seppälä wrote: > This includes creating an AVCodecTag table ff_codec_metadata_tags as > there are for video, audio and subtitles. The tag table is used for > mov-compatiblity. > > Signed-off-by: Erkki Seppälä > Signed-off-by: OZOPlayer > --- > lib

Re: [FFmpeg-devel] [PATCH v3 11/25] avformat/movenc: add the urim sample descriptor

2016-11-05 Thread Michael Niedermayer
On Mon, Sep 19, 2016 at 04:25:34PM +0300, Erkki Seppälä wrote: > This also adds libavformat/movmeta that contains the meta data > information. Actually the sample configuration data is performed with > side packet data AV_PKT_DATA_TIMED_METADATA_INFO where the value is > stored inside AVTimedMetada

Re: [FFmpeg-devel] [PATCH] avformat/framehash: also print channel layout as a string

2016-11-05 Thread Michael Niedermayer
On Wed, Sep 28, 2016 at 09:05:07PM -0300, James Almer wrote: > This should be more useful for users since numerical values for channel > layout can be confusing and unintuitive. > > Signed-off-by: James Almer LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040

Re: [FFmpeg-devel] [PATCH] avformat/mux: remove unnecessary autobsf hack

2016-11-05 Thread Michael Niedermayer
On Fri, Sep 23, 2016 at 02:04:08AM -0300, James Almer wrote: > autobsf has been ported to the new bsf API. > > Signed-off-by: James Almer > --- > libavformat/mux.c | 6 -- > 1 file changed, 6 deletions(-) has this patch been forgotten ? [...] -- Michael GnuPG fingerprint: 9FF2128B147E

Re: [FFmpeg-devel] [PATCH] avformat/mux: remove unnecessary autobsf hack

2016-11-05 Thread James Almer
On 11/5/2016 9:46 PM, Michael Niedermayer wrote: > On Fri, Sep 23, 2016 at 02:04:08AM -0300, James Almer wrote: >> autobsf has been ported to the new bsf API. >> >> Signed-off-by: James Almer >> --- >> libavformat/mux.c | 6 -- >> 1 file changed, 6 deletions(-) > > has this patch been forgot

Re: [FFmpeg-devel] [PATCH v3] ffmpeg: parameters for filter thread counts

2016-11-05 Thread DeHackEd
On 11/05/2016 02:51 PM, Michael Niedermayer wrote: > On Sun, Oct 30, 2016 at 09:25:39AM -0400, DeHackEd wrote: >> Enables specifying how many threads are available to each filtergraph. >> --- >> v2->v3: >> - Typos in docs fixed >> >> v1->v2: >> - Reworded documentation >> - Removed hunk from ffmpeg

Re: [FFmpeg-devel] [PATCH] avformat/mux: remove unnecessary autobsf hack

2016-11-05 Thread James Almer
On 11/5/2016 9:49 PM, James Almer wrote: > On 11/5/2016 9:46 PM, Michael Niedermayer wrote: >> On Fri, Sep 23, 2016 at 02:04:08AM -0300, James Almer wrote: >>> autobsf has been ported to the new bsf API. >>> >>> Signed-off-by: James Almer >>> --- >>> libavformat/mux.c | 6 -- >>> 1 file chang

Re: [FFmpeg-devel] [PATCH] avformat/framehash: also print channel layout as a string

2016-11-05 Thread James Almer
On 11/5/2016 8:55 PM, Michael Niedermayer wrote: > On Wed, Sep 28, 2016 at 09:05:07PM -0300, James Almer wrote: >> This should be more useful for users since numerical values for channel >> layout can be confusing and unintuitive. >> >> Signed-off-by: James Almer > > LGTM > > thx Pushed. _

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Fallback to finding non-keyframe in fix_index, if keyframe search fails.

2016-11-05 Thread Sasi Inguva
When I play the file using Quicktime Player 7 , it errs out saying "invalid public movie atom found" . Quicktime 10.4 shows a black screen . Even vlc doesn't play the video. It's true that it used to work before, but don't know if we need to support it. On Sat, Nov 5, 2016 at 10:25 PM, Derek Buit