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 b

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 f

[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 ffm

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. > > Signe

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 wrote: >> >>> Signed-off-by: James Almer >>> --- >>> libavcodec/avcodec.h | 3 +++ >>> libavcodec/version.h | 4 ++-- >>> libavcodec/vp9.c | 22 +

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]Fix cropdetect for >8 bit

2015-05-04 Thread Carl Eugen Hoyos
Michael Niedermayer 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 you,

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

2015-05-04 Thread Carl Eugen Hoyos
Carl Eugen Hoyos 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
codemill.se> writes: > > The original patch was successfully tested: > > http://ffmpeg.gusari.org/viewtopic.php?f=11&t=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

[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 --- libavformat/avidec.c | 1 + 1 file changed, 1 ins

Re: [FFmpeg-devel] [PATCH] avidec: check for valid bit_rate range

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 09:07:52PM +0200, Andreas Cadhalpun wrote: > If bit_rate is negative, it can trigger an av_assert2 in av_rescale_rnd. > > Since av_rescale returns int64_t, but st->codec_bit_rate is int, it can > also overflow into a negative value. > > Signed-off-by: Andreas Cadhalpun >

[FFmpeg-devel] [PATCH] avidec: check for valid bit_rate range

2015-05-04 Thread Andreas Cadhalpun
If bit_rate is negative, it can trigger an av_assert2 in av_rescale_rnd. Since av_rescale returns int64_t, but st->codec_bit_rate is int, it can also overflow into a negative value. Signed-off-by: Andreas Cadhalpun --- libavformat/avidec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion

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=11&t=2065 No ide

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: >

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 > >> --- > >> libavformat/ma

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

2015-05-04 Thread wm4
On Mon, 4 May 2015 10:15:54 + (UTC) Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > > Carl Eugen Hoyos ag.or.at> writes: > > > > > > > Clément Bœsch pkh.me> writes: > > > > > > > > > > +text->linebreaks = ""; > > > > > > > > > > i'm afraid this won't be enough to supp

[FFmpeg-devel] [PATCH] avcodec/mips: MSA (MIPS-SIMD-Arch) optimizations for HEVC uni copy, uni horizontal and uni vertical mc functions

2015-05-04 Thread shivraj.patil
From: Shivraj Patil Signed-off-by: Shivraj Patil --- libavcodec/mips/hevcdsp_init_mips.c | 26 + libavcodec/mips/hevcdsp_mips.h | 40 + libavcodec/mips/hevcdsp_msa.c | 1822 +++ libavutil/mips/generic_macros_msa.h | 533 ++ 4 files changed

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] 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] 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 02:14

[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 >> --- >> libavformat/matroskadec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff -

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 ag.or.at> writes: > > > > > Clément Bœsch pkh.me> writes: > > > > > > > > +text->linebreaks = ""; > > > > > > > > i'm afraid this won't be

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

2015-05-04 Thread Michael Niedermayer
On Mon, May 04, 2015 at 02:35:45AM +1200, Jonathan Viney wrote: > Hi, > > First patch here. I wrote this format to be able to get the keyframe > indexes faster than seemed possible with ffprobe, and also to get more > familiar with ffmpeg. > > ffmpeg -i input.mkv -c copy -f keyframes -keyframes_f

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 > --- > libavformat/matroskadec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/matroskadec.c b/libavformat/matroskad

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=11&t=2065 > No idea really. Can't this flag be taken autom

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

2015-05-04 Thread Carl Eugen Hoyos
Clément Bœsch pkh.me> writes: > > Carl Eugen Hoyos ag.or.at> writes: > > > > > Clément Bœsch pkh.me> writes: > > > > > > > > +text->linebreaks = ""; > > > > > > > > i'm afraid this won't be enough to support > > > > "", "", "", ... > > > > > > I only looked at the sample we have, are t

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 ag.or.at> writes: > > > Clément Bœsch pkh.me> writes: > > > > > > +text->linebreaks = ""; > > > > > > i'm afraid this won't be enough to support > > > "", "", "", ... > > > > I only looked at the sample

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

2015-05-04 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > Clément Bœsch pkh.me> writes: > > > > +text->linebreaks = ""; > > > > i'm afraid this won't be enough to support > > "", "", "", ... > > I only looked at the sample we have, are they > all valid line breaks? Ping, I don't understand your comment. C