Re: [FFmpeg-devel] [PATCH v5 4/4 v2] lavf/utils: Normalize AVPacket.data to native endian in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 08:29 AM, Mats Peterson wrote: Due to Morton Balint's recent addition of ff_standardize_creation_time(), the previous patch would fail. Please add this one before any more changes occur. In my book, using native uint32_t when dealing with the palette on muxing is clearly the

[FFmpeg-devel] [PATCH v5 4/4 v2] lavf/utils: Normalize AVPacket.data to native endian in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
Due to Morton Balint's recent addition of ff_standardize_creation_time(), the previous patch would fail. Please add this one before any more changes occur. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 08b75292ed7ede8de32bdecf5ec72fedc18b1aa9 Mon Sep 17 00:00:00 2001 From: Mats

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 07:22 AM, Mats Peterson wrote: On 03/03/2016 07:15 AM, Mats Peterson wrote: On 03/03/2016 06:52 AM, Mats Peterson wrote: On 03/03/2016 06:39 AM, Mats Peterson wrote: On 03/03/2016 06:33 AM, Mats Peterson wrote: On 03/03/2016 03:05 AM, Michael Niedermayer wrote: with this API

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 07:15 AM, Mats Peterson wrote: On 03/03/2016 06:52 AM, Mats Peterson wrote: On 03/03/2016 06:39 AM, Mats Peterson wrote: On 03/03/2016 06:33 AM, Mats Peterson wrote: On 03/03/2016 03:05 AM, Michael Niedermayer wrote: with this API each muxer must convert the palette from native

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 06:52 AM, Mats Peterson wrote: On 03/03/2016 06:39 AM, Mats Peterson wrote: On 03/03/2016 06:33 AM, Mats Peterson wrote: On 03/03/2016 03:05 AM, Michael Niedermayer wrote: with this API each muxer must convert the palette from native endian to whatever the format requires feels

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 06:39 AM, Mats Peterson wrote: On 03/03/2016 06:33 AM, Mats Peterson wrote: On 03/03/2016 03:05 AM, Michael Niedermayer wrote: with this API each muxer must convert the palette from native endian to whatever the format requires feels a bit odd to extact the palette from

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 06:33 AM, Mats Peterson wrote: On 03/03/2016 03:05 AM, Michael Niedermayer wrote: with this API each muxer must convert the palette from native endian to whatever the format requires feels a bit odd to extact the palette from AVPacket.data where its stored as the container needs

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 03:05 AM, Michael Niedermayer wrote: with this API each muxer must convert the palette from native endian to whatever the format requires feels a bit odd to extact the palette from AVPacket.data where its stored as the container needs (for some containers) and then convert that to

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 03:05 AM, Michael Niedermayer wrote: On Thu, Mar 03, 2016 at 12:36:21AM +0100, Mats Peterson wrote: -- Mats Peterson http://matsp888.no-ip.org/~mats/ internal.h |7 +-- utils.c| 23 --- 2 files changed, 21 insertions(+), 9 deletions(-)

Re: [FFmpeg-devel] [PATCH] Add test foravpriv_get_trc_function_from_trc function

2016-03-02 Thread NagaChaitanya Vellanki
Passes on my mac ~/Documents/projects/FFmpeg/libavutil (master ✘)✭ ᐅ file color_utils-test color_utils-test: Mach-O executable i386 ~/Documents/projects/FFmpeg/libavutil (master ✘)✭ ᐅ cd .. ~/Documents/projects/FFmpeg (master ✘)✭ ᐅ make fate-color_utils TESTcolor_utils Will test this on my

Re: [FFmpeg-devel] [PATCH] videotoolboxenc: Fix compile failure on OS X 10.10

2016-03-02 Thread Mark Harris
> --- > libavcodec/videotoolboxenc.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c > index bbecb24..342c83c 100644 > --- a/libavcodec/videotoolboxenc.c > +++ b/libavcodec/videotoolboxenc.c > @@ -32,6 +32,14 @@ >

[FFmpeg-devel] [PATCH] fate: fix filter-hls tests dependencies

2016-03-02 Thread James Almer
Signed-off-by: James Almer --- tests/fate/filter-audio.mak| 15 +-- tests/ref/fate/{filter-hls2 => filter-hls} | 0 2 files changed, 9 insertions(+), 6 deletions(-) rename tests/ref/fate/{filter-hls2 => filter-hls} (100%) diff --git

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: remove *_NULLABLE annotations; fixes pre-10.11 build

