Re: [FFmpeg-devel] [PATCH 0/6] Improvements for EBU R128 plugin (third round)

2018-10-10 Thread Paul B Mahol
On 10/10/18, Daniel Molkentin wrote: > On 09.10.2018 14:32, Daniel Molkentin wrote: >> Changes over second round: >> >> - gauge option: Add 'm' and 's' as alias parameter for >>momentary and short-term >> - fix spelling error in documentation >> - remove "mabsolute" documentation which was

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-10 Thread Marc-Antoine ARNAUD
For me it's the only codec who use picture colorspace as source. All others uses only the CodecContext. I don't know the exact reason, but I suppose it can be easiest to manage output colorspace during merge of video, as a video can have only one "static" video colorspace. So for me it made sense

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-10 Thread Paul B Mahol
On 10/10/18, Marc-Antoine ARNAUD wrote: > For me it's the only codec who use picture colorspace as source. > All others uses only the CodecContext. > I don't know the exact reason, but I suppose it can be easiest to manage > output colorspace during merge of video, as a video can have only one >

Re: [FFmpeg-devel] avcodec/prores aw improvment

2018-10-10 Thread Marc-Antoine ARNAUD
Great patches ;-) I have submit a patch too, regarding colospace in prores: http://ffmpeg.org/pipermail/ffmpeg-devel/2018-October/235034.html You have also in your patch 2, the same purpose (take colorspace from picture), do you have any objection to take it from the CodecContext as it works like

[FFmpeg-devel] [PATCH] LICENSE: add libdavs2 and libxavs2 into compatible libraries section

2018-10-10 Thread hwren
Signed-off-by: hwren --- LICENSE.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/LICENSE.md b/LICENSE.md index ba65b05..1340ee4 100644 --- a/LICENSE.md +++ b/LICENSE.md @@ -83,11 +83,13 @@ affect the licensing of binaries resulting from the combination. The following libraries are

Re: [FFmpeg-devel] avcodec/prores aw improvment

2018-10-10 Thread Martin Vignali
Le mer. 10 oct. 2018 à 11:52, Marc-Antoine ARNAUD < arnaud.marcanto...@gmail.com> a écrit : > Great patches ;-) > I have submit a patch too, regarding colospace in prores: > http://ffmpeg.org/pipermail/ffmpeg-devel/2018-October/235034.html > You have also in your patch 2, the same purpose (take

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-10 Thread Marc-Antoine ARNAUD
I have updated the patch with our discussion. It took information only from the codec context. Marc-Antoine Le mer. 10 oct. 2018 à 11:36, Paul B Mahol a écrit : > On 10/10/18, Marc-Antoine ARNAUD wrote: > > For me it's the only codec who use picture colorspace as source. > > All others uses

Re: [FFmpeg-devel] [PATCH 0/6] Improvements for EBU R128 plugin (third round)

2018-10-10 Thread Paul B Mahol
On 10/10/18, Paul B Mahol wrote: > On 10/10/18, Daniel Molkentin wrote: >> On 09.10.2018 14:32, Daniel Molkentin wrote: >>> Changes over second round: >>> >>> - gauge option: Add 'm' and 's' as alias parameter for >>>momentary and short-term >>> - fix spelling error in documentation >>> -

Re: [FFmpeg-devel] [PATCH] avcodec/wmaprodec: improve WMAPRO/XMA gapless output

2018-10-10 Thread Rostislav Pehlivanov
On Wed, 10 Oct 2018 at 00:15, wrote: > From: bnnm > > Improves trac issue #6722. Fixes truncated WMAPRO/XMA output due to delay > samples and partially applies bitstream gapless info. > > This can be tested with files encoding a small nb_samples (like 400), > which couldn't output anything due

[FFmpeg-devel] [PATCH] avformat/dashenc: Dont ignore the codec tag from codec parameters

2018-10-10 Thread Jeyapal, Karthick
My git send-email is not working. Hence attaching the patch. Regards, Karthick 0001-avformat-dashenc-Dont-ignore-the-codec-tag-from-code.patch Description: 0001-avformat-dashenc-Dont-ignore-the-codec-tag-from-code.patch ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] avcodec: add Amuse Graphics decoder

2018-10-10 Thread Paul B Mahol
This work is partially sponsored by VideoLAN. Signed-off-by: Paul B Mahol --- The AGM3 variant decodes with some artifacts. --- libavcodec/Makefile | 1 + libavcodec/agm.c| 692 libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h|

Re: [FFmpeg-devel] [PATCH] avcodec/wmaprodec: improve WMAPRO/XMA gapless output

2018-10-10 Thread Banana M.
> No, you need to set this during init. > You should also set avctx->delay to the same value. This value is part of the bitstream, so theoretically can appear in any packet after the first. Can you confirm it doesn't? How could it be set on init? You'd need to read the bitstream then rewind? And

Re: [FFmpeg-devel] [PATCH] avcodec/wmaprodec: improve WMAPRO/XMA gapless output

2018-10-10 Thread Banana M.
> Can you confirm that if you break the decoder intentionally, the test > (still) fails? > > Thank you, Carl Eugen It does. Since file now writes 1 extra packet (as intended) so it'll always differ SIZE_TOLERANCE at min, so decoder changes will go over than value, I think. I tried -frames:a 108

Re: [FFmpeg-devel] swscale/x86/rgb2rgb : port shuffle2103 to external asm

2018-10-10 Thread Michael Niedermayer
On Tue, Oct 09, 2018 at 10:52:58PM +0200, Martin Vignali wrote: > > > > porting code and removing code sounds like 2 seperate things, > > that should not be in the same patch > > > > Split patch in attach the split should be the removal of the code that stays removed the porting of the remaining

[FFmpeg-devel] [PATCH 3/4] avcodec/ilbcdec: Fix multiple invalid left shifts

2018-10-10 Thread Michael Niedermayer
Fixes: 10651/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ILBC_fuzzer-5202341540659200 Fixes: left shift of negative value -512 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/ilbcdec.c

[FFmpeg-devel] [PATCH 4/4] avcodec/ilbcdec: Fix multiple integer overflows

2018-10-10 Thread Michael Niedermayer
Fixes: 10651/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ILBC_fuzzer-5202341540659200 Fixes: signed integer overflow: -1707705920 - 1703592888 cannot be represented in type 'int' This tries to follow the webrtc code. For example using cliping and 64 bit as in

[FFmpeg-devel] [PATCH 2/4] avcodec/ilbcdec: Limit indexes in create_augmented_vector()

2018-10-10 Thread Michael Niedermayer
These limits are based on limiting done in WebRtcIlbcfix_CreateAugmentedVec() Fixes: out of array accesses Fixes: 10652/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ILBC_fuzzer-5638941487661056 Fixes: 10655/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ILBC_fuzzer-5699970020147200

Re: [FFmpeg-devel] [PATCH] avcodec/qsvdec:flush the buffered data before reinit

2018-10-10 Thread Li, Zhong
> From: Fu, Linjie > Sent: Sunday, September 30, 2018 9:01 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie ; Li, Zhong > Subject: [PATCH] avcodec/qsvdec:flush the buffered data before reinit > > Flush the buffered data in libmfx before video param reinit in case the > frames drop. > > Cache