Re: [FFmpeg-devel] [PATCH]Support linebreaks in onCaption subtitles

2015-05-04 Thread Clément Bœsch
On Mon, May 04, 2015 at 09:41:35AM +, Carl Eugen Hoyos wrote: Carl Eugen Hoyos cehoyos at ag.or.at writes: Clément Bœsch u at pkh.me writes: +text-linebreaks = br/; i'm afraid this won't be enough to support br, BR, br /, ... I only looked at the sample we have,

Re: [FFmpeg-devel] [PATCH]Set interlaced flag when muxing interlaced dnxhd into mxf

2015-05-04 Thread Carl Eugen Hoyos
On Monday 20 April 2015 12:26:50 am Tomas Härdin wrote: On Fri, 2015-04-17 at 00:31 +0200, Carl Eugen Hoyos wrote: Hi! Completely untested. The original patch was successfully tested: http://ffmpeg.gusari.org/viewtopic.php?f=11t=2065 No idea really. Can't this flag be taken automagically

Re: [FFmpeg-devel] [PATCH] matroskadec: check s-streams[k] before using it

2015-05-04 Thread Michael Niedermayer
On Sun, May 03, 2015 at 11:55:20PM +0200, Andreas Cadhalpun wrote: This fixes a segmentation fault. Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] [PATCH]Support linebreaks in onCaption subtitles

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 11:48:04AM +0200, Clément Bœsch wrote: On Mon, May 04, 2015 at 09:41:35AM +, Carl Eugen Hoyos wrote: Carl Eugen Hoyos cehoyos at ag.or.at writes: Clément Bœsch u at pkh.me writes: +text-linebreaks = br/; i'm afraid this won't be enough to

[FFmpeg-devel] Re: [PATCH] matroskadec: check s-streams[k] before using it

2015-05-04 Thread Andreas Cadhalpun
On 04.05.2015 12:39, Michael Niedermayer wrote: On Sun, May 03, 2015 at 11:55:20PM +0200, Andreas Cadhalpun wrote: This fixes a segmentation fault. Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-04 Thread Anshul
On 05/04/2015 03:09 AM, Michael Niedermayer wrote: On Sun, May 03, 2015 at 08:53:14PM +0200, Clément Bœsch wrote: On Sun, May 03, 2015 at 06:46:15PM +0530, Anshul wrote: On 05/02/2015 08:24 PM, Clément Bœsch wrote: On Tue, Apr 28, 2015 at 07:50:15PM +0530, Anshul wrote: On 04/28/2015

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-04 Thread Anshul
On 05/04/2015 05:27 PM, Anshul wrote: On 05/04/2015 03:09 AM, Michael Niedermayer wrote: On Sun, May 03, 2015 at 08:53:14PM +0200, Clément Bœsch wrote: On Sun, May 03, 2015 at 06:46:15PM +0530, Anshul wrote: On 05/02/2015 08:24 PM, Clément Bœsch wrote: On Tue, Apr 28, 2015 at 07:50:15PM

Re: [FFmpeg-devel] [PATCH] libavformat/keyframes: output keyframe indexes

2015-05-04 Thread Jonathan Viney
Hi, Thanks for the feedback. After having another look, this can actually already be done with ffprobe. I was incorrectly using -show_frames instead of -show_packets, causing a big performance hit. This works well, with the same performance: ffprobe -show_packets -print_format compact input.mkv

Re: [FFmpeg-devel] [PATCH] matroskadec: check s-streams[k] before using it

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 01:24:08PM +0200, Andreas Cadhalpun wrote: On 04.05.2015 12:39, Michael Niedermayer wrote: On Sun, May 03, 2015 at 11:55:20PM +0200, Andreas Cadhalpun wrote: This fixes a segmentation fault. Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com ---

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 05:30:05PM +0530, Anshul wrote: On 05/04/2015 05:27 PM, Anshul wrote: On 05/04/2015 03:09 AM, Michael Niedermayer wrote: On Sun, May 03, 2015 at 08:53:14PM +0200, Clément Bœsch wrote: On Sun, May 03, 2015 at 06:46:15PM +0530, Anshul wrote: On 05/02/2015 08:24

