[FFmpeg-devel] [PATCH] movtextenc: Use default style information from ASS style

2015-08-22 Thread Niklesh Lalwani
From: Niklesh Generate the default style for movtext from the ASS style. Earlier, we used a fixed default style. Signed-off-by: Niklesh --- libavcodec/movtextenc.c | 110 1 file changed, 82 insertions(+), 28 deletions(-) diff --git a/libavcode

Re: [FFmpeg-devel] [PATCHv2] avdevice/lavfi: fix self assignment warning

2015-08-22 Thread Ganesh Ajjanagadde
On Sat, Aug 22, 2015 at 11:24 PM, Ganesh Ajjanagadde wrote: > FAIL(ret) expands to statements including a silly ret=ret. > This triggers a -Wself-assign on confirmed clang 3.6, and so we fix it. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavdevice/lavfi.c | 2 +- > 1 file changed, 1 inserti

[FFmpeg-devel] [PATCHv2] avdevice/lavfi: fix self assignment warning

2015-08-22 Thread Ganesh Ajjanagadde
FAIL(ret) expands to statements including a silly ret=ret. This triggers a -Wself-assign on confirmed clang 3.6, and so we fix it. Signed-off-by: Ganesh Ajjanagadde --- libavdevice/lavfi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi

Re: [FFmpeg-devel] [FFmpeg-cvslog] disable deprecation warnings in deprecated code

2015-08-22 Thread James Almer
> ffmpeg | branch: master | Andreas Cadhalpun googlemail.com> | Sat Aug 8 10:41:33 2015 +0200| > [095347ffe4c73143dbeb7b05cde8891fd1289389] | committer: Andreas Cadhalpun > > disable deprecation warnings in deprecated code > > Reviewed-by: wm4 > Signed-off-by: Andreas Cadhalpun > > > http://gi

[FFmpeg-devel] [PATCH] avdevice/lavfi: fix self assignment warning

2015-08-22 Thread Ganesh Ajjanagadde
FAIL(ret) expands to statements including a silly ret=ret. This triggers a -Wself-assign on clang 5+, and so we fix it. Signed-off-by: Ganesh Ajjanagadde --- libavdevice/lavfi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c index d9

Re: [FFmpeg-devel] [PATCH 2/3] avformat/hls: comment out unused function

2015-08-22 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 7:27 PM, Ganesh Ajjanagadde wrote: > Fixes -Wunused-function from > http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1 > > Signed-off-by: Ganesh Ajjanagadde > --- > libavformat/hls.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deleti

Re: [FFmpeg-devel] [PATCH] all: silence clang -Wabsolute-value for unsigned subtractions

