[FFmpeg-devel] [PATCH v2] Add MediaFoundation wrapper

2017-04-03 Thread wm4
Can do audio decoding, audio encoding, video decoding, video encoding, video HW encoding. I also had video HW decoding, but removed it for now, as thw hwframes integration wasn't very sane. Some of the MS codecs aren't well tested, and might not work properly. --- Didn't actually test compilation

Re: [FFmpeg-devel] [PATCH] HDR Transcode VUI Info Copy

2017-04-03 Thread wm4
On Mon, 3 Apr 2017 19:50:07 + Ben Chang wrote: > From c8092bc538ba7ae32357ca166e67809eb3705f60 Mon Sep 17 00:00:00 2001 > From: Ben Chang > Date: Mon, 27 Mar 2017 17:00:19 -0700 > Subject: [PATCH] Allow transfer of HDR-required VUI info from dec ctx to enc

[FFmpeg-devel] [PATCH] huffyuv: assign correct per-thread avctx pointer to HYuvContext::avctx.

2017-04-03 Thread Ronald S. Bultje
Fixes the following tsan warning when running fate-vsynth_lena-ffvhuff: WARNING: ThreadSanitizer: data race (pid=6484) Write of size 8 at 0x7d64000154b8 by main thread (mutexes: write M1331): #0 update_context_from_user src/libavcodec/pthread_frame.c:331 (ffmpeg+0x00dca887) [..]

Re: [FFmpeg-devel] [PATCH] mjpegenc: enable optimal huffman coding by default

