[libav-devel] [PATCH 1/4] h264_slice: Drop pointless assert

--- libavcodec/h264_slice.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c index 8d79740..87d6582 100644 --- a/libavcodec/h264_slice.c +++ b/libavcodec/h264_slice.c @@ -286,8 +286,6 @@ static int

Re: [libav-devel] [PATCH] fate.sh: Allow setting other make flags for running tests

On Tue, Nov 22, 2016 at 10:10:09PM +0200, Martin Storsjö wrote: > If makeopts_fate is set, these makeopts are used for running the > tests instead of the normal makeopts. If it isn't set, the normal > makeopts variable is used as before. > > This is useful if remote testing on a lesser machine

[libav-devel] [PATCH 3/4] ac3enc: Reshuffle functions to avoid forward declarations

--- libavcodec/ac3enc_fixed.c| 58 ++-- libavcodec/ac3enc_float.c| 70 +++- libavcodec/ac3enc_template.c | 9 -- 3 files changed, 64 insertions(+), 73 deletions(-) diff --git a/libavcodec/ac3enc_fixed.c

[libav-devel] [PATCH 4/4] ac3enc: Avoid unnecessary macro indirections

--- libavcodec/ac3enc_fixed.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/ac3enc_fixed.c b/libavcodec/ac3enc_fixed.c index c1cf825..cb1c58d 100644 --- a/libavcodec/ac3enc_fixed.c +++ b/libavcodec/ac3enc_fixed.c @@ -104,7 +104,7 @@ static CoefType

[libav-devel] [PATCH 2/4] ac3enc: Reshuffle some float/fixed-mode ifdefs to avoid a dummy function

--- libavcodec/ac3enc_fixed.c| 2 ++ libavcodec/ac3enc_float.c| 10 -- libavcodec/ac3enc_template.c | 4 +--- 3 files changed, 3 insertions(+), 13 deletions(-) diff --git a/libavcodec/ac3enc_fixed.c b/libavcodec/ac3enc_fixed.c index 2bb82ef..c26f9c0 100644 ---

Re: [libav-devel] [PATCH 1/3] ape: Restructure DEBUG ifdefs to avoid unused function parameter warnings

