On Fri, 31 Mar 2017 03:49:16 +0200
Michael Niedermayer wrote:
> On Thu, Mar 30, 2017 at 05:05:36PM +0200, wm4 wrote:
> > Not sure if it behaves ideally in presence of decoding errors.
> > ---
> > The original version of this patch is actually 1 year old.
> > ---
> > ffprobe.c | 42 ++
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
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
> ctx
>
> ---
> ffmpeg.c | 3 +++
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)
[..]
Previ
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 << n
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 tables, disabling\n");
>
> Iiuc, together wi
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 when encoding
AMV, th
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 insertions(+)
applied
[...]
--
Michael GnuP
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
> > ---
> > tests/fate/fi
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 +++
> 1 file changed, 7 insertions(+)
reference file upl
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 +-
tests/ref/lavf/jpg
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 something
> > i cared abou
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:
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 with expand behaviour.
>> >>
>> >>
>> > How so? Still works fine with squar
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 | 4 ++--
tests/
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 --git a/libavcodec/mj
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
+++ b/libavcodec/mpegvideo_enc
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: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts
>>>
>>> Please comme
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 -hwac
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 Eugen
>
> Breaks indentation, nak.
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;
> +if(!(imme_image = (nu
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 squared pixels:
> > $ ffmpeg -f lavfi -i "color=s=hd720
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 -vframes 1 -f null
> - -v verbose 2>&1 | grep
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 @ 01c953343940] w:1280
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 wrote:
>> >> > Signed-off-by: Paul B Mahol
>> >> > ---
>> >> > doc/filters.te
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)
> [..]
> Previous write of size
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: write
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
> >> > ---
> >> > doc/filters.texi | 3 +++
> >> > libavfilter/vf_pad.c | 14
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 del
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 +++
>> > libavfilter/vf_pad.c | 14 ++
>> > 2 files changed, 17 insertions(+)
>> >
>>
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.
___
ffmp
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(+)
> >
> > +if (adjusted_aspect.num && adjusted_aspect.
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, 3 Apr 2017 19:03:29
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
http://ffmpeg.org/mailman
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:
>> >
>> >>
>> >> This is the Boost Software license, very similar to BSD/MIT, so if
>> >
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 src/l
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 the vmaf filter later
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
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 optim
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 si
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 --git a/libavcodec/h
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 FATE succeeds and the result is a c
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 file.
diff --git
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
--- a/ffmpeg.c
+++ b
---
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 @@ s
---
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
+++ b/doc/examples/extrac
---
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
+++ b/doc/examples/extrac
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
> +++--
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 S
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
> >> that license is all that matters, its obviously fine.
>
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 Lee wrote:
> > > > On 04/03/2017 06:35 PM, Nicolas George 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 Lee wrote:
> > > On 04/03/2017 06:35 PM, Nicolas George wrote:
> > > >Hi. Thanks for the patch.
> > > >
> > > >Le quartidi 14 germinal, an CCXX
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 100644
--- a/doc/
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 all along.
>
> I apologise for wasting y
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 :
> > >>Capture is an
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.
Carl Eugen
_
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 be ok but maybe wai
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 the latest 10.8.5 SDK and checked every fi
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 b/libavcodec/hevc_parse.c
> index 6c1138e01
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
> cach
> 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
___
ffmpeg-de
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
frame timestamps in the tra
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 d
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 +++
>>> libavutil/spherical.c | 15 +++
>>> libavutil/sph
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
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
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 +
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 (supp
68 matches
Mail list logo
