Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-12 Thread Thomas Mundt
Hi Thierry, 2017-09-12 3:25 GMT+02:00 Thierry Foucu : > Fix ticket #2674 > Tested with examples from ticket 2674. > --- > > Update the Patch with the correct number of duplicate showing. > For exmaple, in case we up-sample one second video at 24fps by 2, we > should be getting 24 duplicate frames

Re: [FFmpeg-devel] [PATCH 8/8] doc: update filter_design.txt.

2017-09-12 Thread Nicolas George
Le primidi 21 fructidor, an CCXXV, Nicolas George a écrit : > Signed-off-by: Nicolas George > --- > doc/filter_design.txt | 251 > +++--- > 1 file changed, 135 insertions(+), 116 deletions(-) Series pushed. (Without the zoompan fixes, this heap of bu

Re: [FFmpeg-devel] [PATCH 12/14] lavfi: Add OpenCL overlay filter

2017-09-12 Thread Nicolas George
Le quintidi 25 fructidor, an CCXXV, James Almer a écrit : > Yes, it moves things to configure where many filters already have > assorted non lavfi dependencies listed there (fft, etc), so the end > result of having the Makefile listing only the filter specific file(s) > for each OBJS entry is IMO w

[FFmpeg-devel] [PATCH] lavfi/framesync: reword repeatlast option help.

2017-09-12 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/filters.texi| 6 +++--- libavfilter/framesync.c | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 4111532512..830de54909 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -337,9 +337

[FFmpeg-devel] [PATCH] lavfi/framesync: remove dead code.

2017-09-12 Thread Nicolas George
Fix CID 1416960. Signed-off-by: Nicolas George --- libavfilter/framesync.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavfilter/framesync.c b/libavfilter/framesync.c index ebfbcaf090..be9f99c451 100644 --- a/libavfilter/framesync.c +++ b/libavfilter/framesync.c @@ -

[FFmpeg-devel] [PATCH] lavfi/buffersink: deprecate non-AVOption init.

2017-09-12 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/APIchanges | 3 +++ libavfilter/buffersink.c | 10 ++ libavfilter/buffersink.h | 12 3 files changed, 21 insertions(+), 4 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index cc67cbf6f8..be136ca11e 100644 --- a/doc/

Re: [FFmpeg-devel] [PATCH] lavfi/framesync: reword repeatlast option help.

2017-09-12 Thread Paul B Mahol
On 9/12/17, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > doc/filters.texi| 6 +++--- > libavfilter/framesync.c | 2 +- > 2 files changed, 4 insertions(+), 4 deletions(-) > lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] [PATCH] lavfi/framesync: remove dead code.

2017-09-12 Thread Paul B Mahol
On 9/12/17, Nicolas George wrote: > Fix CID 1416960. > > Signed-off-by: Nicolas George > --- > libavfilter/framesync.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpe

Re: [FFmpeg-devel] [PATCH] lavfi/framesync: reword repeatlast option help.

2017-09-12 Thread Nicolas George
Le sextidi 26 fructidor, an CCXXV, Paul B Mahol a écrit : > lgtm Thanks, pushed. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpe

Re: [FFmpeg-devel] [PATCH] lavfi/framesync: remove dead code.

2017-09-12 Thread Nicolas George
Le sextidi 26 fructidor, an CCXXV, Paul B Mahol a écrit : > lgtm Thanks, pushed. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpe

[FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc lpf msa functions

2017-09-12 Thread kaustubh.raste
From: Kaustubh Raste Seperate the filter processing in all strong, all weak and strong + weak cases. Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_lpf_sao_msa.c | 750 ++-- 1 file changed, 556 insertions(+), 194 deletions(-) diff --git a/libavcodec/mi

[FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc idct msa functions

2017-09-12 Thread kaustubh.raste
From: Kaustubh Raste Align the buffers. Remove reduandant constant array. Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_idct_msa.c | 255 ++- 1 file changed, 171 insertions(+), 84 deletions(-) diff --git a/libavcodec/mips/hevc_idct_msa.c b/libavco

Re: [FFmpeg-devel] [PATCH] frame_thread_encoder: make 'exit' member atomic.

2017-09-12 Thread Ronald S. Bultje
Hi, On Mon, Sep 11, 2017 at 5:30 PM, Michael Niedermayer wrote: > On Mon, Sep 11, 2017 at 09:58:55AM -0400, Ronald S. Bultje wrote: > > Should fix the following tsan warning: > > > > WARNING: ThreadSanitizer: data race (pid=19806) > > Read of size 4 at 0x7b8412f0 by thread T9: > > #0 w

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: Replace more literal magic numbers by enum values.

2017-09-12 Thread Michael Niedermayer
On Sun, Sep 10, 2017 at 10:28:05PM +0200, Tomas Härdin wrote: > On Sun, 2017-09-10 at 22:10 +0200, Michael Niedermayer wrote: > >  enum { > >  INDEX_MPEG2 = 0, > >  INDEX_AES3, > > @@ -159,6 +139,26 @@ enum { > >  INDEX_H264, > >  }; > >   > > +static const struct { > > +enum AVCode

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: Replace more literal magic numbers by enum values.

2017-09-12 Thread Michael Niedermayer
On Sun, Sep 10, 2017 at 10:28:05PM +0200, Tomas Härdin wrote: > On Sun, 2017-09-10 at 22:10 +0200, Michael Niedermayer wrote: > >  enum { > >  INDEX_MPEG2 = 0, > >  INDEX_AES3, > > @@ -159,6 +139,26 @@ enum { > >  INDEX_H264, > >  }; > >   > > +static const struct { > > +enum AVCode

[FFmpeg-devel] [PATCH] vp9: fix explicit memory order for report_progress.

2017-09-12 Thread Ronald S. Bultje
--- libavcodec/vp9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 66ccb6c..6b5de19 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -73,7 +73,7 @@ static int vp9_alloc_entries(AVCodecContext *avctx, int n) { static void vp9

Re: [FFmpeg-devel] [PATCH] vp9: fix explicit memory order for report_progress.

2017-09-12 Thread Clément Bœsch
On Tue, Sep 12, 2017 at 03:16:43PM -0400, Ronald S. Bultje wrote: > --- > libavcodec/vp9.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > index 66ccb6c..6b5de19 100644 > --- a/libavcodec/vp9.c > +++ b/libavcodec/vp9.c > @@ -73,7 +73,

Re: [FFmpeg-devel] [PATCH] vp9: fix explicit memory order for report_progress.

2017-09-12 Thread Ronald S. Bultje
Hi, On Tue, Sep 12, 2017 at 3:18 PM, Clément Bœsch wrote: > On Tue, Sep 12, 2017 at 03:16:43PM -0400, Ronald S. Bultje wrote: > > --- > > libavcodec/vp9.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > > index 66ccb6c..6b5de1

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-12 Thread Thierry Foucu
On Tue, Sep 12, 2017 at 1:35 AM, Thomas Mundt wrote: > Hi Thierry, > > 2017-09-12 3:25 GMT+02:00 Thierry Foucu : > >> Fix ticket #2674 >> Tested with examples from ticket 2674. >> --- >> >> Update the Patch with the correct number of duplicate showing. >> For exmaple, in case we up-sample one sec

Re: [FFmpeg-devel] [PATCH 01/18] h264_sei: Add namespace prefix to all SEI values

2017-09-12 Thread Mark Thompson
On 12/09/17 02:43, James Almer wrote: > On 8/20/2017 7:41 PM, Mark Thompson wrote: >> This avoids confusion with equivalent H.265 SEI values when both are >> being used at the same time. >> >> (cherry picked from commit 6ea220cbeec8863e2006a03b73bed52db2b13ee7) >> --- >> libavcodec/h264_parser.c

[FFmpeg-devel] [PATCH 01/20] h264: Add stream constraint values to the common header

2017-09-12 Thread Mark Thompson
With comments describing the derivation of each value. (cherry picked from commit aaf441465080b9bc57f5ca8dea656f9b2c5dc821) --- libavcodec/h264.h | 45 + 1 file changed, 45 insertions(+) diff --git a/libavcodec/h264.h b/libavcodec/h264.h index 86df5eb9

[FFmpeg-devel] [PATCH 00/20] Coded bitstream editing (v2)

2017-09-12 Thread Mark Thompson
Re-merged and updated. Changes: * All overreads fixed. * FATE tests using conformance test streams added. * MPEG-2 VAAPI added. * Many minor fixes, including review comments from v1. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org

[FFmpeg-devel] [PATCH 02/20] hevc: Improve stream constraint values in common header

2017-09-12 Thread Mark Thompson
Add comments to describe the sources of the constraint values expressed here, and add some more related values which will be used in following patches. Fix the incorrect values for SPS and PPS count (they are not the same as those used for H.264), and remove HEVC_MAX_CU_SIZE because it is not used

[FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-09-12 Thread Mark Thompson
(cherry picked from commit 18f1706f331bf5dd565774eae680508c8d3a97ad) (cherry picked from commit 44cde38c8acbef7d5250e6d1b52b1020871e093b) --- configure | 1 + libavcodec/Makefile | 1 + libavcodec/cbs.c | 466 ++ libavc

[FFmpeg-devel] [PATCH 04/20] lavc: Add coded bitstream read/write support for H.264

2017-09-12 Thread Mark Thompson
(cherry picked from commit acf06f45441be24c5cbae0920579cd69427326a1) (cherry picked from commit 768eb9182e94a94bc2ef46f565a0dac7afef3b57) (cherry picked from commit e7f64191b27bcf37cbf7006606f0f439c6cdc24f) --- configure |2 + libavcodec/Makefile |

[FFmpeg-devel] [PATCH 06/20] lavc: Add coded bitstream read/write support for MPEG-2

2017-09-12 Thread Mark Thompson
(cherry picked from commit 2bc9ba8d3c41f3a8e56484bd67b05040c7909a01) (cherry picked from commit a41b69b5eb950c10d8ede472bcc4e88ce4246db9) --- configure | 2 + libavcodec/Makefile| 1 + libavcodec/cbs.c | 3 + libavcodec/c

[FFmpeg-devel] [PATCH 10/20] lavc: Add hevc_metadata bitstream filter

2017-09-12 Thread Mark Thompson
This is able to modify some header metadata found in the VPS/SPS/VUI, and can also add/remove AUDs. (cherry picked from commit b31a9eae0233325c4b382c657f4b687d5d8b0812) --- configure | 1 + doc/bitstream_filters.texi | 54 + libavcodec/Makefile| 1 +

[FFmpeg-devel] [PATCH 05/20] lavc: Add coded bitstream read/write support for H.265

2017-09-12 Thread Mark Thompson
(cherry picked from commit 867381b8b51fa21fa2b8f071f508f3d39cc9c1f0) (cherry picked from commit f763489364416bb6866adc4f4a96012dd2ca1bd0) (cherry picked from commit 067a9ddeb8feff1f724856f0054930c55219f76b) --- configure |2 + libavcodec/Makefile |

[FFmpeg-devel] [PATCH 11/20] lavc: Add mpeg2_metadata bitstream filter

2017-09-12 Thread Mark Thompson
(cherry picked from commit b78c30d7ec26af67c00ce2002709a189f6a87a7e) --- configure | 1 + doc/bitstream_filters.texi | 36 libavcodec/Makefile | 1 + libavcodec/bitstream_filters.c | 1 + libavcodec/mpeg2_metadata_bsf.c | 360

[FFmpeg-devel] [PATCH 12/20] vaapi_h264: Convert to use coded bitstream infrastructure

2017-09-12 Thread Mark Thompson
(cherry picked from commit 7a4fac5e91789b73e07bd4ad20493cfde028df76) --- configure |2 +- libavcodec/Makefile|2 +- libavcodec/vaapi_encode_h264.c | 1425 3 files changed, 551 insertions(+), 878 deletions(-) diff --

[FFmpeg-devel] [PATCH 08/20] lavc: Add h264_metadata bitstream filter

2017-09-12 Thread Mark Thompson
This is able to modify some header metadata found in the SPS/VUI, and can also add/remove AUDs and insert user data in SEI NAL units. (cherry picked from commit 9e93001b6135a23fe4e200196c08fb4fbffed6fc) (cherry picked from commit c42b62d1f9641f10ffc23cad9abbe47d8a4a165b) --- configure

[FFmpeg-devel] [PATCH 13/20] vaapi_h264: Add support for AUD NAL units

2017-09-12 Thread Mark Thompson
Adds a new private option to enable them (off by default). (cherry picked from commit 820a4483af13cf6fd51f13638e57bcd1c3f629d4) --- libavcodec/vaapi_encode_h264.c | 33 + 1 file changed, 33 insertions(+) diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaa

[FFmpeg-devel] [PATCH 09/20] lavc: Add h264_redundant_pps bitstream filter

2017-09-12 Thread Mark Thompson
This applies a specific fixup to some Bluray streams which contain redundant PPSs modifying irrelevant parameters of the stream which confuse other transformations which require correct extradata. A new single global PPS is created, and all of the redundant PPSs within the stream are removed. (ch

[FFmpeg-devel] [PATCH 07/20] lavc: Add trace_headers bitstream filter

2017-09-12 Thread Mark Thompson
Supports all streams that the coded bitstream infrastructure does (currently H.264, H.265 and MPEG-2). (cherry picked from commit f11d8a5e8b185340cc50fcbc8a1437b0fbe7e931) --- configure | 1 + doc/bitstream_filters.texi | 8 +++ libavcodec/Makefile| 1 +

[FFmpeg-devel] [PATCH 14/20] vaapi_h264: Add support for SEI recovery points

2017-09-12 Thread Mark Thompson
Included by default with non-IDR intra frames. (cherry picked from commit a49ee60d5fdbdae1706a44cfbb814abb9793815f) --- libavcodec/vaapi_encode_h264.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_

[FFmpeg-devel] [PATCH 17/20] vaapi_h265: Add support for AUD NAL units

2017-09-12 Thread Mark Thompson
Matching the H.264 encoder. (cherry picked from commit e3e8eab359238486dc233f7aa89b7bb3cb19ec38) --- libavcodec/vaapi_encode_h265.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c index fd

[FFmpeg-devel] [PATCH 15/20] vaapi_h264: Fix CPB/DPB delays

2017-09-12 Thread Mark Thompson
This should be ticks, not time_scale steps - it was wrong for all framerates not a multiple of 1/2. (cherry picked from commit 30645174e333c7db1946ab09be769db9be9d7481) --- libavcodec/vaapi_encode_h264.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/vaapi_en

[FFmpeg-devel] [PATCH 18/20] mpeg12: Move finding the best frame rate to common code

2017-09-12 Thread Mark Thompson
Previously in the mpeg2_metadata filter. Also adds a test. (cherry picked from commit b5859e0b04bdbe12c97cb12ac10a45d51d2d73c9) --- libavcodec/Makefile| 1 + libavcodec/mpeg12.h| 4 ++ libavcodec/mpeg12framerate.c | 64 libavco

[FFmpeg-devel] [PATCH 16/20] vaapi_h265: Convert to use coded bitstream infrastructure

2017-09-12 Thread Mark Thompson
Also improves the metadata and generally makes the configuration a bit cleaner. (cherry picked from commit ac12486714b48f9bd5d9167f90b77c936751d6ef) --- configure |2 +- libavcodec/Makefile|2 +- libavcodec/vaapi_encode_h265.c | 1536 +++---

[FFmpeg-devel] [PATCH 19/20] vaapi_mpeg2: Convert to use coded bitstream infrastructure

2017-09-12 Thread Mark Thompson
(cherry picked from commit 10eb496d9ae94df6f792b0e1d8750738eb3a0952) --- configure | 2 +- libavcodec/vaapi_encode_mpeg2.c | 638 ++-- 2 files changed, 424 insertions(+), 216 deletions(-) diff --git a/configure b/configure index 2e617aed

[FFmpeg-devel] [PATCH 20/20] cbs: Add some read/write tests

2017-09-12 Thread Mark Thompson
Use the appropriate metadata filter for each codec - in the absence of any options to modify the stream, the output bitstream should be identical to the input (though the output file may differ in padding). All tests use conformance bitstreams, the MPEG-2 streams are newly added from the conforman

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-12 Thread Michael Niedermayer
On Mon, Sep 11, 2017 at 06:25:08PM -0700, Thierry Foucu wrote: > Fix ticket #2674 > Tested with examples from ticket 2674. > --- > > Update the Patch with the correct number of duplicate showing. > For exmaple, in case we up-sample one second video at 24fps by 2, we should > be getting 24 duplica

Re: [FFmpeg-devel] [PATCH 2/2] avformat/fitsenc: fill header line with spaces

2017-09-12 Thread James Almer
On 9/2/2017 4:07 PM, Paras Chadha wrote: > Signed-off-by: Paras Chadha > --- > This fixes the failed fate tests on msvc12: > http://fate.ffmpeg.org/report.cgi?time=20170901190511&slot=x86_32-msvc12-windows-native > http://fate.ffmpeg.org/report.cgi?time=20170901211336&slot=x86_64-msvc12-windows-na

Re: [FFmpeg-devel] [PATCH] avcodec/fitsdec: change to le pixel formats

2017-09-12 Thread James Almer
On 9/3/2017 12:05 PM, Paras Chadha wrote: > Signed-off-by: Paras Chadha > --- > This fixes the failed fate tests on ppc64be: > http://fate.ffmpeg.org/report.cgi?time=20170903050743&slot=ppc64be-RHEL7.0-gcc-4.8.5-ibmcrl > > libavcodec/fitsdec.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 del

[FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-12 Thread Thierry Foucu
Fix ticket #2674 Tested with examples from ticket 2674. --- Sorry Michael, I forgot to configure using --enable-gpl. Please find new patch. libavfilter/vf_fps.c | 44 +++- tests/ref/fate/filter-fps| 6 ++ tests/ref/fate/filter-fps-r

[FFmpeg-devel] [PATCH] avcodec/dnxhdenc: fix DNxHR 444 encoding crashes see https://trac.ffmpeg.org/ticket/6649

2017-09-12 Thread Frederic Devernay
From: Frédéric Devernay --- libavcodec/dnxhdenc.c | 16 libavcodec/dnxhdenc.h | 4 ++-- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/libavcodec/dnxhdenc.c b/libavcodec/dnxhdenc.c index 5a0e6de6a5..0d80381a2d 100644 --- a/libavcodec/dnxhdenc.c +++ b/libavcode