On 11/05/2022 01:32, Soft Works wrote:
[...]
The prefixing can be implemented as a function and then be used
in file_open.c.
Other file system related functions like mkdir, rename, rmdir, etc.
are already intercepted in os_support.h, and the prefixing can be
applied there.
Maybe I missed some
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Tobias Rapp
> Sent: Wednesday, May 11, 2022 9:46 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v11 1/6]
> libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and
> utf8toansi
>
> On 11/05/2022 01:32,
On Wed, May 11, 2022 at 9:58 AM Soft Works wrote:
>
>
>
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of
> > Tobias Rapp
> > Sent: Wednesday, May 11, 2022 9:46 AM
> > To: ffmpeg-devel@ffmpeg.org
> > Subject: Re: [FFmpeg-devel] [PATCH v11 1/6]
> > libavutil/wchar_filename.h: Add
---
fftools/ffmpeg.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 9fa0719cf6..69b9c42822 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -840,15 +840,15 @@ static int encode_frame(OutputFile *of, OutputStream
---
fftools/ffmpeg.c | 75 +++-
1 file changed, 16 insertions(+), 59 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 0e62c39522..9fa0719cf6 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -836,12 +836,12 @@ static int encode_
do_video_stats() is only ever called for video.
---
fftools/ffmpeg.c | 48 +++-
1 file changed, 23 insertions(+), 25 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 69b9c42822..2bc612f448 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/f
Call do_video_stats() for every video packets produced by the encoder,
rather than for every frame sent to the encoder.
---
fftools/ffmpeg.c | 145 ---
1 file changed, 60 insertions(+), 85 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
in
It retrieves libavformat's internal dts value (contrary to the
function's name), which is not only incorrect in general, but also
unnecessary because we can access the packet directly.
---
fftools/ffmpeg.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/fftools/ffmpe
---
fftools/ffmpeg.c | 93
1 file changed, 46 insertions(+), 47 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 2bc612f448..ae3cc57bef 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -127,7 +127,6 @@ typedef struct Bench
---
fftools/ffmpeg.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 5983c57410..fe7c37db62 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -837,7 +837,7 @@ static double psnr(double d)
static void do_video_stats(OutputStre
Its use for muxing is not documented, in practice it is incremented per
each packet successfully passed to the muxer's write_packet(). Since
there is a lot of indirection between ffmpeg receiving a packet from the
encoder and it actually being written (e.g. bitstream filters, the
interleaving queue
This is a more appropriate place for this code, since the values we read
from AV_PKT_DATA_QUALITY_STATS side data are primarily written into
video stats. This ensures that the values written into stats actually
apply to the right packet.
Rename the function to update_video_stats() to better reflec
In case anyone cares about the context - these patches were previously a
part of the muxer-threading set, but I've decided to factor them out and
reshuffle them. do_video_stats() is now kept close to the encoder rather
than being moved to the muxing code.
--
Anton Khirnov
> I think that can be changed easily.
How about writing the necessary patches yourself?
> A path using forward slashes can still be prefixed with '\\?\'
It cannot. Only backslashes are valid in \\?\ paths.
> The prefixing can be implemented as a function and then be used
> in file_open.c.
> Ot
> I'm not sure how much you had followed, so please excuse in case you
> had already read it: the manifest approach does not work on a default
> Windows installation.
> To activate long path support, the users needs to opt-in to a behavior
> that is probably deactivated by default for some reason.
Quoting Zane van Iperen (2022-04-24 12:14:03)
> +void av_uuid_nil_set(AVUUID uu)
^^^
sounds weird
av_uuid_zero()?
av_uuid_reset()?
--
Anton Khirnov
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/lis
Quoting Gyan Doshi (2022-05-10 13:40:54)
> So far, -metadata:s:v rotate would only be applied to streamcopied
> video streams.
Using -metadata for this functionality is a hack that should be removed,
not extended.
--
Anton Khirnov
___
ffmpeg-devel mail
Quoting lance.lmw...@gmail.com (2022-05-08 09:17:01)
> From: Limin Wang
>
> some samples include both A53 and SCTE20 data. Before the commit, both
> of the will be exported, so the CC data will be repeated or garbarge
Why would it be garbage? That sounds like a bug.
Why can't we just export both
On 2022-05-11 05:26 pm, Anton Khirnov wrote:
Quoting Gyan Doshi (2022-05-10 13:40:54)
So far, -metadata:s:v rotate would only be applied to streamcopied
video streams.
Using -metadata for this functionality is a hack that should be removed,
not extended.
When there's a replacement for CLI
On Wed, May 11, 2022 at 02:00:10PM +0200, Anton Khirnov wrote:
> Quoting lance.lmw...@gmail.com (2022-05-08 09:17:01)
> > From: Limin Wang
> >
> > some samples include both A53 and SCTE20 data. Before the commit, both
> > of the will be exported, so the CC data will be repeated or garbarge
>
> W
On 11/05/2022 09:57, Soft Works wrote:
[...]
I'm not sure how much you had followed, so please excuse in case you
had already read it: the manifest approach does not work on a default
Windows installation.
To activate long path support, the users needs to opt-in to a behavior
that is probably d
From: Limin Wang
Signed-off-by: Limin Wang
---
libavformat/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/Makefile b/libavformat/Makefile
index 3b9995a9d3..8e612b6cc7 100644
--- a/libavformat/Makefile
+++ b/libavformat/Makefile
@@ -514,7 +514,7 @@ OBJS-
From: Limin Wang
Signed-off-by: Limin Wang
---
libavcodec/ccaption_dec.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c
index 34f0513b1a..8f61e8aa03 100644
--- a/libavcodec/ccaption_dec.c
+++ b/libavcodec/ccaption_
From: Limin Wang
Signed-off-by: Limin Wang
---
libavformat/sccenc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavformat/sccenc.c b/libavformat/sccenc.c
index c8c4d097e4..2b924ba6e7 100644
--- a/libavformat/sccenc.c
+++ b/libavformat/sccenc.c
@@ -72,11 +72,11 @@ s
From: Limin Wang
Signed-off-by: Limin Wang
---
libavcodec/dvdsubenc.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/libavcodec/dvdsubenc.c b/libavcodec/dvdsubenc.c
index fc3b7d1816..d29db7d49c 100644
--- a/libavcodec/dvdsubenc.c
+++ b/libavcodec/dvdsubenc.c
@@ -376,6 +376,12 @@ stat
On 10/5/22 23:18, Andreas Rheinhardt wrote:
+int av_uuid_parse_range(const char *in_start, const char *in_end, AVUUID uu)
+{
+int i;
+const char *cp;
+char buf[3];
+
+if ((in_end - in_start) != 36)
+return -1;
+
+for (i = 0, cp = in_start; i < 36; i++, cp++) {
+
On 11/5/22 21:53, Anton Khirnov wrote:
Quoting Zane van Iperen (2022-04-24 12:14:03)
+void av_uuid_nil_set(AVUUID uu)
^^^
sounds weird
av_uuid_zero()?
av_uuid_reset()?
Good point, `av_uuid_zero()` has a nice ring to it.
___
It is unneeded, as this decoder does not call ff_get_buffer().
Signed-off-by: Andreas Rheinhardt
---
libavcodec/wrapped_avframe.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/libavcodec/wrapped_avframe.c b/libavcodec/wrapped_avframe.c
index 3af007d478..06c274eed0 100644
--- a/libavco
Currently, every call to ff_get_buffer() allocates FrameDecodeData,
although only very few decoders (those that might use nvdec or
videotoolbox hardware acceleration) actually need it.
This commit addresses this by adding an internal codec cap
to ensure that FrameDecodeData is only allocated for c
On 2022-05-10 05:10 pm, Gyan Doshi wrote:
So far, -metadata:s:v rotate would only be applied to streamcopied
video streams.
Plan to push tomorrow.
Gyan
---
fftools/ffmpeg.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index a85ed18b08..
On Wed, May 4, 2022 at 10:15 AM Vignesh Venkatasubramanian
wrote:
>
> Add an AVIF muxer by re-using the existing the mov/mp4 muxer.
>
> AVIF Specification: https://aomediacodec.github.io/av1-avif
>
> Sample usage for still image:
> ffmpeg -i image.png -c:v libaom-av1 -avif-image 1 image.avif
>
> S
On 2022-05-11 10:24 pm, Vignesh Venkatasubramanian wrote:
On Wed, May 4, 2022 at 10:15 AM Vignesh Venkatasubramanian
wrote:
Add an AVIF muxer by re-using the existing the mov/mp4 muxer.
AVIF Specification: https://aomediacodec.github.io/av1-avif
Sample usage for still image:
ffmpeg -i imag
Andreas Rheinhardt:
> The general decoding API uses bitstream filters and an AVFifo
> and therefore AVCodecInternal contains pointers to an AVBSFContext
> and to an AVFifo and lavc/internal.h includes lavc/bsf.h and
> lavu/fifo.h.
> Yet actually, only two files are supposed to use these, namely
> a
Quoting Gyan Doshi (2022-05-11 18:30:41)
>
>
> On 2022-05-10 05:10 pm, Gyan Doshi wrote:
> > So far, -metadata:s:v rotate would only be applied to streamcopied
> > video streams.
>
> Plan to push tomorrow.
WTF? This is not how development works.
I had an objection to your patch, it was not res
Quoting Gyan Doshi (2022-05-11 14:18:49)
>
>
> On 2022-05-11 05:26 pm, Anton Khirnov wrote:
> > Quoting Gyan Doshi (2022-05-10 13:40:54)
> >> So far, -metadata:s:v rotate would only be applied to streamcopied
> >> video streams.
> > Using -metadata for this functionality is a hack that should be
On Wed, May 11, 2022 at 10:16:47AM +0200, Anton Khirnov wrote:
> ---
> fftools/ffmpeg.c | 75 +++-
> 1 file changed, 16 insertions(+), 59 deletions(-)
breaks 2pass with ffv1
./ffmpeg -y -i mm-short.mpg -an -vcodec ffv1 -strict -2 -slices 4 -t 1 -coder
On 2022-05-12 12:19 am, Anton Khirnov wrote:
Quoting Gyan Doshi (2022-05-11 14:18:49)
On 2022-05-11 05:26 pm, Anton Khirnov wrote:
Quoting Gyan Doshi (2022-05-10 13:40:54)
So far, -metadata:s:v rotate would only be applied to streamcopied
video streams.
Using -metadata for this functional
why?
___
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".
sdl2 recently changed their versioning, moving the patch level to minor level
https://github.com/libsdl-org/SDL/commit/cd7c2f1de7d9e418bb554047d714dd7cacc020ff
and have said that they will instead ship sdl3.pc for 3.0.0
trac: https://trac.ffmpeg.org/ticket/9768
Signed-off-by: Christopher Degawa
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Tobias Rapp
> Sent: Wednesday, May 11, 2022 3:33 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v11 1/6]
> libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and
> utf8toansi
>
> On 11/05/2022 09:57,
On Wed, May 11, 2022 at 09:47:52PM +0200, Paul B Mahol wrote:
> why?
assuming the len is 1, the following code will access the next 3
array anymore, I think it's better to check the i with len -2.
for (i = 0; i < len; i += 3) {
to
for (i = 0; i < len - 2; i += 3) {
for the return, I think it's
1. Return error if filter frame fail before set outlink to EOF in none
pass through mode.
2. Set outlink to EOF before return success in pass through mode.
Fix endless cmd:
ffmpeg -hwaccel qsv -qsv_device /dev/dri/renderD128 -hwaccel_output_format \
qsv -v debug -c:v hevc_qsv -i 4k.h265
On 2022-05-12 01:41 am, Christopher Degawa wrote:
sdl2 recently changed their versioning, moving the patch level to minor level
https://github.com/libsdl-org/SDL/commit/cd7c2f1de7d9e418bb554047d714dd7cacc020ff
and have said that they will instead ship sdl3.pc for 3.0.0
Will apply.
Regards,
Quoting Gyan Doshi (2022-05-11 21:15:15)
>
>
> On 2022-05-12 12:19 am, Anton Khirnov wrote:
> > Quoting Gyan Doshi (2022-05-11 14:18:49)
> >>
> >> On 2022-05-11 05:26 pm, Anton Khirnov wrote:
> >>> Quoting Gyan Doshi (2022-05-10 13:40:54)
> So far, -metadata:s:v rotate would only be applied
Prerequisites that must be met before adding new codec
- Added new entry to codec IDs list
- Added new entry to the codec descriptor list
- Bumped libavcodec minor version
- Changes in Changelog and MAINTAINERS files
Signed-off-by: Dawid Kozinski
---
Changelog | 3 ++-
MAINTAINERS
- Added xeve encoder wrapper
- Added xevd dencoder wrapper
- Added documentation for xeve and xevd wrappers
- Added parser for EVC format
- Changes in project configuration file and libavcodec Makefile
Signed-off-by: Dawid Kozinski
---
configure |8 +
doc/decoders.texi
- Added muxer for EVC format (MP4, raw)
- Added demuxer for EVC format (MP4)
- Added evc extension to the list of extensions for ff_mov_demuxer
Signed-off-by: Dawid Kozinski
---
doc/muxers.texi | 6 ++
libavformat/Makefile | 2 +
libavformat/allformats.c | 2 +
libavformat/evc
Thank you for your review. All your suggestions have been taken into account
and new patches have just been put on the mailing list.
-Original Message-
From: ffmpeg-devel On Behalf Of Andreas
Rheinhardt
Sent: Tuesday, April 5, 2022 7:37 PM
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpe
On Thu, May 12, 2022 at 1:39 AM wrote:
> On Wed, May 11, 2022 at 09:47:52PM +0200, Paul B Mahol wrote:
> > why?
>
> assuming the len is 1, the following code will access the next 3
> array anymore, I think it's better to check the i with len -2.
>
> for (i = 0; i < len; i += 3) {
> to
> for (i =
On 2022-05-12 10:35 am, Gyan Doshi wrote:
On 2022-05-12 01:41 am, Christopher Degawa wrote:
sdl2 recently changed their versioning, moving the patch level to
minor level
https://github.com/libsdl-org/SDL/commit/cd7c2f1de7d9e418bb554047d714dd7cacc020ff
and have said that they will instead
50 matches
Mail list logo