[FFmpeg-devel] [PATCH] tests: add some ASS/SSA/MKS remux and transcode tests

2015-05-25 Thread Clément Bœsch
--- http://b.pkh.me/1ededcbd7b.ass http://b.pkh.me/a9-misc.ssa 825502bec102d064edcb3a76301bc438 fate-samples/sub/1ededcbd7b.ass e39d15a63e914b5e683331e098dc fate-samples/sub/a9-misc.ssa --- tests/fate/subtitles.mak| 9 tests/ref/fate/binsub-mksenc| 1 + t

Re: [FFmpeg-devel] [PATCH] libshine: fix support for shine 3.0

2015-05-25 Thread Andreas Cadhalpun
On 24.05.2015 18:56, Michael Niedermayer wrote: > On Sun, May 24, 2015 at 03:52:21PM +0200, Andreas Cadhalpun wrote: >> shine_encode_buffer expects written to be an int pointer, while the >> previous shine_encode_frame expected it to be a long pointer. >> >> Thus encoding with libshine currently al

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Andreas Cadhalpun
On 24.05.2015 19:23, Michael Niedermayer wrote: > On Sun, May 24, 2015 at 03:21:25PM +0200, Andreas Cadhalpun wrote: >> mov.c |5 + >> 1 file changed, 5 insertions(+) >> 199a14800f0d79aa85b8dc01c2c1dc2743c3fb0d >> 0001-mov-check-for-negative-stsc-count.patch >> From 1d66ce9f96ccd4e130837

Re: [FFmpeg-devel] [PATCH] Improve upon dynamic arrays- movtext subtitles

2015-05-25 Thread Philip Langdale
On Mon, 25 May 2015 00:37:03 +0530 Niklesh Lalwani wrote: > From: Niklesh > > Signed-off-by: Niklesh > --- > libavcodec/movtextdec.c | 32 > 1 file changed, 16 insertions(+), 16 deletions(-) > > diff --git a/libavcodec/movtextdec.c b/libavcodec/movtextdec.c >

Re: [FFmpeg-devel] [PATCH] tests: add some ASS/SSA/MKS remux and transcode tests

2015-05-25 Thread Michael Niedermayer
On Mon, May 25, 2015 at 02:33:49PM +0200, Clément Bœsch wrote: > --- > > http://b.pkh.me/1ededcbd7b.ass > http://b.pkh.me/a9-misc.ssa samples uploaded patch should be ok in 24h [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Old school: Use the lowest level la

Re: [FFmpeg-devel] [PATCH] Improve upon dynamic arrays- movtext subtitles

2015-05-25 Thread Niklesh Lalwani
Updated Patch. Thanks, Niklesh On Mon, May 25, 2015 at 9:10 PM, Philip Langdale wrote: > On Mon, 25 May 2015 00:37:03 +0530 > Niklesh Lalwani wrote: > > > From: Niklesh > > > > Signed-off-by: Niklesh > > --- > > libavcodec/movtextdec.c | 32 > > 1 file chan

Re: [FFmpeg-devel] [PATCH] Improve upon dynamic arrays- movtext subtitles

2015-05-25 Thread Philip Langdale
On Mon, 25 May 2015 21:51:36 +0530 Niklesh Lalwani wrote: > Updated Patch. > > Thanks, > > Niklesh use "return AVERROR(ENOMEM);" > On Mon, May 25, 2015 at 9:10 PM, Philip Langdale > wrote: > > > On Mon, 25 May 2015 00:37:03 +0530 > > Niklesh Lalwani wrote: > > > > > From: Niklesh > > > >

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Michael Niedermayer
On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: > On 24.05.2015 19:23, Michael Niedermayer wrote: > > On Sun, May 24, 2015 at 03:21:25PM +0200, Andreas Cadhalpun wrote: > >> mov.c |5 + > >> 1 file changed, 5 insertions(+) > >> 199a14800f0d79aa85b8dc01c2c1dc2743c3fb0d

[FFmpeg-devel] [PATCH] avcodec/vdpau: Don't leak h264.h into all VDPAU codecs

2015-05-25 Thread Philip Langdale
h264.h and hevc.h are mutually exclusive due to defining some of the same names. As such, we need to avoid forcing h264.h to be included if we want hevc decode acceleration to be possible. Signed-off-by: Philip Langdale --- libavcodec/vc1dec.c | 1 + libavcodec/vdpau_internal.h | 3 ++-