2016-03-02 Thread Richard Kern
> On Mar 3, 2016, at 11:46 AM, Rodger Combs wrote: > > These macros were added in OS X 10.11, and the file compiles without warnings > on both 10.10 and 10.11 with them removed. > > Thanks to mark4o on IRC for pointing out the failure and testing the patch. Lgtm -

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: remove *_NULLABLE annotations; fixes pre-10.11 build

2016-03-02 Thread Rodger Combs
These macros were added in OS X 10.11, and the file compiles without warnings on both 10.10 and 10.11 with them removed. Thanks to mark4o on IRC for pointing out the failure and testing the patch. --- libavcodec/videotoolboxenc.c | 14 +++--- 1 file changed, 7 insertions(+), 7

Re: [FFmpeg-devel] [PATCH] Add test foravpriv_get_trc_function_from_trc function

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 06:36:26PM -0800, NagaChaitanya Vellanki wrote: > --- > libavutil/Makefile | 1 + > libavutil/color_utils.c| 29 + > tests/fate/libavutil.mak | 4 + > tests/ref/fate/color_utils | 285 > + > 4 files

[FFmpeg-devel] [PATCH] videotoolboxenc: Fix compile failure on OS X 10.10

2016-03-02 Thread Mark Harris
--- libavcodec/videotoolboxenc.c | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index bbecb24..342c83c 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec/videotoolboxenc.c @@ -32,6 +32,14 @@ #include "internal.h"

Re: [FFmpeg-devel] [PATCH] fate: add filter-hls

2016-03-02 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 04:59:46AM +0100, Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 12:32:27AM -0300, James Almer wrote: > > On 3/1/2016 12:24 AM, Michael Niedermayer wrote: > > > On Tue, Mar 01, 2016 at 12:17:33AM -0300, James Almer wrote: > > >> On 2/29/2016 11:47 PM, Michael

[FFmpeg-devel] [PATCH] Add test foravpriv_get_trc_function_from_trc function

2016-03-02 Thread NagaChaitanya Vellanki
--- libavutil/Makefile | 1 + libavutil/color_utils.c| 29 + tests/fate/libavutil.mak | 4 + tests/ref/fate/color_utils | 285 + 4 files changed, 319 insertions(+) create mode 100644 tests/ref/fate/color_utils diff --git

Re: [FFmpeg-devel] [PATCH] Add test foravpriv_get_trc_function_from_trc function

2016-03-02 Thread NagaChaitanya Vellanki
Sending in the new patch :-). On Wed, Mar 2, 2016 at 5:20 PM, Michael Niedermayer wrote: > On Wed, Mar 02, 2016 at 04:35:08PM -0800, NagaChaitanya Vellanki wrote: > > --- > > libavutil/Makefile | 1 + > > libavutil/color_utils.c| 65 >

Re: [FFmpeg-devel] [PATCH] tests/gapless: add gapless aac tests

2016-03-02 Thread Michael Niedermayer
On Thu, Mar 03, 2016 at 02:27:52AM +0100, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > tests/fate/gapless.mak | 3 +++ > tests/ref/fate/gapless-aac | 5 + > 2 files changed, 8 insertions(+) > create mode 100644 tests/ref/fate/gapless-aac seems to fail on

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Michael Niedermayer
On Thu, Mar 03, 2016 at 12:36:21AM +0100, Mats Peterson wrote: > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > internal.h |7 +-- > utils.c| 23 --- > 2 files changed, 21 insertions(+), 9 deletions(-) > 3f86328b63621ddba1ee0730ce639010fc38aec3 >

Re: [FFmpeg-devel] [PATCH] avformat/y4m: fix seeking in partial files

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 08:59:34PM +0100, Paul B Mahol wrote: > Hi, > > patch attached. > yuv4mpegdec.c |3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > 3e38c18c187adfbc6aa88e52c9e01177de126847 > 0001-avformat-yuv4mpegdec-fix-seeking-for-partial-files.patch > From

[FFmpeg-devel] [PATCH] tests/gapless: add gapless aac tests

