Re: [FFmpeg-devel] [PATCH] delete the old segment file from hls list

2014-08-30 Thread Anshul
On August 30, 2014 8:57:12 AM IST, Steven Liu lingjiujia...@gmail.com wrote: On Aug 28, 2014, at 4:52 AM, Anshul anshul.ffm...@gmail.com wrote: On August 22, 2014 8:01:20 AM IST, Steven Liu lingjiujia...@gmail.com wrote: repost new patch: when update the hls m3u8 list, the old file is not

[FFmpeg-devel] [PATCH] avformat/rtpdec_qt: add cast to silence discards const qualifier from pointer target type warning

2014-08-30 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/rtpdec_qt.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtpdec_qt.c b/libavformat/rtpdec_qt.c index ee8a48a..58253b6 100644 --- a/libavformat/rtpdec_qt.c +++ b/libavformat/rtpdec_qt.c @@

Re: [FFmpeg-devel] [RFC] [PATCH] proresenc_kostya: warn/reject on incorrect profile

2014-08-30 Thread Christophe Gisquet
Hi, 2014-08-24 18:56 GMT+02:00 Michael Niedermayer michae...@gmx.at: aggree, a PRORES_PROFILE_AUTO seems like a good idea with it mismatching profile and pixel format can be errors while by default the profile can be selected based on the pixel format The attached patch makes it selects

Re: [FFmpeg-devel] [PATCH] avcodec/vp3data: use more compact data types

2014-08-30 Thread Paul B Mahol
On 8/30/14, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vp3data.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libavcodec/vp3data.h b/libavcodec/vp3data.h index

Re: [FFmpeg-devel] [PATCH] avcodec/vp3data: use more compact data types

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 01:31:04PM +0200, Paul B Mahol wrote: On 8/30/14, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vp3data.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

Re: [FFmpeg-devel] [RFC] [PATCH] proresenc_kostya: warn/reject on incorrect profile

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 01:13:56PM +0200, Christophe Gisquet wrote: Hi, 2014-08-24 18:56 GMT+02:00 Michael Niedermayer michae...@gmx.at: aggree, a PRORES_PROFILE_AUTO seems like a good idea with it mismatching profile and pixel format can be errors while by default the profile can be

Re: [FFmpeg-devel] [PATCH] lavu/avstring: check for overlong encodings

2014-08-30 Thread Stefano Sabatini
On date Friday 2014-08-29 15:57:32 +0200, wm4 encoded: On Thu, 28 Aug 2014 17:39:27 +0200 Stefano Sabatini stefa...@gmail.com wrote: Fix reopened trac ticket #1163. --- libavutil/avstring.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] [PATCH] lavu/avstring: check for overlong encodings

2014-08-30 Thread Nicolas George
Le tridi 13 fructidor, an CCXXII, Stefano Sabatini a écrit : It is mathematically impossible that the length will be larger than 5. Added an assert for that though, alternatively I could add a check. In ffprobe, the function is used with values coming directly from the file's metadata: an

Re: [FFmpeg-devel] [PATCH] lavu/avstring: check for overlong encodings

2014-08-30 Thread Stefano Sabatini
On date Saturday 2014-08-30 15:06:14 +0200, Nicolas George encoded: Le tridi 13 fructidor, an CCXXII, Stefano Sabatini a écrit : It is mathematically impossible that the length will be larger than 5. Added an assert for that though, alternatively I could add a check. In ffprobe, the

Re: [FFmpeg-devel] [PATCH] lavu/hash.c: Add missing static const.

2014-08-30 Thread Reimar Döffinger
On Fri, Aug 29, 2014 at 09:29:46PM +0200, Clément Bœsch wrote: On Fri, Aug 29, 2014 at 06:34:52PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavutil/hash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH 5/6] xv.c: Add missing const to lookup table.

2014-08-30 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavdevice/xv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavdevice/xv.c b/libavdevice/xv.c index aeb3e41..c19c15c 100644 --- a/libavdevice/xv.c +++ b/libavdevice/xv.c @@ -67,7 +67,7 @@ typedef struct

