Re: [FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-21 Thread James Almer
On 20/10/14 5:03 PM, Lukasz Marek wrote: > On 20.10.2014 14:55, Michael Niedermayer wrote: >> On Mon, Oct 20, 2014 at 12:14:14PM +0200, Lukasz Marek wrote: >>> On 20 October 2014 08:47, Reimar Döffinger wrote: >>> On 19.10.2014, at 13:39, Lukasz Marek wrote: > Signed-off-by: Lukasz

Re: [FFmpeg-devel] [PATCH] configure: add build tree's directories to rpath

2014-10-21 Thread Carl Eugen Hoyos
Lukasz Marek gmail.com> writes: > if you run > configure --enable-rpath --disable-static --enable-shared > make > make install > then installed binaries of the fftolls will have more > paths defined to look libraries for. This sounds like a very bad idea to me > but The goal is to allow to run

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

2014-10-21 Thread Carl Eugen Hoyos
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 index c70ddf3..bc77623 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@

Re: [FFmpeg-devel] Help with writing an RMS-LMS decoder

2014-10-21 Thread Thilo Borgmann
Hi, > I was of the impression that even though the time for submitting patches is > running out, any propective OPW applicant could still continue working on > her tasks even after the application period is over right up to the > selections are announced for the OPW? I think this is correct. And

Re: [FFmpeg-devel] Help with writing an RMS-LMS decoder

2014-10-21 Thread Thilo Borgmann
Am 20.10.14 um 13:30 schrieb Ngassa Finjap: > Thilo, Please can we have an IRC chat about this. I can still implement the > RLS-LMS decoder I'm usually not in IRC - I cannot stay there during my every day job and I can't plan my free time as much as I want to. So mail is my preferred channel of c

[FFmpeg-devel] [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata reads.

2014-10-21 Thread Thilo Borgmann
Hi, the last commit to lavf/mov.c might leave the metadata buffer allocated in certain error conditions. This patch fixes it. -Thilo From a2fabe107796cf0af5cd18310fad178c5650b168 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Tue, 21 Oct 2014 10:10:55 +0200 Subject: [PATCH] lavf/mov.c: Prev

[FFmpeg-devel] [PATCH] lavfi/afade: fix cur_sample computation

2014-10-21 Thread Stefano Sabatini
Use the correct timebase conversion. --- libavfilter/af_afade.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_afade.c b/libavfilter/af_afade.c index fbf9802..35b2a84 100644 --- a/libavfilter/af_afade.c +++ b/libavfilter/af_afade.c @@ -221,6 +221,8 @@ static

[FFmpeg-devel] [PATCH] lavfi/concat: accept a single segment

2014-10-21 Thread Stefano Sabatini
Simplify scripting operations, so that n=1 is not to be considered as a special case. --- libavfilter/avf_concat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/avf_concat.c b/libavfilter/avf_concat.c index 18f373c..fa9e6b9 100644 --- a/libavfilter/avf_concat.c ++

Re: [FFmpeg-devel] [PATCH] configure: add build tree's directories to rpath

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 8:21 AM, Carl Eugen Hoyos wrote: > This sounds like a very bad idea to me I agree. It's not nice to have remnants of the build process in the installed binaries. If you need to run them during development, use LD_LIBRARY_PATH or or build statically. - Derek

Re: [FFmpeg-devel] [PATCH] lavfi/afade: fix cur_sample computation

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 10:47 AM, Stefano Sabatini wrote: > -int64_t cur_sample = av_rescale_q(buf->pts, (AVRational){1, > outlink->sample_rate}, outlink->time_base); > +int64_t cur_sample = av_rescale_q(buf->pts, inlink->time_base, > (AVRational){1, inlink->sample_rate}); Probably OK. - Derek __

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata reads.

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 9:12 AM, Thilo Borgmann wrote: > Subject: [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata > reads. > > --- > libavformat/mov.c | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) LGTM. - Derek ___

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

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 8:32 AM, Carl Eugen Hoyos wrote: > +The filter only works for strictly constant frame rate input. If your input > +has variable frame rate, try the @ref{pullup} filter. Perhaps the filter should output a warning as well? - Derek ___ ffmpeg

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: don't use avpriv_ prefix for internal functions

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 7:56 AM, James Almer wrote: > No need to keep the old symbols around until a major bump since lavd > functions > with the avpriv_ prefix were never exposed. lol. > Signed-off-by: James Almer > --- > "avpriv_*;" can be added to libavdevice.v after this patch, but personally > I'd

Re: [FFmpeg-devel] [PATCH 1/4] ffserver: move configuration code to separate file

2014-10-21 Thread Stefano Sabatini
On date Monday 2014-10-20 23:56:59 +0200, Lukasz Marek encoded: > This commit doesn't change any existing logic. > It moves ffserver configuration related code to separate file. > It intends to make maintaining easier. I'm not opposed, but please tell why you think it's better this way. I still ha

Re: [FFmpeg-devel] [PATCH] lavfi/concat: accept a single segment

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 10:47 AM, Stefano Sabatini wrote: > - AV_OPT_TYPE_INT, { .i64 = 2 }, 2, INT_MAX, V|A|F}, > + AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, V|A|F}, I'm not sure I follow how this 'allows' n=1? Doesn't this just change the default? - Derek __

