Same logic as tile_cols and tile_rows, this saves CBS users having to
recalculate these values, and makes them available for all frames within a
Temporal Unit.
Signed-off-by: James Almer
---
Not a fan of adding more derived values to "raw" structs, but much like the
tile ones, these will be usefu
> On Aug 23, 2020, at 7:04 AM, Andreas Rheinhardt
> wrote:
>
> Casting const away directly before return is completely pointless
> here as avfilter_get_by_name() already returns a const AVFilter *.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavfilter/allfilters.c | 2 +-
> 1 file changed,
Signed-off-by: James Almer
---
libavcodec/av1_parser.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/av1_parser.c b/libavcodec/av1_parser.c
index cd426a2b0f..0a34f20a59 100644
--- a/libavcodec/av1_parser.c
+++ b/libavcodec/av1_parser.c
@@ -127,8 +127,8 @@ stat
The values contained in these fields are only valid for the last frame parsed
by CBS. Unlike tile_rows and tile_cols, these are not used in functions within
cbs_av1_syntax_template.c that don't have access to the relevant
AV1RawFrameHeader, and since each frame now has their relevant derived values
On Sat, Aug 22, 2020 at 07:36:24PM +0200, Marton Balint wrote:
>
>
> On Sat, 15 Aug 2020, lance.lmw...@gmail.com wrote:
>
> > From: Limin Wang
> >
> > Signed-off-by: Limin Wang
> > ---
> > configure | 2 +-
> > libavformat/mpegts.h| 16 ++
> > libavformat/mpegtsenc.c
On Sat, Aug 22, 2020 at 07:27:30PM +0200, Marton Balint wrote:
>
>
> On Sat, 22 Aug 2020, lance.lmw...@gmail.com wrote:
>
> > On Wed, Jul 22, 2020 at 11:27:38PM +0800, lance.lmw...@gmail.com wrote:
> > > From: Limin Wang
> > >
> > > copy the atsc ac3 audio in ts like below:
> > > ./ffmpeg -i a
parse_inputs() uses a temporary linked list to parse the labeled inputs
of a filter; said linked list owns its elements (and their names). On
success, the list of unlabeled inputs is appened to the end of the list
of labeled inputs and the new list is returned; yet on failures, nothing
frees the al
Casting const away directly before return is completely pointless
here as avfilter_get_by_name() already returns a const AVFilter *.
Signed-off-by: Andreas Rheinhardt
---
libavfilter/allfilters.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/allfilters.c b/libav
This happened in parse_link_name() if there was a '[' without matching
']'. While this is not undefined behaviour (pointer arithmetic one
beyond the end of an array works fine as long as there are no accesses),
it is potentially dangerous. It currently isn't (all callers of
parse_link_name() treat
Signed-off-by: Anamitra Ghorui
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/configure b/configure
index 6faff9bc7b..a80e300cdb 100755
--- a/configure
+++ b/configure
@@ -3307,6 +3307,7 @@ fifo_muxer_deps="threads"
flac_demuxer_select="flac_parser"
flv_muxer_select="aac_adts
Signed-off-by: Anamitra Ghorui
---
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/flifdec.c| 431 +++
libavformat/version.h| 2 +-
4 files changed, 434 insertions(+), 1 deletion(-)
create mode 100644 libavformat/flifd
Signed-off-by: Anamitra Ghorui
---
Changelog| 3 ++-
doc/general.texi | 2 ++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/Changelog b/Changelog
index 7467e73306..cfa21c280b 100644
--- a/Changelog
+++ b/Changelog
@@ -15,7 +15,8 @@ version :
- Argonaut Games ASF muxer
-
They are not explicitly in the bitstream in this case, but it is helpful
to be able to use these values without always needing to check the flag
beforehand.
---
On 22/08/2020 23:01, Mark Thompson wrote:
On 21/08/2020 06:29, Fei Wang wrote:
...
+
+ if (s->raw_frame_header.uniform_tile_spacing_
On 21/08/2020 06:29, Fei Wang wrote:
This av1 decoder is now only used for av1 hardware acceleration
decoder. Consider it can be extend to a local decoder like hevc
or vp9 in the future, so define its name as "av1" and put it into
external libraries codec list.
Signed-off-by: Fei Wang
---
Cha
Am Sa., 22. Aug. 2020 um 23:15 Uhr schrieb Jan Ekström :
>
> On Sat, Aug 22, 2020 at 11:42 PM Carl Eugen Hoyos wrote:
> >
> > Am Sa., 22. Aug. 2020 um 22:40 Uhr schrieb Jan Ekström :
> > >
> > > Validates the set channel layout as well as verifies that the received
> > > layout to the function mat
On 21/08/2020 06:49, Moritz Barsnick wrote:
On Thu, Aug 20, 2020 at 22:49:16 +0100, Mark Thompson wrote:
With this patch, -disable_auto_conversion_filters does what you want.
[...]
+if (!po->name) {
+/* Try to match a boolean option with a negative prefix. */
+for (int i =
On Sat, Aug 22, 2020 at 11:42 PM Carl Eugen Hoyos wrote:
>
> Am Sa., 22. Aug. 2020 um 22:40 Uhr schrieb Jan Ekström :
> >
> > Validates the set channel layout as well as verifies that the received
> > layout to the function matches the reference layout, so that it matches
> > the implemented re-or
On Sat, Aug 22, 2020 at 11:39 PM Jan Ekström wrote:
>
> Validates the set channel layout as well as verifies that the received
> layout to the function matches the reference layout, so that it matches
> the implemented re-ordering logic.
>
> Fixes #8845
> ---
> libavcodec/aacdec_template.c | 21 +
On Fri Aug 21 15:35:38 EEST 2020, Nicolas George wrote:
> 1. What would you think about putting the documentation for
> libavfilter/vf_foobar.c into libavfilter/doc/vf_foobar.texi …
> 2. What would you think about switching from texinfo to a small basic
> subset of HTML for new documenta
Jan Ekström:
> Validates the set channel layout as well as verifies that the received
> layout to the function matches the reference layout, so that it matches
> the implemented re-ordering logic.
>
> Fixes #8845
What about the new tickets #8859, #8860?
- Andreas
Am Sa., 22. Aug. 2020 um 22:40 Uhr schrieb Jan Ekström :
>
> Validates the set channel layout as well as verifies that the received
> layout to the function matches the reference layout, so that it matches
> the implemented re-ordering logic.
>
> Fixes #8845
Did you also see tickets #8859 and #886
Validates the set channel layout as well as verifies that the received
layout to the function matches the reference layout, so that it matches
the implemented re-ordering logic.
Fixes #8845
---
libavcodec/aacdec_template.c | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(
On Sat, Aug 22, 2020 at 02:12:13PM +0200, Nicolas George wrote:
> Clement Boesch (12020-08-22):
> > AVOption.help and AVFilter.description (as well as AVCodec.description
> > etc) do contain redundant information with the documentation.
>
> There is truth in what you say, but I disagree this is re
On Sat, Aug 22, 2020 at 05:01:05PM +0300, Jan Ekström wrote:
> On Sat, Aug 22, 2020 at 4:44 PM Michael Niedermayer
> wrote:
> >
> > On Sat, Aug 22, 2020 at 02:36:09PM +0300, Jan Ekström wrote:
> > > On Sat, Aug 22, 2020 at 2:17 PM Michael Niedermayer
> > > wrote:
> > > >
> > > > On Sat, Aug 22, 2
Hi all,
hi Paul!
Shouldn't it have been also reflected in the commit's metadata,
author or message, that the encoder was originally written by
Todd Kirby and David Adler?
Did you base it on the version from Jaikrishnan Menon?
If yes, I think it should have been mentioned in the commit message too
On Sat, 15 Aug 2020, lance.lmw...@gmail.com wrote:
From: Limin Wang
Signed-off-by: Limin Wang
---
configure | 2 +-
libavformat/mpegts.h| 16 ++
libavformat/mpegtsenc.c | 84 +++--
3 files changed, 99 insertions(+), 3 delet
[...]
Patchset applied, thanks for the review
--
Clément B.
___
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
On Fri, Aug 21, 2020 at 10:22:23PM +0200, Clément Bœsch wrote:
> On Fri, Aug 21, 2020 at 06:24:58PM +0200, Alexander Strasser wrote:
> > On 2020-08-21 14:32 +0200, Clément Bœsch wrote:
> > > On Tue, Aug 18, 2020 at 11:43:29PM +0200, Michael Niedermayer wrote:
> > > > On Tue, Aug 18, 2020 at 10:45:5
On Sat, 22 Aug 2020, lance.lmw...@gmail.com wrote:
On Wed, Jul 22, 2020 at 11:27:38PM +0800, lance.lmw...@gmail.com wrote:
From: Limin Wang
copy the atsc ac3 audio in ts like below:
./ffmpeg -i atsc_audio.ts -c:v copy -c:a copy out.ts
Stream #0:6[0x64](eng): Audio: ac3 (AC-3 / 0x332D4341),
On 2020-08-22 12:59 +0200, Nicolas George wrote:
> Alexander Strasser (12020-08-15):
> > If I'm not mistaken that reason for the rewrite didn't make
> > it into the commit message:
>
> You are right. The reason for the complete rewrite was that the original
> code was completely broken which I did
Hi Nicolas,
hi Clement!
Wow the discussion gets up to speed quickly :)
Thanks for bringing up all those good points!
I'm intentionally top-posting, because I want to say something
concerning the points you discuss below. Though I couldn't find
a good way to do it in a comment style, so instead I'
Nicolas George:
> Andreas Rheinhardt (12020-08-21):
>> This only works if one only works with one list without owner at a time;
>> if one has several such lists at the same time and an error happens when
>> working with one of these lists, one has to free the other lists.
>>
>> This scenario can of
On Sat, Aug 22, 2020 at 4:44 PM Michael Niedermayer
wrote:
>
> On Sat, Aug 22, 2020 at 02:36:09PM +0300, Jan Ekström wrote:
> > On Sat, Aug 22, 2020 at 2:17 PM Michael Niedermayer
> > wrote:
> > >
> > > On Sat, Aug 22, 2020 at 12:57:59AM +0300, Jan Ekström wrote:
> > > > This way we can check tha
On Sat, Aug 22, 2020 at 02:36:09PM +0300, Jan Ekström wrote:
> On Sat, Aug 22, 2020 at 2:17 PM Michael Niedermayer
> wrote:
> >
> > On Sat, Aug 22, 2020 at 12:57:59AM +0300, Jan Ekström wrote:
> > > This way we can check that we have exactly the required things for 22.2.
> > >
> > > Fixes #8845
>
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Nicolas George
> Sent: Saturday, August 22, 2020 2:12 PM
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] Documentation: proposed changes in the
> structure
>
> Clement Boesch (1202
Andreas Rheinhardt (12020-08-21):
> This only works if one only works with one list without owner at a time;
> if one has several such lists at the same time and an error happens when
> working with one of these lists, one has to free the other lists.
>
> This scenario can of course usually be avo
Andreas Rheinhardt (12020-08-21):
> If adding the list of input formats to its AVFilterLink fails, the list
> of output formats (which has not been attached to permanent storage yet)
> leaks. This has been fixed by not creating the lists of in- and output
> formats simultaneously. Instead creating
Clement Boesch (12020-08-22):
> AVOption.help and AVFilter.description (as well as AVCodec.description
> etc) do contain redundant information with the documentation.
There is truth in what you say, but I disagree this is really redundant:
AVOption.help and AV*.description are supposed to be very
On Sat, Aug 15, 2020 at 10:27:38PM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang
>
> Signed-off-by: Limin Wang
> ---
> configure | 2 +-
> libavformat/mpegts.h| 16 ++
> libavformat/mpegtsenc.c | 84
> +++--
> 3 f
On 22-08-2020 04:23 pm, Nicolas George wrote:
Signed-off-by: Nicolas George
---
doc/texi2pod.pl | 9 +
1 file changed, 9 insertions(+)
diff --git a/doc/texi2pod.pl b/doc/texi2pod.pl
index 9a9b34fc15..c7f67afe8c 100644
--- a/doc/texi2pod.pl
+++ b/doc/texi2pod.pl
@@ -172,6 +172,9 @@
On Sat, Aug 22, 2020 at 2:21 PM Michael Niedermayer
wrote:
>
> On Sat, Aug 22, 2020 at 12:58:00AM +0300, Jan Ekström wrote:
> > This was quite useful to verify that 22.2 got properly handled,
> > among other things.
> > ---
> > libavcodec/aacdec_template.c | 26 ++
> > 1 f
On Sat, Aug 22, 2020 at 2:17 PM Michael Niedermayer
wrote:
>
> On Sat, Aug 22, 2020 at 12:57:59AM +0300, Jan Ekström wrote:
> > This way we can check that we have exactly the required things for 22.2.
> >
> > Fixes #8845
> > ---
> > libavcodec/aacdec_template.c | 47 ++
On Wed, Jul 22, 2020 at 11:27:38PM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang
>
> copy the atsc ac3 audio in ts like below:
> ./ffmpeg -i atsc_audio.ts -c:v copy -c:a copy out.ts
> Stream #0:6[0x64](eng): Audio: ac3 (AC-3 / 0x332D4341), 48000 Hz, stereo,
> fltp, 192 kb/s
>
> ./ffmp
On Sat, Aug 22, 2020 at 12:58:00AM +0300, Jan Ekström wrote:
> This was quite useful to verify that 22.2 got properly handled,
> among other things.
> ---
> libavcodec/aacdec_template.c | 26 ++
> 1 file changed, 26 insertions(+)
>
> diff --git a/libavcodec/aacdec_template
On Sat, Aug 22, 2020 at 12:57:59AM +0300, Jan Ekström wrote:
> This way we can check that we have exactly the required things for 22.2.
>
> Fixes #8845
> ---
> libavcodec/aacdec_template.c | 47 ++--
> 1 file changed, 45 insertions(+), 2 deletions(-)
>
> diff --gi
Nicolas George (12020-08-09):
> If an input has an undefined timestamp, the computation
> for silence at stitches can overflow.
>
> Partial fix for trac ticket #8843.
>
> Signed-off-by: Nicolas George
> ---
> libavfilter/avf_concat.c | 4
> 1 file changed, 4 insertions(+)
Ping? Will apply
Alexander Strasser (12020-08-15):
> If I'm not mistaken that reason for the rewrite didn't make
> it into the commit message:
You are right. The reason for the complete rewrite was that the original
code was completely broken which I did not want to include permanently
in the history.
By complete
On Fri, Aug 21, 2020 at 09:16:52PM -0700, h...@riseup.net wrote:
> ffmpeg documentation says the NUT container supports SubStation Alpha
> This brings actual functionality in line with documentation.
> ---
> libavformat/nut.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavformat/
It will allow to include it.
Signed-off-by: Nicolas George
---
doc/general.texi| 1415 +--
doc/{general.texi => general_contents.texi} | 14 -
2 files changed, 1 insertion(+), 1428 deletions(-)
copy doc/{general.texi => general_contents.texi} (99%)
Signed-off-by: Nicolas George
---
doc/ffmpeg.texi | 1 +
doc/ffplay.texi | 1 +
doc/ffprobe.texi | 1 +
3 files changed, 3 insertions(+)
diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi
index 267ddfe8b5..329ee3019f 100644
--- a/doc/ffmpeg.texi
+++ b/doc/ffmpeg.texi
@@ -2040,6 +2040,7 @@ ffmpeg -
Signed-off-by: Nicolas George
---
doc/texi2pod.pl | 9 +
1 file changed, 9 insertions(+)
diff --git a/doc/texi2pod.pl b/doc/texi2pod.pl
index 9a9b34fc15..c7f67afe8c 100644
--- a/doc/texi2pod.pl
+++ b/doc/texi2pod.pl
@@ -172,6 +172,9 @@ INF: while(<$inf>) {
} elsif ($ended =~ /^(
On 8/22/20, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/rpzaenc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
lgtm
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/li
52 matches
Mail list logo
