2016-09-25 21:47 GMT+02:00 Clément Bœsch :
>> +{ "write_tmcd", "force or disable writing tmcd",
>> offsetof(MOVMuxContext, write_tmcd), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 1,
>> AV_OPT_FLAG_ENCODING_PARAM},
>
> AV_OPT_TYPE_BOOL
Applied with that change.
[...]
> Didn't test nor looked in dep
Hi!
As requested by Clément, no opinion here.
Please comment, Carl Eugen
From 233aaff6511c3b105a7ac2dad2d55d03455df153 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos
Date: Mon, 26 Sep 2016 08:52:36 +0200
Subject: [PATCH] lavf/movenc: Only write timecode track for mov by default.
---
Changelog
On Sep 26, 2016 09:29, "Carl Eugen Hoyos" wrote:
>
> 2016-09-26 1:52 GMT+02:00 Jan Ekström :
>
> If this fixes anything (I am not sure I understand: Are you changing a
> type which introduces a possible undefined behaviour, so you add an
> additional check at the same time?) it should be part of a
2016-09-26 1:52 GMT+02:00 Jan Ekström :
> * Renames the `width` and `height` entries to `stored_{width,height}`
> according to the specification's naming.
This should be a separate patch. (Although I wonder how useful it is:
Are you planning to add yourself as mxf maintainer? This would be
very
2016-09-25 23:55 GMT+02:00 Michael Niedermayer :
>> > probably ok
>> > it might be ultimately easier to reject demuxer output that has
>> > timestamps close to INT64 MAX/MIN instead of protecting all
>> > computations one by one though
>>
>> I don't disagree but I am probably unable to improve the
On Sun, Sep 25, 2016 at 02:43:47PM -0500, Rodger Combs wrote:
> ---
> libavformat/mpegtsenc.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
probably ok or rather ive no better idea either
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Re
On Tue, Aug 09, 2016 at 05:15:30PM +0200, Jean-Yves Simon wrote:
> ---
> libavcodec/hevc.c | 3 +++
> 1 file changed, 3 insertions(+)
applied
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
While the State exists there can be no freedom; when there is free
On Mon, 26 Sep 2016, Jan Ekström wrote:
* Updates the width/height sanity check to check all values against
INT_MAX.
* Correctly utilizes the stored width or height by default, and if
sampled or display values are available they are utilized.
Signed-off-by: Jan Ekström
---
libavformat/mxfde
* Renames the `width` and `height` entries to `stored_{width,height}`
according to the specification's naming.
* Switches the data type of widths and heights to uint32_t according
to specification, adds additional checks to make sure we don't
overflow INT_MAX as codecpar->{width,height} are s
* Updates the width/height sanity check to check all values against
INT_MAX.
* Correctly utilizes the stored width or height by default, and if
sampled or display values are available they are utilized.
Signed-off-by: Jan Ekström
---
libavformat/mxfdec.c | 30 +++---
On Sat, Sep 03, 2016 at 11:22:43AM +0200, Michael Niedermayer wrote:
> On Fri, Aug 05, 2016 at 11:24:35PM +0200, Carl Eugen Hoyos wrote:
> > Hi!
> >
> > 2016-08-04 16:01 GMT+02:00 James Almer :
> > >> So the advantage is that compilation gets measurably faster
> > >> with your patch?
> > >
> > > N
On Sun, Sep 25, 2016 at 09:04:30PM +0200, Carl Eugen Hoyos wrote:
> 2016-09-25 20:59 GMT+02:00 Michael Niedermayer :
> > On Sat, Sep 24, 2016 at 03:28:36PM +0200, Carl Eugen Hoyos wrote:
> >> Hi!
> >>
> >> Attached patch hopefully fixes ticket #5136.
> >>
> >> Please review, Carl Eugen
> >
> >> ut
Allowing larger timestamps makes it impossible to calculate basic things like
the
difference of 2 timestamps or their sum without checking each individual
computation for
overflow.
This should avoid a significant number of overflow checks
Fixes Ticket5136
Signed-off-by: Michael Niedermayer
---
> On Sep 25, 2016, at 1:49 PM, Michael Niedermayer
> wrote:
>
> On Sat, Sep 24, 2016 at 10:40:35PM +0200, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Not everybody is happy about a timecode track in mp4,
>> see ticket #5492.
>>
>> Please comment, Carl Eugen
>
>> doc/muxers.texi |3 +++
>>
On Sat, Sep 24, 2016 at 10:40:35PM +0200, Carl Eugen Hoyos wrote:
> Hi!
>
> Not everybody is happy about a timecode track in mp4,
> see ticket #5492.
>
> Please comment, Carl Eugen
> From 47a813709643106b0d0a2eceff822107c395d15c Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos
> Date: Sat, 24
---
libavformat/mpegtsenc.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index fd849e5..ac27b81 100644
--- a/libavformat/mpegtsenc.c
+++ b/libavformat/mpegtsenc.c
@@ -1790,11 +1790,15 @@ static int mpegts_check_bitstr
On 9/24/2016 9:29 PM, Josh de Kock wrote:
> On 24/09/2016 19:24, James Almer wrote:
>> [...]
>
> Both patches look good to me.
>
> --
> Josh
Pushed, thanks.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmp
2016-09-25 20:59 GMT+02:00 Michael Niedermayer :
> On Sat, Sep 24, 2016 at 03:28:36PM +0200, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Attached patch hopefully fixes ticket #5136.
>>
>> Please review, Carl Eugen
>
>> utils.c |4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>> 519f20990142
On Sat, Sep 24, 2016 at 03:28:36PM +0200, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch hopefully fixes ticket #5136.
>
> Please review, Carl Eugen
> utils.c |4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
> 519f209901429efaf9dffec381290a6a83573deb
> 0001-lavf-utils-Do-not-ov
On Sat, Sep 24, 2016 at 10:40:35PM +0200, Carl Eugen Hoyos wrote:
> Hi!
>
> Not everybody is happy about a timecode track in mp4,
> see ticket #5492.
>
> Please comment, Carl Eugen
> doc/muxers.texi |3 +++
> libavformat/movenc.c |6 --
> libavformat/movenc.h |1 +
> 3 fil
On Sun, Sep 25, 2016 at 06:36:15PM +0100, Josh de Kock wrote:
> On 25/09/2016 18:35, Clément Bœsch wrote:
> > ---
> > What else is missing?
> > ---
> > doc/libav-merge.txt | 10 ++
> > 1 file changed, 10 insertions(+)
> >
> > diff --git a/doc/libav-merge.txt b/doc/libav-merge.txt
> > inde
On 25/09/2016 18:35, Clément Bœsch wrote:
---
What else is missing?
---
doc/libav-merge.txt | 10 ++
1 file changed, 10 insertions(+)
diff --git a/doc/libav-merge.txt b/doc/libav-merge.txt
index 4fd863e..e200bdb 100644
--- a/doc/libav-merge.txt
+++ b/doc/libav-merge.txt
@@ -84,4 +84,14
---
What else is missing?
---
doc/libav-merge.txt | 10 ++
1 file changed, 10 insertions(+)
diff --git a/doc/libav-merge.txt b/doc/libav-merge.txt
index 4fd863e..e200bdb 100644
--- a/doc/libav-merge.txt
+++ b/doc/libav-merge.txt
@@ -84,4 +84,14 @@ whitespace differences) are passed into c
On date Saturday 2016-09-24 15:21:11 +0200, wm4 encoded:
> On Fri, 23 Sep 2016 19:46:16 +0200
> Stefano Sabatini wrote:
>
> > On date Friday 2016-09-23 09:34:19 +0200, wm4 encoded:
> > > On Thu, 22 Sep 2016 18:50:27 +0200
> > > Stefano Sabatini wrote:
> > [...]
> > > > Ping. I'd like to commit
On 24/09/2016 19:01, Josh de Kock wrote:
Signed-off-by: Josh de Kock
---
libavdevice/sdl2.c | 27 +--
1 file changed, 5 insertions(+), 22 deletions(-)
[...]
Applied.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
htt
On 25/09/2016 14:51, Josh de Kock wrote:
On 25/09/2016 14:50, Carl Eugen Hoyos wrote:
2016-09-25 15:48 GMT+02:00 Josh de Kock :
On 25/09/2016 14:43, Carl Eugen Hoyos wrote:
2016-09-25 15:40 GMT+02:00 Josh de Kock :
On 25/09/2016 14:38, Carl Eugen Hoyos wrote:
This is missing the importan
Le quartidi 4 vendémiaire, an CCXXV, Carl Eugen Hoyos a écrit :
> (Massive) memory consumption.
That would be if the consumption was necessary.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
f
2016-09-25 16:00 GMT+02:00 Nicolas George :
> Le quartidi 4 vendémiaire, an CCXXV, Carl Eugen Hoyos a écrit :
>> > And without this, ffmpeg keeps using more and more memory.
>> > So while the memory is eventually freed when the encoding is done
>> I believe this defines it as not being a leak.
>
>
2016-09-25 15:59 GMT+02:00 Timo Rothenpieler :
>>> And without this, ffmpeg keeps using more and more memory.
>>> So while the memory is eventually freed when the encoding is done
>>
>> I believe this defines it as not being a leak.
>
> What else would it be?
(Massive) memory consumption.
Carl E
Le quartidi 4 vendémiaire, an CCXXV, Carl Eugen Hoyos a écrit :
> > And without this, ffmpeg keeps using more and more memory.
> > So while the memory is eventually freed when the encoding is done
> I believe this defines it as not being a leak.
I believe you are wrong. It is not the same kind of
On 9/25/2016 3:55 PM, Carl Eugen Hoyos wrote:
> 2016-09-25 15:47 GMT+02:00 Timo Rothenpieler :
>> On 9/25/2016 3:44 PM, Carl Eugen Hoyos wrote:
>>> 2016-09-25 15:43 GMT+02:00 Timo Rothenpieler :
When the input frames contain side data, it will accumulate
endlessly in the coded frame, as a
2016-09-25 15:47 GMT+02:00 Timo Rothenpieler :
> On 9/25/2016 3:44 PM, Carl Eugen Hoyos wrote:
>> 2016-09-25 15:43 GMT+02:00 Timo Rothenpieler :
>>> When the input frames contain side data, it will accumulate
>>> endlessly in the coded frame, as av_frame_copy_props will
>>> append any new side data
On 25/09/2016 14:50, Carl Eugen Hoyos wrote:
2016-09-25 15:48 GMT+02:00 Josh de Kock :
On 25/09/2016 14:43, Carl Eugen Hoyos wrote:
2016-09-25 15:40 GMT+02:00 Josh de Kock :
On 25/09/2016 14:38, Carl Eugen Hoyos wrote:
This is missing the important change to
the help output.
These chang
2016-09-25 15:48 GMT+02:00 Josh de Kock :
> On 25/09/2016 14:43, Carl Eugen Hoyos wrote:
>>
>> 2016-09-25 15:40 GMT+02:00 Josh de Kock :
>>>
>>> On 25/09/2016 14:38, Carl Eugen Hoyos wrote:
>>
This is missing the important change to
the help output.
>>> These changes are only to not
On 25/09/2016 14:43, Carl Eugen Hoyos wrote:
2016-09-25 15:40 GMT+02:00 Josh de Kock :
On 25/09/2016 14:38, Carl Eugen Hoyos wrote:
This is missing the important change to
the help output.
These changes are only to not break current scripts,
users should use --disable-sdl2.
Why?
This mak
On 9/25/2016 3:44 PM, Carl Eugen Hoyos wrote:
> 2016-09-25 15:43 GMT+02:00 Timo Rothenpieler :
>> When the input frames contain side data, it will accumulate endlessly in
>> the coded frame, as av_frame_copy_props will append any new side data.
>
> But there is no leak, no?
> Is this related to ti
2016-09-25 15:43 GMT+02:00 Timo Rothenpieler :
> When the input frames contain side data, it will accumulate endlessly in
> the coded frame, as av_frame_copy_props will append any new side data.
But there is no leak, no?
Is this related to ticket #5799?
Carl Eugen
2016-09-25 15:40 GMT+02:00 Josh de Kock :
> On 25/09/2016 14:38, Carl Eugen Hoyos wrote:
>> This is missing the important change to
>> the help output.
>>
>
> These changes are only to not break current scripts,
> users should use --disable-sdl2.
Why?
This makes no sense, we don't use version
inf
When the input frames contain side data, it will accumulate endlessly in
the coded frame, as av_frame_copy_props will append any new side data.
---
libavcodec/mpegvideo_enc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
index 87d7954..
On 25/09/2016 14:38, Carl Eugen Hoyos wrote:
2016-09-25 15:29 GMT+02:00 Josh de Kock :
This commit also adds an sdl alias for the configure script.
Signed-off-by: Josh de Kock
---
configure | 2 ++
This is missing the important change to
the help output.
These changes are only to
2016-09-25 15:29 GMT+02:00 Josh de Kock :
> This commit also adds an sdl alias for the configure script.
>
> Signed-off-by: Josh de Kock
> ---
> configure | 2 ++
This is missing the important change to
the help output.
Carl Eugen
___
ffmpeg-d
This commit also adds an sdl alias for the configure script.
Signed-off-by: Josh de Kock
---
configure | 2 ++
libavdevice/sdl2.c | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index f593191..899057d 100755
--- a/configure
+++ b/configure
@@
2016-09-25 14:59 GMT+02:00 Josh de Kock :
>
>
> On 25/09/2016 13:57, Carl Eugen Hoyos wrote:
>>
>> 2016-09-25 14:51 GMT+02:00 Josh de Kock :
>>>
>>> On 25/09/2016 13:45, Carl Eugen Hoyos wrote:
2016-09-25 14:28 GMT+02:00 Josh de Kock :
>
>
> Signed-off-by: Josh de Kock
>
On 25/09/2016 13:57, Carl Eugen Hoyos wrote:
2016-09-25 14:51 GMT+02:00 Josh de Kock :
On 25/09/2016 13:45, Carl Eugen Hoyos wrote:
2016-09-25 14:28 GMT+02:00 Josh de Kock :
Signed-off-by: Josh de Kock
---
The --enable-sdl2 option is not being renamed because it's a fairly
big change a
2016-09-25 14:51 GMT+02:00 Josh de Kock :
> On 25/09/2016 13:45, Carl Eugen Hoyos wrote:
>>
>> 2016-09-25 14:28 GMT+02:00 Josh de Kock :
>>>
>>> Signed-off-by: Josh de Kock
>>> ---
>>>
>>> The --enable-sdl2 option is not being renamed because it's a fairly
>>> big change and sdl2 is incompatible
On 25/09/2016 13:45, Carl Eugen Hoyos wrote:
2016-09-25 14:28 GMT+02:00 Josh de Kock :
Signed-off-by: Josh de Kock
---
The --enable-sdl2 option is not being renamed because it's a fairly
big change and sdl2 is incompatible with sdl.
How is (old) sdl in FFmpeg incompatible with the new sdl2
2016-09-25 14:28 GMT+02:00 Josh de Kock :
> Signed-off-by: Josh de Kock
> ---
>
> The --enable-sdl2 option is not being renamed because it's a fairly
> big change and sdl2 is incompatible with sdl.
How is (old) sdl in FFmpeg incompatible with the new sdl2 in FFmpeg?
Please rename it.
Patch ok
Hi folks,
I am Pallavi Kumari, a recent graduate in Computer Science from IIIT
Hyderabad, India. My areas of interest includes Information Retrieval and
Extraction, Machine learning, Deep learning , algorithms. I have sound
knowledge C, C++, Python and am familiar with various open source
technolo
Hi!
Attached patch fixes a long-time regression.
Please comment, Carl Eugen
From 2b991a66c4bf14774210e2c7bfb0bb5f0e4f1b8f Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos
Date: Sun, 25 Sep 2016 14:27:43 +0200
Subject: [PATCH] lavc/8bps: Fix 32bit output of 24bit video.
Regression since / partial
Signed-off-by: Josh de Kock
---
The --enable-sdl2 option is not being renamed because it's a fairly
big change and sdl2 is incompatible with sdl.
libavdevice/sdl2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavdevice/sdl2.c b/libavdevice/sdl2.c
index 217ccc0..5d9e9
On Sun, Sep 25, 2016 at 12:55:24PM +0100, Josh de Kock wrote:
> Signed-off-by: Josh de Kock
> ---
> configure | 2 +-
> libavdevice/sdl2.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index f593191..4cfd840 100755
> --- a/configure
On 25/09/2016 12:55, Josh de Kock wrote:
Signed-off-by: Josh de Kock
---
configure | 2 +-
libavdevice/sdl2.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index f593191..4cfd840 100755
--- a/configure
+++ b/configure
@@ -291,7 +291,7 @@ E
Signed-off-by: Josh de Kock
---
configure | 2 +-
libavdevice/sdl2.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index f593191..4cfd840 100755
--- a/configure
+++ b/configure
@@ -291,7 +291,7 @@ External library support:
[...]
Patch applied.
Thank you, Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
---
libavcodec/aarch64/Makefile | 2 +-
libavcodec/aarch64/h264cmc_neon.S | 2 +-
libavcodec/arm/h264cmc_neon.S | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavcodec/aarch64/Makefile b/libavcodec/aarch64/Makefile
index 36d9d34..c3df887 100644
--- a/libavcodec
2016-09-25 5:59 GMT+02:00 James Almer :
> As Lukas suggested in another email, it would be a good idea
> to keep using "sdl" instead of "sdl2" on things like the outdev
> name and the configure option (--enable/disable-sdl). So
> basically, anything an user could interact with.
If this gets chang
2016-09-25 8:20 GMT+02:00 Xiaolei Yu :
> ---
> libavcodec/aarch64/Makefile | 2 +-
> libavcodec/aarch64/h264cmc_neon.S | 2 +-
> libavcodec/arm/h264cmc_neon.S | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/aarch64/Makefile b/libavcodec/aarch64/Make
57 matches
Mail list logo
