[FFmpeg-devel] [PATCH] avformat/async: fix integer conversion warning

2015-09-06 Thread Zhang Rui
--- libavformat/async.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/libavformat/async.c b/libavformat/async.c index 15355fb..003212d 100644 --- a/libavformat/async.c +++ b/libavformat/async.c @@ -50,7 +50,7 @@ typedef struct Context { URLContext

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: remove fourcc values from AVCodecIDs

2015-09-06 Thread Paul B Mahol
Dana 7. 9. 2015. 02:38 osoba "James Almer" napisala je: > > Based on patch by wm4. > > Signed-off-by: James Almer > --- > libavcodec/avcodec.h | 122 +++ > 1 file changed, 64 insertions(+), 58 deletions(-) Whole patchset lgtm, assuming relevant as

Re: [FFmpeg-devel] : compile samba

2015-09-06 Thread 李洁
> hi, I want to add samba into ffmpeg, I use the command as followed. it does not work, i am looking forward for your help ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ff

Re: [FFmpeg-devel] [PATCH 2/2] avformat/http: exponentially delay reconnection to avoid flooding the server

2015-09-06 Thread Zhang Rui
>> Doesn't this have overflow issues? >> More precisely, are you sure this behaves correctly when the int >> reconnect_delay wraps around? > > i intend to add a reconnect_max that limits reconnects and prevents > overflow, just wanted to first make sure noone has objections to > these 2 patches bef

Re: [FFmpeg-devel] [PATCH] avformat: implement query_codec for the image2 muxer.

2015-09-06 Thread Michael Niedermayer
On Sun, Sep 06, 2015 at 11:07:20AM -0400, tab wrote: > Allows avformat_query_codec to be used to check for valid image2 encoders. > Reuses the existing ff_guess_image2_codec ID table. > --- > libavformat/img2.c| 10 +++--- > libavformat/img2.h| 7 +++ > libavformat/img2enc.c | 12

[FFmpeg-devel] [PATCH 2/2] avcodec: remove fourcc values from AVCodecIDs

2015-09-06 Thread James Almer
Based on patch by wm4. Signed-off-by: James Almer --- libavcodec/avcodec.h | 122 +++ 1 file changed, 64 insertions(+), 58 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 32235ab..070db50 100644 --- a/libavcodec/avcodec

[FFmpeg-devel] [PATCH 1/2] avcodec: remove duplicate AVCodecIDs

2015-09-06 Thread James Almer
Signed-off-by: James Almer --- libavcodec/avcodec.h| 60 + libavcodec/codec_desc.c | 115 libavcodec/utils.c | 20 - 3 files changed, 21 insertions(+), 174 deletions(-) diff --git a/libavcodec/avcodec.h b

Re: [FFmpeg-devel] [PATCH 7/8] avfilter/deband: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Paul B Mahol
Dana 6. 9. 2015. 19:48 osoba "Clément Bœsch" napisala je: > > --- > libavfilter/vf_deband.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > LGTM > diff --git a/libavfilter/vf_deband.c b/libavfilter/vf_deband.c > index 5de3db1..f0485dd 100644 > --- a/libavfilter/vf_deband.c > +++ b/lib

Re: [FFmpeg-devel] [PATCH 6/8] avfilter/dynaudnorm: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Paul B Mahol
Dana 6. 9. 2015. 19:48 osoba "Clément Bœsch" napisala je: > > --- > libavfilter/af_dynaudnorm.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Lgtm > > diff --git a/libavfilter/af_dynaudnorm.c b/libavfilter/af_dynaudnorm.c > index 0b80b2a..d71a79a 100644 > --- a/libavfilter/af_dy

Re: [FFmpeg-devel] [PATCH 5/8] swscale: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Michael Niedermayer
On Sun, Sep 06, 2015 at 07:43:51PM +0200, Clément Bœsch wrote: > --- > libswscale/options.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The bravest are surely those who have the clearest v

Re: [FFmpeg-devel] [PATCH 2/2] avformat/http: exponentially delay reconnection to avoid flooding the server

2015-09-06 Thread Michael Niedermayer
On Sun, Sep 06, 2015 at 02:05:05PM -0700, Ganesh Ajjanagadde wrote: > On Sun, Sep 6, 2015 at 1:52 PM, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/http.c |7 ++- > > 1 file changed, 6 insertions(+), 1 deletio

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_TYPE_BOOL

2015-09-06 Thread Michael Niedermayer
On Sun, Sep 06, 2015 at 07:43:47PM +0200, Clément Bœsch wrote: > TODO: bump lavu minor > --- > libavutil/opt.c| 69 > +- > libavutil/opt.h| 1 + > tests/ref/fate/opt | 22 +++-- > 3 files changed, 89 insertions(+), 3 deletio

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Ganesh Ajjanagadde
On Sun, Sep 6, 2015 at 2:03 PM, James Almer wrote: > On 9/6/2015 5:23 PM, Ganesh Ajjanagadde wrote: >> On Sun, Sep 6, 2015 at 1:07 PM, Nicolas George wrote: >>> Le decadi 20 fructidor, an CCXXIII, James Almer a écrit : Is it confirmed that we are not ABI compatible at all with libav? >>> >>>

Re: [FFmpeg-devel] [PATCH 2/2] avformat/http: exponentially delay reconnection to avoid flooding the server

2015-09-06 Thread Ganesh Ajjanagadde
On Sun, Sep 6, 2015 at 1:52 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavformat/http.c |7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/libavformat/http.c b/libavformat/http.c > index aa60aef..126

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread James Almer
On 9/6/2015 5:23 PM, Ganesh Ajjanagadde wrote: > On Sun, Sep 6, 2015 at 1:07 PM, Nicolas George wrote: >> Le decadi 20 fructidor, an CCXXIII, James Almer a écrit : >>> Is it confirmed that we are not ABI compatible at all with libav? >> >> It is confirmed that we are not 100% compatible. Nothing m

[FFmpeg-devel] [PATCH 2/2] avformat/http: exponentially delay reconnection to avoid flooding the server

2015-09-06 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavformat/http.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavformat/http.c b/libavformat/http.c index aa60aef..126c7b3 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -28,6 +28,7

[FFmpeg-devel] [PATCH 1/2] avformat/http: Add reconnect_at_eof and reconnect_streamed options

2015-09-06 Thread Michael Niedermayer
From: Michael Niedermayer They allow reconnecting endless live streams which fail with eof TODO: docs Signed-off-by: Michael Niedermayer --- libavformat/http.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index d

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread James Almer
On 9/6/2015 5:37 PM, Paul B Mahol wrote: > Alternative is to remove fourcc and add gap, fine for me too. That's what the patch does. Removes duplicate entries, fourcc values and adds a gap between libav's latest codecid for each group and the first of our own additions. Will send it later today.

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Paul B Mahol
Dana 6. 9. 2015. 22:03 osoba "James Almer" napisala je: > > On 9/6/2015 5:32 AM, Nicolas George wrote: > > Avoid giving the same SONAME as the fork if ABI compatibility > > is not offered. > > > > Not necessary for libpostproc and libswresample but done for > > consistency. > > > > Not done for li

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Clément Bœsch
On Sun, Sep 06, 2015 at 01:23:10PM -0700, Ganesh Ajjanagadde wrote: > On Sun, Sep 6, 2015 at 1:07 PM, Nicolas George wrote: > > Le decadi 20 fructidor, an CCXXIII, James Almer a écrit : > >> Is it confirmed that we are not ABI compatible at all with libav? > > > > It is confirmed that we are not 1

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Ganesh Ajjanagadde
On Sun, Sep 6, 2015 at 1:07 PM, Nicolas George wrote: > Le decadi 20 fructidor, an CCXXIII, James Almer a écrit : >> Is it confirmed that we are not ABI compatible at all with libav? > > It is confirmed that we are not 100% compatible. Nothing more accurate than > that. > > But that does not matte

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Nicolas George
Le decadi 20 fructidor, an CCXXIII, James Almer a écrit : > Is it confirmed that we are not ABI compatible at all with libav? It is confirmed that we are not 100% compatible. Nothing more accurate than that. But that does not matter, it could be fixed. What matters is what we want. Apparently, pe

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread James Almer
On 9/6/2015 5:32 AM, Nicolas George wrote: > Avoid giving the same SONAME as the fork if ABI compatibility > is not offered. > > Not necessary for libpostproc and libswresample but done for > consistency. > > Not done for libavresample. Is it confirmed that we are not ABI compatible at all with

Re: [FFmpeg-devel] [PATCH] Update md5 muxer docs to indicate that it ignores timestamps.

2015-09-06 Thread Michael Niedermayer
On Sun, Sep 06, 2015 at 12:57:42PM -0400, Ronald S. Bultje wrote: > --- > doc/muxers.texi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) ok Thanks! [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Dictatorship naturally arises out of democracy, and the

Re: [FFmpeg-devel] [PATCH] avfilter/vf_thumbnail: use the name 's' for the pointer to the private context

2015-09-06 Thread Ganesh Ajjanagadde
On Sun, Sep 6, 2015 at 4:52 AM, Clément Bœsch wrote: > On Fri, Sep 04, 2015 at 08:20:53AM -0700, Ganesh Ajjanagadde wrote: >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/vf_thumbnail.c | 58 >> +++--- >> 1 file changed, 29 insertions(+), 29 de

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Nicolas George
Le decadi 20 fructidor, an CCXXIII, Carl Eugen Hoyos a écrit : > When this was suggested to Debian, their answer - > iirc - was that this would not fix a possible > compatibility issue. That may not fix theirs, but that would fix a lot of issues still. Two libraries with different ABI should not

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Nicolas George
Le decadi 20 fructidor, an CCXXIII, Ronald S. Bultje a écrit : > I want you to imagine for a second that a user downloads one of these UIs > that is essentially ffmpeg.exe or avconv.exe with some extras, and tries to > use it, and sees a user-equivalent of the above. Media files don't load, > color

[FFmpeg-devel] [PATCH 8/8] avfilter/interlace: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
--- libavfilter/vf_interlace.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_interlace.c b/libavfilter/vf_interlace.c index a520776..28315c3 100644 --- a/libavfilter/vf_interlace.c +++ b/libavfilter/vf_interlace.c @@ -46,8 +46,8 @@ static const AVOption int

[FFmpeg-devel] [PATCH 6/8] avfilter/dynaudnorm: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
--- libavfilter/af_dynaudnorm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavfilter/af_dynaudnorm.c b/libavfilter/af_dynaudnorm.c index 0b80b2a..d71a79a 100644 --- a/libavfilter/af_dynaudnorm.c +++ b/libavfilter/af_dynaudnorm.c @@ -82,9 +82,9 @@ static const AVOpt

[FFmpeg-devel] [PATCH 7/8] avfilter/deband: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
--- libavfilter/vf_deband.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_deband.c b/libavfilter/vf_deband.c index 5de3db1..f0485dd 100644 --- a/libavfilter/vf_deband.c +++ b/libavfilter/vf_deband.c @@ -58,7 +58,7 @@ static const AVOption deband_options[] = {

[FFmpeg-devel] [PATCH 5/8] swscale: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
--- libswscale/options.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libswscale/options.c b/libswscale/options.c index 18a6594..ac35af7 100644 --- a/libswscale/options.c +++ b/libswscale/options.c @@ -75,9 +75,7 @@ static const AVOption swscale_options[] = { { "ed",

[FFmpeg-devel] [PATCH 3/8] avdevice/dshow: use AV_OPT_TYPE_BOOL (UNTESTED)

2015-09-06 Thread Clément Bœsch
--- libavdevice/dshow.c | 32 1 file changed, 8 insertions(+), 24 deletions(-) diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c index 0c2eca0..31764a9 100644 --- a/libavdevice/dshow.c +++ b/libavdevice/dshow.c @@ -1268,36 +1268,20 @@ static const AVOption op

[FFmpeg-devel] [PATCH 4/8] avfilter/kerndeint: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
--- libavfilter/vf_kerndeint.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_kerndeint.c b/libavfilter/vf_kerndeint.c index 0e2417a..70ec2bf 100644 --- a/libavfilter/vf_kerndeint.c +++ b/libavfilter/vf_kerndeint.c @@ -52,8 +52,8 @@ static const AVOption ker

[FFmpeg-devel] [PATCH 2/8] avcodec/aacenc: use AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
--- libavcodec/aacenc.c | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/libavcodec/aacenc.c b/libavcodec/aacenc.c index 2b2d039..bb90048 100644 --- a/libavcodec/aacenc.c +++ b/libavcodec/aacenc.c @@ -885,18 +885,10 @@ static const AVOption aacenc_options[] = {

[FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_TYPE_BOOL

2015-09-06 Thread Clément Bœsch
TODO: bump lavu minor --- libavutil/opt.c| 69 +- libavutil/opt.h| 1 + tests/ref/fate/opt | 22 +++-- 3 files changed, 89 insertions(+), 3 deletions(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index 4030fa8..9912d9e 1

[FFmpeg-devel] Add AV_OPT_TYPE_BOOL and use it in various places

2015-09-06 Thread Clément Bœsch
$subj (many other places to add obviously) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] Update md5 muxer docs to indicate that it ignores timestamps.

2015-09-06 Thread Ronald S. Bultje
--- doc/muxers.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index ccd222c..86ca4ad 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -549,7 +549,7 @@ MD5 testing format. This muxer computes and prints the MD5 hash of all the input

[FFmpeg-devel] [PATCH] avfilter/vf_super2xsai: use the name 's' for the pointer to the private context

2015-09-06 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavfilter/vf_super2xsai.c | 54 ++--- 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/libavfilter/vf_super2xsai.c b/libavfilter/vf_super2xsai.c index 67e9f97..cbb3f62 100644 --- a/libavfilter/vf_super2

Re: [FFmpeg-devel] Opinons on adding "tickets fixed" to the Changelog

2015-09-06 Thread Clément Bœsch
On Sun, Sep 06, 2015 at 08:27:08AM -0700, Ganesh Ajjanagadde wrote: > Hi all, > > I noticed that glibc lists all the tickets/bug reports fixed in the > Changelog, and was wondering whether it would be beneficial if ffmpeg > did the same for its releases. > > The benefit I see is that this is a "u

[FFmpeg-devel] Opinons on adding "tickets fixed" to the Changelog

2015-09-06 Thread Ganesh Ajjanagadde
Hi all, I noticed that glibc lists all the tickets/bug reports fixed in the Changelog, and was wondering whether it would be beneficial if ffmpeg did the same for its releases. The benefit I see is that this is a "user-facing" change and thus ideally should be documented in the relase notes. The

Re: [FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Carl Eugen Hoyos
Nicolas George nsup.org> writes: > Avoid giving the same SONAME as the fork if ABI > compatibility is not offered. (I am neither a supporter of this patch nor do I in any way object.) When this was suggested to Debian, their answer - iirc - was that this would not fix a possible compatibili

Re: [FFmpeg-devel] Gathering opinions from FFmpeg developers about a reunification with Libav developers

2015-09-06 Thread Ganesh Ajjanagadde
On Sun, Sep 6, 2015 at 2:27 AM, Nicolas George wrote: > Le quartidi 14 thermidor, an CCXXIII, Clement Boesch a écrit : >> Since Michael decided to step down as a leader, the question of >> reunification with Libav came up once more naturally. Before negotiations >> start again, I think it's import

[FFmpeg-devel] [PATCH] avformat: implement query_codec for the image2 muxer.

2015-09-06 Thread tab
Allows avformat_query_codec to be used to check for valid image2 encoders. Reuses the existing ff_guess_image2_codec ID table. --- libavformat/img2.c| 10 +++--- libavformat/img2.h| 7 +++ libavformat/img2enc.c | 12 3 files changed, 22 insertions(+), 7 deletions(-)

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Carl Eugen Hoyos
Ronald S. Bultje gmail.com> writes: > On Sun, Sep 6, 2015 at 8:46 AM, Carl Eugen Hoyos wrote: > > > Ronald S. Bultje gmail.com> writes: > > > > > Are there users that care about this feature? > > > > > Why didn't they complain that it didn't work? > > > > Why do you think it doesn't work? > >

Re: [FFmpeg-devel] [PATCH] lavc/srtdec: handle leading whitespace before tag names

2015-09-06 Thread Clément Bœsch
On Wed, Jul 15, 2015 at 12:27:04PM -0500, Rodger Combs wrote: > This fixes cases like ``. > --- > libavcodec/srtdec.c | 16 ++-- > 1 file changed, 10 insertions(+), 6 deletions(-) > rebased, slightly modified, and pushed. thx & sorry for the delay -- Clément B. pgpSkg6koda9t.pgp

Re: [FFmpeg-devel] [PATCH] avfilter/sine: support expression in the number of output samples

2015-09-06 Thread Clément Bœsch
On Sat, Aug 15, 2015 at 07:45:56PM +0200, Nicolas George wrote: > L'octidi 28 thermidor, an CCXXIII, Clement Boesch a écrit : > > The expression is evaluated once per frame, so very low overhead. > > > > But for the record, I tried the following: > > >0.10% ffmpeg_g ffmpeg_g [.] av_expr_ev

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Ronald S. Bultje
Hi, On Sun, Sep 6, 2015 at 8:46 AM, Carl Eugen Hoyos wrote: > Ronald S. Bultje gmail.com> writes: > > > Are there users that care about this feature? > > > Why didn't they complain that it didn't work? > > Why do you think it doesn't work? "(Just for reference, I tested running avconv's FATE

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Carl Eugen Hoyos
Ronald S. Bultje gmail.com> writes: > Are there users that care about this feature? > Why didn't they complain that it didn't work? Why do you think it doesn't work? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/

[FFmpeg-devel] [PATCH] vp9: fix integer overflows in sse2 version of iadst4.

2015-09-06 Thread Ronald S. Bultje
--- libavcodec/x86/vp9itxfm.asm | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavcodec/x86/vp9itxfm.asm b/libavcodec/x86/vp9itxfm.asm index e1cb637..02d4d0a 100644 --- a/libavcodec/x86/vp9itxfm.asm +++ b/libavcodec/x86/vp9itxfm.asm @@ -66,7 +66,8 @@ pw_1

Re: [FFmpeg-devel] [PATCH] avfilter/vf_thumbnail: use the name 's' for the pointer to the private context

2015-09-06 Thread Clément Bœsch
On Fri, Sep 04, 2015 at 08:20:53AM -0700, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/vf_thumbnail.c | 58 > +++--- > 1 file changed, 29 insertions(+), 29 deletions(-) > Yeah sure ok -- Clément B. pgp08nD2mgwKs.p

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/flac: remove avpriv parsing API cruft

2015-09-06 Thread Hendrik Leppkes
On Sun, Sep 6, 2015 at 1:37 PM, Paul B Mahol wrote: > Dana 6. 9. 2015. 11:10 osoba "Hendrik Leppkes" > napisala je: >> >> --- >> libavcodec/flac.c | 15 --- >> libavcodec/flac.h | 8 >> 2 files changed, 23 deletions(-) >> > > I guess its ok if libav removed it too or doesnt

Re: [FFmpeg-devel] [PATCH] avcodec/alac: remove dead code cruft

2015-09-06 Thread Paul B Mahol
Dana 6. 9. 2015. 12:04 osoba "Hendrik Leppkes" napisala je: > > The output is always planar since two major bumps, remove all code > related to packed output. > --- > libavcodec/alac.c | 40 +++- > 1 file changed, 3 insertions(+), 37 deletions(-) Should be ok.

Re: [FFmpeg-devel] [PATCH 3/4] avformat/samidec:do not include trailing sami

2015-09-06 Thread Clément Bœsch
On Sun, Aug 30, 2015 at 07:24:49AM -0700, Yayoi wrote: > footer in the last packet > --- > libavformat/samidec.c | 8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) > Fixed commit message and applied -- Clément B. pgpbm0YBMiP88.pgp Description: PGP signature __

Re: [FFmpeg-devel] [PATCH 4/4] fate/subtitles: add a new test for SAMI demuxer

2015-09-06 Thread Clément Bœsch
On Sun, Aug 30, 2015 at 07:24:50AM -0700, Yayoi wrote: > and decoder > --- > tests/fate/subtitles.mak | 3 ++ > tests/ref/fate/sub-sami2 | 91 > > 2 files changed, 94 insertions(+) > create mode 100644 tests/ref/fate/sub-sami2 > Fixed commit me

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/samidec: use ff_htmlmarkup_to_ass()

2015-09-06 Thread Clément Bœsch
On Sun, Aug 30, 2015 at 07:24:48AM -0700, Yayoi wrote: > --- > libavcodec/Makefile | 2 +- > libavcodec/samidec.c| 45 - > tests/ref/fate/sub-sami | 18 +- > 3 files changed, 38 insertions(+), 27 deletions(-) > dropped a lot mo

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/srtdec: factor out HTML parsing code

2015-09-06 Thread Clément Bœsch
On Sun, Aug 30, 2015 at 07:24:47AM -0700, Yayoi wrote: [...] > +int len, tag_close, sptr = 1, line_start = 1, an = 0, end = 0, count; removed this "count" variable you forgot to drop, and applied. -- Clément B. pgpV7RXIbMMBa.pgp Description: PGP signature __

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/vorbis: remove avpriv vorbis parser cruft

2015-09-06 Thread Paul B Mahol
Dana 6. 9. 2015. 11:10 osoba "Hendrik Leppkes" napisala je: > > It was replaced by a public API > --- > libavcodec/vorbis_parser.c | 21 --- > libavcodec/vorbis_parser_internal.h | 40 - > 2 files changed, 61 deletions(-) Probably ok.

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/flac: remove avpriv parsing API cruft

2015-09-06 Thread Paul B Mahol
Dana 6. 9. 2015. 11:10 osoba "Hendrik Leppkes" napisala je: > > --- > libavcodec/flac.c | 15 --- > libavcodec/flac.h | 8 > 2 files changed, 23 deletions(-) > I guess its ok if libav removed it too or doesnt have it. > diff --git a/libavcodec/flac.c b/libavcodec/flac.c >

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Ronald S. Bultje
Hi Nicolas, On Sun, Sep 6, 2015 at 3:55 AM, Nicolas George wrote: > Le nonidi 19 fructidor, an CCXXIII, Ronald S. Bultje a écrit : > > Ah, ok, so we did test it, and it didn't work. That sounds like an > amazing > > reason to remove it. Do we really want people to think corresponds to > > "FFmpe

Re: [FFmpeg-devel] [PATCH] avfilter/af_asyncts: use llabs for int64_t

2015-09-06 Thread Michael Niedermayer
On Sat, Sep 05, 2015 at 08:42:02PM -0700, Ganesh Ajjanagadde wrote: > long may not be 64 bit on all platforms; so labs on int64_t is unsafe. > This fixes a warning reported in: > http://fate.ffmpeg.org/log.cgi?time=20150905071512&log=compile&slot=i386-darwin-clang-polly-3.7 > > Signed-off-by: Gane

[FFmpeg-devel] [PATCH] avcodec/alac: remove dead code cruft

2015-09-06 Thread Hendrik Leppkes
The output is always planar since two major bumps, remove all code related to packed output. --- libavcodec/alac.c | 40 +++- 1 file changed, 3 insertions(+), 37 deletions(-) diff --git a/libavcodec/alac.c b/libavcodec/alac.c index 13607b3..827c0db 100644 --- a

Re: [FFmpeg-devel] Gathering opinions from FFmpeg developers about a reunification with Libav developers

2015-09-06 Thread Nicolas George
Le quartidi 14 thermidor, an CCXXIII, Clement Boesch a écrit : > Since Michael decided to step down as a leader, the question of > reunification with Libav came up once more naturally. Before negotiations > start again, I think it's important if we can all individually start > thinking about what a

Re: [FFmpeg-devel] [PATCH] avformat/async: replace strerror with av_err2str

2015-09-06 Thread Michael Niedermayer
On Sun, Sep 06, 2015 at 12:51:54PM +0800, Zhang Rui wrote: > Fixes CID1322337 > --- > libavformat/async.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) applied btw, if you want git write access (to maintain async* and or any other work) then send me your public ssh key th

[FFmpeg-devel] [PATCH 2/3] avcodec/flac: remove avpriv parsing API cruft

2015-09-06 Thread Hendrik Leppkes
--- libavcodec/flac.c | 15 --- libavcodec/flac.h | 8 2 files changed, 23 deletions(-) diff --git a/libavcodec/flac.c b/libavcodec/flac.c index 5ff004e..f5154b9 100644 --- a/libavcodec/flac.c +++ b/libavcodec/flac.c @@ -235,18 +235,3 @@ void ff_flac_parse_streaminfo(AVCodec

[FFmpeg-devel] [PATCH 3/3] avcodec/vorbis: remove avpriv vorbis parser cruft

2015-09-06 Thread Hendrik Leppkes
It was replaced by a public API --- libavcodec/vorbis_parser.c | 21 --- libavcodec/vorbis_parser_internal.h | 40 - 2 files changed, 61 deletions(-) diff --git a/libavcodec/vorbis_parser.c b/libavcodec/vorbis_parser.c index 8fa6d99..0b

[FFmpeg-devel] [PATCH 1/3] avcodec/alac: remove dead code cruft

2015-09-06 Thread Hendrik Leppkes
--- libavcodec/alac.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/alac.c b/libavcodec/alac.c index 13607b3..016741b 100644 --- a/libavcodec/alac.c +++ b/libavcodec/alac.c @@ -593,7 +593,6 @@ static int alac_set_info(ALACContext *alac) static av_cold int al

[FFmpeg-devel] [PATCH] all: bump all libraries versions by 100.

2015-09-06 Thread Nicolas George
Avoid giving the same SONAME as the fork if ABI compatibility is not offered. Not necessary for libpostproc and libswresample but done for consistency. Not done for libavresample. Signed-off-by: Nicolas George --- libavcodec/flac.c | 2 +- libavcodec/flac.h

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Nicolas George
Le decadi 20 fructidor, an CCXXIII, Clement Boesch a écrit : > To me, losing the ABI compatibility should not be an issue, because so far > it seems no downstream trusted it. To my understanding, it was introduced > as a way to simplify distribution work into integrating both FFmpeg and > Libav. Th

Re: [FFmpeg-devel] [PATCH 3/3] lavc: remove Libav ABI compat hacks

2015-09-06 Thread Nicolas George
Le nonidi 19 fructidor, an CCXXIII, Ronald S. Bultje a écrit : > Ah, ok, so we did test it, and it didn't work. That sounds like an amazing > reason to remove it. Do we really want people to think corresponds to > "FFmpeg quality"? I would be embarrassed if people pointed this out to me. > > When