[FFmpeg-devel] [PATCH] lavf/avienc: Store palette at keyframes

Here is an experimental and much simpler solution that stores the last used palette at keyframes regardless. Maybe too simple, Michael, I don't know. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From f244806bf67595bf42d3449dccd3defd4433119d Mon Sep 17 00:00:00 2001 From: Mats

Re: [FFmpeg-devel] [PATCH] lavf/avienc: Store palette at keyframes

On 03/05/2016 11:23 PM, Mats Peterson wrote: On 03/05/2016 11:20 PM, Mats Peterson wrote: On 03/05/2016 11:00 PM, Mats Peterson wrote: Here is an experimental and much simpler solution that stores the last used palette at keyframes regardless. Maybe too simple, Michael, I don't know. This

Re: [FFmpeg-devel] [PATCH] avformat/dump: Fix context/level for payload dump

On Sat, Mar 05, 2016 at 09:10:00AM -0800, Mark Harris wrote: > Use the context and level specified to av_pkt_dump_log2(), > instead of panic level (0), for dumping packet payload. > --- > libavformat/dump.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] --

Re: [FFmpeg-devel] [PATCH] avformat/avienc: store palette at keyframes if it differs from the global one

On 03/05/2016 10:26 PM, Mats Peterson wrote: On 03/05/2016 10:13 PM, Mats Peterson wrote: On 03/05/2016 10:12 PM, Mats Peterson wrote: Doesn't seem to work quite as expected, Michael. Try ffmpeg -i toon.avi -vcodec copy out.avi. The destination file will still contain only two xxpc chunks.

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

> iam not sure zthis is still an issue as the last build is a few days > ago so maybe this is alread fixed but, theres a build failure on: > http://fatebeta.ffmpeg.org/log/x86_64-darwin-gcc-4.8/20160303011009/compile This was fixed by ecba35bbe305c022d59da2d7bf066e3325693c26.

Re: [FFmpeg-devel] [PATCH] avformat/avienc: store palette at keyframes if it differs from the global one

On 03/05/2016 10:13 PM, Mats Peterson wrote: On 03/05/2016 10:12 PM, Mats Peterson wrote: Doesn't seem to work quite as expected, Michael. Try ffmpeg -i toon.avi -vcodec copy out.avi. The destination file will still contain only two xxpc chunks. Mats Well, it works with -vcodec rawvideo,

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

On Wed, Mar 02, 2016 at 08:11:40PM +0800, Rick Kern wrote: > Autodetected by default. Encode using -codec:v h264_videotoolbox. > > Signed-off-by: Rick Kern > --- > MAINTAINERS |1 + > configure| 26 +- > libavcodec/Makefile |

[FFmpeg-devel] [PATCH 2/2] version.sh: Add next release to the version

Also make consistent the fallback versions. This patch uses a RELEASE-based approach rather than a tag-based one. There are several reasons for this. First, the -dev tags never convey more information than RELEASE. Second, RELEASE can provide a version even without Git. The new script is

Re: [FFmpeg-devel] [PATCH] lavf/avienc: Store palette at keyframes

On 03/05/2016 11:20 PM, Mats Peterson wrote: On 03/05/2016 11:00 PM, Mats Peterson wrote: Here is an experimental and much simpler solution that stores the last used palette at keyframes regardless. Maybe too simple, Michael, I don't know. This simplicity leads to storing an xxpc chunk with

Re: [FFmpeg-devel] [PATCH] lavf/avienc: Store palette at keyframes

Mats Peterson skrev: (5 mars 2016 23:31:40 CET) >On 03/05/2016 11:23 PM, Mats Peterson wrote: >> On 03/05/2016 11:20 PM, Mats Peterson wrote: >>> On 03/05/2016 11:00 PM, Mats Peterson wrote: Here is an experimental and much simpler solution that stores the

Re: [FFmpeg-devel] [PATCH] avformat/avienc: store palette at keyframes if it differs from the global one

