It was done only in check_mvset(), while mv_scale() is called also by
dist_scale().
Sample-Id: 1579-google
Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/hevc_mvs.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a
On Fri, Feb 07, 2014 at 03:41:34PM -0500, Andrew Kelley wrote:
> --- a/libavformat/oggenc.c
> +++ b/libavformat/oggenc.c
> @@ -616,7 +616,15 @@ AVOutputFormat ff_ogg_muxer = {
> +#ifdef CONFIG_LIBVORBIS_ENCODER
> +.audio_codec = AV_CODEC_ID_VORBIS,
> +#else
> .audio_codec = AV_
Here's a cleaner version.
--
Christophe
From 9a8381fef85b50bb8034ea650f20a3868a146e3c Mon Sep 17 00:00:00 2001
From: Christophe Gisquet
Date: Wed, 19 Dec 2012 20:26:05 +0100
Subject: [PATCH 04/10] x86: dcadsp: implement SSE lfe_dir
Results for Arrandale/Windows:
32: 1670 -> 316
64: 728 -> 298
For the record, patch dropped.
--
Christophe
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On Fri, 7 Feb 2014, Janne Grunau wrote:
Hi,
whoever pushes "[PATCH 03/11] dcadsp: split lfe_dir cases" should
squash this patch into it. Original patch is ok this one needs review.
Janne
--->0---
---
libavcodec/arm/dcadsp_init_arm.c | 46
libavcodec/arm/
2014-02-07 Janne Grunau :
> whoever pushes "[PATCH 03/11] dcadsp: split lfe_dir cases" should
> squash this patch into it. Original patch is ok this one needs review.
I can't actually review, but from I kind of see, the only saving is
not doing a test/conditional assignment/jump inside of the func
Hi,
whoever pushes "[PATCH 03/11] dcadsp: split lfe_dir cases" should
squash this patch into it. Original patch is ok this one needs review.
Janne
--->0---
---
libavcodec/arm/dcadsp_init_arm.c | 46
libavcodec/arm/dcadsp_neon.S | 18 ++--
2014-02-07 Janne Grunau :
> Do you have someone who's keen to review x86 asm?
Actually I got a review from Loren, who pointed the following
improvements/fixes:
- the SSE yasm function was using a SSE2 instruction, which is corrected
- the macro parameter was the number of SSE regs, which is unnece
Hi,
2014-02-06 Christophe Gisquet :
> 2014-02-06 Diego Biurrun :
>> Why did you place this inside fft? A separate init file sounds more
>> logical to me.
>
> A year ago, before my work was delayed, that's where it was.
>
> arm now places it in arm/dcadsp_init_arm.c, that's where I'd put for
> con
On 02/07/2014 03:41 PM, Andrew Kelley wrote:
Since 2007, the Xipth.org Foundation recommends that .ogg only be used
for Ogg Vorbis audio files.
typo. it's Xiph.org not Xipth.org
Source: http://wiki.xiph.org/index.php/MIME_Types_and_File_Extensions
However we only do it if we have libvorbis a
On 2014-02-07 21:57:08 +0100, Christophe Gisquet wrote:
> From 87983deb56aa52c2cdcfbf248dd76bccb97d694a Mon Sep 17 00:00:00 2001
> From: Christophe Gisquet
> Date: Fri, 11 May 2012 11:25:30 +0200
> Subject: [PATCH 02/10] x86: dcadsp: implement int8x8_fmul_int32
>
> For the callable function (as o
On 2014-02-07 21:51:02 +0100, Christophe Gisquet wrote:
> From c233b96fa29150b0385776499e90afc9b59405b5 Mon Sep 17 00:00:00 2001
> From: Christophe Gisquet
> Date: Fri, 11 May 2012 11:17:36 +0200
> Subject: [PATCH 01/10] dcadsp: add int8x8_fmul_int32 to DSP context
>
> It is currently declared as
Hi,
here's an updated version also taking into account the changes
suggested for first patch of the series.
--
Christophe
From 87983deb56aa52c2cdcfbf248dd76bccb97d694a Mon Sep 17 00:00:00 2001
From: Christophe Gisquet
Date: Fri, 11 May 2012 11:25:30 +0200
Subject: [PATCH 02/10] x86: dcadsp: imp
Hi,
2014-02-06 Luca Barbato :
>> I though more of something like below which is smaller and has imho
>> clearer preprocessor use.
>>
>> Janne
>>
>
> av_unused might be added.
>
> That way it should not make a difference between Christophe macro and
> this speedwise.
Following your suggestion, her
Since 2007, the Xipth.org Foundation recommends that .ogg only be used
for Ogg Vorbis audio files.
Source: http://wiki.xiph.org/index.php/MIME_Types_and_File_Extensions
However we only do it if we have libvorbis available because the
built in vorbis encoder is not very good.
---
libavformat/ogge
On 2014-02-07 21:00:03 +0100, Diego Biurrun wrote:
> On Fri, Feb 07, 2014 at 07:24:28PM +0100, Janne Grunau wrote:
> > ---
> > configure | 12
> > 1 file changed, 12 insertions(+)
>
> Why? A little bit more explanation would be appreciated.
>
> > --- a/configure
> > +++ b/configure
>
On Fri, Feb 07, 2014 at 07:24:28PM +0100, Janne Grunau wrote:
> ---
> configure | 12
> 1 file changed, 12 insertions(+)
Why? A little bit more explanation would be appreciated.
> --- a/configure
> +++ b/configure
> @@ -4208,6 +4208,18 @@ elif enabled_any msvc icl; then
> fi
>
On Fri, 7 Feb 2014, Janne Grunau wrote:
On 2014-02-07 20:40:43 +0200, Martin Storsjö wrote:
On Fri, 7 Feb 2014, Janne Grunau wrote:
---
configure | 12
1 file changed, 12 insertions(+)
diff --git a/configure b/configure
index 7a305c3..abee247 100755
--- a/configure
+++ b/configur
On 2014-02-07 21:03:12 +0200, Martin Storsjö wrote:
> This avoids interpreting instructions such as e.g. bic as
> a branch with the condition 'ic'.
> ---
> Updated to check the condition against the list of conditions
> from ARM ARM.
> ---
> gas-preprocessor.pl | 15 ++-
> 1 file chang
This avoids interpreting instructions such as e.g. bic as
a branch with the condition 'ic'.
---
Updated to check the condition against the list of conditions
from ARM ARM.
---
gas-preprocessor.pl | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/gas-preprocessor.
On Fri, Feb 7, 2014 at 2:00 PM, Luca Barbato wrote:
> We could enable vorbis output if the libvorbis encoder is available,
> otherwise fallback to flac.
Sounds like a good compromise. And since the debian package of libav
depends on libvorbis, this will effectively produce the desired behavior
On 07/02/14 19:31, Andrew Kelley wrote:
> On Fri, Feb 7, 2014 at 1:07 PM, Justin Ruggles
> wrote:
>
>> As you can see from .extensions, the ogg muxer isn't only used for .ogg
>> files (although maybe we could add .oga to that list). FLAC was chosen for
>> the default because it is lossless and we
On 2014-02-07 20:40:43 +0200, Martin Storsjö wrote:
> On Fri, 7 Feb 2014, Janne Grunau wrote:
>
> >---
> >configure | 12
> >1 file changed, 12 insertions(+)
> >
> >diff --git a/configure b/configure
> >index 7a305c3..abee247 100755
> >--- a/configure
> >+++ b/configure
> >@@ -4208,6 +
On 2/7/2014 6:39 PM, Kostya Shishkov wrote:
>> +
>> +/* Header + at least once slice (4) */
>
> one slice
Fixed locally. I won't bother sending a new patch for such a
trivial change.
- Derek
___
libav-devel mailing list
libav-devel@libav.org
https
On Fri, 7 Feb 2014, Janne Grunau wrote:
---
configure | 12
1 file changed, 12 insertions(+)
diff --git a/configure b/configure
index 7a305c3..abee247 100755
--- a/configure
+++ b/configure
@@ -4208,6 +4208,18 @@ elif enabled_any msvc icl; then
fi
fi
+case $ld_type in
+clan
On Fri, Feb 07, 2014 at 06:32:01PM +, Derek Buitenhuis wrote:
> From: Kostya Shishkov
>
> Does not contain cursor rendering yet.
>
> Signed-off-by: Derek Buitenhuis
> ---
> All review comments from Kostya and Diego have been addressed.
> ---
> Changelog | 1 +
> configure
From: Kostya Shishkov
Does not contain cursor rendering yet.
Signed-off-by: Derek Buitenhuis
---
All review comments from Kostya and Diego have been addressed.
---
Changelog | 1 +
configure | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
On Fri, Feb 7, 2014 at 1:07 PM, Justin Ruggles wrote:
> As you can see from .extensions, the ogg muxer isn't only used for .ogg
> files (although maybe we could add .oga to that list). FLAC was chosen for
> the default because it is lossless and we have a good native encoder. An
> alternative coul
---
configure | 12
1 file changed, 12 insertions(+)
diff --git a/configure b/configure
index 7a305c3..abee247 100755
--- a/configure
+++ b/configure
@@ -4208,6 +4208,18 @@ elif enabled_any msvc icl; then
fi
fi
+case $ld_type in
+clang)
+ check_ldflags -Qunused-argu
On 2014-02-07 19:12:30 +0100, Diego Biurrun wrote:
> On Fri, Feb 07, 2014 at 06:01:01PM +, Janne Grunau wrote:
> > On 2014-02-07 18:45:50 +0100, Diego Biurrun wrote:
> > > On Fri, Feb 07, 2014 at 06:39:38PM +0100, Janne Grunau wrote:
> > > > --- a/libavcodec/dcadec.c
> > > > +++ b/libavcodec/dc
On 07 Feb 2014, at 19:12, Diego Biurrun wrote:
> On Fri, Feb 07, 2014 at 06:01:01PM +, Janne Grunau wrote:
>> On 2014-02-07 18:45:50 +0100, Diego Biurrun wrote:
>>> On Fri, Feb 07, 2014 at 06:39:38PM +0100, Janne Grunau wrote:
--- a/libavcodec/dcadec.c
+++ b/libavcodec/dcadec.c
>>>
On Fri, Feb 07, 2014 at 06:01:01PM +, Janne Grunau wrote:
> On 2014-02-07 18:45:50 +0100, Diego Biurrun wrote:
> > On Fri, Feb 07, 2014 at 06:39:38PM +0100, Janne Grunau wrote:
> > > --- a/libavcodec/dcadec.c
> > > +++ b/libavcodec/dcadec.c
> > > @@ -1587,7 +1587,12 @@ static int dca_exss_parse
On 02/07/2014 12:43 PM, Andrew Kelley wrote:
Since 2007, the Xipth.org Foundation recommends that .ogg only be used
for Ogg Vorbis audio files.
Source: http://wiki.xiph.org/index.php/MIME_Types_and_File_Extensions
---
libavformat/oggenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
On 2014-02-07 18:45:50 +0100, Diego Biurrun wrote:
> On Fri, Feb 07, 2014 at 06:39:38PM +0100, Janne Grunau wrote:
> > --- a/libavcodec/dcadec.c
> > +++ b/libavcodec/dcadec.c
> > @@ -1587,7 +1587,12 @@ static int dca_exss_parse_asset_header(DCAContext *s)
> > case 2: extensions_mask = DCA_EXT_
On Fri, Feb 07, 2014 at 06:39:38PM +0100, Janne Grunau wrote:
> --- a/libavcodec/dcadec.c
> +++ b/libavcodec/dcadec.c
> @@ -1587,7 +1587,12 @@ static int dca_exss_parse_asset_header(DCAContext *s)
> case 2: extensions_mask = DCA_EXT_EXSS_LBR; break;
> -case 3: extensions_mask = 0; /* a
Since 2007, the Xipth.org Foundation recommends that .ogg only be used
for Ogg Vorbis audio files.
Source: http://wiki.xiph.org/index.php/MIME_Types_and_File_Extensions
---
libavformat/oggenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/oggenc.c b/libavformat/
---
libavcodec/dcadec.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c
index 66eb03f..0624209 100644
--- a/libavcodec/dcadec.c
+++ b/libavcodec/dcadec.c
@@ -1587,7 +1587,12 @@ static int dca_exss_parse_asset_header(DCAContext *s)
On Fri, Feb 07, 2014 at 04:50:55PM +, Derek Buitenhuis wrote:
> From: Kostya Shishkov
>
> Does not contain cursor rendering yet.
>
> Signed-off-by: Derek Buitenhuis
> ---
> Changelog | 1 +
> libavcodec/Makefile | 1 +
> libavcodec/allcodecs.c | 1 +
> libavcodec/a
On Fri, Feb 07, 2014 at 04:50:55PM +, Derek Buitenhuis wrote:
> From: Kostya Shishkov
>
> Does not contain cursor rendering yet.
>
> Signed-off-by: Derek Buitenhuis
> ---
> Changelog | 1 +
> libavcodec/Makefile | 1 +
> libavcodec/allcodecs.c | 1 +
> libavcodec/a
From: Kostya Shishkov
Does not contain cursor rendering yet.
Signed-off-by: Derek Buitenhuis
---
Changelog | 1 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 7 ++
libavcodec/fic.c| 311 +
On 2/7/2014 4:39 PM, Derek Buitenhuis wrote:
> +int slice_off = AV_RB32(src + tsize + 0x1B + slice * 4);
This is changed fro 0x1B to 27 locally.
- Derek
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo
From: Kostya Shishkov
Does not contain cursor rendering yet.
Signed-off-by: Derek Buitenhuis
---
Changelog | 1 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 7 ++
libavcodec/fic.c| 309 +
On 2014-02-07 17:01:06 +0200, Martin Storsjö wrote:
>
> >> +if ($cond ne "ic") {
> >> +if (exists $thumb_labels{$label}) {
> >> +print ASMFILE ".thumb_func $label\n";
> >> +} else {
> >> +$call_targets{$label}++;
> >> +}
>
On February 7, 2014 4:51:19 PM EET, Janne Grunau wrote:
>On 2014-02-07 16:31:38 +0200, Martin Storsjö wrote:
>
>Do not instead of Don't in the commit msg
>
>> ---
>> This doesn't help much right now, but in a number of experimental
>> branches I've added things in this clause, and I've had to ha
On 2014-02-07 16:31:38 +0200, Martin Storsjö wrote:
Do not instead of Don't in the commit msg
> ---
> This doesn't help much right now, but in a number of experimental
> branches I've added things in this clause, and I've had to hack
> around misidentifying bic as a branch in all of them.
> ---
>
If linking in an object file without this attribute set, the
linker will assume that an executable stack might be needed.
---
libavutil/arm/asm.S | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavutil/arm/asm.S b/libavutil/arm/asm.S
index f4523ea..45fdf55 100644
--- a/libavutil/arm/asm.S
++
---
This doesn't help much right now, but in a number of experimental
branches I've added things in this clause, and I've had to hack
around misidentifying bic as a branch in all of them.
---
gas-preprocessor.pl | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/
---
libavcodec/h264.c | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/libavcodec/h264.c b/libavcodec/h264.c
index da2d4a5..628dbcd 100644
--- a/libavcodec/h264.c
+++ b/libavcodec/h264.c
@@ -4787,7 +4787,9 @@ again:
break;
case
On Fri, Feb 7, 2014 at 1:47 PM, Reinhard Tartler wrote:
> On Thu, Feb 6, 2014 at 11:24 PM, Reinhard Tartler wrote:
>>
>> Depending on the feedback, I'd suggest to decide next weekend what to
>> do next.
>
> For avoidance of any doubt, I'm not referring to the weekend starting
> tomorrow, but the
On 2014-02-07 14:03:59 +0200, Martin Storsjö wrote:
> On Fri, 7 Feb 2014, Janne Grunau wrote:
>
> >On 2014-02-07 12:06:56 +0200, Martin Storsjö wrote:
> >>Don't rely on the fact that an unprefixed label currently exists.
> >>---
> >> libavcodec/arm/fmtconvert_neon.S | 2 +-
> >> libavcodec/a
On Thu, Feb 6, 2014 at 11:24 PM, Reinhard Tartler wrote:
>
> Depending on the feedback, I'd suggest to decide next weekend what to
> do next.
For avoidance of any doubt, I'm not referring to the weekend starting
tomorrow, but the week after, that is, February 8.
--
regards,
Reinhard
__
On 2014-02-07 11:51:18 +0100, Vittorio Giovara wrote:
> ---
> libavcodec/h264.c | 19 ++-
> 1 file changed, 14 insertions(+), 5 deletions(-)
>
> diff --git a/libavcodec/h264.c b/libavcodec/h264.c
> index da2d4a5..0cbe944 100644
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
>
On Fri, 7 Feb 2014, Janne Grunau wrote:
On 2014-02-07 12:06:56 +0200, Martin Storsjö wrote:
Don't rely on the fact that an unprefixed label currently exists.
---
libavcodec/arm/fmtconvert_neon.S | 2 +-
libavcodec/arm/h264idct_neon.S | 16
libavcodec/arm/hpeldsp
On 2014-02-07 12:06:57 +0200, Martin Storsjö wrote:
> This makes the generated assembly more internally consistent,
> avoiding declaring two labels for the same function (for cases
> where EXTERN_ASM is empty) and not declaring a separate unprefixed
> label in other cases.
>
> This also makes sure
On 2014-02-07 12:06:56 +0200, Martin Storsjö wrote:
> Don't rely on the fact that an unprefixed label currently exists.
> ---
> libavcodec/arm/fmtconvert_neon.S | 2 +-
> libavcodec/arm/h264idct_neon.S | 16
> libavcodec/arm/hpeldsp_armv6.S | 4 ++--
> liba
On 07 Feb 2014, at 01:28, Anton Khirnov wrote:
>
> On Thu, 6 Feb 2014 23:30:22 +0100, Tim Walker wrote:
>> On 04 Feb 2014, at 22:34, Anton Khirnov wrote:
>>
>>> ---
>>> This patch is dedicated to av500's tshirt
>>> ---
>>> avplay.c| 33 +
>>> doc/avpla
---
libavcodec/h264.c | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/libavcodec/h264.c b/libavcodec/h264.c
index da2d4a5..0cbe944 100644
--- a/libavcodec/h264.c
+++ b/libavcodec/h264.c
@@ -4787,7 +4787,9 @@ again:
break;
case
This makes the generated assembly more internally consistent,
avoiding declaring two labels for the same function (for cases
where EXTERN_ASM is empty) and not declaring a separate unprefixed
label in other cases.
This also makes sure the .func and .type delcarations have the same
prefix. They hav
Don't rely on the fact that an unprefixed label currently exists.
---
libavcodec/arm/fmtconvert_neon.S | 2 +-
libavcodec/arm/h264idct_neon.S | 16
libavcodec/arm/hpeldsp_armv6.S | 4 ++--
libavcodec/arm/mdct_neon.S | 2 +-
libavcodec/arm/mpegv
On Fri, Feb 07, 2014 at 07:48:42AM +0100, Kostya Shishkov wrote:
> On Thu, Feb 06, 2014 at 09:30:16PM +0100, Luca Barbato wrote:
> > On 06/02/14 21:26, Tim Walker wrote:
> >
> > > Also, regarding the commit message, any opinions regarding "444" vs.
> > > "4:4:4"?
> > >
> >
> > I have no opinion
60 matches
Mail list logo
