Re: [libav-devel] [PATCH] h2645_parse: Allocate a single buffer per packet

2017-11-03 Thread Luca Barbato
On 03/11/2017 20:52, James Almer wrote: From: Kieran Kunhya Drastically reduces memory usage on pathological streams. --- Fixed so it may apply cleanly, and with the memset() change removed. Thank you :) I'm happy with it now, anybody is against landing it? lu

Re: [libav-devel] PATCH] h2645: Allocate a single buffer per packet. Drastically reduces memory usage on pathological streams.

2017-11-03 Thread James Almer
On 11/3/2017 5:04 PM, Diego Biurrun wrote: > On Fri, Nov 03, 2017 at 04:50:56PM -0300, James Almer wrote: >> On 11/3/2017 4:42 PM, Luca Barbato wrote: >>> On 03/11/2017 19:23, Kieran Kunhya wrote: This patch fixes very high memory usage on pathological streams. >>> >>> this hunk seems

Re: [libav-devel] PATCH] h2645: Allocate a single buffer per packet. Drastically reduces memory usage on pathological streams.

2017-11-03 Thread Diego Biurrun
On Fri, Nov 03, 2017 at 04:50:56PM -0300, James Almer wrote: > On 11/3/2017 4:42 PM, Luca Barbato wrote: > > On 03/11/2017 19:23, Kieran Kunhya wrote: > >> This patch fixes very high memory usage on pathological streams. > > > > this hunk seems spurious (and should not even compile with gcc). >

[libav-devel] [PATCH] h2645_parse: Allocate a single buffer per packet

2017-11-03 Thread James Almer
From: Kieran Kunhya Drastically reduces memory usage on pathological streams. --- Fixed so it may apply cleanly, and with the memset() change removed. libavcodec/h2645_parse.c | 22 -- libavcodec/h2645_parse.h | 10 -- libavcodec/h264_parser.c | 11

Re: [libav-devel] PATCH] h2645: Allocate a single buffer per packet. Drastically reduces memory usage on pathological streams.

2017-11-03 Thread James Almer
On 11/3/2017 4:42 PM, Luca Barbato wrote: > On 03/11/2017 19:23, Kieran Kunhya wrote: >> This patch fixes very high memory usage on pathological streams. > > this hunk seems spurious (and should not even compile with gcc). It does, but complains about mixed declarations and code. It's in any

Re: [libav-devel] PATCH] h2645: Allocate a single buffer per packet. Drastically reduces memory usage on pathological streams.

2017-11-03 Thread Luca Barbato
On 03/11/2017 19:23, Kieran Kunhya wrote: This patch fixes very high memory usage on pathological streams. this hunk seems spurious (and should not even compile with gcc). diff --git a/libavcodec/h264_parse.c b/libavcodec/h264_parse.c index a7c71d9..2ddbbf9 100644 ---

[libav-devel] PATCH] h2645: Allocate a single buffer per packet. Drastically reduces memory usage on pathological streams.

2017-11-03 Thread Kieran Kunhya
This patch fixes very high memory usage on pathological streams. 0001-h2645-Allocate-a-single-buffer-per-packet.-Drastical.patch Description: Binary data ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH 4/5] configure: Improve restrict keyword handling

2017-11-03 Thread Diego Biurrun
--- configure| 23 +-- libavcodec/binkdsp.h | 2 -- libavcodec/dnxhdenc.h| 2 -- libavcodec/h263.h| 2 -- libavcodec/idctdsp.h | 2 -- libavcodec/pixblockdsp.h | 2 -- libavutil/float_dsp.h| 2 -- 7 files changed, 5 insertions(+),

[libav-devel] [PATCH 5/5] Drop some unnecessary config.h #includes

2017-11-03 Thread Diego Biurrun
--- Noticed while working on the previous patch, spun off into a separate commit. libavcodec/thread.h | 1 - libavformat/tls.h| 4 ++-- libavutil/aarch64/cpu.h | 1 - libavutil/arm/cpu.h | 1 - libavutil/cpu_internal.h | 2 ++ libavutil/ppc/cpu.h | 1 -