Re: [FFmpeg-devel] [PATCH]Set interlaced flag when muxing interlaced dnxhd into mxf

2015-05-04 Thread tomas . hardin
On 2015-05-04 12:18, Carl Eugen Hoyos wrote: On Monday 20 April 2015 12:26:50 am Tomas Härdin wrote: On Fri, 2015-04-17 at 00:31 +0200, Carl Eugen Hoyos wrote: Hi! Completely untested. The original patch was successfully tested: http://ffmpeg.gusari.org/viewtopic.php?f=11t=2065 No idea

[FFmpeg-devel] ffserver jpg patch

2015-05-04 Thread ill
I changed it to git and add the .write header to the last part. The first part compiled after I changed ffserver.c, so it is no longer corrupted. Muxer detection seems to work when I tried it for the middle part. It should be good to fix ffserver now? diff -Nrup ffmpeg-git--orig/ffserver.c

Re: [FFmpeg-devel] ffserver jpg patch

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 06:35:36PM -0600, ill wrote: I changed it to git and add the .write header to the last part. The first part compiled after I changed ffserver.c, so it is no longer corrupted. Muxer detection seems to work when I tried it for the middle part. It should be good to fix

Re: [FFmpeg-devel] ffserver jpg patch

2015-05-04 Thread ill
patch attached. On Mon, May 04, 2015 at 06:35:36PM -0600, ill wrote: I changed it to git and add the .write header to the last part. The first part compiled after I changed ffserver.c, so it is no longer corrupted. Muxer detection seems to work when I tried it for the middle part. It should

Re: [FFmpeg-devel] [PATCH] libavformat/keyframes: output keyframe indexes

2015-05-04 Thread Lou Logan
On Mon, May 4, 2015, at 04:09 AM, Jonathan Viney wrote: This works well, with the same performance: ffprobe -show_packets -print_format compact input.mkv | grep -n flags=K | cut -f1 -d: Using -show_entries packet=flags may be faster than -show_packets.

Re: [FFmpeg-devel] [PATCH] avidec: avoid infinite loop due to negative ast-sample_size

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 11:01:45PM +0200, Andreas Cadhalpun wrote: If max in clean_index is set to a negative ast-sample_size, the following loop never ends: while (max 1024) max += max; Thus set ast-sample_size to 0 if it would otherwise be negative. Signed-off-by:

[FFmpeg-devel] [PATCH] avidec: avoid infinite loop due to negative ast-sample_size

2015-05-04 Thread Andreas Cadhalpun
If max in clean_index is set to a negative ast-sample_size, the following loop never ends: while (max 1024) max += max; Thus set ast-sample_size to 0 if it would otherwise be negative. Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com ---

Re: [FFmpeg-devel] [PATCH]Fix cropdetect for 8 bit

2015-05-04 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: Attached patch fixes cropdetect here for yuv422p10le (no other formats tested), I don't know if the change to diff makes sense or not. LGTM i think the div change is not needed but i might be wrong Merged without the div change. Thank

Re: [FFmpeg-devel] [PATCH] vp9: add profile names

2015-05-04 Thread James Almer
On 03/05/15 11:35 PM, James Almer wrote: On 03/05/15 3:55 AM, Ronald S. Bultje wrote: Hi, On Sun, May 3, 2015 at 2:15 AM, James Almer jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/avcodec.h | 3 +++ libavcodec/version.h | 4 ++-- libavcodec/vp9.c

Re: [FFmpeg-devel] [PATCH]Allow easy png streamcopying

2015-05-04 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: On Thursday 23 April 2015 03:05:48 pm Michael Niedermayer wrote: without really thinking about it, the idea to remove the extension for apng or change it to .apng, seems like a good idea New patch attached. This variant was merged. Carl Eugen

Re: [FFmpeg-devel] [PATCH]Set interlaced flag when muxing interlaced dnxhd into mxf

2015-05-04 Thread Carl Eugen Hoyos
tomas.hardin at codemill.se writes: The original patch was successfully tested: http://ffmpeg.gusari.org/viewtopic.php?f=11t=2065 No idea really. Can't this flag be taken automagically from the codec somehow? New patch attached. Short and simple. Looks fine to me Merged by