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

2014-10-21 Thread Rodger Combs
On Oct 20, 2014, at 20:42, Michael Niedermayer michae...@gmx.at wrote: On Mon, Oct 20, 2014 at 06:50:12PM -0500, Rodger Combs wrote: On Oct 17, 2014, at 03:19, Rodger Combs rodger.co...@gmail.com wrote: On Oct 15, 2014, at 06:00, Rodger Combs rodger.co...@gmail.com

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

2014-10-21 Thread Rodger Combs
On Oct 19, 2014, at 23:34, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 17:40, Michael Niedermayer michae...@gmx.at mailto:michae...@gmx.at wrote: On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger

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

2014-10-21 Thread James Almer
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 jamr...@gmail.com --- avpriv_*; can be added to libavdevice.v after this patch, but personally I'd say to wait until it's needed rather than

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 of

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

[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 thilo.borgm...@mail.de Date: Tue, 21 Oct 2014 10:10:55 +0200 Subject:

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

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 jamr...@gmail.com --- avpriv_*; can be added to libavdevice.v after this patch, but

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 hate

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 lukasz.m.lu...@gmail.com --- ffserver_config.c | 16 +--- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/ffserver_config.c b/ffserver_config.c index 927b6fe..351c4e9

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 lukasz.m.lu...@gmail.com --- ffserver.c|6 + ffserver_config.c | 1138 ++--- ffserver_config.h |7 + 3 files changed, 566

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't this

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 jamr...@gmail.com LGTM. [...] -- FFmpeg = Fast and Faithless Multimedia

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

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

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 jamr...@gmail.com --- tools/crypto_bench.c | 45 + 1 file changed, 45 insertions(+) LGTM [...] -- Michael GnuPG fingerprint:

[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

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 stefa...@gmail.com 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

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 Moronic

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 derek.buitenh...@gmail.com 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

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 michae...@gmx.at wrote: On Mon, Oct 20, 2014 at 06:50:12PM -0500, Rodger Combs wrote: On Oct 17, 2014, at 03:19, Rodger Combs rodger.co...@gmail.com wrote: On Oct 15,

[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

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 stefa...@gmail.com 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

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

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 si...@gmx.net wrote: http://www.facebook.com/ffmpeg.devel, http://www.facebook.com/ffmpeg.framework, http://www.facebook.com/multimedia.framework, ... @all:

[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

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

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

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 stefa...@gmail.com wrote: On date Monday 2014-10-20 23:57:01 +0200, Lukasz Marek encoded: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver.c|6 + ffserver_config.c | 1138

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

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

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 lukasz.m.lu...@gmail.com wrote: --- libavdevice/alsa-audio-common.c | 60 + libavdevice/alsa-audio-dec.c| 6 + libavdevice/alsa-audio-enc.c| 6 +

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 lukasz.m.lu...@gmail.com --- 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 lukasz.m.lu...@gmail.com --- 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 singh.ee...@gmail.com Date: Tue, 21

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 lukasz.m.lu...@gmail.com [...] +int show_sinks(void *optctx, const char

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 reimar.doeffin...@gmx.de wrote: On 19.10.2014, at 13:39,

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

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

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

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

2014-10-21 Thread Carl Eugen Hoyos
Derek Buitenhuis derek.buitenhuis at 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 at ref{pullup} filter. Perhaps the filter

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 lukasz.m.lu...@gmail.com --- cmdutils.c | 191 +

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 rodger.combs at 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

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

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 jamr...@gmail.com --- 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;michae...@gmx.at; Date: Fri, Oct 17, 2014 11:26 PM To:

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

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 rodger.co...@gmail.com wrote: On Oct 17, 2014, at 17:40, Michael Niedermayer michae...@gmx.at mailto:michae...@gmx.at wrote: On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael

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 #if