Re: [FFmpeg-devel] [PATCH 2/4] ffserver_config: clean up variable initialization

2014-10-21 Thread Stefano Sabatini
On date Monday 2014-10-20 23:57:00 +0200, Lukasz Marek encoded: > Signed-off-by: Lukasz Marek > --- > ffserver_config.c | 16 +--- > 1 file changed, 5 insertions(+), 11 deletions(-) > > diff --git a/ffserver_config.c b/ffserver_config.c > index 927b6fe..351c4e9 100644 > --- a/ffserve

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_conf: factorize parse function per config tag

2014-10-21 Thread Stefano Sabatini
On date Monday 2014-10-20 23:57:01 +0200, Lukasz Marek encoded: > Signed-off-by: Lukasz Marek > --- > ffserver.c|6 + > ffserver_config.c | 1138 > ++--- > ffserver_config.h |7 + > 3 files changed, 566 insertions(+), 585 deletions(

Re: [FFmpeg-devel] [PATCH 0/4] FFserver configuration

2014-10-21 Thread Stefano Sabatini
On date Monday 2014-10-20 23:56:58 +0200, Lukasz Marek encoded: > Some time ago I wanted to setup internet camera streaming via ffserver > and I faced the same probalem as #1275. > It was over a year ago, so I don't remember correctly, but a conclusion > from debugging was that codec's private data

Re: [FFmpeg-devel] [PATCH] lavfi/concat: accept a single segment

2014-10-21 Thread Stefano Sabatini
On date Tuesday 2014-10-21 11:08:47 +0100, Derek Buitenhuis encoded: > On 10/21/2014 10:47 AM, Stefano Sabatini wrote: > > - AV_OPT_TYPE_INT, { .i64 = 2 }, 2, INT_MAX, V|A|F}, > > + AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, V|A|F}, > > I'm not sure I follow how this 'allows' n=1? Doesn'

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: don't use avpriv_ prefix for internal functions

2014-10-21 Thread Stefano Sabatini
On date Tuesday 2014-10-21 03:56:06 -0300, James Almer encoded: > No need to keep the old symbols around until a major bump since lavd > functions > with the avpriv_ prefix were never exposed. > > Signed-off-by: James Almer LGTM. [...] -- FFmpeg = Fast and Faithless Multimedia Political Eage

