[FFmpeg-devel] [PATCH v3] fate/api-h264-slice-test: use cleaner error handling

2018-10-30 Thread joshdk
From: Josh de Kock Fix ticket #7521 --- tests/api/api-h264-slice-test.c | 88 +++-- 1 file changed, 51 insertions(+), 37 deletions(-) diff --git a/tests/api/api-h264-slice-test.c b/tests/api/api-h264-slice-test.c index 57e7dc79c3..ebc90b5b57 100644 ---

[FFmpeg-devel] [PATCH v2] fate/api-h264-slice-test: use cleaner error handling

2018-10-29 Thread joshdk
From: Josh de Kock --- tests/api/api-h264-slice-test.c | 85 +++-- 1 file changed, 48 insertions(+), 37 deletions(-) diff --git a/tests/api/api-h264-slice-test.c b/tests/api/api-h264-slice-test.c index 57e7dc79c3..b3f9f91ab3 100644 ---

[FFmpeg-devel] [PATCH] fate/api-h264-slice-test: use cleaner error handling

2018-10-29 Thread joshdk
From: Josh de Kock --- tests/api/api-h264-slice-test.c | 74 +++-- 1 file changed, 43 insertions(+), 31 deletions(-) diff --git a/tests/api/api-h264-slice-test.c b/tests/api/api-h264-slice-test.c index 57e7dc79c3..08d5d57941 100644 ---

[FFmpeg-devel] [PATCH vFINAL] fate: add api-h264-slice test

2018-10-24 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- Actually send fix. tests/api/Makefile | 1 + tests/api/api-h264-slice-test.c | 221 +++ tests/fate/api.mak

[FFmpeg-devel] [PATCH v5] fate: add api-h264-slice test

2018-10-23 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- Prints a warning to the user when threading is not enabled, slice NALUs should still be checked for proper decoding irregardless of whether threads

[FFmpeg-devel] [PATCH v4] fate: add api-h264-slice test

2018-10-15 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- Tested on Linux 32/64, MinGW + WINE 32/64. tests/api/Makefile | 1 + tests/api/api-h264-slice-test.c | 216 ++

[FFmpeg-devel] [PATCH v3] fate: add api-h264-slice test

2018-10-12 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- Ignore the previous patch, had some dead code in it. This now works in MinGW as well. tests/api/Makefile | 1 +

[FFmpeg-devel] [PATCH v2 4/4] fate: add h264 timecode test

2018-10-12 Thread joshdk
From: Josh de Kock --- tests/fate/h264.mak | 2 + tests/ref/fate/h264-timecode | 305 +++ 2 files changed, 307 insertions(+) create mode 100644 tests/ref/fate/h264-timecode diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak index

[FFmpeg-devel] [PATCH v4 2/4] h264/pic_timing: support multiple timecodes

2018-10-12 Thread joshdk
From: Josh de Kock --- There are no cosmetic changes which are not functional here. libavcodec/h264_sei.c | 21 --- libavcodec/h264_sei.h | 28 +-- libavcodec/h264_slice.c | 59 +++-- 3 files changed, 65 insertions(+), 43

[FFmpeg-devel] [PATCH v2 1/4] lavc/h264: create AVFrame side data from H.264 timecodes

2018-10-12 Thread joshdk
From: Devin Heitmueller Create SMPTE ST 12-1 timecodes based on H.264 SEI picture timing info. For framerates > 30 FPS, the field flag is used in conjunction with pairs of frames which contain the same frame timestamp in S12M. Ensure the field is properly set per the spec. ---

[FFmpeg-devel] [PATCH] fate: add api-h264-slice test

2018-10-12 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- tests/api/Makefile | 1 + tests/api/api-h264-slice-test.c | 213 ++ tests/fate/api.mak | 4 +

[FFmpeg-devel] [PATCH 1/3] lavc/h264: create AVFrame side data from H.264 timecodes

