On Thu, 2 Dec 2021, lance.lmw...@gmail.com wrote:
From: Limin Wang
Signed-off-by: Limin Wang
---
libavformat/rtsp.c | 23 +--
1 file changed, 9 insertions(+), 14 deletions(-)
diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
index e6a4993..249a019 100644
--- a/libavform
The mpeg4 encoder is slice-threaded and its output depends upon
the number of threads used. Therefore all tests of this encoder
use a hardcoded number of threads (ENC_OPTS in fate-run.sh contains
"-threads 1"; only the vsynth%-mpeg4-thread tests override this
for the mpeg4 encoder, but they also us
The mpeg4 encoder is slice-threaded and its output depends upon
the number of threads used. Therefore all tests of this encoder
use a hardcoded number of threads (ENC_OPTS in fate-run.sh contains
"-threads 1"; only the vsynth%-mpeg4-thread tests override this
for the mpeg4 encoder, but they also us
2 Dec 2021, 03:18 by jianhua...@intel.com:
> Ping.
>
>> Wu Jianhua:
>> Lynne:
>> >>ffm...@gyani.pro：
>> >>> On 2021-11-26 04:33 pm, Lynne wrote:
>> >>>
>> 26 Nov 2021, 11:37 by ffm...@gyani.pro:
>>
>> >
>> > On 2021-11-26 03:08 pm, Lynne wrote:
>> >
>> >> 26 Nov 2021, 10:
> On Dec 2, 2021, at 6:04 PM, Andreas Rheinhardt
> wrote:
>
> The mpeg4 encoder is slice-threaded and its output depends upon
> the number of threads used. Therefore all tests of this encoder
> use a hardcoded number of threads (ENC_OPTS in fate-run.sh contains
> "-threads 1"; only the vsynth
2 Dec 2021, 02:53 by wenbin.c...@intel.com:
>> The vaapi can import external frame, but the planes of the external
>> frames should be in the same drm object. A new option "contiguous_planes"
>> is added to device. This flag tells device to allocate places in one
>> memory. When device is derived
On Thu, Dec 02, 2021 at 10:04:37AM +0200, Martin Storsjö wrote:
> On Thu, 2 Dec 2021, lance.lmw...@gmail.com wrote:
>
> > From: Limin Wang
> >
> > Signed-off-by: Limin Wang
> > ---
> > libavformat/rtsp.c | 23 +--
> > 1 file changed, 9 insertions(+), 14 deletions(-)
> >
> >
From: Limin Wang
this allows getting rid of the hardcoded max size of SDP.
Reviewed-by: Martin Storsjö
Signed-off-by: Limin Wang
---
libavformat/rtsp.c | 25 +
1 file changed, 9 insertions(+), 16 deletions(-)
diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
index
From: Limin Wang
Signed-off-by: Limin Wang
---
libavformat/rtspdec.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c
index 0e91e3e..2ada29a 100644
--- a/libavformat/rtspdec.c
+++ b/libavformat/rtspdec.c
@@ -172,7 +172,
On Thu, 2 Dec 2021, lance.lmw...@gmail.com wrote:
From: Limin Wang
this allows getting rid of the hardcoded max size of SDP.
Reviewed-by: Martin Storsjö
Signed-off-by: Limin Wang
---
libavformat/rtsp.c | 25 +
1 file changed, 9 insertions(+), 16 deletions(-)
LGTM
/
On Thu, 2 Dec 2021, lance.lmw...@gmail.com wrote:
From: Limin Wang
Signed-off-by: Limin Wang
---
libavformat/rtspdec.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
LGTM, thanks!
If this was the last use of SDP_MAX_SIZE, I guess this patch could remove
the define too?
On Thu, Dec 02, 2021 at 01:20:00PM +0200, Martin Storsjö wrote:
> On Thu, 2 Dec 2021, lance.lmw...@gmail.com wrote:
>
> > From: Limin Wang
> >
> > Signed-off-by: Limin Wang
> > ---
> > libavformat/rtspdec.c | 10 --
> > 1 file changed, 8 insertions(+), 2 deletions(-)
>
> LGTM, thanks!
>
Reproduced when using the VAAPI va module on VLC 4.0. No leaks when
setting thread count to 1.
---
libavcodec/pthread_frame.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index 73b1b7d7d9..4c578aa44a 100644
--- a/libavcodec/pthread_fra
From: Limin Wang
Old system is:
OSX version: 10.11.6
Apple LLVM version 8.0.0 (clang-800.0.42.1)
Target: x86_64-apple-darwin15.6.0
Signed-off-by: Limin Wang
---
libavcodec/videotoolbox.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/videotoolbox.c b/libavco
Thomas Guillem:
> Reproduced when using the VAAPI va module on VLC 4.0. No leaks when
> setting thread count to 1.
> ---
> libavcodec/pthread_frame.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
> index 73b1b7d7d9..4c578aa44a 100
On Thu, Dec 2, 2021, at 14:38, Andreas Rheinhardt wrote:
> Thomas Guillem:
>> Reproduced when using the VAAPI va module on VLC 4.0. No leaks when
>> setting thread count to 1.
>> ---
>> libavcodec/pthread_frame.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/libavcodec/pthread_fr
Thomas Guillem:
>
>
> On Thu, Dec 2, 2021, at 14:38, Andreas Rheinhardt wrote:
>> Thomas Guillem:
>>> Reproduced when using the VAAPI va module on VLC 4.0. No leaks when
>>> setting thread count to 1.
>>> ---
>>> libavcodec/pthread_frame.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff -
Fixes FATE failures if e.g. libavdevice is disabled.
Signed-off-by: Andreas Rheinhardt
---
tests/fate/ffmpeg.mak | 20 +---
1 file changed, 17 insertions(+), 3 deletions(-)
diff --git a/tests/fate/ffmpeg.mak b/tests/fate/ffmpeg.mak
index f9bc5bedf5..4ba73a8dfa 100644
--- a/tests
On Thu, Dec 2, 2021 at 2:20 PM wrote:
>
> From: Limin Wang
>
> Old system is:
> OSX version: 10.11.6
> Apple LLVM version 8.0.0 (clang-800.0.42.1)
> Target: x86_64-apple-darwin15.6.0
>
> Signed-off-by: Limin Wang
> ---
> libavcodec/videotoolbox.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 d
On Thu, Dec 02, 2021 at 03:39:44PM +0100, Hendrik Leppkes wrote:
> On Thu, Dec 2, 2021 at 2:20 PM wrote:
> >
> > From: Limin Wang
> >
> > Old system is:
> > OSX version: 10.11.6
> > Apple LLVM version 8.0.0 (clang-800.0.42.1)
> > Target: x86_64-apple-darwin15.6.0
> >
> > Signed-off-by: Limin Wang
From: Limin Wang
Old system is:
OSX version: 10.11.6
Apple LLVM version 8.0.0 (clang-800.0.42.1)
Target: x86_64-apple-darwin15.6.0
Signed-off-by: Limin Wang
---
libavcodec/videotoolbox.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/videotoolbox.c b/libavco
On Thu, Dec 2, 2021 at 11:39 AM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
> Fixes FATE failures if e.g. libavdevice is disabled.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> tests/fate/ffmpeg.mak | 20 +---
> 1 file changed, 17 insertions(+), 3 deletions(-)
>
>
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 4eb1f56f7d..826dafbd00 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -356,8 +356,6 @@ end:
av_free(p
It is not documented that freeing the last (and only) entry of
an AVDictionary frees the dictionary.
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 826dafbd00..d9083ad98
An AVBPrint's internal string is always already zero-terminated;
writing another '\0' is unnecessary as long as one treats
the string only as a C-string.
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/libavdevice/lavfi.c b/libavdev
The issue is that if ffmpeg is compiled without Vulkan, and an API
user includes and uses the functions exposed in hwcontext_vulkan.h,
then a linking error will happen, as the hwcontext_vulkan.c file has
not been compiled.
Move the functions to another file, and make it always compiled. The
funct
On Thu, Dec 2, 2021 at 3:01 PM Lynne wrote:
> The issue is that if ffmpeg is compiled without Vulkan, and an API
> user includes and uses the functions exposed in hwcontext_vulkan.h,
> then a linking error will happen, as the hwcontext_vulkan.c file has
> not been compiled.
>
> Move the functions
Lynne:
> The issue is that if ffmpeg is compiled without Vulkan, and an API
> user includes and uses the functions exposed in hwcontext_vulkan.h,
> then a linking error will happen, as the hwcontext_vulkan.c file has
> not been compiled.
>
> Move the functions to another file, and make it always
On Thu, Dec 2, 2021, at 15:05, Andreas Rheinhardt wrote:
> 1. hw_device_ctx is in the public AVCodecContext, not in the private
> AVCodecInternal.
> 2. ctx->internal is intended to be allocated for all the internal
> AVCodecContexts; the check is only there to ensure that we don't crash
> if an a
LGTM
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
probably ok
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 25 +
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 2659c8508a..209ebed5fd 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -317,26 +3
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 209ebed5fd..878bb193af 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -320,27 +320,27 @@ av
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 878bb193af..b208b1a928 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -269,11 +269,10 @@ av_cold st
The length of this list is a compile-time constant, so there is
no need to calculate it again at runtime.
(This also avoids an implicit requirement of -1 == AV_SAMPLE_FMT_NONE.)
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
Signed-off-by: Andreas Rheinhardt
---
libavdevice/lavfi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index d63a6f66b3..710ad7c530 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -76,7 +76,7 @@ static int *create_all_
These lists have size fields since
e48ded8551172b58a78f30303a81dfce125344e0.
Signed-off-by: Andreas Rheinhardt
---
libavfilter/buffersink.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/libavfilter/buffersink.c b/libavfilter/buffersink.c
index b8ddafec35..c021566
On Thu, Dec 02, 2021 at 06:37:38PM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang
>
> this allows getting rid of the hardcoded max size of SDP.
>
> Reviewed-by: Martin Storsjö
> Signed-off-by: Limin Wang
> ---
> libavformat/rtsp.c | 25 +
> 1 file changed, 9 i
On Thu, Dec 02, 2021 at 11:52:23PM +0100, Michael Niedermayer wrote:
> On Thu, Dec 02, 2021 at 06:37:38PM +0800, lance.lmw...@gmail.com wrote:
> > From: Limin Wang
> >
> > this allows getting rid of the hardcoded max size of SDP.
> >
> > Reviewed-by: Martin Storsjö
> > Signed-off-by: Limin Wang
From: Limin Wang
without it, read_packet_wrapper() will return AVERROR(EINVAL) and avio_read
will be failed.
Signed-off-by: Limin Wang
---
It'll fix the issue reported by Michael in below patch:
https://patchwork.ffmpeg.org/project/ffmpeg/patch/1638441459-21819-1-git-send-email-lance.lmw...@gma
From: Limin Wang
This is allowed to read fewer bytes than requested. The missing bytes can
be read in the next call.
Signed-off-by: Limin Wang
---
libavformat/aviobuf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c
index 14688
---
libavformat/movenc.c | 8 +
libavformat/movenc.h | 1 +
tests/ref/fate/movenc | 80 +--
3 files changed, 49 insertions(+), 40 deletions(-)
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index f8731d33c5..01dfd21a43 100644
--- a/libavfo
---
libavformat/movenc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index 38ff90833a..634a829f28 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -4427,7 +4427,7 @@ static int mov_write_mfhd_tag(AVIOCo
---
libavformat/movenc.c | 17 -
libavformat/movenc.h | 1 +
tests/ref/fate/movenc | 4 ++--
3 files changed, 19 insertions(+), 3 deletions(-)
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index 634a829f28..06d3819591 100644
--- a/libavformat/movenc.c
+++ b/libavfor
traf_number indicates the ‘traf’ number that contains the sync sample.
The number ranges from 1 (the first ‘traf’ is numbered 1) in each ‘moof’.
For A-V interleaved fmp4 with two traf boxes inside a moof, tfra box
was broken.
---
libavformat/movenc.c | 4 +++-
libavformat/movenc.h | 2 ++
test
---
libavformat/movenc.c | 42 +-
1 file changed, 21 insertions(+), 21 deletions(-)
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index 01dfd21a43..3c7c951c7d 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -4939,16 +4939,16 @@ st
> On Dec 3, 2021, at 1:06 PM, Zhao Zhili wrote:
>
> ---
> libavformat/movenc.c | 8 +
> libavformat/movenc.h | 1 +
> tests/ref/fate/movenc | 80 +--
> 3 files changed, 49 insertions(+), 40 deletions(-)
>
> diff --git a/libavformat/movenc.c b/libav
> -Original Message-
> From: ffmpeg-devel On Behalf Of Anton
> Khirnov
> Sent: Wednesday, December 1, 2021 11:55 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v2 1/1] avcodec/vpp_qsv: Copy side data
> from input to output frame
>
> Quoting Soft Works (2021-11-30
Signed-off-by: softworkz
---
V3: Split commits, add version bump, use flag param
doc/APIchanges | 3 +++
libavutil/frame.c | 58 ++---
libavutil/frame.h | 16 +
libavutil/version.h | 4 ++--
4 files changed, 54 insertions(+), 27 dele
Signed-off-by: softworkz
---
libavfilter/qsvvpp.c | 5 +
libavfilter/vf_overlay_qsv.c | 19 +++
2 files changed, 20 insertions(+), 4 deletions(-)
diff --git a/libavfilter/qsvvpp.c b/libavfilter/qsvvpp.c
index d1218355c7..b291216292 100644
--- a/libavfilter/qsvvpp.c
+
50 matches
Mail list logo
