On Wed, Nov 18, 2015 at 05:04:06PM +0200, Maksym Veremeyenko wrote:
> Hi,
>
> FLV remuxed from MPEG-TS stream does not contains *AAC sequence
> header* as result no audio with some RTMP servers like *crtmpserver*
> and YouTube.
>
> I am not a first who faced with such problem:
>
On Tue, Nov 17, 2015 at 03:06:27PM -0800, Zach Swena wrote:
> Hi Pavel,
>
> I can confirm that there is a problem with the UDP packet engine in
> FFmpeg. FFmpeg has excessive jitter in it's UDP streaming output to the
> point where hardware decoders can't handle it. My solution was to buffer
>
On Wed, Nov 18, 2015 at 11:13:37AM +, Vicente Olivert Riera wrote:
> Signed-off-by: Vicente Olivert Riera
> ---
> Changes v1 -> v2:
> - Remove a block of code for mipsdspr2 which wasn't intended to be in
>this patch and it was causing a syntax error in the
Dear Michael Niedermayer,
On 11/18/2015 04:59 PM, Michael Niedermayer wrote:
> On Wed, Nov 18, 2015 at 11:13:38AM +, Vicente Olivert Riera wrote:
>> Having a configure option with the same name as a MIPS ISA is confusing,
>> so better to remove it. This option was being used to add some
>>
On Wed, Nov 18, 2015 at 11:13:38AM +, Vicente Olivert Riera wrote:
> Having a configure option with the same name as a MIPS ISA is confusing,
> so better to remove it. This option was being used to add some
> optimizations to a specific core (p5600). We will add the optimizations
> just when
> i suggest you read the mpeg specs, they detail when things should be
> sent down to each byte IIRC
> also IIRC its not that complicated, more like timestamp + bytepos/rate
Only for strict CBR streams which FFmpeg cannot know a priori. That's
why the only correct way is to index the stream as
On Tue, Nov 17, 2015 at 04:00:30PM +, Derek Buitenhuis wrote:
> This way, it never starts with 0xFFF0, and never trips the
> ADTS "Detection" code in movenc.c.
>
> Signed-off-by: Derek Buitenhuis
> ---
> libavformat/movenc-test.c | 4 ++--
>
On Tue, Nov 17, 2015 at 07:27:55AM -0800, Michael Bradshaw wrote:
> Hi,
>
> Attached patch adds support for OpenJPEG 2.0/2.1. Please review.
>
> Thanks,
>
> Michael Bradshaw
> configure |5
> libavcodec/libopenjpegdec.c | 173
>
On Wed, Nov 18, 2015 at 6:21 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 11/17/2015 3:27 PM, Michael Bradshaw wrote:
> > Attached patch adds support for OpenJPEG 2.0/2.1. Please review.
>
> This seems like quite a large change. Does it perhaps warrant its own
> file/decoder
On 11/18/2015 3:15 PM, Michael Niedermayer wrote:
> this looks a bit strange
> doesnt this write the PTS into bytes which are never used or read?
Uh, yep. Need to update size too.
- Derek
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
This way, it never starts with 0xFFF0, and never trips the
ADTS "Detection" code in movenc.c.
Signed-off-by: Derek Buitenhuis
---
libavformat/movenc-test.c | 6 +++---
tests/ref/fate/movenc | 40
2 files changed, 23
On Wed, Nov 18, 2015 at 03:51:16PM +, Derek Buitenhuis wrote:
> This way, it never starts with 0xFFF0, and never trips the
> ADTS "Detection" code in movenc.c.
>
> Signed-off-by: Derek Buitenhuis
> ---
> libavformat/movenc-test.c | 6 +++---
>
Yea, how it is supposed to happen isn't complicated. To be of any help, I
have to wrap my mind around how FFmpeg currently times the output stream.
That mechanism currently does not produce as stable of a transmission rate
as multiplexer and decoder hardware requires. Since there is a bounty
From: Rick Kern
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 308 ++---
1 file changed, 154 insertions(+), 154 deletions(-)
diff --git a/libavcodec/vtenc.c b/libavcodec/vtenc.c
index f497c38..b363492
From: Rick Kern
Use correct dictionary key/value to enable hardware encoding on OSX.
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavcodec/vtenc.c b/libavcodec/vtenc.c
index 06c5360..f497c38 100644
From: Rick Kern
Moved all declarations to the top of their block.
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 409 ++---
1 file changed, 260 insertions(+), 149 deletions(-)
diff --git
From: Rick Kern
Updated patch with requested changes.
Rick Kern (10):
Added VideoToolbox H.264 encoder.
vtenc: Enable hardware encoding on OSX.
vtenc: Removed trailing whitespace.
vtenc: Fixed iOS compilation.
vtenc: Check for min iOS/OSX versions.
Better
From: Rick Kern
Removed dictionary keys that aren't supported on iOS builds.
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavcodec/vtenc.c b/libavcodec/vtenc.c
index b363492..7132411 100644
---
From: Rick Kern
configure was checking for the SDK version being used to compile. Now it
compares against the minimum deployment target.
Signed-off-by: Rick Kern
---
configure | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
From: Rick Kern
The CMSampleBufferRef passed into the encoder can be NULL. Added check.
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/vtenc.c b/libavcodec/vtenc.c
index
From: Rick Kern
Some memory allocations were unchecked.
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/libavcodec/vtenc.c b/libavcodec/vtenc.c
index f0d8289..583244c 100644
---
From: Rick Kern
Condition was not being verified after conditional variable unblocked.
Signed-off-by: Rick Kern
---
libavcodec/vtenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/vtenc.c b/libavcodec/vtenc.c
index
From: Rick Kern
---
configure | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/configure b/configure
index 718f285..f99fbd9 100755
--- a/configure
+++ b/configure
@@ -5474,17 +5474,7 @@ enabled openssl && { check_lib openssl/ssl.h
From: Rick Kern
Enable with configure --enable-vtenc and encode using -codec:v vtenc_h264.
Signed-off-by: Rick Kern
---
MAINTAINERS|1 +
configure | 14 +
libavcodec/Makefile|1 +
libavcodec/allcodecs.c |1 +
> Only for strict CBR streams which FFmpeg cannot know a priori. That's
> why the only correct way is to index the stream as it's received and
> play out at a smooth rate like multicat does.
I am not sure what you are referencing with regards to multicat. Is this
an output option? Smooth output
On 18.11.2015 19:37, Michael Niedermayer wrote:
[...]
the first field of the context must be a AVClass
updated patch attached
--
Maksym Veremeyenko
From 665751883fd515c753ada7bbf4d24c89a24f3d30 Mon Sep 17 00:00:00 2001
From: Maksym Veremeyenko
Date: Wed, 18 Nov 2015 23:44:08
On Tue, Nov 17, 2015 at 04:54:10PM -0500, Ganesh Ajjanagadde wrote:
> isnan and isinf are actually macros as per the standard. In particular,
> the existing implementation has incorrect signature. Furthermore, this
> results in undefined behavior for e.g double values outside float range
> as per
On Wed, Nov 18, 2015 at 2:58 PM, Michael Niedermayer wrote:
> On Tue, Nov 17, 2015 at 04:54:10PM -0500, Ganesh Ajjanagadde wrote:
>> isnan and isinf are actually macros as per the standard. In particular,
>> the existing implementation has incorrect signature. Furthermore, this
Are you referring to this seperate applciation?
https://github.com/mmalecki/multicat/blob/master/trunk/README
While using that makes sense for Pavel's application, why should FFmpeg
produce a problematic UDP stream in the first place? For applications like
I need that involve encoding the
Having a configure option with the same name as a MIPS ISA is confusing,
so better to remove it. This option was being used to add some
optimizations to a specific core (p5600). We will add the optimizations
just when the p5600 core has been detected, in a later patch.
Signed-off-by: Vicente
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Remove a block of code for mipsdspr2 which wasn't intended to be in
this patch and it was causing a syntax error in the configure script.
Highlighted by Michael Niedermayer:
On Tue, 17 Nov 2015 17:39:31 -0500
Ganesh Ajjanagadde wrote:
> ffio_ensure_seekback can fail due to e.g ENOMEM. This return value is
> checked here and a diagnostic is logged.
>
> All usage of ffio_ensure_seekback in the codebase now has the return value
> checked.
>
>
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Nothing.
configure |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/configure b/configure
index d14b02b..ca5c784 100755
--- a/configure
+++ b/configure
@@ -4098,6 +4098,7 @@ elif
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Nothing.
configure | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/configure b/configure
index ca5c784..761c4cb 100755
--- a/configure
+++ b/configure
@@ -1623,6 +1623,7
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Fix a grammar mistake in the commit log. Change "do has" by "does
have". Thanks to Moritz Barsnick:
https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2015-November/183437.html
configure |1 -
1 files
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Nothing.
configure | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/configure b/configure
index 761c4cb..b31567e 100755
--- a/configure
+++ b/configure
@@ -1624,6 +1624,7
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Add a block of code for mipsdspr2 which was previously placed in the
first patch of this series causing a syntax error in the configure
script.
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Nothing.
configure |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/configure b/configure
index 8598fe1..d0685f3 100755
--- a/configure
+++ b/configure
@@ -4101,9 +4101,10 @@
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Nothing.
configure |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index 83b2a64..ea129be 100755
--- a/configure
+++ b/configure
@@ -4094,8 +4094,9 @@ elif
Understanding the mips32r6 and mips64r6 ISAs in the configure script is
not enough. In order to have full support for MIPS R6 in FFmpeg we need
to be able to build it, and for that we need to make sure we don't use
incompatible assembler code which makes the build fail. Ifdefing the
offending code
There is no point to have a "generic" core when we can catch all
unsupported cores with the "*" option, so remove it.
Signed-off-by: Vicente Olivert Riera
---
Changes v1 -> v2:
- Nothing.
configure |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff
Dear Michael Niedermayer,
On 11/17/2015 09:17 PM, Michael Niedermayer wrote:
> On Mon, Nov 16, 2015 at 02:57:07PM +, Vicente Olivert Riera wrote:
> [...]
>> index 9a736ce..0461a97 100755
>> --- a/configure
>> +++ b/configure
>> @@ -380,7 +380,7 @@ Optimization options (experts only):
>>
Second patchset addresses feedback after discussion with Martin. It is now
accounting for the fact that avg_frame_rate may not always be set.
On Wed, Nov 18, 2015 at 1:13 AM, Bryan Huh wrote:
> DASH manifest should have framerate specified as an attribute in the
> AdaptationSet
DASH manifest should have framerate specified as an attribute in the
AdaptationSet element and Representation elements. Though ISO/IEC
23009-1:2014 doesn't seem to define frameRate as a required attribute,
it is at least optional, and DASH-IF IOP 3.0 seems to require it. See
section 3.2.4 of
On Tue, Nov 17, 2015 at 07:04:05PM +0100, Paul B Mahol wrote:
> On 11/17/15, Michael Niedermayer wrote:
> > From: Michael Niedermayer
> >
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavformat/genh.c |2 ++
> > 1
From: Michael Niedermayer
Signed-off-by: Michael Niedermayer
---
libavutil/mem.c |4
libavutil/mem.h | 15 +++
2 files changed, 19 insertions(+)
diff --git a/libavutil/mem.c b/libavutil/mem.c
index 323b183..8dfaad8 100644
On Wed, Nov 18, 2015 at 01:13:14AM -0800, Bryan Huh wrote:
> DASH manifest should have framerate specified as an attribute in the
> AdaptationSet element and Representation elements. Though ISO/IEC
> 23009-1:2014 doesn't seem to define frameRate as a required attribute,
> it is at least optional,
On Wed, 18 Nov 2015 15:09:48 +0100
Michael Niedermayer wrote:
> On Tue, Nov 17, 2015 at 03:19:29PM +0100, wm4 wrote:
> > If videotoolbox_common_end_frame failed, then the AVFrame was returned
> > to the API user with the dummy buffer (in AVFrame.buf[0]) still set, and
> >
From: Michael Niedermayer
In case of bitstream errors the deblock filter and slices can access
uninitialized
top_borders from previous slices which did not fill them as they stoped halfway
due
to error or where entirely missing
Found-by: Tyson Smith
Signed-off-by:
On Wed, Nov 18, 2015 at 7:31 AM, wm4 wrote:
> On Tue, 17 Nov 2015 17:39:31 -0500
> Ganesh Ajjanagadde wrote:
>
>> ffio_ensure_seekback can fail due to e.g ENOMEM. This return value is
>> checked here and a diagnostic is logged.
>>
>> All usage of
Hi,
FLV remuxed from MPEG-TS stream does not contains *AAC sequence header*
as result no audio with some RTMP servers like *crtmpserver* and YouTube.
I am not a first who faced with such problem:
http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-May/157791.html
On 11/17/2015 4:00 PM, Derek Buitenhuis wrote:
> This way, it never starts with 0xFFF0, and never trips the
> ADTS "Detection" code in movenc.c.
>
> Signed-off-by: Derek Buitenhuis
> ---
> libavformat/movenc-test.c | 4 ++--
> tests/ref/fate/movenc | 40
I previously had an option for gcrypt too but it seemed to not work (FFmpeg
crashed, iirc), so I just gave up on it.
On 18 November 2015 at 05:41, Matt Oliver wrote:
> On 15 November 2015 at 01:09, Ricardo Constantino
> wrote:
>
>> Enables RTMP(T)E
On Tue, Nov 17, 2015 at 05:20:08PM +0100, Stefano Sabatini wrote:
> On date Tuesday 2015-11-17 17:12:46 +0100, wm4 encoded:
> > On Tue, 17 Nov 2015 17:00:29 +0100
> > Stefano Sabatini wrote:
> [...]
> > > No, just in case ret == AVERROR(EAGAIN), in all other failure cases it
>
On 11/17/2015 3:27 PM, Michael Bradshaw wrote:
> Attached patch adds support for OpenJPEG 2.0/2.1. Please review.
This seems like quite a large change. Does it perhaps warrant its own
file/decoder name?
- Derek
___
ffmpeg-devel mailing list
On 16.11.2015 02:59, Michael Niedermayer wrote:
> On Fri, Nov 13, 2015 at 10:32:47PM +0100, Andreas Cadhalpun wrote:
>> Because these samples aren't tested with the aac_fixed decoder:
>> * aac/ct_faac-adts.aac is only used to test the aac demuxer.
>> * aac/al07_96.mp4 is for some reason only
On Thu, Nov 19, 2015 at 12:31:17AM +0100, Andreas Cadhalpun wrote:
> On 16.11.2015 13:46, Michael Niedermayer wrote:
> > On Fri, Nov 13, 2015 at 11:19:44PM +0100, Andreas Cadhalpun wrote:
> >> Well, unfortunately just rejecting 0 in sbr_dequant is no solution,
> >> because, as you noticed, that
On Thu, Nov 19, 2015 at 02:36:19AM +0100, Michael Niedermayer wrote:
> On Thu, Nov 19, 2015 at 01:25:53AM +0100, Andreas Cadhalpun wrote:
> > On 18.11.2015 15:07, Michael Niedermayer wrote:
> > > On Tue, Nov 17, 2015 at 05:20:08PM +0100, Stefano Sabatini wrote:
> > >> On date Tuesday 2015-11-17
On Thu, Nov 19, 2015 at 01:25:53AM +0100, Andreas Cadhalpun wrote:
> On 18.11.2015 15:07, Michael Niedermayer wrote:
> > On Tue, Nov 17, 2015 at 05:20:08PM +0100, Stefano Sabatini wrote:
> >> On date Tuesday 2015-11-17 17:12:46 +0100, wm4 encoded:
> >>> On Tue, 17 Nov 2015 17:00:29 +0100
> >>>
Hi,
On Wed, Nov 18, 2015 at 9:28 PM, Zach Swena wrote:
> Are you referring to this seperate applciation?
>
> https://github.com/mmalecki/multicat/blob/master/trunk/README
>
Probably what was meant was
http://www.videolan.org/projects/multicat.html , but that might be
---
libavformat/tee.c | 43 ++-
1 file changed, 2 insertions(+), 41 deletions(-)
diff --git a/libavformat/tee.c b/libavformat/tee.c
index af52a49..c527854 100644
--- a/libavformat/tee.c
+++ b/libavformat/tee.c
@@ -405,45 +405,6 @@ fail:
return ret;
}
---
ffmpeg.c | 46 --
ffmpeg.h | 1 -
ffmpeg_opt.c | 6 +-
3 files changed, 9 insertions(+), 44 deletions(-)
diff --git a/ffmpeg.c b/ffmpeg.c
index c4e9280..cae680d 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -683,47 +683,10 @@ static void
---
libavformat/avformat.h | 8
libavformat/utils.c| 49 +
2 files changed, 57 insertions(+)
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index 5283085..96d3cfd 100644
--- a/libavformat/avformat.h
+++
If the input contains too many too large values, the imdct can overflow.
Even if it didn't, the output would be larger than the valid range of 29
bits.
Note that this is a very delicate limit: Allowing values up to 1<<25
does not prevent input larger than 1<<29 from arriving at
sbr_sum_square,
On 16.11.2015 15:39, Nedeljko Babic wrote:
>>> On 11.11.2015 13:46, Michael Niedermayer wrote:
>> Comments fro AAC and SBR experts very welcome!
>
> This code was developed a while ago, but based on informations that I have
> this part of code was analysed regarding possibility of overflow and
>
On 18 November 2015 at 19:28, Zach Swena wrote:
> Are you referring to this seperate applciation?
>
> https://github.com/mmalecki/multicat/blob/master/trunk/README
>
> While using that makes sense for Pavel's application, why should FFmpeg
> produce a problematic UDP
On 16.11.2015 09:02, tim nicholson wrote:
> On 15/11/15 18:07, Andreas Cadhalpun wrote:
>> Previously only the edit_rate of material_track was checked.
>> If it's negative, it causes assertion failures in av_rescale_rnd.
>>
>> Signed-off-by: Andreas Cadhalpun
>>
From: Michael Niedermayer
It appears rets_count is redundant
Signed-off-by: Michael Niedermayer
---
libavcodec/pthread_slice.c |5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/libavcodec/pthread_slice.c
On Wed, Nov 18, 2015 at 7:01 PM, Andreas Cadhalpun
wrote:
> On 16.11.2015 15:39, Nedeljko Babic wrote:
On 11.11.2015 13:46, Michael Niedermayer wrote:
>>> Comments fro AAC and SBR experts very welcome!
>>
>> This code was developed a while ago, but based on
On 18.11.2015 15:07, Michael Niedermayer wrote:
> On Tue, Nov 17, 2015 at 05:20:08PM +0100, Stefano Sabatini wrote:
>> On date Tuesday 2015-11-17 17:12:46 +0100, wm4 encoded:
>>> On Tue, 17 Nov 2015 17:00:29 +0100
>>> Stefano Sabatini wrote:
>> [...]
No, just in case ret
> A proper ts remuxer is quite a different beast to what FFmpeg is
> designed to do. As much as people try it can't be hacked into the
> FFmpeg paradigm which is largely based around converting static files
> of fixed codecs/resolutions in a nonrealtime environment from A to B.
Why is there a udp
---
libavformat/avformat.h | 9 +
libavformat/utils.c| 20
2 files changed, 29 insertions(+)
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index 6e9ffe2..cedcb8f 100644
--- a/libavformat/avformat.h
+++ b/libavformat/avformat.h
@@ -2776,6 +2776,15
---
libavformat/matroskaenc.c | 43 +++
1 file changed, 35 insertions(+), 8 deletions(-)
diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
index 10cf0a0..41aa8e0 100644
--- a/libavformat/matroskaenc.c
+++ b/libavformat/matroskaenc.c
@@
On 16.11.2015 13:46, Michael Niedermayer wrote:
> On Fri, Nov 13, 2015 at 11:19:44PM +0100, Andreas Cadhalpun wrote:
>> Well, unfortunately just rejecting 0 in sbr_dequant is no solution,
>> because, as you noticed, that only happens via underflow.
>
> a value that has underflowed should be 0, so
This solves the problem discussed in
https://ffmpeg.org/pipermail/ffmpeg-devel/2015-September/179238.html
by allowing AVCodec::write_header to be delayed until after packets have been
run through required bitstream filters in order to generate global extradata.
It also provides a mechanism by
> Because FFmpeg again cannot know a priori whether the input signal is
> fully VBV compliant for use within MPEGTS.
> An RTMP feed is an example of a feed which certainly isn't and so
> FFmpeg has to make some numbers up in order to make a valid TS.
> Such guessing might work in the file world
On Wed, Nov 18, 2015 at 8:00 AM, Michael Niedermayer wrote:
> On Tue, Nov 17, 2015 at 07:27:55AM -0800, Michael Bradshaw wrote:
> > Hi,
> >
> > Attached patch adds support for OpenJPEG 2.0/2.1. Please review.
> >
> > Thanks,
> >
> > Michael Bradshaw
>
> > configure
> Of course if the user copies a video stream that isn't complaint there will
> be problems if the decoder requires that. The issue here is two fold.
> First, the current udp transmission model can cause problems with
> networking equipment because of it's bursty nature. Second, FFmpeg can
>
On Wed, Nov 18, 2015 at 02:41:05PM +0100, Michael Niedermayer wrote:
> From: Michael Niedermayer
>
> In case of bitstream errors the deblock filter and slices can access
> uninitialized
> top_borders from previous slices which did not fill them as they stoped
> halfway
79 matches
Mail list logo