Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-05-04 Thread Michael Niedermayer
On Wed, May 04, 2016 at 08:40:04AM +0300, Andrey Utkin wrote: > On Wed, May 04, 2016 at 05:07:10AM +0200, Michael Niedermayer wrote: > > doesnt this require an update to teh docs ? > > Indeed, thanks for commenting. > > But I wonder how should I format reference to "aspect" correctly and pretty.

Re: [FFmpeg-devel] [PATCH] fate: Add test for broken SPS (Ticket 2580)

2016-05-04 Thread Michael Niedermayer
On Tue, May 03, 2016 at 04:38:12PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate/h264.mak|2 + > tests/ref/fate/h264-brokensps-2580 | 84 > > 2 files changed, 86 insertions(+) > create mode 10

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Derek Buitenhuis
On 5/5/2016 1:26 AM, Michael Niedermayer wrote: > It was possible to write and register external protocols, codecs, > (de)muxers,... in the distant past. Was there even a such thing as a public API at the time? ;) > i always liked to allow and support "Plugins" / externally registering > stuff, b

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Michael Niedermayer
On Thu, May 05, 2016 at 12:29:11AM +0100, Derek Buitenhuis wrote: > On 5/4/2016 11:58 PM, Thierry Foucu wrote: > >> Can i ask why it cannot be part of the public API? > > See answer below. > > >> It was really practical just to register new prefix, specially, when you > >> have to create X number

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Derek Buitenhuis
On 5/4/2016 11:58 PM, Thierry Foucu wrote: >> Can i ask why it cannot be part of the public API? See answer below. >> It was really practical just to register new prefix, specially, when you >> have to create X number of them. It's not harder to do via AVIO callbacks in actual code using the API

Re: [FFmpeg-devel] [PATCH] vc2enc: do not print lavc version when the bitexact flag is enabled

