On Wed, Apr 6, 2011 at 6:35 PM, rukhsana afroz wrote:
>
>
> On Wed, Apr 6, 2011 at 9:24 AM, Michael Niedermayer wrote:
>
>> I suspect the If statement is there as the implementation only supports
>> cblk_style==0
>> the others simply are not implemented
>>
>> Do you think you can implement them?
On Mon, Apr 11, 2011 at 10:22:31PM -0700, Alex Converse wrote:
> $subj
> From c104a6e90226eaa253aeba471c8859e0da922da7 Mon Sep 17 00:00:00 2001
> From: Michael Niedermayer
> Date: Mon, 11 Apr 2011 11:49:07 +0200
> Subject: [PATCH] oggdec: use av_freep() instead of av_free()
> Signed-off-by: Mich
$subj
From c104a6e90226eaa253aeba471c8859e0da922da7 Mon Sep 17 00:00:00 2001
From: Michael Niedermayer
Date: Mon, 11 Apr 2011 11:49:07 +0200
Subject: [PATCH] oggdec: use av_freep() instead of av_free()
Signed-off-by: Michael Niedermayer
---
libavformat/oggdec.c |4 ++--
1 files changed, 2
On Mon, Apr 11, 2011 at 9:49 PM, Alex Converse wrote:
> On Sat, Apr 9, 2011 at 5:48 PM, Luca Barbato wrote:
>> On 04/10/2011 02:41 AM, Alex Converse wrote:
>>> Passes make fate, altivec untested.
>>
>> Altivec should not have problems (the variable is already unsigned).
>>
>> lu
>>
>> --
>
> push
On Mon, Mar 28, 2011 at 3:36 AM, Anton Khirnov wrote:
> ---
> libavcodec/Makefile | 1 +
> libavformat/mp3enc.c | 74 ++---
> 2 files changed, 70 insertions(+), 5 deletions(-)
>
> diff --git a/libavcodec/Makefile b/libavcodec/Makefile
> index 85d9
On Mon, Apr 11, 2011 at 1:34 AM, Luca Barbato wrote:
>> (1<<31) is undefined and seems to be evaluated by gcc to -2^31 when
>> these formulae require 2^31.
>> ---
>> libavcodec/audioconvert.c | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> Looks ok, put a note about adding te
On Sat, Apr 9, 2011 at 5:48 PM, Luca Barbato wrote:
> On 04/10/2011 02:41 AM, Alex Converse wrote:
>> Passes make fate, altivec untested.
>
> Altivec should not have problems (the variable is already unsigned).
>
> lu
>
> --
pushed
___
libav-devel maili
On Mon, Apr 11, 2011 at 2:24 PM, Luca Barbato wrote:
> On 04/11/2011 11:14 PM, Manjunath Siddaiah wrote:
>> Hi,
>>
>> When we do "-acodec copy" if input is "mpegts" and output is non-mpegts
>> ( MOV, MP4, 3GP, FLV, ISM) ADTS header also got copied.
>> AAC audio do not need ADTS header in MOV, MP4
On Sat, Apr 09, 2011 at 05:10:35PM -0400, Ronald S. Bultje wrote:
[...]
> > -@item
> > - Did you provide a suggestion for a clear commit log message?
>
> I don't think this is right. Git still doesn't write your log messages
> for you. You could reword it as "did you provide a clear git commit
On Sun, Apr 10, 2011 at 12:52:57AM +0200, Stefano Sabatini wrote:
> [CC-ing ffmpeg-devel]
>
> On date Saturday 2011-04-09 12:35:58 +0200, Anton Khirnov wrote:
> > On Sat, Apr 09, 2011 at 02:47:35AM +0200, Stefano Sabatini wrote:
> > > From b563c12cb285f1e6eb8dc19d1a18323cd9280ea1 Mon Sep 17 00:00:
On 04/11/2011 11:14 PM, Manjunath Siddaiah wrote:
> Hi,
>
> When we do "-acodec copy" if input is "mpegts" and output is non-mpegts
> ( MOV, MP4, 3GP, FLV, ISM) ADTS header also got copied.
> AAC audio do not need ADTS header in MOV, MP4, 3GP, FLV, ISM containers.
> This patch skips the ADTS head
Hi,
When we do "-acodec copy" if input is "mpegts" and output is non-mpegts ( MOV,
MP4, 3GP, FLV, ISM) ADTS header also got copied. AAC audio do not need ADTS
header in MOV, MP4, 3GP, FLV, ISM containers. This patch skips the ADTS header
copying to non-mpegts formats.
-Manjunath H Siddaiah
On 04/11/2011 10:34 PM, Luca Barbato wrote:
> Right, they are leftovers...
updated my github branch.
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/li
On 04/11/2011 10:21 PM, Aℓex Converse wrote:
> On Mon, Apr 11, 2011 at 1:01 PM, Luca Barbato wrote:
>> It is used only for x86 so it will be moved once the code is moved in
>> the arch dir
>>
>> Question: should we build an accurate BGR2YV12 variant?
>> ---
>> libswscale/rgb2rgb.c |4
-Original Message-
From: libav-devel-boun...@libav.org [mailto:libav-devel-boun...@libav.org] On
Behalf Of Kostya
Sent: Thursday, March 24, 2011 3:50 AM
To: libav development
Subject: Re: [libav-devel] [PATCH] RTMP streaming crashes after some time with
Flash Media Server (FMS), Issue 223
On Mon, Apr 11, 2011 at 1:01 PM, Luca Barbato wrote:
> It is used only for x86 so it will be moved once the code is moved in
> the arch dir
>
> Question: should we build an accurate BGR2YV12 variant?
> ---
> libswscale/rgb2rgb.c | 4 ++--
> libswscale/swscale.c | 2 --
> l
From: Josh Allmann
Simplify the fallthrough case when no accelerated functions
can be initialized.
---
libswscale/swscale.c |9 ++---
1 files changed, 2 insertions(+), 7 deletions(-)
diff --git a/libswscale/swscale.c b/libswscale/swscale.c
index e830bb6..277a0d3 100644
--- a/libswscale/
From: Josh Allmann
Move x86-specific constants out of swscale.c
---
libswscale/swscale.c | 57 -
libswscale/x86/swscale_template.c | 54 +++
2 files changed, 54 insertions(+), 57 deletions(-)
diff --git a/libsw
It is used only for x86 so it will be moved once the code is moved in
the arch dir
Question: should we build an accurate BGR2YV12 variant?
---
libswscale/rgb2rgb.c |4 ++--
libswscale/swscale.c |2 --
libswscale/swscale_internal.h |2 ++
3 files changed, 4 insertions
I squashed the patches and reworded the commit messages.
I hope they look a little better, now.
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On 04/11/2011 07:03 PM, Diego Biurrun wrote:
> On Mon, Apr 11, 2011 at 09:34:11AM -0700, Aℓex Converse wrote:
>> On Mon, Apr 11, 2011 at 3:06 AM, Diego Biurrun wrote:
>>> On Mon, Apr 11, 2011 at 11:07:38AM +0200, Luca Barbato wrote:
---
libswscale/rgb2rgb_template.c | 28 +
On Mon, Apr 11, 2011 at 09:34:11AM -0700, Aℓex Converse wrote:
> On Mon, Apr 11, 2011 at 3:06 AM, Diego Biurrun wrote:
> > On Mon, Apr 11, 2011 at 11:07:38AM +0200, Luca Barbato wrote:
> >> ---
> >> libswscale/rgb2rgb_template.c | 28 ++--
> >> 1 files changed, 14 insert
On Mon, Apr 11, 2011 at 3:06 AM, Diego Biurrun wrote:
> On Mon, Apr 11, 2011 at 11:07:38AM +0200, Luca Barbato wrote:
>> ---
>> libswscale/rgb2rgb_template.c | 28 ++--
>> 1 files changed, 14 insertions(+), 14 deletions(-)
>
> Why? The log message is descriptive, but no
On Mon, Apr 11, 2011 at 09:44:49AM -0400, Ronald S. Bultje wrote:
>
> On Sun, Apr 10, 2011 at 3:44 PM, Diego Biurrun wrote:
> > When HAVE_7REGS was not defined these functions had an empty body
> > causing the following warnings during compilation.
> > In file included from libswscale/x86/yuv2rgb
On Mon, Apr 11, 2011 at 10:06:43AM -0400, Ronald S. Bultje wrote:
> Hi,
>
> On Mon, Apr 11, 2011 at 10:01 AM, Luca Barbato wrote:
> > On 04/11/2011 03:44 PM, Ronald S. Bultje wrote:
> >> This isn't right. You're declaring empty functions for stuff that is
> >> supposed to do something. In x86/yuv
Hi,
On Mon, Apr 11, 2011 at 10:01 AM, Luca Barbato wrote:
> On 04/11/2011 03:44 PM, Ronald S. Bultje wrote:
>> This isn't right. You're declaring empty functions for stuff that is
>> supposed to do something. In x86/yuv2rgb_mmx.c, you'll see these
>> functions are only called under if (HAVE_7REGS
On 04/11/2011 03:44 PM, Ronald S. Bultje wrote:
> This isn't right. You're declaring empty functions for stuff that is
> supposed to do something. In x86/yuv2rgb_mmx.c, you'll see these
> functions are only called under if (HAVE_7REGS) anyway, so the proper
> fix is to not declare these functions a
Hi,
On Sun, Apr 10, 2011 at 3:44 PM, Diego Biurrun wrote:
> When HAVE_7REGS was not defined these functions had an empty body
> causing the following warnings during compilation.
> In file included from libswscale/x86/yuv2rgb_mmx.c:58:
> libswscale/x86/yuv2rgb_template.c: In function ‘yuva420_rgb
On Mon, Apr 11, 2011 at 11:25:49AM +0200, Luca Barbato wrote:
> On 04/10/2011 09:44 PM, Diego Biurrun wrote:
> > When HAVE_7REGS was not defined these functions had an empty body
> > causing the following warnings during compilation.
> > In file included from libswscale/x86/yuv2rgb_mmx.c:58:
> > li
On Sun, Apr 10, 2011 at 09:54:56PM +0200, Luca Barbato wrote:
> On 04/10/2011 09:46 PM, Diego Biurrun wrote:
> > The printf function is forbidden in library code, but not in test programs,
> > so #undefine the printf macro that disables the system printf function.
>
> ok
Pushed.
Diego
__
On Mon, Apr 11, 2011 at 01:48:04PM +0200, Luca Barbato wrote:
> On 04/11/2011 12:16 PM, Diego Biurrun wrote:
> > On Mon, Apr 11, 2011 at 11:07:33AM +0200, Luca Barbato wrote:
> >> Hi this patchset does still apply today.
> >> Even if not complete I'm considering to merge it since seems not
> >> int
On Mon, 11 Apr 2011, Luca Barbato wrote:
> On 04/08/2011 12:21 AM, Martin Storsjö wrote:
> > Changelog |1 +
> > configure |6 ++
> > doc/general.texi |2 +-
> > libavcodec/Makefile |1 +
> > libavcodec/allcodecs.c|1 +
> > li
On Mon, 11 Apr 2011, Luca Barbato wrote:
> On 04/08/2011 12:22 AM, Martin Storsjö wrote:
>
> > The wrapper code is based on the libamr wrapper removed in SVN rev 19365.
> > ---
> > Changelog |1 +
> > configure |6 ++
> > doc/general.texi|
On 04/08/2011 12:21 AM, Martin Storsjö wrote:
> Changelog |1 +
> configure |6 ++
> doc/general.texi |2 +-
> libavcodec/Makefile |1 +
> libavcodec/allcodecs.c|1 +
> libavcodec/libvo-aacenc.c | 128
> ++
On 04/08/2011 12:22 AM, Martin Storsjö wrote:
> The wrapper code is based on the libamr wrapper removed in SVN rev 19365.
> ---
> Changelog |1 +
> configure |6 ++
> doc/general.texi|3 +-
> libavcodec/Makefile |1 +
> libav
On 04/11/2011 12:16 PM, Diego Biurrun wrote:
> On Mon, Apr 11, 2011 at 11:07:33AM +0200, Luca Barbato wrote:
>> Hi this patchset does still apply today.
>> Even if not complete I'm considering to merge it since seems not
>> introduce changes and already moves most of the x86 code in its
>> director
On 04/11/2011 12:09 PM, Diego Biurrun wrote:
> On Mon, Apr 11, 2011 at 11:07:43AM +0200, Luca Barbato wrote:
>> ---
>> libswscale/swscale.c | 13 +
>> 1 files changed, 5 insertions(+), 8 deletions(-)
>
> "Fixup" what? Please be more descriptive.
>
> Diego
> ___
On Mon, Apr 11, 2011 at 11:07:42AM +0200, Luca Barbato wrote:
> Always call first the C init and make the optional optimization
> init follow.
> The unaccelerated function pointer shouldn't be set again,
> likewise the other common logic should be present only on the C
> init.
Factorize initiali
On Mon, Apr 11, 2011 at 11:07:41AM +0200, Luca Barbato wrote:
> move all the functions to the _c suffix
Please add the rationale for this change to the log message :)
> --- a/libswscale/swscale_template.c
> +++ b/libswscale/swscale_template.c
> @@ -18,7 +18,7 @@
>
> -static inline void RENAME(y
On Mon, Apr 11, 2011 at 11:26:16AM +0200, Kostya wrote:
> On Mon, Apr 11, 2011 at 11:07:39AM +0200, Luca Barbato wrote:
> > With this commit we should have all the arch specific code moved away.
> > Init pattern now:
> > - generic C init first
> > - arch specific init later overwriting
> >
> > ---
On Mon, Apr 11, 2011 at 11:07:39AM +0200, Luca Barbato wrote:
> With this commit we should have all the arch specific code moved away.
> Init pattern now:
> - generic C init first
> - arch specific init later overwriting
>
> --- a/libswscale/rgb2rgb.c
> +++ b/libswscale/rgb2rgb.c
> @@ -130,21 +51,
On Mon, Apr 11, 2011 at 11:07:33AM +0200, Luca Barbato wrote:
> Hi this patchset does still apply today.
> Even if not complete I'm considering to merge it since seems not
> introduce changes and already moves most of the x86 code in its
> directory.
A general remark: I find the log messages a bit
On Mon, Apr 11, 2011 at 11:07:37AM +0200, Luca Barbato wrote:
> ---
> libswscale/rgb2rgb.c |9 +-
> libswscale/rgb2rgb_template.c | 174
> 2 files changed, 91 insertions(+), 92 deletions(-)
I'd be happy to see you explain the reason for this
On Mon, Apr 11, 2011 at 11:07:43AM +0200, Luca Barbato wrote:
> ---
> libswscale/swscale.c | 13 +
> 1 files changed, 5 insertions(+), 8 deletions(-)
"Fixup" what? Please be more descriptive.
Diego
___
libav-devel mailing list
libav-deve
On Mon, Apr 11, 2011 at 11:07:38AM +0200, Luca Barbato wrote:
> ---
> libswscale/rgb2rgb_template.c | 28 ++--
> 1 files changed, 14 insertions(+), 14 deletions(-)
Why? The log message is descriptive, but not explanatory...
> --- a/libswscale/rgb2rgb_template.c
> +++ b
On Mon, Apr 11, 2011 at 11:07:34AM +0200, Luca Barbato wrote:
> Move the function pointers in a separate file.
> It will removed once the pointers will be factored in a more structured
> form
>
> --- /dev/null
> +++ b/libswscale/swscale_functions.c
> @@ -0,0 +1,67 @@
> +#include
stdint.h is enou
On Mon, Apr 11, 2011 at 11:07:47AM +0200, Luca Barbato wrote:
> From: Josh Allmann
looks ok
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On Mon, Apr 11, 2011 at 11:07:46AM +0200, Luca Barbato wrote:
> From: Josh Allmann
ok
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On Mon, Apr 11, 2011 at 11:07:45AM +0200, Luca Barbato wrote:
> That makes the output consistent, to be investigated if it could be
> avoided. Thanks to Josh for spotting it.
> ---
isn't the whole context zeroed before initialisation? But in general looks ok
___
On Mon, Apr 11, 2011 at 11:07:44AM +0200, Luca Barbato wrote:
> This unbreak darwin.
unbreakS
Also maybe you should merge it with patch 1 then?
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On Mon, Apr 11, 2011 at 11:07:43AM +0200, Luca Barbato wrote:
> ---
> libswscale/swscale.c | 13 +
> 1 files changed, 5 insertions(+), 8 deletions(-)
ok
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/lis
On Mon, Apr 11, 2011 at 11:07:42AM +0200, Luca Barbato wrote:
> Always call first the C init and make the optional optimization
> init follow.
> The unaccelerated function pointer shouldn't be set again,
> likewise the other common logic should be present only on the C
> init.
looks reasonable
___
On Mon, Apr 11, 2011 at 11:07:41AM +0200, Luca Barbato wrote:
> move all the functions to the _c suffix
> ---
> libswscale/swscale_template.c | 130
> 1 files changed, 65 insertions(+), 65 deletions(-)
looks ok but somebody would also ask breaking those l
On Mon, Apr 11, 2011 at 11:07:39AM +0200, Luca Barbato wrote:
> With this commit we should have all the arch specific code moved away.
> Init pattern now:
> - generic C init first
> - arch specific init later overwriting
>
> In future the arch specific init will overwrite just the generics for
> w
On 04/10/2011 09:44 PM, Diego Biurrun wrote:
> When HAVE_7REGS was not defined these functions had an empty body
> causing the following warnings during compilation.
> In file included from libswscale/x86/yuv2rgb_mmx.c:58:
> libswscale/x86/yuv2rgb_template.c: In function ‘yuva420_rgb32_MMX’:
> libs
On Mon, Apr 11, 2011 at 11:07:38AM +0200, Luca Barbato wrote:
> ---
> libswscale/rgb2rgb_template.c | 28 ++--
> 1 files changed, 14 insertions(+), 14 deletions(-)
Could you also deniedermayerize it? I find "var= value" style improper.
But in general ok.
On Mon, Apr 11, 2011 at 11:07:37AM +0200, Luca Barbato wrote:
> ---
> libswscale/rgb2rgb.c |9 +-
> libswscale/rgb2rgb_template.c | 174
> 2 files changed, 91 insertions(+), 92 deletions(-)
looks ok
___
On Mon, Apr 11, 2011 at 11:13:29AM +0200, Kostya wrote:
> On Mon, Apr 11, 2011 at 11:07:35AM +0200, Luca Barbato wrote:
> > It is used only for x86 so it will be moved once the code is moved in
> > the arch dir
> >
> > Question: should we build an accurate BGR2YV12 variant?
>
> maybe when we have
On Mon, Apr 11, 2011 at 11:07:35AM +0200, Luca Barbato wrote:
> It is used only for x86 so it will be moved once the code is moved in
> the arch dir
>
> Question: should we build an accurate BGR2YV12 variant?
maybe when we have optimisations for it
___
On Mon, Apr 11, 2011 at 11:07:34AM +0200, Luca Barbato wrote:
> Move the function pointers in a separate file.
> It will removed once the pointers will be factored in a more structured
> form
> ---
probably ok (and having YUV-to-YUV conversion in rgb2rgb.c is somehow a bit
baffling)
__
From: Josh Allmann
---
libswscale/swscale.c |9 ++---
1 files changed, 2 insertions(+), 7 deletions(-)
diff --git a/libswscale/swscale.c b/libswscale/swscale.c
index e830bb6..277a0d3 100644
--- a/libswscale/swscale.c
+++ b/libswscale/swscale.c
@@ -1211,8 +1211,6 @@ SwsFunc ff_getSwsFunc
From: Josh Allmann
---
libswscale/swscale.c | 57 -
libswscale/x86/swscale_template.c | 54 +++
2 files changed, 54 insertions(+), 57 deletions(-)
diff --git a/libswscale/swscale.c b/libswscale/swscale.c
index
That makes the output consistent, to be investigated if it could be
avoided. Thanks to Josh for spotting it.
---
libswscale/x86/swscale_template.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/libswscale/x86/swscale_template.c
b/libswscale/x86/swscale_template.c
index
This unbreak darwin.
---
libswscale/Makefile|2 +-
libswscale/rgb2rgb.c | 65 ++
libswscale/swscale_functions.c | 67
3 files changed, 66 insertions(+), 68 deletions(-)
delete mode 100644 li
---
libswscale/swscale.c | 13 +
1 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/libswscale/swscale.c b/libswscale/swscale.c
index 8d8c625..fe96141 100644
--- a/libswscale/swscale.c
+++ b/libswscale/swscale.c
@@ -1197,10 +1197,7 @@ static inline void monoblack2Y(uint8_
Always call first the C init and make the optional optimization
init follow.
The unaccelerated function pointer shouldn't be set again,
likewise the other common logic should be present only on the C
init.
---
libswscale/swscale.c |6 +--
libswscale/x86/swscale_template.c | 73 +
move all the functions to the _c suffix
---
libswscale/swscale_template.c | 130
1 files changed, 65 insertions(+), 65 deletions(-)
diff --git a/libswscale/swscale_template.c b/libswscale/swscale_template.c
index c983fc0..cac9725 100644
--- a/libswscale/s
With this commit we should have all the arch specific code moved away.
Init pattern now:
- generic C init first
- arch specific init later overwriting
In future the arch specific init will overwrite just the generics for
which we have an optimization.
---
libswscale/Makefile |3 +-
libsw
---
libswscale/rgb2rgb_template.c | 28 ++--
1 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/libswscale/rgb2rgb_template.c b/libswscale/rgb2rgb_template.c
index 30c3ad2..69f0683 100644
--- a/libswscale/rgb2rgb_template.c
+++ b/libswscale/rgb2rgb_template.
---
libswscale/rgb2rgb.c |9 +-
libswscale/rgb2rgb_template.c | 174
2 files changed, 91 insertions(+), 92 deletions(-)
diff --git a/libswscale/rgb2rgb.c b/libswscale/rgb2rgb.c
index dd34c9d..e6d7971 100644
--- a/libswscale/rgb2rgb.c
+++ b/li
It is used only for x86 so it will be moved once the code is moved in
the arch dir
Question: should we build an accurate BGR2YV12 variant?
---
libswscale/rgb2rgb.c |2 --
libswscale/swscale.c |2 --
libswscale/swscale_internal.h |2 ++
3 files changed, 2 insertions(+
Move the function pointers in a separate file.
It will removed once the pointers will be factored in a more structured
form
---
libswscale/Makefile|2 +-
libswscale/rgb2rgb.c | 66 ---
libswscale/swscale_functions.c | 67 +++
Hi this patchset does still apply today.
Even if not complete I'm considering to merge it since seems not
introduce changes and already moves most of the x86 code in its
directory.
You can fork it from my libav repo on github
https://github.com/lu-zero/libav/tree/swscale-cleanup
Many thanks to J
On Sun, Apr 10, 2011 at 09:44:51PM +0200, Diego Biurrun wrote:
> When HAVE_7REGS was not defined these functions had an empty body
> causing the following warnings during compilation.
> In file included from libswscale/x86/yuv2rgb_mmx.c:58:
> libswscale/x86/yuv2rgb_template.c: In function ‘yuva420_
On Mon, Apr 11, 2011 at 09:40:32AM +0400, Konstantin Pavlov wrote:
>
> --- a/libavutil/libm.h
> +++ b/libavutil/libm.h
> @@ -38,6 +38,14 @@
>
> +#if !HAVE_ISNAN
> +#undef isnan
> +#define isnan(x) \
> + (sizeof (x) == sizeof (long double) ? __builtin_isnanl ((long double)(x))
> : \
> + size
On Sun, 10 Apr 2011, Kostya wrote:
> On Sun, Apr 10, 2011 at 08:39:43PM +0300, Martin Storsjö wrote:
> > This fixes a failing assert in ff_raw_read_header (in
> > fate-g722dec-1), where bits_per_coded_sample is set using this
> > function and is required to have a positive value.
> > ---
>
> yes,
> (1<<31) is undefined and seems to be evaluated by gcc to -2^31 when
> these formulae require 2^31.
> ---
> libavcodec/audioconvert.c |4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
Looks ok, put a note about adding tests.
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu
On 04/10/2011 11:59 PM, Kieran Kunhya wrote:
> Blackmagic hardware SDI cards output V210 with a custom stride.
> This patch allows libav to decode these streams.
>
How to test it? Not sure we need it as global avcodec parameter, as
avoption might be nicer.
lu
--
Luca Barbato
Gentoo/linux
htt
78 matches
Mail list logo