Re: [FFmpeg-devel] [PATCH] lavf/mpegtsenc: fix autobsf when the first NAL is 0x1 bytes

2016-09-25 Thread Michael Niedermayer
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: [FFmpeg-devel] [PATCH] Add YUV420P10 to vaapi hardware decode - permits hardware decoding of HEVC Main 10 on AMD RX 480

2016-09-25 Thread Michael Niedermayer
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

Re: [FFmpeg-devel] [PATCH 2/2] lavf/mxfdec: begin utilizing the newly parsed widths and heights

2016-09-25 Thread Marton Balint
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

[FFmpeg-devel] [PATCH 1/2] lavf/mxfdec: add support for all of the picture size and offset fields

2016-09-25 Thread Jan Ekström
* 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

[FFmpeg-devel] [PATCH 2/2] lavf/mxfdec: begin utilizing the newly parsed widths and heights

2016-09-25 Thread Jan Ekström
* 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

Re: [FFmpeg-devel] [PATCH] avcodec: don't include vdpau_compat.h when vdpau is not enabled

2016-09-25 Thread Michael Niedermayer
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

Re: [FFmpeg-devel] [PATCH]lavf/utils: Do not overflow in update_initial_timestamps().

2016-09-25 Thread Michael Niedermayer
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

[FFmpeg-devel] [PATCH] avformat/utils: Discard huge timestamps which would cause overflows if used in basic computations

2016-09-25 Thread Michael Niedermayer
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

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Allow to disable writing the timecode track

2016-09-25 Thread Dave Rice
> 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 > >>

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Allow to disable writing the timecode track

2016-09-25 Thread Clément Bœsch
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

[FFmpeg-devel] [PATCH] lavf/mpegtsenc: fix autobsf when the first NAL is 0x1 bytes

2016-09-25 Thread Rodger Combs
--- 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

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: fix memleak of bitstream filter context on failure

2016-09-25 Thread James Almer
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

Re: [FFmpeg-devel] [PATCH]lavf/utils: Do not overflow in update_initial_timestamps().

2016-09-25 Thread Carl Eugen Hoyos
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

Re: [FFmpeg-devel] [PATCH]lavf/utils: Do not overflow in update_initial_timestamps().

2016-09-25 Thread 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(-) > 519f209901429efaf9dffec381290a6a83573deb >

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Allow to disable writing the timecode track

2016-09-25 Thread Michael Niedermayer
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

Re: [FFmpeg-devel] [PATCH] doc/libav-merge: complete TODO section

2016-09-25 Thread Clément Bœsch
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 > >

Re: [FFmpeg-devel] [PATCH] doc/libav-merge: complete TODO section

2016-09-25 Thread Josh de Kock
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

[FFmpeg-devel] [PATCH] doc/libav-merge: complete TODO section

2016-09-25 Thread Clément Bœsch
--- 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

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-09-25 Thread Stefano Sabatini
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: >

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: remove unused code

2016-09-25 Thread Josh de Kock
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

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Josh de Kock
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

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Nicolas George
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

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Carl Eugen Hoyos
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

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Carl Eugen Hoyos
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)

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread 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. I believe you are wrong. It is not the same kind of

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Timo Rothenpieler
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

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Carl Eugen Hoyos
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

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Josh de Kock
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

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Carl Eugen Hoyos
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.

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread 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 break current scripts, users should use

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread 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. > > But there is no leak, no?

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread Carl Eugen Hoyos
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

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Carl Eugen Hoyos
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

[FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: fix memory leak

2016-09-25 Thread 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. --- libavcodec/mpegvideo_enc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Josh de Kock
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

Re: [FFmpeg-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Carl Eugen Hoyos
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-devel] [PATCH v3] lavd/sdl2: add sdl alias

2016-09-25 Thread Josh de Kock
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

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: add sdl alias

2016-09-25 Thread Carl Eugen Hoyos
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

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: add sdl alias

2016-09-25 Thread 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

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: add sdl alias

2016-09-25 Thread Carl Eugen Hoyos
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

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: add sdl alias

2016-09-25 Thread 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 with sdl. How is (old) sdl in

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: add sdl alias

2016-09-25 Thread Carl Eugen Hoyos
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-devel] [OPW] OPW Project Proposal

2016-09-25 Thread Pallavi Kumari
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

[FFmpeg-devel] [PATCH]lavc/8bps: Fix 32bit output of 24bit video

2016-09-25 Thread Carl Eugen Hoyos
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.

[FFmpeg-devel] [PATCH] lavd/sdl2: add sdl alias

2016-09-25 Thread 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. libavdevice/sdl2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavdevice/sdl2.c b/libavdevice/sdl2.c

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: rename user-facing names to sdl

2016-09-25 Thread Clément Bœsch
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

Re: [FFmpeg-devel] [PATCH] lavd/sdl2: rename user-facing names to sdl

2016-09-25 Thread Josh de Kock
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 @@

[FFmpeg-devel] [PATCH] lavd/sdl2: rename user-facing names to sdl

2016-09-25 Thread Josh de Kock
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:

Re: [FFmpeg-devel] [PATCH v2] avcodec: fix vc1dsp dependencies

2016-09-25 Thread Carl Eugen Hoyos
[...] Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH v2] avcodec: fix vc1dsp dependencies

2016-09-25 Thread 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/Makefile index 36d9d34..c3df887 100644 ---

Re: [FFmpeg-devel] [PATCH 0/5] Add SDL2 support & drop SDL1 support

2016-09-25 Thread Carl Eugen Hoyos
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.

[FFmpeg-devel] avcodec: fix vc1dsp dependencies

2016-09-25 Thread 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/Makefile index 36d9d34..c3df887 100644 ---