[FFmpeg-devel] [PATCH 4/6] lavf: add const/static const to pix_fmts arrays.

2014-08-30 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavfilter/vf_blackdetect.c| 2 +- libavfilter/vf_lenscorrection.c | 2 +- libavfilter/vf_signalstats.c| 2 +- libavfilter/vf_tinterlace.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git

[FFmpeg-devel] [PATCH 3/6] sipr16kdata.h: add forgotten const.

2014-08-30 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/sipr16kdata.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/sipr16kdata.h b/libavcodec/sipr16kdata.h index 96bf0e9..16a653d 100644 --- a/libavcodec/sipr16kdata.h +++ b/libavcodec/sipr16kdata.h

[FFmpeg-devel] [PATCH 1/6] fft: add ff_ prefix to some global arrays.

2014-08-30 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/fft_init_table.c | 4 ++-- libavcodec/fft_table.h | 4 ++-- libavcodec/fft_template.c | 12 ++-- libavcodec/mips/fft_mips.c | 8 4 files changed, 14 insertions(+), 14 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] oggdec: fix invalid free on error

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 03:39:15PM +0200, wm4 wrote: The read_packet callback passes a pointer to a stack-allocated AVPacket. Attempting to free it with av_free() makes no sense. --- Found by looking at the code; there's no sample file. --- libavformat/oggdec.c | 1 - 1 file changed, 1

[FFmpeg-devel] [PATCH] aacsbr: support hardcoding tables.

2014-08-30 Thread Reimar Döffinger
For sbr_qmf_window_us there is even a question if it maybe should be fully hardcoded all the time. Since half of it is coded, it ends up in .data and not .bss. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/Makefile | 4 +- libavcodec/aacsbr.c | 10

Re: [FFmpeg-devel] [PATCH 4/6] lavf: add const/static const to pix_fmts arrays.

2014-08-30 Thread Reimar Döffinger
On Sat, Aug 30, 2014 at 08:07:29AM -0700, Timothy Gu wrote: On Aug 30, 2014 7:44 AM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: It's lavfi, not lavf. Fixed locally. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 5/6] xv.c: Add missing const to lookup table.

2014-08-30 Thread Paul B Mahol
On 8/30/14, Reimar Doeffinger reimar.doeffin...@gmx.de wrote: Signed-off-by: Reimar Doeffinger reimar.doeffin...@gmx.de --- libavdevice/xv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavdevice/xv.c b/libavdevice/xv.c index aeb3e41..c19c15c 100644 ---

[FFmpeg-devel] [PATCH] WMA: add const to avoid warnings with hardcoded tables.

2014-08-30 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/wma.h | 2 +- libavcodec/wmaprodec.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/wma.h b/libavcodec/wma.h index c4056ec..01244a0 100644 --- a/libavcodec/wma.h +++

[FFmpeg-devel] [PATCH] Add av_cold to table generation functions.

2014-08-30 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/aac_tablegen.h | 2 +- libavcodec/aacps_tablegen.h| 4 ++-- libavcodec/aacsbr_tablegen.h | 2 +- libavcodec/cbrt_tablegen.h | 3 ++- libavcodec/dv_tablegen.h | 3 ++-

Re: [FFmpeg-devel] [PATCH 5/6] xv.c: Add missing const to lookup table.

2014-08-30 Thread Reimar Döffinger
On Sat, Aug 30, 2014 at 05:55:39PM +0200, Paul B Mahol wrote: On 8/30/14, Reimar Doeffinger reimar.doeffin...@gmx.de wrote: Signed-off-by: Reimar Doeffinger reimar.doeffin...@gmx.de --- libavdevice/xv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) lgtm Thanks, pushed.

Re: [FFmpeg-devel] [PATCH] WMA: add const to avoid warnings with hardcoded tables.

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 06:00:20PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/wma.h | 2 +- libavcodec/wmaprodec.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) LGTM thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH] Add av_cold to table generation functions.

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 06:00:43PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/aac_tablegen.h | 2 +- libavcodec/aacps_tablegen.h| 4 ++-- libavcodec/aacsbr_tablegen.h | 2 +- libavcodec/cbrt_tablegen.h