2018-10-09 Thread joshdk
From: Devin Heitmueller Create SMPTE ST 12-1 timecodes based on H.264 SEI picture timing info. For framerates > 30 FPS, the field flag is used in conjunction with pairs of frames which contain the same frame timestamp in S12M. Ensure the field is properly set per the spec. ---

[FFmpeg-devel] [PATCH 3/3] h264/pic_timing: support multiple timecodes

2018-10-09 Thread joshdk
From: Josh de Kock --- libavcodec/h264_sei.c | 21 -- libavcodec/h264_sei.h | 28 +-- libavcodec/h264_slice.c | 59 ++- libavfilter/vf_showinfo.c | 10 +++ 4 files changed, 75 insertions(+), 43 deletions(-) diff

[FFmpeg-devel] [PATCH 0/3] Timecode support

2018-10-09 Thread joshdk
This set requires Derek's patch to h264 which fixes #7083. 'h264_slice: Copy the value of x264_build before calling h264_slice_header_init during thread init' Josh ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH 2/3] lavf/timecode: document SMPTE struct

2018-10-09 Thread joshdk
From: Devin Heitmueller There are a number of different binary representations in which SMPTE timecodes can use. Make clear that the specific representation that ffmpeg refers to corresponds to the DV video spec, which is SMPTE S314M:2005 for standard definition video and ST 370-2013 for high

[FFmpeg-devel] [PATCH v3] fate: add api-h264-slice test

2018-10-09 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- Tested with `./configure --cc='cc -m32'` now, seems to be fine. Will send sample to Michael on IRC. tests/api/Makefile | 1 +

[FFmpeg-devel] [PATCH v2] fate: add api-h264-slice test

2018-09-27 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- Tested 32bit with --arch=x86_32 tests/api/Makefile | 1 + tests/api/api-h264-slice-test.c | 212 ++

[FFmpeg-devel] [PATCH 2/2] fate: add api-h264-slice test

2018-09-26 Thread joshdk
From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- tests/api/Makefile | 1 + tests/api/api-h264-slice-test.c | 208 + tests/fate/api.mak | 4 +

[FFmpeg-devel] [PATCH 1/2] lavc/h264dec: don't error out when receiving multiple IDR slices

2018-09-26 Thread joshdk
From: Josh de Kock This error isn't particularly helpful as checking for mixed IDR/non-IDR NALUs would need to be done at a higher level to actually be accurate. Removing the error allows an API user to send individual slice NALUs (i.e. incomplete frames) so they can take advantage of slice

[FFmpeg-devel] [PATCH] lavc/h264: warn on mixed non-IDR/IDR NAL units

2018-09-04 Thread joshdk
From: Josh de Kock No segfault on sample ticket 4408. --- libavcodec/h264dec.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index 8d115fa040..2ab52f57c0 100644 --- a/libavcodec/h264dec.c +++

[FFmpeg-devel] [PATCH v3] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-17 Thread joshdk
From: Kieran Kunhya Signed-off-by: Josh de Kock --- libavcodec/h264_sei.c | 15 --- libavcodec/h264_sei.h | 3 +-- libavcodec/h264_slice.c | 22 ++ libavcodec/h264dec.c| 5 +++-- 4 files changed, 26 insertions(+), 19 deletions(-) diff --git

[FFmpeg-devel] [PATCH v2] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-17 Thread joshdk
From: Kieran Kunhya Signed-off-by: Josh de Kock --- libavcodec/h264_sei.c | 15 --- libavcodec/h264_sei.h | 3 +-- libavcodec/h264_slice.c | 26 ++ libavcodec/h264dec.c| 5 +++-- 4 files changed, 30 insertions(+), 19 deletions(-) diff --git

[FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-16 Thread joshdk
From: Kieran Kunhya --- This is also useful when CC is fully contained in the first field. libavcodec/h264_sei.c | 15 --- libavcodec/h264_sei.h | 3 +-- libavcodec/h264_slice.c | 15 ++- libavcodec/h264dec.c| 5 +++-- 4 files changed, 22 insertions(+), 16