2015-08-22 Thread Ganesh Ajjanagadde
On Sat, Aug 22, 2015 at 10:26 AM, Nicolas George wrote: > Le quintidi 5 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : >> +/* cast for -Wabsolute-value in clang 3.5+ */ >> +return abs((int) (t1 - t2)) + abs((int) ((c1 & 0x00ff) - (c2 & >> 0x00ff))) + >> +abs((int) (((

Re: [FFmpeg-devel] [PATCH] avformat: Remove use of AVFrac and AVStream->pts

2015-08-22 Thread Michael Niedermayer
On Sat, Aug 22, 2015 at 06:06:59PM +0200, Andreas Cadhalpun wrote: > On 17.08.2015 16:45, Michael Niedermayer wrote: > > On Mon, Aug 17, 2015 at 04:36:52PM +0200, Nicolas George wrote: > >> Le decadi 30 thermidor, an CCXXIII, Michael Niedermayer a écrit : > >>> From: Michael Niedermayer > >>> > >>

Re: [FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Michael Niedermayer
On Sat, Aug 22, 2015 at 09:21:09PM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Aug 22, 2015 at 8:21 PM, Michael Niedermayer > wrote: > > > On Sat, Aug 22, 2015 at 07:12:13PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Sat, Aug 22, 2015 at 6:54 PM, Michael Niedermayer > > > > >

Re: [FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Ronald S. Bultje
Hi, On Sat, Aug 22, 2015 at 8:21 PM, Michael Niedermayer wrote: > On Sat, Aug 22, 2015 at 07:12:13PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, Aug 22, 2015 at 6:54 PM, Michael Niedermayer > > > wrote: > > > > > On Sat, Aug 22, 2015 at 07:57:33PM +0200, Andreas Cadhalpun wrote: > >

Re: [FFmpeg-devel] [PATCH 2/2] libx265: Enable 12-bit encoding

2015-08-22 Thread James Almer
On 22/08/15 10:03 PM, Stephen Hutchinson wrote: >> I don't really think checking for 60 is enough to get working 12bit >> non-experimental encoding. 68 will probably be a safer bet. >> And regardless of which we end up checking for, this change will break >> support for libx265 1.7, which is not a

Re: [FFmpeg-devel] [PATCH 2/2] libx265: Enable 12-bit encoding

2015-08-22 Thread Stephen Hutchinson
On 8/22/2015 3:26 PM, James Almer wrote: The warnings were removed from the master branch of libx265 three days ago, and X265_BUILD in that branch is 71. libx265 1.8 will be tagged from the stable branch in the coming days, which is not up to date with master (X265_BUILD is 68) and of course mean

Re: [FFmpeg-devel] [PATCH] avcodec/x86/mpegaudiodsp: correct asm guards

2015-08-22 Thread Michael Niedermayer
On Fri, Aug 21, 2015 at 01:43:43PM -0400, Ganesh Ajjanagadde wrote: > Fixes -Wunused-function warnings when compiling with --disable-yasm on x86. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/x86/mpegaudiodsp.c | 2 ++ > 1 file changed, 2 insertions(+) applied thanks [...] -- Mich

Re: [FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Michael Niedermayer
On Sat, Aug 22, 2015 at 07:12:13PM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Aug 22, 2015 at 6:54 PM, Michael Niedermayer > wrote: > > > On Sat, Aug 22, 2015 at 07:57:33PM +0200, Andreas Cadhalpun wrote: > > > This fixes fate with FF_API_LAVF_BITEXACT disabled. > > > > > > Signed-off-by:

Re: [FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Ronald S. Bultje
Hi, On Sat, Aug 22, 2015 at 6:54 PM, Michael Niedermayer wrote: > On Sat, Aug 22, 2015 at 07:57:33PM +0200, Andreas Cadhalpun wrote: > > This fixes fate with FF_API_LAVF_BITEXACT disabled. > > > > Signed-off-by: Andreas Cadhalpun > > --- > > tests/Makefile | 2 +- > > tests/fate-run

Re: [FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Michael Niedermayer
On Sat, Aug 22, 2015 at 07:57:33PM +0200, Andreas Cadhalpun wrote: > This fixes fate with FF_API_LAVF_BITEXACT disabled. > > Signed-off-by: Andreas Cadhalpun > --- > tests/Makefile | 2 +- > tests/fate-run.sh| 18 +- > tests/fate/mpeg4.mak | 2 +- > tests/

Re: [FFmpeg-devel] Make libavfilter/vf_drawtext.c build under uclibc environments

2015-08-22 Thread Nicolas George
Le quintidi 5 fructidor, an CCXXIII, Timothy Gu a écrit : > What would you suggest? I had thought of: int double_to_intXX_t(double v, intXX_t *r); Returning an AVERROR code and converting the result reliably. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH 1/4] avcodec: remove FF_API_LOWRES around max_lowres

2015-08-22 Thread Michael Niedermayer
On Sat, Aug 22, 2015 at 07:56:53PM +0200, Andreas Cadhalpun wrote: > This fixes building with FF_API_LOWRES disabled. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/avcodec.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 4

Re: [FFmpeg-devel] Make libavfilter/vf_drawtext.c build under uclibc environments

2015-08-22 Thread Timothy Gu
On Sat, Aug 22, 2015 at 1:18 PM Paul B Mahol wrote: > On 8/22/15, Rene Rheaume wrote: > > In function func_eval_expr_int_format inside > > libavfilter/vf_drawtext.c, the functions feclearexcept and > > fetestexcept are used. They are missing on uclibc. On that platform, > > instead of checking i

Re: [FFmpeg-devel] Make libavfilter/vf_drawtext.c build under uclibc environments

2015-08-22 Thread Paul B Mahol
On 8/22/15, Rene Rheaume wrote: > In function func_eval_expr_int_format inside > libavfilter/vf_drawtext.c, the functions feclearexcept and > fetestexcept are used. They are missing on uclibc. On that platform, > instead of checking if the FPU raised an exception after trying to > convert a floati

Re: [FFmpeg-devel] [PATCH] swscale/alphablend: fix pointer cast

2015-08-22 Thread Paul B Mahol
On 8/22/15, Ganesh Ajjanagadde wrote: > Commit 7c72a4bbd3a0ca421c7419abad2fb65aa713dd8b was slightly wrong > > Signed-off-by: Ganesh Ajjanagadde > --- > libswscale/alphablend.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libswscale/alphablend.c b/libswscale/alphablen

Re: [FFmpeg-devel] [PATCH 2/2] libx265: Enable 12-bit encoding

2015-08-22 Thread James Almer
On 22/08/15 3:17 PM, Stephen Hutchinson wrote: > This was introduced in x265 in July, and the experimental warnings > about it in libx265 were recently removed, so there shouldn't be > any reason to have it as experimental here. > > The configure detection is bumped to X265_BUILD >= 60, as the > r

Re: [FFmpeg-devel] [PATCH 13/13] disable deprecation warnings in deprecated code

2015-08-22 Thread James Almer
On 08/08/15 8:33 AM, Andreas Cadhalpun wrote: > diff --git a/libavutil/opt.c b/libavutil/opt.c > index 62db1b5..b5df9a3 100644 > --- a/libavutil/opt.c > +++ b/libavutil/opt.c > @@ -1178,11 +1178,13 @@ int av_opt_show2(void *obj, void *av_log_obj, int > req_flags, int rej_flags) > void av_opt_set_

Re: [FFmpeg-devel] [PATCH 1/4] avcodec: remove FF_API_LOWRES around max_lowres

2015-08-22 Thread wm4
On Sat, 22 Aug 2015 20:23:28 +0200 Andreas Cadhalpun wrote: > On 22.08.2015 20:16, wm4 wrote: > > On Sat, 22 Aug 2015 19:56:53 +0200 > > Andreas Cadhalpun wrote: > > > >> This fixes building with FF_API_LOWRES disabled. > >> > >> Signed-off-by: Andreas Cadhalpun > >> --- > >> libavcodec/avcod

Re: [FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Ronald S. Bultje
Hi, On Sat, Aug 22, 2015 at 1:57 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > This fixes fate with FF_API_LAVF_BITEXACT disabled. Does this fix lavf-mov? I couldn't get that one to work... Patch OK. Ronald ___ ffmpeg-devel mail

Re: [FFmpeg-devel] [PATCH 1/4] avcodec: remove FF_API_LOWRES around max_lowres

2015-08-22 Thread James Almer
Could you please configure git to send patchsets in a single thread? It's confusing as is and clutters people's inboxes. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/4] avcodec: remove FF_API_LOWRES around max_lowres

2015-08-22 Thread Andreas Cadhalpun
On 22.08.2015 20:16, wm4 wrote: > On Sat, 22 Aug 2015 19:56:53 +0200 > Andreas Cadhalpun wrote: > >> This fixes building with FF_API_LOWRES disabled. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/avcodec.h | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/libavcodec/avc

[FFmpeg-devel] [PATCH 2/2] libx265: Enable 12-bit encoding

2015-08-22 Thread Stephen Hutchinson
This was introduced in x265 in July, and the experimental warnings about it in libx265 were recently removed, so there shouldn't be any reason to have it as experimental here. The configure detection is bumped to X265_BUILD >= 60, as the requisite 12-bit querying ability in x265_api_get was added

[FFmpeg-devel] [PATCH 1/2] libx265: 4:2:2 and 4:4:4 are no longer experimental

2015-08-22 Thread Stephen Hutchinson
x265 itself removed its warnings about these a while ago. --- libavcodec/libx265.c | 8 1 file changed, 8 deletions(-) diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c index a15ef1a..40a8daf 100644 --- a/libavcodec/libx265.c +++ b/libavcodec/libx265.c @@ -82,14 +82,6 @@ static av

[FFmpeg-devel] [PATCH 0/2] libx265: 4:2:2, 4:4:4, and 12-bit

2015-08-22 Thread Stephen Hutchinson
The patch that was submitted in July hasn't had any more replies, aside from my comments a few days ago. The change in x265 itself that removes the warnings about 12-bit likely makes the other patch outdated anyway, so I went ahead and polished my local patches that made similar changes. Stephen

Re: [FFmpeg-devel] [PATCH 1/4] avcodec: remove FF_API_LOWRES around max_lowres

2015-08-22 Thread wm4
On Sat, 22 Aug 2015 19:56:53 +0200 Andreas Cadhalpun wrote: > This fixes building with FF_API_LOWRES disabled. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/avcodec.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 41f373

Re: [FFmpeg-devel] [PATCH 2/2] avutil: Move prototype of av_ctz to common.h

2015-08-22 Thread Hendrik Leppkes
On Sat, Aug 22, 2015 at 7:58 PM, Timothy Gu wrote: > On Sat, Aug 22, 2015 at 10:31 AM Hendrik Leppkes > wrote: > >> On Sat, Aug 22, 2015 at 7:15 PM, Timothy Gu wrote: >> > intmath.h is not installed so it cannot be considered "public". >> > >> > Instead, handle av_ctz the same way as av_log2. >>

[FFmpeg-devel] [PATCH 4/4] fate: use 'c' for setting the channel_layout

2015-08-22 Thread Andreas Cadhalpun
Without this fate-filter-join fails with FF_API_GET_CHANNEL_LAYOUT_COMPAT disabled. Signed-off-by: Andreas Cadhalpun --- The filter-formats reference needs to be updated, when FF_API_GET_CHANNEL_LAYOUT_COMPAT gets disabled: --- a/tests/ref/fate/filter-formats +++ b/tests/ref/fate/filter-formats

Re: [FFmpeg-devel] [PATCH 2/2] avutil: Move prototype of av_ctz to common.h

2015-08-22 Thread Timothy Gu
On Sat, Aug 22, 2015 at 10:31 AM Hendrik Leppkes wrote: > On Sat, Aug 22, 2015 at 7:15 PM, Timothy Gu wrote: > > intmath.h is not installed so it cannot be considered "public". > > > > Instead, handle av_ctz the same way as av_log2. > > --- > > libavcodec/flacenc.c | 2 +- > > libavutil/common

[FFmpeg-devel] [PATCH 3/4] fate: add -fflags +bitexact to the relevant targets

2015-08-22 Thread Andreas Cadhalpun
This fixes fate with FF_API_LAVF_BITEXACT disabled. Signed-off-by: Andreas Cadhalpun --- tests/Makefile | 2 +- tests/fate-run.sh| 18 +- tests/fate/mpeg4.mak | 2 +- tests/fate/subtitles.mak | 4 ++-- tests/fate/vqf.mak | 2 +- 5 files changed, 14

[FFmpeg-devel] [PATCH 2/4] move the FF_QSCALE_TYPE_* defines to libavutil/internal.h

2015-08-22 Thread Andreas Cadhalpun
This fixes building with FF_API_QSCALE_TYPE disabled. Signed-off-by: Andreas Cadhalpun --- libavcodec/version.h | 3 --- libavutil/internal.h | 7 +++ libavutil/version.h | 3 +++ 3 files changed, 10 insertions(+), 3 deletions(-) diff --git a/libavcodec/version.h b/libavcodec/version.h ind

[FFmpeg-devel] [PATCH 1/4] avcodec: remove FF_API_LOWRES around max_lowres

2015-08-22 Thread Andreas Cadhalpun
This fixes building with FF_API_LOWRES disabled. Signed-off-by: Andreas Cadhalpun --- libavcodec/avcodec.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 41f373b..12a6b54 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -3493

Re: [FFmpeg-devel] [PATCH 2/2] avutil: Move prototype of av_ctz to common.h

2015-08-22 Thread Hendrik Leppkes
On Sat, Aug 22, 2015 at 7:15 PM, Timothy Gu wrote: > intmath.h is not installed so it cannot be considered "public". > > Instead, handle av_ctz the same way as av_log2. > --- > libavcodec/flacenc.c | 2 +- > libavutil/common.h | 10 ++ > libavutil/intmath.c | 1 + > libavutil/intmath

[FFmpeg-devel] Make libavfilter/vf_drawtext.c build under uclibc environments

2015-08-22 Thread René Rhéaume
In function func_eval_expr_int_format inside libavfilter/vf_drawtext.c, the functions feclearexcept and fetestexcept are used. They are missing on uclibc. On that platform, instead of checking if the FPU raised an exception after trying to convert a floating-point number to an integer, do the check

Re: [FFmpeg-devel] [PATCH 13/13] disable deprecation warnings in deprecated code

2015-08-22 Thread Andreas Cadhalpun
On 22.08.2015 19:01, wm4 wrote: > On Sat, 8 Aug 2015 13:33:24 +0200 > Andreas Cadhalpun wrote: > >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/mpeg12dec.c | 5 - >> libavcodec/options_table.h | 3 +++ >> libavcodec/pthread_frame.c | 9 - >> libavcodec/resample.c

Re: [FFmpeg-devel] [PATCH 09/13] fate: replace deprecated request_channels with request_channel_layout

2015-08-22 Thread Andreas Cadhalpun
On 22.08.2015 18:59, wm4 wrote: > On Sat, 22 Aug 2015 18:47:26 +0200 > Andreas Cadhalpun wrote: > >> On 08.08.2015 13:32, Andreas Cadhalpun wrote: >>> Signed-off-by: Andreas Cadhalpun >>> --- >>> tests/fate/ac3.mak | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/t

Re: [FFmpeg-devel] [PATCH 06/13] openal-dec: replace av_destruct_packet with av_free_packet

2015-08-22 Thread Andreas Cadhalpun
On 22.08.2015 18:58, wm4 wrote: > On Sat, 8 Aug 2015 13:32:22 +0200 > Andreas Cadhalpun wrote: > >> Signed-off-by: Andreas Cadhalpun >> --- >> libavdevice/openal-dec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavdevice/openal-dec.c b/libavdevice/openal-dec.c

Re: [FFmpeg-devel] [PATCH 04/13] add missing FF_API_DESTRUCT_PACKET guards

2015-08-22 Thread Andreas Cadhalpun
On 22.08.2015 18:56, wm4 wrote: > On Sat, 8 Aug 2015 13:32:07 +0200 > Andreas Cadhalpun wrote: > >> --- >> ffmpeg.c | 8 +++- >> libavdevice/iec61883.c | 4 >> libavformat/mux.c | 4 >> libavformat/tee.c | 8 +++- >> 4 files changed, 22 insertions(+), 2

[FFmpeg-devel] [PATCH 2/2] avutil: Move prototype of av_ctz to common.h

2015-08-22 Thread Timothy Gu
intmath.h is not installed so it cannot be considered "public". Instead, handle av_ctz the same way as av_log2. --- libavcodec/flacenc.c | 2 +- libavutil/common.h | 10 ++ libavutil/intmath.c | 1 + libavutil/intmath.h | 8 +--- 4 files changed, 13 insertions(+), 8 deletions(-

[FFmpeg-devel] [PATCH 1/2] avcodec: Remove unused includes of lavu/intmath.h

2015-08-22 Thread Timothy Gu
--- libavcodec/mpegvideo_enc.c | 1 - libavcodec/proresdec_lgpl.c | 1 - libavcodec/snow.c | 1 - libavcodec/snowdec.c| 1 - libavcodec/snowenc.c| 1 - 5 files changed, 5 deletions(-) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index 488ec51..71

Re: [FFmpeg-devel] [PATCH 13/13] disable deprecation warnings in deprecated code

2015-08-22 Thread wm4
On Sat, 8 Aug 2015 13:33:24 +0200 Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpeg12dec.c | 5 - > libavcodec/options_table.h | 3 +++ > libavcodec/pthread_frame.c | 9 - > libavcodec/resample.c | 2 ++ > libavfilter/audio.c| 2 ++

Re: [FFmpeg-devel] [PATCH 09/13] fate: replace deprecated request_channels with request_channel_layout

2015-08-22 Thread wm4
On Sat, 22 Aug 2015 18:47:26 +0200 Andreas Cadhalpun wrote: > On 08.08.2015 13:32, Andreas Cadhalpun wrote: > > Signed-off-by: Andreas Cadhalpun > > --- > > tests/fate/ac3.mak | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak

Re: [FFmpeg-devel] [PATCH 06/13] openal-dec: replace av_destruct_packet with av_free_packet

2015-08-22 Thread wm4
On Sat, 8 Aug 2015 13:32:22 +0200 Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavdevice/openal-dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavdevice/openal-dec.c b/libavdevice/openal-dec.c > index b552158..cab1eb6 100644 > --- a/li

Re: [FFmpeg-devel] [PATCH 04/13] add missing FF_API_DESTRUCT_PACKET guards

2015-08-22 Thread wm4
On Sat, 8 Aug 2015 13:32:07 +0200 Andreas Cadhalpun wrote: > --- > ffmpeg.c | 8 +++- > libavdevice/iec61883.c | 4 > libavformat/mux.c | 4 > libavformat/tee.c | 8 +++- > 4 files changed, 22 insertions(+), 2 deletions(-) > > diff --git a/ffmpeg.c b/ff

Re: [FFmpeg-devel] [PATCH]Postpone vdpau removal

2015-08-22 Thread Philip Langdale
On Sat, 22 Aug 2015 13:59:10 +0200 Andreas Cadhalpun wrote: > On 22.08.2015 05:12, Philip Langdale wrote: > > I've sent a patch to the mplayer list that removes these final bits. > > Thanks. > > > I've verified that mplayer compiles and then runs correctly, but I > > can't actually build ffmpeg

Re: [FFmpeg-devel] [PATCH 13/13] disable deprecation warnings in deprecated code

2015-08-22 Thread Andreas Cadhalpun
On 08.08.2015 13:33, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpeg12dec.c | 5 - > libavcodec/options_table.h | 3 +++ > libavcodec/pthread_frame.c | 9 - > libavcodec/resample.c | 2 ++ > libavfilter/audio.c| 2 ++ > libavfilter/

Re: [FFmpeg-devel] [PATCH 09/13] fate: replace deprecated request_channels with request_channel_layout

2015-08-22 Thread Andreas Cadhalpun
On 08.08.2015 13:32, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > tests/fate/ac3.mak | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak > index 6760de3..e636271 100644 > --- a/tests/fate/ac3.mak > +++ b/tests/fa

Re: [FFmpeg-devel] [PATCH 06/13] openal-dec: replace av_destruct_packet with av_free_packet

2015-08-22 Thread Andreas Cadhalpun
On 08.08.2015 13:32, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavdevice/openal-dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavdevice/openal-dec.c b/libavdevice/openal-dec.c > index b552158..cab1eb6 100644 > --- a/libavdevice/opena

Re: [FFmpeg-devel] [PATCH 04/13] add missing FF_API_DESTRUCT_PACKET guards

2015-08-22 Thread Andreas Cadhalpun
On 08.08.2015 13:32, Andreas Cadhalpun wrote: > --- > ffmpeg.c | 8 +++- > libavdevice/iec61883.c | 4 > libavformat/mux.c | 4 > libavformat/tee.c | 8 +++- > 4 files changed, 22 insertions(+), 2 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index

Re: [FFmpeg-devel] [PATCH 03/13] avfilter: add missing FF_API_AVFILTERBUFFER guards

2015-08-22 Thread Andreas Cadhalpun
On 16.08.2015 18:15, Andreas Cadhalpun wrote: > av_buffersink_poll_frame has been deprecated since 2013 (commit > 7e6c67dd7e6c67dd). > By now it isn't really used anymore, so I doubt many would miss it. I intend to push the patch as is tomorrow. If you want to keep av_buffersink_poll_frame, that

Re: [FFmpeg-devel] [PATCH] avformat: Remove use of AVFrac and AVStream->pts

2015-08-22 Thread Andreas Cadhalpun
On 17.08.2015 16:45, Michael Niedermayer wrote: > On Mon, Aug 17, 2015 at 04:36:52PM +0200, Nicolas George wrote: >> Le decadi 30 thermidor, an CCXXIII, Michael Niedermayer a écrit : >>> From: Michael Niedermayer >>> >>> Move field to internal part of AVStream and struct to internal.h >>> >>> Sign

Re: [FFmpeg-devel] [PATCH 10/10] ffv1: Add missing ff_ prefixes

2015-08-22 Thread Timothy Gu
On Fri, Aug 21, 2015 at 09:46:57PM -0400, Ganesh Ajjanagadde wrote: > LGTM Pushed. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 08/10] vp9dsp: Add missing ff_ prefixes

2015-08-22 Thread Timothy Gu
On Fri, Aug 21, 2015 at 10:00:25PM -0400, Ganesh Ajjanagadde wrote: > LGTM Pushed. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 07/10] aacsbr_fixed: Make fixed_{exp, log}_table static const

2015-08-22 Thread Timothy Gu
On Fri, Aug 21, 2015 at 10:03:06PM -0400, Ganesh Ajjanagadde wrote: > Relatively recent code essentially from > f85bc147fb87de048ccc5767e186ac59ec0284ef. > Been over a month, so LGTM. Pushed. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 06/10] aacdec_fixed: Make exp2tab static const

2015-08-22 Thread Timothy Gu
On Fri, Aug 21, 2015 at 10:05:06PM -0400, Ganesh Ajjanagadde wrote: > Also relatively recent, anyway LGTM. Pushed. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 09/10] fmtconvert: Remove float_interleave*

2015-08-22 Thread Timothy Gu
On Fri, Aug 21, 2015 at 09:56:01PM -0400, Ganesh Ajjanagadde wrote: > I don't know if this code will get used at some point in the future. Judging from the comment "ffdshow custom code," this is unlikely. > As such, LGTM. Pushed. Timothy ___ ffmpeg-de

[FFmpeg-devel] [PATCH] lavf/http: Export headers as AVDictionary

2015-08-22 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- libavformat/http.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index fba87ac..064239b 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -48,6 +48,7 @@ #define MAX_REDIRECTS 8 #define HTTP

Re: [FFmpeg-devel] [PATCH] lavf/http: Export headers as AVDictionary

2015-08-22 Thread Stephan Holljes
On Sat, Aug 22, 2015 at 11:00 AM, Nicolas George wrote: > Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : >> Actually I just noticed that Pragma-headers may appear multiple times. >> What would be a good way to handle that? I thought about expanding the >> header tag from "Pragma" to "

Re: [FFmpeg-devel] [PATCH] version: Fix two more typos

2015-08-22 Thread Timothy Gu
On Fri, Aug 21, 2015 at 09:51:12PM -0800, Lou Logan wrote: > On Fri, Aug 21, 2015, at 09:41 PM, Timothy Gu wrote: > > Same as cafba99b5146cf10d60ceed4257f6e82380a020e but applied to > > lavf and lavu. > > --- > > libavformat/version.h | 4 ++-- > > libavutil/version.h | 4 ++-- > > 2 files chang

Re: [FFmpeg-devel] [PATCH] all: silence clang -Wabsolute-value for unsigned subtractions

2015-08-22 Thread Nicolas George
Le quintidi 5 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : > +/* cast for -Wabsolute-value in clang 3.5+ */ > +return abs((int) (t1 - t2)) + abs((int) ((c1 & 0x00ff) - (c2 & > 0x00ff))) + > +abs((int) (((c1 & 0xff00) >> 8) - ((c2 & 0xff00) >> 8))) + > +

[FFmpeg-devel] [PATCH] all: silence clang -Wabsolute-value for unsigned subtractions

2015-08-22 Thread Ganesh Ajjanagadde
Recent clang (3.5 onwards): http://llvm.org/releases/3.5.0/tools/clang/docs/ReleaseNotes.html, http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20141006/116174.html introduces a new warning -Wabsolute-value. This patch shuts it up when we do safe unsigned subtractions with no overflow is

Re: [FFmpeg-devel] [PATCH] Put remaining pieces of CODEC_FLAG_EMU_EDGE under FF_API_EMU_EDGE.

2015-08-22 Thread Andreas Cadhalpun
Hi Ronald, On 22.08.2015 15:31, Ronald S. Bultje wrote: > On Sat, Aug 22, 2015 at 8:53 AM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> On 17.08.2015 18:25, Ronald S. Bultje wrote: >>> diff --git a/libavcodec/snowenc.c b/libavcodec/snowenc.c >>> index 16c7e05..9f74c9e 100644 >

Re: [FFmpeg-devel] [PATCH] Put remaining pieces of CODEC_FLAG_EMU_EDGE under FF_API_EMU_EDGE.

2015-08-22 Thread Ronald S. Bultje
Hi Andreas, On Sat, Aug 22, 2015 at 8:53 AM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 17.08.2015 18:25, Ronald S. Bultje wrote: > > The amv one probably looks suspicious, but since it's an intra-only > > codec, I couldn't possibly imagine what it would use the edge for, >

Re: [FFmpeg-devel] [PATCH] Put remaining pieces of CODEC_FLAG_EMU_EDGE under FF_API_EMU_EDGE.

2015-08-22 Thread Andreas Cadhalpun
On 17.08.2015 18:25, Ronald S. Bultje wrote: > The amv one probably looks suspicious, but since it's an intra-only > codec, I couldn't possibly imagine what it would use the edge for, > and the vsyncht fate result doesn't change, so it's probably OK. > --- > ffmpeg_opt.c | 2 ++ > ffplay.

Re: [FFmpeg-devel] [PATCH]Postpone vdpau removal

2015-08-22 Thread Andreas Cadhalpun
On 22.08.2015 05:12, Philip Langdale wrote: > I've sent a patch to the mplayer list that removes these final bits. Thanks. > I've verified that mplayer compiles and then runs correctly, but I > can't actually build ffmpeg without the API CAPs as the full-decoder > vdpau code is still being compil

Re: [FFmpeg-devel] [GSoC] BDA (DTV) Capture / tuning -- work-in-progress

2015-08-22 Thread Michael Niedermayer
On Sat, Aug 22, 2015 at 05:18:04AM +0200, Máté Sebők wrote: > > > > fails to build with mingw due to missing header > > In file included from ffmpeg/libavdevice/dshow.c:33:0: > > ffmpeg/libavdevice/bdadefs.h:8:24: fatal error: combaseapi.h: No such > > file or directory > > compilation terminated.

Re: [FFmpeg-devel] [PATCH][RFC] avutil/x86/asm: rename REG_SP to REG_sp

2015-08-22 Thread Ganesh Ajjanagadde
On Fri, Aug 21, 2015 at 11:27 PM, Ganesh Ajjanagadde wrote: > On Fri, Aug 21, 2015 at 9:07 PM, Michael Niedermayer > wrote: >> On Fri, Aug 21, 2015 at 02:21:48PM -0400, Ganesh Ajjanagadde wrote: >>> On Fri, Aug 21, 2015 at 2:20 PM, Ganesh Ajjanagadde >>> wrote: >>> > On Fri, Aug 21, 2015 at 2:17

Re: [FFmpeg-devel] [PATCH] version: Fix two more typos

2015-08-22 Thread Michael Niedermayer
On Fri, Aug 21, 2015 at 11:23:52PM -0700, Timothy Gu wrote: > El viernes, 21 de agosto de 2015, Lou Logan escribió: > > > > Good, of course. No need to send a patch for trivial comment typo fixes > > like these; you can just push. > > > > I don't think I have Git write access to git.v.o right now

Re: [FFmpeg-devel] [PATCH] lavf/http: Export headers as AVDictionary

2015-08-22 Thread Nicolas George
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : > Actually I just noticed that Pragma-headers may appear multiple times. > What would be a good way to handle that? I thought about expanding the > header tag from "Pragma" to "Pragma-%02d" with either an incrementing > number or just usin

Re: [FFmpeg-devel] [PATCH 02/10] aacenc: Add missing ff_ prefixes

2015-08-22 Thread Nicolas George
Le quintidi 5 fructidor, an CCXXIII, Claudio Freire a écrit : > They were included in the symbol table but only as local, the libavcodec.v > file makes sure to make everything not explicitly mentioned for export > local. > > Though it's possible that it depends on the compiler version? It depends