Re: [FFmpeg-devel] [PATCH] avformat/oggenc: fix page duration calculation when granule differs from timestamp

2016-07-07 Thread James Almer
On 7/7/2016 9:58 PM, Michael Niedermayer wrote: > On Thu, Jul 07, 2016 at 07:13:47PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> >> libavformat/oggenc.c | 10 +- >> 1 file changed, 5 insertions(+), 5 deletions(-) >> >> diff --git a/libavformat/oggenc.c b/libavformat/

Re: [FFmpeg-devel] [PATCH 3/4] avformat/oggenc: make Vorbis audio the only default for ogg muxer

2016-07-07 Thread James Almer
On 7/7/2016 10:55 PM, Michael Niedermayer wrote: > On Wed, Jul 06, 2016 at 08:25:26PM -0300, James Almer wrote: >> While not enforced, RFC 5334 mentions that the .ogg extension should >> be used for Vorbis audio files only. >> >> Signed-off-by: James Almer >> --- >> This patch turns the ogg muxer

Re: [FFmpeg-devel] [PATCH 3/4] avformat/oggenc: make Vorbis audio the only default for ogg muxer

2016-07-07 Thread Michael Niedermayer
On Wed, Jul 06, 2016 at 08:25:26PM -0300, James Almer wrote: > While not enforced, RFC 5334 mentions that the .ogg extension should > be used for Vorbis audio files only. > > Signed-off-by: James Almer > --- > This patch turns the ogg muxer into an audio only muxer to follow > the RFC's recommend

Re: [FFmpeg-devel] [PATCH] avformat/oggenc: fix page duration calculation when granule differs from timestamp

2016-07-07 Thread Michael Niedermayer
On Thu, Jul 07, 2016 at 07:13:47PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > > libavformat/oggenc.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c > index f998af3..296028e 100644 > --- a/libavf

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Michael Niedermayer
On Thu, Jun 30, 2016 at 03:36:31PM +0200, Michael Niedermayer wrote: > On Wed, Jun 29, 2016 at 04:15:12PM +, Dan Parrot wrote: > > This patch addresses Trac ticket #5570. The optimized functions are in file > > libswscale/ppc/input_vsx.c. Each optimized function name is a concatenation > > of

Re: [FFmpeg-devel] [PATCH 1/2] h2645_parse: only read avc length code at the correct position

2016-07-07 Thread Michael Niedermayer
On Thu, Jul 07, 2016 at 08:21:17PM +0200, Hendrik Leppkes wrote: > Reading it from any other position would result in a wrong size being > read, instead fallback to the re-sync mechanic in the else clause. > --- > libavcodec/h2645_parse.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[FFmpeg-devel] [PATCH] avformat/oggenc: fix page duration calculation when granule differs from timestamp

2016-07-07 Thread James Almer
Signed-off-by: James Almer --- libavformat/oggenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c index f998af3..296028e 100644 --- a/libavformat/oggenc.c +++ b/libavformat/oggenc.c @@ -263,10 +263,10 @@ static int ogg_

Re: [FFmpeg-devel] [GSoC] MLP/TrueHD encoder

2016-07-07 Thread Jai Luthra
On Thu, Jul 7, 2016 at 3:19 PM, Carl Eugen Hoyos wrote: > > Not necessarily related: Do you have an idea about how > to fix ticket #5039? > https://trac.ffmpeg.org/ticket/5039 Hi, Looks like either ffmpeg's decoder is messing up when all samples in a packet are same (likely; this might also expla

Re: [FFmpeg-devel] [PATCH] libvpx: Enable vp9 alpha encoding

2016-07-07 Thread Ronald S. Bultje
Hi Vignesh, On Thu, Jul 7, 2016 at 2:44 PM, Vignesh Venkatasubramanian < vigneshv-at-google@ffmpeg.org> wrote: > On Wed, Jul 6, 2016 at 4:50 PM, Vignesh Venkatasubramanian > wrote: > > On Fri, Jul 1, 2016 at 3:03 PM, Ronald S. Bultje > wrote: > >> Hi, > >> > >> On Fri, Jul 1, 2016 at 5:27 P

[FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-07 Thread Vignesh Venkatasubramanian
VPx (VP8/VP9) alpha encoding has been part of FFmpeg. Now, add the ability to decode such files with alpha channel. Signed-off-by: Vignesh Venkatasubramanian --- libavcodec/libvpxdec.c | 111 --- tests/fate/vpx.mak | 3 + tests/ref/fate/vp8

Re: [FFmpeg-devel] [PATCH] libvpx: Enable vp9 alpha encoding

2016-07-07 Thread Vignesh Venkatasubramanian
On Wed, Jul 6, 2016 at 4:50 PM, Vignesh Venkatasubramanian wrote: > On Fri, Jul 1, 2016 at 3:03 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Fri, Jul 1, 2016 at 5:27 PM, Vignesh Venkatasubramanian < >> vigneshv-at-google@ffmpeg.org> wrote: >> >>> On Fri, Jul 1, 2016 at 12:48 PM, Ronald S. Bult

[FFmpeg-devel] [PATCH 2/2] h2645_parse: don't overread AnnexB NALs within an avc stream

2016-07-07 Thread Hendrik Leppkes
We know the maximum size of an AnnexB NAL, signaling it as the maximum NAL size allows ff_h2645_extract_rbsp to determine the correct size. --- libavcodec/h2645_parse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c index 09f

[FFmpeg-devel] [PATCH 1/2] h2645_parse: only read avc length code at the correct position

2016-07-07 Thread Hendrik Leppkes
Reading it from any other position would result in a wrong size being read, instead fallback to the re-sync mechanic in the else clause. --- libavcodec/h2645_parse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c index 9979b6

Re: [FFmpeg-devel] [PATCH] libavformat: Add FIFO pseudo-muxer

2016-07-07 Thread Nicolas George
Le primidi 11 messidor, an CCXXIV, Jan Sebechlebsky a écrit : > I don't understand this note - the fifo_format option is used (and seems to > work)? My bad, I did not notice the field had a different name. > Unfortunately :( Do you think cmd muxer initialization could be easily > modified in a wa

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Carl Eugen Hoyos
Ronald S. Bultje gmail.com> writes: > - measure. Use START/STOP_TIMER, nothing else, around the > caller with/without -cpuflags 0 and look only at the last > reported cycle count line. If -cpuflags 0 works for the tested case (this wasn't true so far). Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Dominik 'Rathann' Mierzejewski
On Thursday, 07 July 2016 at 14:51, Ronald S. Bultje wrote: [...] > - start with one function. Take a really simple one. Don't do 20 at a time. > Especially if this is your first time writing ppc64 assembly. > - measure speedups on other archs with similar register width. Best > example: measure SS

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: add hwaccel support

2016-07-07 Thread Matthieu Bouron
On Mon, Jun 27, 2016 at 03:14:34PM +0200, Matthieu Bouron wrote: > On Fri, Jun 24, 2016 at 06:18:02PM +0200, Michael Niedermayer wrote: > > On Fri, Jun 24, 2016 at 11:17:41AM +0200, Matthieu Bouron wrote: > > > On Thu, Apr 07, 2016 at 02:51:44PM +0200, Matthieu Bouron wrote: > > > > On Wed, Mar 23,

Re: [FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

2016-07-07 Thread Josh de Kock
Hi, On Thu, Jul 7, 2016, at 10:47 AM, Carl Eugen Hoyos wrote: > Josh de Kock itanimul.li> writes: > > > +ret = openmpt_could_open_propability( > > I don't know anything about openmpt but what was > wrong with the original probe function sent by you? It probed one format, and not very well,

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Ronald S. Bultje
Hi, On Thu, Jul 7, 2016 at 7:38 AM, Michael Niedermayer wrote: > On Thu, Jul 07, 2016 at 07:14:43AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Thu, Jul 7, 2016 at 7:07 AM, Michael Niedermayer > > > wrote: > > > > > On Wed, Jul 06, 2016 at 07:28:27AM -0500, Dan Parrot wrote: > > > > On We

Re: [FFmpeg-devel] core infrastructure badge for FFmpeg

2016-07-07 Thread Ganesh Ajjanagadde
06.07.2016, 09:20, "Jean-Baptiste Kempf" : >   On 06 Jul, Ganesh Ajjanagadde wrote : >>    >  A custom script is not a common build system tool. >> >>    How configure.ac that gets fed into autotools etc is acceptable and >> configure is not is beyond me. >>    As for their meaning, I don't know.

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Michael Niedermayer
On Thu, Jul 07, 2016 at 07:14:43AM -0400, Ronald S. Bultje wrote: > Hi, > > On Thu, Jul 7, 2016 at 7:07 AM, Michael Niedermayer > wrote: > > > On Wed, Jul 06, 2016 at 07:28:27AM -0500, Dan Parrot wrote: > > > On Wed, 2016-07-06 at 09:07 +0200, Hendrik Leppkes wrote: > > [...] > > > > > > > > One

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Ronald S. Bultje
Hi, On Thu, Jul 7, 2016 at 7:07 AM, Michael Niedermayer wrote: > On Wed, Jul 06, 2016 at 07:28:27AM -0500, Dan Parrot wrote: > > On Wed, 2016-07-06 at 09:07 +0200, Hendrik Leppkes wrote: > [...] > > > > > One other thing: why didn't this come up when the earlier patch was > > submitted and appli

Re: [FFmpeg-devel] [PATCH] PPC64: Add versions of functions in libswscale/input.c optimized for POWER8 VSX SIMD.

2016-07-07 Thread Michael Niedermayer
On Wed, Jul 06, 2016 at 07:28:27AM -0500, Dan Parrot wrote: > On Wed, 2016-07-06 at 09:07 +0200, Hendrik Leppkes wrote: [...] > > One other thing: why didn't this come up when the earlier patch was > submitted and applied? community patch review is not a reproduceable process, depending on who h

Re: [FFmpeg-devel] PPC & threats

2016-07-07 Thread Ronald S. Bultje
Hi, On Thu, Jul 7, 2016 at 3:14 AM, Clément Bœsch wrote: > Hi folks, > > I think we should reconsider accepting contributions from Dan Parrot. Just > received that in my mailbox. Uhm, yes, I agree this is not acceptable. That went downhill very quickly... :( Ronald ___

Re: [FFmpeg-devel] [GSoC] MLP/TrueHD encoder

2016-07-07 Thread Carl Eugen Hoyos
Jai Luthra jailuthra.in> writes: > This is an update for the TrueHD encoder gsoc project. This looks great, congratulations! Not necessarily related: Do you have an idea about how to fix ticket #5039? https://trac.ffmpeg.org/ticket/5039 Carl Eugen _

Re: [FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

2016-07-07 Thread Carl Eugen Hoyos
Josh de Kock itanimul.li> writes: > +ret = openmpt_could_open_propability( I don't know anything about openmpt but what was wrong with the original probe function sent by you? Iiuc, this code would slow down probing everything or am I missing something? Thank you, Carl Eugen ___

[FFmpeg-devel] PPC & threats

2016-07-07 Thread Clément Bœsch
Hi folks, I think we should reconsider accepting contributions from Dan Parrot. Just received that in my mailbox. - Forwarded message from Dan Parrot - Date: Wed, 06 Jul 2016 19:11:57 -0500 From: Dan Parrot To: Clément Bœsch Subject: keep your opinions to yourself X-Mailer: Evolution

[FFmpeg-devel] [dan.par...@mail.com: Re: [PATCH] PPC64: The following is probably gratuitous, but after witnessing a couple of months of this type of abrupt interjection ...]

2016-07-07 Thread Clément Bœsch
Apparently this mail is meant for anyone in addition to myself, so I'm forwarding to the mailing-list. - Forwarded message from Dan Parrot - Date: Wed, 06 Jul 2016 19:12:54 -0500 From: Dan Parrot To: Clément Bœsch Subject: Re: [FFmpeg-devel] [PATCH] PPC64: The following is probably gr