On 3/12/20, Marshall Murmu wrote:
> This patch is part of the qualification task for Audio Tones Source Filter.
> ---
> Changelog | 1 +
> configure | 4 +
> doc/filters.texi | 26 +
> libavfilter/Makefile | 1 +
> libavfilt
On 3/13/20, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch fixes ticket #8568 for me.
>
> Please comment, Carl Eugen
>
whatever
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, vis
Hi,
I am using ffmpeg to convert RTSP stream to HLS chunk files. incase there
is network error and the rtsp url is not receiving any data. ffmpeg doesn't
retry for the data after the network issue is fixed.
Regards,
Siddhant Agarwal
Back-End Developer
_
With the description in frame size with refs semantics (SPEC 7.2.5),
it is a requirement of bitstream conformance that for at least one
reference frame has the valid dimensions.
Modify the check to make sure the decoder works well in SINGLE_REFERENCE
mode that not all reference frames have valid d
> From: Fu, Linjie
> Sent: Friday, March 13, 2020 17:18
> To: ffmpeg-devel@ffmpeg.org
> Cc: Fu, Linjie
> Subject: [PATCH] lavc/vp9: fix reference frame dimensions check for
> SINGLE_REFERENCE mode
>
> With the description in frame size with refs semantics (SPEC 7.2.5),
> it is a requirement of b
>
> > +}
>> > +
>> > +static int query_formats(AVFilterContext *ctx)
>> > +{
>> > +FluidSynthContext *fluidsynth = ctx->priv;
>> > +
>> > +AVFilterChannelLayouts *chanlayout = NULL;
>> > +int64_t chanlayouts =
>> >
>> av_get_default_channel_layout(2*fluid_synth_count_audio_channels(flui
---
libavcodec/dxva2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/dxva2.c b/libavcodec/dxva2.c
index 32416112bf..6add893f3e 100644
--- a/libavcodec/dxva2.c
+++ b/libavcodec/dxva2.c
@@ -840,7 +840,7 @@ int ff_dxva2_commit_buffer(AVCodecContext *avctx,
And 4:2:0 12 bits as well.
---
libavcodec/hevcdec.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
index 8f1c162ace..04496a002b 100644
--- a/libavcodec/hevcdec.c
+++ b/libavcodec/hevcdec.c
@@ -420,6 +420,13 @@ static enum AVPixelForm
---
libavcodec/hevcdec.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
index 04496a002b..4c1650c416 100644
--- a/libavcodec/hevcdec.c
+++ b/libavcodec/hevcdec.c
@@ -432,6 +432,16 @@ static enum AVPixelFormat get_format(HEVCContext *s, c
Mimick the existing structure and add the extra fields from the Range Extension
in a wrapping structure.
The FF_DXVA2_WORKAROUND_HEVC_REXT is set by the decoder user to signal the
selected decoder is expecting this extended structure rather than the default
one.
---
libavcodec/d3d11va.h| 1 +
This is extremely fragile against reordering and hides what is actually
being copied. Copy all the fields manually instead.
---
libavcodec/mpeg4videodec.c | 27 ++-
1 file changed, 26 insertions(+), 1 deletion(-)
diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4vi
Specifically, between the user-facing one and the first frame thread
one.
This is fragile and dangerous, allocate separate private data for each
per-thread context.
---
libavcodec/pthread_frame.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/libavcodec/pthread_frame.
Use the AVFrame API to properly allocate and free frames for delayed
release.
---
libavcodec/pthread_frame.c | 40 --
1 file changed, 25 insertions(+), 15 deletions(-)
diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index 143fee238f..a166b3
This is a constant codec property, so a capability flag is more appropriate.
---
doc/multithreading.txt | 5 +++--
libavcodec/ffv1dec.c | 4 +---
libavcodec/h264dec.c | 5 ++---
libavcodec/hevcdec.c | 5 ++---
libavcodec/internal.h | 20 +---
libavcod
Currently the frame pool used by the default get_buffer2()
implementation is a single struct, allocated when opening the decoder.
A pointer to it is simply copied to each frame thread and we assume that
no thread attempts to modify it at an unexpected time. This is rather
fragile and potentially da
This SPS is not necessarily the one that will be used. Export the chroma
location along with all the other SPS properties.
---
libavcodec/h264_ps.c| 5 +++--
libavcodec/h264_ps.h| 2 ++
libavcodec/h264_slice.c | 1 +
libavcodec/h264dec.c| 2 --
4 files changed, 6 insertions(+), 4 delet
It is very fragile against fields being moved and hides what is actually
being copied. Copy all the fields explicitly instead.
---
libavcodec/vp3.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c
index 0a8c8ad286..81d0b9b7bb 1006
This is extremely fragile against reordering and hides what is actually
being copied. Copy all the fields manually instead.
---
libavcodec/pthread_frame.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index a166b3f
There are no failure cases left in this function.
---
libavcodec/h264_slice.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c
index e24d41ca50..ed4f711ac5 100644
--- a/libavcodec/h264_slice.c
+++ b/libavcodec/h264_slice.
The name is misleading, this function does a lot more than just flushing
the DPB.
---
libavcodec/h264dec.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c
index 79663d8083..a119fa7641 100644
--- a/libavcodec/h264dec.c
+++ b/libav
The current design, where
- proper init is called for the first per-thread context
- first thread's private data is copied into private data for all the
other threads
- a "fixup" function is called for all the other threads to e.g.
allocate dynamically allocated data
is very fragile and hard to
Makes sure it is only used for logging and nothing else.
---
libavcodec/h264_ps.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
index 1951bb1161..4ef25aa514 100644
--- a/libavcodec/h264_ps.c
+++ b/libavcodec/h264_
This tells the parsing functions the payload size and prevents them from
overreading.
---
libavcodec/h264_sei.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c
index a565feabe2..a2452141ca 100644
---
It represents the relationship between them more naturally and will be
useful in the following commits.
Allows significantly more frames in fate-h264-attachment-631 to be
decoded.
---
libavcodec/h264_ps.c | 29 +-
libavcodec/h264_ps.h | 3 +
libavcodec/h264_slic
Those SEIs refer to the currently active SPS. However, since the SEI
NALUs precede the coded picture data in the bitstream, the active SPS is
in general not known when we are decoding the SEI.
Therefore, store the content of the picture timing SEIs and actually
parse it when the active SPS is know
---
libavcodec/decode.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/libavcodec/decode.c b/libavcodec/decode.c
index 1b8c76352e..7671e8e70a 100644
--- a/libavcodec/decode.c
+++ b/libavcodec/decode.c
@@ -1595,7 +1595,7 @@ static int update_frame_pool(AVCodecContext *a
Such errors are not necessarily fatal and decoding might still be
possible, e.g. it happens for MVC streams where we do not handle the
subset SPS thus failing to parse its corresponding PPS.
---
libavcodec/h264dec.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/l
This has been cargo culted from mpegvideo and serves no useful purpose.
It will be initialize correctly in h264_field_start()
---
libavcodec/h264dec.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c
index 04b1f74cc2..6838b13eac 100644
--- a/libavcodec
Signed-off-by: Andreas Rheinhardt
---
libavformat/mxfenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
index 5e0dc0e889..55c715d776 100644
--- a/libavformat/mxfenc.c
+++ b/libavformat/mxfenc.c
@@ -72,10 +72,10 @@ typedef struct M
as well as includes of libavutil/timer.h.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/cabac.c | 1 -
libavcodec/dnxhddec.c | 3 --
libavcodec/dnxhdenc.c | 4 +-
libavcodec/ffv1.c | 1 -
libavcodec/ffv1.h | 1 -
It already initializes the packet.
Signed-off-by: Andreas Rheinhardt
---
Resending because of 3117f47f19d051d47ba29c9b78c2ca525f0fdb45.
fftools/ffplay.c | 2 +-
libavcodec/qsvdec_h2645.c | 2 +-
libavcodec/qsvdec_other.c | 2 +-
libavformat/fifo.c| 1 -
libavformat/img2enc.c
Renamed the filter from fluidsynth to atone
---
Changelog | 2 +-
configure | 2 +-
doc/filters.texi | 5 +-
libavfilter/Makefile | 2 +-
libavfilter/allfilters.c
Patch should not be difference from previous sent patch.
But always difference to current ffmpeg master tree.
So use git to rebase your work correctly.
On 3/13/20, Marshall Murmu wrote:
> Renamed the filter from fluidsynth to atone
> ---
> Changelog | 2 +-
>
Signed-off-by: numberwolf
---
libavcodec/mpegutils.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/libavcodec/mpegutils.c b/libavcodec/mpegutils.c
index c0ee3aa..148d3e7 100644
--- a/libavcodec/mpegutils.c
+++ b/libavcodec/mpegutils.c
@@ -34,6 +34,12 @@ static int
Signed-off-by: numberwolf
---
libavcodec/mpegutils.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/libavcodec/mpegutils.c b/libavcodec/mpegutils.c
index c0ee3aa..148d3e7 100644
--- a/libavcodec/mpegutils.c
+++ b/libavcodec/mpegutils.c
@@ -34,6 +34,12 @@ static int
On Fri, Mar 13, 2020 at 11:25 AM Steve Lhomme wrote:
>
> Mimick the existing structure and add the extra fields from the Range
> Extension
> in a wrapping structure.
>
> The FF_DXVA2_WORKAROUND_HEVC_REXT is set by the decoder user to signal the
> selected decoder is expecting this extended struct
On 3/13/2020 7:28 AM, Anton Khirnov wrote:
> This is extremely fragile against reordering and hides what is actually
> being copied. Copy all the fields manually instead.
> ---
> libavcodec/mpeg4videodec.c | 27 ++-
> 1 file changed, 26 insertions(+), 1 deletion(-)
>
> dif
On 3/13/2020 7:28 AM, Anton Khirnov wrote:
> Those SEIs refer to the currently active SPS. However, since the SEI
> NALUs precede the coded picture data in the bitstream, the active SPS is
> in general not known when we are decoding the SEI.
>
> Therefore, store the content of the picture timing S
On Thu, Mar 12, 2020, at 9:06 PM, Steven Liu wrote:
> Signed-off-by: Steven Liu
> ---
> generate-doc.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/generate-doc.sh b/generate-doc.sh
> index d8d01d3..e316c05 100755
> --- a/generate-doc.sh
> +++ b/generate-doc.sh
> @@
On Thu, Mar 12, 2020, at 8:32 PM, Gyan Doshi wrote:
>
> I already started on this after I added the MOV demuxer options.
>
> But this is a good start. I'll probably want to expand descriptions
> later on.
>
> Will have a look and push this weekend.
Thanks.
Here is my muxer.texi todo list if yo
On 3/13/2020 7:28 AM, Anton Khirnov wrote:
> diff --git a/libavcodec/wavpack.c b/libavcodec/wavpack.c
> index edc0f7911d..3e0b8f5db3 100644
> --- a/libavcodec/wavpack.c
> +++ b/libavcodec/wavpack.c
> @@ -603,15 +603,6 @@ static av_cold int wv_alloc_frame_context(WavpackContext
> *c)
> return
On 3/13/2020 7:28 AM, Anton Khirnov wrote:
> This is a constant codec property, so a capability flag is more appropriate.
> ---
> doc/multithreading.txt | 5 +++--
> libavcodec/ffv1dec.c | 4 +---
> libavcodec/h264dec.c | 5 ++---
> libavcodec/hevcdec.c | 5 ++---
> libav
On Fri, Mar 13, 2020, at 1:14 AM, Siddhant Agarwal wrote:
> Hi,
>
> I am using ffmpeg to convert RTSP stream to HLS chunk files. incase there
> is network error and the rtsp url is not receiving any data. ffmpeg doesn't
> retry for the data after the network issue is fixed.
>
> Regards,
> Siddhan
Since commit e134c203 strdups of several elements of a manifest are kept
in the DASHContext; but said commit completely forgot to free these
strings again (with xmlFree()). Given that these strings are never used
at all, this commit closes this leak by reverting said commit.
Signed-off-by: Andreas
Up until e7ddafd5, the Matroska muxer wrote two SeekHeads: One at the
beginning referencing the main level 1 elements (i.e. not the Clusters)
and one at the end, referencing the Clusters. This second SeekHead was
useless and has therefore been removed. Yet the SeekHead-related
functions and structu
On Fri, Mar 13, 2020 at 11:28:47AM +0100, Anton Khirnov wrote:
> The name is misleading, this function does a lot more than just flushing
> the DPB.
> ---
> libavcodec/h264dec.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
not tested (as a earlier patch didnt apply automatically)
On Fri, Mar 13, 2020 at 03:27:53PM -0300, James Almer wrote:
> On 3/13/2020 7:28 AM, Anton Khirnov wrote:
> > diff --git a/libavcodec/wavpack.c b/libavcodec/wavpack.c
> > index edc0f7911d..3e0b8f5db3 100644
> > --- a/libavcodec/wavpack.c
> > +++ b/libavcodec/wavpack.c
> > @@ -603,15 +603,6 @@ stati
On Fri, Mar 13, 2020 at 11:28:50AM +0100, Anton Khirnov wrote:
> Makes sure it is only used for logging and nothing else.
> ---
> libavcodec/h264_ps.c | 18 +-
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
> index 19
On Fri, Mar 13, 2020 at 11:28:42AM +0100, Anton Khirnov wrote:
> There are no failure cases left in this function.
> ---
> libavcodec/h264_slice.c | 8 ++--
> 1 file changed, 2 insertions(+), 6 deletions(-)
untested but LGTM
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF
When the Matroska muxer writes the Cues (the index), it groups index
entries with the same timestamp into the same CuePoint to save space.
But given Matroska's variable-length length fields, it either needs
to have an upper bound of the final size of the CuePoint before writing it
or the CuePoint h
It is no longer needed given that the function pointers of the child
muxer's AVOutputFormat are no longer called directly.
Signed-off-by: Andreas Rheinhardt
---
The difference to the previous version is that this version is not based
upon my patch [1], so that the patches to webm_chunk are not bl
Andreas Rheinhardt:
> If an AVFormatContext could be allocated, but white-/blacklists couldn't
> be copied, the AVFormatContext would leak as it was only accessible
> through a local variable that goes out of scope when one goes to fail.
>
> Furthermore, in case writing a header of a submuxer fail
On Fri, Mar 13, 2020 at 10:30:01PM +0800, numberwolf wrote:
> Signed-off-by: numberwolf
> ---
> libavcodec/mpegutils.c | 10 --
> 1 file changed, 8 insertions(+), 2 deletions(-)
breaks fate
--- ./tests/ref/fate/filter-codecview-mvs 2020-02-28 20:37:13.372518075
+0100
+++ tests/dat
Reviewed-by: Lou Logan
Signed-off-by: Steven Liu
---
generate-doc.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/generate-doc.sh b/generate-doc.sh
index d8d01d3..e316c05 100755
--- a/generate-doc.sh
+++ b/generate-doc.sh
@@ -34,7 +34,7 @@ export FA_ICONS=true
rm -rf bu
> From: ffmpeg-devel On Behalf Of
> Nicolas George
> Sent: Tuesday, February 18, 2020 03:02
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: add -autoscale
> to disable/enable the default scale
>
> Gyan Doshi (12020-0
On 13-03-2020 10:02 am, Gyan Doshi wrote:
On 13-03-2020 12:08 am, Lou Logan wrote:
Signed-off-by: Lou Logan
---
The options for this muxer have a separate section about fragmenting.
I tried to organize the missing options into the proper sections, but
I don't use fragmenting so I may have p
As per https://johnvansickle.com/ffmpeg/
---
src/download | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/download b/src/download
index 6292a7e..d9155ee 100644
--- a/src/download
+++ b/src/download
@@ -70,7 +70,7 @@
Linux Static Builds
57 matches
Mail list logo