Re: [FFmpeg-devel] [PATCH] doc/filters: Add ascii graphics to clarify what the currently implemented tinterlace modes do

2014-12-04 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/12/14 17:13, Michael Niedermayer wrote: > On Tue, Dec 02, 2014 at 05:02:49PM +, tim nicholson wrote: >> On 02/12/14 15:33, Michael Niedermayer wrote: >>> Signed-off-by: Michael Niedermayer >>> --- >>> doc/filters.texi | 131 >>> ++

Re: [FFmpeg-devel] [PATCH] avfilter/tinterlace: merge code with interlace

2014-12-04 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/12/14 13:08, Michael Niedermayer wrote: > On Tue, Dec 02, 2014 at 07:43:47PM +0100, Clément Bœsch wrote: >> --- >> After this commit, interlace doesn't behave the same if the input frames >> are already interlaced. Does anyone wants me to keep th

Re: [FFmpeg-devel] [PATCH] avfilter/tinterlace: merge code with interlace

2014-12-04 Thread tim nicholson
On 02/12/14 18:43, Clément Bœsch wrote: > --- > After this commit, interlace doesn't behave the same if the input frames > are already interlaced. Does anyone wants me to keep this behaviour? > > Also, the PTS adjustments might be different, but I didn't follow the > current discussions about it s

Re: [FFmpeg-devel] [PATCH] avfilter/tinterlace: merge code with interlace

2014-12-04 Thread Clément Bœsch
On Thu, Dec 04, 2014 at 09:31:47AM +, tim nicholson wrote: > On 02/12/14 18:43, Clément Bœsch wrote: > > --- > > After this commit, interlace doesn't behave the same if the input frames > > are already interlaced. Does anyone wants me to keep this behaviour? > > > > Also, the PTS adjustments m

Re: [FFmpeg-devel] [PATCH 2/2] RELEASE: Bump

2014-12-04 Thread Michael Niedermayer
On Wed, Dec 03, 2014 at 08:15:20PM -0800, Timothy Gu wrote: > Signed-off-by: Timothy Gu > --- > RELEASE | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid

[FFmpeg-devel] Fix bug for POWER LE: libavcodec/ppc/vp8dsp_altivec.c

2014-12-04 Thread rongyan
Hi, We propose this patch to fix vp8 bugs for POWER8 little endian. Functions put_vp8_epel_h_altivec_core(), puta_vp8_epel_v_altivec_core(), and put_vp8_pixels16_altivec() are fixed. The passed test cases change from 2296/2328 to 2328/2328. Thanks for your review. Rong Yan --

Re: [FFmpeg-devel] Fix bug for POWER LE: avcodec/ppc/vc1dsp_altivec.c

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 02:06:34PM +0800, rongyan wrote: > Hi, > We propose this patch to fix vc1 bugs for POWER8 little endian. Function > vc1_inv_trans_8x4_altivec() is fixed. > The passed test cases change from 2296/2328 to 2303/2328‍. > > ‍‍ > > Thanks for your review. > Rong Yan > >

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Nicolas George
Le quartidi 14 frimaire, an CCXXIII, Anshul a écrit : > It would be valid, c708 is backward compatible as far as > I have seen while working with ccextractor. > C708 is made in such a way that people dont have to throw there older TV > sets or device. > if any decoder is not able to decode c708 par

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Kieran Kunhya
> Furthermore, I do not see any ASS styling in the decoder, that would be a > good test case. Captions shouldn't be ASS styled - they should be styled as per the specification (see VLC). Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http:

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Clément Bœsch
On Thu, Dec 04, 2014 at 11:32:40AM +, Kieran Kunhya wrote: > > Furthermore, I do not see any ASS styling in the decoder, that would be a > > good test case. > > Captions shouldn't be ASS styled - they should be styled as per the > specification (see VLC). > I'd be surprised if you can't get

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Nicolas George
Le quartidi 14 frimaire, an CCXXIII, Kieran Kunhya a écrit : > Captions shouldn't be ASS styled - they should be styled as per the > specification (see VLC). You are certainly right, but we are talking about a decoder, and FFmpeg's internal format for decoded text subtitles is still ASS. If someth

Re: [FFmpeg-devel] [PATCH] lavfi: add tdiff filter