Re: [FFmpeg-devel] [PATCH] cabac: initialize all of ff_h264_cabac_tables programmatically.

2014-08-30 Thread wm4
On Sat, 30 Aug 2014 18:41:52 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 18:29, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sat, Aug 30, 2014 at 06:20:56PM +0200, wm4 wrote: On Sat, 30 Aug 2014 18:18:41 +0200 Reimar Döffinger reimar.doeffin...@gmx.de

Re: [FFmpeg-devel] [PATCH] libavdevice/v4l2: fix of crash caused by assert

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 08:19:37PM +0400, Dmitry Volyntsev wrote: To understand the problem one needs to see the original code around and to think what would happen if from time to time while capturing condition (s-frame_size 0 buf.bytesused != s-frame_size) happens to be true (this is not

Re: [FFmpeg-devel] [PATCH] cabac: initialize all of ff_h264_cabac_tables programmatically.

2014-08-30 Thread Reimar Döffinger
On 30.08.2014, at 19:46, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 19:01, wm4 nfx...@googlemail.com wrote: but still I wonder why you'd optimize for binary size, in exchange for higher RAM usage. What is your motivation for doing this? The only higher RAM usage

Re: [FFmpeg-devel] [PATCH] cabac: initialize all of ff_h264_cabac_tables programmatically.

2014-08-30 Thread wm4
On Sat, 30 Aug 2014 19:58:11 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 19:46, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 19:01, wm4 nfx...@googlemail.com wrote: but still I wonder why you'd optimize for binary size, in exchange for

Re: [FFmpeg-devel] [PATCH] lavu/hash.c: Add missing static const.

2014-08-30 Thread Clément Bœsch
On Sat, Aug 30, 2014 at 04:36:57PM +0200, Reimar Döffinger wrote: On Fri, Aug 29, 2014 at 09:29:46PM +0200, Clément Bœsch wrote: On Fri, Aug 29, 2014 at 06:34:52PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavutil/hash.c | 2 +- 1

Re: [FFmpeg-devel] [PATCH] lavu/hash.c: Add missing static const.

2014-08-30 Thread Timothy Gu
On Aug 30, 2014 7:37 AM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: But mostly I care about having it in .rodata, everything else I'm willing to adapt to what others prefer. What are the gains for using .rodata instead of .data? Timothy ___

Re: [FFmpeg-devel] [PATCH] lavu/hash.c: Add missing static const.

2014-08-30 Thread Reimar Döffinger
On Sat, Aug 30, 2014 at 12:58:35PM -0700, Timothy Gu wrote: On Aug 30, 2014 7:37 AM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: But mostly I care about having it in .rodata, everything else I'm willing to adapt to what others prefer. What are the gains for using .rodata instead of

Re: [FFmpeg-devel] [PATCH 4/6] lavf: add const/static const to pix_fmts arrays.

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 04:43:42PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavfilter/vf_blackdetect.c| 2 +- libavfilter/vf_lenscorrection.c | 2 +- libavfilter/vf_signalstats.c| 2 +- libavfilter/vf_tinterlace.c | 2 +- 4

Re: [FFmpeg-devel] [PATCH 3/6] sipr16kdata.h: add forgotten const.

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 04:43:41PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/sipr16kdata.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 6/6] ac3dec_fixed: add missing static const.

2014-08-30 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 04:43:44PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/ac3dec_fixed.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-30 Thread wm4
On Sat, 30 Aug 2014 18:16:27 -0400 Marcus Johnson bumblebritche...@gmail.com wrote: Here's a PGS subtitle sample from Eac3to: https://dl.dropboxusercontent.com/u/52358991/English.zip Thanks! The DTS fields in this one are all 0 too. ___

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-30 Thread Timothy Gu
On Sat, Aug 30, 2014 at 9:41 PM, Marcus Johnson bumblebritche...@gmail.com wrote: If you don't mind me asking, what is DTS? because when I hear it I think of the DTS audio codec, but I know it's obviously not that, and googling DTS in the codec context won't get me anywhere. DTS = decoding