More readable and less breakable.
---
libavutil/imgutils.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/libavutil/imgutils.c b/libavutil/imgutils.c
index 8956ae3..a189a50 100644
--- a/libavutil/imgutils.c
+++ b/libavutil/imgutils.c
@@ -236,11 +236,21 @@ typed
tr -d '\r' ?
Timothy
El El lun, 16 de nov de 2015 a las 4:26 AM, Marton Balint
escribió:
> Sed \r is not portable, it does not work on freebsd, hopefully awk will.
>
> Signed-off-by: Marton Balint
> ---
> tests/fate-run.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
On Sun, Nov 8, 2015 at 7:24 AM Kieran Kunhya wrote:
> On 8 November 2015 at 00:30, Timothy Gu wrote:
> > Allows one to do:
> >
> > ffmpeg -s 1920x1080 -i blah.v210 ...
> > ffmpeg -s 1920x1080 -f v210x -i blah.yuv10 ...
> >
> > Fixes #1869
On Mon, Nov 9, 2015 at 10:26 AM Calvin Walton
wrote:
> The ffmpeg command-line tool actually maps the "-s" and "-r" input
> options to "video_size" and "framerate" respectively on the input
> format. This will work automatically.
>
Thanks for the clarification :)
Timothy
___
On Sun, Nov 8, 2015 at 9:28 AM Claudio Freire
wrote:
> This particular piece of code is going to disappear soon, so not sure
> it's worth applying the patch.
>
Oh, okay then.
> Is the warning causing any troubles?
>
No, other than the minor annoyance it causes when compiling.
Timothy
___
---
libavformat/genh.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/libavformat/genh.c b/libavformat/genh.c
index 260e320..44862b3 100644
--- a/libavformat/genh.c
+++ b/libavformat/genh.c
@@ -40,7 +40,6 @@ static int genh_read_header(AVFormatContext *s)
{
unsigned s
---
libavcodec/aaccoder_twoloop.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/aaccoder_twoloop.h b/libavcodec/aaccoder_twoloop.h
index e53257f..328d7c8 100644
--- a/libavcodec/aaccoder_twoloop.h
+++ b/libavcodec/aaccoder_twoloop.h
@@ -691,7 +691,7 @@ static void
On Sun, Nov 08, 2015 at 03:04:21AM +0100, Michael Niedermayer wrote:
> On Sat, Nov 07, 2015 at 03:58:28PM -0800, Timothy Gu wrote:
> > ---
> >
> > Consolidates BUF_SIZE macros
> >
> > ---
> > tests/checkasm/Makefile | 1 +
> > tests/checka
On Sat, Nov 07, 2015 at 10:22:34PM -0300, James Almer wrote:
> On 11/7/2015 9:45 PM, Timothy Gu wrote:
> > +#define OFFSET(x) offsetof(V210DemuxerContext, x)
> > +#define DEC AV_OPT_FLAG_DECODING_PARAM
> > +static const AVOption v210_options[] = {
> > +{ "video
/null
+++ b/libavformat/v210.c
@@ -0,0 +1,133 @@
+/*
+ * Raw v210 video demuxer
+ * Copyright (c) 2015 Tiancheng "Timothy" Gu
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser Gene
@@
+/*
+ * Raw v210 video demuxer
+ * Copyright (c) 2015 Tiancheng "Timothy" Gu
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software
@@
+/*
+ * Raw v210 video demuxer
+ * Copyright (c) 2015 Tiancheng "Timothy" Gu
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software
diff --git a/tests/checkasm/pixblockdsp.c b/tests/checkasm/pixblockdsp.c
new file mode 100644
index 000..70e7f74
--- /dev/null
+++ b/tests/checkasm/pixblockdsp.c
@@ -0,0 +1,107 @@
+/*
+ * Copyright (c) 2015 Tiancheng "Timothy" Gu
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free
On Fri, Nov 06, 2015 at 09:11:40PM +0100, Andreas Cadhalpun wrote:
> Subject: [FFmpeg-devel] [PATCH] jvdec: avoid unsized overflow in comparison
*unsigned
[...]
Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/lis
On Fri, Nov 06, 2015 at 08:18:39PM +0100, Andreas Cadhalpun wrote:
[...]
> diff --git a/src/download b/src/download
> index 5691fbd..6d3517e 100644
> --- a/src/download
> +++ b/src/download
> @@ -53,13 +53,22 @@
>
> Linux Packages
>
> + href="https://track
On Sun, Nov 1, 2015 at 8:59 AM Timothy Gu wrote:
> ---
> libavcodec/x86/pixblockdsp.asm | 66
> --
> 1 file changed, 31 insertions(+), 35 deletions(-)
>
Ping set.
Timothy
___
ffmpeg-devel mail
On Tue, Nov 3, 2015 at 4:47 AM Ganesh Ajjanagadde > - GCC vectorization
slows down compilation A LOT in all versions. The newer
> > the worse.
>
> A ~ 20% slowdown on a build for a ~ 20% improvement in an overall FATE
> bench - sounds like a win to me especially with ccache.
Of course, but unfor
On Mon, Nov 2, 2015 at 8:23 PM Rostislav Pehlivanov
wrote:
> >if one removes the crippling
> >-fno-tree-vectorize
> Yes, I think a config option to turn this flag on (like the unsafe
> bitstream reader) would be good. Defaulting to off by default if it doesn't
> break anything for at least a few
On Sun, Nov 1, 2015 at 4:45 PM Michael Niedermayer
wrote:
> On Sun, Nov 01, 2015 at 10:40:03AM -0800, Timothy Gu wrote:
> > ---
> > libavfilter/vf_boxblur.c | 110
> +++
> > 1 file changed, 44 insertions(+), 66 deletions(-)
On Sun, Nov 1, 2015 at 11:10 AM Ganesh Ajjanagadde wrote:
> Have not tested, but just a general comment. Personally, I follow the
> twice repition is ok, thrice means it is good to factor out. I would
> have been happier if the diff-stat was better than 44+/66- in terms of
> deletions.
The most
---
libavfilter/vf_boxblur.c | 110 +++
1 file changed, 44 insertions(+), 66 deletions(-)
diff --git a/libavfilter/vf_boxblur.c b/libavfilter/vf_boxblur.c
index ef01cf9..6934076 100644
--- a/libavfilter/vf_boxblur.c
+++ b/libavfilter/vf_boxblur.c
@@ -20
b/tests/checkasm/pixblockdsp.c
new file mode 100644
index 000..d59d162
--- /dev/null
+++ b/tests/checkasm/pixblockdsp.c
@@ -0,0 +1,107 @@
+/*
+ * Copyright (c) 2015 Tiancheng "Timothy" Gu
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and
---
libavcodec/x86/pixblockdsp.asm | 66 --
1 file changed, 31 insertions(+), 35 deletions(-)
diff --git a/libavcodec/x86/pixblockdsp.asm b/libavcodec/x86/pixblockdsp.asm
index 7c5377b..a7d9816 100644
--- a/libavcodec/x86/pixblockdsp.asm
+++ b/libavcodec/x8
On Tue, Oct 20, 2015 at 11:29 PM Michael Niedermayer
wrote:
> > diff --git a/libavcodec/pixblockdsp.c b/libavcodec/pixblockdsp.c
> > index 322e1dd..0f23d8a 100644
> > --- a/libavcodec/pixblockdsp.c
> > +++ b/libavcodec/pixblockdsp.c
> > @@ -23,12 +23,38 @@
> > #include "avcodec.h"
> > #include
On Sat, Oct 24, 2015 at 11:03 AM James Almer wrote:
> Gentoo and Debian both seem to ship it. Arch does as well but on their
> Community repository (ffmpeg adopting it may be incentive enough for them
> to move it to Extra).
>
In fact, at least two of the reverse dependencies (out of 8, not count
On Thu, Oct 22, 2015 at 5:01 PM Ganesh Ajjanagadde
wrote:
> Sample benchmark (x86-64, Haswell, GNU/Linux), fraps-v2 from FATE:
> new:
> 280110 decicycles in qsort, 1 runs, 0 skips
> 268260 decicycles in qsort, 2 runs, 0 skips
>
> old:
> 1469910 decicycles in qsort, 1 r
On Wed, Oct 21, 2015 at 10:46 AM Christophe Gisquet <
christophe.gisq...@gmail.com> wrote:
> 2015-10-18 2:47 GMT+02:00 Timothy Gu :
> > This function is only used within other inline asm functions, hence the
> > HAVE_MMX_INLINE guard. Per recent discussions, we should no
On Wed, Oct 21, 2015 at 10:32 AM Timothy Gu wrote:
> On Tue, Oct 20, 2015 at 7:36 PM James Almer wrote:
>
>> On 10/20/2015 10:32 PM, Timothy Gu wrote:
>>
> > +; mov type used for src1q, dstq, first reg, second reg
>> > +%macro DIFF_BYTES_LOOP_CORE 4
>>
On Tue, Oct 20, 2015 at 7:36 PM James Almer wrote:
> On 10/20/2015 10:32 PM, Timothy Gu wrote:
> > +; mov type used for src1q, dstq, first reg, second reg
> > +%macro DIFF_BYTES_LOOP_CORE 4
> > +%if regsize != 16
>
> %if mmsize != 16
>
> By checking regsize you
On Tue, Oct 20, 2015 at 7:09 PM Ganesh Ajjanagadde wrote:
> Hi all,
>
> It is known that there exist at least certain parts of the codebase
> that do not work correctly if ints are 64 bits. One of them I noticed
> was in avutil/intmath.h: ff_ctz_c does not compute the right thing if
> int is 64 b
SSE2 version 4%-35% faster than MMX depending on the width.
AVX2 version 1%-13% faster than SSE2 depending on the width.
---
Addressed James's and Henrik's advices. Removed heuristics based on width.
Made available both aligned and unaligned versions. For AVX2 version,
gracefully fall back on SSE2
On Tue, Oct 20, 2015 at 2:18 PM James Almer wrote:
> Removing this will make the INLINE_MMXEXT if statement below fail to
> compile on
> builds with inline asm disabled (msvc, etc). Even with dead code
> elimination in
> mind you'd need at least a prototype for the relevant functions.
> Just move
Before:
15543 decicycles in get_pixels, 4193214 runs, 1090 skips
After:
5713 decicycles in get_pixels, 8387564 runs, 1044 skips
---
libavcodec/pixblockdsp.c | 36 ++-
libavcodec/pixblockdsp_template.c | 40 ---
2
On Tue, Oct 20, 2015 at 9:30 AM Michael Niedermayer
wrote:
> On Sat, Oct 17, 2015 at 06:05:45PM -0700, Timothy Gu wrote:
> > This reverts commit 628e6d0164febc8e69b0f10dfa487e8a2dd1a28a and uses
> > a better fix.
> >
> > Before:
> > 4483 decicycles in get_pix
llard
+;* Copyright (c) 2002-2004 Michael Niedermayer
+;*
+;* MMX optimization by Nick Kurshev
+;* Conversion to NASM format by Tiancheng "Timothy" Gu
+;*
+;* This file is part of FFmpeg.
+;*
+;* FFmpeg is free software; you can redistribute it and/or
+;* modify it under the terms of the GNU
On Mon, Oct 19, 2015 at 3:53 PM Michael Niedermayer
wrote:
> On Mon, Oct 19, 2015 at 01:00:43PM -0700, Timothy Gu wrote:
> > It is done this way in huffyuvdsp as well.
> > ---
> > libavcodec/huffyuvencdsp.c | 4 ++--
> > libavcodec/huffyuvencdsp.h |
On Sat, Oct 17, 2015 at 4:49 PM Timothy Gu wrote:
> That's where all yasm initiation steps are. Also removes the overlap
> between the two files.
> ---
> libavcodec/x86/vc1dsp_init.c | 36 +---
> libavcodec/x86/vc1dsp_mmx.c | 27
On Sat, Oct 17, 2015 at 6:31 PM Michael Niedermayer
wrote:
> On Sat, Oct 17, 2015 at 04:48:59PM -0700, Timothy Gu wrote:
> > This function does not exist.
> > ---
> > libavcodec/x86/fpel.h | 2 --
> > 1 file changed, 2 deletions(-)
>
>
On Sat, Oct 17, 2015 at 4:49 PM Timothy Gu wrote:
> ---
> libavcodec/x86/fpel.h | 6 ++
> libavcodec/x86/h264_qpel.c | 4
> 2 files changed, 6 insertions(+), 4 deletions(-)
>
Pushed.
Timothy
___
ffmpeg-devel mailing li
On Mon, Oct 19, 2015 at 1:44 PM Ganesh Ajjanagadde wrote:
> On Mon, Oct 19, 2015 at 4:41 PM, Hendrik Leppkes
> wrote:
> > On Mon, Oct 19, 2015 at 10:36 PM, Ganesh Ajjanagadde
> wrote:
> >> On Mon, Oct 19, 2015 at 4:00 PM, Timothy Gu
> wrote:
> >>> A
About 16% faster on large clips (>1200px width), more than 2x slower on small
clips
(352px). So using a heuristic to select with one to use.
---
libavcodec/huffyuvenc.c| 6 +++---
libavcodec/huffyuvencdsp.c | 4 ++--
libavcodec/huffyuvencdsp.h | 4 ++--
libavcodec/pnge
4% to 35% faster depending on the width.
---
libavcodec/x86/huffyuvencdsp.asm | 31 ---
libavcodec/x86/huffyuvencdsp_mmx.c | 8 +++-
2 files changed, 27 insertions(+), 12 deletions(-)
diff --git a/libavcodec/x86/huffyuvencdsp.asm b/libavcodec/x86/huffyuvencdsp.a
@@
+;
+;* SIMD-optimized HuffYUV encoding functions
+;* Copyright (c) 2000, 2001 Fabrice Bellard
+;* Copyright (c) 2002-2004 Michael Niedermayer
+;*
+;* MMX optimization by Nick Kurshev
+;* Conversion to NASM format by Tiancheng "Ti
It is done this way in huffyuvdsp as well.
---
libavcodec/huffyuvencdsp.c | 4 ++--
libavcodec/huffyuvencdsp.h | 4 ++--
libavcodec/x86/huffyuvencdsp_mmx.c | 5 +++--
3 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/libavcodec/huffyuvencdsp.c b/libavcodec/huffyuvencds
Before:
15543 decicycles in get_pixels, 4193214 runs, 1090 skips
After:
5713 decicycles in get_pixels, 8387564 runs, 1044 skips
---
libavcodec/pixblockdsp.c | 38 -
libavcodec/pixblockdsp_template.c | 40 ---
This reverts commit 628e6d0164febc8e69b0f10dfa487e8a2dd1a28a and uses
a better fix.
Before:
4483 decicycles in get_pixels_8x4_sym, 131032 runs, 40 skips
After:
2569 decicycles in get_pixels_8x4_sym, 131054 runs, 18 skips
---
libavcodec/dnxhdenc.c | 24
1 file c
This function is only used within other inline asm functions, hence the
HAVE_MMX_INLINE guard. Per recent discussions, we should not worry about
the performance of inline asm-only builds.
---
The conversion process has to start _somewhere_...
Difference from previous version: use mova/h, correct
This function is only used within other inline asm functions, hence the
HAVE_MMX_INLINE guard. Per recent discussions, we should not worry about
the performance of inline asm-only builds.
---
libavcodec/x86/vc1dsp.asm | 89 -
libavcodec/x86/vc1dsp_mmx.
That's where all yasm initiation steps are. Also removes the overlap
between the two files.
---
libavcodec/x86/vc1dsp_init.c | 36 +---
libavcodec/x86/vc1dsp_mmx.c | 27 ---
2 files changed, 25 insertions(+), 38 deletions(-)
diff --git a/li
---
libavcodec/x86/fpel.h | 6 ++
libavcodec/x86/h264_qpel.c | 4
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/libavcodec/x86/fpel.h b/libavcodec/x86/fpel.h
index 4d93959..625c47d 100644
--- a/libavcodec/x86/fpel.h
+++ b/libavcodec/x86/fpel.h
@@ -22,6 +22,10 @@
#in
This function does not exist.
---
libavcodec/x86/fpel.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/libavcodec/x86/fpel.h b/libavcodec/x86/fpel.h
index 625c47d..4e83cf7 100644
--- a/libavcodec/x86/fpel.h
+++ b/libavcodec/x86/fpel.h
@@ -40,8 +40,6 @@ void ff_put_pixels4_mmx(uint8_t *block,
On Sat, Oct 17, 2015 at 7:59 AM Ganesh Ajjanagadde
wrote:
> On Thu, Oct 15, 2015 at 6:26 AM, Ganesh Ajjanagadde
> wrote:
> > On Thu, Oct 15, 2015 at 1:45 AM, Timothy Gu
> wrote:
> >> On Wed, Oct 14, 2015 at 8:05 PM Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
On Tue, Oct 13, 2015 at 9:33 PM Timothy Gu wrote:
> ---
> libavfilter/vf_chromakey.c | 25 +
> 1 file changed, 17 insertions(+), 8 deletions(-)
>
Reviewed by Timo on IRC and pushed.
Timothy
___
ffmpeg-devel
On Wed, Oct 14, 2015 at 2:21 PM Michael Niedermayer
wrote:
> On Tue, Oct 13, 2015 at 09:48:35PM -0700, Timothy Gu wrote:
> > ---
> > libavutil/opencl.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
>
> LGTM
>
&g
On Wed, Oct 14, 2015 at 8:05 PM Ganesh Ajjanagadde
wrote:
> This clarifies and Doxygen's the comment for ff_fmt_is_in.
^
gajjanag.c:1:23: error: unexpected token ‘adjective’
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavfilter/internal.h | 8 +++-
> 1 file change
On Wed, Oct 14, 2015 at 8:08 PM Ganesh Ajjanagadde
wrote:
> pushed. Please let me know if such things are ok or not in the future
> - technically I guess doc maintainers need to respond before I can
> push, but as this change was trivial I thought I would do it.\
>
As a documentation maintainer,
On Tue, Oct 13, 2015 at 6:24 PM Ganesh Ajjanagadde wrote:
> On Tue, Oct 13, 2015 at 9:20 PM, Timothy Gu wrote:
> > Patch LGTM.
>
> Go ahead and push
Pushed, thanks.
Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
ht
Silences warnings regarding `clCreateCommandQueue` being deprecated.
Only a very limited number of products support 2.0. Since the
replacement API (`clCreateCommandQueueWithProperties`) is only available
in 2.0, we should not update it just yet.
---
libavutil/opencl.h | 1 +
1 file changed, 1 ins
All other classes use lowercase names.
---
libavutil/opencl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/opencl.c b/libavutil/opencl.c
index f720ce0..62706f3 100644
--- a/libavutil/opencl.c
+++ b/libavutil/opencl.c
@@ -84,7 +84,7 @@ static const AVOption opencl_o
Useful when debugging.
---
libavutil/opencl.c | 33 ++---
1 file changed, 30 insertions(+), 3 deletions(-)
diff --git a/libavutil/opencl.c b/libavutil/opencl.c
index c2956fd..f720ce0 100644
--- a/libavutil/opencl.c
+++ b/libavutil/opencl.c
@@ -449,12 +449,14 @@ end:
c
---
libavutil/opencl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavutil/opencl.c b/libavutil/opencl.c
index 8686493..c2956fd 100644
--- a/libavutil/opencl.c
+++ b/libavutil/opencl.c
@@ -484,7 +484,8 @@ cl_program av_opencl_compile(const char *program_name,
const cha
---
libavfilter/vf_chromakey.c | 25 +
1 file changed, 17 insertions(+), 8 deletions(-)
diff --git a/libavfilter/vf_chromakey.c b/libavfilter/vf_chromakey.c
index 47fdea631..3309748 100644
--- a/libavfilter/vf_chromakey.c
+++ b/libavfilter/vf_chromakey.c
@@ -35,6 +35,9 @@
lp".
Type "apropos word" to search for commands related to "word"...
Reading symbols from ffmpeg_g...done.
(gdb) r -loglevel debug -f lavfi -i allrgb -vf chromakey=green:opencl=1 -f
null -
Starting program: /home/timothy-gu/ffmpeg/ffmpeg/ffmpeg_g -loglevel debug
-f lavfi -i allrg
El martes, 13 de octubre de 2015, Michael Niedermayer
escribió:
> On Sun, Oct 11, 2015 at 04:27:49PM -0400, Ganesh Ajjanagadde wrote:
> > ---
> > .gitignore | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/.gitignore b/.gitignore
> > index 0b26f68..b215828 100644
> > --- a/.gitigno
On Sun, Oct 11, 2015 at 6:31 PM Ganesh Ajjanagadde wrote:
> Just to make this clear: suppose (hypothetically) av_ctz was public.
> If my application never used av_ctz, but I (or my distro) upgrades
> libav*, then I don't need to recompile/relink my code?
>
If we are only talking about av_ctz her
On Sun, Oct 11, 2015 at 12:16 PM Ganesh Ajjanagadde
wrote:
> Alternatively (for now, more sustainable is a proper hook), could you
> post here your alias to do this?
>
git remote set-url origin --push --add g...@github.com:FFmpeg/FFmpeg.git
Timothy
__
On Sun, Oct 11, 2015 at 11:29 AM Ronald S. Bultje
wrote:
> It's an a non-installed header and only used in one place (flacenc).
> Since ff_ctz is static inline, it's fine to use that instead.
> ---
> libavcodec/flacenc.c | 2 +-
> libavutil/intmath.c | 5 -
> libavutil/intmath.h | 8 --
On Sun, Oct 11, 2015 at 11:44 AM Ganesh Ajjanagadde
wrote:
> I noticed that the Github mirror: https://github.com/FFmpeg/FFmpeg is
> over 3 hours out of sync with the main repos, making it unusable as a
> fetch url for development. Anyone knows why this is the case?
>
It is not an automatic mirr
On Sun, Oct 11, 2015 at 8:21 AM Ganesh Ajjanagadde
wrote:
> Fixes Ticket4922.
>
> Commit 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure.
>
> From: Jean-Yves Avenard
>
You might want to use `git commit --author` instead of manually adding a
"From" line in the commit message body.
Timo
On Sun, Oct 11, 2015 at 9:10 AM Ganesh Ajjanagadde wrote:
> Isn't that an API change and not an ABI change (it was in doc/APIchanges)?
>
No, since intmath.h is not public.
Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org
On Sat, Oct 10, 2015 at 7:20 AM Ganesh Ajjanagadde wrote:
> On Thu, Oct 8, 2015 at 8:37 AM, Michael Niedermayer
> wrote:
> > Hi
> >
> > ill make 2.8.1 soon from release/2.8
> > if you want something backported push it to release/2.8 soon!
> > (or ask me to pull from your git tree, or if its triv
On Thu, Oct 8, 2015 at 9:55 AM Ivan Uskov wrote:
> Hello Hendrik,
>
> Thursday, October 8, 2015, 7:03:36 PM, you wrote:
>
> HL> That doesn't seem correct to me. The mfx dispatcher library has the BSD
> HL> license plastered all over it. BSD is compatible with the GPL.
> Dispatcher, yes, but mfx l
On Sun, Oct 4, 2015 at 2:21 PM Michael Niedermayer wrote:
> there is a spec ?
>
Don't think so for the codec itself.
> i only know of soe reference source code and i looked yesterday
> and it usesed the full 8 bits unless i missed something
>
https://alac.macosforge.org/trac/browser/trunk/co
On Fri, Oct 2, 2015 at 10:08 AM James Darnley
wrote:
> The third patch uses them in the remaining inline assembly.
>
That's the crux of the problem: inline asm uses these constants, but will
be unable to without yasm. Either we drop compatibility for inline asm for
x86 platforms w/o yasm, or we
El domingo, 27 de septiembre de 2015, Gonzalo escribió:
>
>
> El 27/09/15 a las 22:45, Gonzalo escribió:
>
>> $ ffplay /media/Linux/anims/wildlife/wildlife.%04d.png
>> ffplay version N-75563-g235381e Copyright (c) 2003-2015 the FFmpeg
>> developers
>> built with gcc 4.8 (Ubuntu 4.8.4-2ubuntu1~1
On Sat, Sep 26, 2015 at 7:47 AM, Ganesh Ajjanagadde
wrote:
> diff --git a/ffplay.c b/ffplay.c
> index d302793..4f3322b 100644
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -1348,6 +1353,25 @@ static void toggle_pause(VideoState *is)
> is->step = 0;
> }
>
> +static void toggle_mute(VideoState *is)
>
On Mon, Sep 21, 2015 at 6:51 PM Rodger Combs wrote:
> ---
> doc/bitstream_filters.texi | 11 ---
> libavcodec/aac_adtstoasc_bsf.c | 74
> ++
> libavformat/flvenc.c | 6
> libavformat/movenc.c | 6
> 4 files changed,
On Mon, Sep 21, 2015 at 6:50 PM Rodger Combs wrote:
> ---
> libavcodec/aacdec_template.c | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/libavcodec/aacdec_template.c b/libavcodec/aacdec_template.c
> index c2d7d05..a999013 100644
> --- a/libavcodec/aacdec_template.c
> +++ b/libav
On Wed, Sep 16, 2015 at 6:09 PM Michael Niedermayer
wrote:
> this is wrong, the buffer is constant
> if the content of the buffer would change that would be a serious
> bug.
> Before this change the compiler would detect direct changes done to
> the buffer
>
Commit reverted before a consensus is
On Wed, Sep 16, 2015 at 2:20 PM, Ganesh Ajjanagadde
wrote:
> gamma_convert is only used with the old code. Thus, it is
> placed under a header guard. This patch silences a -Wunused-function
> observed on GCC 5.2.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libswscale/swscale.c | 2 ++
> 1 file
On Wed, Sep 16, 2015 at 2:25 PM, Ganesh Ajjanagadde
wrote:
> Silences a -Wunused-variable and -Wunused-function observed under GCC 5.2.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavformat/mpjpegdec.c | 16
> 1 file changed, 16 deletions(-)
Applied, thanks.
Timothy
__
On Wed, Sep 16, 2015 at 3:50 PM, Ganesh Ajjanagadde
wrote:
> lpd.buf is non-const and discards the const qualifier of zerobuffer.
> This fixes -Wdiscarded-qualifiers observed with GCC 5.2.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavformat/format.c | 2 +-
> 1 file changed, 1 insertion(+)
On Mon, Sep 14, 2015 at 5:09 PM Ganesh Ajjanagadde wrote:
> Shouldn't this function be renamed, since it no longer necessarily
> prints in 3 columns?
>
Applied with that change.
Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmp
On Sat, Sep 12, 2015 at 11:28 PM Ganesh Ajjanagadde
wrote:
> op== is confusing, as it may easily be mistook for an equality check,
> while in reality it is doing an assigment to '='.
> This patch has the additional benefit of cleaning up 2 warnings from
> shellcheck.net.
>
> Signed-off-by: Ganesh
---
configure | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index da18e70..3dfbb91 100755
--- a/configure
+++ b/configure
@@ -425,7 +425,10 @@ if test -t 1 && which tput >/dev/null; then
error_color=$(tput setaf 1)
reset_color=$(tp
On Wed, Sep 9, 2015 at 11:15 AM Ganesh Ajjanagadde
wrote:
> Signed-off-by: Ganesh Ajjanagadde
> ---
> configure | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index 056feea..4842b52 100755
> --- a/configure
> +++ b/configure
> @@ -421,6 +421,7
On Tue, Sep 8, 2015 at 12:18 PM Ganesh Ajjanagadde
wrote:
> quotes='""'
> -if test -t 1 && which tput >/dev/null; then
> -ncolors=$(tput colors)
> -if test -n "$ncolors" && test $ncolors -ge 8; then
> -bold_color=$(tput bold)
> -warn_color=$(tput setaf 3)
> -reset
On Tue, Sep 1, 2015 at 6:46 PM Rostislav Pehlivanov
wrote:
> /**
> + * List of currently supported profiles, anything not listed isn't
> supported.
> + */
>
> +struct AACProfileOptions aacenc_profiles[] = {
>
static const?
> +{FF_PROFILE_AAC_LOW,
> +{ /* Default profile, these
On Thu, Aug 27, 2015 at 06:36:00PM -0400, Ganesh Ajjanagadde wrote:
> Please add the area (e.g configure:...) to the commit message.
> More generally, I suggest a slight cleanup of the commit message:
> first line-> "configure: print out enabled programs"
> skip a blank line, body-> "This prints ou
On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote:
> This patch introduces color warning messages (yellow).
> Tested with zsh, bash, and sh (bash under --posix).
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> configure | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
I
On Tue, Aug 25, 2015 at 09:25:14PM -0400, Ganesh Ajjanagadde wrote:
> Ok. Please place the function outside of all the opt_* stuff;
> since this is not setting options, it should not be in the middle of them.
> Otherwise, patch LGTM.
Pushed with the changes.
Timothy
__
On Tue, Aug 25, 2015 at 08:19:59PM -0400, Ganesh Ajjanagadde wrote:
> For consistency, prefix with an opt_?
This name is chosen to be consistent with other show_ functions. opt_*
are exclusively used for setting some options AFAICT.
> Maybe you could check the return value of printf?
Absolutely
On Tue, Aug 25, 2015 at 08:28:43PM -0400, Ganesh Ajjanagadde wrote:
> On Tue, Aug 25, 2015 at 8:08 PM, Timothy Gu wrote:
> >
> > cmdutils.c is shared for all ff* programs. -hwaccel is only supported in
> > ffmpeg.
>
> Why?
Because nobody has implemented it for other f
On Tue, Aug 25, 2015 at 5:06 PM Ganesh Ajjanagadde wrote:
> On Tue, Aug 25, 2015 at 7:47 PM, Timothy Gu wrote:
> > ---
> > doc/ffmpeg.texi | 4
> > ffmpeg_opt.c| 14 ++
> > 2 files changed, 18 insertions(+)
> >
> > diff --git a/doc/
---
doc/ffmpeg.texi | 4
ffmpeg_opt.c| 14 ++
2 files changed, 18 insertions(+)
diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi
index 51a4ec5..e1d8562 100644
--- a/doc/ffmpeg.texi
+++ b/doc/ffmpeg.texi
@@ -698,6 +698,10 @@ is not specified, the value of the @var{DISPLAY}
env
On Tue, Aug 25, 2015 at 7:37 AM Paul B Mahol wrote:
> +@section hstack
> +Stack streams horizontally.
> +
> +All streams must be of same pixel format and of same height.
>
Add a new line here
> +Note: this filter is faster then using @ref{overlay} and @ref{pad} filter
+to create same output.
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
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.
> > ---
> >
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(-
---
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
201 - 300 of 626 matches
Mail list logo