2014-12-04 Thread Stefano Sabatini
On date Sunday 2014-11-23 14:57:41 +0100, Clément Bœsch encoded: > On Sun, Nov 23, 2014 at 02:50:51PM +0100, Stefano Sabatini wrote: > > On date Sunday 2014-11-23 02:01:16 +0100, Clément Bœsch encoded: > > > On Sat, Nov 22, 2014 at 11:44:29PM +0100, Stefano Sabatini wrote: > > > > TODO: bump minor

Re: [FFmpeg-devel] [PATCH 3/3] lavc/libxvid: workaround for bug in libxvidcore

2014-12-04 Thread Clément Bœsch
On Thu, Dec 04, 2014 at 12:39:29AM +0100, Lukasz Marek wrote: > On 24.11.2014 01:17, Lukasz Marek wrote: > >libxvidcore calculate number of threads basing on video height. > >If height is small enough it allocates 0 bytes long memory and > >writes to it. > >Setting thread_count to 0 uses 1 thread a

[FFmpeg-devel] [PATCH] Incorrect aspect ratio in MPEG2 essence

2014-12-04 Thread Erik Johansson
Hi, Setting setting sample_aspect_ratio in AVContext does not produce the expected aspect index value in the MPEG2 sequence header. Problem seems to be around line 263 in mpeg12enc.c. Setting the sample_aspect_ratio to 16/9 leads the current error minimization routine to determine that error -=

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Nicolas George
Le quartidi 14 frimaire, an CCXXIII, Anshul a écrit : > Yes while doing that only I realized, if we use "i" then cc must be in > first stream. > When I had multiple stream(with audio), then I was not able to map subcc > to my video stream. > using stream_idx , we are saying that if we can use out

Re: [FFmpeg-devel] [PATCH] web: Update Bower dependencies

2014-12-04 Thread Michael Niedermayer
On Wed, Dec 03, 2014 at 08:56:11PM -0800, Timothy Gu wrote: > On Tue, Nov 25, 2014 at 7:50 PM, Timothy Gu wrote: > > Also explicitly add jQuery as a dependency. It is copied from the Makefile. > > > > Signed-off-by: Timothy Gu > > --- > > bower.json| 9 +- > > htdoc

Re: [FFmpeg-devel] [PATCH 4/4] style: Update from ffmpeg-web

2014-12-04 Thread Michael Niedermayer
On Wed, Dec 03, 2014 at 09:34:11PM -0800, Timothy Gu wrote: > Signed-off-by: Timothy Gu > --- > > Forgot this one > > -- > doc/style.min.css | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applying: style: Update from ffmpeg-web fatal: corrupt patch at line 31 [...] -- Michael Gn

[FFmpeg-devel] [PATCH] lavd/lavfi: allow to extract subcc.

2014-12-04 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/indevs.texi | 14 libavdevice/lavfi.c | 92 + 2 files changed, 99 insertions(+), 7 deletions(-) Correctly rebased today, with some extra doc. If the decoder is applied before this patch, then

Re: [FFmpeg-devel] [PATCH] avfilter/tinterlace: merge code with interlace

2014-12-04 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/12/14 09:59, Clément Bœsch wrote: > On Thu, Dec 04, 2014 at 09:31:47AM +, tim nicholson wrote: >> On 02/12/14 18:43, Clément Bœsch wrote: >>> --- >>> After this commit, interlace doesn't behave the same if the input frames >>> are already int

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Anshul Maheshwari
On Thu, Dec 4, 2014 at 5:43 PM, Nicolas George wrote: > Le quartidi 14 frimaire, an CCXXIII, Anshul a écrit : > > Yes while doing that only I realized, if we use "i" then cc must be in > > first stream. > > When I had multiple stream(with audio), then I was not able to map subcc > > to my video

Re: [FFmpeg-devel] [PATCH] avfilter/tinterlace: merge code with interlace