2017-04-03 Thread Michael Niedermayer
On Mon, Apr 03, 2017 at 10:31:27PM +0100, Rostislav Pehlivanov wrote: > As it gives excellent encoding gains at an insignificant speed increase > and passes fate without problems, it should now be safe to enable by > default. Still breaks slice threading both Assertion n <= 31 && value < (1U <<

Re: [FFmpeg-devel] [PATCH 2/3] mjpegenc: disable huffman coding with AMV

2017-04-03 Thread Rostislav Pehlivanov
On 4 April 2017 at 00:02, Carl Eugen Hoyos wrote: > 2017-04-03 22:13 GMT+02:00 Rostislav Pehlivanov : > > > +if (s->huffman && avctx->codec_id == AV_CODEC_ID_AMV) { > > +av_log(avctx, AV_LOG_WARNING, "AMV doesn't support optimized > huffman

Re: [FFmpeg-devel] [PATCH 2/3] mjpegenc: disable huffman coding with AMV

2017-04-03 Thread Carl Eugen Hoyos
2017-04-03 22:13 GMT+02:00 Rostislav Pehlivanov : > +if (s->huffman && avctx->codec_id == AV_CODEC_ID_AMV) { > +av_log(avctx, AV_LOG_WARNING, "AMV doesn't support optimized huffman > tables, disabling\n"); Iiuc, together with 3/3 this prints a warning by default

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mov: Check creation_time for overflow

2017-04-03 Thread Michael Niedermayer
On Sat, Apr 01, 2017 at 07:18:36PM +0200, Michael Niedermayer wrote: > Fixes integer overflow > Fixes: 701640 > > Found-by: Found-by: Thomas Guilbert > Signed-off-by: Michael Niedermayer > --- > libavformat/mov.c | 6 ++ > 1 file changed, 6

Re: [FFmpeg-devel] [PATCH] fate: add owdenoise test

2017-04-03 Thread Michael Niedermayer
On Mon, Apr 03, 2017 at 02:32:15PM -0700, Thomas Turner wrote: > On Apr 2, 2017 5:21 PM, "Michael Niedermayer" > wrote: > > On Sun, Apr 02, 2017 at 03:27:57PM -0700, Thomas Turner wrote: > > Reference found at: http://bit.ly/2ogQygb > > > > Signed-off-by: Thomas Turner

Re: [FFmpeg-devel] [PATCH] fate: add owdenoise test

2017-04-03 Thread Thomas Turner
On Apr 2, 2017 5:21 PM, "Michael Niedermayer" wrote: On Sun, Apr 02, 2017 at 03:27:57PM -0700, Thomas Turner wrote: > Reference found at: http://bit.ly/2ogQygb > > Signed-off-by: Thomas Turner > --- > tests/fate/filter-video.mak | 7 +++ >

[FFmpeg-devel] [PATCH] mjpegenc: enable optimal huffman coding by default

2017-04-03 Thread Rostislav Pehlivanov
As it gives excellent encoding gains at an insignificant speed increase and passes fate without problems, it should now be safe to enable by default. Signed-off-by: Rostislav Pehlivanov --- Had to add some missing tests --- libavcodec/mjpegenc.c| 2 +-

Re: [FFmpeg-devel] [PATCH] frame_thread_encoder: make task indexing deterministic.

2017-04-03 Thread Michael Niedermayer
On Sun, Apr 02, 2017 at 07:30:00AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Apr 1, 2017 at 8:19 PM, Michael Niedermayer > wrote: > > > Supporting changing delay and number of threads at runtime was a > > big factor in the design of the frame thread encoder. Its

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Also read three-character language descriptors

2017-04-03 Thread Marton Balint
On Mon, 3 Apr 2017, Carl Eugen Hoyos wrote: Hi! Attached patch fixes setting the language for the sample from vlc ticket #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts Please comment, Carl Eugen From d8b30253cc6b1c2156f1e8898e0a0c2ff3c3f900 Mon Sep 17 00:00:00 2001 From:

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Paul B Mahol
On 4/3/17, Ricardo Constantino wrote: > On 3 April 2017 at 20:04, Paul B Mahol wrote: > >> On 4/3/17, Ricardo Constantino wrote: >> > On 3 April 2017 at 19:58, Paul B Mahol wrote: >> > >> >> >> >> Yes, but that conflict

[FFmpeg-devel] [PATCH 3/3] mjpegenc: enable optimal huffman coding by default

2017-04-03 Thread Rostislav Pehlivanov
As it gives excellent encoding gains at an insignificant speed increase and passes fate without problems, it should now be safe to enable by default. Signed-off-by: Rostislav Pehlivanov --- libavcodec/mjpegenc.c| 2 +- tests/ref/vsynth/vsynth1-mjpeg

[FFmpeg-devel] [PATCH 1/3] mjpegenc_common: check for codec ID before using avctx->priv_data

2017-04-03 Thread Rostislav Pehlivanov
When coding lossless jpeg the priv context will be pointing to LJpegEncContext rather than MpegEncContext, which the function expects. Signed-off-by: Rostislav Pehlivanov --- libavcodec/mjpegenc_common.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff

[FFmpeg-devel] [PATCH 2/3] mjpegenc: disable huffman coding with AMV

2017-04-03 Thread Rostislav Pehlivanov
Isn't supported. Signed-off-by: Rostislav Pehlivanov --- libavcodec/mpegvideo_enc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index 71a858fc72..8cc964be23 100644 --- a/libavcodec/mpegvideo_enc.c +++

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Also read three-character language descriptors

2017-04-03 Thread Paul B Mahol
On 4/3/17, Carl Eugen Hoyos wrote: > 2017-04-03 20:13 GMT+02:00 Paul B Mahol : >> On 4/3/17, Carl Eugen Hoyos wrote: >>> Hi! >>> >>> Attached patch fixes setting the language for the sample from vlc ticket >>> #10830:

[FFmpeg-devel] [PATCH] HDR Transcode VUI Info Copy

2017-04-03 Thread Ben Chang
Hi, This patch adds copy of HDR VUI info from decode ctx to encode ctx. Currently, information under colour_description_present_flag (eg. colour primaries, transfer_characteristics, matrix_coeffs) do not get copied to output stream when trancode happens. Testing performed: ffmpeg.exe -y

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Also read three-character language descriptors

2017-04-03 Thread Carl Eugen Hoyos
2017-04-03 20:13 GMT+02:00 Paul B Mahol : > On 4/3/17, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes setting the language for the sample from vlc ticket >> #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts >> >> Please comment, Carl

Re: [FFmpeg-devel] [PATCH] avfilter: add ANSNR filter

2017-04-03 Thread Ronald S. Bultje
Hi Betty, On Mon, Apr 3, 2017 at 11:00 AM, Betty Wu wrote: > +typedef double number_t; > Why? +static int ansnr_filter(const uint8_t* src_image, number_t* dst_image, int > img_row, int img_col, const double *filter, int stride) > [..] > +number_t **imme_image; > +

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Ricardo Constantino
On 3 April 2017 at 20:04, Paul B Mahol wrote: > On 4/3/17, Ricardo Constantino wrote: > > On 3 April 2017 at 19:58, Paul B Mahol wrote: > > > >> > >> Yes, but that conflict with expand behaviour. > >> > >> > > How so? Still works fine with

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Paul B Mahol
On 4/3/17, Ricardo Constantino wrote: > On 3 April 2017 at 19:58, Paul B Mahol wrote: > >> >> Yes, but that conflict with expand behaviour. >> >> > How so? Still works fine with squared pixels: > $ ffmpeg -f lavfi -i "color=s=hd720" -vf pad=aspect=4/3

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Ricardo Constantino
On 3 April 2017 at 19:58, Paul B Mahol wrote: > > Yes, but that conflict with expand behaviour. > > How so? Still works fine with squared pixels: $ ffmpeg -f lavfi -i "color=s=hd720" -vf pad=aspect=4/3 -vframes 1 -f null - -v verbose 2>&1 | grep Parsed_pad [Parsed_pad_0 @

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Paul B Mahol
On 4/3/17, Ricardo Constantino wrote: > On 3 April 2017 at 19:18, Paul B Mahol wrote: > >> On 4/3/17, Ricardo Constantino wrote: >> > On 2 April 2017 at 22:13, Paul B Mahol wrote: >> > >> >> On 4/2/17, Paul B Mahol

Re: [FFmpeg-devel] [PATCH] png: set AVFrame flags/fields before calling setup_finished().

2017-04-03 Thread Paul B Mahol
On 4/3/17, Ronald S. Bultje wrote: > Fixes tsan warnings in fate-apng: > > WARNING: ThreadSanitizer: data race (pid=51230) > Read of size 4 at 0x7d5042fc by main thread (mutexes: write M1000): > #0 frame_copy_props frame.c:302 (ffmpeg:x86_64+0x1019a35d6) > [..] >

[FFmpeg-devel] [PATCH] png: set AVFrame flags/fields before calling setup_finished().

2017-04-03 Thread Ronald S. Bultje
Fixes tsan warnings in fate-apng: WARNING: ThreadSanitizer: data race (pid=51230) Read of size 4 at 0x7d5042fc by main thread (mutexes: write M1000): #0 frame_copy_props frame.c:302 (ffmpeg:x86_64+0x1019a35d6) [..] Previous write of size 4 at 0x7d5042fc by thread T1 (mutexes:

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Ricardo Constantino
On 3 April 2017 at 19:18, Paul B Mahol wrote: > On 4/3/17, Ricardo Constantino wrote: > > On 2 April 2017 at 22:13, Paul B Mahol wrote: > > > >> On 4/2/17, Paul B Mahol wrote: > >> > Signed-off-by: Paul B Mahol

[FFmpeg-devel] [PATCH] vf_pad: center image on padded area if negative x/y

2017-04-03 Thread Ricardo Constantino
or if x/y go beyond padded area. This is mostly useful when paired with the aspect option. Defaults aren't changed. Idea for this was taken from mpv's soon-to-be-removed expand vf. --- doc/filters.texi | 3 +++ libavfilter/vf_pad.c | 12 +++- 2 files changed, 10 insertions(+), 5

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Paul B Mahol
On 4/3/17, Ricardo Constantino wrote: > On 2 April 2017 at 22:13, Paul B Mahol wrote: > >> On 4/2/17, Paul B Mahol wrote: >> > Signed-off-by: Paul B Mahol >> > --- >> > doc/filters.texi | 3 +++ >> >

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Also read three-character language descriptors

2017-04-03 Thread Paul B Mahol
On 4/3/17, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes setting the language for the sample from vlc ticket > #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts > > Please comment, Carl Eugen > Breaks indentation, nak.

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pad: add aspect option

2017-04-03 Thread Ricardo Constantino
On 2 April 2017 at 22:13, Paul B Mahol wrote: > On 4/2/17, Paul B Mahol wrote: > > Signed-off-by: Paul B Mahol > > --- > > doc/filters.texi | 3 +++ > > libavfilter/vf_pad.c | 14 ++ > > 2 files changed, 17 insertions(+) >

[FFmpeg-devel] [PATCH]lavf/mpegts: Also read three-character language descriptors

2017-04-03 Thread Carl Eugen Hoyos
Hi! Attached patch fixes setting the language for the sample from vlc ticket #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts Please comment, Carl Eugen From d8b30253cc6b1c2156f1e8898e0a0c2ff3c3f900 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon,

Re: [FFmpeg-devel] [PATCH] doc/muxers: fix default value for image2 option start_number.

2017-04-03 Thread Lou Logan
On Mon, Apr 3, 2017, at 02:14 AM, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > doc/muxers.texi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

2017-04-03 Thread Carl Eugen Hoyos
2017-04-03 13:34 GMT+02:00 Ricardo Constantino : > On 3 April 2017 at 10:56, Carl Eugen Hoyos wrote: > >> 2017-04-01 16:37 GMT+02:00 Ricardo Constantino : >> > On 1 April 2017 at 14:03, Hendrik Leppkes wrote: >> > >>

Re: [FFmpeg-devel] [PATCH] png: split header state and data state in two separate variables.

2017-04-03 Thread Michael Niedermayer
On Mon, Apr 03, 2017 at 10:09:30AM -0400, Ronald S. Bultje wrote: > Fixes a reported (but false) race condition in tsan for fate-apng: > > WARNING: ThreadSanitizer: data race (pid=6274) > Read of size 4 at 0x7d680001ec78 by main thread (mutexes: write M1338): > #0 update_thread_context

Re: [FFmpeg-devel] [PATCH] avfilter: add ANSNR filter

2017-04-03 Thread Steven Liu
2017-04-03 23:00 GMT+08:00 Betty Wu : > A new filter ANSNR is added. libavfilter/Makefile is changed. > Run 'ffmpeg -i input1 -i input2 -lavfi ansnr -f null -' to get an overall > score while per-frame value is stored but not printed. > This implementation is for constructing

[FFmpeg-devel] [PATCH] avfilter: add ANSNR filter

2017-04-03 Thread Betty Wu
A new filter ANSNR is added. libavfilter/Makefile is changed. Run 'ffmpeg -i input1 -i input2 -lavfi ansnr -f null -' to get an overall score while per-frame value is stored but not printed. This implementation is for constructing the vmaf filter later since ANSNR is one of individual tools used

[FFmpeg-devel] [PATCH] pthread_frame: allow per-field ThreadFrame owners.

2017-04-03 Thread Ronald S. Bultje
This tries to handle cases where separate invocations of decode_frame() (each running in separate threads) write to respective fields in the same AVFrame->data[]. Having per-field owners makes interaction between readers (the referencing thread) and writers (the decoding thread) slightly more

[FFmpeg-devel] [PATCH] png: split header state and data state in two separate variables.

2017-04-03 Thread Ronald S. Bultje
Fixes a reported (but false) race condition in tsan for fate-apng: WARNING: ThreadSanitizer: data race (pid=6274) Read of size 4 at 0x7d680001ec78 by main thread (mutexes: write M1338): #0 update_thread_context src/libavcodec/pngdec.c:1456 (ffmpeg+0x00dacf0c) [..] Previous write of

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/hevc_parse: check for parameter set decoding failure

2017-04-03 Thread James Almer
On 4/3/2017 7:00 AM, Michael Niedermayer wrote: > On Sun, Apr 02, 2017 at 10:45:41PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavcodec/hevc_parse.c | 32 +--- >> 1 file changed, 25 insertions(+), 7 deletions(-) >> >> diff

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: use reordered duration for stream PTS.

2017-04-03 Thread wm4
On Mon, 3 Apr 2017 14:52:15 +0200 Nicolas George wrote: > Signed-off-by: Nicolas George > --- > ffmpeg.c | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > > Not sure if this is correct, the timestamp handling is awfully complicated. > But

[FFmpeg-devel] [PATCH 2/2] ffmpeg: use reordered duration for stream PTS.

2017-04-03 Thread Nicolas George
Signed-off-by: Nicolas George --- ffmpeg.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) Not sure if this is correct, the timestamp handling is awfully complicated. But FATE succeeds and the result is a correct value of next_pts after the last frame with a VFR

[FFmpeg-devel] [PATCH 1/2] ffmpeg: rename a variable.

2017-04-03 Thread Nicolas George
Makes the reason of the "FIXME" comment more obvious. Avoid name conflicts for the next commit. Signed-off-by: Nicolas George --- ffmpeg.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 11faf0d4a8..02ff073615 100644

[FFmpeg-devel] [PATCH 2/3] doc/examples/extract_mvs: make pkt local to the main function

2017-04-03 Thread Matthieu Bouron
--- doc/examples/extract_mvs.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c index d6730db3a2..552a7334cf 100644 --- a/doc/examples/extract_mvs.c +++ b/doc/examples/extract_mvs.c @@ -31,13 +31,11 @@

[FFmpeg-devel] [PATCH 1/3] doc/examples/extract_mvs: switch to new decoding API

2017-04-03 Thread Matthieu Bouron
--- doc/examples/extract_mvs.c | 72 ++ 1 file changed, 35 insertions(+), 37 deletions(-) diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c index 8b22b40c11..d6730db3a2 100644 --- a/doc/examples/extract_mvs.c +++

[FFmpeg-devel] [PATCH 3/3] doc/examples/extract_mvs: re-indent after previous commit

2017-04-03 Thread Matthieu Bouron
--- doc/examples/extract_mvs.c | 56 +++--- 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c index 552a7334cf..7ae934ead3 100644 --- a/doc/examples/extract_mvs.c +++

Re: [FFmpeg-devel] [PATCH v3] avutil/avstring: improve av_strreplace implement

2017-04-03 Thread Nicolas George
Le quartidi 14 germinal, an CCXXV, Steven Liu a écrit : > change name from av_strreplace to av_strireplace > Use AVBprint to implement av_strireplace > add av_strireplace test case TEST_STRIREPLACE > > Signed-off-by: Steven Liu > --- > libavutil/avstring.c | 76 >

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

2017-04-03 Thread Nicolas George
Le quartidi 14 germinal, an CCXXV, Ricardo Constantino a écrit : > I was talking about the headers, which is the only part that matters to > FFmpeg. I do not know the specifics, so just for references: The license that matter is the license of ALL the files that are needed at build time. > The

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

2017-04-03 Thread Ricardo Constantino
On 3 April 2017 at 10:56, Carl Eugen Hoyos wrote: > 2017-04-01 16:37 GMT+02:00 Ricardo Constantino : > > On 1 April 2017 at 14:03, Hendrik Leppkes wrote: > > > >> > >> This is the Boost Software license, very similar to BSD/MIT, so if

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread wm4
On Mon, 3 Apr 2017 18:20:35 +0800 Steven Liu wrote: > 2017-04-03 18:12 GMT+08:00 wm4 : > > > On Mon, 3 Apr 2017 11:25:39 +0200 > > Michael Niedermayer wrote: > > > > > On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread Steven Liu
2017-04-03 18:12 GMT+08:00 wm4 : > On Mon, 3 Apr 2017 11:25:39 +0200 > Michael Niedermayer wrote: > > > On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote: > > > On 04/03/2017 06:35 PM, Nicolas George wrote: > > > >Hi. Thanks for the patch.

[FFmpeg-devel] [PATCH] doc/muxers: fix default value for image2 option start_number.

2017-04-03 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/muxers.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) In principle I would rather fix the default value, but that may break a lot of users' workflow. diff --git a/doc/muxers.texi b/doc/muxers.texi index 166c929369..844bbce990

Re: [FFmpeg-devel] H.261 chroma motion vector rounding.

2017-04-03 Thread wm4
On Mon, 3 Apr 2017 10:31:15 +0100 Christina Brien wrote: > > For both positive and negative numbers. > > > > 3 / 2 = 1 > > -3 / 2 = -1 > > You’re right. I was confusing integer division by a power of two, with > right-shifting. The bug was in my code

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread wm4
On Mon, 3 Apr 2017 11:25:39 +0200 Michael Niedermayer wrote: > On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote: > > On 04/03/2017 06:35 PM, Nicolas George wrote: > > >Hi. Thanks for the patch. > > > > > >Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit

Re: [FFmpeg-devel] [RFC] FFmpeg 3.3

2017-04-03 Thread Carl Eugen Hoyos
2017-04-02 21:15 GMT+02:00 Michael Niedermayer : > i intend to make 3.3 from this branch within a few days Please wait for a resolution for the decklink license issue: Either the library is free (as explained here) or non-free as was claimed on the Debian bugtracker.

Re: [FFmpeg-devel] [PATCH 3/5] avcodec/hevc_parse: ignore all non parameter set NAL units in extradata

2017-04-03 Thread Michael Niedermayer
On Sun, Apr 02, 2017 at 10:45:43PM -0300, James Almer wrote: > While they shouldn't be present, they are harmless if they are. > > Signed-off-by: James Almer > --- > libavcodec/hevc_parse.c | 21 ++--- > 1 file changed, 2 insertions(+), 19 deletions(-) should

Re: [FFmpeg-devel] [PATCH]configure: Fix decklink license dependency

2017-04-03 Thread Carl Eugen Hoyos
2017-04-01 16:37 GMT+02:00 Ricardo Constantino : > On 1 April 2017 at 14:03, Hendrik Leppkes wrote: > >> >> This is the Boost Software license, very similar to BSD/MIT, so if >> that license is all that matters, its obviously fine. >> >> > Just downloaded

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/hevc_parse: check for parameter set decoding failure

2017-04-03 Thread Michael Niedermayer
On Sun, Apr 02, 2017 at 10:45:41PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/hevc_parse.c | 32 +--- > 1 file changed, 25 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/hevc_parse.c

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread Nicolas George
Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit : > Thanks for your quick reply. Regarding the almost direct copy of code from > cache.c, I previously submitted a patch on 31 March that adds a "cache_file" > option to the cache protocol. It was intended to allow a specifically named >

Re: [FFmpeg-devel] H.261 chroma motion vector rounding.

2017-04-03 Thread Christina Brien
> For both positive and negative numbers. > > 3 / 2 = 1 > -3 / 2 = -1 You’re right. I was confusing integer division by a power of two, with right-shifting. The bug was in my code all along. I apologise for wasting your time. Christina ___

Re: [FFmpeg-devel] Match source video timestamp

2017-04-03 Thread Eran Kornblau
Ping From: Eran Kornblau Sent: Tuesday, March 28, 2017 3:13 PM To: FFmpeg development discussions and patches Subject: Match source video timestamp Hi all, I'm trying to transcode some video file (MP4/h264 baseline vfr/AAC) with ffmpeg, and I would like to have the

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread Michael Niedermayer
On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote: > On 04/03/2017 06:35 PM, Nicolas George wrote: > >Hi. Thanks for the patch. > > > >Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit : > >>Capture is an input stream capture protocol that dumps the input stream to a > >>file. The

Re: [FFmpeg-devel] [PATCH 1/3] avutil/spherical: add av_spherical_projection_name()

2017-04-03 Thread Vittorio Giovara
On Fri, Mar 31, 2017 at 7:15 PM, James Almer wrote: > On 3/31/2017 5:42 AM, Vittorio Giovara wrote: >> On Wed, Mar 29, 2017 at 4:55 AM, James Almer wrote: >>> Signed-off-by: James Almer >>> --- >>> doc/APIchanges| 3 +++ >>>

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread Timothy Lee
On 04/03/2017 06:35 PM, Nicolas George wrote: Hi. Thanks for the patch. Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit : Capture is an input stream capture protocol that dumps the input stream to a file. The default name of the output file is "capture.dat", but it can be changed using

Re: [FFmpeg-devel] [PATCH 1/3] libavformat: add "capture:" protocol

2017-04-03 Thread Nicolas George
Hi. Thanks for the patch. Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit : > Capture is an input stream capture protocol that dumps the input stream to a > file. The default name of the output file is "capture.dat", but it can be > changed using the "capture_file" option. > > capture.c

[FFmpeg-devel] [PATCH 2/3] lavc: Add Content Light Level side metadata found in HEVC

2017-04-03 Thread Steve Lhomme
These data are necessary when transmitting HDR over HDMI. -- update the previous patch: - rename CEA 861.3 to CTA-861.3 - update with MaxCLL/MaxFALL changes in avutil update the previous patch: - rebased and updated version bump --- libavcodec/avcodec.h | 7 +++ libavcodec/avpacket.c | 1

[FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-03 Thread Steve Lhomme
As found in HEVC. I put the code in mastering_display_metadata as they usually go together in Blu-Ray UHD sources. -- update the previous patch: - rename CEA 861.3 to CTA-861.3 - use MaxCLL and MaxFALL names that are more commonly found - use unsigned integer rather than rational numbers as

Re: [FFmpeg-devel] H.261 chroma motion vector rounding.

2017-04-03 Thread Hendrik Leppkes
On Sun, Apr 2, 2017 at 11:14 PM, Christina Brien wrote: > >> On 2 Apr 2017, at 22:03, Hendrik Leppkes wrote: >> >> C integer divison truncates, that equals rounding towards zero. > > For positive numbers, yes. > 3 (0b