Re: [FFmpeg-devel] [PATCH]lavc/libvpx: Fix high-bitrate colour spaces

On Thu, Feb 18, 2016 at 11:40 PM, Carl Eugen Hoyos wrote: > On Friday 19 February 2016 07:59:37 am James Zern wrote: > >> The decoder won't change when built on a big-endian platform, so I don't >> think this is correct. > > New patch attached. > Sorry, I meant all of it, high

Re: [FFmpeg-devel] [PATCH]lavc/libvpx: Fix high-bitrate colour spaces

On Friday 19 February 2016 07:59:37 am James Zern wrote: > The decoder won't change when built on a big-endian platform, so I don't > think this is correct. New patch attached. Thank you, Carl Eugen diff --git a/libavcodec/libvpx.c b/libavcodec/libvpx.c index a60d186..1e7010b 100644 ---

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Thu, Feb 18, 2016 at 3:02 PM, Hendrik Leppkes wrote: > On Thu, Feb 18, 2016 at 11:59 PM, Carl Eugen Hoyos wrote: >> On Thursday 18 February 2016 09:40:01 pm Nicolas George wrote: >>> Initial patch by Carl Eugen Hoyos. >>> >>> Fix trac ticket #5249. >> >>

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Thu, Feb 18, 2016 at 3:34 PM, Carl Eugen Hoyos wrote: >> Alternatively we could bump the minimum required version and remove most of >> the existing checks. 1.3.0 is pretty old by now and it would let us clean >> these files. > > I would prefer if this could be done

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Thu, Feb 18, 2016 at 3:10 PM, James Almer wrote: > On 2/18/2016 7:59 PM, Carl Eugen Hoyos wrote: >> +case AV_PIX_FMT_GBRP: >> +ctx->vpx_cs = VPX_CS_SRGB; > > This is only available since libvpx 1.4.0, so you need to wrap all this new > code inside some

Re: [FFmpeg-devel] [PATCH]lavc/libvpx: Fix high-bitrate colour spaces

Hi, On Thu, Feb 18, 2016 at 3:30 PM, Carl Eugen Hoyos wrote: > Please comment, Carl Eugen > diff --git a/libavcodec/libvpx.c b/libavcodec/libvpx.c > index a60d186..1e7010b 100644 > --- a/libavcodec/libvpx.c > +++ b/libavcodec/libvpx.c > @@ -46,14 +46,14 @@ static const enum

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

--- configure | 14 +++--- libavdevice/jack.c | 12 2 files changed, 23 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 287896f..f2f7c3d 100755 --- a/configure +++ b/configure @@ -1732,6 +1732,7 @@ BUILTIN_LIST=" mm_empty rdtsc

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Friday 19 February 2016 12:18:06 am James Almer wrote: > On 2/18/2016 8:10 PM, James Almer wrote: > > On 2/18/2016 7:59 PM, Carl Eugen Hoyos wrote: > >> +case AV_PIX_FMT_GBRP: > >> +ctx->vpx_cs = VPX_CS_SRGB; > > > > This is only available since libvpx 1.4.0, so you need to wrap all

[FFmpeg-devel] [PATCH] checkasm: fix dependencies for vf_blend tests

They will now compile if avcodec is disabled Signed-off-by: James Almer --- tests/checkasm/Makefile | 6 +- tests/checkasm/checkasm.c | 8 +--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile index

[FFmpeg-devel] [PATCH]lavc/libvpx: Fix high-bitrate colour spaces

