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 them for

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.texi

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 jamr...@gmail.com --- The entire hpeldsp_rnd_template.c file should be ported to yasm, and/or this

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

2014-10-23 Thread Rodger Combs
As mentioned in https://trac.ffmpeg.org/ticket/4054#comment:1 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

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 know the

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 actual

[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 t.r...@noa-audio.com Date: Thu, 23 Oct 2014 15:10:28 +0200 Subject: [PATCH] ffprobe.xsd: Add

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 the

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 geo...@nsup.org 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

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

2014-10-23 Thread wm4
On Thu, 23 Oct 2014 13:56:54 +0200 Nicolas George geo...@nsup.org 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

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 to

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

2014-10-23 Thread wm4
On Thu, 23 Oct 2014 16:12:52 +0200 Nicolas George geo...@nsup.org 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

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 wm4
On Thu, 23 Oct 2014 17:12:40 +0200 Michael Niedermayer michae...@gmx.at wrote: On Thu, Oct 23, 2014 at 04:18:29PM +0200, wm4 wrote: On Thu, 23 Oct 2014 16:12:52 +0200 Nicolas George geo...@nsup.org wrote: Le duodi 2 brumaire, an CCXXIII, wm4 a écrit : I kind of disagree. I

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, Carl

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 u...@pkh.me 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,

[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

[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

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(+), 3

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 ++ 1

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 michae...@gmx.at 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

[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 +

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 cehoyos at 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 ___

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 michae...@gmx.at 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

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:

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

2014-10-23 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- 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