2014-12-04 Thread Clément Bœsch
On Thu, Dec 04, 2014 at 12:41:47PM +, tim nicholson wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 04/12/14 09:59, Clément Bœsch wrote: > > On Thu, Dec 04, 2014 at 09:31:47AM +, tim nicholson wrote: > >> On 02/12/14 18:43, Clément Bœsch wrote: > >>> --- > >>> After this co

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-04 Thread Nicolas George
Le quartidi 14 frimaire, an CCXXIII, Anshul Maheshwari a écrit : > while with stream_idx, even if I you dont know where is video and audio it > works. I do not think so, I suspect it only worked by chance. You may as well end up trying to extract subcc from audio, or in a completely random order.

Re: [FFmpeg-devel] Fix bug for POWER LE: libavcodec/ppc/vp8dsp_altivec.c

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 06:48:18PM +0800, rongyan wrote: > Hi, > We propose this patch to fix vp8 bugs for POWER8 little endian. Functions > put_vp8_epel_h_altivec_core(), puta_vp8_epel_v_altivec_core(), and > put_vp8_pixels16_altivec() are fixed. > The passed test cases change from 2296/2328 to

[FFmpeg-devel] revised patches for dv profile

2014-12-04 Thread Steve Jiekak
- fixed defect with invalid framerate - included libavformat\dvenc.c in second patch Steve ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 1/2] add av_dv_codec_profile2 : uses framerate to select best matching profile. default on first matching profile

2014-12-04 Thread Steve Jiekak
Signed-off-by: Steve Jiekak --- doc/APIchanges |3 +++ libavcodec/dv_profile.c | 24 ++-- libavcodec/dv_profile.h |6 ++ libavcodec/version.h|2 +- 4 files changed, 32 insertions(+), 3 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges in

[FFmpeg-devel] [PATCH 2/2] replaced av_dv_codec_profile by av_dv_codec_profile2 in encoder and dv muxers

2014-12-04 Thread Steve Jiekak
Signed-off-by: Steve Jiekak --- libavcodec/dvenc.c |2 +- libavformat/dv.c|4 ++-- libavformat/dvenc.c |3 ++- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/dvenc.c b/libavcodec/dvenc.c index 5d810e3..7061508 100644 --- a/libavcodec/dvenc.c +++ b/libav

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/pngdec: use memcpy instead of byte loops for P frames.

2014-12-04 Thread Benoit Fouet
Hi, Le 03/12/2014 14:31, Benoit Fouet a écrit : Rely on the way memcpy is optimized for one's system instead of looping on a byte buffer for buffer copies to handle P frames. --- libavcodec/pngdec.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] doc/filters: Add ascii graphics to clarify what the currently implemented tinterlace modes do

2014-12-04 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/12/14 08:24, tim nicholson wrote: > On 02/12/14 17:13, Michael Niedermayer wrote: >> On Tue, Dec 02, 2014 at 05:02:49PM +, tim nicholson wrote: >>> On 02/12/14 15:33, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --

Re: [FFmpeg-devel] [PATCH] doc/filters: Add ascii graphics to clarify what the currently implemented tinterlace modes do

2014-12-04 Thread Clément Bœsch
On Thu, Dec 04, 2014 at 04:03:00PM +, tim nicholson wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 04/12/14 08:24, tim nicholson wrote: > > On 02/12/14 17:13, Michael Niedermayer wrote: > >> On Tue, Dec 02, 2014 at 05:02:49PM +, tim nicholson wrote: > >>> On 02/12/14 15:33

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/pngdec: use memcpy instead of byte loops for P frames.

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 03:28:32PM +0100, Benoit Fouet wrote: > Hi, > > Le 03/12/2014 14:31, Benoit Fouet a écrit : > >Rely on the way memcpy is optimized for one's system instead of looping > >on a byte buffer for buffer copies to handle P frames. > >--- > > libavcodec/pngdec.c | 25

Re: [FFmpeg-devel] [PATCH] Incorrect aspect ratio in MPEG2 essence

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 01:11:08PM +0100, Erik Johansson wrote: > Hi, > > Setting setting sample_aspect_ratio in AVContext does not produce the > expected aspect index value in the MPEG2 sequence header. > > Problem seems to be around line 263 in mpeg12enc.c. Setting the > sample_aspect_ratio to

Re: [FFmpeg-devel] [PATCH] doc/filters: Add ascii graphics to clarify what the currently implemented tinterlace modes do

