[FFmpeg-devel] [PATCH] avformat/rtmpproto: Handle NetStream.Play.PublishNotify by re-creating stream.

2016-10-22 Thread Ahmed Ammar
--- libavformat/rtmpproto.c | 49 + 1 file changed, 37 insertions(+), 12 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 95d1c1d..8a3 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -227,6

Re: [FFmpeg-devel] [PATCH] mpegts: handle AVMEDIA_TYPE_UNKNOWN correctly

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 01:22:21AM +0200, Andreas Cadhalpun wrote: > It is negative, so can't be used for left shifting. > > This fixes ubsan runtime error: shift exponent -1 is negative > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/mpegts.c | 3 ++- > 1 file changed, 2 insertions(+)

Re: [FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

2016-10-22 Thread Marton Balint
On Sat, 22 Oct 2016, Kieran Kunhya wrote: On Fri, 21 Oct 2016, 20:05 Marton Balint, wrote: On Thu, 20 Oct 2016, Michael Niedermayer wrote: > On Tue, Oct 18, 2016 at 05:36:27PM -0700, Carlos Fernandez Sanz wrote: >> From: Carlos Fernandez >> >> Signed-off-by: Carlos Fernandez >> --- >> l

[FFmpeg-devel] [PATCH] lavf/mpegts: add missed fixes to scte35 section callback

2016-10-22 Thread Marton Balint
They somehow got lost along the patch versions. Signed-off-by: Marton Balint --- libavformat/mpegts.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 97a2225..bcc13e3 100644 --- a/libavformat/mpegts.c +++ b/libavforma

Re: [FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

2016-10-22 Thread Kieran Kunhya
On Sat, 22 Oct 2016, 06:13 Rostislav Pehlivanov, wrote: > On 22 October 2016 at 01:17, Kieran Kunhya wrote: > > > On Fri, 21 Oct 2016, 20:05 Marton Balint, wrote: > > > > > > > > On Thu, 20 Oct 2016, Michael Niedermayer wrote: > > > > > > > On Tue, Oct 18, 2016 at 05:36:27PM -0700, Carlos Ferna

[FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- doc/patchwork | 9 + 1 file changed, 9 insertions(+) create mode 100644 doc/patchwork diff --git a/doc/patchwork b/doc/patchwork new file mode 100644 index 000..9486e07 --- /dev/null +++ b/doc/patchwork @@ -0,0 +1,9 @@ +Patchwork states + +NEW:

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Clément Bœsch
On Sat, Oct 22, 2016 at 01:38:47PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/patchwork | 9 + > 1 file changed, 9 insertions(+) > create mode 100644 doc/patchwork > > diff --git a/doc/patchwork b/doc/patchwork > new file mode 100644 > index 0

Re: [FFmpeg-devel] [PATCH 1/7] avcodec/svq1enc: Clear MMX state after svq1_encode_plane()

2016-10-22 Thread Michael Niedermayer
On Fri, Oct 21, 2016 at 03:31:40PM +0200, Michael Niedermayer wrote: > svq1_encode_plane() uses SIMD and we call libc memory allocation functions > after it > > Signed-off-by: Michael Niedermayer > --- > libavcodec/svq1enc.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) appl

Re: [FFmpeg-devel] [PATCH 2/7] avcodec/cavsdec: Clear MMX state after MB decode loop

2016-10-22 Thread Michael Niedermayer
On Fri, Oct 21, 2016 at 03:31:41PM +0200, Michael Niedermayer wrote: > The MMX state must be cleared between using MMX and using memory allocation > thats basically the only location between the 2 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/cavsdec.c | 1 + > 1 file changed, 1 inse

Re: [FFmpeg-devel] [PATCH 3/7] avcodec/utils: Move emms_c() before memory allocation functions in avcodec_encode_video2()

2016-10-22 Thread Michael Niedermayer
On Fri, Oct 21, 2016 at 03:31:42PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/utils.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Modern terror

Re: [FFmpeg-devel] [PATCH 4/7] avcodec/mpegvideo_enc: Add missing emms_c() to clear MMX state after SIMD use

2016-10-22 Thread Michael Niedermayer
On Fri, Oct 21, 2016 at 03:42:09PM +0200, Michael Niedermayer wrote: > On Fri, Oct 21, 2016 at 03:31:43PM +0200, Michael Niedermayer wrote: > > Fixes undefined behavior due to calling libc allocation with unclean FPU > > state > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/mp

Re: [FFmpeg-devel] [PATCH 5/7] avcodec/snowenc: Clear MMX state after edge drawing and picture encode

2016-10-22 Thread Michael Niedermayer
On Fri, Oct 21, 2016 at 03:31:44PM +0200, Michael Niedermayer wrote: > Fixes undefined behavior from calling libc allocation with unclean FPU state. > > Signed-off-by: Michael Niedermayer > --- > libavcodec/snowenc.c | 2 ++ > 1 file changed, 2 insertions(+) applied [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 6/7] avutil/avassert: Add av_assertX_fpu()

2016-10-22 Thread Michael Niedermayer
On Fri, Oct 21, 2016 at 03:31:45PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavutil/avassert.h | 9 + > libavutil/utils.c| 13 + > 2 files changed, 22 insertions(+) applied with version bump [...] -- Michael GnuPG fingerprint

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 02:12:18PM +0200, Clément Bœsch wrote: > On Sat, Oct 22, 2016 at 01:38:47PM +0200, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > doc/patchwork | 9 + > > 1 file changed, 9 insertions(+) > > create mode 100644 doc/patchwork > > > >

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 02:25:19PM +0200, Michael Niedermayer wrote: > On Sat, Oct 22, 2016 at 02:12:18PM +0200, Clément Bœsch wrote: > > On Sat, Oct 22, 2016 at 01:38:47PM +0200, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > > --- > > > doc/patchwork | 9 + > > >

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 02:31:06PM +0200, Michael Niedermayer wrote: > On Sat, Oct 22, 2016 at 02:25:19PM +0200, Michael Niedermayer wrote: > > On Sat, Oct 22, 2016 at 02:12:18PM +0200, Clément Bœsch wrote: > > > On Sat, Oct 22, 2016 at 01:38:47PM +0200, Michael Niedermayer wrote: > > > > Signed-of

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-22 Thread Ronald S. Bultje
Hi, On Fri, Oct 21, 2016 at 4:36 PM, Moritz Barsnick wrote: > On Thu, Oct 20, 2016 at 12:06:14 -0400, Ronald S. Bultje wrote: > > +s->did_range_warn = 1; > > +} > > // fall-through > > +} > > case AVCOL_RANGE_MPEG: > > The fall-through comment seems misplace

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-22 Thread Moritz Barsnick
On Sat, Oct 22, 2016 at 08:43:26 -0400, Ronald S. Bultje wrote: > Is there documentation on what coverity expects? I found this: https://lost-contact.mit.edu/afs/cs.stanford.edu/pkg/prevent-4.3.1/i386_linux26/opt/prevent-linux-4.3.1/cgi-bin/doc/checker_ref.html#c_checker_MISSING_BREAK Moritz

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Stephen Hutchinson
On 10/22/2016 8:25 AM, Michael Niedermayer wrote: On Sat, Oct 22, 2016 at 02:12:18PM +0200, Clément Bœsch wrote: On Sat, Oct 22, 2016 at 01:38:47PM +0200, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- doc/patchwork | 9 + 1 file changed, 9 insertions(+) create mod

Re: [FFmpeg-devel] [PATCH] mpegts: handle AVMEDIA_TYPE_UNKNOWN correctly

2016-10-22 Thread Andreas Cadhalpun
On 22.10.2016 11:41, Michael Niedermayer wrote: > On Sat, Oct 22, 2016 at 01:22:21AM +0200, Andreas Cadhalpun wrote: >> It is negative, so can't be used for left shifting. >> >> This fixes ubsan runtime error: shift exponent -1 is negative >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavfor

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Bodecs Bela
2016.10.22. 18:29 keltezéssel, Stephen Hutchinson írta: On 10/22/2016 8:25 AM, Michael Niedermayer wrote: On Sat, Oct 22, 2016 at 02:12:18PM +0200, Clément Bœsch wrote: On Sat, Oct 22, 2016 at 01:38:47PM +0200, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- doc/patchwork

[FFmpeg-devel] [PATCH] faq: use relative links to own documentation

2016-10-22 Thread Andreas Cadhalpun
This way locally installed documentation refers to itself instead of the website. Bud-Id: https://bugs.debian.org/841501 Signed-off-by: Andreas Cadhalpun --- doc/faq.texi | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/doc/faq.texi b/doc/faq.texi index ef111c7.

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Lou Logan
On Sat, Oct 22, 2016, at 09:49 AM, Bodecs Bela wrote: > > under review: someone marked it because he/she investigate this patch. > So the patch submitter and other potential reviewers may feel/be_sure > that this patch is already handled by someone else. > I suggest to use it. This is psychologic

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight

2016-10-22 Thread James Almer
On 10/20/2016 2:05 PM, Nicolas George wrote: > Le nonidi 29 vendémiaire, an CCXXV, Dave Rice a écrit : >> You could also make the storage simpler by considering whether or not to >> write DisplayWidth and DisplayHeight independently. For instance: >> PixelWidth=720 >> PixelHeight=480 >> DisplayWidt

[FFmpeg-devel] [PATCH 02/13] avcodec: Clear MMX state in ff_thread_report_progress(INT_MAX)

2016-10-22 Thread Michael Niedermayer
This decreases the number of FPU state violations in fate on x86-64 from 309 to 79 Signed-off-by: Michael Niedermayer --- libavcodec/pthread_frame.c | 3 +++ libavcodec/utils.c | 2 ++ 2 files changed, 5 insertions(+) diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c

[FFmpeg-devel] [PATCH 03/13] avcodec/pngdec: Clear MMX state in png_handle_row()

2016-10-22 Thread Michael Niedermayer
zlib calls malloc/free thus emms is needed between inflate and png_handle_row() Signed-off-by: Michael Niedermayer --- libavcodec/pngdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 36275ae..916e68f 100644 --- a/libavcodec/pngdec.c +++ b/l

[FFmpeg-devel] [PATCH 01/13] avcodec/diracdec: Clear MMX state after dirac_decode_frame_internal()

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/diracdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/diracdec.c b/libavcodec/diracdec.c index b183fad..e295b14 100644 --- a/libavcodec/diracdec.c +++ b/libavcodec/diracdec.c @@ -2184,6 +2184,7 @@ static int dirac_decode_data_u

[FFmpeg-devel] [PATCH 05/13] avcodec/utvideoenc: Clear MMX state after predict loop

2016-10-22 Thread Michael Niedermayer
The subsequent huffman code performs several memory allocations, MMX state thus needs to be cleared between Signed-off-by: Michael Niedermayer --- libavcodec/utvideoenc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/utvideoenc.c b/libavcodec/utvideoenc.c index 8ffc263..fd27caa

[FFmpeg-devel] [PATCH 04/13] avcodec/error_resilience: Clear MMX state after the MC/ME loop

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/error_resilience.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/error_resilience.c b/libavcodec/error_resilience.c index 2790599..83b5c92 100644 --- a/libavcodec/error_resilience.c +++ b/libavcodec/error_resilience.c @@ -1187,6

[FFmpeg-devel] [PATCH 07/13] avcodec/vp56: Clear mmx state after mb decode loop

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/vp56.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c index 6319248..2125000 100644 --- a/libavcodec/vp56.c +++ b/libavcodec/vp56.c @@ -668,6 +668,7 @@ static int ff_vp56_decode_mbs(AVCodecContext *avctx,

[FFmpeg-devel] [PATCH 06/13] avcodec/vp3: Move emms_c() before early exit claus in vp3_draw_horiz_band()

2016-10-22 Thread Michael Niedermayer
There are calls to *malloc() after it Signed-off-by: Michael Niedermayer --- libavcodec/vp3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c index fa749be..c9add4a 100644 --- a/libavcodec/vp3.c +++ b/libavcodec/vp3.c @@ -1432,6 +1432,7 @@

[FFmpeg-devel] [PATCH 13/13] avcodec/interplayvideo: Clear MMX state after ipvideo_decode_opcodes()

2016-10-22 Thread Michael Niedermayer
Fixes last failing fate test on x86_64 Signed-off-by: Michael Niedermayer --- libavcodec/interplayvideo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/interplayvideo.c b/libavcodec/interplayvideo.c index aa03015..66f6d23 100644 --- a/libavcodec/interplayvideo.c +++ b/libavcodec

[FFmpeg-devel] [PATCH 11/13] avcodec/mpegutils: Move emms_c() before conditional return paths in ff_draw_horiz_band()

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/mpegutils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpegutils.c b/libavcodec/mpegutils.c index 62cc36a..1e6620e 100644 --- a/libavcodec/mpegutils.c +++ b/libavcodec/mpegutils.c @@ -42,6 +42,8 @@ void ff_d

[FFmpeg-devel] [PATCH 12/13] avutil/tests/pixelutils: Add emms_c() after SIMD use before calling libc (malloc/free) code

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavutil/tests/pixelutils.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavutil/tests/pixelutils.c b/libavutil/tests/pixelutils.c index ec4dc8f..fbb66e5 100644 --- a/libavutil/tests/pixelutils.c +++ b/libavutil/tests/pixelutils.c @@ -67,6 +67,8

[FFmpeg-devel] [PATCH 10/13] avcodec/h264dec: Move emms_c() call before conditional return paths in ff_h264_draw_horiz_band()

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/h264dec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index ed0b724..2e17124 100644 --- a/libavcodec/h264dec.c +++ b/libavcodec/h264dec.c @@ -116,6 +116,8 @@ void ff_h264_draw

[FFmpeg-devel] [PATCH 08/13] avcodec/h264_slice: Clear MMX state after decode_slice()

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/h264_slice.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c index 25a5890..b5f2e2d 100644 --- a/libavcodec/h264_slice.c +++ b/libavcodec/h264_slice.c @@ -2608,6 +2608,7 @@ int ff_h264_execute_

[FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/svq1dec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c index 2b72e08..0fe222e 100644 --- a/libavcodec/svq1dec.c +++ b/libavcodec/svq1dec.c @@ -744,6 +744,7 @@ static int svq1_decode_frame(AVCode

Re: [FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 10:41:29AM -0800, Lou Logan wrote: > On Sat, Oct 22, 2016, at 09:49 AM, Bodecs Bela wrote: > > > > under review: someone marked it because he/she investigate this patch. > > So the patch submitter and other potential reviewers may feel/be_sure > > that this patch is alread

Re: [FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

2016-10-22 Thread Carlos Fernandez Sanz
On Sat, Oct 22, 2016 at 4:15 AM, Kieran Kunhya wrote: > > Getting patches merged by sheer attrition is a shameful way of "running" an > open source project. I think ignoring follow-up questions from the patch submitter, like you are doing with mine, it's plain rude. Basically your position is "S

[FFmpeg-devel] doc/filters : add metadata info for blackframe filter

2016-10-22 Thread Martin Vignali
Hello, in attach a patch to add information about the metadata export by the blackframe filter. Martin From d9b68a8176d202853b64924e3e1e1b97b9698083 Mon Sep 17 00:00:00 2001 From: Martin Vignali Date: Sat, 22 Oct 2016 21:57:07 +0200 Subject: [PATCH] doc/filters : add metadata information for bla

[FFmpeg-devel] [PATCH] avformat/matroskaenc: use display aspect ratio for DisplayWidth and DisplayHeight when possible

2016-10-22 Thread James Almer
This avoids potential rounding errors and guarantees the source aspect ratio is preserved. Keep writing pixel values when Stereo 3D Mode is enabled and for WebM, as the format doesn't support anything else. This fixes ticket #5743, implementing the suggestion from ticket #5903. Signed-off-by: Jam

Re: [FFmpeg-devel] [PATCH 1/3] avformat/matroskaenc: support writting Chroma Location elements

2016-10-22 Thread James Almer
On 10/15/2016 12:40 PM, James Almer wrote: > Signed-off-by: James Almer > --- > libavformat/matroskaenc.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c > index a48f5bb..d6557a1 100644 > --- a/libavformat/matroskaenc.c > +++

Re: [FFmpeg-devel] [PATCH] lavf/mpegts: add missed fixes to scte35 section callback

2016-10-22 Thread Marton Balint
On Sat, 22 Oct 2016, Marton Balint wrote: They somehow got lost along the patch versions. Signed-off-by: Marton Balint --- libavformat/mpegts.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 97a2225..bcc13e3 100644

Re: [FFmpeg-devel] [PATCH 2/7] avutil/mips: loongson add mmi utils header file

2016-10-22 Thread Michael Niedermayer
On Mon, Oct 10, 2016 at 04:07:59PM +0800, 周晓勇 wrote: > From 51abc8e9981b99d23c7d253bfb603b25aaabf4aa Mon Sep 17 00:00:00 2001 > From: Zhou Xiaoyong > Date: Mon, 10 Oct 2016 14:21:55 +0800 > Subject: [PATCH 2/7] avutil/mips: loongson add mmi utils header file > > > 1.mmiutils.h defined MMI_ load/

Re: [FFmpeg-devel] [PATCH 3/7] avcodec/mips: loongson optimize mmi load and store operators

2016-10-22 Thread Michael Niedermayer
On Mon, Oct 10, 2016 at 04:09:12PM +0800, 周晓勇 wrote: > From ec6fd41adff8541180f4e43e019e31ff06867789 Mon Sep 17 00:00:00 2001 > From: Zhou Xiaoyong > Date: Mon, 10 Oct 2016 14:31:39 +0800 > Subject: [PATCH 3/7] avcodec/mips: loongson optimize mmi load and store > operators > > > 1.MMI_ load/sto

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 12:19 PM, Moritz Barsnick wrote: > On Sat, Oct 22, 2016 at 08:43:26 -0400, Ronald S. Bultje wrote: > > Is there documentation on what coverity expects? > > I found this: > https://lost-contact.mit.edu/afs/cs.stanford.edu/pkg/ > prevent-4.3.1/i386_linux26/opt/prevent-l

Re: [FFmpeg-devel] [PATCH 02/13] avcodec: Clear MMX state in ff_thread_report_progress(INT_MAX)

2016-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer wrote: > This decreases the number of FPU state violations in fate on x86-64 from > 309 to 79 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/pthread_frame.c | 3 +++ > libavcodec/utils.c | 2 ++ > 2 files changed, 5 in

Re: [FFmpeg-devel] [PATCH 07/13] avcodec/vp56: Clear mmx state after mb decode loop

2016-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/vp56.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c > index 6319248..2125000 100644 > --- a/libavcodec/vp56.c > +++ b/libavc

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-22 Thread Ronald S. Bultje
Hi, general comment about all other dec patches. On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/svq1dec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c > index 2b72e08

Re: [FFmpeg-devel] [PATCH 07/13] avcodec/vp56: Clear mmx state after mb decode loop

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 10:07:21PM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer > wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/vp56.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavcodec/vp56.c b/li

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-22 Thread Michael Niedermayer
On Sat, Oct 22, 2016 at 10:10:01PM -0400, Ronald S. Bultje wrote: > Hi, > > general comment about all other dec patches. > > On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer > wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/svq1dec.c | 2 ++ > > 1 file changed, 2 in

[FFmpeg-devel] [PATCH] avutil/x86/emms: Document the emms_c() vs alloc/free relation.

2016-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavutil/x86/emms.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/x86/emms.h b/libavutil/x86/emms.h index 6fda6e2..42c18e2 100644 --- a/libavutil/x86/emms.h +++ b/libavutil/x86/emms.h @@ -31,6 +31,8 @@ void avpriv_emms_yasm(void); * Empt

Re: [FFmpeg-devel] [PATCH] faq: use relative links to own documentation

2016-10-22 Thread James Almer
On 10/22/2016 3:17 PM, Andreas Cadhalpun wrote: > This way locally installed documentation refers to itself instead of the > website. > > Bud-Id: https://bugs.debian.org/841501 > Signed-off-by: Andreas Cadhalpun > --- > doc/faq.texi | 14 +++--- > 1 file changed, 7 insertions(+), 7 delet

[FFmpeg-devel] [PATCH] libavcodec/tests: Added test for libavcodec/avpacket.c

2016-10-22 Thread Thomas Turner
Function(s) Tested: av_packet_clone(). This test checks if av_packet_clone() can successfully make a copy of an AVPacket. Compares all data members in AVPacket EXCEPT for "buf" because "buf" is initialized to NIL in the original AVPacket [to be cloned]. This test also prints out the all the con

[FFmpeg-devel] [PATCH] libavcodec/tests: Added test for libavcodec/avpacket.c

2016-10-22 Thread Thomas Turner
Function(s) Tested: av_packet_clone(). This test checks if av_packet_clone() can successfully make a copy of an AVPacket. Compares all data members in AVPacket EXCEPT for "buf" because "buf" is initialized to NIL in the original AVPacket [to be cloned]. This test also prints out the all the con