2016-05-04 Thread Rostislav Pehlivanov
On 4 May 2016 at 03:56, Michael Niedermayer wrote: > On Wed, May 04, 2016 at 01:39:21AM +0100, Rostislav Pehlivanov wrote: > > Signed-off-by: Rostislav Pehlivanov > > --- > > libavcodec/vc2enc.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/vc2enc.c

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Hendrik Leppkes
On Thu, May 5, 2016 at 12:46 AM, Thierry Foucu wrote: > On Wed, May 4, 2016 at 1:58 PM, Derek Buitenhuis > wrote: > >> On 5/4/2016 9:52 PM, wm4 wrote: >> > This was never allowed and was never public API. Use custom I/O instead >> > (creating an avio context with your own read/write callbacks). >>

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Thierry Foucu
On Wed, May 4, 2016 at 3:46 PM, Thierry Foucu wrote: > > > On Wed, May 4, 2016 at 1:58 PM, Derek Buitenhuis < > derek.buitenh...@gmail.com> wrote: > >> On 5/4/2016 9:52 PM, wm4 wrote: >> > This was never allowed and was never public API. Use custom I/O instead >> > (creating an avio context with

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Thierry Foucu
On Wed, May 4, 2016 at 1:58 PM, Derek Buitenhuis wrote: > On 5/4/2016 9:52 PM, wm4 wrote: > > This was never allowed and was never public API. Use custom I/O instead > > (creating an avio context with your own read/write callbacks). > > Can i ask why it cannot be part of the public API? It was re

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/dtsdec: detect core-less streams

2016-05-04 Thread James Almer
On 5/3/2016 10:06 AM, foo86 wrote: > --- > > Updated detection algorithm. > > libavformat/dtsdec.c | 47 --- > 1 file changed, 40 insertions(+), 7 deletions(-) FATE passes, some dts samples i tried still work, and the lbr samples are now properly reco

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Derek Buitenhuis
On 5/4/2016 9:52 PM, wm4 wrote: > This was never allowed and was never public API. Use custom I/O instead > (creating an avio context with your own read/write callbacks). Yes, I was thinking something akin to: int my_open(...) { if (filename_has_prefix("myownproto://")) { ctx->opaque

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread wm4
On Wed, 4 May 2016 13:08:14 -0700 Thierry Foucu wrote: > Hi Team > > On Mon, Feb 29, 2016 at 8:51 AM, Derek Buitenhuis wrote: > > > ffmpeg | branch: master | Derek Buitenhuis | > > Mon Feb 29 16:50:39 2016 +| [9c75148e6ebc88a0501e3d0242defb6dbdc3c23d] > > | committer: Derek Buitenhuis > >

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Derek Buitenhuis
On 5/4/2016 9:08 PM, Thierry Foucu wrote: > I noticed that this commit remove the function ffurl_register_protocol > which allows people to register their own protocol when using libavformat. This is intended. It was never, ever a public API on purpose, and it was never supported or allowed. > Bu

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Thierry Foucu
Hi Team On Mon, Feb 29, 2016 at 8:51 AM, Derek Buitenhuis wrote: > ffmpeg | branch: master | Derek Buitenhuis | > Mon Feb 29 16:50:39 2016 +| [9c75148e6ebc88a0501e3d0242defb6dbdc3c23d] > | committer: Derek Buitenhuis > > Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d' > > This commi

Re: [FFmpeg-devel] [PATCH v3 00/12] videotoolbox encoder features and fixes

2016-05-04 Thread wm4
On Tue, 3 May 2016 11:17:54 -0400 Richard Kern wrote: > ping > All applied, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

2016-05-04 Thread Felt, Patrick
On 5/4/16, 9:12 AM, "ffmpeg-devel on behalf of Hendrik Leppkes" wrote: >On Wed, May 4, 2016 at 5:01 PM, Felt, Patrick >wrote: >>> >>>[...] +/* libavfilter documentation says that filter init will be called only once. ffmpeg calls the init twice to enable it to validate + * th

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

2016-05-04 Thread Hendrik Leppkes
On Wed, May 4, 2016 at 5:01 PM, Felt, Patrick wrote: >> >>[...] >>> +/* libavfilter documentation says that filter init will be called only >>> once. ffmpeg calls the init twice to enable it to validate >>> + * the complex filtering has the right input and output pads. this allows >>> us to byp

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

2016-05-04 Thread Felt, Patrick
On 5/4/16, 5:52 AM, "ffmpeg-devel on behalf of Clément Bœsch" wrote: >On Wed, May 04, 2016 at 12:15:26AM +, Felt, Patrick wrote: >> Afternoon all, >> I apologize if this isn’t right way to submit a patch. > >You're supposed to git commit and use git format-patch. Mind throwing some p

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-04 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 1:53 PM, Ronald S. Bultje wrote: > --- > doc/filters.texi | 13 > libavfilter/colorspacedsp.c | 12 > libavfilter/colorspacedsp.h | 6 ++ > libavfilter/colorspacedsp_template.c | 128 > +++

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

2016-05-04 Thread Clément Bœsch
On Wed, May 04, 2016 at 12:15:26AM +, Felt, Patrick wrote: > Afternoon all, > I apologize if this isn’t right way to submit a patch. You're supposed to git commit and use git format-patch. [...] > +/* libavfilter documentation says that filter init will be called only once. > ffmpeg calls

Re: [FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-04 Thread Hendrik Leppkes
On Wed, May 4, 2016 at 12:44 PM, wm4 wrote: > On Tue, 3 May 2016 15:21:00 -0400 > "Ronald S. Bultje" wrote: > >> I have no idea why the first hunk uses ost->enc_ctx, because as far as >> I understand, that is never used in case of -c:v copy, but this code >> block is only entered if encoding_nee

Re: [FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-04 Thread wm4
On Tue, 3 May 2016 15:21:00 -0400 "Ronald S. Bultje" wrote: > I have no idea why the first hunk uses ost->enc_ctx, because as far as > I understand, that is never used in case of -c:v copy, but this code > block is only entered if encoding_needed=0, which means stream_copy=1. > My point being: r

[FFmpeg-devel] A Hackathon for Codec Develovers

2016-05-04 Thread Shruti Vishnoi
Hi, We at HackerEarth are pleased to inform that Acuiti +Seequestor is conducting a Hackathon

Re: [FFmpeg-devel] [PATCH 1/2] vc2enc: prevent random data

2016-05-04 Thread Christophe Gisquet
Hi, 2016-05-04 3:06 GMT+02:00 Rostislav Pehlivanov : > vc2hqencode is not the reference encoder, vc2-reference is. It's even worse > though. Sorry, I thought authoritative could mean "from the authors", so I didn't mean it as "the" reference/"the authority". Just a good reference in case the spec