2014-12-04 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/12/14 16:09, Clément Bœsch wrote: > [..] > > This representation requires way more efforts to me to understand than the > one from Michael... > Just shows how different minds think/work ;) As the saying goes, one man's meat is another man's p

Re: [FFmpeg-devel] [PATCH] web: Update Bower dependencies

2014-12-04 Thread db0 company
Did you check that the latest versions of the dependencies still make the website look the same? I remember having retro-compatibility issues with Bootstrap in the past, but that was a long time ago, so it's probably OK. -- db0 - Deby Barbara Lepage http://db0.fr/ __

Re: [FFmpeg-devel] [PATCH] imc: make issue non-fatal

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:04:50AM +0100, Christophe Gisquet wrote: > Until a noticeably distorted output or crash is reported, it's better > ignoring the "error", as this results in better output on the affected > samples. (ticket #3886) > > On the other hand, I have a hard time understanding the

Re: [FFmpeg-devel] [PATCH 2/2] replaced av_dv_codec_profile by av_dv_codec_profile2 in encoder and dv muxers

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 02:48:15PM +0100, Steve Jiekak wrote: > Signed-off-by: Steve Jiekak > --- > libavcodec/dvenc.c |2 +- > libavformat/dv.c|4 ++-- > libavformat/dvenc.c |3 ++- > 3 files changed, 5 insertions(+), 4 deletions(-) applied thx [...] -- Michael GnuPG fin

Re: [FFmpeg-devel] [PATCH 1/2] add av_dv_codec_profile2 : uses framerate to select best matching profile. default on first matching profile

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 02:48:14PM +0100, Steve Jiekak wrote: > Signed-off-by: Steve Jiekak > --- > doc/APIchanges |3 +++ > libavcodec/dv_profile.c | 24 ++-- > libavcodec/dv_profile.h |6 ++ > libavcodec/version.h|2 +- > 4 files changed, 32 in

Re: [FFmpeg-devel] [PATCH] web: Update Bower dependencies

2014-12-04 Thread Timothy Gu
On Thu, Dec 4, 2014 at 4:22 AM, Michael Niedermayer wrote: > On Wed, Dec 03, 2014 at 08:56:11PM -0800, Timothy Gu wrote: >> On Tue, Nov 25, 2014 at 7:50 PM, Timothy Gu wrote: >> > Also explicitly add jQuery as a dependency. It is copied from the Makefile. >> > >> > Signed-off-by: Timothy Gu >> >

Re: [FFmpeg-devel] [PATCH] web: Update Bower dependencies

2014-12-04 Thread Timothy Gu
On Thu, Dec 4, 2014 at 9:07 AM, db0 company wrote: > Did you check that the latest versions of the dependencies still make > the website look the same? I did, and it's OK. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/

Re: [FFmpeg-devel] [PATCH] web: Update Bower dependencies

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:55:05PM -0800, Timothy Gu wrote: > And with patch > From 51ff20706034f2d74e32a643727c71f404aeebaa Mon Sep 17 00:00:00 2001 > From: Timothy Gu > Date: Tue, 25 Nov 2014 19:50:06 -0800 > Subject: [PATCH] web: Update Bower dependencies > > Also explicitly add jQuery as a d

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c refactor resolving MultiDescriptor and remove essence group hack

2014-12-04 Thread Tomas Härdin
On Tue, 2014-12-02 at 16:43 -0800, Mark Reid wrote: > I think this is a better way to deal with single frame essence data then my > previous way. > > --- > libavformat/mxfdec.c | 62 > > 1 file changed, 38 insertions(+), 24 deletions(-) > > d

[FFmpeg-devel] parallelization over outputs

2014-12-04 Thread Daniel Oberhoff
Hello, I was wondering if ffmpeg parallelizes over outputs, i.e. if I have one input, but multiple outputs, with different codecs and/or filters, since these outputs are independent from each other they could be rendered/encoded in different threads (or thread groups), is this the case? Best

Re: [FFmpeg-devel] [PATCH 3/3] lavc/libxvid: workaround for bug in libxvidcore

