Re: [FFmpeg-devel] Filters

2014-08-24 Thread Clément Bœsch
On Sun, Aug 24, 2014 at 12:06:46AM +0100, JULIAN GARDNER wrote: [...] I tried your suggestion of splitting the -map [vid[0:a] into -map [vid] -map [0:a] and I get the following error message Output with label '0:a' does not exist in any defined filter graph, or is already used elsewhere.

[FFmpeg-devel] [PATCH 4/5] x86: hevc_mc: convert to ssse3

2014-08-24 Thread Christophe Gisquet
The only sse4 instruction is pextrw, which is used on rather minor functions for small blocks. Therefore use whichever GPR is available to extract the output word. Before (sse4), for block_w == 6: 4627 decicycles in epel_uni, 16377 runs, 7 skips 7422 decicycles in epel_bi, 65501 runs, 35 skips

[FFmpeg-devel] [PATCH 2/5] x86: hevc_mc: correct unneeded use of SSE4 code

2014-08-24 Thread Christophe Gisquet
--- libavcodec/x86/hevc_mc.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/hevc_mc.asm b/libavcodec/x86/hevc_mc.asm index e2236ec..9ce6bd1 100644 --- a/libavcodec/x86/hevc_mc.asm +++ b/libavcodec/x86/hevc_mc.asm @@ -890,7 +890,7 @@ cglobal

Re: [FFmpeg-devel] [PATCH 2/5] x86: hevc_mc: correct unneeded use of SSE4 code

2014-08-24 Thread Michael Niedermayer
On Sun, Aug 24, 2014 at 08:46:31AM +, Christophe Gisquet wrote: --- libavcodec/x86/hevc_mc.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB When you are offended at any man's

Re: [FFmpeg-devel] [PATCH]Fix pnm encoding with bpc set

2014-08-24 Thread Christophe Gisquet
Hi, 2014-08-24 12:13 GMT+02:00 Carl Eugen Hoyos ceho...@ag.or.at: [...] -if( avctx-bits_per_raw_sample ) +if ( avctx-bits_per_raw_sample + av_pix_fmt_desc_get(avctx-pix_fmt)-comp[0].depth_minus1 7) maxdepth = (1 avctx-bits_per_raw_sample) - 1; So

Re: [FFmpeg-devel] [PATCH]Fix pnm encoding with bpc set

2014-08-24 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: Attached patch fixes pnm encoding for me if bits_per_raw_sample is set to a value different from the pix_fmt's native value. I should have mentioned that this was planned as a [RFC] since I am not completely convinced we want this but current

Re: [FFmpeg-devel] [PATCH] Add generate-doc.sh to generate makeinfo-based ffmpeg docs

2014-08-24 Thread Michael Niedermayer
On Sun, Jul 20, 2014 at 02:22:37PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- README| 12 +--- generate-doc.sh | 41 + src/template_doctitle | 1 - 3 files changed, 42 insertions(+), 12

Re: [FFmpeg-devel] [PATCH]Fix pnm encoding with bpc set

2014-08-24 Thread Carl Eugen Hoyos
Christophe Gisquet christophe.gisquet at gmail.com writes: Hi, 2014-08-24 12:48 GMT+02:00 Carl Eugen Hoyos cehoyos at ag.or.at: I think [PATCH 2/5] pnmenc: use bits_per_raw_sample is wrong then Definitely, I still wonder how you tested it. ljpeg 36bits from #2966 and having the

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Thilo Borgmann
Am 24.08.14 00:23, schrieb Michael Niedermayer: On Sat, Aug 23, 2014 at 09:24:33PM +0200, Clément Bœsch wrote: --- This is 100% untested and probably doesn't even compile. it does compile, but i dont have a ppc so i cant say if it would work I can test that later today. 'make test' is what

Re: [FFmpeg-devel] [PATCH]Fix pnm encoding with bpc set

2014-08-24 Thread Christophe Gisquet
Hi, I think a first point must be cleared, seeing your reaction to my pnmdec comment. Here' are the opinions: - Mine: bits_per_raw_sample should indicate the dynamics of the signal when it is different from the colorspace bitdepth - ffmpeg's and yours (if I'm not mistaken): the signal bitdepth

Re: [FFmpeg-devel] [PATCH]Fix pnm encoding with bpc set

2014-08-24 Thread Carl Eugen Hoyos
Christophe Gisquet christophe.gisquet at gmail.com writes: - ffmpeg's and yours (if I'm not mistaken): the signal bitdepth should always match the colorspace one, and bits_per_raw_sample should indicate the original bitdepth I believe that my opinion is mostly irrelevant but that FFmpeg's

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Carl Eugen Hoyos
Clément Bœsch u at pkh.me writes: +OBJS-$(CONFIG_PIXELUTILS) += ppc/pixelutils_init.o + ALTIVEC-OBJS += ppc/float_dsp_altivec.o \ +ALTIVEC-OBJS-$(CONFIG_PIXELUTILS) += ppc/pixelutils_altivec.o \ Something is wrong here, I suspect the \ have to be

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread wm4
On Sun, 24 Aug 2014 00:28:56 +0200 Clément Bœsch u...@pkh.me wrote: Hi, Kieran suggested tonight on #ffmpeg-devel to have a common mailing-list between the two projects to start communicating again in sane terms. The proposition would be a mailing-list where the 2 projects would send the

Re: [FFmpeg-devel] [RFC] [PATCH] proresenc_kostya: warn/reject on incorrect profile

2014-08-24 Thread Michael Niedermayer
On Mon, Aug 18, 2014 at 07:26:32PM +0200, Christophe Gisquet wrote: Hi, 2014-08-18 13:40 GMT+02:00 Carl Eugen Hoyos ceho...@ag.or.at: Christophe Gisquet christophe.gisquet at gmail.com writes: If a profile was set, the automatic setting should probably not overwrite it. (If this is

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread Lou Logan
On Sat, Aug 23, 2014, at 02:28 PM, Clément Bœsch wrote: On the technical side, we potentially need a neutral party to setup such mailing-list. Is anyone willing to do that? The neutral party should also be willing to regularly parse the pending moderator requests for valid messages from

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread compn
On Sun, 24 Aug 2014 08:57:04 -0800 Lou Logan l...@lrcd.com wrote: On Sat, Aug 23, 2014, at 02:28 PM, Clément Bœsch wrote: On the technical side, we potentially need a neutral party to setup such mailing-list. Is anyone willing to do that? The neutral party should also be willing to

Re: [FFmpeg-devel] [PATCH] libx264: fix -b_qfactor and -chromaoffset

2014-08-24 Thread Michael Niedermayer
On Sat, Aug 23, 2014 at 08:59:41PM +0800, Hii wrote: Currently -b_qfactor and -chromaoffset have no effect in libx264, the attached patch is an attempt to fix the issue. Move the corresponding lines after x264_param_default_preset() to prevent them being overwritten by it, make the two

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread Michael Niedermayer
On Sun, Aug 24, 2014 at 02:54:19PM +0200, wm4 wrote: On Sun, 24 Aug 2014 00:28:56 +0200 Clément Bœsch u...@pkh.me wrote: Hi, Kieran suggested tonight on #ffmpeg-devel to have a common mailing-list between the two projects to start communicating again in sane terms. The

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-24 Thread Clément Bœsch
On Wed, Aug 13, 2014 at 02:11:26PM -0300, James Almer wrote: On 13/08/14 1:52 PM, Reimar Döffinger wrote: On 13.08.2014, at 12:48, Ronald S. Bultje rsbul...@gmail.com wrote: Might it be useful to add some of those special samples to fate? Probably. The first 10 frames of the sample

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread wm4
On Sun, 24 Aug 2014 19:35:28 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Aug 24, 2014 at 02:54:19PM +0200, wm4 wrote: On Sun, 24 Aug 2014 00:28:56 +0200 Clément Bœsch u...@pkh.me wrote: Hi, Kieran suggested tonight on #ffmpeg-devel to have a common mailing-list

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-24 Thread Michael Niedermayer
On Sun, Aug 24, 2014 at 07:37:59PM +0200, Clément Bœsch wrote: On Wed, Aug 13, 2014 at 02:11:26PM -0300, James Almer wrote: On 13/08/14 1:52 PM, Reimar Döffinger wrote: On 13.08.2014, at 12:48, Ronald S. Bultje rsbul...@gmail.com wrote: Might it be useful to add some of those

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Check alpha_mode

2014-08-24 Thread Michael Niedermayer
On Wed, Aug 20, 2014 at 06:47:02PM +0200, Michael Niedermayer wrote: Fixes CID1231992 Suggested-by: Timothy Gu timothyg...@gmail.com Signed-off-by: Michael Niedermayer michae...@gmx.at applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB In a

Re: [FFmpeg-devel] [PATCH]Fix pnm encoding with bpc set

2014-08-24 Thread Reimar Döffinger
On 24.08.2014, at 14:02, Christophe Gisquet christophe.gisq...@gmail.com wrote: Hi, I think a first point must be cleared, seeing your reaction to my pnmdec comment. Here' are the opinions: - Mine: bits_per_raw_sample should indicate the dynamics of the signal when it is different from the