> -Original Message-
> From: ffmpeg-devel On Behalf Of Soft Works
> Sent: Saturday, November 27, 2021 8:19 AM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v16 08/16] fftools/ffmpeg: Replace
> sub2video with subtitle frame filtering
>
>
>
> > -
> -Original Message-
> From: ffmpeg-devel On Behalf Of Andreas
> Rheinhardt
> Sent: Friday, November 26, 2021 2:02 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v16 08/16] fftools/ffmpeg: Replace
> sub2video with subtitle frame filtering
>
> > -int subtitle_o
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:10 by jianhua...@intel.com:
>>
>> This filter flips the input video both horizontally and v
> -Original Message-
> From: ffmpeg-devel On Behalf Of Andreas
> Rheinhardt
> Sent: Friday, November 26, 2021 12:14 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v16 06/16] avfilter/sbuffer: Add
> sbuffersrc and sbuffersink filters
>
> > +#include "libavcodec/av
Adds demuxer for Square Enux SCD files.
Based off [1] and personal investigation.
This has only been tested against Drakengard 3 (PS3) *_SCD.XXX files
(big-endian). As it is highly likely that FFXIV (PC) files are little-endian,
this demuxer is marked as experimental until this can be confirmed.
> -Original Message-
> From: ffmpeg-devel On Behalf Of Andreas
> Rheinhardt
> Sent: Friday, November 26, 2021 11:35 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v16 01/16] global: Prepare AVFrame for
> subtitle handling
Hi Andreas,
thanks for the detailed revie
On Fri, Nov 26, 2021 at 10:01:23PM +, Soft Works wrote:
>
>
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of Michael
> > Niedermayer
> > Sent: Friday, November 26, 2021 10:26 PM
> > To: FFmpeg development discussions and patches
> > Subject: Re: [FFmpeg-devel] [PATCH v3 1/
> -Original Message-
> From: ffmpeg-devel On Behalf Of Anton
> Khirnov
> Sent: Thursday, November 25, 2021 4:56 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v15 01/16] global: Prepare AVFrame for
> subtitle handling
>
> Quoting Soft Works (2021-11-25 01:48:11)
> -Original Message-
> From: ffmpeg-devel On Behalf Of Michael
> Niedermayer
> Sent: Friday, November 26, 2021 10:26 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v3 1/1] fftools/ffprobe: print size of
> attachment streams (extradata_size)
>
>
On Fri, Nov 26, 2021 at 07:55:40PM +, Soft Works wrote:
>
>
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of Michael
> > Niedermayer
> > Sent: Friday, November 26, 2021 7:50 PM
> > To: FFmpeg development discussions and patches
> > Subject: Re: [FFmpeg-devel] [PATCH v3 1/1
> -Original Message-
> From: ffmpeg-devel On Behalf Of Michael
> Niedermayer
> Sent: Friday, November 26, 2021 7:50 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v3 1/1] fftools/ffprobe: print size of
> attachment streams (extradata_size)
>
>
On Sun, Nov 14, 2021 at 08:30:28PM +0100, Michael Niedermayer wrote:
> Fixes: Timeout
> Fixes:
> 40882/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TXD_fuzzer-4893364584054784
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/txd.c | 30 +++---
> 1 file chan
On Sun, Nov 14, 2021 at 08:30:29PM +0100, Michael Niedermayer wrote:
> Fixes: Timeout
> Fixes:
> 40950/clusterfuzz-testcase-minimized-ffmpeg_dem_AVI_fuzzer-6478873068437504
>
> Found-by: continuous fuzzing process
> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> Signed-off-by:
> -Original Message-
> From: ffmpeg-devel On Behalf Of Soft Works
> Sent: Friday, November 26, 2021 8:29 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v4 1/1] avutils/hwcontext: When deriving a
> hwdevice, search for existing device in both di
> -Original Message-
> From: ffmpeg-devel On Behalf Of Anton
> Khirnov
> Sent: Friday, November 26, 2021 8:12 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v4 1/1] avutils/hwcontext: When deriving a
> hwdevice, search for existing device in bo
Signed-off-by: Andreas Rheinhardt
---
fftools/ffmpeg.c | 3 ---
fftools/ffmpeg.h | 7 ---
fftools/ffmpeg_opt.c | 43 ---
3 files changed, 12 insertions(+), 41 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index c838e2604c..0bc
Fixes Coverity issue #1493344.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vqavideo.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/libavcodec/vqavideo.c b/libavcodec/vqavideo.c
index 5466e25cdf..61f0a2c950 100644
--- a/libavcodec/vqavideo.c
+++ b/libavcodec/vqav
Quoting Soft Works (2021-11-26 19:43:58)
> Maybe I'm missing something, but hw device contexts are refcounted.
> What happens in hwdevice_ctx_free() is this:
>
> av_buffer_unref(&ctx->internal->source_device);
IIUC this only happens after the parent device is freed. My concern is
the following si
On Thu, Nov 25, 2021 at 02:45:26PM -0500, Alex Xu (Hello71) wrote:
> Roughly doubles webp performance for common applications (ffmpeg,
> ffprobe, mpv) due to speeding up avformat_find_stream_info.
>
> Lossy needs no patches since vp8.c already implements skip_frame.
> Lossless needs hook to unders
On Thu, Nov 25, 2021 at 04:59:41PM +, Soft Works wrote:
> Another attempt: Created on Linux and zipped...
tested and works
LGTM
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Many that live deserve death. And some that die deserve life. Can you give
i
> -Original Message-
> From: ffmpeg-devel On Behalf Of Anton
> Khirnov
> Sent: Friday, November 26, 2021 5:15 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v4 1/1] avutils/hwcontext: When deriving a
> hwdevice, search for existing device in bo
> -Original Message-
> From: ffmpeg-devel On Behalf Of Anton
> Khirnov
> Sent: Friday, November 26, 2021 5:21 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v16 01/16] global: Prepare AVFrame for
> subtitle handling
>
> You have completely disregarded my comments
> -Original Message-
> From: ffmpeg-devel On Behalf Of Anton
> Khirnov
> Sent: Friday, November 26, 2021 5:15 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH v4 1/1] avutils/hwcontext: When deriving a
> hwdevice, search for existing device in bo
Andreas Rheinhardt:
> For most check_bitstream() functions this just avoids having
> to dereference s->streams[pkt->stream_index] themselves; but for
> meta-muxers it will allow to forward the packet to stream with
> a different stream_index (belonging to a different AVFormatContext)
> without usin
Anton Khirnov:
> img2enc keeps a private (and possibly truncated) copy of the url that is
> never modified. Just use AVFormatContext.url instead.
> ---
> libavformat/img2enc.c | 15 ++-
> 1 file changed, 6 insertions(+), 9 deletions(-)
>
> diff --git a/libavformat/img2enc.c b/libavfor
Anton Khirnov:
> device and cap are local to the loop iteration, there is no need for
> them to retain their values. Especially for device it may be dangerous,
> since it points to av_malloc'ed data.
> ---
> libavdevice/v4l2.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff -
Anton Khirnov:
> fd is local to the loop iteration, it is better to store it on stack
> than modify the context.
> ---
> libavdevice/v4l2.c | 13 ++---
> 1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/libavdevice/v4l2.c b/libavdevice/v4l2.c
> index b5997fba33..777867db86
Zane van Iperen:
>
>
> On 27/11/21 01:32, Andreas Rheinhardt wrote:
>> Zane van Iperen:
>>> +
>>> +static int scd_seek(AVFormatContext *s, int stream_index,
>>> + int64_t pts, int flags)
>>> +{
>>> + SCDDemuxContext *ctx = s->priv_data;
>>> + SCDTrackHeader *trk = ctx->t
Signed-off-by: quietvoid
---
Files are available here:
https://drive.google.com/drive/folders/1NDhtSoJ-mP5Yi62V6j1wjjPJNq84nP3J
They're all blank frames encoded with x265, remuxed with FFmpeg
after the mov box is added by dlb_mp4base.
Let me know if the hosting is not appropriate.
I thought it
On 27/11/21 01:32, Andreas Rheinhardt wrote:
Zane van Iperen:
+
+static int scd_seek(AVFormatContext *s, int stream_index,
+int64_t pts, int flags)
+{
+SCDDemuxContext *ctx = s->priv_data;
+SCDTrackHeader *trk = ctx->tracks + stream_index;
+
+if (pts != 0)
+
You have completely disregarded my comments on the previous patch. Stop
doing that, it is rude.
Your patchset is unacceptable until you deal with them.
--
Anton Khirnov
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/li
Quoting Soft Works (2021-11-25 18:02:54)
>
>
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of Anton
> > Khirnov
> > Sent: Thursday, November 25, 2021 5:40 PM
> > To: ffmpeg-devel@ffmpeg.org
> > Subject: Re: [FFmpeg-devel] [PATCH v4 1/1] avutils/hwcontext: When deriving
> > a
>
Quoting Soft Works (2021-11-25 18:20:31)
>
>
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of Anton
> > Khirnov
> > Sent: Thursday, November 25, 2021 4:56 PM
> > To: ffmpeg-devel@ffmpeg.org
> > Subject: Re: [FFmpeg-devel] [PATCH v15 01/16] global: Prepare AVFrame for
> > subtit
On Fri, Nov 26, 2021 at 4:09 PM Andreas Rheinhardt
wrote:
>
> Soft Works:
> > Also introduce deferred loading of ass headers for all cases where it can't
> > be taken from the context of a decoder.
>
> This should be a commit of its own.
>
> >
> > Signed-off-by: softworkz
> > ---
> > libavcodec
Anton Khirnov:
> Maximum output size with a 32-bit int is 17 bytes, or 26 with a 64-bit
> int.
>
> Silences the following gcc 10 warning:
> src/libavdevice/jack.c: In function ‘audio_read_header’:
> src/libavdevice/jack.c:171:45: warning: ‘snprintf’ output may be truncated
> before the last forma
Zane van Iperen:
> +
> +static int scd_seek(AVFormatContext *s, int stream_index,
> +int64_t pts, int flags)
> +{
> +SCDDemuxContext *ctx = s->priv_data;
> +SCDTrackHeader *trk = ctx->tracks + stream_index;
> +
> +if (pts != 0)
> +return AVERROR(EINVAL);
> +
Soft Works:
> Also introduce deferred loading of ass headers for all cases where it can't
> be taken from the context of a decoder.
This should be a commit of its own.
>
> Signed-off-by: softworkz
> ---
> libavcodec/assenc.c| 81 -
> libavcodec/avcodec.h
Soft Works:
> - overlaytextsubs {VS -> V)
> Overlay text subtitles onto a video stream.
>
> - textsubs2video {S -> V)
> Converts text subtitles to video frames
>
> Signed-off-by: softworkz
> ---
> configure| 2 +
> doc/filters.texi | 113 ++
> l
The same flags should be used when linking than when compiling.
Without this actual errors have been observed when using LTO.
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/configure b/configure
index e6cee21142..97c51f20a7 100755
--- a/configure
+++ b/configure
@@ -5805,6 +5805
Soft Works:
> This commit actually enables subtitle filtering in ffmpeg by
> sending and receiving subtitle frames to and from a filtergraph.
>
> The heartbeat functionality from the previous sub2video implementation
> is retained and applied to all subtitle frames (bitmap, text, ..).
>
> The oth
On Fri, Nov 26, 2021 at 01:50:33PM +0200, Martin Storsjö wrote:
> On Fri, 26 Nov 2021, lance.lmw...@gmail.com wrote:
>
> > From: Limin Wang
> >
> > Reviewed-by: Martin Storsjö
>
> Technically, I haven't reviewed this version yet - including such a header
> would indicate that I have approvd it
From: Limin Wang
Signed-off-by: Limin Wang
---
doc/filters.texi| 9 +
libavfilter/src_movie.c | 4 +++-
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/doc/filters.texi b/doc/filters.texi
index 3731a14..3b70aca 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
The same flags should be used when linking than when compiling.
Without this actual errors have been observed when using LTO.
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/configure b/configure
index e6cee21142..97c51f20a7 100755
--- a/configure
+++ b/configure
@@ -5805,6 +5805
On Fri, 26 Nov 2021, lance.lmw...@gmail.com wrote:
From: Limin Wang
Reviewed-by: Martin Storsjö
Technically, I haven't reviewed this version yet - including such a header
would indicate that I have approvd it, which I haven't.
Although in practice, this version looks fine to me, so yeah
26 Nov 2021, 12:20 by 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:10 by jianhua...@intel.com:
> This filter flips the input video both horizontally and ve
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:10 by jianhua...@intel.com:
This filter flips the input video both horizontally and vertically
in one compute pipeline, and it's no need to use two pipelines
Soft Works:
> Signed-off-by: softworkz
> ---
> configure| 2 +-
> libavfilter/allfilters.c | 2 ++
> libavfilter/buffersink.c | 63 +++
> libavfilter/buffersink.h | 15 +
> libavfilter/buffersrc.c | 72
26 Nov 2021, 11:37 by ffm...@gyani.pro:
>
>
> On 2021-11-26 03:08 pm, Lynne wrote:
>
>> 26 Nov 2021, 10:10 by jianhua...@intel.com:
>>
>>> This filter flips the input video both horizontally and vertically
>>> in one compute pipeline, and it's no need to use two pipelines for
>>> hflip_vulkan,vfli
26 Nov 2021, 03:54 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 fro
On Fri, Nov 26, 2021 at 11:12 AM wrote:
> On Fri, Nov 26, 2021 at 10:48:07AM +0100, Paul B Mahol wrote:
> > On Fri, Nov 26, 2021 at 10:38 AM wrote:
> >
> > > From: Limin Wang
> > >
> > > Signed-off-by: Limin Wang
> > > ---
> > > doc/filters.texi| 5 +
> > > libavfilter/src_movie.c
From: Limin Wang
Reviewed-by: Martin Storsjö
Signed-off-by: Limin Wang
---
support for both sdp and rtp for v2:
1, -localaddr xx.xx.xx.xx -i 1.sdp
2, -localaddr xx.xx.xx.xx -i rtp://xx.xx.xx.xx:xx
3, -i rtp://xx.xx.xx.xx:xx?localadd xx.xx.xx.xx
doc/protocols.texi | 4
libavformat/rt
Wu Jianhua:
> +user_layers_str = av_strdup(user_layers->value);
> +if (!user_layers_str) {
> +err = AVERROR(EINVAL);
Wrong error code.
> +goto fail;
> +}
> +
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmp
On Fri, Nov 26, 2021 at 11:26:39AM +0100, Andreas Rheinhardt wrote:
> lance.lmw...@gmail.com:
> > From: Limin Wang
> >
> > Signed-off-by: Limin Wang
> > ---
> > libavformat/udp.c | 10 --
> > 1 file changed, 4 insertions(+), 6 deletions(-)
> >
> > diff --git a/libavformat/udp.c b/libav
On 2021-11-26 03:08 pm, Lynne wrote:
26 Nov 2021, 10:10 by jianhua...@intel.com:
This filter flips the input video both horizontally and vertically
in one compute pipeline, and it's no need to use two pipelines for
hflip_vulkan,vflip_vulkan anymore.
Signed-off-by: Wu Jianhua
---
configur
Soft Works:
> Root commit for adding subtitle filtering capabilities.
> In detail:
>
> - Add type (AVMediaType) field to AVFrame
> Replaces previous way of distinction which was based on checking
> width and height to determine whether a frame is audio or video
> - Add subtitle fields to AVFra
lance.lmw...@gmail.com:
> From: Limin Wang
>
> Signed-off-by: Limin Wang
> ---
> libavformat/udp.c | 10 --
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/libavformat/udp.c b/libavformat/udp.c
> index bbe5aec..b8b0e19 100644
> --- a/libavformat/udp.c
> +++ b/libavfo
On Fri, Nov 26, 2021 at 10:48:07AM +0100, Paul B Mahol wrote:
> On Fri, Nov 26, 2021 at 10:38 AM wrote:
>
> > From: Limin Wang
> >
> > Signed-off-by: Limin Wang
> > ---
> > doc/filters.texi| 5 +
> > libavfilter/src_movie.c | 4 +++-
> > 2 files changed, 8 insertions(+), 1 deletion
On Fri, Nov 26, 2021 at 9:17 AM Nicolas George wrote:
> James Almer (12021-11-25):
> > Have all the concerns people had been addressed? I'm also certain the
> > design itself wasn't well received.
>
> Indeed. The whole libavfilter part is nowhere near anything acceptable.
> It should not be appli
On Fri, Nov 26, 2021 at 10:38 AM wrote:
> From: Limin Wang
>
> Signed-off-by: Limin Wang
> ---
> doc/filters.texi| 5 +
> libavfilter/src_movie.c | 4 +++-
> 2 files changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/doc/filters.texi b/doc/filters.texi
> index 3731a14..f2d595
From: Limin Wang
Signed-off-by: Limin Wang
---
doc/filters.texi| 5 +
libavfilter/src_movie.c | 4 +++-
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/doc/filters.texi b/doc/filters.texi
index 3731a14..f2d5957 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
@@ -2
26 Nov 2021, 10:10 by jianhua...@intel.com:
> This filter flips the input video both horizontally and vertically
> in one compute pipeline, and it's no need to use two pipelines for
> hflip_vulkan,vflip_vulkan anymore.
>
> Signed-off-by: Wu Jianhua
> ---
> configure| 1 +
>
26 Nov 2021, 10:10 by jianhua...@intel.com:
> Validation layer is an indispensable part of developing on Vulkan.
>
> The following commands is on how to enable validation layers:
>
> ffmpeg -init_hw_device
> vulkan=0,debug=1,validation_layers=VK_LAYER_LUNARG_monitor+VK_LAYER_LUNARG_api_dump
>
> S
On Fri, Nov 26, 2021 at 07:36:33AM +0100, Andreas Rheinhardt wrote:
> lance.lmw...@gmail.com:
> > From: Limin Wang
> >
> > Signed-off-by: Limin Wang
> > ---
> > doc/filters.texi| 5 +
> > libavfilter/src_movie.c | 8 +++-
> > 2 files changed, 12 insertions(+), 1 deletion(-)
> >
This filter flips the input video both horizontally and vertically
in one compute pipeline, and it's no need to use two pipelines for
hflip_vulkan,vflip_vulkan anymore.
Signed-off-by: Wu Jianhua
---
configure| 1 +
libavfilter/allfilters.c | 1 +
libavfilter/vf_flip_vul
Validation layer is an indispensable part of developing on Vulkan.
The following commands is on how to enable validation layers:
ffmpeg -init_hw_device
vulkan=0,debug=1,validation_layers=VK_LAYER_LUNARG_monitor+VK_LAYER_LUNARG_api_dump
Signed-off-by: Wu Jianhua
---
libavutil/hwcontext_vulkan.
James Almer (12021-11-25):
> Have all the concerns people had been addressed? I'm also certain the
> design itself wasn't well received.
Indeed. The whole libavfilter part is nowhere near anything acceptable.
It should not be applied at all.
Regards,
--
Nicolas George
25 Nov 2021, 23:49 by c...@passwd.hu:
>
>
> On Thu, 25 Nov 2021, Lynne wrote:
>
>> This adds a time_base field (currently unused), analogue to the
>> AVPacket.time_base field.
>> This allows for API clients to exchange AVFrames directly, without
>> needing to plumb extra data from sources via side
67 matches
Mail list logo