[libav-devel] the big configure cleanup (act V)

2017-11-03 Thread Diego Biurrun
miscellaneous minor changes and refactorings ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 2/5] configure: Coalesce some arch configuration and PIC handling

2017-11-03 Thread Diego Biurrun
--- configure | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/configure b/configure index 33c52240aa..169a6fa33f 100755 --- a/configure +++ b/configure @@ -2595,6 +2595,7 @@ pkg_config_default=pkg-config ranlib="ranlib" strip="strip"

[libav-devel] [PATCH 3/5] configure: Factorize check_64_bit()

2017-11-03 Thread Diego Biurrun
--- I'm assuming that the change for MIPS is safe. configure | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/configure b/configure index 169a6fa33f..21f24e8e0f 100755 --- a/configure +++ b/configure @@ -3764,8 +3764,7 @@ check_host_cflags $host_cflags_speed

[libav-devel] [PATCH 1/5] configure: Miscellaneous minor changes

2017-11-03 Thread Diego Biurrun
- Move a variable closer to where it is used - Add an explanatory comment - Simplify a crosscompile check - Minor SHFLAGS simplification - Coalesce some threads tests --- configure | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/configure b/configure index

Re: [libav-devel] [PATCH 9/9] Revert "configure: Detect AIX ar command instead of hardcoding it in the OS section"

2017-11-03 Thread Luca Barbato
On 03/11/2017 16:32, Diego Biurrun wrote: On Wed, Oct 18, 2017 at 03:22:15AM +0200, Diego Biurrun wrote: This reverts commit 4822ee3ca620a92cd2b0a9a03ea9e34288192c79. AIX is a fringe oddity. Do not clutter the main codepath with AIX workarounds. --- configure | 4 +--- 1 file changed, 1

Re: [libav-devel] [PATCH 9/9] Revert "configure: Detect AIX ar command instead of hardcoding it in the OS section"

2017-11-03 Thread Diego Biurrun
On Wed, Oct 18, 2017 at 03:22:15AM +0200, Diego Biurrun wrote: > This reverts commit 4822ee3ca620a92cd2b0a9a03ea9e34288192c79. > AIX is a fringe oddity. Do not clutter the main codepath with AIX workarounds. > --- > configure | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) We need a

Re: [libav-devel] [PATCH v2] matroskadec: don't warn about unknown spherical medata when none is present

2017-11-03 Thread Luca Barbato
On 03/11/2017 00:53, James Almer wrote: track->video.projection.type is set to 0 (a Matroska specific "No spherical metadata present" value, with no related AVSphericalMapping) by default on files without the element. This removes bogus warnings on every single matroska file without Spherical

Re: [libav-devel] [PATCH] libspeexenc: Use speex_lib_get_mode instead of the speex_foo_mode data symbols

2017-11-03 Thread Luca Barbato
On 03/11/2017 11:05, Martin Storsjö wrote: This avoids issues linking to a DLL version of libspeex, since the libspeex headers lack proper dllimport declarations for the data symbols. This isn't an issue when building with mingw with GNU binutils, since GNU ld can fix up that kind of data

Re: [libav-devel] [PATCH] libspeexenc: Use speex_lib_get_mode instead of the speex_foo_mode data symbols

2017-11-03 Thread Martin Storsjö
On Fri, 3 Nov 2017, Martin Storsjö wrote: This avoids issues linking to a DLL version of libspeex, since the libspeex headers lack proper dllimport declarations for the data symbols. This isn't an issue when building with mingw with GNU binutils, since GNU ld can fix up that kind of data

[libav-devel] [PATCH] libspeexenc: Use speex_lib_get_mode instead of the speex_foo_mode data symbols

2017-11-03 Thread Martin Storsjö
This avoids issues linking to a DLL version of libspeex, since the libspeex headers lack proper dllimport declarations for the data symbols. This isn't an issue when building with mingw with GNU binutils, since GNU ld can fix up that kind of data import automatically. ---