2014-12-04 Thread Lukasz Marek
On 04.12.2014 12:42, Clément Bœsch wrote: On Thu, Dec 04, 2014 at 12:39:29AM +0100, Lukasz Marek wrote: On 24.11.2014 01:17, Lukasz Marek wrote: libxvidcore calculate number of threads basing on video height. If height is small enough it allocates 0 bytes long memory and writes to it. Setting t

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c refactor resolving MultiDescriptor and remove essence group hack

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 11:11:48PM +0100, Tomas Härdin wrote: > On Tue, 2014-12-02 at 16:43 -0800, Mark Reid wrote: > > I think this is a better way to deal with single frame essence data then my > > previous way. > > > > --- > > libavformat/mxfdec.c | 62 > >

Re: [FFmpeg-devel] [PATCH 2/2] lavc/options: fix leaks in avcodec_copy_context

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:54:30AM +0100, Lukasz Marek wrote: > On 27.11.2014 04:17, Michael Niedermayer wrote: > >On Thu, Nov 27, 2014 at 12:43:57AM +0100, Lukasz Marek wrote: > >>On 24.11.2014 05:16, Lukasz Marek wrote: > >>>Signed-off-by: Lukasz Marek > >>>--- > >>> libavcodec/options.c | 23 +

[FFmpeg-devel] [PATCH 1/3] avformat/hlsenc: added segment filename template

2014-12-04 Thread Christian Suloway
Signed-off-by: Christian Suloway --- libavformat/hlsenc.c | 87 +--- 1 file changed, 69 insertions(+), 18 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index e13f438..0a48919 100644 --- a/libavformat/hlsenc.c +++ b/libavform

[FFmpeg-devel] [PATCH 3/3] avformat/hlsenc: added HLS encryption

2014-12-04 Thread Christian Suloway
HLS encryption with key URL & key containing file specified with -hls_key_info_file (key info updates when modified). Option for IV from segment number or random generation (-hls_random_iv). Signed-off-by: Christian Suloway --- libavformat/hlsenc.c | 225 +

[FFmpeg-devel] [PATCH 2/3] avformat/hlsenc: added segment file deletion

2014-12-04 Thread Christian Suloway
Option removes segments no longer in playlist when older than playlist + segment duration Signed-off-by: Christian Suloway --- libavformat/hlsenc.c | 72 1 file changed, 67 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/li

Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:50:06AM +0100, Lukasz Marek wrote: > On 03.12.2014 01:28, Michael Niedermayer wrote: > >On Wed, Dec 03, 2014 at 01:24:55AM +0100, Lukasz Marek wrote: > >>On 03.12.2014 01:15, Michael Niedermayer wrote: > >>>On Wed, Dec 03, 2014 at 12:06:47AM +0100, Lukasz Marek wrote: > >

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: [...] > index 49c8b40..4148752 100644 > --- a/libavcodec/options.c > +++ b/libavcodec/options.c > @@ -303,3 +303,176 @@ const AVClass *avcodec_get_subtitle_rect_class(void) > { > return &av_subtitle_rect_class; > } > + > +#define

[FFmpeg-devel] [PATCH] Fixed PNG decoding with Paeth filter for RGBA 64-bits.

2014-12-04 Thread Dominique Leroux
Found and fixed an artifact in the last column of decoded RGBA 64-bits PNG images. The code was dealing with a SIMD-optimized version of the function without taking into account that we can have RGB/RGBA images that are respectively 6 or 8 bytes per pixel (not just 3 or 4). Dominique 0001-F

Re: [FFmpeg-devel] [PATCH 1/3] avformat/hlsenc: added segment filename template

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 05:05:24PM -0600, Christian Suloway wrote: > Signed-off-by: Christian Suloway > --- > libavformat/hlsenc.c | 87 > +--- > 1 file changed, 69 insertions(+), 18 deletions(-) missing documentation > > diff --git a/libavform

Re: [FFmpeg-devel] [PATCH 3/3] lavc/libxvid: workaround for bug in libxvidcore

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 11:33:03PM +0100, Lukasz Marek wrote: > On 04.12.2014 12:42, Clément Bœsch wrote: > >On Thu, Dec 04, 2014 at 12:39:29AM +0100, Lukasz Marek wrote: > >>On 24.11.2014 01:17, Lukasz Marek wrote: > >>>libxvidcore calculate number of threads basing on video height. > >>>If height