Re: [FFmpeg-devel] [PATCH] avformat/flvenc: fix ticket 5976 and use old commit

2016-11-25 Thread Steven Liu
2016-11-26 9:39 GMT+08:00 Lou Logan : > On Sat, 26 Nov 2016 09:19:42 +0800, Steven Liu wrote: > > > 2016-11-24 11:49 GMT+08:00 Steven Liu : > > > > > mythtv have problem with non-seekable dont write duration and filesize > > > and there have problem with some

Re: [FFmpeg-devel] [PATCH] avformat/flvenc: fix ticket 5976 and use old commit

2016-11-25 Thread Lou Logan
On Sat, 26 Nov 2016 09:19:42 +0800, Steven Liu wrote: > 2016-11-24 11:49 GMT+08:00 Steven Liu : > > > mythtv have problem with non-seekable dont write duration and filesize > > and there have problem with some other server and player with 0 value > > duation and filesize. >

Re: [FFmpeg-devel] [PATCH] avformat/flvenc: fix ticket 5976 and use old commit

2016-11-25 Thread Steven Liu
2016-11-24 11:49 GMT+08:00 Steven Liu : > mythtv have problem with non-seekable dont write duration and filesize > and there have problem with some other server and player with 0 value > duation and filesize. > So add a flv flags to fix the ticket and make a choose for

Re: [FFmpeg-devel] [PATCH] avformat/flvdec: move set bit_rate from parse AMF OBJECT to create_stream

2016-11-25 Thread Steven Liu
2016-11-26 2:49 GMT+08:00 Michael Niedermayer : > On Fri, Nov 25, 2016 at 05:18:43PM +0800, Steven Liu wrote: > > before patch: > > Stream #0:0: Video: h264 (High), yuv420p(progressive), 1280x714 [SAR 1:1 > DAR 640:357], 25 fps, 25 tbr, 1k tbn, 50 tbc > > > > after patch:

Re: [FFmpeg-devel] [PATCH] lavf: always forward codec_whitelist in avformat_find_stream_info

2016-11-25 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 09:57:57PM +0100, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavformat/utils.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] pngdec: check if previous frame exists instead of trusting sequence_number

2016-11-25 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 10:13:06PM +0100, Andreas Cadhalpun wrote: > This fixes a segmentation fault caused by calling memcpy with NULL as > second argument in handle_p_frame_apng. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/pngdec.c | 2 +- > 1

Re: [FFmpeg-devel] [PATCH 1/2] swresample/soxr: fix invalid use of linear_interp

2016-11-25 Thread Muhammad Faiz
On 11/25/16, Michael Niedermayer wrote: > On Fri, Nov 25, 2016 at 04:55:34PM +0700, Muhammad Faiz wrote: >> give very bad quality for soxr resampler. >> linear_interp is intended for using linear interpolation >> between filter bank so quality will be better. >> >> i

Re: [FFmpeg-devel] [PATCH] avcodec/aac_adtstoasc: validate and forward extradata if the stream is already ASC

2016-11-25 Thread James Almer
On 11/24/2016 9:18 PM, James Almer wrote: > Fixes ticket #5973 > > Signed-off-by: James Almer > --- > libavcodec/aac_adtstoasc_bsf.c | 12 ++-- > 1 file changed, 10 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/aac_adtstoasc_bsf.c

Re: [FFmpeg-devel] [PATCH] Remove the ffserver program and the ffm muxer/demuxer

2016-11-25 Thread Andreas Cadhalpun
On 25.11.2016 22:22, wm4 wrote: > On Fri, 25 Nov 2016 21:13:54 +0100 > Andreas Cadhalpun wrote: > >> On 25.11.2016 10:36, wm4 wrote: >>> I'd like to add that removing the ffserver support from the libs, and >>> all deprecated stuff it depends on, will make it

Re: [FFmpeg-devel] [PATCH 1/2] vc1dec: support multiple slices in frame coded images with hwaccel

2016-11-25 Thread Hendrik Leppkes
On Sun, Nov 20, 2016 at 7:14 PM, Carl Eugen Hoyos wrote: > 2016-11-20 17:50 GMT+01:00 Mark Thompson : >> On 20/11/16 16:16, Carl Eugen Hoyos wrote: > >>> There is no API to read the version of the mesa driver? >> >> The one useful call is vaQueryVendorString(),

Re: [FFmpeg-devel] [PATCH] matroska read_seek: make max diff for last known subtitle configurable

2016-11-25 Thread wm4
On Fri, 25 Nov 2016 07:48:40 +0100 Rainer Hochecker wrote: > I was tempted to submit a patch with this feature deleted. Unfortunately I > was not able to find the original reason > why this was implemented. I am sure there was a good reason and this reason > may still be a

Re: [FFmpeg-devel] [PATCH] Remove the ffserver program and the ffm muxer/demuxer