[FFmpeg-devel] [PATCH 1/2] fft: x86: Clear __SECT__ before declaring a structure

2015-05-25 Thread Timothy Gu
This commit reverts 599888a48. This silences the following warning: libavcodec/x86/fft.asm:54: warning: section flags ignored on section redeclaration The cause of this warning is that because `struc` and `endstruc` attempts to revert to the previous section state [1]. The section state

[FFmpeg-devel] [PATCH 2/2] resample: x86: Clear __SECT__ before declaring a structure

2015-05-25 Thread Timothy Gu
Silences a warning: libswresample/x86/resample.asm:48: warning: section flags ignored on section redeclaration --- libswresample/x86/resample.asm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libswresample/x86/resample.asm b/libswresample/x86/resample.asm index a57ff37..10f29b0 10

Re: [FFmpeg-devel] [PATCH] Improve upon dynamic arrays- movtext subtitles

2015-05-25 Thread Niklesh Lalwani
Updated. Thanks, Niklesh From 00647a47883b0b547da77d3101f3f890634a85b4 Mon Sep 17 00:00:00 2001 From: Niklesh Date: Mon, 25 May 2015 22:46:04 +0530 Subject: [PATCH] Improve upon dynamic arrays- movtext subtitles Signed-off-by: Niklesh --- libavcodec/movtextdec.c | 34 ++---

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Andreas Cadhalpun
On 25.05.2015 18:31, Michael Niedermayer wrote: > On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: >> On 24.05.2015 19:23, Michael Niedermayer wrote: >>> ISO/IEC 14496-12:2012(E) says the field is unsigned so it cannot be >>> negative >>> >>> 8.7.4.2 Syntax >>> aligned(8) class

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: Don't leak h264.h into all VDPAU codecs

2015-05-25 Thread Michael Niedermayer
On Mon, May 25, 2015 at 09:46:14AM -0700, Philip Langdale wrote: > h264.h and hevc.h are mutually exclusive due to defining some of the same > names. As such, we need to avoid forcing h264.h to be included if we want > hevc decode acceleration to be possible. > > Signed-off-by: Philip Langdale >

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Michael Niedermayer
On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: [...] > mov.c |7 +++ > 1 file changed, 7 insertions(+) > 7ff306f094f2ecd47b720deb20ea318c24efaf4d > 0002-mov-reject-zero-bytes_per_frame-with-non-zero-sample.patch > From 42c8b0c216b39fd2cb8b329669737ce771ecdd20 Mon Sep

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: Don't leak h264.h into all VDPAU codecs

2015-05-25 Thread Philip Langdale
On Mon, 25 May 2015 20:34:34 +0200 Michael Niedermayer wrote: > On Mon, May 25, 2015 at 09:46:14AM -0700, Philip Langdale wrote: > > h264.h and hevc.h are mutually exclusive due to defining some of > > the same names. As such, we need to avoid forcing h264.h to be > > included if we want hevc dec

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Andreas Cadhalpun
On 25.05.2015 21:38, Michael Niedermayer wrote: > On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: > [...] > >> mov.c |7 +++ >> 1 file changed, 7 insertions(+) >> 7ff306f094f2ecd47b720deb20ea318c24efaf4d >> 0002-mov-reject-zero-bytes_per_frame-with-non-zero-sample.pat

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Michael Niedermayer
On Mon, May 25, 2015 at 11:22:32PM +0200, Andreas Cadhalpun wrote: > On 25.05.2015 21:38, Michael Niedermayer wrote: > > On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: > > [...] > > > >> mov.c |7 +++ > >> 1 file changed, 7 insertions(+) > >> 7ff306f094f2ecd47b720deb20

[FFmpeg-devel] [PATCH] configure: Check if ID3D11VideoDecoder exists, not just the header to enable d3d11va

2015-05-25 Thread Michael Niedermayer
Fixes build with "mingw-w64-x86-64-dev 3.2.0-2" Signed-off-by: Michael Niedermayer --- configure |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 52448af..c21d475 100755 --- a/configure +++ b/configure @@ -1815,6 +1815,7 @@ TOOLCHAIN_FEATURES=

