Steven Liu 于2020年8月2日周日 下午10:35写道：
>
> Steven Liu 于2020年7月25日周六 上午10:49写道：
> >
> > Steven Liu 于2020年7月22日周三 下午5:16写道：
> > >
> > > fix ticket: 8783
> > > Because in single file by encryption mode, it cannot get the last one
> > > block of the file, it need ff_format_io_close for get full file siz
take following addvices from James:
Handle meta data before seq and frame in obu_probe
Use read_obu_with_size instead of change parse_obu_header
Use memmove instead of memcpy in obu_read_data
Add "av1_frame_merge_bsf av1_parser" as dependency to configure.
Change name from "av1" to "obu".
---
libavcodec/av1_parse.h | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavcodec/av1_parse.h b/libavcodec/av1_parse.h
index 01bcd646c2..ae0ebb5a18 100644
--- a/libavcodec/av1_parse.h
+++ b/libavcodec/av1_parse.h
@@ -27,6 +27,9 @@
#include "avcodec.h"
#include "get_bi
It's defined in Section 5.2, used by netflix.
see http://download.opencontent.netflix.com/?prefix=AV1/Chimera/
---
configure| 1 +
libavformat/allformats.c | 1 +
libavformat/av1dec.c | 266 +++
3 files changed, 245 insertions(+), 23 dele
---
tests/fate/demux.mak | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak
index 9f3a6be276..763452a533 100644
--- a/tests/fate/demux.mak
+++ b/tests/fate/demux.mak
@@ -13,6 +13,9 @@ fate-aea-demux: CMD = crc -i $(TARGET_SAMPLES)/aea/chirp.aea
-c:a
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Steve Lhomme
> Sent: Monday, August 10, 2020 8:19 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer
> copies are done before submitting them
>
> On 2020-08-08 8:24, Soft Wo
Signed-off-by: Mingyu Yin
---
libavfilter/dnn/dnn_backend_native_layer_mathunary.c | 4
libavfilter/dnn/dnn_backend_native_layer_mathunary.h | 1 +
tests/dnn/dnn-layer-mathunary-test.c | 4
tools/python/convert_from_tensorflow.py | 2 +-
tools/python/convert
> -Original Message-
> From: ffmpeg-devel On Behalf Of Ting Fu
> Sent: 2020年8月10日 0:33
> To: ffmpeg-devel@ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH V7 2/2] FATE/dnn: add unit test for dnn avgpool
> layer
>
> 'make fate-dnn-layer-avgpool' to run the test
>
> Signed-off-by: Ting Fu
>
On 2020-08-10 12:04, Soft Works wrote:
the very least the locks are needed inside libavcodec to avoid
setting DXVA buffers concurrently from different threads. It will
most likely result in very bad distortions if not crashes. Maybe
you're only using 1 decoding thread with DXVA (which a lot of pe
Up until now, opening a section filter works as follows: A filter is
opened and (on success) attached to the MpegTSContext. Then a buffer for
said filter is allocated and upon success attached to the section
filter; on error, the filter is simply freed without removing it from
the MpegTSContext, le
On Sun, Aug 02, 2020 at 01:23:23AM +0200, Michael Niedermayer wrote:
> Fixes: out of array read
> Fixes:
> 24487/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_JPEG2000_fuzzer-5165847820369920
>
> Found-by: continuous fuzzing process
> https://github.com/google/oss-fuzz/tree/master/projects/f
by allocating it together with the AVBSFContext.
Signed-off-by: Andreas Rheinhardt
---
Similar things can of course be done with other structures. I did only
this one to see whether everyone is ok with this.
libavcodec/bsf.c | 25 +
1 file changed, 13 insertions(+), 12 d
On 8/10/2020 10:55 AM, Andreas Rheinhardt wrote:
> by allocating it together with the AVBSFContext.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> Similar things can of course be done with other structures. I did only
> this one to see whether everyone is ok with this.
Personally, i don't like it
On Sat, 8 Aug 2020, at 19:57, Nicolas George wrote:
> +@item
> +If the code contains tables of numbers or other data, their origin should be
> +documented in a comment, so that other developers can rebuild them if
> +necessary.
I would add "as honestly as possible" after documented... or something
Signed-off-by: James Almer
---
configure | 3 ++-
libavcodec/libdav1d.c | 30 ++
2 files changed, 32 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index 37fe01e8be..83f051fdee 100755
--- a/configure
+++ b/configure
@@ -3211,6 +3211,7 @@
Signed-off-by: James Almer
---
configure | 2 +-
libavcodec/hevc_sei.c | 49 +++
2 files changed, 9 insertions(+), 42 deletions(-)
diff --git a/configure b/configure
index 5fd966035f..762e74a8ab 100755
--- a/configure
+++ b/configure
@@ -2743,
Signed-off-by: James Almer
---
configure | 2 +-
libavcodec/h264_sei.c | 48 ++-
2 files changed, 3 insertions(+), 47 deletions(-)
diff --git a/configure b/configure
index 83f051fdee..5fd966035f 100755
--- a/configure
+++ b/configure
@@ -2738,
Signed-off-by: James Almer
---
configure| 9 +---
libavcodec/atsc_a53.c| 43
libavcodec/atsc_a53.h| 17 ++
libavcodec/internal.h| 15 -
libavcodec/libx264.c | 1 +
libavcodec/nve
Signed-off-by: James Almer
---
configure | 1 +
libavcodec/Makefile | 1 +
libavcodec/atsc_a53.c | 76 +++
libavcodec/atsc_a53.h | 39 ++
4 files changed, 117 insertions(+)
create mode 100644 libavcodec/atsc_a53.c
creat
On 8/10/2020 1:18 PM, Lynne wrote:
> Aug 10, 2020, 17:49 by jamr...@gmail.com:
>
>> Signed-off-by: James Almer
>> ---
>> configure | 2 +-
>> libavcodec/hevc_sei.c | 49 +++
>> 2 files changed, 9 insertions(+), 42 deletions(-)
>>
>> diff --git
From: Gautam Ramakrishnan
This patch updates the documentation by adding all options
for JPEG2000 encoder.
---
doc/encoders.texi | 36 +++-
1 file changed, 35 insertions(+), 1 deletion(-)
diff --git a/doc/encoders.texi b/doc/encoders.texi
index de0472e225..426baa
Source: Videolan sample database (samples/sub/dvbsub)
---
Can someone upload
https://streams.videolan.org/samples/sub/dvbsub/dvbsubtest.ts to
fate-samples/sub?
---
tests/fate/subtitles.mak | 3 ++
tests/ref/fate/sub-dvb | 75
2 files changed, 78 insert
It will be useful in the next commit.
Signed-off-by: James Almer
---
libavcodec/internal.h | 2 ++
libavcodec/utils.c| 10 +-
2 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/libavcodec/internal.h b/libavcodec/internal.h
index 0a1c0a17ec..a302cb24eb 100644
--- a/libavc
Mention an example scenario where the function should be used.
Signed-off-by: James Almer
---
libavcodec/bsf.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/bsf.h b/libavcodec/bsf.h
index 7ed51677e8..3b5faa85cb 100644
--- a/libavcodec/bsf.h
+++ b/libavcodec/bsf.h
This function lets the API user reuse an AVBSFContext in case of stream
parameter changes.
Signed-off-by: James Almer
---
Missing version bump and APIChanges entry.
libavcodec/bsf.c | 21 +
libavcodec/bsf.h | 14 ++
2 files changed, 35 insertions(+)
diff --git a
On Mon, 10 Aug 2020, Andreas Rheinhardt wrote:
Up until now, opening a section filter works as follows: A filter is
opened and (on success) attached to the MpegTSContext. Then a buffer for
said filter is allocated and upon success attached to the section
filter; on error, the filter is simply
James Almer:
> This function lets the API user reuse an AVBSFContext in case of stream
> parameter changes.
>
> Signed-off-by: James Almer
> ---
> Missing version bump and APIChanges entry.
>
> libavcodec/bsf.c | 21 +
> libavcodec/bsf.h | 14 ++
> 2 files change
Clément Bœsch:
> Source: Videolan sample database (samples/sub/dvbsub)
> ---
> Can someone upload
> https://streams.videolan.org/samples/sub/dvbsub/dvbsubtest.ts to
> fate-samples/sub?
A 20MB sample? Can't one use a smaller sample or filter out the
interesting PIDs from this sample? (If I remux
On 8/10/2020 3:33 PM, Andreas Rheinhardt wrote:
> James Almer:
>> This function lets the API user reuse an AVBSFContext in case of stream
>> parameter changes.
>>
>> Signed-off-by: James Almer
>> ---
>> Missing version bump and APIChanges entry.
>>
>> libavcodec/bsf.c | 21 +
>
This function lets the API user reuse an AVBSFContext in case of stream
parameter changes.
Signed-off-by: James Almer
---
libavcodec/bsf.c | 26 ++
libavcodec/bsf.h | 14 ++
2 files changed, 40 insertions(+)
diff --git a/libavcodec/bsf.c b/libavcodec/bsf.c
in
On Wed, 5 Aug 2020, Nicolas George wrote:
Signed-off-by: Nicolas George
---
libavformat/tests/url.c | 34
libavformat/url.c | 70 +
libavformat/url.h | 41
tests/ref/fate/url | 45 +
On Wed, 5 Aug 2020, Nicolas George wrote:
Also add and update some tests.
Change the semantic a little, because for filesytem paths
symlinks complicate things.
See the comments in the code for detail.
Add to commit message that this fixes tickets 8813, 8814.
Signed-off-by: Nicolas George
Aug 10, 2020, 17:49 by jamr...@gmail.com:
> Signed-off-by: James Almer
> ---
> configure | 2 +-
> libavcodec/hevc_sei.c | 49 +++
> 2 files changed, 9 insertions(+), 42 deletions(-)
>
> diff --git a/configure b/configure
> index 5fd966035f..7
On Tue, Aug 04, 2020 at 13:59:42 +0200, Anton Khirnov wrote:
> --- a/fftools/ffmpeg.c
> +++ b/fftools/ffmpeg.c
> @@ -2883,7 +2883,9 @@ static int init_input_stream(int ist_index, char
> *error, int error_len)
> ist->dec_ctx->opaque= ist;
> ist->dec_ctx->get_format
On Mon, Aug 10, 2020 at 08:36:13PM +0200, Andreas Rheinhardt wrote:
> Clément Bœsch:
> > Source: Videolan sample database (samples/sub/dvbsub)
> > ---
> > Can someone upload
> > https://streams.videolan.org/samples/sub/dvbsub/dvbsubtest.ts to
> > fate-samples/sub?
>
> A 20MB sample? Can't one us
Hi Zane,
thanks for addressing my comments in v3.
AFAICS you also took Andreas' feedback into account, but I didn't
check that.
No more remarks from me.
Alexander
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/list
James Almer (12020-08-10):
> Personally, i don't like it. It's extra complexity to save a single 8 or
> 12 byte allocation that happens once during bsf alloc. It's kind of a
> pointless micro-optimization.
I do not agree at all.
First, it is not extra complexity, it actually makes the code simple
Andreas Rheinhardt (12020-08-10):
> by allocating it together with the AVBSFContext.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> Similar things can of course be done with other structures. I did only
> this one to see whether everyone is ok with this.
I wholeheartedly approve.
> libavcodec/
Clément Bœsch:
> On Mon, Aug 10, 2020 at 08:36:13PM +0200, Andreas Rheinhardt wrote:
>> Clément Bœsch:
>>> Source: Videolan sample database (samples/sub/dvbsub)
>>> ---
>>> Can someone upload
>>> https://streams.videolan.org/samples/sub/dvbsub/dvbsubtest.ts to
>>> fate-samples/sub?
>>
>> A 20MB s
On 8/10/2020 7:11 PM, Nicolas George wrote:
> James Almer (12020-08-10):
>> Personally, i don't like it. It's extra complexity to save a single 8 or
>> 12 byte allocation that happens once during bsf alloc. It's kind of a
>> pointless micro-optimization.
>
> I do not agree at all.
>
> First, it i
James Almer (12020-08-10):
> It adds an extra struct and makes the code harder to read. Might as well
> just do
>
> ctx = av_mallocz(sizeof(*ctx) + sizeof(AVBSFInternal));
> ctx->internal = &ctx[1];
This is exactly what the code does, except without the undefined
behaviors, because what you just
On Mon, 10 Aug 2020 23:33:14 +0200
"Alexander Strasser" wrote:
>
> Hi Zane,
>
> thanks for addressing my comments in v3.
>
> AFAICS you also took Andreas' feedback into account, but I didn't
> check that.
>
No worries. I'm pretty sure I handled everything...
> No more remarks from me.
>
Co
On 8/10/2020 7:34 PM, Nicolas George wrote:
> James Almer (12020-08-10):
>> It adds an extra struct and makes the code harder to read. Might as well
>> just do
>>
>> ctx = av_mallocz(sizeof(*ctx) + sizeof(AVBSFInternal));
>> ctx->internal = &ctx[1];
>
> This is exactly what the code does
I know,
James Almer:
> On 8/10/2020 7:11 PM, Nicolas George wrote:
>> James Almer (12020-08-10):
>>> Personally, i don't like it. It's extra complexity to save a single 8 or
>>> 12 byte allocation that happens once during bsf alloc. It's kind of a
>>> pointless micro-optimization.
>>
>> I do not agree at a
Andreas Rheinhardt:
> James Almer:
>> On 8/10/2020 7:11 PM, Nicolas George wrote:
>>> James Almer (12020-08-10):
Personally, i don't like it. It's extra complexity to save a single 8 or
12 byte allocation that happens once during bsf alloc. It's kind of a
pointless micro-optimization
James Almer:
> This function lets the API user reuse an AVBSFContext in case of stream
> parameter changes.
>
> Signed-off-by: James Almer
> ---
> libavcodec/bsf.c | 26 ++
> libavcodec/bsf.h | 14 ++
> 2 files changed, 40 insertions(+)
>
> diff --git a/libav
On 8/10/2020 8:12 PM, Andreas Rheinhardt wrote:
> James Almer:
>> On 8/10/2020 7:11 PM, Nicolas George wrote:
>>> James Almer (12020-08-10):
Personally, i don't like it. It's extra complexity to save a single 8 or
12 byte allocation that happens once during bsf alloc. It's kind of a
On 8/10/2020 8:26 PM, Andreas Rheinhardt wrote:
> James Almer:
>> This function lets the API user reuse an AVBSFContext in case of stream
>> parameter changes.
>>
>> Signed-off-by: James Almer
>> ---
>> libavcodec/bsf.c | 26 ++
>> libavcodec/bsf.h | 14 ++
>>
James Almer:
> On 8/10/2020 8:12 PM, Andreas Rheinhardt wrote:
>> James Almer:
>>> On 8/10/2020 7:11 PM, Nicolas George wrote:
James Almer (12020-08-10):
> Personally, i don't like it. It's extra complexity to save a single 8 or
> 12 byte allocation that happens once during bsf alloc.
Up until now, the TiVo demuxer parse an array of SEQ entries, yet it has
never ever made any use of them. In fact, parse_master, the function
parsing said table, only influenced the outside world in three ways: Via
an excessive amount of error message in case a certain parameter is not
what it expe
On 8/10/2020 9:14 PM, Andreas Rheinhardt wrote:
> James Almer:
>> On 8/10/2020 8:12 PM, Andreas Rheinhardt wrote:
>>> James Almer:
On 8/10/2020 7:11 PM, Nicolas George wrote:
> James Almer (12020-08-10):
>> Personally, i don't like it. It's extra complexity to save a single 8 or
>>
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Steve Lhomme
> Sent: Monday, August 10, 2020 2:15 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer
> copies are done before submitting them
>
> > @Steven
>
> My name is S
> On Aug 11, 2020, at 8:14 AM, Andreas Rheinhardt
> wrote:
>
> James Almer:
>> On 8/10/2020 8:12 PM, Andreas Rheinhardt wrote:
>>> James Almer:
On 8/10/2020 7:11 PM, Nicolas George wrote:
> James Almer (12020-08-10):
>> Personally, i don't like it. It's extra complexity to save a
g«°Æeñ0Í,{,x;QL3 Á[ì
õ5ËõRɯð5öS»ÍuE)àC^æ¸'㡾?¤\ãÕpbÙ&zô5Û¯HöEòåÊB¿ÌÏ"h¾a>*0G}Õ2Ø|×ùeP÷qÞq°1H¬úa70PìlÍìªßÕ
µ&ê.·iÆaTøÝдwrêWt'H<©w%~*¼#©äK
Hý
ÐóMÍÖs?g÷Ç/J_Dó¿òÁ_WùhTBÏ#ʽ
>JVV´¾f}E¶¤pì6,
ë*øÔIU.Í¿A,¾Ôf!QÒ&Ã×uBõ´¦'%¦~`'H5ârWlÉj㥴µ½JHU±Á:׺½vwL
~ýEµ.ÕË£¨/\à#·Y
On 2020-08-11 3:52, Soft Works wrote:
-Original Message-
From: ffmpeg-devel On Behalf Of
Steve Lhomme
Sent: Monday, August 10, 2020 2:15 PM
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer
copies are done before submitting them
@S
Previously, the only way to input the master display and content light metadata
required for HDR10 was through x265-params. Obviously, that only worked with
x265. If you wanted to use a different encoder like nvenc, you were out of
luck. The options specified are written to the container (mov or
Sorry, please disregard this patch. I just realized that my test changes are
not in it.
Kenny
> On Aug 10, 2020, at 7:36 PM, Kenny McClive wrote:
>
> Previously, the only way to input the master display and content light
> metadata required for HDR10 was through x265-params. Obviously, that
Previously, the only way to input the master display and content light metadata
required for HDR10 was through x265-params. Obviously, that only worked with
x265. If you wanted to use a different encoder like nvenc, you were out of
luck. The options specified are written to the container (mov or
Previously, the only way to input the master display and content light metadata
required for HDR10 was through x265-params. Obviously, that only worked with
x265. If you wanted to use a different encoder like nvenc, you were out of
luck. The options specified are written to the container (mov or
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Steve Lhomme
> Sent: Tuesday, August 11, 2020 6:35 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer
> copies are done before submitting them
[...]
> > But SetMultithreadP
60 matches
Mail list logo