2016-11-25 Thread wm4
On Fri, 25 Nov 2016 21:13:54 +0100 Andreas Cadhalpun wrote: > On 25.11.2016 10:36, wm4 wrote: > > I'd like to add that removing the ffserver support from the libs, and > > all deprecated stuff it depends on, will make it actually easier to > > readd ffserver in

[FFmpeg-devel] [PATCH] pngdec: check if previous frame exists instead of trusting sequence_number

2016-11-25 Thread Andreas Cadhalpun
This fixes a segmentation fault caused by calling memcpy with NULL as second argument in handle_p_frame_apng. Signed-off-by: Andreas Cadhalpun --- libavcodec/pngdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/pngdec.c

[FFmpeg-devel] [PATCH] lavf: always forward codec_whitelist in avformat_find_stream_info

2016-11-25 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavformat/utils.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 2634317..b1d0a0a 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: Prevent access of elements after freeing by set nb_elem to zero.

2016-11-25 Thread Andreas Cadhalpun
On 25.11.2016 11:04, Schenk, Michael wrote: > From 9d2d5127a3113b6581b0a562776c3ec92c547bdb Mon Sep 17 00:00:00 2001 > From: Michael Schenk > Date: Fri, 25 Nov 2016 09:36:20 +0100 > Subject: [PATCH] set nb_elem to 0 after freeing to avoid further access which >

Re: [FFmpeg-devel] [PATCH] Remove the ffserver program and the ffm muxer/demuxer

2016-11-25 Thread Andreas Cadhalpun
On 25.11.2016 10:36, wm4 wrote: > I'd like to add that removing the ffserver support from the libs, and > all deprecated stuff it depends on, will make it actually easier to > readd ffserver in a cleaner way. It reduces the risk that the new > ffserver accidentally depends on mechanisms that were

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-25 Thread Andreas Cadhalpun
On 25.11.2016 02:59, Michael Niedermayer wrote: > On Fri, Nov 25, 2016 at 02:26:24AM +0100, Andreas Cadhalpun wrote: >> mss2.c |8 +++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> 958ee0811485404a0662a1540fcb8b131423947b >>

[FFmpeg-devel] [PATCH] avcodec/nvenc: Delay identification of underlying format of cuda frames

2016-11-25 Thread Philip Langdale
When input surfaces are cuda frames, we will not know what the actual underlying format (nv12, p010, etc) is at surface allocation time. On the other hand, we will know when the input frames are actually registered and associated with a surface. So, let's delay format discovery until

Re: [FFmpeg-devel] [PATCH 1/2] img2: added support for %t output pattern

2016-11-25 Thread Roger Pack
On 11/18/16, Roger Pack wrote: > On 11/15/16, James Almer wrote: >> On 11/10/2016 4:26 PM, Roger Pack wrote: >>> On 11/1/16, James Almer wrote: > On 11/1/2016 6:43 PM, James Almer wrote: > >> On 10/25/2016 9:38 PM, Roger Pack

Re: [FFmpeg-devel] [PATCH] avformat/flvdec: move set bit_rate from parse AMF OBJECT to create_stream

2016-11-25 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 05:18:43PM +0800, Steven Liu wrote: > before patch: > Stream #0:0: Video: h264 (High), yuv420p(progressive), 1280x714 [SAR 1:1 DAR > 640:357], 25 fps, 25 tbr, 1k tbn, 50 tbc > > after patch: > Stream #0:0: Video: h264 (High), yuv420p(progressive), 1280x714 [SAR 1:1 DAR >

Re: [FFmpeg-devel] [PATCH] matroska read_seek: make max diff for last known subtitle configurable

2016-11-25 Thread Rainer Hochecker
I was tempted to submit a patch with this feature deleted. Unfortunately I was not able to find the original reason why this was implemented. I am sure there was a good reason and this reason may still be a good one. ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] [RFC]MAINTAINERS: Add developers who have git write access but are otherwise not listed

2016-11-25 Thread Marton Balint
On Fri, 25 Nov 2016, James Almer wrote: On 11/25/2016 9:15 AM, Michael Niedermayer wrote: On Fri, Nov 25, 2016 at 11:11:23AM +0100, Marton Balint wrote: On Mon, 21 Nov 2016, Michael Niedermayer wrote: I omitted developers who do not use their account and i felt would prefer not to be

Re: [FFmpeg-devel] [PATCH] [RFC]MAINTAINERS: Add developers who have git write access but are otherwise not listed

2016-11-25 Thread wm4
On Fri, 25 Nov 2016 10:31:58 -0300 James Almer wrote: > On 11/25/2016 9:15 AM, Michael Niedermayer wrote: > > On Fri, Nov 25, 2016 at 11:11:23AM +0100, Marton Balint wrote: > >> > >> On Mon, 21 Nov 2016, Michael Niedermayer wrote: > >> > >>> I omitted developers who do not

Re: [FFmpeg-devel] [PATCH] [RFC]MAINTAINERS: Add developers who have git write access but are otherwise not listed