Re: [FFmpeg-devel] [PATCH] lavfi/concat: accept a single segment

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 11:22 AM, Stefano Sabatini wrote: > - AV_OPT_TYPE_INT, { .i64 = 2 }, 2, INT_MAX, V|A|F}, > + AV_OPT_TYPE_INT, { .i64 = 2 }, 1, INT_MAX, V|A|F}, OK. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/m

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata reads.

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 11:04:28AM +0100, Derek Buitenhuis wrote: > On 10/21/2014 9:12 AM, Thilo Borgmann wrote: > > Subject: [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata > > reads. > > > > --- > > libavformat/mov.c | 14 -- > > 1 file changed, 8 insertions(+),

Re: [FFmpeg-devel] [PATCH] tools/crypto_bench: add CAST5 support

2014-10-21 Thread Michael Niedermayer
On Sat, Oct 18, 2014 at 06:10:39PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > tools/crypto_bench.c | 45 + > 1 file changed, 45 insertions(+) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB C

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

2014-10-21 Thread Stefano Sabatini
On date Monday 2014-10-20 23:57:02 +0200, Lukasz Marek encoded: > So far AVCodecContext was created without codec specified. > This causes internal data to not be initialized to defaults. > > This commit postpone context creation until all information are gathered. > > Partially fixes #1275 > ---

[FFmpeg-devel] Finding residuals in h264 decoding in ffmpeg

2014-10-21 Thread Norbert Schmitt
Hello, I am trying to understand how the H.264 Decoder in ffmpeg works and saving data (Macroblock types and motion vectors for example) from the decoding of a H.264 video. I also want to save the residuals for each frame but I cant seem to find them. There is something I am missing. It g

Re: [FFmpeg-devel] [PATCH 1/4] ffserver: move configuration code to separate file

2014-10-21 Thread Lukasz Marek
On 21 October 2014 12:08, Stefano Sabatini wrote: > On date Monday 2014-10-20 23:56:59 +0200, Lukasz Marek encoded: > > This commit doesn't change any existing logic. > > It moves ffserver configuration related code to separate file. > > It intends to make maintaining easier. > > I'm not opposed,

Re: [FFmpeg-devel] [PATCH] lavfi/concat: accept a single segment

2014-10-21 Thread Stefano Sabatini
On date Tuesday 2014-10-21 11:33:54 +0100, Derek Buitenhuis encoded: > On 10/21/2014 11:22 AM, Stefano Sabatini wrote: > > - AV_OPT_TYPE_INT, { .i64 = 2 }, 2, INT_MAX, V|A|F}, > > + AV_OPT_TYPE_INT, { .i64 = 2 }, 1, INT_MAX, V|A|F}, > > OK. Applied. -- FFmpeg = Fierce and Foolish Moron

Re: [FFmpeg-devel] [PATCH] lavfi/afade: fix cur_sample computation