Re: [FFmpeg-devel] [libav-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Andreas Cadhalpun
On 25.05.2015 23:41, Luca Barbato wrote: > On 25/05/15 23:22, Andreas Cadhalpun wrote: >> On 25.05.2015 21:38, Michael Niedermayer wrote: >>> On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: >>> [...] >>> mov.c |7 +++ 1 file changed, 7 insertions(+) 7ff3

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-25 Thread Andreas Cadhalpun
On 25.05.2015 23:44, Michael Niedermayer wrote: > On Mon, May 25, 2015 at 11:22:32PM +0200, Andreas Cadhalpun wrote: >> On 25.05.2015 21:38, Michael Niedermayer wrote: >>> On Mon, May 25, 2015 at 05:25:18PM +0200, Andreas Cadhalpun wrote: >>> [...] >>> mov.c |7 +++ 1 file change

[FFmpeg-devel] [PATCH 1/2] Revert "Move struc FFTContext below SECTION_RODATA"

2015-05-25 Thread Timothy Gu
This reverts commit 599888a48040f4086c9086aae958b7231b37afcc. The commit does not silence the warning on ELF-based systems, and will be fixed in the subsequent commit. Conflicts: libavcodec/x86/fft_mmx.asm --- libavcodec/x86/fft.asm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(

[FFmpeg-devel] [PATCH 2/2] x86inc: Clear __SECT__

2015-05-25 Thread Timothy Gu
This commit silences warning(s) like: libavcodec/x86/fft.asm:93: warning: section flags ignored on section redeclaration The cause of this warning is that because `struc` and `endstruc` attempts to revert to the previous section state [1]. The section state is stored in the macro __SECT__

[FFmpeg-devel] [PATCH] avcodec/vdpau: Re-factor pre-hwaccel helper functions into separate header

2015-05-25 Thread Philip Langdale
h264.h and hevc.h are mutually exclusive due to defining some of the same names. As such, we need to avoid forcing h264.h to be included if we want hevc decode acceleration to be possible. However, some of the pre-hwaccel helper functions need h264.h. To avoid messy collisions, let's move the decl

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: Don't leak h264.h into all VDPAU codecs

2015-05-25 Thread Philip Langdale
On Mon, 25 May 2015 13:14:05 -0700 Philip Langdale wrote: > On Mon, 25 May 2015 20:34:34 +0200 > Michael Niedermayer wrote: > > > On Mon, May 25, 2015 at 09:46:14AM -0700, Philip Langdale wrote: > > > h264.h and hevc.h are mutually exclusive due to defining some of > > > the same names. As such

Re: [FFmpeg-devel] [PATCH] configure: Check if ID3D11VideoDecoder exists, not just the header to enable d3d11va

2015-05-25 Thread Michael Niedermayer
On Tue, May 26, 2015 at 12:24:44AM +0200, Michael Niedermayer wrote: > Fixes build with "mingw-w64-x86-64-dev 3.2.0-2" > > Signed-off-by: Michael Niedermayer > --- > configure |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: Re-factor pre-hwaccel helper functions into separate header

2015-05-25 Thread Michael Niedermayer
On Mon, May 25, 2015 at 04:29:12PM -0700, Philip Langdale wrote: > h264.h and hevc.h are mutually exclusive due to defining some of the same > names. As such, we need to avoid forcing h264.h to be included if we want > hevc decode acceleration to be possible. > > However, some of the pre-hwaccel h

Re: [FFmpeg-devel] [PATCH] ffprobe: add stream_index to frame data

2015-05-25 Thread Dave Rice
> On May 14, 2015, at 3:53 AM, Stefano Sabatini wrote: > > [...] >> From c655acffb8d9acc97e19553c2e464c0d78586063 Mon Sep 17 00:00:00 2001 >> From: Dave Rice >> Date: Wed, 13 May 2015 13:38:29 -0400 >> Subject: [PATCH] ffprobe: print stream_index with frame data >> >> otherwise the frame data

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: Re-factor pre-hwaccel helper functions into separate header

2015-05-25 Thread Philip Langdale
On Tue, 26 May 2015 02:21:43 +0200 Michael Niedermayer wrote: > On Mon, May 25, 2015 at 04:29:12PM -0700, Philip Langdale wrote: > > h264.h and hevc.h are mutually exclusive due to defining some of > > the same names. As such, we need to avoid forcing h264.h to be > > included if we want hevc dec