On 03/05/2016 10:12 PM, Mats Peterson wrote: Doesn't seem to work quite as expected, Michael. Try ffmpeg -i toon.avi -vcodec copy out.avi. The destination file will still contain only two xxpc chunks. Mats Well, it works with -vcodec rawvideo, but not -vcodec copy. Mats -- Mats Peterson

[FFmpeg-devel] [PATCH] [RFC] proof-of-concept minimal inflate.

FFmpeg currently lacks a fallback inflate algorithm when zlib is not available. We have a lot of infrastructure for it already available though, like VLC code reading (though only in libavcodec, not libavutil). This is a hackish quick-and-dirty version. It certainly is not optimized, and I would

[FFmpeg-devel] [PATCH 0/2] New versioning (was: version.sh: Always use latest tag for generated version number)

This patch set implements the versioning algorithm I recently proposed. The first patch addresses an already existing problem with version.sh. The second is the one actually implementing the change. To Michael and all release makers: after this patchset when making a release in a release branch,

[FFmpeg-devel] [PATCH 1/2] version.sh: Fix Git directory

This is necessary for cases where the FFmpeg directory does not have Git metadata but the parent directories do. See e.g. https://github.com/mxe/mxe/issues/1219. --- version.sh | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/version.sh b/version.sh index

[FFmpeg-devel] [PATCH] avformat/avienc: store palette at keyframes if it differs from the global one

--- libavformat/avienc.c |9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavformat/avienc.c b/libavformat/avienc.c index 0cfffb7..8804ece 100644 --- a/libavformat/avienc.c +++ b/libavformat/avienc.c @@ -84,6 +84,7 @@ typedef struct AVIStream { uint32_t

Re: [FFmpeg-devel] [PATCH]lavf/img2dec: xbm autodetection

On Sat, Mar 05, 2016 at 07:29:01PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Not sure how useful this patch is. > > Please comment, Carl Eugen > Makefile |1 + > allformats.c |1 + > img2dec.c|8 > 3 files changed, 10 insertions(+) >

[FFmpeg-devel] [PATCH v2] avfilter/vf_bwdif: add x86 SIMD

This new patch adds x86 SIMD support up to 12 bit. Please comment. Signed-off-by: Thomas Mundt --- libavfilter/bwdif.h | 72 +++ libavfilter/vf_bwdif.c | 69 +++ libavfilter/x86/Makefile| 2 + libavfilter/x86/vf_bwdif.asm|

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

On 03/05/2016 05:54 PM, Reimar Döffinger wrote: On Sat, Mar 05, 2016 at 02:32:56PM +0100, Mats Peterson wrote: On 03/05/2016 02:24 PM, Mats Peterson wrote: The original toon.avi doesn't contain xxpc chunks at keyframes either, if that can be of any comfort. It only contains two xxpc chunks,

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

On 03/05/2016 06:14 PM, Michael Niedermayer wrote: On Sat, Mar 05, 2016 at 05:54:05PM +0100, Reimar Döffinger wrote: On Sat, Mar 05, 2016 at 02:32:56PM +0100, Mats Peterson wrote: On 03/05/2016 02:24 PM, Mats Peterson wrote: The original toon.avi doesn't contain xxpc chunks at keyframes

Re: [FFmpeg-devel] [PATCH] avformat/avienc: store palette at keyframes if it differs from the global one

On 03/05/2016 08:44 PM, Michael Niedermayer wrote: --- libavformat/avienc.c |9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavformat/avienc.c b/libavformat/avienc.c index 0cfffb7..8804ece 100644 --- a/libavformat/avienc.c +++ b/libavformat/avienc.c @@ -84,6

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

On Sat, Mar 05, 2016 at 02:32:56PM +0100, Mats Peterson wrote: > On 03/05/2016 02:24 PM, Mats Peterson wrote: > > > >The original toon.avi doesn't contain xxpc chunks at keyframes either, > >if that can be of any comfort. It only contains two xxpc chunks, at > >that. If you're able to invent

Re: [FFmpeg-devel] [PATCH] avcodec: try to document timebase and framerate a bit more

On Sat, Mar 05, 2016 at 01:41:32PM +0100, Michael Niedermayer wrote: > + * Note, for variable framerate material this has undefined meaning > currently > + * and is not set to the actual framerate nor {0,1} but can be set to > + * 1/timebase (FIXME) the code in some parts assumes

[FFmpeg-devel] [PATCH] avformat/dump: Fix context/level for payload dump

Use the context and level specified to av_pkt_dump_log2(), instead of panic level (0), for dumping packet payload. --- libavformat/dump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dump.c b/libavformat/dump.c index 9e7c12b..86bb82d 100644 ---

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

On Sat, Mar 05, 2016 at 12:21:14PM +0100, Mats Peterson wrote: > I've noticed that avienc.c is close to useless for use with stdout. > It only checks pb->seekable in a few places, and for example the > ff_end_tag() function uses avio_seek(), but that one is used in > several places where there is

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

On Thu, Mar 03, 2016 at 12:35:36AM +0100, Mats Peterson wrote: > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > riffenc.c | 20 +++- > 1 file changed, 11 insertions(+), 9 deletions(-) > 70ca679c14ab31934806895eea5f10f633aeb778 >

[FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

Signed-off-by: Michael Niedermayer --- libavformat/avienc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavformat/avienc.c b/libavformat/avienc.c index 0cfffb7..357dd34 100644 --- a/libavformat/avienc.c +++ b/libavformat/avienc.c @@ -306,8

Re: [FFmpeg-devel] [PATCH] avcodec: try to document timebase and framerate a bit more

On Sat, Mar 05, 2016 at 06:01:11PM +0100, Reimar Döffinger wrote: > On Sat, Mar 05, 2016 at 01:41:32PM +0100, Michael Niedermayer wrote: > > + * Note, for variable framerate material this has undefined meaning > > currently > > + * and is not set to the actual framerate nor {0,1} but can

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

On 03/05/2016 07:33 AM, Mats Peterson wrote: Only copy/compare enough bytes for the bit depth in question. Here's another good file with xxpc chunks that you can try. It is encoded with Microsoft Video 1 (CRAM). https://samples.ffmpeg.org/avi/palette_change/toon.avi Mats -- Mats Peterson

Re: [FFmpeg-devel] Regression maybe

Ive now got the bisect down to the problem appearing between 75753 and 75759 but when i build the versions inbetween i get avconv build and not ffmpeg This is how the bisect went78890 BAD72615 GOOD75753 GOOD77321 BAD..75759 BADAVCONV built joolz  Joolz On Friday, 4 March 2016, 0:17,

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

On 03/05/2016 09:08 AM, Mats Peterson wrote: On 03/05/2016 07:33 AM, Mats Peterson wrote: Only copy/compare enough bytes for the bit depth in question. Here's another good file with xxpc chunks that you can try. It is encoded with Microsoft Video 1 (CRAM).

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

On Sat, Mar 05, 2016 at 04:48:21AM +0100, Mats Peterson wrote: > On 03/05/2016 04:43 AM, Mats Peterson wrote: > > > > > > > >___ > >ffmpeg-devel mailing list > >ffmpeg-devel@ffmpeg.org > >http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > This one

Re: [FFmpeg-devel] [PATCH] version.sh: Always use latest tag for generated version number

Timothy Gu gmail.com> writes: > The versioning scheme is supposed to make people's life easier. The patch would make my life more difficult . Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] version.sh: Always use latest tag for generated version number

On Sat, Mar 5, 2016 at 11:26 AM, Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > >> The versioning scheme is supposed to make people's life easier. > > The patch would make my life more difficult . > If you can't even explain why, then I wouldn't expect many people

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

I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several places where there is no check for a seekable stream. Anyway, this version adds the

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

On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several places where there is no check for a seekable

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

On Fri, Mar 04, 2016 at 04:19:18PM -0800, Sergey Volk wrote: > Ok, something like this for now, then? your original patch contained a nice commit message, this one doesnt > I'm new to ffmpeg development. When is the next version bump going to happen? you can make changes at the next bump by

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

On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several places where there is no check for a seekable

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

On 03/05/2016 12:41 PM, Mats Peterson wrote: On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several

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

On 03/05/2016 01:10 PM, Mats Peterson wrote: On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several

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

On 03/05/2016 01:18 PM, Mats Peterson wrote: On 03/05/2016 01:10 PM, Mats Peterson wrote: On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses

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

On 03/05/2016 02:24 PM, Mats Peterson wrote: On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several

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

On 03/05/2016 12:21 PM, Mats Peterson wrote: I've noticed that avienc.c is close to useless for use with stdout. It only checks pb->seekable in a few places, and for example the ff_end_tag() function uses avio_seek(), but that one is used in several places where there is no check for a seekable

[FFmpeg-devel] [PATCH] avcodec: try to document timebase and framerate a bit more

Hi see the 2 attached patches ISP still has probems with git send email here -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is dangerous to be right in matters on which the established authorities are wrong. -- Voltaire From

[FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

Hi! I see the following issue here with extractplanes: If the input file has a 16bit little endian pix_fmt not supported by extractplanes (gbra12le), the filter system has to choose a compatible pix_fmt supported by the filter and chooses rgba64be here. The extractplanes filter returns

Re: [FFmpeg-devel] [PATCH]lavf/img2dec: Use jpeg constants in jpeg_probe()

On Sat, Mar 05, 2016 at 07:26:36PM +0100, Carl Eugen Hoyos wrote: > Hi! > > I believe attached patch makes the jpeg probe function more readable. > > Please comment, Carl Eugen > img2dec.c | 73 > +++--- > 1 file changed, 37

[FFmpeg-devel] [PATCH] avcodec/cfhd: Fixes cfhd_odd.mov which has a resolution of 496x241

In this case container width/height is better however. Thanks to koda for the sample --- libavcodec/cfhd.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index e37bef0..98f16d3 100644 --- a/libavcodec/cfhd.c +++ b/libavcodec/cfhd.c @@ -467,6 +467,9 @@

[FFmpeg-devel] [PATCH]lavf/img2dec: Use jpeg constants in jpeg_probe()

Hi! I believe attached patch makes the jpeg probe function more readable. Please comment, Carl Eugendiff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index 019793f..fe0e346 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -33,6 +33,7 @@ #include "avio_internal.h"

[FFmpeg-devel] [PATCH]lavf/img2dec: xbm autodetection

Hi! Not sure how useful this patch is. Please comment, Carl Eugen diff --git a/libavformat/Makefile b/libavformat/Makefile index dc931d9..8981e08 100644 --- a/libavformat/Makefile +++ b/libavformat/Makefile @@ -227,6 +227,7 @@ OBJS-$(CONFIG_IMAGE_SGI_PIPE_DEMUXER) += img2dec.o img2.o

Re: [FFmpeg-devel] [libav-devel] [PATCH] avcodec/cfhd: Fixes cfhd_odd.mov which has a resolution of 496x241

On Sat, Mar 5, 2016 at 1:06 PM, Kieran Kunhya wrote: > In this case container width/height is better however. > Thanks to koda for the sample > --- > libavcodec/cfhd.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c > index

Re: [FFmpeg-devel] [PATCH] avcodec: try to document timebase and framerate a bit more

On Sat, Mar 05, 2016 at 06:56:30PM +0100, Michael Niedermayer wrote: > On Sat, Mar 05, 2016 at 06:01:11PM +0100, Reimar Döffinger wrote: > > On Sat, Mar 05, 2016 at 01:41:32PM +0100, Michael Niedermayer wrote: > > > + * Note, for variable framerate material this has undefined meaning > > >