[FFmpeg-devel] [PATCH] avformat/rdt: Forward whitelists to rdt demuxer

2014-10-23 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/rdt.c | 21 +++-- 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/libavformat/rdt.c b/libavformat/rdt.c index e8dc8f5..fd99bb1 100644 --- a/libavformat/rdt.c +++ b/libavformat/rdt.c @@ -523,18 +523,26 @@ static Pa

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: postpone codec context creation

2014-10-23 Thread Reynaldo H. Verdejo Pinochet
Hi Lukasz On 10/22/2014 06:44 PM, Lukasz Marek wrote: > [..] > To save your time: > 1. I updated ffserver_apply_stream_config function, > 2. added comments in FFServerConfig struct in header (according to > Stefano's remark) > 3. in ffserver_parse_config_stream, at the end added: > av_free

Re: [FFmpeg-devel] [PATCH 1/2] rtsp: pass return code from ffurl_open() on its failure

2014-10-23 Thread Michael Niedermayer
On Fri, Oct 24, 2014 at 12:39:21AM +0400, Andrey Utkin wrote: > On Thu, Oct 23, 2014 at 10:23 PM, Michael Niedermayer > wrote: > > why doesnt this write into err directly without ret ? > > ffurl_open() returns >= 0 in case of success. > Positive value of err can be confusing and can lead to unex

Re: [FFmpeg-devel] [PATCH]Print an error if both -r and -vsync 0 are used

2014-10-23 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > +if (frame_rate && video_sync_method == VSYNC_PASSTHROUGH) > +av_log(NULL, AV_LOG_ERROR, "Using -vsync 0 and -r > produces invalid output files\n"); Merged by Michael. Thank you, Carl Eugen ___ ffmpeg

[FFmpeg-devel] [PATCH] avformat: add AVFMT_FULLREAD for standalone subtitles formats

2014-10-23 Thread Clément Bœsch
--- Requested by wm4, who is concerned about opening a thousands of subtitles files at once. I'm a bit concerned about the fact that the avio is not closed until later. --- doc/APIchanges | 5 + libavformat/aqtitledec.c | 1 + libavformat/assdec.c | 1 + libavformat/

Re: [FFmpeg-devel] [PATCH 1/2] rtsp: pass return code from ffurl_open() on its failure

2014-10-23 Thread Andrey Utkin
On Thu, Oct 23, 2014 at 10:23 PM, Michael Niedermayer wrote: > why doesnt this write into err directly without ret ? ffurl_open() returns >= 0 in case of success. Positive value of err can be confusing and can lead to unexpected behaviour, also at future changes (the modified function is quite la

Re: [FFmpeg-devel] [PATCH 2/2] http: pass return code from http_open_cnx_internal() on its failure

2014-10-23 Thread Michael Niedermayer
On Thu, Oct 23, 2014 at 08:55:46PM +0400, Andrey Utkin wrote: > Previously, AVERROR(EIO) was returned on failure of > http_open_cnx_internal(). Now the value is passed to upper level, thus > it is possible to distinguish ECONNREFUSED, ETIMEDOUT, ENETUNREACH etc. > --- > libavformat/http.c | 2 ++ >

Re: [FFmpeg-devel] [PATCH 1/2] rtsp: pass return code from ffurl_open() on its failure

2014-10-23 Thread Michael Niedermayer
On Thu, Oct 23, 2014 at 08:55:45PM +0400, Andrey Utkin wrote: > Previously, AVERROR(EIO) was returned. Now the value is passed from > lower level, thus it is possible to distinguish ECONNREFUSED, ETIMEDOUT, > ENETUNREACH etc. > --- > libavformat/rtsp.c | 7 --- > 1 file changed, 4 insertions(+

[FFmpeg-devel] [PATCH 1/2] rtsp: pass return code from ffurl_open() on its failure

2014-10-23 Thread Andrey Utkin
Previously, AVERROR(EIO) was returned. Now the value is passed from lower level, thus it is possible to distinguish ECONNREFUSED, ETIMEDOUT, ENETUNREACH etc. --- libavformat/rtsp.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c in

[FFmpeg-devel] [PATCH 2/2] http: pass return code from http_open_cnx_internal() on its failure

2014-10-23 Thread Andrey Utkin
Previously, AVERROR(EIO) was returned on failure of http_open_cnx_internal(). Now the value is passed to upper level, thus it is possible to distinguish ECONNREFUSED, ETIMEDOUT, ENETUNREACH etc. --- libavformat/http.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/http.c b/libav

Re: [FFmpeg-devel] [PATCH] x86/hpeldsp: fix loop in {avg, avg_no_rnd}_pixels16_x2_mmx

2014-10-23 Thread James Almer
On 23/10/14 8:25 AM, Michael Niedermayer wrote: > On Thu, Oct 23, 2014 at 12:50:33AM -0300, James Almer wrote: >> Handle it inside the __asm__() block. >> Fixes fate-vc1_ilaced_twomv when using the gcc-usan toolchain. >> >> Signed-off-by: James Almer >> --- >> The entire hpeldsp_rnd_template.c fil

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

2014-10-23 Thread Dave Rice
On Oct 23, 2014, at 4:05 AM, Clément Bœsch wrote: > On Tue, Oct 21, 2014 at 09:32:39AM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> It appears to me that we all know that fieldmatch needs cfr input, >> but it isn't mentioned in the documentation. >> Related to ticket #3968. >> >> Please comment,

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

2014-10-23 Thread Carl Eugen Hoyos
On Thursday 23 October 2014 10:05:07 am Clément Bœsch wrote: > > +The filter only works for strictly constant frame rate input. If your > > input +has variable frame rate, try the @ref{pullup} filter. > > + > > Well... isn't telecined content supposed to be CFR anyway? Patch updated. Thank you, C

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread wm4
On Thu, 23 Oct 2014 17:12:40 +0200 Michael Niedermayer wrote: > On Thu, Oct 23, 2014 at 04:18:29PM +0200, wm4 wrote: > > On Thu, 23 Oct 2014 16:12:52 +0200 > > Nicolas George wrote: > > > > > Le duodi 2 brumaire, an CCXXIII, wm4 a écrit : > > > > I kind of disagree. > > > > > > I kind of have

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread Nicolas George
Le duodi 2 brumaire, an CCXXIII, Michael Niedermayer a écrit : > I dont understand either side or the proposed API nearly well enough > to really argue about this but iam concerned that this fight could > lead to a deadlock again and noone implementing any API, which would > be a loss to all. For

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread Michael Niedermayer
On Thu, Oct 23, 2014 at 04:18:29PM +0200, wm4 wrote: > On Thu, 23 Oct 2014 16:12:52 +0200 > Nicolas George wrote: > > > Le duodi 2 brumaire, an CCXXIII, wm4 a écrit : > > > I kind of disagree. > > > > I kind of have gathered that. > > > > > With your approach, all data gets (pot

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread wm4
On Thu, 23 Oct 2014 16:12:52 +0200 Nicolas George wrote: > Le duodi 2 brumaire, an CCXXIII, wm4 a écrit : > > I kind of disagree. > > I kind of have gathered that. > > > With your approach, all data gets (potentially) > > trashed on opening, and it's hard to change the encodin

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread Nicolas George
Le duodi 2 brumaire, an CCXXIII, wm4 a écrit : > I kind of disagree. I kind of have gathered that. > With your approach, all data gets (potentially) > trashed on opening, and it's hard to change the encoding afterwards. > You'd have to reopen the demuxer. And then you'd have t

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread wm4
On Thu, 23 Oct 2014 13:56:54 +0200 Nicolas George wrote: > Le duodi 2 brumaire, an CCXXIII, Rodger Combs a écrit : > > As mentioned in https://trac.ffmpeg.org/ticket/4054#comment:1 > > Let me quote for completeness: > > 11rcombs: > >>> Sometimes, especially when ffmpeg is being called programma

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2014-10-23 Thread wm4
On Wed, 22 Oct 2014 23:45:42 +0200 Nicolas George wrote: > > [ CCing Anton, as most that is written here also apply to libav too, and > this would be a good occasion to try a cross-fork cooperation; if that is > not wanted, please let us know so we can drop the cc. ] > > 1. Problems with the cu

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2014-10-23 Thread Clément Bœsch
On Wed, Oct 22, 2014 at 11:45:42PM +0200, Nicolas George wrote: > > [ CCing Anton, as most that is written here also apply to libav too, and > this would be a good occasion to try a cross-fork cooperation; if that is > not wanted, please let us know so we can drop the cc. ] > > 1. Problems with t

[FFmpeg-devel] [PATCH] ffprobe.xsd: Add packets_and_frames element definition

2014-10-23 Thread Tobias Rapp
Attached patch allows validation of ffprobe XML output if -show_packets and -show_frames are both set. From 0c10cf28993f61cbb8365caf8d06f5d5b6111a80 Mon Sep 17 00:00:00 2001 From: Tobias Rapp Date: Thu, 23 Oct 2014 15:10:28 +0200 Subject: [PATCH] ffprobe.xsd: Add packets_and_frames element defini

Re: [FFmpeg-devel] [PATCH] lavd/alsa: implement get_device_list callbacks

2014-10-23 Thread Nicolas George
Le primidi 1er brumaire, an CCXXIII, Lukasz Marek a écrit : > It gives the same list as aplay -L (upper case). For example (ffmpeg will > display it in different way, but the content is the same) I thought for a time that "dmix:CARD=PCH,DEV=0" had placeholders: PCH needed to be replaced by an actu

Re: [FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread Nicolas George
Le duodi 2 brumaire, an CCXXIII, Rodger Combs a écrit : > As mentioned in https://trac.ffmpeg.org/ticket/4054#comment:1 Let me quote for completeness: 11rcombs: >>> Sometimes, especially when ffmpeg is being called programmatically, it is >>> difficult or impossible for the caller (or user) to kn

[FFmpeg-devel] Discussion: Feature: Subtitle charenc detection

2014-10-23 Thread Rodger Combs
As mentioned in https://trac.ffmpeg.org/ticket/4054#comment:1 , it'd probably be useful to discuss this concept here. There are a lot of nuances to this, it'll require linking at least one (and possibly 3 or more) new dependencies, and it'll probab

Re: [FFmpeg-devel] [PATCH] x86/hpeldsp: fix loop in {avg, avg_no_rnd}_pixels16_x2_mmx

2014-10-23 Thread Michael Niedermayer
On Thu, Oct 23, 2014 at 12:50:33AM -0300, James Almer wrote: > Handle it inside the __asm__() block. > Fixes fate-vc1_ilaced_twomv when using the gcc-usan toolchain. > > Signed-off-by: James Almer > --- > The entire hpeldsp_rnd_template.c file should be ported to yasm, and/or this > function opt

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

2014-10-23 Thread Clément Bœsch
On Tue, Oct 21, 2014 at 09:32:39AM +0200, Carl Eugen Hoyos wrote: > Hi! > > It appears to me that we all know that fieldmatch needs cfr input, > but it isn't mentioned in the documentation. > Related to ticket #3968. > > Please comment, Carl Eugen > diff --git a/doc/filters.texi b/doc/filters.t

Re: [FFmpeg-devel] Facebook page

2014-10-23 Thread tim nicholson
On 21/10/14 20:21, Lou Logan wrote: > On Tue, 21 Oct 2014 19:30:39 +0200, Thomas Volkert wrote: > >> There was also no response from Facebook within the last days. > > The behemoth is probably slow to respond. Let's give them some more > time then perhaps prod them again after not hearing from th