[FFmpeg-devel] [PATCH V2 0/3] Change options location and kvazaar

2018-12-15 Thread Jun Zhao
V2: - Fix the fate break issue after change options location (Tks Micheal's review) V1: - Change some options location in opt table - Use avctx->frame_rate first for framerate setting in kvazaar encoder - Indent the code in kvazaar wrapper to follow FFmpeg's coding style Jun Zhao (3):

[FFmpeg-devel] [PATCH V2 3/3] lavc/libkvazaar: indent the code

2018-12-15 Thread Jun Zhao
indent the code to follow FFmpeg's coding style Signed-off-by: Jun Zhao --- libavcodec/libkvazaar.c | 11 +-- 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c index 50910b7..a89ca7f 100644 --- a/libavcodec/libkvazaar.c +++

[FFmpeg-devel] [PATCH V2 2/3] lavc/libkvazaar: Use avctx->frame_rate first for framerate setting

2018-12-15 Thread Jun Zhao
perfer avctx->frame_rate first than use avctx->time_base when setting the frame rate to encoder. Signed-off-by: Jun Zhao --- libavcodec/libkvazaar.c | 22 -- 1 files changed, 16 insertions(+), 6 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c in

[FFmpeg-devel] [PATCH V2 1/3] lavc/options_table: Change some options location in opt table.

2018-12-15 Thread Jun Zhao
Change the some options location in avcodec_options to make code more readable. And update the fate test with this change. Signed-off-by: Jun Zhao --- libavcodec/options_table.h | 48 +- tests/ref/fate/api-mjpeg-codec-param | 24 tes

Re: [FFmpeg-devel] [PATCH] avutil/tests/random_seed: seeds[] is uint32_t, therefore use PRIX32 macro

2018-12-15 Thread Michael Niedermayer
On Sat, Dec 15, 2018 at 01:07:18PM +1100, Peter Ross wrote: > squelch format type warning > --- > libavutil/tests/random_seed.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/tests/random_seed.c b/libavutil/tests/random_seed.c > index 78067dbe41..bf0c6c7986 100

Re: [FFmpeg-devel] [PATCH 1/3] lavc/options_table: Change some options location in opt table.

2018-12-15 Thread Michael Niedermayer
On Sat, Dec 15, 2018 at 07:31:35PM +0800, Jun Zhao wrote: > Change the some options location in avcodec_options to make code more > readable. > > Signed-off-by: Jun Zhao > --- > libavcodec/options_table.h | 48 > ++-- > 1 files changed, 24 insertions(+)

Re: [FFmpeg-devel] [PATCH] swscale/output: Altivec-optimize float yuv2plane1

2018-12-15 Thread Michael Niedermayer
On Sat, Dec 15, 2018 at 06:32:31PM +0200, Lauri Kasanen wrote: > This function wouldn't benefit from VSX instructions, and input > and output share alignment, so I put it under altivec. > > ./ffmpeg_g -f rawvideo -pix_fmt rgb24 -s hd1080 -i /dev/zero -pix_fmt > grayf32le \ > -f null -vframes 100

Re: [FFmpeg-devel] [PATCH] lavf/mov: ensure only one tkhd per trak

2018-12-15 Thread Baptiste Coudurier
Hi Chris, > On Dec 14, 2018, at 3:12 PM, Chris Cunningham > wrote: > >> >> from a quick look, i did not find a file this breaks >> > > Woot. Baptiste, I'm happy with this last patch if you are. Yes, looks great. Thanks a lot! — Baptiste Coudurier ___

[FFmpeg-devel] [PATCH] avformat: add vividas demuxer

2018-12-15 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/vividas.c| 715 +++ 3 files changed, 717 insertions(+) create mode 100644 libavformat/vividas.c diff --git a/libavformat/Makefile b/libavformat

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: skip subtitle PES packets if PCR not available

2018-12-15 Thread Jan Ekström
On Sat, Dec 15, 2018 at 5:13 PM Jan Ekström wrote: > > On Sat, Dec 15, 2018 at 4:57 PM Marton Balint wrote: > > > > On Sat, 15 Dec 2018, Jan Ekström wrote: > > > > > Fixes issues when a subtitle packet is received before PCR for the > > > program has been received, leading to wildly jumping times

[FFmpeg-devel] [PATCH] avformat/mpegts: unset DTS/PTS for subtitle PES packets if PCR not available

2018-12-15 Thread Jan Ekström
Fixes issues when a subtitle packet is received before PCR for the program has been received, leading to wildly jumping timestamps on the lavf client side as well as in the re-ordering logic. This usually happens in case of multiplexes where the PCR of a program is not taken into account with subt

[FFmpeg-devel] [PATCH] swscale/output: Altivec-optimize float yuv2plane1

2018-12-15 Thread Lauri Kasanen
This function wouldn't benefit from VSX instructions, and input and output share alignment, so I put it under altivec. ./ffmpeg_g -f rawvideo -pix_fmt rgb24 -s hd1080 -i /dev/zero -pix_fmt grayf32le \ -f null -vframes 100 -v error -nostats - 3743 UNITS in planar1, 65495 runs, 41 skips -cp

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: skip subtitle PES packets if PCR not available

2018-12-15 Thread Jan Ekström
On Sat, Dec 15, 2018 at 4:57 PM Marton Balint wrote: > > On Sat, 15 Dec 2018, Jan Ekström wrote: > > > Fixes issues when a subtitle packet is received before PCR for the > > program has been received, leading to wildly jumping timestamps > > on the lavf client side as well as in the re-ordering lo

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: skip subtitle PES packets if PCR not available

2018-12-15 Thread Marton Balint
On Sat, 15 Dec 2018, Jan Ekström wrote: Fixes issues when a subtitle packet is received before PCR for the program has been received, leading to wildly jumping timestamps on the lavf client side as well as in the re-ordering logic. This usually happens in case of multiplexes where the PCR of

Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/mxfdec: fix error check in macro

2018-12-15 Thread James Almer
On 12/15/2018 6:18 AM, Hendrik Leppkes wrote: > On Sat, Dec 15, 2018 at 9:50 AM Paul B Mahol wrote: >> >> On 12/15/18, James Almer wrote: ffmpeg | branch: master | Paul B Mahol >>> > | Thu Dec 13 23:51:02 2018 +0100| [e5a0013c4a6248fe7e2

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/gif: enable encoding single gif image per frame

2018-12-15 Thread Paul B Mahol
On 12/15/18, Paul B Mahol wrote: > On 12/15/18, Paul B Mahol wrote: >> On 12/15/18, Carl Eugen Hoyos wrote: >>> 2018-12-13 19:32 GMT+01:00, Paul B Mahol : ffmpeg | branch: master | Paul B Mahol | Wed Dec 12 13:19:33 2018 +0100| [bb0984cc29e93e56506722fe0ce5c6d43b6d4326] | committ

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/gif: enable encoding single gif image per frame

2018-12-15 Thread Paul B Mahol
On 12/15/18, Paul B Mahol wrote: > On 12/15/18, Carl Eugen Hoyos wrote: >> 2018-12-13 19:32 GMT+01:00, Paul B Mahol : >>> ffmpeg | branch: master | Paul B Mahol | Wed Dec 12 >>> 13:19:33 2018 +0100| [bb0984cc29e93e56506722fe0ce5c6d43b6d4326] | >>> committer: >>> Paul B Mahol >>> >>> avcodec/gif:

[FFmpeg-devel] [PATCH 1/2] avcodec/g723_1: add support for stereo files

2018-12-15 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/g723_1.h| 13 ++ libavcodec/g723_1dec.c | 54 -- libavcodec/g723_1enc.c | 14 ++- 3 files changed, 47 insertions(+), 34 deletions(-) diff --git a/libavcodec/g723_1.h b/libavcodec/g723_1.h index

[FFmpeg-devel] [PATCH 2/2] avcodec/g723_1dec: reindent after last commit

2018-12-15 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/g723_1dec.c | 234 - 1 file changed, 117 insertions(+), 117 deletions(-) diff --git a/libavcodec/g723_1dec.c b/libavcodec/g723_1dec.c index 798d464c71..bc8629fbec 100644 --- a/libavcodec/g723_1dec.c +++ b/libavcod

[FFmpeg-devel] [PATCH 2/3] lavc/libkvazaar: Use avctx->frame_rate first for framerate setting

2018-12-15 Thread Jun Zhao
perfer avctx->frame_rate first than use avctx->time_base when setting the frame rate to encoder. Signed-off-by: Jun Zhao --- libavcodec/libkvazaar.c | 22 -- 1 files changed, 16 insertions(+), 6 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c in

[FFmpeg-devel] [PATCH 3/3] lavc/libkvazaar: indent the code

2018-12-15 Thread Jun Zhao
indent the code to follow FFmpeg's coding style Signed-off-by: Jun Zhao --- libavcodec/libkvazaar.c | 11 +-- 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c index 50910b7..a89ca7f 100644 --- a/libavcodec/libkvazaar.c +++

[FFmpeg-devel] [PATCH 1/3] lavc/options_table: Change some options location in opt table.

2018-12-15 Thread Jun Zhao
Change the some options location in avcodec_options to make code more readable. Signed-off-by: Jun Zhao --- libavcodec/options_table.h | 48 ++-- 1 files changed, 24 insertions(+), 24 deletions(-) diff --git a/libavcodec/options_table.h b/libavcodec/opt

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: skip subtitle PES packets if PCR not available

2018-12-15 Thread Jan Ekström
On Sat, Dec 15, 2018 at 3:50 AM Carl Eugen Hoyos wrote: > > 2018-12-15 2:31 GMT+01:00, Jan Ekström : > > Fixes issues when a subtitle packet is received before PCR for the > > program has been received, leading to wildly jumping timestamps > > on the lavf client side as well as in the re-ordering

Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/mxfdec: fix error check in macro

2018-12-15 Thread Hendrik Leppkes
On Sat, Dec 15, 2018 at 9:50 AM Paul B Mahol wrote: > > On 12/15/18, James Almer wrote: > >> ffmpeg | branch: master | Paul B Mahol >> > | Thu Dec 13 23:51:02 > >> 2018 +0100| [e5a0013c4a6248fe7e2a651db1fda6b9bb2cd743] | committer: Paul B > >> Ma

Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/mxfdec: fix error check in macro

2018-12-15 Thread Paul B Mahol
On 12/15/18, James Almer wrote: >> ffmpeg | branch: master | Paul B Mahol > > | Thu Dec 13 23:51:02 >> 2018 +0100| [e5a0013c4a6248fe7e2a651db1fda6b9bb2cd743] | committer: Paul B >> Mahol >> >> avformat/mxfdec: fix error check in macro >> >> Fixes #

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/gif: enable encoding single gif image per frame

2018-12-15 Thread Paul B Mahol
On 12/15/18, Carl Eugen Hoyos wrote: > 2018-12-13 19:32 GMT+01:00, Paul B Mahol : >> ffmpeg | branch: master | Paul B Mahol | Wed Dec 12 >> 13:19:33 2018 +0100| [bb0984cc29e93e56506722fe0ce5c6d43b6d4326] | >> committer: >> Paul B Mahol >> >> avcodec/gif: enable encoding single gif image per frame

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/rasc: Check that the number of moves is less than or equal the number of pixels

2018-12-15 Thread Paul B Mahol
On 12/15/18, Michael Niedermayer wrote: > Fixes: OOM > Fixes: > 10307/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_RASC_fuzzer-5393974559244288 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer > ---