[FFmpeg-devel] [PATCH] lavc/rawdec: Align AV_PIX_FMT_RGB24 correctly

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From ea996121595ebe61de297858092e038fa7a52415 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sun, 21 Feb 2016 06:50:41 +0100 Subject: [PATCH] lavc/rawdec: Align AV_PIX_FMT_RGB24 correctly --- libavcodec/rawdec.c | 11

Re: [FFmpeg-devel] [PATCH v5] lavf/movenc: Add palette to video sample description

On 02/21/2016 06:09 AM, Mats Peterson wrote: On 02/21/2016 06:01 AM, Mats Peterson wrote: Checking for pixel format to be RGB (including monochrome) and not YUV. I should add that converting an odd-width file to raw 24-bit QuickTime produces a skewed file. Something is not entirely correct

Re: [FFmpeg-devel] [PATCH v5] lavf/movenc: Add palette to video sample description

On 02/21/2016 06:01 AM, Mats Peterson wrote: Checking for pixel format to be RGB (including monochrome) and not YUV. I should add that converting an odd-width file to raw 24-bit QuickTime produces a skewed file. Something is not entirely correct in the calculations in

[FFmpeg-devel] [PATCH v5] lavf/movenc: Add palette to video sample description

Checking for pixel format to be RGB (including monochrome) and not YUV. -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 483569ce254ce31a279213e2b9f16840e1fc495f Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sun, 21 Feb 2016 05:58:12 +0100 Subject: [PATCH v5]

Re: [FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

On 02/21/2016 04:47 AM, Mats Peterson wrote: On 02/21/2016 04:33 AM, Mats Peterson wrote: i think this would also get run for raw yuv formats I see. Do you have a suggestion on how to "isolate" it? MAts Is trk->enc->bits_per_coded_sample perhaps zero when using raw yuv? Hardly. Sorry

Re: [FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

On 02/21/2016 04:33 AM, Mats Peterson wrote: i think this would also get run for raw yuv formats I see. Do you have a suggestion on how to "isolate" it? MAts Is trk->enc->bits_per_coded_sample perhaps zero when using raw yuv? Hardly. Sorry for that stupid assumption. Mats

Re: [FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

On 02/21/2016 04:28 AM, Mats Peterson wrote: On 02/21/2016 04:23 AM, Michael Niedermayer wrote: On Sun, Feb 21, 2016 at 02:00:14AM +0100, Mats Peterson wrote: Forgot movenc.h. -- Mats Peterson http://matsp888.no-ip.org/~mats/ movenc.c | 50

Re: [FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

On 02/21/2016 04:23 AM, Michael Niedermayer wrote: On Sun, Feb 21, 2016 at 02:00:14AM +0100, Mats Peterson wrote: Forgot movenc.h. -- Mats Peterson http://matsp888.no-ip.org/~mats/ movenc.c | 50 +- movenc.h |3 +++ 2 files changed,

Re: [FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

On Sun, Feb 21, 2016 at 02:00:14AM +0100, Mats Peterson wrote: > Forgot movenc.h. > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > movenc.c | 50 +- > movenc.h |3 +++ > 2 files changed, 52 insertions(+), 1 deletion(-) >

Re: [FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

Mats Peterson skrev: (21 februari 2016 02:00:14 CET) >Forgot movenc.h. I should add that converting an odd file to raw 24-bit QuickTime produces a skewed file. Something is not entirely correct in the calculations in ff_reshuffle_raw_rgb(), probably related

Re: [FFmpeg-devel] [PATCH v3 4/4] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

Michael Niedermayer skrev: (21 februari 2016 04:02:21 CET) >On Sat, Feb 20, 2016 at 01:04:51PM +0100, Mats Peterson wrote: >> >> -- >> Mats Peterson >> http://matsp888.no-ip.org/~mats/ > >> rawdec.c | 19 --- >> 1 file changed, 19 deletions(-) >>

Re: [FFmpeg-devel] [PATCH v3 4/4] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

On Sat, Feb 20, 2016 at 01:04:51PM +0100, Mats Peterson wrote: > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > rawdec.c | 19 --- > 1 file changed, 19 deletions(-) > dd6beae8ed2d83e7cdeb28df6e7f05616ed0d799 >

Re: [FFmpeg-devel] [PATCH v3 1/4] lavf/riffenc: Write space for palette

Mats Peterson skrev: (21 februari 2016 03:41:30 CET) >Michael Niedermayer skrev: (21 februari 2016 >03:35:02 CET) >>On Sat, Feb 20, 2016 at 01:11:47PM +0100, Mats Peterson wrote: >>> On 02/20/2016 01:02 PM, Mats Peterson wrote: >>> > >>>

Re: [FFmpeg-devel] [PATCH v3 1/4] lavf/riffenc: Write space for palette

Michael Niedermayer skrev: (21 februari 2016 03:35:02 CET) >On Sat, Feb 20, 2016 at 01:11:47PM +0100, Mats Peterson wrote: >> On 02/20/2016 01:02 PM, Mats Peterson wrote: >> > >> > >> > >> >___ >> >ffmpeg-devel mailing list >>

Re: [FFmpeg-devel] [PATCH v3 1/4] lavf/riffenc: Write space for palette

On Sat, Feb 20, 2016 at 01:11:47PM +0100, Mats Peterson wrote: > On 02/20/2016 01:02 PM, Mats Peterson wrote: > > > > > > > >___ > >ffmpeg-devel mailing list > >ffmpeg-devel@ffmpeg.org > >http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > Sorry for

Re: [FFmpeg-devel] [PATCH v3 3/4] tests/ref/vsynth: Update 1 bpp files for pal8

On Sat, Feb 20, 2016 at 01:04:07PM +0100, Mats Peterson wrote: > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > vsynth1-bpp1 |8 > vsynth2-bpp1 |8 > vsynth3-bpp1 |8 > vsynth_lena-bpp1 |8 > 4 files changed, 16

Re: [FFmpeg-devel] [PATCH v3 2/4 v2] lavf/avienc: Add palette after BITMAPINFOHEADER

On Sat, Feb 20, 2016 at 02:42:51PM +0100, Mats Peterson wrote: > Forgot to remove an av_free(). > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > avienc.c | 23 +++ > 1 file changed, 19 insertions(+), 4 deletions(-) > 32295d2f96b7651139b6859e38a5042a78c6687e >

[FFmpeg-devel] [PATCH v4] lavf/movenc: Add palette to video sample description

Forgot movenc.h. -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From dac790207f7057a460a6f460a6146e937fc89211 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sun, 21 Feb 2016 01:59:17 +0100 Subject: [PATCH v4] lavf/movenc: Add palette to video sample description ---

[FFmpeg-devel] [PATCH v3] lavf/movenc: Add palette to video sample description

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From d6dd1dd2f2ac378da0a8e85a47f9e1b925ce8bce Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sun, 21 Feb 2016 01:55:37 +0100 Subject: [PATCH v3] lavf/movenc: Add palette to video sample description ---

[FFmpeg-devel] [PATCH] avcodec/webp: move exif_metadata outside of WebPContext

Signed-off-by: James Almer --- libavcodec/webp.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/libavcodec/webp.c b/libavcodec/webp.c index 5c2961f..81b2e91 100644 --- a/libavcodec/webp.c +++ b/libavcodec/webp.c @@ -196,7 +196,6 @@ typedef struct

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

Hi, > 2nd version attached. These files fails to decode: http://www.datafilehost.com/d/d5a2d63f Regards ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Fixed issue #43 JACK indev support on OSX

On Fri, Feb 19, 2016 at 08:27:50PM +, Josh de Kock wrote: > --- > configure | 13 ++--- > libavdevice/jack.c | 10 ++ > 2 files changed, 20 insertions(+), 3 deletions(-) > > diff --git a/configure b/configure > index 287896f..78f9618 100755 > --- a/configure > +++

[FFmpeg-devel] [PATCH v2] lavf/movenc: Add palette to video sample description

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From ea8f4f037a2fcde2a0f95ddf3a1ee2bee871b7a4 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 20 Feb 2016 22:45:22 +0100 Subject: [PATCH v2] lavf/movenc: Add palette to video sample description ---

Re: [FFmpeg-devel] [PATCH] lavf/movenc: Add palette to video sample description

On 02/20/2016 01:31 PM, Mats Peterson wrote: On 02/20/2016 01:06 PM, Mats Peterson wrote: This one is somewhat experimental so far, since it doesn't work well with ff_reshuffle_raw_rgb() for some reason. Perhaps you could look into the reason why, Michael. Remember that QuickTime strides are

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

On Sat, Feb 20, 2016 at 05:41:40PM +, Derek Buitenhuis wrote: > > ffmpeg | branch: master | Michael Niedermayer | > > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] > > | committer: Michael Niedermayer > > [...] > > > +if (align < 4) { > > +

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

On 2/20/16, Paul B Mahol wrote: > Hi, > > patch attached. > 2nd version attached. From 31997a1f3e78050dc1bb6f19e8475fcc4efbd662 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 19 Feb 2016 14:35:53 +0100 Subject: [PATCH] IFF ANIM support Currently only

Re: [FFmpeg-devel] [PATCH] libvpxenc: quiet unused-variable warning

On 2/20/2016 4:26 PM, James Zern wrote: > with older versions of libvpx > since: > 432be63 lavc/libvpx: Fix support for RGB colorspace. > > Signed-off-by: James Zern > --- > libavcodec/libvpxenc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH] lavc/lavf: transmit stream_id information for mpegts KLV data packets

On date Saturday 2016-02-13 21:06:23 +0100, Marton Balint encoded: > > On Sat, 13 Feb 2016, Stefano Sabatini wrote: > > >>Subject: [PATCH] lavc/lavf: transmit stream_id information for mpegts KLV > >>data > >> packets > >> > > [...] > > >Someone else wants to have a look, or should I push as

[FFmpeg-devel] [PATCH] libvpxenc: quiet unused-variable warning

with older versions of libvpx since: 432be63 lavc/libvpx: Fix support for RGB colorspace. Signed-off-by: James Zern --- libavcodec/libvpxenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index 1239103..b1114bc 100644

[FFmpeg-devel] [PATCH] checkasm: bench each vf_blend mode once

Also test a smaller buffer. This drastically reduces --bench runtime and reports smaller, more readable numbers. Signed-off-by: James Almer --- tests/checkasm/vf_blend.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/checkasm/vf_blend.c

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

Hi, > What this VirtualDub thing use for ANIM decoding? It has internal decoder for IFF ANIM - see "InputFileANIM.cpp". Regards ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

On 2/20/16, Piotr Bandurski wrote: > Hi, > >> >> The player I have shows same artefacts with AmigaRHQ sample. >> > >> > AmigaRHQ doesn't decode here: >> >> Because i have fix locally, I asked do you have player that plays >> it without artefacts? Both xanim and iffanimplay shows

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

Hi, > >> The player I have shows same artefacts with AmigaRHQ sample. > > > > AmigaRHQ doesn't decode here: > > Because i have fix locally, I asked do you have player that plays > it without artefacts? Both xanim and iffanimplay shows artefacts. No, Amiga software plays this file with artefacts

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

On 2/20/16, Piotr Bandurski wrote: > Hi, > >> The player I have shows same artefacts with AmigaRHQ sample. > > AmigaRHQ doesn't decode here: Because i have fix locally, I asked do you have player that plays it without artefacts? Both xanim and iffanimplay shows artefacts. >>

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

Hi, > The player I have shows same artefacts with AmigaRHQ sample. AmigaRHQ doesn't decode here: D:\>ffmpegm -i AmigaRHQ -b:v 2000k out.avi ffmpeg version 3.0.git Copyright (c) 2000-2016 the FFmpeg developers built with gcc 4.6.2 (GCC) configuration: --disable-pthreads --disable-ffprobe

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

On Sat, 20 Feb 2016 17:41:40 + Derek Buitenhuis wrote: > > ffmpeg | branch: master | Michael Niedermayer | > > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] > > | committer: Michael Niedermayer > > [...] > > > +if (align

[FFmpeg-devel] [PATCH] Fixed issues preventing ffserver write_index and files_size from being set correctly which was breaking ffserver streaming.

Hi all My first patch/appearance in these parts so please go easy on me if I’ve done anything wrong! I discovered that ffserver streaming was broken (it seems like it has been since 20th November) and I opened a ticket for this (https://trac.ffmpeg.org/ticket/5250

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

On 2/20/16, Piotr Bandurski wrote: > Hi, > >> patch attached. > > Very nice. > > I tested it a bit. Here are some problematic samples (some files doesn't > decode, some decodes with artefacts): > > http://www.datafilehost.com/d/65783e33 The player I have shows same artefacts

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

> ffmpeg | branch: master | Michael Niedermayer | > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] | > committer: Michael Niedermayer [...] > +if (align < 4) { > +av_log(NULL, AV_LOG_ERROR, "Formats with a palette require a > minimum alignment

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

On Sun, 14 Feb 2016 16:25:02 +0100 Michael Niedermayer wrote: > This was suggested by wm4 and stefano. > After this patch using align=1 the size used by various functions would not > contain padding, while the palette would be aligned at align>1 > > This patch makes it

Re: [FFmpeg-devel] [PATCH] IFF ANIM support

Hi, > patch attached. Very nice. I tested it a bit. Here are some problematic samples (some files doesn't decode, some decodes with artefacts): http://www.datafilehost.com/d/65783e33 Regards ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] lavf/md5enc: New streammd5 muxer

Le duodi 2 ventôse, an CCXXIV, Moritz Barsnick a écrit : > md5enc is literally waiting to become "hashenc": Yes, of course. But changing the name or the default hash, both theoretically a good thing, would break user scripts, it can not be done lightly. Regards, -- Nicolas George

[FFmpeg-devel] [PATCH v3 2/4 v2] lavf/avienc: Add palette after BITMAPINFOHEADER

Forgot to remove an av_free(). -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 5f495f33fcef40bc7ee9dc07b440cd1da0824eeb Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 20 Feb 2016 14:40:22 +0100 Subject: [PATCH v3 2/4 v2] lavf/avienc: Add palette after

[FFmpeg-devel] [PATCH] IFF ANIM support

Hi, patch attached. From c7ff2e61fee53a9f02c66042c96ad622f8d6bf23 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 19 Feb 2016 14:35:53 +0100 Subject: [PATCH] IFF ANIM support Currently only most common compression implemented. Signed-off-by: Paul B Mahol

Re: [FFmpeg-devel] [PATCH] lavf/movenc: Add palette to video sample description

On 02/20/2016 01:06 PM, Mats Peterson wrote: This one is somewhat experimental so far, since it doesn't work well with ff_reshuffle_raw_rgb() for some reason. Perhaps you could look into the reason why, Michael. Remember that QuickTime strides are 2-byte aligned, not 4-byte aligned. Mats

Re: [FFmpeg-devel] [PATCH] lavf/md5enc: New streammd5 muxer

On Fri, Feb 19, 2016 at 17:51:00 +0100, Nicolas George wrote: > Changing the name of existing muxers is annoying, but for new features I > would like much better if we did not wave around the name of a hash function > that has known vulnerabilities since 20 years: "streamhash" would be an >

Re: [FFmpeg-devel] [PATCH v3 1/4] lavf/riffenc: Write space for palette

On 02/20/2016 01:02 PM, Mats Peterson wrote: ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Sorry for the trailing whitespace. ___ ffmpeg-devel

[FFmpeg-devel] [PATCH] lavf/movenc: Add palette to video sample description

This one is somewhat experimental so far, since it doesn't work well with ff_reshuffle_raw_rgb() for some reason. Perhaps you could look into the reason why, Michael. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 6a4a59d5e4cf08f14e77710234685239638d583d Mon Sep 17 00:00:00 2001

[FFmpeg-devel] [PATCH v3 4/4] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 4becd6f73a07ed25c9e78d4ee98f9773b3316963 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 20 Feb 2016 12:59:38 +0100 Subject: [PATCH v3 4/4] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a

[FFmpeg-devel] [PATCH v3 3/4] tests/ref/vsynth: Update 1 bpp files for pal8

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 9c8c847464ea6f0a6c14561601d8b47586160195 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 20 Feb 2016 12:59:05 +0100 Subject: [PATCH v3 3/4] tests/ref/vsynth: Update 1 bpp files for pal8 ---

[FFmpeg-devel] [PATCH v3 2/4] lavf/avienc: Add palette after BITMAPINFOHEADER

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From bf0d637d70cd92f80f2e689598aaa5fc137a8f93 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 20 Feb 2016 12:58:19 +0100 Subject: [PATCH v3 2/4] lavf/avienc: Add palette after BITMAPINFOHEADER ---

[FFmpeg-devel] [PATCH v3 1/4] lavf/riffenc: Write space for palette

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From c27daa7279b3ccaf93d43621cc22c5ceac5c406c Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 20 Feb 2016 12:57:50 +0100 Subject: [PATCH v3 1/4] lavf/riffenc: Write space for palette --- libavformat/riffenc.c |

Re: [FFmpeg-devel] [PATCH v2 1/4] lavf/riffenc: Write space for palette

On 02/20/2016 09:26 AM, Mats Peterson wrote: the codec priv_data must not be accessed from a muxer, its private to the codec OK, thanks for the information. Do you have a better way to store the palette offset? There aren't many places accessible from this function. I've solved it in

Re: [FFmpeg-devel] [PATCH] vp9: add superframe merging bitstream filter.

On Fri, 19 Feb 2016 16:28:44 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Fri, Feb 19, 2016 at 3:48 PM, Michael Niedermayer > wrote: > > > On Fri, Feb 19, 2016 at 02:43:48PM -0500, Ronald S. Bultje wrote: > > > Fixes ticket 4313. > > > --- > >

Re: [FFmpeg-devel] [PATCH v2 1/4] lavf/riffenc: Write space for palette

On 02/20/2016 03:40 AM, Michael Niedermayer wrote: On Sat, Feb 20, 2016 at 12:02:15AM +0100, Mats Peterson wrote: -- Mats Peterson http://matsp888.no-ip.org/~mats/ riffenc.c | 23 +++ 1 file changed, 19 insertions(+), 4 deletions(-)