The Opus decoder in particular uses optimised float_dsp functions that expect
32-byte alignment
---
libavcodec/avcodec.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index 93aad35..5cbc175 100644
--- a/libavcodec/avcodec.h
+++
One step in untangling the mpegvideo code and fixing some problems in
the order that initialization is being done in h263dec and h261dec.
---
libavcodec/dnxhdenc.c | 2 +-
libavcodec/h261dec.c | 4 +++-
libavcodec/h263dec.c | 8 ++--
libavcodec/mpeg12dec.c | 5 +
l
ff_MPV_common_init requires the frame dimensions which get parsed in
h261_decode_picture_header.
---
libavcodec/h261dec.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index c3c3ff6..8c34bb5 100644
--- a/libavcodec/h261de
ff_MPV_common_init requires the frame dimensions which get parsed in
*_decode_picture_header.
---
libavcodec/h263dec.c | 25 -
1 file changed, 12 insertions(+), 13 deletions(-)
diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c
index d0e9f4d..e0e5600 100644
--- a/lib
On 08/09/2014 03:19 PM, John Stebbins wrote:
> On 08/09/2014 01:42 PM, Diego Biurrun wrote:
>> On Sat, Aug 09, 2014 at 09:25:36AM -0700, John Stebbins wrote:
>>> --- a/libavcodec/mpegvideo.c
>>> +++ b/libavcodec/mpegvideo.c
>>> @@ -379,7 +379,6 @@ av_cold int ff_dct_common_init(MpegEncContext *s)
>
On 08/09/2014 01:42 PM, Diego Biurrun wrote:
> On Sat, Aug 09, 2014 at 09:25:36AM -0700, John Stebbins wrote:
>> --- a/libavcodec/mpegvideo.c
>> +++ b/libavcodec/mpegvideo.c
>> @@ -379,7 +379,6 @@ av_cold int ff_dct_common_init(MpegEncContext *s)
>> {
>> ff_blockdsp_init(&s->bdsp, s->avctx);
---
Updated according the review, the AVDictionary helper will appear later,
now avresample_config() does not include avresample_open().
doc/APIchanges | 6 +++
libavresample/avresample.h | 69
libavresample/utils.c | 131 +
On 08/09/2014 04:37 PM, Luca Barbato wrote:
On 02/08/14 01:58, Luca Barbato wrote:
Otherwise trailing zeroes would appear.
---
This patch requires to update the reference files.
Shall I upload a v3?
libavfilter/af_resample.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavfilter/af
On Sat, Aug 09, 2014 at 09:25:37AM -0700, John Stebbins wrote:
> ff_MPV_common_init requires the frame dimensions which gets parsed in
get_
same typo in 3/3
> *_decode_picture_header.
> ---
> libavcodec/h263dec.c | 25 -
> 1 file changed, 12 insertions(+), 13 deletions(-
On Sat, Aug 09, 2014 at 09:25:36AM -0700, John Stebbins wrote:
> --- a/libavcodec/mpegvideo.c
> +++ b/libavcodec/mpegvideo.c
> @@ -379,7 +379,6 @@ av_cold int ff_dct_common_init(MpegEncContext *s)
> {
> ff_blockdsp_init(&s->bdsp, s->avctx);
> ff_hpeldsp_init(&s->hdsp, s->avctx->flags);
>
On 02/08/14 01:58, Luca Barbato wrote:
> Otherwise trailing zeroes would appear.
> ---
>
> This patch requires to update the reference files.
> Shall I upload a v3?
>
> libavfilter/af_resample.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/libavfilter/af_resample.c b/libavfilter/af_
On Sat, Aug 09, 2014 at 10:48:29PM +0300, Martin Storsjö wrote:
> ---
> This goes on top of Diego's RFC patch.
> ---
> configure | 9 +
> 1 file changed, 5 insertions(+), 4 deletions(-)
LGTM
Diego
___
libav-devel mailing list
libav-devel@libav.
On Sat, Aug 09, 2014 at 10:48:28PM +0300, Martin Storsjö wrote:
> ---
> configure | 4
> 1 file changed, 4 insertions(+)
OK
Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On Sat, 9 Aug 2014, Martin Storsjö wrote:
---
configure | 4
1 file changed, 4 insertions(+)
diff --git a/configure b/configure
index 7e3f6ff..4d2439e 100755
--- a/configure
+++ b/configure
@@ -2881,6 +2881,10 @@ probe_cc(){
_cflags_size="-O2 -Munroll=c:1 $opt_common"
_cflag
On 09/08/14 21:48, Martin Storsjo wrote:
> This allows doing a simple no-op to test that gas-preprocessor
> exists and works, since armasm doesn't have a -v option.
> ---
> gas-preprocessor.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gas-preprocessor.pl b/gas-prepr
On 09/08/14 21:48, Martin Storsjö wrote:
> ---
> configure | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/configure b/configure
> index 7e3f6ff..4d2439e 100755
> --- a/configure
> +++ b/configure
> @@ -2881,6 +2881,10 @@ probe_cc(){
> _cflags_size="-O2 -Munroll=c:1 $opt_com
This allows doing a simple no-op to test that gas-preprocessor
exists and works, since armasm doesn't have a -v option.
---
gas-preprocessor.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl
index 86293d1..e60de8c 100755
--- a/gas-prep
Since plain 'cpp' is used for preprocessing, it inherits whatever
platform specific predefined defines.
Previously this was set on the libav configure line, such as
--as='gas-preprocessor.pl -as-type armasm -- armasm -nologo -U__ELF__ -ignore
4509',
but if gas-preprocessor is to be enabled automa
---
This goes on top of Diego's RFC patch.
---
configure | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/configure b/configure
index 4d2439e..877f240 100755
--- a/configure
+++ b/configure
@@ -3782,17 +3782,18 @@ if enabled_any arm aarch64 || enabled_all ppc altivec
&
---
configure | 4
1 file changed, 4 insertions(+)
diff --git a/configure b/configure
index 7e3f6ff..4d2439e 100755
--- a/configure
+++ b/configure
@@ -2881,6 +2881,10 @@ probe_cc(){
_cflags_size="-O2 -Munroll=c:1 $opt_common"
_cflags_noopt="-O1"
_flags_filter=pgi_
On Sat, 9 Aug 2014, Janne Grunau wrote:
On 2014-08-09 08:06:12 -0700, Diego Biurrun wrote:
---
Implemented Janne's suggestion. Untested since I don't have the right system.
configure | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/configure b/configure
i
On 09/08/14 19:48, John Stebbins wrote:
> +
> +if (!s->context_initialized)
> +if ((ret = ff_MPV_common_init(s)) < 0)
> +return ret;
> +
> if (!s->context_initialized) {
> ret = ff_set_dimensions(avctx, s->width, s->height);
> if (ret < 0)
>
Thanks
On Sat, 9 Aug 2014, Janne Grunau wrote:
'#' at the start of the line indicates on all archs the start of a
comment or a preprocessor directive. This is after preprocessing so
removing the lines as comment is safe. Line number comments are stripped
by this too since gas-preprocessor.pl ignores li
On Thu, 31 Jul 2014, Andrew Stone wrote:
Currently, only onMetadata is used, but some providers (wrongly)
put metadata into onCuePoint events, and it's still nice to be
able to use that data.
---
libavformat/flvdec.c | 82 +++-
1 file changed, 42 in
On Thu, 31 Jul 2014, Andrew Stone wrote:
It won't hurt servers that don't care about the header,
and those that do will include it by default.
---
Changelog | 1 +
doc/protocols.texi | 2 +-
libavformat/http.c | 2 +-
3 files changed, 3 insertions(+), 2 deletions(-)
I'm slightly sceptica
---
libavformat/mpegtsenc.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index 457b2e7..b210b02 100644
--- a/libavformat/mpegtsenc.c
+++ b/libavformat/mpegtsenc.c
@@ -165,7 +165,7 @@ static int mpegts_write_section
ff_MPV_common_init requires the frame dimensions which gets parsed in
h261_decode_picture_header.
---
libavcodec/h261dec.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index dcd7c54..d57f5ab 100644
--- a/libavcodec/h261d
On 09/08/14 18:25, John Stebbins wrote:
> One step in untangling the mpegvideo code and fixing some problems in
> the order that initialization is being done in h263dec and h261dec.
> ---
> libavcodec/dnxhdenc.c | 2 +-
> libavcodec/h261dec.c | 4 +++-
> libavcodec/h263dec.c |
On 09/08/14 18:25, John Stebbins wrote:
> ff_MPV_common_init requires the frame dimensions which gets parsed in
> h261_decode_picture_header.
> ---
> libavcodec/h261dec.c | 10 ++
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.
On 09/08/14 17:47, Janne Grunau wrote:
> '#' at the start of the line indicates on all archs the start of a
> comment or a preprocessor directive. This is after preprocessing so
> removing the lines as comment is safe. Line number comments are stripped
> by this too since gas-preprocessor.pl ignore
On 09/08/14 16:21, Felix Abecassis wrote:
> Ping. I think both options you mentioned (safety vs hiding missing
> initialization) would be valid. Did you have time to think which one
> would be best?
>
> If you do plan to store large arrays in side data like Anton
> mentioned, then I guess I can se
On Fri, 8 Aug 2014 18:21:03 +, Christophe Gisquet
wrote:
> The position is either rounded or not checked, so delay the wait to
> check the proper value.
>
> Reviewed-by: Mickaël Raulet
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/hevc_mvs.c | 3 ++-
> 1 file changed, 2 insert
On Sat, Aug 09, 2014 at 05:30:25PM +0200, Janne Grunau wrote:
> On 2014-08-09 08:06:12 -0700, Diego Biurrun wrote:
> > ---
> >
> > Implemented Janne's suggestion. Untested since I don't have the right
> > system.
> >
> > configure | 15 ---
> > 1 file changed, 12 insertions(+), 3 d
On Fri, 8 Aug 2014 18:21:01 +, Christophe Gisquet
wrote:
> The merge list only needs to be derived up to the merge index.
>
> Signed-off-by: Michael Niedermayer
>
> Conflicts:
> libavcodec/hevc_mvs.c
> ---
> libavcodec/hevc_mvs.c | 35 ++-
> 1 file
On Fri, 8 Aug 2014 18:21:00 +, Christophe Gisquet
wrote:
> When the candidate has been found, no need to derive others.
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/hevc_mvs.c | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavcodec/hevc_mvs.c b/libavcodec/h
On Fri, 8 Aug 2014 18:20:59 +, Christophe Gisquet
wrote:
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/hevc_mvs.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/hevc_mvs.c b/libavcodec/hevc_mvs.c
> index 2fe4dbb..f930f55 100644
> ---
ff_MPV_common_init requires the frame dimensions which gets parsed in
*_decode_picture_header.
---
libavcodec/h263dec.c | 25 -
1 file changed, 12 insertions(+), 13 deletions(-)
diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c
index f0b275b..0ef5ef8 100644
--- a/li
ff_MPV_common_init requires the frame dimensions which gets parsed in
h261_decode_picture_header.
---
libavcodec/h261dec.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index dcd7c54..613e1f0 100644
--- a/libavcodec/h261d
This patch set fixes a problem that occurs with h263 and h261 decoding
when not using libavformat for demux. Initialization order is incorrect
and results in incorrect decoding.
The reason decoding works when libavformat is used for demux is
AVCodecContext widht/height gets initialized during avf
One step in untangling the mpegvideo code and fixing some problems in
the order that initialization is being done in h263dec and h261dec.
---
libavcodec/dnxhdenc.c | 2 +-
libavcodec/h261dec.c | 4 +++-
libavcodec/h263dec.c | 8 ++--
libavcodec/mpeg12dec.c | 5 +
l
On Sat, Aug 9, 2014 at 4:42 PM, Anton Khirnov wrote:
> From: Michael Niedermayer
>
> Such changes are not allowed nor supported
>
> Found-by: ami_stuff
> Signed-off-by: Anton Khirnov
> ---
> libavcodec/ffv1dec.c | 32 +---
> 1 file changed, 25 insertions(+), 7 deleti
On Fri, 8 Aug 2014 18:20:58 +, Christophe Gisquet
wrote:
> iterate over memory in a more continuous order
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/hevc.c | 12 ++--
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/libavcodec/hevc.c b/libavcodec
On Sat, 9 Aug 2014 08:40:13 -0700, Diego Biurrun wrote:
> From: Reinhard Tartler
>
> ---
> libavcodec/libgsm.c | 1 +
> libavcodec/libopencore-amr.c | 1 +
> libavcodec/libspeexdec.c | 1 +
> 3 files changed, 3 insertions(+)
>
> diff --git a/libavcodec/libgsm.c b/libavcodec/libg
'#' at the start of the line indicates on all archs the start of a
comment or a preprocessor directive. This is after preprocessing so
removing the lines as comment is safe. Line number comments are stripped
by this too since gas-preprocessor.pl ignores line numbers. So it
removes just misleading d
From: Michael Niedermayer
Such changes are not allowed nor supported
Found-by: ami_stuff
Signed-off-by: Anton Khirnov
---
libavcodec/ffv1dec.c | 32 +---
1 file changed, 25 insertions(+), 7 deletions(-)
diff --git a/libavcodec/ffv1dec.c b/libavcodec/ffv1dec.c
index
From: Reinhard Tartler
---
libavcodec/libgsm.c | 1 +
libavcodec/libopencore-amr.c | 1 +
libavcodec/libspeexdec.c | 1 +
3 files changed, 3 insertions(+)
diff --git a/libavcodec/libgsm.c b/libavcodec/libgsm.c
index 1098bce..a1fbb55 100644
--- a/libavcodec/libgsm.c
+++ b/libavcodec
On 2014-08-09 08:06:12 -0700, Diego Biurrun wrote:
> ---
>
> Implemented Janne's suggestion. Untested since I don't have the right system.
>
> configure | 15 ---
> 1 file changed, 12 insertions(+), 3 deletions(-)
>
> diff --git a/configure b/configure
> index 46b8391..7c2fab6 1007
---
Implemented Janne's suggestion. Untested since I don't have the right system.
configure | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/configure b/configure
index 46b8391..7c2fab6 100755
--- a/configure
+++ b/configure
@@ -2199,8 +2199,6 @@ ranlib="ranl
On 08/09/2014 10:36 AM, wm4 wrote:
On Sat, 09 Aug 2014 10:26:16 -0400
Justin Ruggles wrote:
On 08/02/2014 10:56 AM, Luca Barbato wrote:
---
+int avresample_config(AVAudioResampleContext *avr, AVFrame *out, AVFrame *in,
+ AVDictionary **opts)
+{
+int ret;
+
+if (av
On Sat, 09 Aug 2014 10:26:16 -0400
Justin Ruggles wrote:
> On 08/02/2014 10:56 AM, Luca Barbato wrote:
> > ---
> > +int avresample_config(AVAudioResampleContext *avr, AVFrame *out, AVFrame
> > *in,
> > + AVDictionary **opts)
> > +{
> > +int ret;
> > +
> > +if (avres
On 08/02/2014 10:56 AM, Luca Barbato wrote:
---
doc/APIchanges | 4 ++
libavresample/avresample.h | 71
libavresample/utils.c | 134 +
libavresample/version.h| 2 +-
libavutil/error.h |
Ping. I think both options you mentioned (safety vs hiding missing
initialization) would be valid. Did you have time to think which one
would be best?
If you do plan to store large arrays in side data like Anton
mentioned, then I guess I can send a patch to fix STEREO3D
initialization instead.
20
On 31/07/14 11:16, Luca Barbato wrote:
> This way is possible to enable features requiring pkg-config checked
> libraries without having pkg-config installed.
> ---
> configure | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/configure b/configure
> index 03742f0..
On 08/09/2014 09:19 AM, Diego Elio Pettenò wrote:
Instead of simulating a grayscale palette, use real grayscale pixels, if no
palette is actually defined.
Signed-off-by: Diego Elio Pettenò
---
libavcodec/tiff.c | 14 +++---
1 file changed, 3 insertions(+), 11 deletions(-)
diff --git
Instead of simulating a grayscale palette, use real grayscale pixels, if no
palette is actually defined.
Signed-off-by: Diego Elio Pettenò
---
libavcodec/tiff.c | 14 +++---
1 file changed, 3 insertions(+), 11 deletions(-)
diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c
index 2aff45a
On 2014-08-06 12:24:24 +0200, Diego Biurrun wrote:
> On Tue, Aug 05, 2014 at 01:35:22PM +0200, Janne Grunau wrote:
> > From: Janne Grunau
> > --- a/configure
> > +++ b/configure
> > @@ -3760,22 +3760,32 @@ unsigned int endian = 'B' << 24 | 'I' << 16 | 'G'
> > << 8 | 'E';
> >
> > -
> > -if enabl
On 2014-08-08 14:58:39 +0200, Diego Biurrun wrote:
> On Thu, Aug 07, 2014 at 06:39:33PM +0200, Diego Biurrun wrote:
> > On Wed, Aug 06, 2014 at 04:41:24PM +0200, Janne Grunau wrote:
> > > On 2014-08-06 05:57:39 -0700, Diego Biurrun wrote:
> > >
> > > the commit message describes the commit only pa
The automated script did not perfectly replace all the instances nor
added internal.h in all the files requiring it.
---
And obviously doing the whole thing at 4am did not help.
libavcodec/mxpegdec.c| 1 +
libavcodec/pthread.c | 2 +-
libavcodec/roqvideoenc.c | 1 +
3 files changed, 3 in
From: Reinhard Tartler
The transition to ff_get_buffer() requires the header "internal.h",
which was missed in two files
Signed-off-by: Reinhard Tartler
---
libavcodec/mxpegdec.c| 1 +
libavcodec/roqvideoenc.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/libavcodec/mxpegdec.c b/li
On 09/08/14 08:46, Nidhi Makhijani wrote:
> ---
> libavformat/mpegtsenc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
> index 457b2e7..cdaf276 100644
> --- a/libavformat/mpegtsenc.c
> +++ b/libavformat/mpegtsenc.c
> @@ -
60 matches
Mail list logo
