Updated patch:
>From 204a8da14f92fcdefe63dda341261bdd25d6ce87 Mon Sep 17 00:00:00 2001
From: Daniel Oberhoff
Date: Mon, 28 Jul 2014 23:58:12 +0200
Subject: [PATCH] added rectification filter
---
libavfilter/Makefile | 1 +
libavfilter/allfilters.c | 1 +
libavfilter/vf_recti
First off thanks for taking the time to review.
Am 29.07.2014 um 01:16 schrieb Carl Eugen Hoyos :
> Daniel Oberhoff gmail.com> writes:
>
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License
>
> Your patch is mis
On 13/07/14 8:39 AM, Kieran Kunhya wrote:
> On 13 July 2014 07:00, James Almer wrote:
>> Signed-off-by: James Almer
>
> LGTM. There's the odd use of 3-operand in some of the macros. Have you
> benched AVX?
Yes. Tested luma with an 8bit sample and it's ~%5 faster.
Sorry for the late reply. I'll
On Tue, Jul 29, 2014 at 01:29:16AM +0200, Nicolas George wrote:
> Le decadi 10 thermidor, an CCXXII, Michael Niedermayer a écrit :
> > Breaks ABI, AVFrame size is depending on libavutil and may be non
> > constant for a libavfilter build
>
> That was intentional (that makes the code simpler with l
On Sun, Jul 27, 2014 at 10:27:35AM +0200, Christophe Gisquet wrote:
> One may interleave even more vertical and horizontal passes, but this
> is less straightforward (more code changes than just moves).
>
> --
> Christophe
> hevc_filter.c | 90
> +-
On Mon, Jul 28, 2014 at 09:11:49PM -0300, James Almer wrote:
> It's slightly faster
>
> Signed-off-by: James Almer
> ---
> libavcodec/x86/hevc_deblock.asm | 8
> 1 file changed, 8 insertions(+)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78704
It's slightly faster
Signed-off-by: James Almer
---
libavcodec/x86/hevc_deblock.asm | 8
1 file changed, 8 insertions(+)
diff --git a/libavcodec/x86/hevc_deblock.asm b/libavcodec/x86/hevc_deblock.asm
index f7cd031..7d67a07 100644
--- a/libavcodec/x86/hevc_deblock.asm
+++ b/libavcodec/x
On 2014-07-23 13:42 -0800, Lou Logan wrote:
> On Tue, 22 Jul 2014 10:36:31 -0800, Lou Logan wrote:
[...]
> Actually I'd like to wait on this anchor patch since I contacted
> Alexander to add some redirects instead, so the new, cleaner names can
> be used. If that's not possible then I'll push this
Le decadi 10 thermidor, an CCXXII, Michael Niedermayer a écrit :
> Breaks ABI, AVFrame size is depending on libavutil and may be non
> constant for a libavfilter build
That was intentional (that makes the code simpler with less error checks):
the frame is only allocated and used internally, so thi
Daniel Oberhoff gmail.com> writes:
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License
Your patch is missing a configure hunk that
makes sure that the filter is only compiled
if --enable-gpl was specified.
Search
---
doc/APIchanges|4
libavcodec/avcodec.h |5 -
libavcodec/mpeg12dec.c| 21 -
libavcodec/version.h |5 -
libavfilter/vf_showinfo.c |3 +++
libavutil/frame.h | 12
libavutil/version.h |2 +
Hi again,
Since at least in the web view of the list attachements seem to be stripped, I
send it here in the clear:
thanks for reviewing!
Daniel
>From 7b5b3b42804714ddc56d2b8cd2c5709b4bb91f70 Mon Sep 17 00:00:00 2001
From: Daniel Oberhoff
Date: Mon, 28 Jul 2014 23:58:12 +0200
Subject: [PATCH]
Hello,
In the last days I effectively rewrote the rectification filter found in the
frei0r suite as a libavfilter video filter and would like to submit this to
ffmpeg. It probably needs some cleanup, which I will do by checking your commit
guidelines, but I would like to get a first opinion als
Following up, this is a patch for web/news kickstarting the fundraiser
for OPW. OK to commit?
Once posted, it would be great if you can set aside 2 minutes to
tweet/blogpost/whatever about it so it gets noticed. We need all the
$ help we can get.
Bests,
--
Reynaldo
>From 857556764e2c3c09437bac
On Mon, Jul 28, 2014 at 10:54:47PM +0200, Andreas Cadhalpun wrote:
> On 28.07.2014 20:51, Michael Niedermayer wrote:
> >On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote:
> >>On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote:
> >>>On Mon, Jul 28, 2014 at 04:30:02AM
On Mon, Jul 28, 2014 at 07:21:37PM +0200, Christophe Gisquet wrote:
> 2014-07-23 21:20 GMT+02:00 Christophe Gisquet :
> > Didn't refresh the patch with update documentation. Here it is.
>
> It didn't contain dummy functions in libavcodec/utils.c.
>
> The attached patch should fix failures to buil
On 28.07.2014 20:51, Michael Niedermayer wrote:
On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote:
On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote:
On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote:
[...]
in absence of further comments i mi
Oliver Fromme fromme.com> writes:
> That's why I appreciate the patch very much
> that Carl Eugen has created.
Did you test the patch?
I think it will not get applied without a test.
[...]
> Maybe I'll try to make a patch that can do this
If you consider to invest time in DVD reading,
please
On Thu, Jul 24, 2014 at 03:39:43PM +0200, Nicolas George wrote:
> Also deprecate adding a NULL frame to mark EOF.
>
> TODO APIchanges entry, version bump.
>
[...]
> diff --git a/libavfilter/buffersrc.h b/libavfilter/buffersrc.h
> index ea34c04..28ca545 100644
> --- a/libavfilter/buffersrc.h
> +++
On Thu, Jul 24, 2014 at 03:39:45PM +0200, Nicolas George wrote:
> This makes the FPS filter duplicate the last frame to take
> its duration into account, exactly like the other ones.
>
> TODO find corresponding trac ticket(s).
>
> Signed-off-by: Nicolas George
> ---
> libavfilter/vf_fps.c | 59
On Thu, Jul 24, 2014 at 03:39:42PM +0200, Nicolas George wrote:
[...]
> @@ -1162,6 +1205,17 @@ int ff_filter_frame(AVFilterLink *link, AVFrame *frame)
> }
> }
>
> +int ff_filter_link_close(AVFilterLink *link, int64_t pts)
> +{
> +AVFrame frame = { };
Breaks ABI, AVFrame size is depend
On Mon, Jul 28, 2014 at 04:17:35PM -0300, James Almer wrote:
> Reading 8 bytes is enough.
>
> Signed-off-by: James Almer
> ---
> libavcodec/x86/hevc_deblock.asm | 16
> 1 file changed, 8 insertions(+), 8 deletions(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF
On Mon, Jul 28, 2014 at 04:17:34PM -0300, James Almer wrote:
> This fixes a warning spam when using NASM
>
> Signed-off-by: James Almer
> ---
> libavcodec/x86/hevc_idct.asm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B
Reading 8 bytes is enough.
Signed-off-by: James Almer
---
libavcodec/x86/hevc_deblock.asm | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/libavcodec/x86/hevc_deblock.asm b/libavcodec/x86/hevc_deblock.asm
index f7cd031..ecebd36 100644
--- a/libavcodec/x86/hevc
This fixes a warning spam when using NASM
Signed-off-by: James Almer
---
libavcodec/x86/hevc_idct.asm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/x86/hevc_idct.asm b/libavcodec/x86/hevc_idct.asm
index 2ad6e97..481726d 100644
--- a/libavcodec/x86/hevc_idct.asm
+
Am Montag 28 Juli 2014, 07:00:12 schrieb Frantisek Korbel:
> Good day to all,
> congratulation to ffmpeg.org new website, now information is better
> structured and users can find data easily.
> One question is a dark background - initially I also used it on my web,
> because it is easier to read w
> Could you maybe add a comment to where this comes from/where it is
> specified, and if you know possible future plan?
Myself and others from Opus/Videolan etc are drafting a specification
for Opus in TS:
https://people.xiph.org/~tterribe/opus/ETSI_TS_opus-v0.1.2-draft.doc
We are registered with
On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote:
> On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote:
> > On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote:
[...]
> in absence of further comments i might apply this patch to
> master and release/2
On Mon, Jul 28, 2014 at 08:33:14PM +0200, Hendrik Leppkes wrote:
> Am 28.07.2014 04:30 schrieb "Michael Niedermayer" :
> > diff --git a/libavutil/version.h b/libavutil/version.h
> > index 6d8d6f0..1deb6e4 100644
> > --- a/libavutil/version.h
> > +++ b/libavutil/version.h
> > @@ -138,7 +138,7 @@
> >
On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote:
> On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote:
> > This works around ABI issues with applications which depend on libavutil
> > internal values like sizeof(AVFrame)
> > One such application is VLC 2.1.4 as wel
Am 28.07.2014 04:30 schrieb "Michael Niedermayer" :
>
> This works around ABI issues with applications which depend on libavutil
> internal values like sizeof(AVFrame)
> One such application is VLC 2.1.4 as well as 2.1.5
>
> Found-by: Andreas Cadhalpun
> Signed-off-by: Michael Niedermayer
> ---
>
On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote:
> This works around ABI issues with applications which depend on libavutil
> internal values like sizeof(AVFrame)
> One such application is VLC 2.1.4 as well as 2.1.5
As you said yourself, we have been and will be relying on this
On Sun, Jul 27, 2014 at 01:23:34AM +0100, Kieran Kunhya wrote:
> ---
> libavcodec/opus.c| 11 +---
> libavcodec/opus.h|9 +++
> libavcodec/opus_parser.c | 139
> +-
> libavcodec/opusdec.c |1 +
> libavformat/mpegts.c |
With patch.
From 0b5d56a563df7efdfcf206013680510c62e2569c Mon Sep 17 00:00:00 2001
From: Christophe Gisquet
Date: Mon, 28 Jul 2014 11:48:21 +0200
Subject: [PATCH 10/15] x86: hevc_mc: fix register count usage
A macro was using a fixed register, causing too many GPRs to be
declared as used.
---
li
Le decadi 10 thermidor, an CCXXII, Clément Bœsch a écrit :
> It's a static website, how are you going to make the selection persistent?
I believe javascript can do that. I blame the browser developers for not
providing a cleaner way of doing it.
Regards,
--
Nicolas George
On Mon, Jul 28, 2014 at 09:46:31AM -0800, Lou Logan wrote:
> On Mon, 28 Jul 2014 07:00:12 -0700, Frantisek Korbel wrote:
>
> > For this reasons I strongly recommend to set the white background
> > please try it from the 1st August.
>
> If you are interested you could supply a patch to ffmpeg-web
On Mon, Jul 28, 2014 at 4:00 PM, Frantisek Korbel
wrote:
> For this reasons I strongly recommend to set the white background
> please try it from the 1st August.
My main inspiration was this website I really like: http://nodejs.org/
We can add a switch option using javascript to use a white styl
On Mon, 28 Jul 2014 07:00:12 -0700, Frantisek Korbel wrote:
> For this reasons I strongly recommend to set the white background
> please try it from the 1st August.
If you are interested you could supply a patch to ffmpeg-web to
somehow implement a user selectable CSS to enable a light background
On Mon, Jul 28, 2014 at 07:00:12AM -0700, Frantisek Korbel wrote:
> Good day to all,
> congratulation to ffmpeg.org new website, now information is better
> structured and users can find data easily.
> One question is a dark background - initially I also used it on my web,
> because it is easier to
Hi,
the count is obviously inflated. This patch is actually required for
"[PATCH 0/4] Exploit compile-time constant" (figures there are
produced used this patch).
--
Christophe
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mai
2014-07-23 21:20 GMT+02:00 Christophe Gisquet :
> Didn't refresh the patch with update documentation. Here it is.
It didn't contain dummy functions in libavcodec/utils.c.
The attached patch should fix failures to build with pthread disabled.
--
Christophe
From 4a144438e5dcd66be173b2d23f310bb4ea
The dststride parameter is always MAX_PB_SIZE.
---
libavcodec/hevc.c | 4 ++--
libavcodec/hevcdsp.h | 4 ++--
libavcodec/hevcdsp_template.c | 27 +--
libavcodec/x86/hevc_mc.asm| 16
libavcodec/x86/hevcdsp.h | 2 +-
libavcode
---
libavcodec/hevc.c | 8 +++
libavcodec/hevcdsp.h | 8 +++
libavcodec/hevcdsp_template.c | 56 +--
libavcodec/x86/hevc_mc.asm| 18 +++---
libavcodec/x86/hevcdsp.h | 6 ++---
libavcodec/x86/hevcdsp_init.c | 1
The stride for some buffers is known.
---
libavcodec/x86/hevc_mc.asm | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/x86/hevc_mc.asm b/libavcodec/x86/hevc_mc.asm
index ea4447b..2c34927 100644
--- a/libavcodec/x86/hevc_mc.asm
+++ b/libavcodec/x86/hevc_mc.asm
@@ -1
MAX_PB_SIZE is used or assumed for various buffer strides. In some cases,
it is used as constant parameter(s) to functions.
Make use of that knowledge to:
- not pass the parameter
- avoid extra GPR usage
- precompute addresses / offsets
Premature optimization and overall not that useful.
Before:
---
libavcodec/hevc.h| 1 -
libavcodec/hevcdsp.h | 2 ++
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavcodec/hevc.h b/libavcodec/hevc.h
index 8420f38..0369d8f 100644
--- a/libavcodec/hevc.h
+++ b/libavcodec/hevc.h
@@ -56,7 +56,6 @@
#define MAX_TRANSFORM_DEPTH 5
#define
Le decadi 10 thermidor, an CCXXII, Frantisek Korbel a écrit :
> - All most visited webs like Facebook, Google, Wikipedia, Yahoo etc.
> use black text on white background: http://www.alexa.com/topsites
> - Some sensitive people can avoid to visit ffmpeg.org only because
> of its black background
> -
On Mon, 28 Jul 2014 07:00:12 -0700
"Frantisek Korbel" wrote:
> - Dark background often use warez and other illegal webs, so some
lol.
thanks for your opinion. i am not against changing bg color, but i just
thought this was hilarious.
-compn
___
ffmp
Good day to all,
congratulation to ffmpeg.org new website, now information is better
structured and users can find data easily.
One question is a dark background - initially I also used it on my web,
because it is easier to read when ambient light is low and also it
saves energy - monitor powers mu
Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit :
> Ah! Thank you very much for pointing me to the concat demuxer.
> I wasn't aware that it can be (ab)used to declare the presence
> of streams in an input file. I will definitely give that a try.
Please let me know of any issue.
> Well,
Hi,
2014-07-27 19:18 GMT+02:00 Michael Niedermayer :
>> It is unconditionally set to 64, which is quite higher than the
>> actual block size.
>
> is this faster?
No, which surprises me, but so be it.
On the other hand, if this stride is PB_MAX_SIZE throughout the code,
then maybe its value could
---
libavcodec/opus.c| 11 +---
libavcodec/opus.h|9 +++
libavcodec/opus_parser.c | 138 +-
libavcodec/opusdec.c |1 +
libavformat/mpegts.c | 54 ++
5 files changed, 192 insertions(+), 21 deletions(-)
On 28.07.2014 12:20, Thorsten Glaser wrote:
Andreas Cadhalpun wrote:
* Do you intend to replace Libav by FFmpeg in Debian?
No, there is no need to replace anything as long as it is maintained.
Currently the main goal is to give multimedia maintainers a choice
between the two sets
On Sun, Jul 27, 2014 at 08:08:09PM -0700, Timothy Gu wrote:
> On Sun, Jul 27, 2014 at 5:49 PM, Michael Niedermayer wrote:
> > release branches are detected by checking if "git" is not in RELEASE
> > This changes "N-64706-g2f71aeb" to "n2.3-8-g2f71aeb"
> > for git master theres no change
> >
> > Th
Andreas Cadhalpun wrote:
> * Do you intend to replace Libav by FFmpeg in Debian?
>No, there is no need to replace anything as long as it is maintained.
>Currently the main goal is to give multimedia maintainers a choice
>between the two sets of libraries to link against, and our users
Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit :
> There's another problem: The palette of the subtitle streams
> is contained in the IFO files, too. So, in order to get the
> palette right, my script uses "mencoder -vobsubout", then
> greps the palette from the resulting .idx file and
Le decadi 10 thermidor, an CCXXII, Eli Kara a écrit :
> It would solve the problem for DVDs with structure, which is probably what
> most people have when they rip a DVD.
Certainly. But the other scenarios where the user only have a big MPEG-PS
file and nothing else, are likely too.
> Just out of
From: ePirat
Fix header.
---
Add Icecast protocol, a convenience wrapper for the HTTP protocol
---
Changelog| 3 +
configure| 1 +
doc/general.texi | 1 +
doc/protocols.texi | 42 ++
libavformat/Makefile | 1 +
libavformat/allfor
From: ePirat
Fix: Last password char was removed by accident.
Code cleanup.
---
Add Icecast protocol, a convenience wrapper for the HTTP protocol
---
Changelog| 3 +
configure| 1 +
doc/general.texi | 1 +
doc/protocols.texi | 42 ++
> From: ffmpeg-devel-boun...@ffmpeg.org
> [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Nicolas George
> Sent: Monday, July 28, 2014 1:10 AM
> That would be nice, thanks for proposing. But that would not fix the issue:
> not all MPEG-PS files come with a DVD structure, even ones with subt
60 matches
Mail list logo
