[FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
From: Petri Hintukainen phint...@users.sourceforge.net Fixes ticket #2208 --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supenc.c | 61 3 files changed, 63 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread wm4
On Fri, 29 Aug 2014 13:31:22 +0300 Petri Hintukainen phint...@gmail.com wrote: From: Petri Hintukainen phint...@users.sourceforge.net Fixes ticket #2208 --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supenc.c | 61

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Hendrik Leppkes
On Fri, Aug 29, 2014 at 12:31 PM, Petri Hintukainen phint...@gmail.com wrote: From: Petri Hintukainen phint...@users.sourceforge.net Fixes ticket #2208 Is the format officially documented somewhere, so interoperability with other tools is ensured? - Hendrik

Re: [FFmpeg-devel] [PATCH] avformat/wtvdec: seek over broken chunks

2014-08-29 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 04:42:04PM +1000, Peter Ross wrote: Fixes ticket #3898 Signed-off-by: Peter Ross pr...@xvid.org --- libavformat/wtvdec.c | 29 +++-- 1 file changed, 27 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/rtpdec_asf: fix compiler warning about const qualifier being discarded

2014-08-29 Thread Michael Niedermayer
On Thu, Aug 28, 2014 at 05:22:42PM -0700, Timothy Gu wrote: On Aug 28, 2014 6:14 AM, Michael Niedermayer michae...@gmx.at wrote: buf is assigned to AVIOContext.buffer, which is written to by some functions using AVIOContext Doesn't const only apply to this particular function? In other

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

2014-08-29 Thread wm4
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 a/libavutil/avstring.c b/libavutil/avstring.c index a63fb84..df27d5e

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Carl Eugen Hoyos
Petri Hintukainen phintuka at gmail.com writes: From: Petri Hintukainen phintuka at users.sourceforge.net Fixes ticket #2208 Do I understand correctly that the produced files are compatible with eac3to? Are there other programs that accept these files? Thank you, Carl Eugen

Re: [FFmpeg-devel] [PATCH 2/2] Add missing const all over the place.

2014-08-29 Thread Carl Eugen Hoyos
Reimar Döffinger Reimar.Doeffinger at gmx.de writes: There is a risk that one in the code that is not always compiled might break things. We will only find out if you push;-) Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 5/5] lavfi/vf_fps: accept EOF timestamp.

2014-08-29 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: Fix trac ticket #2674. libavfilter/vf_fps.c | 33 + 1 file changed, 25 insertions(+), 8 deletions(-) probably ok Ping? (I didn't test myself.) Carl Eugen ___

Re: [FFmpeg-devel] [PATCH 2/2] Add missing const all over the place.

2014-08-29 Thread Reimar Döffinger
On Thu, Aug 28, 2014 at 05:20:44PM -0700, Timothy Gu wrote: On Aug 28, 2014 3:46 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: There is a risk that one in the code that is not always compiled might break things. What do you mean by 'one in the code'? The intended grouping was

[FFmpeg-devel] [PATCH 1/3] fft: add missing const.

2014-08-29 Thread Reimar Döffinger
This table was the largest object in FFmpeg's .data, and really should be in .rodata. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/fft_init_table.c | 2 +- libavcodec/fft_table.h | 2 +- libavcodec/fft_template.c | 2 +- 3 files changed, 3 insertions(+), 3

[FFmpeg-devel] [PATCH 3/3] fft_template: Move variables to where they are used.

2014-08-29 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/fft_template.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/libavcodec/fft_template.c b/libavcodec/fft_template.c index 3abf948..23ea453 100644 --- a/libavcodec/fft_template.c +++

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

2014-08-29 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavutil/hash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/hash.c b/libavutil/hash.c index 773f29e..979fdd9 100644 --- a/libavutil/hash.c +++ b/libavutil/hash.c @@ -63,7 +63,7 @@ typedef struct

[FFmpeg-devel] [PATCH] add silenceremove filter

2014-08-29 Thread Paul B Mahol
Signed-off-by: Paul B Mahol one...@gmail.com --- doc/filters.texi | 62 ++ libavfilter/Makefile | 1 + libavfilter/af_silenceremove.c | 478 + libavfilter/allfilters.c | 1 + 4 files changed, 542 insertions(+) create

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-29 Thread Michael Niedermayer
Hi Vittorio also please dont remove ffmpeg-devel from the CC I had missed that you removed it so my reply went just to debian-devel full quote left below for ffmpeg-devel, no further inline comments below Thanks On Fri, Aug 29, 2014 at 07:17:55PM +0200, Michael Niedermayer wrote: Hi Vittorio

[FFmpeg-devel] Symbian port

2014-08-29 Thread Федор
How directly use gas-preprocessor.pl? I download bundle from here -  https://git.libav.org/?p=gas-preprocessor.git;a=tree  ang try it like described here -  https://wiki.libav.org/Tools/gas-preprocessor . I got error :  gas-preprocessor.pl gas Unrecognized input filetype at

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

2014-08-29 Thread Clément Bœsch
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 a/libavutil/hash.c b/libavutil/hash.c index 773f29e..979fdd9 100644 ---

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-29 Thread Kieran Kunhya
think discussing the Libav-FFmpeg split and ways to resolve it at VDD makes a lot of sense, quite litterally more than 90% of the developers wont be there. I also wont be there Lame excuse. You're the leader and the main reason why people want you there. also iam quite confident that if

Re: [FFmpeg-devel] [PATCH 2/3] fft: add ff_ prefix to some global arrays.

2014-08-29 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 06:33:00PM +0200, Reimar Döffinger wrote: 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 ++-- 3 files changed, 10 insertions(+), 10

Re: [FFmpeg-devel] [PATCH 1/3] fft: add missing const.

2014-08-29 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 06:32:59PM +0200, Reimar Döffinger wrote: This table was the largest object in FFmpeg's .data, and really should be in .rodata. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavcodec/fft_init_table.c | 2 +- libavcodec/fft_table.h | 2 +-

Re: [FFmpeg-devel] Symbian port

2014-08-29 Thread Lou Logan
On Fri, Aug 29, 2014, at 11:24 AM, Федор wrote: How directly use gas-preprocessor.pl? I download bundle from here -  https://git.libav.org/?p=gas-preprocessor.git;a=tree  ang try it like described here -  https://wiki.libav.org/Tools/gas-preprocessor . I got error :  gas-preprocessor.pl

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
On pe, 2014-08-29 at 12:51 +0200, wm4 wrote: Do you plan on writing a demuxer too? If not, then I'd give it a try. Probably not (at least in next week or two). It should be simple to parse :). After the header there's always segment type byte and length (16-bits BE). Without index seeking is

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
On pe, 2014-08-29 at 14:06 +0200, Hendrik Leppkes wrote: On Fri, Aug 29, 2014 at 12:31 PM, Petri Hintukainen phint...@gmail.com wrote: From: Petri Hintukainen phint...@users.sourceforge.net Fixes ticket #2208 Is the format officially documented somewhere, so interoperability with

Re: [FFmpeg-devel] [PATCH] added synchronous KLV codec ID and mpegts PAT and PMT descriptors as specified in stanag4609 annex C

2014-08-29 Thread Carl Eugen Hoyos
paolovergori webinostestbed at gmail.com writes: +AV_CODEC_ID_SMPTE_KLV_SYNC = MKBETAG('K','L','V','S'), Please just add this line, do not reindent the other defines, at least not in the same commit. { MKTAG('K','L','V','A'), AVMEDIA_TYPE_DATA, AV_CODEC_ID_SMPTE_KLV }, +{

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Carl Eugen Hoyos
Peter Kasting pkasting at google.com writes: The attached patch fixes instances of MSVC warning C4244 about possible value truncation (e.g. when assigning double to float or int64_t to int). This warning is currently disabled in Chromium's MSVC build and I'm trying to enable it. Why?

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Peter Kasting
On Fri, Aug 29, 2014 at 3:48 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Peter Kasting pkasting at google.com writes: The attached patch fixes instances of MSVC warning C4244 about possible value truncation (e.g. when assigning double to float or int64_t to int). This warning is

[FFmpeg-devel] [PATCH]Clean up if filter initialisation failed

2014-08-29 Thread Carl Eugen Hoyos
Hi! Attached patch fixes a memleak with ffmpeg if the filter initialisation fails, fixes ticket #3858 for me. Please review, Carl Eugen diff --git a/ffmpeg_filter.c b/ffmpeg_filter.c index b6dc42f..7acff28 100644 --- a/ffmpeg_filter.c +++ b/ffmpeg_filter.c @@ -898,8 +898,11 @@ int

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Reimar Döffinger
On 30.08.2014, at 00:44, Peter Kasting pkast...@google.com wrote: Hi FFMPEG devs, please forgive any errors here as I'm normally a Chromium developer and this is my first submission to FFMPEG. The attached patch fixes instances of MSVC warning C4244 about possible value truncation (e.g. when

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Carl Eugen Hoyos
Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Second, I believe powf and sinf are less commonly available than pow and sin, so I think this will break compilation on some platforms I fear this is correct. Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Michael Niedermayer
On Sat, Aug 30, 2014 at 01:26:46AM +0200, Reimar Döffinger wrote: On 30.08.2014, at 00:44, Peter Kasting pkast...@google.com wrote: Hi FFMPEG devs, please forgive any errors here as I'm normally a Chromium developer and this is my first submission to FFMPEG. The attached patch fixes

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread James Almer
On 29/08/14 8:26 PM, Reimar Döffinger wrote: On 30.08.2014, at 00:44, Peter Kasting pkast...@google.com wrote: Hi FFMPEG devs, please forgive any errors here as I'm normally a Chromium developer and this is my first submission to FFMPEG. The attached patch fixes instances of MSVC warning

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Peter Kasting
On Fri, Aug 29, 2014 at 4:26 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: First, this needs very, very careful review. I am not at all convinced that these will not change behaviour. I strongly agree that it needs careful review. I think that reinforces why a change like this is

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-08-29 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 04:38:28PM -0700, Peter Kasting wrote: On Fri, Aug 29, 2014 at 4:26 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: First, this needs very, very careful review. I am not at all convinced that these will not change behaviour. I strongly agree that it needs

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-29 Thread Christoph Anton Mitterer
Hey. On 12/08/2014 18:30, Michael Niedermayer wrote: Also ive offered my resignation in the past. I do still offer to resign from the FFmpeg leader position, if it resolves this split between FFmpeg and Libav and make everyone work together again. I have absolutely no opinion on the

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

2014-08-29 Thread Steven Liu
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 unlinked this patch can do this operation delete the old ts segment which

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

2014-08-29 Thread Michael Niedermayer
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 427827b..da31f6d 100644 --- a/libavcodec/vp3data.h +++ b/libavcodec/vp3data.h