2016-03-02 Thread Marton Balint
Signed-off-by: Marton Balint --- tests/fate/gapless.mak | 3 +++ tests/ref/fate/gapless-aac | 5 + 2 files changed, 8 insertions(+) create mode 100644 tests/ref/fate/gapless-aac diff --git a/tests/fate/gapless.mak b/tests/fate/gapless.mak index 2fb005f..c47ab94 100644

Re: [FFmpeg-devel] [PATCH] Add test foravpriv_get_trc_function_from_trc function

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 04:35:08PM -0800, NagaChaitanya Vellanki wrote: > --- > libavutil/Makefile | 1 + > libavutil/color_utils.c| 65 > ++ > tests/fate/libavutil.mak | 4 +++ > tests/ref/fate/color_utils | 38

Re: [FFmpeg-devel] [PATCH 3/3] ffmpeg: remove hardcoded 'now' creation_time support

2016-03-02 Thread Marton Balint
On Sun, 28 Feb 2016, Marton Balint wrote: Every date parsing routine now uses av_parse_time which handles 'now' and provides greater precision as well. This change also enables the segmenter muxer to set the proper 'now' creation time at the beginning of each segment. Signed-off-by: Marton

Re: [FFmpeg-devel] [PATCH 2/3] avformat: use ff_standardize_creation_time for formats writing all format string metadata

2016-03-02 Thread Marton Balint
On Sun, 28 Feb 2016, wm4 wrote: On Sun, 28 Feb 2016 14:28:20 +0100 Marton Balint wrote: Signed-off-by: Marton Balint --- libavformat/apetag.c| 1 + libavformat/cafenc.c| 1 + libavformat/flvenc.c| 1 + libavformat/id3v2enc.c | 1 +

[FFmpeg-devel] [PATCH] Add test foravpriv_get_trc_function_from_trc function

2016-03-02 Thread NagaChaitanya Vellanki
--- libavutil/Makefile | 1 + libavutil/color_utils.c| 65 ++ tests/fate/libavutil.mak | 4 +++ tests/ref/fate/color_utils | 38 +++ 4 files changed, 108 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH v5 1/4] lavf/avienc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/03/2016 12:34 AM, Mats Peterson wrote: Hopefully this patch set will work on big-endian machines as well. Please try stream copy to and from avi/mov with the files below: QuickTime Animation (RLE): https://drive.google.com/open?id=0B3_pEBoLs0faREo1SlRydmV1LU0 QuickTime Graphics (SMC):