2014-10-21 Thread Stefano Sabatini
On date Tuesday 2014-10-21 11:03:23 +0100, Derek Buitenhuis encoded: > On 10/21/2014 10:47 AM, Stefano Sabatini wrote: > > -int64_t cur_sample = av_rescale_q(buf->pts, (AVRational){1, > > outlink->sample_rate}, outlink->time_base); > > +int64_t cur_sample = av_rescale_q(buf->pts, inlink->t

Re: [FFmpeg-devel] [PATCH] configure: add build tree's directories to rpath

2014-10-21 Thread Lukasz Marek
On 21 October 2014 12:01, Derek Buitenhuis wrote: > On 10/21/2014 8:21 AM, Carl Eugen Hoyos wrote: > > This sounds like a very bad idea to me > > I agree. It's not nice to have remnants of the build process in the > installed binaries. > > If you need to run them during development, use LD_LIBRAR

Re: [FFmpeg-devel] PATCH: Make -copyts a tri-state

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 01:23:55AM -0500, Rodger Combs wrote: > > > On Oct 20, 2014, at 20:42, Michael Niedermayer wrote: > > > > On Mon, Oct 20, 2014 at 06:50:12PM -0500, Rodger Combs wrote: > >> > >>> On Oct 17, 2014, at 03:19, Rodger Combs wrote: > >>> > > On Oct 15, 2014, at 06

[FFmpeg-devel] [PATCH] mov.c: reasonable bitrate for fragmented mp4

2014-10-21 Thread Mika Raento
If using MFRA for timestamps, the stream may start from a large offset and/or have gaps. With this change we calculate the bitrate based on frames we've seen. --- libavformat/mov.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 80549ec.

Re: [FFmpeg-devel] [PATCH 1/4] ffserver: move configuration code to separate file

2014-10-21 Thread Stefano Sabatini
On date Tuesday 2014-10-21 13:18:01 +0200, Lukasz Marek encoded: > On 21 October 2014 12:08, Stefano Sabatini wrote: > > > On date Monday 2014-10-20 23:56:59 +0200, Lukasz Marek encoded: > > > This commit doesn't change any existing logic. > > > It moves ffserver configuration related code to sep

[FFmpeg-devel] [RFC] How to deal with libavfilter buffer overflows?

2014-10-21 Thread Stefano Sabatini
$ ffmpeg -f lavfi -i "testsrc=d=10, split[t0][t1]; [t0]select='gte(t,5)',setpts=PTS-STARTPTS[s0]; [t1] select='lt(t,5)', setpts=PTS-STARTPTS[s1]; [s0][s1] concat" -y outtest.mp4 [...] Input #0, lavfi, from 'testsrc=d=10, split[t0][t1]; [t0]select='gte(t,5)',setpts=PTS-STARTPTS[s0]; [t1] select='

Re: [FFmpeg-devel] Facebook page

2014-10-21 Thread Thomas Volkert
>> Please, vote for a Facebook url here: >> http://goo.gl/forms/AHVFvPxp8X >> >> This will show us the community opinion about this. > Does this mean that just "ffmpeg" is used by someone else? > Yes, "ffmpeg" is already used by someone else. Best regards, Thomas. _

Re: [FFmpeg-devel] Facebook page

2014-10-21 Thread Thomas Volkert
Am 15.10.2014 10:49, schrieb Thomas Volkert: > On 10/05/2014 09:09 PM, compn wrote: >> On Sun, 05 Oct 2014 20:19:49 +0200 >> Thomas Volkert wrote: >> >>> http://www.facebook.com/ffmpeg.devel, >>> http://www.facebook.com/ffmpeg.framework, >>> http://www.facebook.com/multimedia.framework, ... >>> @a

Re: [FFmpeg-devel] Facebook page

2014-10-21 Thread James Almer
On 15/10/14 5:44 AM, Thomas Volkert wrote: > On 10/05/2014 11:19 PM, Anshul wrote: >> I think we should wait for day or 2 for Fan reply. >>> ffmpeg-devel mailing list >>> ffmpeg-devel@ffmpeg.org >>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> > > There was no reply within the last days. >

[FFmpeg-devel] [PATCH]Addedd Kate demuxer and decoder

2014-10-21 Thread Hima Dangi
diff --git a/libavformat/Makefile b/libavformat/Makefile index 86064ea..4d41283 100644 --- a/libavformat/Makefile +++ b/libavformat/Makefile @@ -433,6 +433,7 @@ OBJS-$(CONFIG_VOBSUB_DEMUXER)+= subtitles.o # mpeg demuxer is in the OBJS-$(CONFIG_VOC_DEMUXER) += vocdec.o voc

Re: [FFmpeg-devel] Facebook page

2014-10-21 Thread Thomas Volkert
Am 21.10.2014 18:12, schrieb James Almer: > ffmpeg is a registered trademark. I'm sure Facebook can solve this is asked. > There was also no response from Facebook within the last days. Best regards, Thomas. ___ ffmpeg-devel mailing list ffmpeg-devel@ff

Re: [FFmpeg-devel] [PATCH] mov.c: reasonable bitrate for fragmented mp4

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 03:04:10PM +0300, Mika Raento wrote: > If using MFRA for timestamps, the stream may start from a large offset > and/or have gaps. With this change we calculate the bitrate based on > frames we've seen. > --- > libavformat/mov.c | 11 +++ > 1 file changed, 11 inserti

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_conf: factorize parse function per config tag

2014-10-21 Thread Reimar Döffinger
On 21.10.2014, at 12:15, Stefano Sabatini wrote: > On date Monday 2014-10-20 23:57:01 +0200, Lukasz Marek encoded: >> Signed-off-by: Lukasz Marek >> --- >> ffserver.c|6 + >> ffserver_config.c | 1138 >> ++--- >> ffserver_config.h |7

Re: [FFmpeg-devel] [PATCH] libutvideo: remove libutvideo wrapper

2014-10-21 Thread Lou Logan
On Fri, 17 Oct 2014 05:02:55 +0200, Michael Niedermayer wrote: > this removes the possibility to easily test our decoder against > libutvideo. Ok. Just thought I'd throw it out there/generate some discussion. Anyway, patch dropped. ___ ffmpeg-devel ma

Re: [FFmpeg-devel] Facebook page

2014-10-21 Thread Lou Logan
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 a week. I'd rather have "ffmpeg" or not

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

2014-10-21 Thread Lukasz Marek
On 19.10.2014 16:27, wm4 wrote: On Sat, 18 Oct 2014 20:36:22 +0200 Lukasz Marek wrote: --- libavdevice/alsa-audio-common.c | 60 + libavdevice/alsa-audio-dec.c| 6 + libavdevice/alsa-audio-enc.c| 6 + libavdevice/alsa-audio.h

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_conf: factorize parse function per config tag

2014-10-21 Thread Lukasz Marek
On 21.10.2014 20:25, Reimar Döffinger wrote: On 21.10.2014, at 12:15, Stefano Sabatini wrote: On date Monday 2014-10-20 23:57:01 +0200, Lukasz Marek encoded: Signed-off-by: Lukasz Marek --- ffserver.c|6 + ffserver_config.c | 1138 ++-

Re: [FFmpeg-devel] [PATCH 1/2] lavd/fbdev_common: report error during probing fbdev device

2014-10-21 Thread Lukasz Marek
On 19.10.2014 00:28, Michael Niedermayer wrote: On Sat, Oct 18, 2014 at 08:25:29PM +0200, Lukasz Marek wrote: Signed-off-by: Lukasz Marek --- libavdevice/fbdev_common.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) LGTM Pushed __

