[libav-devel] [PATCH] maint: Ignore dot dirs

2019-05-04 Thread Luca Barbato
They are usually created by tools and editors. --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 1a08fd15c5..693fa5636a 100644 --- a/.gitignore +++ b/.gitignore @@ -29,3 +29,4 @@ /coverage.info /lcov/ /mapfile +.*/ -- 2.12.2

[libav-devel] [PATCH 14/14] avcodec/cbs_h2645: use AVBufferRef to store list of active parameter sets

2019-05-04 Thread Luca Barbato
From: James Almer Removes unnecessary data copies, and partially fixes potential issues with dangling references held in said lists. Reviewed-by: Mark Thompson Signed-off-by: James Almer --- libavcodec/cbs_h264.h | 2 ++ libavcodec/cbs_h2645.c | 46

[libav-devel] [PATCH 13/14] cbs_h264: Need [] in the name when subscript is required

2019-05-04 Thread Luca Barbato
From: Haihao Xiang Otherwise it will hit an assert in the function ff_cbs_trace_syntax_element() in cbs.c, line 400. Signed-off-by: Haihao Xiang --- libavcodec/cbs_h264_syntax_template.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cbs_h264_syntax_template.c

[libav-devel] [PATCH 12/14] cbs: Add support for array subscripts in trace output

2019-05-04 Thread Luca Barbato
From: Mark Thompson This makes the trace output for arrays significantly nicer. --- libavcodec/cbs.c | 44 ++-- libavcodec/cbs_h2645.c | 83 --- libavcodec/cbs_h264_syntax_template.c | 101 +--

[libav-devel] [PATCH 09/14] avcodec/cbs_mpeg2: create a reference to the existing buffer when decomposing slice units

2019-05-04 Thread Luca Barbato
From: James Almer Reviewed-by: Mark Thompson Signed-off-by: James Almer --- libavcodec/cbs_mpeg2.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 8974bcabac..d65d84bd3d 100644 --- a/libavcodec/cbs_mpeg2.c +++

[libav-devel] [PATCH] cbs_h264: Fix overflow in shifts

2019-05-04 Thread Luca Barbato
From: Mark Thompson The type of the result of a shift operation is unaffected by the type of the right operand, so some existing code overflows with undefined behaviour when the element length is 32. Add a helper macro to calculate the maximum value correctly and then use it everywhere this

[libav-devel] [PATCH 05/14] cbs_h265: Use helper macro for maximum values of fixed-width elements

2019-05-04 Thread Luca Barbato
From: Mark Thompson Apply the same logic as the previous patch to H.265. There are no cases which currently overflow here, but this is still more consistent. --- libavcodec/cbs_h265_syntax_template.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git

[libav-devel] [PATCH 08/14] avcodec/cbs_h2645: create a reference to the existing buffer when decomposing slice units

2019-05-04 Thread Luca Barbato
From: James Almer Reviewed-by: Mark Thompson Signed-off-by: James Almer --- libavcodec/cbs_h2645.c | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.c index bf16343db5..8cd6db3fb4 100644 ---

[libav-devel] [PATCH 11/14] cbs_h2645: Simplify representation of fixed values

2019-05-04 Thread Luca Barbato
From: Mark Thompson --- libavcodec/cbs_h2645.c| 5 + libavcodec/cbs_h264_syntax_template.c | 30 --- libavcodec/cbs_h265_syntax_template.c | 38 +++ 3 files changed, 34 insertions(+), 39 deletions(-) diff --git

[libav-devel] [PATCH 07/14] avcodec/cbs_mpeg2: use existing buffer reference when splitting fragments

2019-05-04 Thread Luca Barbato
From: James Almer Reviewed-by: Mark Thompson Signed-off-by: James Almer --- libavcodec/cbs_mpeg2.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 7f484267fa..8974bcabac 100644 --- a/libavcodec/cbs_mpeg2.c

[libav-devel] [PATCH 10/14] cbs: Fragment/unit data is always reference counted

2019-05-04 Thread Luca Barbato
From: Mark Thompson Make this clear in the documentation and add some asserts to ensure that it is always true. --- libavcodec/cbs.c | 19 --- libavcodec/cbs.h | 10 ++ 2 files changed, 18 insertions(+), 11 deletions(-) diff --git a/libavcodec/cbs.c b/libavcodec/cbs.c

[libav-devel] [PATCH 03/14] cbs: Add a table of all supported codec IDs

2019-05-04 Thread Luca Barbato
From: Mark Thompson Use it as the set of codec IDs supported by the trace_headers BSF. --- configure | 2 +- libavcodec/cbs.c | 13 + libavcodec/cbs.h | 8 libavcodec/trace_headers_bsf.c | 9 + 4 files changed, 23

[libav-devel] [PATCH 06/14] avcodec/cbs_mpeg2: use memcpy when assembling fragments

2019-05-04 Thread Luca Barbato
From: James Almer before: 419022 decicycles in assemble_fragment,2047 runs, 1 skips after: 104621 decicycles in assemble_fragment,2045 runs, 3 skips Benched with a 2 minutes long 720x480 DVD mpeg2 sample. Reviewed-by: Michael Niedermayer Signed-off-by: James Almer ---

[libav-devel] [PATCH 01/14] maint: Ignore dot dirs

2019-05-04 Thread Luca Barbato
They are usually created by tools and editors. --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 1a08fd15c5..693fa5636a 100644 --- a/.gitignore +++ b/.gitignore @@ -29,3 +29,4 @@ /coverage.info /lcov/ /mapfile +.*/ -- 2.12.2

[libav-devel] [PATCH 02/14] cbs_h264: Fix overflow in shifts

2019-05-04 Thread Luca Barbato
From: Mark Thompson The type of the result of a shift operation is unaffected by the type of the right operand, so some existing code overflows with undefined behaviour when the element length is 32. Add a helper macro to calculate the maximum value correctly and then use it everywhere this

[libav-devel] [PATCH 04/14] avcodec/cbs: use a reference to the assembled CodedBitstreamFragment buffer when writing packets

2019-05-04 Thread Luca Barbato
From: James Almer This saves one malloc + memcpy per packet The CodedBitstreamFragment buffer is padded to follow the requirements of AVPacket. Reviewed-by: jkqxz Signed-off-by: James Almer --- libavcodec/cbs.c | 12 libavcodec/cbs_h2645.c | 8 +---

[libav-devel] Update cbs

2019-05-04 Thread Luca Barbato
Since I need the cbs for AV1 if I want to parse it (and make streamcopy from ivf produce something sane) I picked up the other changes. First batch of probably 3. ___ libav-devel mailing list libav-devel@libav.org