[libav-devel] [COSMETIC] Removal of spaces in arguments

2015-08-07 Thread Paolo Bizzarri
Hi, a simple patch to remove spaces after parenthesis in a file.. For other files, what is the preferred approach? Many files for a patch? Or a single patch per file? Regards PaoloB 0001-Removed-spaces-to-comply-with-formatting-rules.patch Description: Binary data

Re: [libav-devel] [COSMETIC] Removal of spaces in arguments

2015-08-07 Thread Alexandra Hájková
On Fri, Aug 7, 2015 at 10:13 PM, Paolo Bizzarri pibi...@gmail.com wrote: Hi, a simple patch to remove spaces after parenthesis in a file.. For other files, what is the preferred approach? Many files for a patch? Or a single patch per file? Single patch per file is preferred and you should

Re: [libav-devel] [FFmpeg-devel] [PATCH 0/20] removal of deprecated features

2015-08-07 Thread wm4
On Thu, 6 Aug 2015 23:26:05 +0200 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: On 06.08.2015 00:53, wm4 wrote: Well, you sure like to list a lot of projects. No, I don't. I'd like it much more if the list was empty. But what you don't say is that many of these are either

[libav-devel] [PATCH] h264: don't choke on SEI bar data.

2015-08-07 Thread John Högberg
Hi, The current H264 SEI parser chokes if it encounters bar data or EIA-608 subtitles while parsing ATSC1_data. This patch fixes that. Regards, John Högberg From e1f1c47cb50e2471e977911a68e07430fbbff18c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?John=20H=C3=B6gberg?= john.hogb...@ericsson.com

[libav-devel] [PATCH] filtergraph: Select the best pixel format

2015-08-07 Thread Luca Barbato
Give priority to pixel formats closer to the input. --- I kept it as simple as possible on purpose. libavfilter/avfiltergraph.c | 97 + 1 file changed, 97 insertions(+) diff --git a/libavfilter/avfiltergraph.c b/libavfilter/avfiltergraph.c index

Re: [libav-devel] [PATCH] h264: Do not print an error when the buffer has to be refilled

2015-08-07 Thread Kostya Shishkov
On Thu, Aug 06, 2015 at 11:57:53AM +0200, Luca Barbato wrote: Partially amends 9469370fb32679352e66826daf77bdd2e6f067b5 --- My fault for testing only with fate =P libavcodec/h264.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264.c

Re: [libav-devel] [PATCH] h264: don't choke on SEI bar data.

2015-08-07 Thread Luca Barbato
On 07/08/15 21:30, John Högberg wrote: Hi, The current H264 SEI parser chokes if it encounters bar data or EIA-608 subtitles while parsing ATSC1_data. This patch fixes that. Seems fine, do you have a sample that you can share to test? lu ___

Re: [libav-devel] [PATCH] h264: don't choke on SEI bar data.

2015-08-07 Thread Luca Barbato
On 07/08/15 21:30, John Högberg wrote: +skip_bits(h-gb, (size - 1) * 8); This should be a loop over skip_bits(8) due the limitations of the bitstream reader (replacing it won't be too soon...). lu ___ libav-devel mailing list

Re: [libav-devel] [PATCH] h264: don't choke on SEI bar data.

2015-08-07 Thread Hendrik Leppkes
On Sat, Aug 8, 2015 at 1:32 AM, Luca Barbato lu_z...@gentoo.org wrote: On 07/08/15 21:30, John Högberg wrote: +skip_bits(h-gb, (size - 1) * 8); This should be a loop over skip_bits(8) due the limitations of the bitstream reader (replacing it won't be too soon...). There is also

Re: [libav-devel] [PATCH] h264: don't choke on SEI bar data.

2015-08-07 Thread Luca Barbato
On 08/08/15 01:48, Hendrik Leppkes wrote: On Sat, Aug 8, 2015 at 1:32 AM, Luca Barbato lu_z...@gentoo.org wrote: On 07/08/15 21:30, John Högberg wrote: +skip_bits(h-gb, (size - 1) * 8); This should be a loop over skip_bits(8) due the limitations of the bitstream reader (replacing it