Re: [FFmpeg-devel] [PATCH] lavd/fbdev_dec: use default device when not provided

2014-10-21 Thread Lukasz Marek
On 19.10.2014 00:27, Michael Niedermayer wrote: On Sat, Oct 18, 2014 at 07:38:16PM +0200, Lukasz Marek wrote: Signed-off-by: Lukasz Marek --- libavdevice/fbdev_dec.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) LGTM Pushed. ___

Re: [FFmpeg-devel] [PATCH] Added STL demuxer and decoder

2014-10-21 Thread Clément Bœsch
On Tue, Oct 21, 2014 at 02:17:51AM +0530, Eejya Singh wrote: > Updated sample test after rebasing. Please find the patch attached. > I did the little tweak below: [...] > From 798bc500b2df2c90d4fe42b8418059ab1b51fb8a Mon Sep 17 00:00:00 2001 > From: Eejya Singh > Date: Tue, 21 Oct 2014 01:55:39

Re: [FFmpeg-devel] [PATCH 2/2] opts: add list device sources/sinks options

2014-10-21 Thread Lukasz Marek
On 19.10.2014 00:20, Michael Niedermayer wrote: On Sat, Oct 18, 2014 at 08:25:30PM +0200, Lukasz Marek wrote: Allows to list sources/sinks of the devices that implement that functionality. Signed-off-by: Lukasz Marek [...] +int show_sinks(void *optctx, const char *opt, const char *arg) +{ +#

Re: [FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-21 Thread Lukasz Marek
On 21.10.2014 08:59, James Almer wrote: On 20/10/14 5:03 PM, Lukasz Marek wrote: On 20.10.2014 14:55, Michael Niedermayer wrote: On Mon, Oct 20, 2014 at 12:14:14PM +0200, Lukasz Marek wrote: On 20 October 2014 08:47, Reimar Döffinger wrote: On 19.10.2014, at 13:39, Lukasz Marek wrote: Si

Re: [FFmpeg-devel] [PATCH]Addedd Kate demuxer and decoder

2014-10-21 Thread Clément Bœsch
On Tue, Oct 21, 2014 at 10:02:35PM +0530, Hima Dangi wrote: > diff --git a/libavformat/Makefile b/libavformat/Makefile > index 86064ea..4d41283 100644 > --- a/libavformat/Makefile > +++ b/libavformat/Makefile A git format-patch is prefered because it includes authorship, commit message, commit des

Re: [FFmpeg-devel] [PATCH]Addedd Kate demuxer and decoder

2014-10-21 Thread Carl Eugen Hoyos
Hima Dangi gmail.com> writes: > --- /dev/null > +++ b/libavformat/katedec.c > -0,0 +1,158 > + > + > +/** > + * file > + * Kate subtitles format demuxer > + */ Please add a license header, copy it from another file. Carl Eugen ___ ffmpeg-

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

2014-10-21 Thread Carl Eugen Hoyos
Derek Buitenhuis gmail.com> writes: > On 10/21/2014 8:32 AM, Carl Eugen Hoyos wrote: > > +The filter only works for strictly constant frame rate input. I removed "strictly" locally. > > If your input > > +has variable frame rate, try the ref{pullup} filter. > > Perhaps the filter should ou

Re: [FFmpeg-devel] [PATCH 2/2] opts: add list device sources/sinks options

2014-10-21 Thread Michael Niedermayer
On Sat, Oct 18, 2014 at 08:25:30PM +0200, Lukasz Marek wrote: > Allows to list sources/sinks of the devices that implement > that functionality. > > Signed-off-by: Lukasz Marek > --- > cmdutils.c | 191 > + > cmdutils.h |

Re: [FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-21 Thread James Almer
On 21/10/14 6:05 PM, Lukasz Marek wrote: > On 21.10.2014 08:59, James Almer wrote: >> On 20/10/14 5:03 PM, Lukasz Marek wrote: >>> On 20.10.2014 14:55, Michael Niedermayer wrote: On Mon, Oct 20, 2014 at 12:14:14PM +0200, Lukasz Marek wrote: > On 20 October 2014 08:47, Reimar Döffinger >>

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Carl Eugen Hoyos
Rodger Combs gmail.com> writes: > This fixes https://trac.ffmpeg.org/ticket/3934 but I'm > not sure if there was a good reason for this to be > here to begin with. Perhaps a protection against > infinite recursion (though I believe EBML_MAX_DEPTH > serves that purpose to some degree)? Is all

Re: [FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 11:05:40PM +0200, Lukasz Marek wrote: > On 21.10.2014 08:59, James Almer wrote: > >On 20/10/14 5:03 PM, Lukasz Marek wrote: > >>On 20.10.2014 14:55, Michael Niedermayer wrote: > >>>On Mon, Oct 20, 2014 at 12:14:14PM +0200, Lukasz Marek wrote: > On 20 October 2014 08:47,

Re: [FFmpeg-devel] [PATCH] tools/crypto_bench: add CAST5 support

2014-10-21 Thread James Almer
On 21/10/14 8:02 AM, Michael Niedermayer wrote: > On Sat, Oct 18, 2014 at 06:10:39PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> tools/crypto_bench.c | 45 + >> 1 file changed, 45 insertions(+) > > LGTM Pushed, thanks. _

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

2014-10-21 Thread rongyan
Michael, Do you have any suggestion on this patch? Thanks. Rong Yan -- The world has enough for everyone's need, but not enough for everyone's greed. -- Original -- From: "michaelni";; Date: Fri, Oct 17, 2014 11:26 PM To: "FFmpeg developm

[FFmpeg-devel] [PATCH] avformat/matroskadec: Fix cluster parsing loop which gathers seek information

2014-10-21 Thread Michael Niedermayer
Fixes Ticket2263 Fixes Ticket3934 Signed-off-by: Michael Niedermayer --- libavformat/matroskadec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index b742319..2ef3222 100644 --- a/libavformat/matroskadec.c +++ b/

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 01:34:16AM -0500, Rodger Combs wrote: > > > On Oct 19, 2014, at 23:34, Rodger Combs wrote: > > > > > >> On Oct 17, 2014, at 17:40, Michael Niedermayer >> > wrote: > >> > >> On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: >

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

2014-10-21 Thread Michael Niedermayer
On Wed, Oct 22, 2014 at 09:23:43AM +0800, rongyan wrote: > Michael, > Do you have any suggestion on this patch? yes, please remove the code duplication (this also applies to already commited code) instead of #if X code a b1(abc) code c #else code a b2(abc) code c #endif you can write #i

[FFmpeg-devel] [PATCH] avfilter/vf_idet: Fixes issue with idet not flushing last frame.

2014-10-21 Thread Neil Birkbeck
Uses a similar approach as vf_yadif to flush the last frame in idet. Quick test with 50 frames from vsynth1: ./ffmpeg.old -i fate-suite/ffmpeg-synthetic/vsynth1/%02d.pgm -vf idet -f mp4 -y /dev/null 2>&1 | grep Multi (gives) [Parsed_idet_0 @ 0x261ebb0] Multi frame detection: TFF:0 BFF:0 Progre