Please comment, Carl Eugen diff --git a/libavcodec/libvpx.c b/libavcodec/libvpx.c index a60d186..1e7010b 100644 --- a/libavcodec/libvpx.c +++ b/libavcodec/libvpx.c @@ -46,14 +46,14 @@ static const enum AVPixelFormat vp9_pix_fmts_highbd[] = { AV_PIX_FMT_YUV422P, AV_PIX_FMT_YUV440P,

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On 2/18/2016 8:10 PM, James Almer wrote: > On 2/18/2016 7:59 PM, Carl Eugen Hoyos wrote: >> +case AV_PIX_FMT_GBRP: >> +ctx->vpx_cs = VPX_CS_SRGB; > > This is only available since libvpx 1.4.0, so you need to wrap all this new > code inside some preprocessor checks. I think

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Friday 19 February 2016 12:02:59 am Hendrik Leppkes wrote: > On Thu, Feb 18, 2016 at 11:59 PM, Carl Eugen Hoyos wrote: > > On Thursday 18 February 2016 09:40:01 pm Nicolas George wrote: > >> Initial patch by Carl Eugen Hoyos. > >> > >> Fix trac ticket #5249. > > > > Patch

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On 2/18/2016 7:59 PM, Carl Eugen Hoyos wrote: > +case AV_PIX_FMT_GBRP: > +ctx->vpx_cs = VPX_CS_SRGB; This is only available since libvpx 1.4.0, so you need to wrap all this new code inside some preprocessor checks. I think VPX_IMAGE_ABI_VERSION >= 3 should do it, but better wait for

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Thu, Feb 18, 2016 at 11:59 PM, Carl Eugen Hoyos wrote: > On Thursday 18 February 2016 09:40:01 pm Nicolas George wrote: >> Initial patch by Carl Eugen Hoyos. >> >> Fix trac ticket #5249. > > Patch with (automatic) support for 8, 10 and 12 bit gbr attached. > > Please comment,

Re: [FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

On Thursday 18 February 2016 09:40:01 pm Nicolas George wrote: > Initial patch by Carl Eugen Hoyos. > > Fix trac ticket #5249. Patch with (automatic) support for 8, 10 and 12 bit gbr attached. Please comment, Carl Eugen diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index

[FFmpeg-devel] avcodec/cdxl: add support for videos with chunky format

Hi, patch attached. From 7f8779b9be131414a073d051725a5e0aac19d466 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Thu, 18 Feb 2016 23:47:39 +0100 Subject: [PATCH] avcodec/cdxl: add support for videos with chunky format Signed-off-by: Paul B Mahol ---

Re: [FFmpeg-devel] GSoC 2016 own ideas

On Donnerstag, 18. Februar 2016 21:02:09 CET wm4 wrote: > On Thu, 18 Feb 2016 20:41:29 +0100 > > Gerion Entrup wrote: > > On Donnerstag, 18. Februar 2016 20:10:47 CET wm4 wrote: > > > On Thu, 18 Feb 2016 18:27:45 +0100 > > > > > > Gerion Entrup

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On 02/18/2016 11:04 PM, Michael Niedermayer wrote: On Thu, Feb 18, 2016 at 10:40:42PM +0100, Mats Peterson wrote: On 02/18/2016 10:27 PM, Mats Peterson wrote: On 02/18/2016 09:14 PM, Michael Niedermayer wrote: This can be made more efficient, but first and the main goal of this change is to

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On Thu, Feb 18, 2016 at 10:40:42PM +0100, Mats Peterson wrote: > On 02/18/2016 10:27 PM, Mats Peterson wrote: > >On 02/18/2016 09:14 PM, Michael Niedermayer wrote: > >>This can be made more efficient, but first and the main goal of this > >>change is to > >>store it at all > >Great, Michael. I'll

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On 02/18/2016 10:53 PM, Mats Peterson wrote: FFplay plays these files fine, but Windows XP Media Player needs that palette after the BITMAPINFOHEADER. It doesn't seem to care about those 'xxpc' chunks. Interesting, since the Media Player ought to support everything in the AVI specs. That is

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On 02/18/2016 10:40 PM, Mats Peterson wrote: On 02/18/2016 10:27 PM, Mats Peterson wrote: On 02/18/2016 09:14 PM, Michael Niedermayer wrote: This can be made more efficient, but first and the main goal of this change is to store it at all Great, Michael. I'll try it out. Mats I notice

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On 02/18/2016 10:39 PM, Moritz Barsnick wrote: On Thu, Feb 18, 2016 at 21:14:55 +0100, Michael Niedermayer wrote: + * no zerp if a new packet was allocated and ppkt has to be freed "no zerp"? Cute. Probably "non-zero" That's a new term ;)

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On 02/18/2016 10:27 PM, Mats Peterson wrote: On 02/18/2016 09:14 PM, Michael Niedermayer wrote: This can be made more efficient, but first and the main goal of this change is to store it at all Great, Michael. I'll try it out. Mats I notice that the palette inclusion is only made by using

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On Thu, Feb 18, 2016 at 21:14:55 +0100, Michael Niedermayer wrote: > + * no zerp if a new packet was allocated and ppkt has to be freed "no zerp"? Cute. Probably "non-zero" Moritz ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

On 02/18/2016 09:14 PM, Michael Niedermayer wrote: This can be made more efficient, but first and the main goal of this change is to store it at all Great, Michael. I'll try it out. Mats ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avfilter: add firequalizer filter

> +center = s->fir_len / 2; > + > +for (k = 0; k <= center; k++) { > +double u = k * (M_PI/center); > +double win; > +switch (s->wfunc) { > +case WFUNC_RECTANGULAR: > +win = 1.0; > +break; > +

[FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

Initial patch by Carl Eugen Hoyos. Fix trac ticket #5249. Signed-off-by: Nicolas George --- libavcodec/libvpx.c| 1 + libavcodec/libvpxenc.c | 1 + 2 files changed, 2 insertions(+) This is probably quite incomplete (high bit depth?), and there should probably be a

Re: [FFmpeg-devel] pngdsp add bytes padding

On Thu, Feb 18, 2016 at 05:53:46PM +0100, Clément Bœsch wrote: > On Thu, Feb 18, 2016 at 12:42:59PM +0100, Michael Niedermayer wrote: > > On Thu, Feb 18, 2016 at 11:36:18AM +0100, Clément Bœsch wrote: > > > Hi, > > > > > > While working on PNG DSP, I realized the filter up was doing extra > > >

Re: [FFmpeg-devel] [PATCH 3/3] avformat/avienc: Use avi_write_packet_internal() to store raw rgb in a more spec compliant way

On Wed, Feb 17, 2016 at 02:29:08PM +0100, Mats Peterson wrote: > On 02/17/2016 02:16 PM, Mats Peterson wrote: > >On 02/17/2016 02:02 PM, Michael Niedermayer wrote: > >>Please test this with various players if you can > > > >The only player that I have here that can play raw files properly is >

Re: [FFmpeg-devel] [PATCH] avfilter: add BobWeaver deinterlacing filter

>>> Paul B Mahol schrieb am Mi, 17.2.2016: > If nobody plans to comment on code I will apply this soon. I´ve seen you applied it. Thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

This can be made more efficient, but first and the main goal of this change is to store it at all Signed-off-by: Michael Niedermayer --- libavformat/avienc.c | 19 +++ libavformat/internal.h |7 ++- libavformat/rawutils.c |5 +++-- 3

Re: [FFmpeg-devel] GSoC 2016 own ideas

On Thu, 18 Feb 2016 20:41:29 +0100 Gerion Entrup wrote: > On Donnerstag, 18. Februar 2016 20:10:47 CET wm4 wrote: > > On Thu, 18 Feb 2016 18:27:45 +0100 > > Gerion Entrup wrote: > > > > > Good day, > > > > > > I'm a master student

Re: [FFmpeg-devel] GSoC 2016 own ideas

Le decadi 30 pluviôse, an CCXXIV, Gerion Entrup a écrit : > A question for that. If I get it right, ffplay use SDL directly. It is wanted > to "swap it out" to some kind of libavdevice and then use this device out of > ffplay? You will probably get a different answer from each developer, but my

Re: [FFmpeg-devel] GSoC 2016 own ideas

On 2/18/16, Gerion Entrup wrote: >> > - merge of ffmpeg and ffplay (call ffmpeg without output reacts like > ffplay). Maybe this idea is fairly naive. >> > >> >> That sounds hard, and maybe not really what we'd want anyway. (Updating >> ffplay with SDL2 or something

Re: [FFmpeg-devel] GSoC 2016 own ideas

On 2/18/16, Gerion Entrup wrote: > On Donnerstag, 18. Februar 2016 18:53:45 CET Paul B Mahol wrote: >> Dana 18. 2. 2016. 18:27 osoba "Gerion Entrup" >> >> napisala je: >> > >> > Good day, >> > >> > I'm a master student and long term

Re: [FFmpeg-devel] GSoC 2016 own ideas

On Donnerstag, 18. Februar 2016 20:10:47 CET wm4 wrote: > On Thu, 18 Feb 2016 18:27:45 +0100 > Gerion Entrup wrote: > > > Good day, > > > > I'm a master student and long term FFmpeg-user. I want to participate in the GSoC 2016 for FFmpeg. The reason, I write this,

Re: [FFmpeg-devel] GSoC 2016 own ideas

On Donnerstag, 18. Februar 2016 18:53:45 CET Paul B Mahol wrote: > Dana 18. 2. 2016. 18:27 osoba "Gerion Entrup" > napisala je: > > > > Good day, > > > > I'm a master student and long term FFmpeg-user. I want to participate in > the GSoC 2016 for FFmpeg. The reason,

Re: [FFmpeg-devel] GSoC 2016 own ideas

On Thu, 18 Feb 2016 18:27:45 +0100 Gerion Entrup wrote: > Good day, > > I'm a master student and long term FFmpeg-user. I want to participate in the > GSoC 2016 for FFmpeg. The reason, I write this, is that I want to suggest > some own ideas. It could be, that

Re: [FFmpeg-devel] GSoC 2016 own ideas

Dana 18. 2. 2016. 18:27 osoba "Gerion Entrup" napisala je: > > Good day, > > I'm a master student and long term FFmpeg-user. I want to participate in the GSoC 2016 for FFmpeg. The reason, I write this, is that I want to suggest some own ideas. It could be, that some

[FFmpeg-devel] GSoC 2016 own ideas

Good day, I'm a master student and long term FFmpeg-user. I want to participate in the GSoC 2016 for FFmpeg. The reason, I write this, is that I want to suggest some own ideas. It could be, that some of the mentioned things are wrong (because FFmpeg could do this already or it it much more

Re: [FFmpeg-devel] pngdsp add bytes padding

On Thu, Feb 18, 2016 at 12:42:59PM +0100, Michael Niedermayer wrote: > On Thu, Feb 18, 2016 at 11:36:18AM +0100, Clément Bœsch wrote: > > Hi, > > > > While working on PNG DSP, I realized the filter up was doing extra > > overflow checks on the bytes non multiple of the long word size. This > >

Re: [FFmpeg-devel] [PATCH] lavf/options_table: mark use_wallclock_as_timestamps as boolean

On Thu, Feb 18, 2016 at 02:47:10PM +0100, Moritz Barsnick wrote: > It is only used in a boolean context. Also clarify its documentation. > > Signed-off-by: Moritz Barsnick > --- > doc/formats.texi| 2 +- > libavformat/options_table.h | 2 +- > 2 files changed, 2

Re: [FFmpeg-devel] [PATCH 0/4] Revert recent configure changed

On Thu, Feb 18, 2016 at 04:12:25PM +0100, Michael Niedermayer wrote: > On Thu, Feb 18, 2016 at 04:10:22PM +0100, Michael Niedermayer wrote: > > On Thu, Feb 18, 2016 at 03:49:24PM +0100, Hendrik Leppkes wrote: > > > On Thu, Feb 18, 2016 at 3:35 PM, Michael Niedermayer > > >

[FFmpeg-devel] [PATCH 2/4] Revert "Merge commit 'a2bb771a3cded8a05137c0effb34f61a2bc78e22'"

This reverts commit e8ebcb0034c5d4e5df8ff407a5c28d8c53823236, reversing changes made to 3bff005be8ea213c23160ee0ac286634a80a10e1. --- configure | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/configure b/configure index 40821e1..6562e70 100755 --- a/configure +++

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

On Thu, Feb 18, 2016 at 11:12:19AM -0300, James Almer wrote: > On 2/18/2016 10:42 AM, Clément Bœsch wrote: > > On Thu, Feb 18, 2016 at 02:40:06PM +0100, Clément Bœsch wrote: > >> On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: > >>> Clément Bœsch pkh.me> writes: > >>> >

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

On 2/18/2016 10:42 AM, Clément Bœsch wrote: > On Thu, Feb 18, 2016 at 02:40:06PM +0100, Clément Bœsch wrote: >> On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: >>> Clément Bœsch pkh.me> writes: >>> It's not tested on real hardware (qemu only), so I have no idea about

[FFmpeg-devel] [PATCH] lavf/options_table: mark use_wallclock_as_timestamps as boolean

It was probably forgotten to convert this to AV_OPT_TYPE_BOOL, because that's how it is actually used. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] lavf/options_table: mark use_wallclock_as_timestamps as boolean

It is only used in a boolean context. Also clarify its documentation. Signed-off-by: Moritz Barsnick --- doc/formats.texi| 2 +- libavformat/options_table.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/formats.texi b/doc/formats.texi

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

On Thu, Feb 18, 2016 at 02:40:06PM +0100, Clément Bœsch wrote: > On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: > > Clément Bœsch pkh.me> writes: > > > > > It's not tested on real hardware (qemu only), so I have > > > no idea about performance. > > > > I can test this on

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > It's not tested on real hardware (qemu only), so I have > > no idea about performance. > > I can test this on real hardware if you can provide > (exact) command lines. > for ifmt in nv12

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

Clément Bœsch pkh.me> writes: > It's not tested on real hardware (qemu only), so I have > no idea about performance. I can test this on real hardware if you can provide (exact) command lines. Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/3] avformat/avienc: Split avi_write_packet_internal() out

On Wed, Feb 17, 2016 at 02:02:20PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/avienc.c | 19 +-- > 1 file changed, 17 insertions(+), 2 deletions(-) applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/3] avformat: add ff_reshuffle_raw_rgb()

On Wed, Feb 17, 2016 at 04:28:22PM +0100, Nicolas George wrote: > Le nonidi 29 pluviôse, an CCXXIV, Michael Niedermayer a écrit : > > +int ff_reshuffle_raw_rgb(AVFormatContext *s, AVPacket **ppkt, > > AVCodecContext *enc, int expected_stride); > > A little doxy to tell us what it actually does

Re: [FFmpeg-devel] Question about "make distclean" after "git pull"

On 02/18/2016 12:45 PM, Nicolas George wrote: L'octidi 28 pluviôse, an CCXXIV, Nicolas George a écrit : There may have been cases where forgetting distclean would cause bugs more subtle than that, but I do not remember the

Re: [FFmpeg-devel] Question about "make distclean" after "git pull"

L'octidi 28 pluviôse, an CCXXIV, Nicolas George a écrit : > There may > have been cases where forgetting distclean would cause bugs more subtle than > that, but I do not remember the specifics. Just found one: .d files contained the

Re: [FFmpeg-devel] pngdsp add bytes padding

On Thu, Feb 18, 2016 at 11:36:18AM +0100, Clément Bœsch wrote: > Hi, > > While working on PNG DSP, I realized the filter up was doing extra > overflow checks on the bytes non multiple of the long word size. This > logic is present in both C and x86 ASM. Here is the current C code for > reference:

[FFmpeg-devel] pngdsp add bytes padding

Hi, While working on PNG DSP, I realized the filter up was doing extra overflow checks on the bytes non multiple of the long word size. This logic is present in both C and x86 ASM. Here is the current C code for reference: -- 8< // 0x7f7f7f7f or 0x7f7f7f7f7f7f7f7f