Re: [FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
On 03/03/2016 12:36 AM, Mats Peterson wrote: Different interface as well. Now uses a pointer to a palette buffer that the palette is copied to. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/dca: simplify condition

2016-03-02 Thread Hendrik Leppkes
On Wed, Mar 2, 2016 at 8:32 PM, foo86 wrote: > --- > libavcodec/dca_xll.c | 17 ++--- > 1 file changed, 6 insertions(+), 11 deletions(-) > > diff --git a/libavcodec/dca_xll.c b/libavcodec/dca_xll.c > index ffe8ef3..5e6cf35 100644 > --- a/libavcodec/dca_xll.c > +++

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/dca: fix av_cold placement in declarations

2016-03-02 Thread Hendrik Leppkes
On Wed, Mar 2, 2016 at 8:31 PM, foo86 wrote: > --- > libavcodec/dca_xll.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/dca_xll.c b/libavcodec/dca_xll.c > index cd1af81..ffe8ef3 100644 > --- a/libavcodec/dca_xll.c > +++

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/dca: make reading past end of buffer safe

2016-03-02 Thread Hendrik Leppkes
On Thu, Mar 3, 2016 at 12:37 AM, Ronald S. Bultje wrote: > Hi, > > On Wed, Mar 2, 2016 at 2:32 PM, foo86 wrote: > >> Limit the maximum length of unary part of Rice code by the number of >> available bits instead of using an arbitrary constant that happens

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/dca: make reading past end of buffer safe

2016-03-02 Thread Ronald S. Bultje
Hi, On Wed, Mar 2, 2016 at 2:32 PM, foo86 wrote: > Limit the maximum length of unary part of Rice code by the number of > available bits instead of using an arbitrary constant that happens to be > just large enough to work. > > This effectively limits amount of data that can

[FFmpeg-devel] [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette()

2016-03-02 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 75f98197bda12b486971d1d6cc139cb5d22b30ba Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Thu, 3 Mar 2016 00:29:32 +0100 Subject: [PATCH v5 4/4] lavf/utils: Fix endianness in ff_get_packet_palette() ---

[FFmpeg-devel] [PATCH v5 3/4] lavf/riffenc: Handle palette for non-raw codecs

2016-03-02 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 602db1c464b7a432b4f4b471cbeed401cd220e4c Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Thu, 3 Mar 2016 00:28:23 +0100 Subject: [PATCH v5 3/4] lavf/riffenc: Handle palette for non-raw codecs ---

[FFmpeg-devel] [PATCH v5 2/4] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 99e1f56c242cfa3d20a0987a7e766fa358ed35c2 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Thu, 3 Mar 2016 00:27:53 +0100 Subject: [PATCH v5 2/4] lavf/movenc: Add support for palette side data ---

[FFmpeg-devel] [PATCH v5 1/4] lavf/avienc: Add support for palette side data

2016-03-02 Thread Mats Peterson
Hopefully this patch set will work on big-endian machines as well. Please try stream copy to and from avi/mov with the files below: QuickTime Animation (RLE): https://drive.google.com/open?id=0B3_pEBoLs0faREo1SlRydmV1LU0 QuickTime Graphics (SMC):

[FFmpeg-devel] [PATCH] Use matroska TrackNumber for populating AVStream::id

2016-03-02 Thread Sergey Volk
As far as I can see FFmpeg currently doesn't set AVStream::id for matroska/webm streams. I think we could use either MatroskaTrack::num (TrackNumber) or MatroskaTrack::uid (TrackUID) for that. I have found a few discussions claiming that TrackUID could be missing, even though TrackUID is marked as

[FFmpeg-devel] [PATCH] Add encoder stats to the NVENC codec.

2016-03-02 Thread Lucas Cooper
--- libavcodec/nvenc.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index a3b02fa..67232c7 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -1208,31 +1208,35 @@ static int

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/02/2016 11:19 PM, Mats Peterson wrote: On 03/02/2016 11:07 PM, Mats Peterson wrote: On 03/02/2016 10:30 PM, Michael Niedermayer wrote: +ret = 0; +if (trk->enc->pix_fmt == AV_PIX_FMT_PAL8 && !trk->pal_done) { +const uint8_t *pal; +

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/02/2016 11:07 PM, Mats Peterson wrote: On 03/02/2016 10:30 PM, Michael Niedermayer wrote: +ret = 0; +if (trk->enc->pix_fmt == AV_PIX_FMT_PAL8 && !trk->pal_done) { +const uint8_t *pal; +int ret2 = ff_get_packet_palette(s, opkt,

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/02/2016 10:30 PM, Michael Niedermayer wrote: +ret = 0; +if (trk->enc->pix_fmt == AV_PIX_FMT_PAL8 && !trk->pal_done) { +const uint8_t *pal; +int ret2 = ff_get_packet_palette(s, opkt, ret, ); +if (ret2 < 0) +

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/02/2016 09:36 PM, Michael Niedermayer wrote: in which cases of compressed (non raw) video does doing something special for pix_fmt == pal8 fix a problem ? In several cases. QuickTime RLE in 8-bit mode, QuickTime Graphics (SMC) in 8-bit mode, Microsoft Video 1 (CRAM) in 8-bit mode. All

Re: [FFmpeg-devel] [PATCH 0/4] DCA decoder fixes

2016-03-02 Thread James Almer
On 3/2/2016 4:30 PM, foo86 wrote: > This fixes some potential issues I've identified within new DCA decoder. It is > probably a good idea to backport these patches into 3.0 release branch. > > foo86 (4): > avcodec/dca: clear X96 channels if nothing was decoded > avcodec/dca: fix av_cold

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-02 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > personally ill probably abstain, as i even half misunderstood > the use case this was inteded for ;) But that's true for everybody who has commented so far... Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-02 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 10:38:27PM -0500, Ganesh Ajjanagadde wrote: > On Tue, Mar 1, 2016 at 9:17 AM, Paul B Mahol wrote: > > On 3/1/16, Ronald S. Bultje wrote: > >> Hi, > >> > >> On Tue, Mar 1, 2016 at 8:04 AM, Carl Eugen Hoyos wrote: >

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 08:33:42PM +0100, Mats Peterson wrote: > Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > movenc.c | 45 +++-- > 1 file changed, 27 insertions(+),

Re: [FFmpeg-devel] [PATCH 1/2] lavf: allow BSFs to drop packets.

2016-03-02 Thread Ronald S. Bultje
Hi, On Wed, Mar 2, 2016 at 3:51 PM, wm4 wrote: > On Wed, 2 Mar 2016 15:48:11 -0500 > "Ronald S. Bultje" wrote: > > > Hi, > > > > On Mon, Feb 29, 2016 at 10:32 AM, Ronald S. Bultje > > wrote: > > > > > If pkt->size == 0 &&

Re: [FFmpeg-devel] [PATCH 1/2] lavf: allow BSFs to drop packets.

2016-03-02 Thread wm4
On Wed, 2 Mar 2016 15:48:11 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Mon, Feb 29, 2016 at 10:32 AM, Ronald S. Bultje > wrote: > > > If pkt->size == 0 && pkt->side_data_elems == 0 after bsf->filter() > > returns, the packet is considered dropped. >

Re: [FFmpeg-devel] [PATCH 1/2] lavf: allow BSFs to drop packets.

2016-03-02 Thread Ronald S. Bultje
Hi, On Mon, Feb 29, 2016 at 10:32 AM, Ronald S. Bultje wrote: > If pkt->size == 0 && pkt->side_data_elems == 0 after bsf->filter() > returns, the packet is considered dropped. Ping? Ronald ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 09:24:37PM +0100, Mats Peterson wrote: > On 03/02/2016 09:23 PM, Mats Peterson wrote: > >On 03/02/2016 09:21 PM, Michael Niedermayer wrote: > >>On Wed, Mar 02, 2016 at 08:33:42PM +0100, Mats Peterson wrote: > >>>Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 09:23:40PM +0100, Mats Peterson wrote: > On 03/02/2016 09:21 PM, Michael Niedermayer wrote: > >On Wed, Mar 02, 2016 at 08:33:42PM +0100, Mats Peterson wrote: > >>Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. > >> > >>-- > >>Mats Peterson >

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/02/2016 09:23 PM, Mats Peterson wrote: On 03/02/2016 09:21 PM, Michael Niedermayer wrote: On Wed, Mar 02, 2016 at 08:33:42PM +0100, Mats Peterson wrote: Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. -- Mats Peterson http://matsp888.no-ip.org/~mats/ movenc.c

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
On 03/02/2016 09:21 PM, Michael Niedermayer wrote: On Wed, Mar 02, 2016 at 08:33:42PM +0100, Mats Peterson wrote: Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. -- Mats Peterson http://matsp888.no-ip.org/~mats/ movenc.c | 45

Re: [FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 08:33:42PM +0100, Mats Peterson wrote: > Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > movenc.c | 45 +++-- > 1 file changed, 27 insertions(+),

Re: [FFmpeg-devel] [PATCH v10] VideoToolbox H.264 Encoder

2016-03-02 Thread wm4
On Wed, 2 Mar 2016 20:11:40 +0800 Rick Kern wrote: > Autodetected by default. Encode using -codec:v h264_videotoolbox. > > Signed-off-by: Rick Kern > --- Applied. Thanks, and sorry for the delays/revisions. ___

Re: [FFmpeg-devel] FLIC in AVI

2016-03-02 Thread Mats Peterson
On 03/02/2016 09:18 PM, Mats Peterson wrote: On 03/02/2016 09:16 PM, Mats Peterson wrote: On 03/02/2016 09:02 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson wrote: On 03/02/2016 08:45 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson

Re: [FFmpeg-devel] FLIC in AVI

2016-03-02 Thread Mats Peterson
On 03/02/2016 09:16 PM, Mats Peterson wrote: On 03/02/2016 09:02 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson wrote: On 03/02/2016 08:45 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson wrote: I noticed that you, Paul,

Re: [FFmpeg-devel] FLIC in AVI

2016-03-02 Thread Mats Peterson
On 03/02/2016 09:02 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson wrote: On 03/02/2016 08:45 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson wrote: I noticed that you, Paul, added a patch somewhere in 2012 with the

[FFmpeg-devel] [PATCH] avformat/y4m: fix seeking in partial files

2016-03-02 Thread Paul B Mahol
Hi, patch attached. From af75cff37a296072fad21648459b4e005ce1afc6 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Wed, 2 Mar 2016 20:56:01 +0100 Subject: [PATCH] avformat/yuv4mpegdec: fix seeking for partial files Signed-off-by: Paul B Mahol ---

Re: [FFmpeg-devel] FLIC in AVI

2016-03-02 Thread Paul B Mahol
On 3/2/16, Mats Peterson wrote: > On 03/02/2016 08:45 PM, Paul B Mahol wrote: >> On 3/2/16, Mats Peterson wrote: >>> I noticed that you, Paul, added a patch somewhere in 2012 with the >>> purpose of making it possible to use

Re: [FFmpeg-devel] FLIC in AVI

2016-03-02 Thread Mats Peterson
On 03/02/2016 08:45 PM, Paul B Mahol wrote: On 3/2/16, Mats Peterson wrote: I noticed that you, Paul, added a patch somewhere in 2012 with the purpose of making it possible to use FLI(C) in AVI. It doesn't work very well in my book. Try the following command

[FFmpeg-devel] [PATCH 1/4] avcodec/dca: clear X96 channels if nothing was decoded

2016-03-02 Thread foo86
The first X96 channel set can have more channels than core, causing X96 decoding to be skipped. Clear the number of decoded X96 channels to zero in this rudimentary case. --- libavcodec/dca_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/dca_core.c b/libavcodec/dca_core.c

Re: [FFmpeg-devel] FLIC in AVI

2016-03-02 Thread Paul B Mahol
On 3/2/16, Mats Peterson wrote: > I noticed that you, Paul, added a patch somewhere in 2012 with the > purpose of making it possible to use FLI(C) in AVI. It doesn't work very > well in my book. Try the following command with the file below: There is trac for

[FFmpeg-devel] [PATCH v4 2/5 v2] lavf/movenc: Add support for palette side data

2016-03-02 Thread Mats Peterson
Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From c2d0b7a69486afb926e731216b4aff3b04c4ee4a Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Wed, 2 Mar 2016 20:30:59 +0100 Subject: [PATCH v4 2/5

[FFmpeg-devel] [PATCH v4 1/5 v2] lavf/avienc: Add support for palette side data

2016-03-02 Thread Mats Peterson
Check that the track type is AVMEDIA_TYPE_VIDEO in appropriate places. -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 719c9d9ed312da81499c9fef589af24595c3228b Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Wed, 2 Mar 2016 20:30:47 +0100 Subject: [PATCH v4 1/5

Re: [FFmpeg-devel] [PATCH] aacenc: avoid double in quantize_bands.

2016-03-02 Thread Rostislav Pehlivanov
On 1 March 2016 at 21:55, Reimar Döffinger wrote: > I cannot see any point whatsoever to use > double here instead of float. > Using float allows for use of SIMD. > --- > libavcodec/aacenc_utils.h | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff

[FFmpeg-devel] [PATCH 4/4] avcodec/dca: make reading past end of buffer safe

2016-03-02 Thread foo86
Limit the maximum length of unary part of Rice code by the number of available bits instead of using an arbitrary constant that happens to be just large enough to work. This effectively limits amount of data that can be overread per segment by maximum length of binary code per sample multiplied

[FFmpeg-devel] [PATCH 2/4] avcodec/dca: fix av_cold placement in declarations

2016-03-02 Thread foo86
--- libavcodec/dca_xll.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/dca_xll.c b/libavcodec/dca_xll.c index cd1af81..ffe8ef3 100644 --- a/libavcodec/dca_xll.c +++ b/libavcodec/dca_xll.c @@ -602,7 +602,7 @@ static int chs_parse_band_data(DCAXllDecoder *s,

[FFmpeg-devel] [PATCH 3/4] avcodec/dca: simplify condition

2016-03-02 Thread foo86
--- libavcodec/dca_xll.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/libavcodec/dca_xll.c b/libavcodec/dca_xll.c index ffe8ef3..5e6cf35 100644 --- a/libavcodec/dca_xll.c +++ b/libavcodec/dca_xll.c @@ -460,19 +460,14 @@ static int

[FFmpeg-devel] [PATCH 0/4] DCA decoder fixes

2016-03-02 Thread foo86
This fixes some potential issues I've identified within new DCA decoder. It is probably a good idea to backport these patches into 3.0 release branch. foo86 (4): avcodec/dca: clear X96 channels if nothing was decoded avcodec/dca: fix av_cold placement in declarations avcodec/dca: simplify

Re: [FFmpeg-devel] [PATCH] aacenc: optimize cost cache.

2016-03-02 Thread Rostislav Pehlivanov
On 1 March 2016 at 23:34, Reimar Döffinger wrote: > Avoids trashing the CPU cache each time the > cost cache is cleared. > --- > libavcodec/aaccoder_twoloop.h | 20 > libavcodec/aacenc.c | 7 +-- > libavcodec/aacenc.h

Re: [FFmpeg-devel] [PATCH]concatdec: measure duration when it is not available

2016-03-02 Thread Bodecs Bela
2016.03.01. 11:26 keltezéssel, Michael Niedermayer írta: On Tue, Mar 01, 2016 at 10:57:57AM +0100, Nicolas George wrote: Le nonidi 29 pluviôse, an CCXXIV, Bodecs Bela a écrit : I have checked the code where and when duration filled and made some reasoning about it. Duration value for input

Re: [FFmpeg-devel] [PATCH]concatdec: measure duration when it is not available

2016-03-02 Thread Bodecs Bela
2016.03.01. 10:57 keltezéssel, Nicolas George írta: Le nonidi 29 pluviôse, an CCXXIV, Bodecs Bela a écrit : I have checked the code where and when duration filled and made some reasoning about it. Duration value for input files has been populated some time after opening. (estimate_timings

Re: [FFmpeg-devel] [PATCH v4 5/5 v2] lavf/internal.h: Add declaration for ff_get_packet_palette()

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 07:52:30AM +0100, Mats Peterson wrote: > Edited the doxy documentation. > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > internal.h | 12 > 1 file changed, 12 insertions(+) > 9b2a67e03ab88f7806a94c66f3d62410d34cd122 >

Re: [FFmpeg-devel] [PATCH v4 4/5] lavf/utils: New function ff_get_packet_palette()

2016-03-02 Thread Michael Niedermayer
On Wed, Mar 02, 2016 at 03:26:02AM +0100, Mats Peterson wrote: > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > utils.c | 16 > 1 file changed, 16 insertions(+) > 7fbf3f392daebb25a47ba12957917aee6c73db87 > 0004-lavf-utils-New-function-ff_get_packet_palette.patch

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-03-02 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 11:26:45PM +0100, Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 08:01:45PM +0100, Matthieu Bouron wrote: > > On Sat, Feb 27, 2016 at 04:28:43PM +0100, Michael Niedermayer wrote: > > > On Fri, Feb 26, 2016 at 04:54:47PM +0100, Matthieu Bouron wrote: > > > > On Tue,

[FFmpeg-devel] [PATCH 3/3] lavf/hashenc: clone md5 and framemd5 muxers into hash and framehash muxers

2016-03-02 Thread Moritz Barsnick
The two new muxers are functionally identical, except that they default to the SHA-512 algorithm instead of MD5. The original "legacy" muxers continue to provide their MD5 defaults. The currently supported hash functions are now documented. Signed-off-by: Moritz Barsnick ---

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-02 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 09:06:35PM +0100, wm4 wrote: > On Tue, 1 Mar 2016 20:57:12 +0100 > Matthieu Bouron wrote: > > > On Tue, Mar 01, 2016 at 08:55:23PM +0100, Matthieu Bouron wrote: > > > On Tue, Mar 01, 2016 at 08:38:04PM +0100, wm4 wrote: > > > > On Tue, 1 Mar

[FFmpeg-devel] [PATCH 2/3] lavf/hashenc: rename variables where appropriate

2016-03-02 Thread Moritz Barsnick
In preparation for duplication of muxers and functionality. Functions which are applicable to md5 only - those which default to using the MD5 hash algorithm - and will continue to do so keep their names. What's more, the options array, which already carried "hash" in its name instead of "md5", now

[FFmpeg-devel] [PATCH 0/3] lavf: generalize md5enc to hashenc and clone the muxers

2016-03-02 Thread Moritz Barsnick
As suggested in the thread regarding the proposed streammd5 muxer: http://ffmpeg.org/pipermail/ffmpeg-devel/2016-February/189694.html this patchset converts the md5/framemd5 muxers into hash/hashenc muxers with aliased muxers retaining the original names and default algorithms. I am aware that

[FFmpeg-devel] [PATCH 1/3] lavf: rename md5enc source to hashenc

2016-03-02 Thread Moritz Barsnick
To move away from naming it by only one of the supported algorithms. Signed-off-by: Moritz Barsnick --- libavformat/Makefile | 4 +- libavformat/hashenc.c | 171 ++ libavformat/md5enc.c | 171

Re: [FFmpeg-devel] [PATCH] Fix a bug in ff_thread_report_progress in updating progress[field].

2016-03-02 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 8:05 PM, Wan-Teh Chang wrote: > On Tue, Mar 1, 2016 at 7:44 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Tue, Mar 1, 2016 at 9:34 AM, Wan-Teh Chang < > wtc-at-google@ffmpeg.org> > > wrote: > > > >> By the way,

[FFmpeg-devel] [PATCH 2/4] vc2enc: remove useless alignment on slice encoding

2016-03-02 Thread Rostislav Pehlivanov
This was a leftover from before the slices were encoded in parallel. Since the put_bits context is initialized per slice aligning it aferwards is pointless. Signed-off-by: Rostislav Pehlivanov --- libavcodec/vc2enc.c | 1 - 1 file changed, 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH 4/4] vc2enc: only warn on a non-standard base video format

2016-03-02 Thread Rostislav Pehlivanov
This commit is up for discussion. Requiring the user to reduce the strictness just to encode some random file (which decodes fine in all implementations except 1) isn't helping the encoder at all. The encoder needs to be used to get bug reports and to be improved by multiple people. Only

[FFmpeg-devel] [PATCH 3/4] vc2enc: minor cosmetic changes

2016-03-02 Thread Rostislav Pehlivanov
Signed-off-by: Rostislav Pehlivanov --- libavcodec/vc2enc.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/libavcodec/vc2enc.c b/libavcodec/vc2enc.c index 34a4f95..3bc60a3 100644 --- a/libavcodec/vc2enc.c +++ b/libavcodec/vc2enc.c @@ -506,7

[FFmpeg-devel] [PATCH 1/4] vc2enc: do not allocate packet until exact frame size is known

2016-03-02 Thread Rostislav Pehlivanov
This commit solves most of the crashes and issues with the encoder and the bitrate setting. Now the encoder will always allocate the absolute lowest amount of memory regardless of what the bitrate has been set to. Therefore if a user inputs a very low bitrate the encoder will use the maximum

[FFmpeg-devel] [PATCH v10] VideoToolbox H.264 Encoder

2016-03-02 Thread Rick Kern
Autodetected by default. Encode using -codec:v h264_videotoolbox. Signed-off-by: Rick Kern --- MAINTAINERS |1 + configure| 26 +- libavcodec/Makefile |1 + libavcodec/allcodecs.c |1 +

[FFmpeg-devel] [PATCH v10] VideoToolbox H.264 Encoder

2016-03-02 Thread Rick Kern
Uses the --disable-videotoolbox configure switch like the hwaccel, changed encoder filename, changed free() to av_free(). Rick Kern (1): VideoToolbox H.264 Encoder MAINTAINERS |1 + configure| 26 +- libavcodec/Makefile |1 +

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-02 Thread Richard Kern
> On Mar 2, 2016, at 12:11 AM, wm4 wrote: > > On Tue, 01 Mar 2016 14:57:29 + > Timothy Gu > wrote: > >> Hi, >> >> On Mon, Feb 29, 2016 at 9:42 PM Rick Kern wrote: >> >>> Autodetected by

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace sqrtf(Q*sqrtf(Q)) by precomputed value

2016-03-02 Thread Rostislav Pehlivanov
On 2 March 2016 at 04:04, Ganesh Ajjanagadde wrote: > On Tue, Mar 1, 2016 at 7:52 AM, Derek Buitenhuis > wrote: > > On 3/1/2016 3:21 AM, Ganesh Ajjanagadde wrote: > > > > [...] > > > >> --- > >> libavcodec/aacenc_utils.h | 3 +-- > >> 1 file