2016-11-25 Thread James Almer
On 11/25/2016 9:15 AM, Michael Niedermayer wrote: > On Fri, Nov 25, 2016 at 11:11:23AM +0100, Marton Balint wrote: >> >> On Mon, 21 Nov 2016, Michael Niedermayer wrote: >> >>> I omitted developers who do not use their account and i felt would prefer >>> not >>> to be listed. >> >> I think

Re: [FFmpeg-devel] [PATCH 1/2] swresample/soxr: fix invalid use of linear_interp

2016-11-25 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 04:55:34PM +0700, Muhammad Faiz wrote: > give very bad quality for soxr resampler. > linear_interp is intended for using linear interpolation > between filter bank so quality will be better. > > i guess this is misunderstood as 'do not use filter bank, > but directly

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: Remove aspect-ratio decompensation logic

2016-11-25 Thread Timo Rothenpieler
This LGTM, the compensation is indeed gone on all current Nvidia Drivers I tested. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] [RFC]MAINTAINERS: Add developers who have git write access but are otherwise not listed

2016-11-25 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 11:11:23AM +0100, Marton Balint wrote: > > On Mon, 21 Nov 2016, Michael Niedermayer wrote: > > >I omitted developers who do not use their account and i felt would prefer not > >to be listed. > > I think everyone with access should be listed. If somebody does not > use

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-11-25 Thread Martin Vignali
Hello > > If you want to maintain this code, then please add yourself to the > MAINTAINER file too > I'm not very familiar with git. So it's probably not a good idea to be a maintainer. But i can take a look for patch on psd (and exr) if need. Martin

Re: [FFmpeg-devel] [PATCH v2] avformat/rtsp: introduce get_sa_len() function

2016-11-25 Thread Nicolas George
> >Since the Linux implementation of sockaddr doesn't have sa_len as a member, > >but the FreeBSD version does, introduce a get_sa_len() function that > >determines the size based on the address family. > > > >Signed-off-by: Kevin Lo For some reason I do not have the original

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

2016-11-25 Thread Nicolas George
Le quintidi 5 frimaire, an CCXXV, Andreas Cadhalpun a écrit : > I think a better idea would be to require '-strict experimental', > as code disabled by default does neither get build- nor FATE-tested > much. That is an excellent idea! Regards, -- Nicolas George

[FFmpeg-devel] [PATCH 2/2] swresample: enable exact_rational and linear_interp by default

2016-11-25 Thread Muhammad Faiz
have problem with git send-email, so I attach the patch From 4b77fcc028b080ccf7887f6b3fb5830960acf8c1 Mon Sep 17 00:00:00 2001 From: Muhammad Faiz Date: Fri, 25 Nov 2016 16:48:22 +0700 Subject: [PATCH 2/2] swresample: enable exact_rational and linear_interp by default

[FFmpeg-devel] [PATCH 1/2] swresample/soxr: fix invalid use of linear_interp

2016-11-25 Thread Muhammad Faiz
give very bad quality for soxr resampler. linear_interp is intended for using linear interpolation between filter bank so quality will be better. i guess this is misunderstood as 'do not use filter bank, but directly interpolate linearly between samples'. Signed-off-by: Muhammad Faiz

Re: [FFmpeg-devel] [PATCH] [RFC]MAINTAINERS: Add developers who have git write access but are otherwise not listed

2016-11-25 Thread Marton Balint
On Mon, 21 Nov 2016, Michael Niedermayer wrote: I omitted developers who do not use their account and i felt would prefer not to be listed. I think everyone with access should be listed. If somebody does not use his account for a year or so, his/her access should be revoked. Regards,

[FFmpeg-devel] [PATCH] avformat/matroskadec: Prevent access of elements after freeing by set nb_elem to zero.

2016-11-25 Thread Schenk, Michael
Hi folks, retry, because my eMail address was changed. I’m using the decode interrupt feature of ffmpeg for getting a fast user reaction. It looks like this may causing some nasty crashes by accessing previous freed pointers in matroska_read_close. The attached patch will reset nb_elem to zero

Re: [FFmpeg-devel] [PATCH] Remove the ffserver program and the ffm muxer/demuxer

2016-11-25 Thread wm4
On Thu, 24 Nov 2016 21:40:14 -0300 James Almer wrote: > On 11/24/2016 8:48 PM, Andreas Cadhalpun wrote: > > On 24.11.2016 10:53, Josh de Kock wrote: > >> There's no benefit in waiting for this to occur before ffserver is > >> removed as it will always be in git history.

[FFmpeg-devel] [PATCH] avformat/flvdec: move set bit_rate from parse AMF OBJECT to create_stream

2016-11-25 Thread Steven Liu
before patch: Stream #0:0: Video: h264 (High), yuv420p(progressive), 1280x714 [SAR 1:1 DAR 640:357], 25 fps, 25 tbr, 1k tbn, 50 tbc after patch: Stream #0:0: Video: h264 (High), yuv420p(progressive), 1280x714 [SAR 1:1 DAR 640:357], 2576 kb/s, 25 fps, 25 tbr, 1k tbn, 50 tbc Signed-off-by: