Re: [FFmpeg-devel] [PATCH] ffplay: remove empty subtitles from que head

2014-07-12 Thread Marton Balint
On Sat, 12 Jul 2014, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffplay.c |1 + 1 file changed, 1 insertion(+) diff --git a/ffplay.c b/ffplay.c index 7e954fd..6572c42 100644 --- a/ffplay.c +++ b/ffplay.c @@ -1446,6 +1446,7 @@ retry:

Re: [FFmpeg-devel] [PATCH] ffplay: remove dead code

2014-07-24 Thread Marton Balint
On Thu, 24 Jul 2014, Michael Niedermayer wrote: Hi On Thu, Jul 17, 2014 at 06:02:02PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- ffplay.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index af0e199..8abbeae 100644 --- a/ffplay.c

Re: [FFmpeg-devel] [PATCH]Make subtitle properties code in lavc more robust

2014-08-07 Thread Marton Balint
On Thu, 7 Aug 2014, Carl Eugen Hoyos wrote: Hi! Attached patch should allow to set the teletext decoder properties to (AV_CODEC_PROP_TEXT_SUB | AV_CODEC_PROP_BITMAP_SUB). At the moment, the documentation states that if a property of this kind is set, then the decoder will provide

[FFmpeg-devel] [PATCH] mpegts: always parse pcr

2014-08-10 Thread Marton Balint
Previously pcr transmitted without payload but as part of the video stream was not parsed. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/mpegts.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index

Re: [FFmpeg-devel] [PATCH] mpegts: always parse pcr

2014-08-16 Thread Marton Balint
On Sun, 10 Aug 2014, Marton Balint wrote: Previously pcr transmitted without payload but as part of the video stream was not parsed. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/mpegts.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) Hello Michael, Please

[FFmpeg-devel] [PATCH 1/2] ffplay: factorize frame queue operations

2014-09-06 Thread Marton Balint
Create a generic frame and a frame queue struct to handle video picture queues and subtitle picture queues with common code. Also add the possibility to queue AVFrames, however at the moment we only use SDL_Overlay buffers for video and AVSubtitles for subtitles. Signed-off-by: Marton Balint c

Re: [FFmpeg-devel] [RFC]Revert the default for ffplay to -noautoexit

2014-09-07 Thread Marton Balint
On Sun, 7 Sep 2014, Carl Eugen Hoyos wrote: Hi! I have no strong opinion on this but I wonder if ffplay isn't primarily a test application for which it is very useful not to exit. Well, I was thinking about this myself... And I quite hate that in mplayer when you seek past the end of the

Re: [FFmpeg-devel] [PATCH 1/2] ffplay: factorize frame queue operations

2014-09-16 Thread Marton Balint
On Sat, 6 Sep 2014, Marton Balint wrote: Create a generic frame and a frame queue struct to handle video picture queues and subtitle picture queues with common code. Also add the possibility to queue AVFrames, however at the moment we only use SDL_Overlay buffers for video and AVSubtitles

Re: [FFmpeg-devel] [PATCH 1/2] ffplay: factorize frame queue operations

2014-09-17 Thread Marton Balint
On Wed, 17 Sep 2014, Michael Niedermayer wrote: On Wed, Sep 17, 2014 at 03:47:09AM +0200, Michael Niedermayer wrote: On Wed, Sep 17, 2014 at 12:19:49AM +0200, Marton Balint wrote: On Sat, 6 Sep 2014, Marton Balint wrote: Create a generic frame and a frame queue struct to handle video

[FFmpeg-devel] [PATCH 2/2] ffplay: move pts calculating code to generic decoder

2014-09-18 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 67 +++- 1 file changed, 37 insertions(+), 30 deletions(-) diff --git a/ffplay.c b/ffplay.c index f9aac6a..e5d7183 100644 --- a/ffplay.c +++ b/ffplay.c @@ -187,6 +187,10

[FFmpeg-devel] [PATCH 1/2] ffplay: factorize decoder functions to generic decoder

2014-09-18 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 261 ++- 1 file changed, 140 insertions(+), 121 deletions(-) diff --git a/ffplay.c b/ffplay.c index 9982474..f9aac6a 100644 --- a/ffplay.c +++ b/ffplay.c @@ -177,6 +177,18

Re: [FFmpeg-devel] ffplay -fs fix (ticket #3964)

2014-09-22 Thread Marton Balint
61a4f0b2c0ecd883022f06c3655d0f4f936c8615 Mon Sep 17 00:00:00 2001 From: Marton Balint c...@passwd.hu Date: Mon, 22 Sep 2014 22:17:06 +0200 Subject: [PATCH] ffplay: handle fullscreen status on resize event --- ffplay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index

Re: [FFmpeg-devel] [PATCH] ffplay: use av_codec_get_pkt_timebase()

2014-09-24 Thread Marton Balint
On Wed, 24 Sep 2014, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffplay.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index 6eb5aef..7728452 100644 --- a/ffplay.c +++ b/ffplay.c @@ -598,7 +598,7 @@ static int

Re: [FFmpeg-devel] [PATCH] ffplay: Separate AVFrame from AVSubtitle

2014-09-26 Thread Marton Balint
On Fri, 26 Sep 2014, Michael Niedermayer wrote: This is more robust and should avoid false positives in coverity That is if the type is ever mixed up, after this patch that would just be a null pointer dereference instead of a random possibly out of array access Signed-off-by: Michael

[FFmpeg-devel] [PATCH] ffplay: dont leave swresampler in half initialized state

2014-10-08 Thread Marton Balint
On init failure, let's just free it, so next time it will be recreated from start. Also fixes Coverity CID 1241515. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index 8fa5ca3..37983e9 100644

Re: [FFmpeg-devel] [PATCH] ffplay: dont leave swresampler in half initialized state

2014-10-09 Thread Marton Balint
On Thu, 9 Oct 2014, Michael Niedermayer wrote: On Wed, Oct 08, 2014 at 11:48:58PM +0200, Marton Balint wrote: On init failure, let's just free it, so next time it will be recreated from start. Also fixes Coverity CID 1241515. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 4

[FFmpeg-devel] [PATCH 2/3] ffplay: remove manual bug option handling code

2014-10-13 Thread Marton Balint
It is supported as a codec option, and those are already parsed. Signed-off-by: Marton Balint c...@passwd.hu --- doc/ffplay.texi | 2 -- ffplay.c| 3 --- 2 files changed, 5 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 203085c..9bb040e 100644 --- a/doc/ffplay.texi

[FFmpeg-devel] [PATCH 1/3] ffplay: remove delays when paused from video and subtitle thread

2014-10-13 Thread Marton Balint
Once the frame queue is full, we will wait anyway. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/ffplay.c b/ffplay.c index 37983e9..960fdc8 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2122,9 +2122,6 @@ static int video_thread(void

[FFmpeg-devel] [PATCH 3/3] ffplay: update documentation

2014-10-13 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/ffplay.texi | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 9bb040e..4c01e6c 100644 --- a/doc/ffplay.texi +++ b/doc/ffplay.texi @@ -37,10 +37,14 @@ Force

Re: [FFmpeg-devel] [PATCH] mpegts: add the judgement if a new program is ctreated successfully

2014-10-15 Thread Marton Balint
On Wed, 15 Oct 2014, di1028...@samsung.com wrote: From: Di Wu di1028...@samsung.com Add the judement after create a new program to avoid segment fault. Signed-off-by: Di Wu di1028...@samsung.com --- libavformat/mpegts.c |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff

[FFmpeg-devel] [PATCHv2] ffplay: update documentation

2014-10-15 Thread Marton Balint
based on your comments. Here is v2 of the patch. --- Signed-off-by: Marton Balint c...@passwd.hu --- doc/ffplay.texi | 26 -- 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 9bb040e..3c85417 100644 --- a/doc/ffplay.texi

Re: [FFmpeg-devel] [PATCHv2] ffplay: update documentation

2014-10-17 Thread Marton Balint
On Thu, 16 Oct 2014, Lou Logan wrote: On Wed, 15 Oct 2014 23:46:21 +0200, Marton Balint wrote: Thank you for all your feedback, apparently there was some confusion because I have used set both as a synonym to enabled and as an imperative. I have changed all occurences of it to enabled or use

[FFmpeg-devel] [PATCH 1/3] ffplay: implement separete audio decoder thread

2014-10-29 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 265 --- 1 file changed, 153 insertions(+), 112 deletions(-) diff --git a/ffplay.c b/ffplay.c index a979164..24bcae2 100644 --- a/ffplay.c +++ b/ffplay.c @@ -121,7 +121,8

[FFmpeg-devel] [PATCH 2/3] ffplay: fix indentation after last commit

2014-10-29 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 182 +++ 1 file changed, 89 insertions(+), 93 deletions(-) diff --git a/ffplay.c b/ffplay.c index 24bcae2..a3b34fd 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2424,105 +2424,101

[FFmpeg-devel] [PATCH 3/3] ffplay: only output null packet once on EOF

2014-10-29 Thread Marton Balint
The generic decoder properly flushes the codecs with one null packet as well. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/ffplay.c b/ffplay.c index a3b34fd..e04f622 100644 --- a/ffplay.c

Re: [FFmpeg-devel] [PATCH 1/3] ffplay: implement separete audio decoder thread

2014-10-30 Thread Marton Balint
On Thu, 30 Oct 2014, wm4 wrote: On Thu, 30 Oct 2014 00:31:25 +0100 Marton Balint c...@passwd.hu wrote: Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 265 --- 1 file changed, 153 insertions(+), 112 deletions(-) diff

Re: [FFmpeg-devel] [PATCH 1/3] ffplay: implement separete audio decoder thread

2014-11-09 Thread Marton Balint
On Fri, 31 Oct 2014, Marton Balint wrote: On Thu, 30 Oct 2014, wm4 wrote: On Thu, 30 Oct 2014 00:31:25 +0100 Marton Balint c...@passwd.hu wrote: Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 265 --- 1 file

[FFmpeg-devel] [PATCH] avfilter/avf_showwaves: fix off by one error in loop condition

2014-11-09 Thread Marton Balint
It caused segfaults. Signed-off-by: Marton Balint c...@passwd.hu --- libavfilter/avf_showwaves.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/avf_showwaves.c b/libavfilter/avf_showwaves.c index 4cd225a..fa34a52 100644 --- a/libavfilter/avf_showwaves.c +++ b

[FFmpeg-devel] [PATCH] ffplay: signal the frame queue before closing audio

2014-11-10 Thread Marton Balint
Fixed regression caused by 631ac655c00e978e19d05dab572bc1ffd6078c63 when ffplay does not quit if the audio thread is blocked. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index 490bffa..f79161d

Re: [FFmpeg-devel] [PATCH] ffplay: signal the frame queue before closing audio

2014-11-11 Thread Marton Balint
On Mon, 10 Nov 2014, Marton Balint wrote: Fixed regression caused by 631ac655c00e978e19d05dab572bc1ffd6078c63 when ffplay does not quit if the audio thread is blocked. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) Hello

[FFmpeg-devel] [PATCH 1/2] lavfi/ebur128: add support for smaller video sizes

2014-11-16 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/filters.texi| 4 ++-- libavfilter/f_ebur128.c | 11 --- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 53f4cb2..713989c 100644 --- a/doc/filters.texi +++ b/doc

[FFmpeg-devel] [PATCH 2/2] lavfi/ebur128: fix indentation after last commit

2014-11-16 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- libavfilter/f_ebur128.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/f_ebur128.c b/libavfilter/f_ebur128.c index 8780062..63b4dad 100644 --- a/libavfilter/f_ebur128.c +++ b/libavfilter/f_ebur128.c @@ -316,8

Re: [FFmpeg-devel] [PATCH] ffplay: fix mem leak when opening input or parsing options fail.

2014-11-22 Thread Marton Balint
On Fri, 21 Nov 2014, Benoit Fouet wrote: --- ffplay.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index f79161d..1914a66 100644 --- a/ffplay.c +++ b/ffplay.c @@ -3169,8 +3169,9 @@ static int read_thread(void *arg)

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/ebur128: add support for smaller video sizes

2014-11-22 Thread Marton Balint
On Fri, 21 Nov 2014, Clément Bœsch wrote: (sorry for the delay) On Sun, Nov 16, 2014 at 10:53:15PM +0100, Marton Balint wrote: Signed-off-by: Marton Balint c...@passwd.hu --- doc/filters.texi| 4 ++-- libavfilter/f_ebur128.c | 11 --- 2 files changed, 10 insertions(+), 5

[FFmpeg-devel] [PATCH 1/4] ffplay: remove flushed state from decoder context

2014-12-25 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 4 1 file changed, 4 deletions(-) diff --git a/ffplay.c b/ffplay.c index 1914a66..507617e 100644 --- a/ffplay.c +++ b/ffplay.c @@ -185,7 +185,6 @@ typedef struct Decoder { AVCodecContext *avctx; int pkt_serial; int

[FFmpeg-devel] [PATCH 3/4] ffplay: add support for stream specifiers in -ast, -vst, -sst options

2014-12-25 Thread Marton Balint
Also fix the outdated documentation of these options. Signed-off-by: Marton Balint c...@passwd.hu --- doc/ffplay.texi | 29 ++--- ffplay.c| 35 ++- 2 files changed, 36 insertions(+), 28 deletions(-) diff --git a/doc/ffplay.texi b

[FFmpeg-devel] [PATCH 4/4] ffplay: dump format before selecting streams

2014-12-25 Thread Marton Balint
This helps the user to see the available streams just before the error message if the stream specifiers do not match. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ffplay.c b/ffplay.c index 655af6d..72ec35d

Re: [FFmpeg-devel] [PATCH 1/4] ffplay: remove flushed state from decoder context

2015-01-01 Thread Marton Balint
On Sat, 27 Dec 2014, Michael Niedermayer wrote: On Thu, Dec 25, 2014 at 07:57:02PM +0100, Marton Balint wrote: Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 4 1 file changed, 4 deletions(-) LGTM Hello Michael, Please merge from my stable branch for the ffplay patch

[FFmpeg-devel] [PATCH 2/3] ffplay: make eof part of videostate and signal it when opening a stream

2015-02-05 Thread Marton Balint
Otherwise we may not flush a decoder when a new stream is opened during an already eof condition. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/ffplay.c b/ffplay.c index 210bae5..51c3c54 100644

[FFmpeg-devel] [PATCH 3/3] ffplay: factorize thread starting and stopping code into decoder

2015-02-05 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 54 ++ 1 file changed, 22 insertions(+), 32 deletions(-) diff --git a/ffplay.c b/ffplay.c index 51c3c54..8c62f9c 100644 --- a/ffplay.c +++ b/ffplay.c @@ -191,12 +191,11 @@ typedef

[FFmpeg-devel] [PATCH 1/3] ffplay: update frame timer based on last updated clock time when toggling pause

2015-02-05 Thread Marton Balint
It is better than using simply video clock, because video clock may be NAN. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index c112ead..210bae5 100644 --- a/ffplay.c +++ b/ffplay.c @@ -1494,7

Re: [FFmpeg-devel] [RFC][PATCH] ffplay: factorize subtitle rendering code to a private filter

2015-02-16 Thread Marton Balint
On Sat, 14 Feb 2015, Clément Bœsch wrote: On Sat, Feb 14, 2015 at 02:08:15AM +0100, Marton Balint wrote: Signed-off-by: Marton Balint c...@passwd.hu --- Makefile | 1 + doc/ffplay.texi | 4 + ffplay.c | 336

Re: [FFmpeg-devel] [PATCH 1/3] ffplay: update frame timer based on last updated clock time when toggling pause

2015-02-10 Thread Marton Balint
On Fri, 6 Feb 2015, Marton Balint wrote: It is better than using simply video clock, because video clock may be NAN. Signed-off-by: Marton Balint c...@passwd.hu --- Hello Michael, Please merge from my stable branch for the ffplay patch series: 6885009 ffplay: update frame timer based

[FFmpeg-devel] [RFC][PATCH] ffplay: factorize subtitle rendering code to a private filter

2015-02-13 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- Makefile | 1 + doc/ffplay.texi | 4 + ffplay.c | 336 + libavfilter/Makefile | 1 + libavfilter/vf_ffplay_subtitle.c | 347

[FFmpeg-devel] [PATCH] vf_drawtext: add support for setting box border width

2015-03-30 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/filters.texi | 4 libavfilter/vf_drawtext.c | 5 - 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index 15f8ed5..b75ce5a 100644 --- a/doc/filters.texi +++ b/doc/filters.texi

Re: [FFmpeg-devel] [PATCH] ffplay: Switch to show waves mode if allocation/init of RDFT fails

2015-03-03 Thread Marton Balint
On Mon, 2 Mar 2015, Michael Niedermayer wrote: Fixes null pointer dereference Found-by: Paweł pantrom...@gmail.com Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffplay.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index

[FFmpeg-devel] [PATCH] af_channelmap: fix number of channels

2015-03-27 Thread Marton Balint
Fixes segfaults with the following command: ffmpeg -f lavfi -i aevalsrc=0:c=stereo:n=1920 -af 'channelmap=0' test.ac3 Signed-off-by: Marton Balint c...@passwd.hu --- libavfilter/af_channelmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/af_channelmap.c b/libavfilter

[FFmpeg-devel] [PATCH v2] af_channelmap: fix number of channels

2015-03-28 Thread Marton Balint
Fixes segfaults with the following command: ffmpeg -f lavfi -i aevalsrc=0:c=stereo:n=1920 -af 'channelmap=0' test.ac3 Signed-off-by: Marton Balint c...@passwd.hu --- libavfilter/af_channelmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/af_channelmap.c b/libavfilter

Re: [FFmpeg-devel] [PATCH 2/2] ffplay: support snap to 90degree autorotation

2015-05-03 Thread Marton Balint
On Sun, 3 May 2015, Michael Niedermayer wrote: On Sun, May 03, 2015 at 08:04:52PM +0200, wm4 wrote: On Sun, 3 May 2015 03:10:05 +0200 Michael Niedermayer michae...@gmx.at wrote: do people prefer this to be enabled or disabled by default ? Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 1/2] ffplay: unify displaymatrix based rotation code

2015-05-03 Thread Marton Balint
On Sun, 3 May 2015, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffplay.c | 45 +++-- 1 file changed, 23 insertions(+), 22 deletions(-) diff --git a/ffplay.c b/ffplay.c index 826a473..eea00e6 100644 --- a/ffplay.c

[FFmpeg-devel] [PATCH] lavf/img2dec: add option to disable pattern matching

2015-05-11 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 4 libavformat/img2.h| 3 ++- libavformat/img2dec.c | 9 ++--- libavformat/version.h | 2 +- 4 files changed, 13 insertions(+), 5 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 3f7c45e

[FFmpeg-devel] [PATCH 2/4] lavc/utils: change add_metadata_from_side_data to accept avpacket

2015-07-05 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- libavcodec/utils.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index f20cafc..08fad3e 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2296,14 +2296,14 @@ fail

[FFmpeg-devel] [PATCH 4/4] lavc/utils: get rid of add_metadata_from_side_data forward declaration

2015-07-05 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- libavcodec/utils.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index e1870f5..b80b4e7 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -739,7

[FFmpeg-devel] [PATCH 3/4] lavc/utils: call add_metadata_from_side_data in ff_init_buffer_info

2015-07-05 Thread Marton Balint
This should ensure that each frame get its metadata from its proper packet regardless of frame delays caused by reordering or threading. Signed-off-by: Marton Balint c...@passwd.hu --- libavcodec/utils.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/utils.c

[FFmpeg-devel] [PATCH 1/4] lavc/utils: remove redundant call to ff_init_buffer_info

2015-07-05 Thread Marton Balint
It does the same as calling ff_decode_frame_props. Signed-off-by: Marton Balint c...@passwd.hu --- libavcodec/utils.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index e6d5227..f20cafc 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c

Re: [FFmpeg-devel] [PATCH 1/4] concatdec: add support for specifying inpoint of files

2015-07-06 Thread Marton Balint
On Mon, 6 Jul 2015, Nicolas George wrote: L'octidi 18 messidor, an CCXXIII, Marton Balint a écrit : Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 17 + libavformat/concatdec.c | 48 +++- 2 files changed

Re: [FFmpeg-devel] [PATCH 3/4] concatdec: add support for specifying outpoint of files

2015-07-06 Thread Marton Balint
On Mon, 6 Jul 2015, Nicolas George wrote: L'octidi 18 messidor, an CCXXIII, Marton Balint a écrit : Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 13 + libavformat/concatdec.c | 22 -- 2 files changed, 33 insertions(+), 2 deletions

Re: [FFmpeg-devel] [PATCH 4/4] concatdec: add support for injecting packet metadata

2015-07-06 Thread Marton Balint
On Mon, 6 Jul 2015, Nicolas George wrote: L'octidi 18 messidor, an CCXXIII, Marton Balint a écrit : Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 4 libavformat/concatdec.c | 30 ++ 2 files changed, 34 insertions(+) diff --git

[FFmpeg-devel] [PATCH 3/4] concatdec: add support for specifying outpoint of files

2015-07-05 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 13 + libavformat/concatdec.c | 22 -- 2 files changed, 33 insertions(+), 2 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 4bad1c8..4ba797e 100644 --- a/doc

[FFmpeg-devel] [PATCH 4/4] concatdec: add support for injecting packet metadata

2015-07-05 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 4 libavformat/concatdec.c | 30 ++ 2 files changed, 34 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 4ba797e..28244f3 100644 --- a/doc/demuxers.texi +++ b/doc

[FFmpeg-devel] [PATCH 1/4] concatdec: add support for specifying inpoint of files

2015-07-05 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 17 + libavformat/concatdec.c | 48 +++- 2 files changed, 52 insertions(+), 13 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 35a1561

[FFmpeg-devel] [PATCH 2/4] concatdec: store eof condition in context

2015-07-05 Thread Marton Balint
This is needed later for outpoint support which may leave the last file in a not-eof state. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/concatdec.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c index

Re: [FFmpeg-devel] [PATCH] ffplay: Use sws_scale to scale subtitles

2015-07-28 Thread Marton Balint
On Tue, 28 Jul 2015, Michael Niedermayer wrote: From: Michael Niedermayer mich...@niedermayer.cc Fixes some files from Ticket679 This also changes subtitles to 4:2:0 matching the output format and thus simplifying the blend code. This restricts placement to the chroma sample resolution

[FFmpeg-devel] [PATCH] ffplay: do not block audio thread on WIN32

2015-08-03 Thread Marton Balint
The windows SDL audio driver plays the old data in the buffer in a loop if it is not updated in time. So instead of waiting for data and blocking the the audio thread, return silence if no data is available. Should fix ticket #2289. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 7

Re: [FFmpeg-devel] [PATCH] ffplay: add scale_subtitles option to allow disabling the scaling

2015-07-30 Thread Marton Balint
On Wed, 29 Jul 2015, Michael Niedermayer wrote: From: Michael Niedermayer mich...@niedermayer.cc This theoretically is faster and might be useful on some low end embeded systems This issue seems quite theoretical to me. Scaling happens only once per subtitle, usually no more often than

Re: [FFmpeg-devel] [PATCH] ffplay: do not block audio thread on WIN32

2015-08-05 Thread Marton Balint
On Tue, 4 Aug 2015, Michael Niedermayer wrote: On Tue, Aug 04, 2015 at 01:00:52AM +0200, Marton Balint wrote: The windows SDL audio driver plays the old data in the buffer in a loop if it is not updated in time. So instead of waiting for data and blocking the the audio thread, return silence

Re: [FFmpeg-devel] [PATCH] concatdec: fix broken file_inpoint calculation

2015-08-11 Thread Marton Balint
On Tue, 11 Aug 2015, Nicolas George wrote: Le primidi 21 thermidor, an CCXXIII, Marton Balint a écrit : Should fix ticket #4765. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/concatdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM, of course. Sorry

Re: [FFmpeg-devel] [PATCH] doc: make timespec more precise

2015-08-11 Thread Marton Balint
On Tue, 11 Aug 2015, Ganesh Ajjanagadde wrote: Fixes Ticket4595 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- doc/ffmpeg.texi | 11 +++ doc/ffplay.texi | 12 ++-- 2 files changed, 17 insertions(+), 6 deletions(-) [...] diff --git a/doc/ffplay.texi

[FFmpeg-devel] [PATCH] concatdec: fix broken file_inpoint calculation

2015-08-08 Thread Marton Balint
Should fix ticket #4765. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/concatdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c index 1fe294b..969b001 100644 --- a/libavformat/concatdec.c +++ b/libavformat

[FFmpeg-devel] [PATCH 1/2] ffplay: add specific constants for buffer fullness settings when using external clock

2015-08-08 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ffplay.c b/ffplay.c index 7e673f0..6f28694 100644 --- a/ffplay.c +++ b/ffplay.c @@ -67,6 +67,8 @@ const int program_birth_year = 2003; #define MAX_QUEUE_SIZE

[FFmpeg-devel] [PATCH 2/2] ffplay: increase MIN_FRAMES to 25

2015-08-08 Thread Marton Balint
FFplay was using a 5 frame packet buffer, this is not much (e.g. 200 ms for 25fps video), when HLS is requesting a new segment via HTTP, it may take longer for the request to complete. Should fix ticket #4720. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 2 +- 1 file changed, 1

[FFmpeg-devel] [PATCH] mxfdec: set AVFMT_SEEK_TO_PTS demuxer flag

2015-08-09 Thread Marton Balint
Since 53f2ef2c4afb1d49a679dea9163cb0e4671f3117 seeking is done using PTS. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/mxfdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 2d921db..5734976 100644 --- a/libavformat/mxfdec.c

Re: [FFmpeg-devel] [PATCH 2/2] ffplay: increase MIN_FRAMES to 25

2015-08-09 Thread Marton Balint
On Sun, 9 Aug 2015, Michael Niedermayer wrote: On Sat, Aug 08, 2015 at 10:07:00PM +0200, Marton Balint wrote: FFplay was using a 5 frame packet buffer, this is not much (e.g. 200 ms for 25fps video), when HLS is requesting a new segment via HTTP, it may take longer for the request to complete

[FFmpeg-devel] [PATCH] avisynth: fix setting stream timebase

2015-08-10 Thread Marton Balint
Stream timebase should be set using avpriv_set_pts_info, otherwise avctx-pkt_timebase is not correct. This should fix ticket #4766. Patch is only compile tested. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/avisynth.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions

Re: [FFmpeg-devel] [PATCH] ffplay: Use sws_scale to scale subtitles

2015-07-27 Thread Marton Balint
On Sun, 26 Jul 2015, Michael Niedermayer wrote: From: Michael Niedermayer mich...@niedermayer.cc Fixes some files from Ticket679 This also changes subtitles to 4:2:0 matching the output format and thus simplifying the blend code. This restricts placement to the chroma sample resolution

Re: [FFmpeg-devel] [PATCH 6/6] ffplay: auto-insert setfield, tinterlace filters for MXF files with separate fields layout

2015-07-22 Thread Marton Balint
On Wed, 22 Jul 2015, Matthieu Bouron wrote: From: Matthieu Bouron matthieu.bou...@stupeflix.com --- ffplay.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/ffplay.c b/ffplay.c index 58034c6..5038553 100644 --- a/ffplay.c +++ b/ffplay.c @@ -31,6 +31,7 @@ #include

Re: [FFmpeg-devel] [PATCH] mxfdec: calculate the index in display order

2015-07-14 Thread Marton Balint
On Tue, 14 Jul 2015, Tomas Härdin wrote: On Sat, 2015-07-11 at 18:54 +0200, Marton Balint wrote: This should fix seeking for open GOP files as well. Signed-off-by: Marton Balint c...@passwd.hu --- @@ -1411,8 +1417,7 @@ static int mxf_compute_ptses_fake_index(MXFContext *mxf, MXFIndexTable

Re: [FFmpeg-devel] [PATCH 1/5] concatdec: add support for specifying inpoint of files

2015-07-15 Thread Marton Balint
On Sat, 11 Jul 2015, Marton Balint wrote: Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 17 + libavformat/concatdec.c | 39 ++- 2 files changed, 43 insertions(+), 13 deletions(-) Hi Nicolas, Could you please

Re: [FFmpeg-devel] [PATCH 1/5] concatdec: add support for specifying inpoint of files

2015-07-16 Thread Marton Balint
On Thu, 16 Jul 2015, Nicolas George wrote: Le tridi 23 messidor, an CCXXIII, Marton Balint a écrit : Signed-off-by: Marton Balint c...@passwd.hu --- doc/demuxers.texi | 17 + libavformat/concatdec.c | 39 ++- 2 files changed, 43

Re: [FFmpeg-devel] [PATCH 1/5] concatdec: add support for specifying inpoint of files

2015-07-16 Thread Marton Balint
Hello Michael, Please merge from my stable branch for the following patches: 53f2ef2 mxfdec: calculate the index in display order f384132 concatdec: add support for specifying inpoint of files 1ea909a concatdec: store eof condition in context 52bc064 concatdec: add support for specifying

Re: [FFmpeg-devel] [PATCH 2/2] libzvbi-teletextdec: fix AVSubtitleRect pict compatiblity code

2015-10-24 Thread Marton Balint
On Fri, 23 Oct 2015, Marton Balint wrote: Only set pict if we got a valid rect. Signed-off-by: Marton Balint <c...@passwd.hu> --- libavcodec/libzvbi-teletextdec.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) Applied. Regards,

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: use a separate struct for the rescaled YUVA AVSubtitle rectangles

2015-10-24 Thread Marton Balint
On Sat, 24 Oct 2015, Marton Balint wrote: Current code segfaults since the deprecation of AVSubtitleRect.pict because it freed/realloced AVSubtitleRect.pict.data by itself. The new code stores the generated YUVA AVSubtitle rectangles in their own struct and keeps the original AVSubtitle

[FFmpeg-devel] [PATCH 2/2] libzvbi-teletextdec: fix AVSubtitleRect pict compatiblity code

2015-10-23 Thread Marton Balint
Only set pict if we got a valid rect. Signed-off-by: Marton Balint <c...@passwd.hu> --- libavcodec/libzvbi-teletextdec.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/libavcodec/libzvbi-teletextdec.c b/libavcodec/libzvbi-teletextdec.c index c63b1ec..2

[FFmpeg-devel] [PATCH 1/2] ffplay: use AVSubtitleRect.data instead of .pict.data

2015-10-23 Thread Marton Balint
Current code segfaults since the deprecation of AVSubtitleRect.pict because it frees/reallocs AVSubtitleRect.pict.data by itself. Signed-off-by: Marton Balint <c...@passwd.hu> --- ffplay.c | 30 -- 1 file changed, 16 insertions(+), 14 deletions(-) diff

[FFmpeg-devel] [PATCH 4/5] concatdec: add metadata for file start time and duration

2015-10-24 Thread Marton Balint
Signed-off-by: Marton Balint <c...@passwd.hu> --- libavformat/concatdec.c| 6 ++ tests/ref/fate/concat-demuxer-lavf-mxf | 2 +- tests/ref/fate/concat-demuxer-lavf-mxf_d10 | 2 +- tests/ref/fate/concat-demuxer-lavf-ts | 2 +- 4 files changed, 9 insertions

[FFmpeg-devel] [PATCH 2/5] fate: add concat demuxer tests

2015-10-24 Thread Marton Balint
Signed-off-by: Marton Balint <c...@passwd.hu> --- tests/Makefile | 1 + tests/fate-run.sh | 14 tests/fate/concatdec.mak | 12 tests/ref/fate/concat-demuxer-lavf-mxf | 1 + tests/ref/fate/concat-d

[FFmpeg-devel] [PATCH 3/5] concatdec: move duration calculating code to open_file

2015-10-24 Thread Marton Balint
Signed-off-by: Marton Balint <c...@passwd.hu> --- libavformat/concatdec.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c index 7686f28..f262d44 100644 --- a/libavformat/concatdec.c +++ b/libav

[FFmpeg-devel] [PATCH 1/5] ffprobe: add support for printing packet strings metadata as packet tags

2015-10-24 Thread Marton Balint
ffprobe.xsd already contains the tag element. Signed-off-by: Marton Balint <c...@passwd.hu> --- ffprobe.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/ffprobe.c b/ffprobe.c index ac03689..f5930ae 100644 --- a/ffprobe.c +++ b/ffprobe.c @@ -77,6

[FFmpeg-devel] [PATCH 5/5] lavfi/select: add support for concatdec_select option

2015-10-24 Thread Marton Balint
This option can be used to select useful frames from an ffconcat file which is using inpoints and outpoints but where the source files are not intra frame only. Signed-off-by: Marton Balint <c...@passwd.hu> --- doc/filters.texi | 17 + libavfilter/f_select.

Re: [FFmpeg-devel] [PATCH 3/5] concatdec: move duration calculating code to open_file

2015-10-25 Thread Marton Balint
On Sun, 25 Oct 2015, Nicolas George wrote: Le tridi 3 brumaire, an CCXXIV, Marton Balint a écrit : Signed-off-by: Marton Balint <c...@passwd.hu> --- libavformat/concatdec.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavformat/concatd

Re: [FFmpeg-devel] [PATCH 2/5] fate: add concat demuxer tests

2015-10-25 Thread Marton Balint
On Sun, 25 Oct 2015, Nicolas George wrote: [...] --- a/tests/fate-run.sh +++ b/tests/fate-run.sh @@ -249,6 +249,20 @@ gapless(){ do_md5sum $decfile3 } +concat(){ +template=$(target_path $1) Should it really be target_path? The template is in the source. You are right, target

Re: [FFmpeg-devel] [PATCH 4/5] concatdec: add metadata for file start time and duration

2015-10-25 Thread Marton Balint
On Sun, 25 Oct 2015, Nicolas George wrote: Le tridi 3 brumaire, an CCXXIV, Marton Balint a écrit : Signed-off-by: Marton Balint <c...@passwd.hu> --- libavformat/concatdec.c| 6 ++ tests/ref/fate/concat-demuxer-lavf-mxf | 2 +- tests/ref/fate/concat-demuxe

[FFmpeg-devel] [PATCHv2 1/2] ffplay: use a separate struct for the rescaled YUVA AVSubtitle rectangles

2015-10-23 Thread Marton Balint
it is considered invalid API usage. Signed-off-by: Marton Balint <c...@passwd.hu> --- ffplay.c | 48 ++-- 1 file changed, 26 insertions(+), 22 deletions(-) diff --git a/ffplay.c b/ffplay.c index 4a084b4..7c1cb4f 100644 --- a/ffplay.c +++ b/ffplay.c @@

[FFmpeg-devel] [PATCH] ffmpeg: fix ffmpeg.h trailing whitespace

2015-10-21 Thread Marton Balint
How this passed through the commit hook? Signed-off-by: Marton Balint <c...@passwd.hu> --- ffmpeg.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ffmpeg.h b/ffmpeg.h index 5722816..8c232f3 100644 --- a/ffmpeg.h +++ b/ffmpeg.h @@ -273,7 +273,7 @@ typedef

Re: [FFmpeg-devel] [PATCHv2 1/5] ffmpeg: log failed av_write_trailer

2015-10-21 Thread Marton Balint
On Mon, 19 Oct 2015, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:07:34PM +0200, Marton Balint wrote: Signed-off-by: Marton Balint <c...@passwd.hu> --- ffmpeg.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index 36a68fb..252bc0d

Re: [FFmpeg-devel] [PATCH 2/5] ffmpeg: exit on av_write_trailer failure if exit_on_error is set

2015-10-21 Thread Marton Balint
On Sun, 18 Oct 2015, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:38:51AM +0200, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:24:04AM +0200, Marton Balint wrote: Signed-off-by: Marton Balint <c...@passwd.hu> --- ffmpeg.c | 2 ++ 1 file changed, 2 insertions(+) diff

Re: [FFmpeg-devel] [PATCH 3/5] ffmpeg: factorize checking decoder result

2015-10-21 Thread Marton Balint
On Sun, 18 Oct 2015, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:24:05AM +0200, Marton Balint wrote: Signed-off-by: Marton Balint <c...@passwd.hu> --- ffmpeg.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) LGTM thanks Applied,

Re: [FFmpeg-devel] [PATCHv2 4/5] ffmpeg: exit on corrupt packets or decoded frames if exit_on_error flag is present

2015-10-21 Thread Marton Balint
On Mon, 19 Oct 2015, Michael Niedermayer wrote: On Mon, Oct 19, 2015 at 01:57:31AM +0200, Marton Balint wrote: Signed-off-by: Marton Balint <c...@passwd.hu> --- ffmpeg.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) should be ok Applied, thanks.

Re: [FFmpeg-devel] [PATCH] ffmpeg: fix ffmpeg.h trailing whitespace

2015-10-22 Thread Marton Balint
On Thu, 22 Oct 2015, Carl Eugen Hoyos wrote: Marton Balint passwd.hu> writes: How this passed through the commit hook? I'd also like to know but please push. Ok, pushed. Thanks, Marton ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.

Re: [FFmpeg-devel] [PATCH 5/5] ffmpeg: add abort_on option to allow aborting on empty output

2015-10-22 Thread Marton Balint
On Mon, 19 Oct 2015, Marton Balint wrote: On Mon, 19 Oct 2015, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:24:07AM +0200, Marton Balint wrote: Signed-off-by: Marton Balint <c...@passwd.hu> --- doc/ffmpeg.texi | 8 ffmpeg.c| 7 +++ ffmpeg.h

  1   2   3   4   5   6   7   8   9   10   >