On Mon, Nov 21, 2016 at 03:57:08PM +0100, Diego Biurrun wrote: > --- > > Now with 'do {} while(0)' as suggested by Luca. > > libavformat/ape.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) Set OKed by Luca on IRC. Diego ___

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On 23/11/2016 05:10, James Almer wrote: > To make av_stream_add_side_data return error codes consistent with > av_packet_add_side_data's. I'd go the other way round, but sure, staying consistent is good. lu ___ libav-devel mailing list

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On 11/23/2016 12:39 AM, Luca Barbato wrote: > On 22/11/2016 23:16, Vittorio Giovara wrote: >> -tmp = av_realloc_array(st->side_data, st->nb_side_data + 1, >> sizeof(*tmp)); >> +if ((unsigned) st->nb_side_data + 1 >= INT_MAX / sizeof(*st->side_data)) >> +return AVERROR(ERANGE); >>

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On 22/11/2016 23:16, Vittorio Giovara wrote: > -tmp = av_realloc_array(st->side_data, st->nb_side_data + 1, > sizeof(*tmp)); > +if ((unsigned) st->nb_side_data + 1 >= INT_MAX / sizeof(*st->side_data)) > +return AVERROR(ERANGE); > + > +tmp = av_realloc(st->side_data,

Re: [libav-devel] [PATCH] matroska: use av_stream_add_side_data() for stereo3d side data

On 22/11/2016 21:50, Vittorio Giovara wrote: > From: James Almer > > Signed-off-by: James Almer > Signed-off-by: Vittorio Giovara > --- > libavformat/matroska.c | 22 -- > 1 file changed, 8 insertions(+), 14

Re: [libav-devel] [PATCH] fate.sh: Allow setting other make flags for running tests

On 22/11/2016 21:10, Martin Storsjö wrote: > If makeopts_fate is set, these makeopts are used for running the > tests instead of the normal makeopts. If it isn't set, the normal > makeopts variable is used as before. > > This is useful if remote testing on a lesser machine where a large > number

Re: [libav-devel] [PATCH] doc: Add libxavs section

On Tue, Nov 22, 2016 at 5:16 PM, Vittorio Giovara wrote: > From: Diego Biurrun > > --- > doc/general.texi | 8 > 1 file changed, 8 insertions(+) Never mind this email, sorry for the spam. -- Vittorio

[libav-devel] [PATCH] utils: Add av_stream_add_side_data()

From: James Almer Functionally similar to av_packet_add_side_data(). Allows the use of an already allocated buffer as stream side data. Signed-off-by: James Almer Signed-off-by: Vittorio Giovara --- Updated according to James'

[libav-devel] [PATCH] doc: Add libxavs section

From: Diego Biurrun --- doc/general.texi | 8 1 file changed, 8 insertions(+) diff --git a/doc/general.texi b/doc/general.texi index 7b1f11c..d232300 100644 --- a/doc/general.texi +++ b/doc/general.texi @@ -95,6 +95,14 @@ Go to @url{http://www.wavpack.com/} and

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On Tue, Nov 22, 2016 at 5:09 PM, James Almer wrote: > On 11/22/2016 5:32 PM, Vittorio Giovara wrote: >> On Tue, Nov 22, 2016 at 10:04 AM, James Almer wrote: >>> On 11/21/2016 8:49 PM, Vittorio Giovara wrote: From: James Almer

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On 11/22/2016 5:32 PM, Vittorio Giovara wrote: > On Tue, Nov 22, 2016 at 10:04 AM, James Almer wrote: >> On 11/21/2016 8:49 PM, Vittorio Giovara wrote: >>> From: James Almer >>> >>> Functionally similar to av_packet_add_side_data(). Allows the use of an >>>

[libav-devel] [PATCH] matroska: use av_stream_add_side_data() for stereo3d side data

From: James Almer Signed-off-by: James Almer Signed-off-by: Vittorio Giovara --- libavformat/matroska.c | 22 -- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/libavformat/matroska.c

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On Tue, Nov 22, 2016 at 10:04 AM, James Almer wrote: > On 11/21/2016 8:49 PM, Vittorio Giovara wrote: >> From: James Almer >> >> Functionally similar to av_packet_add_side_data(). Allows the use of an >> already allocated buffer as stream side data. >> >>

[libav-devel] [PATCH] fate.sh: Allow setting other make flags for running tests

If makeopts_fate is set, these makeopts are used for running the tests instead of the normal makeopts. If it isn't set, the normal makeopts variable is used as before. This is useful if remote testing on a lesser machine where a large number of parallel jobs might be undesireable, while wanting

Re: [libav-devel] [PATCH 1/1] fate: report ignored tests but ignore the test result

On 2016-11-22 16:43:50 +0100, Diego Biurrun wrote: > On Mon, Nov 21, 2016 at 12:31:24AM +0100, Janne Grunau wrote: > > --- a/configure > > +++ b/configure > > @@ -348,9 +348,8 @@ Developer options (useful when working on Libav itself): > > - --ignore-tests=TESTS whitespace-separated list

Re: [libav-devel] [PATCH 2/2] colorspace: Move cm macro where it belongs

On Mon, Nov 21, 2016 at 10:46:45AM -0500, Vittorio Giovara wrote: > On Mon, Nov 21, 2016 at 8:52 AM, Diego Biurrun wrote: > > --- a/libavcodec/dvbsubdec.c > > +++ b/libavcodec/dvbsubdec.c > > @@ -32,8 +32,6 @@ > > #define DVBSUB_DISPLAYDEFINITION_SEGMENT 0x14 > > #define

Re: [libav-devel] [PATCH 4/9] decode: be more explicit about storing the last packet properties

On Tue, Nov 22, 2016 at 8:36 AM, Anton Khirnov wrote: > The current code stores a pointer to the packet passed to the decoder, > which is then used during get_buffer() for timestamps and side data > passthrough. However, since this is a pointer to user data which we do > not

Re: [libav-devel] [PATCH 2/9] lavc: move encoding-related code from utils.c to a new file

On Tue, Nov 22, 2016 at 8:36 AM, Anton Khirnov wrote: > + > +int ff_alloc_packet(AVPacket *avpkt, int size) > +{ > +if (size > INT_MAX - AV_INPUT_BUFFER_PADDING_SIZE) > +return AVERROR(EINVAL); > + > +if (avpkt->data) { > +AVBufferRef *buf = avpkt->buf;

Re: [libav-devel] [PATCH 1/1] fate: report ignored tests but ignore the test result

On Tue, Nov 22, 2016 at 04:43:50PM +0100, Diego Biurrun wrote: > On Mon, Nov 21, 2016 at 12:31:24AM +0100, Janne Grunau wrote: > > --- a/tests/Makefile > > +++ b/tests/Makefile > > @@ -143,11 +143,17 @@ endif > > > > -fate: $(filter-out $(addprefix fate-,$(IGNORE_TESTS)),$(FATE)) > > +define

Re: [libav-devel] [PATCH 1/1] fate: report ignored tests but ignore the test result

On Mon, Nov 21, 2016 at 12:31:24AM +0100, Janne Grunau wrote: > --- a/configure > +++ b/configure > @@ -348,9 +348,8 @@ Developer options (useful when working on Libav itself): > - --ignore-tests=TESTS whitespace-separated list (without "fate-" prefix > - in the

Re: [libav-devel] [PATCH 1/9] lavc: add a null bitstream filter

On 22/11/2016 14:36, Anton Khirnov wrote: > --- > doc/bitstream_filters.texi | 3 +++ > libavcodec/Makefile| 1 + > libavcodec/bitstream_filters.c | 1 + > libavcodec/null_bsf.c | 44 > ++ > 4 files changed, 49 insertions(+) >

Re: [libav-devel] [PATCH 7/9] qsvdec_h2645: switch to the new generic filtering mechanism

On 22/11/2016 14:36, Anton Khirnov wrote: > Drop the internal manual conversion from the MP4 format to Annex B. > --- > libavcodec/qsvdec_h2645.c | 74 > +++ > 1 file changed, 11 insertions(+), 63 deletions(-) > Looks good.

Re: [libav-devel] [PATCH 9/9] vp9: split superframes in the filtering stage before actual decoding

On 22/11/2016 14:36, Anton Khirnov wrote: > Significantly increases the efficiency of frame threading, since > individual frames in a superframe can now be decoded in parallel. > --- > configure| 2 +- > libavcodec/vp9.c | 74 > ++-- >

Re: [libav-devel] [PATCH 5/9] decode: restructure the core decoding code

On 22/11/2016 14:36, Anton Khirnov wrote: > Currently, the new decoding API is pretty much just a wrapper around > the old deprecated one. This is problematic, since it interferes with > making full use of the flexibility added by the new API. The old API > should also be removed at some future

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On 11/21/2016 8:49 PM, Vittorio Giovara wrote: > From: James Almer > > Functionally similar to av_packet_add_side_data(). Allows the use of an > already allocated buffer as stream side data. > > Signed-off-by: James Almer > Signed-off-by: Vittorio Giovara

Re: [libav-devel] [PATCH 3/9] lavc: move decoding-related code from utils.c to a new file

On Tue, Nov 22, 2016 at 02:36:33PM +0100, Anton Khirnov wrote: > --- > libavcodec/Makefile | 1 + > libavcodec/decode.c | 924 > > libavcodec/utils.c | 889 -- > 3 files changed, 925

Re: [libav-devel] [PATCH 2/9] lavc: move encoding-related code from utils.c to a new file

On Tue, Nov 22, 2016 at 02:36:32PM +0100, Anton Khirnov wrote: > --- > libavcodec/Makefile | 1 + > libavcodec/encode.c | 360 > > libavcodec/utils.c | 331 --- > 3 files changed, 361

Re: [libav-devel] [PATCH 7/9] qsvdec_h2645: switch to the new generic filtering mechanism

On Tue, Nov 22, 2016 at 02:36:37PM +0100, Anton Khirnov wrote: > Drop the internal manual conversion from the MP4 format to Annex B. > --- > libavcodec/qsvdec_h2645.c | 74 > +++ > 1 file changed, 11 insertions(+), 63 deletions(-) The libopenh264

Re: [libav-devel] [PATCH 6/9] lavc: add support for filtering packets before decoding

On Tue, Nov 22, 2016 at 02:36:36PM +0100, Anton Khirnov wrote: > --- a/libavcodec/avcodec.h > +++ b/libavcodec/avcodec.h > @@ -3235,6 +3235,12 @@ typedef struct AVCodec { > * See FF_CODEC_CAP_* in internal.h > */ > int caps_internal; > + > +/** > + * Decoding only, a

Re: [libav-devel] [PATCH 1/9] lavc: add a null bitstream filter

On Tue, Nov 22, 2016 at 02:36:31PM +0100, Anton Khirnov wrote: > --- > doc/bitstream_filters.texi | 3 +++ > libavcodec/Makefile| 1 + > libavcodec/bitstream_filters.c | 1 + > libavcodec/null_bsf.c | 44 > ++ > 4 files changed,

[libav-devel] [PATCH 9/9] vp9: split superframes in the filtering stage before actual decoding

Significantly increases the efficiency of frame threading, since individual frames in a superframe can now be decoded in parallel. --- configure| 2 +- libavcodec/vp9.c | 74 ++-- 2 files changed, 14 insertions(+), 62 deletions(-) diff

[libav-devel] [PATCH 1/9] lavc: add a null bitstream filter

--- doc/bitstream_filters.texi | 3 +++ libavcodec/Makefile| 1 + libavcodec/bitstream_filters.c | 1 + libavcodec/null_bsf.c | 44 ++ 4 files changed, 49 insertions(+) create mode 100644 libavcodec/null_bsf.c diff --git

[libav-devel] [PATCH 3/9] lavc: move decoding-related code from utils.c to a new file

--- libavcodec/Makefile | 1 + libavcodec/decode.c | 924 libavcodec/utils.c | 889 -- 3 files changed, 925 insertions(+), 889 deletions(-) create mode 100644 libavcodec/decode.c diff --git

[libav-devel] [PATCH 7/9] qsvdec_h2645: switch to the new generic filtering mechanism

Drop the internal manual conversion from the MP4 format to Annex B. --- libavcodec/qsvdec_h2645.c | 74 +++ 1 file changed, 11 insertions(+), 63 deletions(-) diff --git a/libavcodec/qsvdec_h2645.c b/libavcodec/qsvdec_h2645.c index a26f150..8ac403b

[libav-devel] [PATCH 6/9] lavc: add support for filtering packets before decoding

--- configure | 1 + libavcodec/avcodec.h | 6 ++ libavcodec/decode.c | 172 ++ libavcodec/internal.h | 8 +++ libavcodec/utils.c| 2 + 5 files changed, 177 insertions(+), 12 deletions(-) diff --git a/configure

[libav-devel] [PATCH 8/9] lavc: add a bitstream filter for splitting VP9 superframes

Partially based on code by Ronald S. Bultje . --- doc/bitstream_filters.texi| 4 + libavcodec/Makefile | 1 + libavcodec/bitstream_filters.c| 1 + libavcodec/vp9_superframe_split_bsf.c | 143 ++ 4

[libav-devel] [PATCH 5/9] decode: restructure the core decoding code

Currently, the new decoding API is pretty much just a wrapper around the old deprecated one. This is problematic, since it interferes with making full use of the flexibility added by the new API. The old API should also be removed at some future point. Reorganize the code so that the new

[libav-devel] [PATCH 4/9] decode: be more explicit about storing the last packet properties

The current code stores a pointer to the packet passed to the decoder, which is then used during get_buffer() for timestamps and side data passthrough. However, since this is a pointer to user data which we do not own, storing it is potentially dangerous. It is also ill defined for the new

[libav-devel] [PATCH 2/9] lavc: move encoding-related code from utils.c to a new file

--- libavcodec/Makefile | 1 + libavcodec/encode.c | 360 libavcodec/utils.c | 331 --- 3 files changed, 361 insertions(+), 331 deletions(-) create mode 100644 libavcodec/encode.c diff --git

Re: [libav-devel] [PATCH] mov: Use av_stream_add_side_data() for displaymatrix side data

On 22/11/2016 00:54, Vittorio Giovara wrote: > From: James Almer > > Signed-off-by: James Almer > Signed-off-by: Vittorio Giovara > --- > libavformat/mov.c | 18 +- > 1 file changed, 5 insertions(+), 13

Re: [libav-devel] [PATCH] utils: Add av_stream_add_side_data()

On 22/11/2016 00:49, Vittorio Giovara wrote: > From: James Almer > > Functionally similar to av_packet_add_side_data(). Allows the use of an > already allocated buffer as stream side data. > > Signed-off-by: James Almer > Signed-off-by: Vittorio Giovara