Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-02-08 Thread Clément Bœsch
On Tue, Jan 20, 2015 at 09:58:33PM +0100, Reimar Döffinger wrote: On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: -echo scale=2; v = $1 - $2; if (v 0) v = -v; if (v $3) r = 1; r | bc +awk BEGIN { v=$1-$2; printf \%d\\n\, ((v0?v:-v) $3) ? 1 : 0 } I'd suggest

Re: [FFmpeg-devel] [PATCH]Read CbYCr dpx files

2015-02-08 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: patch LGTM The patch was merged. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-02-08 Thread Clément Bœsch
On Sun, Feb 08, 2015 at 10:27:14AM +0100, Clément Bœsch wrote: On Tue, Jan 20, 2015 at 09:58:33PM +0100, Reimar Döffinger wrote: On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: -echo scale=2; v = $1 - $2; if (v 0) v = -v; if (v $3) r = 1; r | bc +awk BEGIN {

Re: [FFmpeg-devel] hevc : support deinterlacing inside the decoder

2015-02-08 Thread Carl Eugen Hoyos
Mickaël Raulet mraulet at insa-rennes.fr writes: As we can consider, we won't have 4k interlaced content, copying a field into a frame should be ok. This is what has been done in this implementation. Do you have a sample? I am only interested in testing this. Thank you, Carl Eugen

Re: [FFmpeg-devel] [PATCH]Enable pic on OSX for x86_64

2015-02-08 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: On Sat, Feb 07, 2015 at 08:27:49PM +, Carl Eugen Hoyos wrote: Ronald S. Bultje rsbultje at gmail.com writes: I'm not sure we should ever set -mdynamic-no-pic on Mac x86-64? skiping the -mdynamic-no-pic check and addition on

[FFmpeg-devel] [PATCH] x86/doc/Makefile: DBG=1 to preprocess external asm

2015-02-08 Thread Christophe Gisquet
Suggestions for more efficient make rules, clearer documentation and a better macro than the generic 'DBG' welcome. -- Christophe From 8e2adea82a4ec440744b701716a93e6ecaf211d6 Mon Sep 17 00:00:00 2001 From: Christophe Gisquet christophe.gisq...@gmail.com Date: Sun, 8 Feb 2015 12:18:27 +0100

Re: [FFmpeg-devel] [PATCH] x86/doc/Makefile: DBG=1 to preprocess external asm

2015-02-08 Thread Christophe Gisquet
Hi, 2015-02-08 14:07 GMT+01:00 Carl Eugen Hoyos ceho...@ag.or.at: Doesn't this also need an update for make clean? Right, here's an updated one, also improving dependency generation and allowing more genering (for now) code. -- Christophe From 6795382920c90fdf7e9cef5031627534c9df15b3 Mon Sep

Re: [FFmpeg-devel] [PATCH] x86inc: also warn on sse2 instruction

2015-02-08 Thread Christophe Gisquet
Hi, 2015-02-08 14:28 GMT+01:00 Ronald S. Bultje rsbul...@gmail.com: Cool! Do you know if it catches all assembly types now (e.g. does it catch mmxext in mmx functions)? Or is it =sse only? Good thing that you asked, because I attached an earlier version... So I think what you ask was checked

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/srtdec: attempt to correct SubRip positioning

2015-02-08 Thread Clément Bœsch
On Sun, Feb 08, 2015 at 12:40:19PM +0100, Clément Bœsch wrote: The positioning was completely wrong. First, the coordinates are expressed in ASS playback resolution (which is by default 384x288). Secondly, the coordinates define a drawing rectangle, not a moving area. The previous code was

[FFmpeg-devel] [PATCH 1/2] avcodec/ass: make default playback resolution available to decoders

2015-02-08 Thread Clément Bœsch
--- libavcodec/ass.c | 5 +++-- libavcodec/ass.h | 3 +++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/libavcodec/ass.c b/libavcodec/ass.c index ea247f8..468b8bb 100644 --- a/libavcodec/ass.c +++ b/libavcodec/ass.c @@ -36,8 +36,8 @@ int ff_ass_subtitle_header(AVCodecContext

[FFmpeg-devel] [PATCH 2/2] avcodec/srtdec: attempt to correct SubRip positioning

2015-02-08 Thread Clément Bœsch
The positioning was completely wrong. First, the coordinates are expressed in ASS playback resolution (which is by default 384x288). Secondly, the coordinates define a drawing rectangle, not a moving area. The previous code was making subtitles move from a random position to another random

Re: [FFmpeg-devel] [PATCH] x86inc: also warn on sse2 instruction

2015-02-08 Thread Ronald S. Bultje
Hi, On Sun, Feb 8, 2015 at 6:30 AM, Christophe Gisquet christophe.gisq...@gmail.com wrote: Hi, the attached patch is strictly identical to this x264 one: http://git.videolan.org/?p=x264/x264-sandbox.git;a=commit;h=a7a909fc7b82df85ff369adcbc048927ebec2289 except for the edited paths. The

Re: [FFmpeg-devel] [PATCH] x86/hevc_sao: make sao_band_filter work on x86_32

2015-02-08 Thread Christophe Gisquet
Hi, 2015-02-07 23:06 GMT+01:00 James Almer jamr...@gmail.com: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_sao.asm | 40 libavcodec/x86/hevcdsp_init.c | 24 2 files changed, 48 insertions(+), 16

Re: [FFmpeg-devel] [PATCH] x86/vp9dsp: fix clobbering of xmm6 on IDCT sse2 functions

2015-02-08 Thread Ronald S. Bultje
Hi, On Sat, Feb 7, 2015 at 9:10 PM, James Almer jamr...@gmail.com wrote: On 07/02/15 11:08 PM, James Almer wrote: On 07/02/15 11:05 PM, Ronald S. Bultje wrote: Hi, On Sat, Feb 7, 2015 at 8:33 PM, James Almer jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com ---

Re: [FFmpeg-devel] Adding Force Style option in Subtitles Filter

2015-02-08 Thread Clément Bœsch
On Fri, Feb 06, 2015 at 10:47:09AM +0100, Stefano Sabatini wrote: On date Thursday 2015-02-05 22:12:08 +0530, Eejya Singh encoded: [...] From ed526a03c939055e2b9f931a0cb4e0e22b7b5b04 Mon Sep 17 00:00:00 2001 From: Eejya Singh singh.ee...@gmail.com Date: Wed, 28 Jan 2015 17:41:42 +0530

[FFmpeg-devel] [PATCH] x86inc: also warn on sse2 instruction

2015-02-08 Thread Christophe Gisquet
Hi, the attached patch is strictly identical to this x264 one: http://git.videolan.org/?p=x264/x264-sandbox.git;a=commit;h=a7a909fc7b82df85ff369adcbc048927ebec2289 except for the edited paths. The goal is to catch that last batch of issues in external asm:

Re: [FFmpeg-devel] [PATCHv4] flac: ignore duplicated ID3 tags if vorbis tags exist

2015-02-08 Thread wm4
On Sun, 8 Feb 2015 04:37:31 + (UTC) Ben Boeckel maths...@gmail.com wrote: On Sun, 08 Feb, 2015 at 02:05:45 GMT, Michael Niedermayer wrote: i tried clementine it happily adds and edits id3 tags on ogg files, one has to rename the file to .mp3 though. So no problem as noone would be

Re: [FFmpeg-devel] [PATCH] x86/doc/Makefile: DBG=1 to preprocess external asm

2015-02-08 Thread Carl Eugen Hoyos
Christophe Gisquet christophe.gisquet at gmail.com writes: Suggestions for more efficient make rules, clearer documentation and a better macro than the generic 'DBG' welcome. Doesn't this also need an update for make clean? Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] x86/hevc_sao: make sao_band_filter work on x86_32

2015-02-08 Thread James Almer
On 08/02/15 8:21 AM, Christophe Gisquet wrote: Hi, 2015-02-07 23:06 GMT+01:00 James Almer jamr...@gmail.com: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_sao.asm | 40 libavcodec/x86/hevcdsp_init.c | 24

Re: [FFmpeg-devel] hevc : support deinterlacing inside the decoder

2015-02-08 Thread Kacper Michajłow
2015-02-08 10:48 GMT+01:00 Carl Eugen Hoyos ceho...@ag.or.at: Mickaël Raulet mraulet at insa-rennes.fr writes: As we can consider, we won't have 4k interlaced content, copying a field into a frame should be ok. This is what has been done in this implementation. Do you have a sample? I

Re: [FFmpeg-devel] Enable stream copy of data

2015-02-08 Thread Anshul
On 01/16/2015 07:21 AM, Michael Niedermayer wrote: On Wed, Jan 14, 2015 at 03:11:24PM +0530, Anshul wrote: Hi I have enabled demuxing and muxing path for datat stream Please find attachment Thanks Anshul ffmpeg.c|2 ++ ffmpeg_opt.c| 10 ++

Re: [FFmpeg-devel] Enable stream copy of data

2015-02-08 Thread Kieran Kunhya
It has timestamp inside the data stream, and for making a simple use case scte need -copyts and -vsync 0 param. Otherwise user can ignore the data as they always did. -Anshul Is FFmpeg aware of this PTS and how to place the data packet with respect to PES packets? How are you testing this

Re: [FFmpeg-devel] Enable stream copy of data

2015-02-08 Thread Michael Niedermayer
On Sun, Feb 08, 2015 at 10:42:43PM +0530, Anshul wrote: On 01/16/2015 07:21 AM, Michael Niedermayer wrote: On Wed, Jan 14, 2015 at 03:11:24PM +0530, Anshul wrote: Hi I have enabled demuxing and muxing path for datat stream Please find attachment Thanks Anshul ffmpeg.c

Re: [FFmpeg-devel] [PATCH] x86/hevc_sao: make sao_band_filter work on x86_32

2015-02-08 Thread Christophe Gisquet
Hi, 2015-02-08 18:48 GMT+01:00 James Almer jamr...@gmail.com: +%assign MMSIZE mmsize Why do that? Not a big deal: it's only for my education, if there's something I'm missing. For width 48, the COMPUTE macro is last run after an INIT_XMM cpuname, so mmsize becomes 16 and in the avx2

Re: [FFmpeg-devel] hevc : support deinterlacing inside the decoder

2015-02-08 Thread Carl Eugen Hoyos
Kacper Michajłow kasper93 at gmail.com writes: 2015-02-08 10:48 GMT+01:00 Carl Eugen Hoyos: Mickaël Raulet mraulet at insa-rennes.fr writes: As we can consider, we won't have 4k interlaced content, copying a field into a frame should be ok. This is what has been done in this

Re: [FFmpeg-devel] [PATCH] configure: Check linking against CoreGraphics or, ApplicationServices framework for avfoundation input device.

2015-02-08 Thread Carl Eugen Hoyos
Hi! On Wednesday 04 February 2015 01:48:53 pm Thilo Borgmann wrote: fixes Ticket #4238. Testing for CGImageGetTypeID function is questionable (see trac discussion). If someone has a better idea, please let me know. Alternative patch attached. Also attached is a patch that I believe is

Re: [FFmpeg-devel] [PATCH] configure: Check linking against CoreGraphics or, ApplicationServices framework for avfoundation input device.

2015-02-08 Thread Michael Niedermayer
On Sun, Feb 08, 2015 at 03:19:40PM +0100, Carl Eugen Hoyos wrote: Hi! On Wednesday 04 February 2015 01:48:53 pm Thilo Borgmann wrote: fixes Ticket #4238. Testing for CGImageGetTypeID function is questionable (see trac discussion). If someone has a better idea, please let me know.

Re: [FFmpeg-devel] [PATCHv4] flac: ignore duplicated ID3 tags if vorbis tags exist

2015-02-08 Thread Michael Niedermayer
On Sun, Feb 08, 2015 at 04:37:31AM +, Ben Boeckel wrote: On Sun, 08 Feb, 2015 at 02:05:45 GMT, Michael Niedermayer wrote: i tried clementine it happily adds and edits id3 tags on ogg files, one has to rename the file to .mp3 though. So no problem as noone would be doing that if

[FFmpeg-devel] [PATCH 0/7] using ffmpeg as an RTSP client

2015-02-08 Thread Gilles Chanteperdrix
Hi, I am using ffmpeg 2.5.3 as an RTSP client and found a few missing pieces, which the following series of patches should address. I used some test files streamed with an RTSP server on the xenomai.org server, the files are available for download by replacing rtsp://xenomai.org:8554 with

[FFmpeg-devel] [PATCH 1/7] avformat/rtpdec_ac3: add AC3 RTP depacketization

2015-02-08 Thread Gilles Chanteperdrix
--- libavformat/Makefile | 1 + libavformat/rtpdec.c | 1 + libavformat/rtpdec_ac3.c | 166 +++ libavformat/rtpdec_formats.h | 1 + 4 files changed, 169 insertions(+) create mode 100644 libavformat/rtpdec_ac3.c diff --git

[FFmpeg-devel] [PATCH 2/7] avformat/rtsp: allow receiving subtitles via RTP

2015-02-08 Thread Gilles Chanteperdrix
--- libavformat/rtsp.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index ae62252..77407dd 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -68,10 +68,11 @@ { filter_src, only receive packets from the

[FFmpeg-devel] [CLT2015] FFmpeg at Chemnitzer Linux-Tage

2015-02-08 Thread Alexander Strasser
Hello all, we applied for a booth at `Chemnitzer Linux-Tage' in Germany. Fortunately we were accepted again! The event will take place on 21st 22nd of March in Chemnitz. More information can be found here: https://chemnitzer.linux-tage.de/2015/en/ We invite every FFmpeg

[FFmpeg-devel] [PATCH 7/7] avformat/rtsp: parse lang attribute in SDP

2015-02-08 Thread Gilles Chanteperdrix
--- libavformat/rtsp.c | 12 libavformat/rtsp.h | 2 ++ 2 files changed, 14 insertions(+) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index 77407dd..3a68020 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -477,6 +477,8 @@ static void

[FFmpeg-devel] [PATCH 4/7] avcocdec/mpegaudio_parser: add MP3 ADU headers parser

2015-02-08 Thread Gilles Chanteperdrix
--- libavcodec/mpegaudio_parser.c | 14 -- libavcodec/mpegaudiodecheader.c | 3 ++- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/libavcodec/mpegaudio_parser.c b/libavcodec/mpegaudio_parser.c index 79dbf63..42f4706 100644 --- a/libavcodec/mpegaudio_parser.c +++

[FFmpeg-devel] [PATCH 3/7] avformat/rtpdec: add T.140 RTP depacketization (RFC 4103)

2015-02-08 Thread Gilles Chanteperdrix
--- libavformat/rtpdec.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c index f5557d8..4091bdc 100644 --- a/libavformat/rtpdec.c +++ b/libavformat/rtpdec.c @@ -56,6 +56,12 @@ static RTPDynamicProtocolHandler opus_dynamic_handler = {

[FFmpeg-devel] [PATCH 6/7] avformat/rtpdec_mpeg4: reassemble fragmented AAC frames

2015-02-08 Thread Gilles Chanteperdrix
--- libavformat/rtpdec_mpeg4.c | 84 +- 1 file changed, 76 insertions(+), 8 deletions(-) diff --git a/libavformat/rtpdec_mpeg4.c b/libavformat/rtpdec_mpeg4.c index 53921f2..f791a5c 100644 --- a/libavformat/rtpdec_mpeg4.c +++

[FFmpeg-devel] [PATCH 5/7] avformat/rtpdec_mpeg12: add robust MPEG audio depacketization (RFC 3119)

2015-02-08 Thread Gilles Chanteperdrix
--- libavformat/rtpdec.c | 1 + libavformat/rtpdec_formats.h | 1 + libavformat/rtpdec_mpeg12.c | 201 +++ 3 files changed, 203 insertions(+) diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c index 4091bdc..711a94d 100644 ---

Re: [FFmpeg-devel] [PATCH 2/5] x86: hevc_mc: remove lea in EPEL_LOAD

2015-02-08 Thread Michael Niedermayer
On Sat, Feb 07, 2015 at 06:49:37PM +, Christophe Gisquet wrote: The second parameter to the macro is always an immediate address, so no lea is needed. --- libavcodec/x86/hevc_mc.asm | 19 +++ 1 file changed, 7 insertions(+), 12 deletions(-) applied thanks [...] --

Re: [FFmpeg-devel] [PATCH 1/5] x86: hevc_mc: fewer gpr autoloads for _v filters

2015-02-08 Thread Michael Niedermayer
On Sat, Feb 07, 2015 at 06:49:36PM +, Christophe Gisquet wrote: In that case, it's just to load my, but mx/r3src is not used. --- libavcodec/x86/hevc_mc.asm | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH]Fix msvc compilation with --disable-indev=vfwcap

2015-02-08 Thread Carl Eugen Hoyos
Hi! Attached patch intends to fix compilation with msvc when the vfwcap input device is disabled. Tested by Bernhard Döbler Please comment, Carl Eugen diff --git a/configure b/configure index f31d372..0c6f7ea 100755 --- a/configure +++ b/configure @@ -5371,7 +5371,7 @@ if test $target_os =

Re: [FFmpeg-devel] DSP function ARM NEON patches for hevc

2015-02-08 Thread Michael Niedermayer
On Thu, Feb 05, 2015 at 02:22:28PM +0100, Mickaël Raulet wrote: Michael, Please find some commits that can be cherry picked from https://github.com/OpenHEVC/FFmpeg/commits/ffmpeg_patch Optimized deblocking filter (8bits only) 1b9ee47d2f43b0a029a9468233626102eb1473b8 Optimzed transform

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/ass: make default playback resolution available to decoders

2015-02-08 Thread Michael Niedermayer
On Sun, Feb 08, 2015 at 12:40:18PM +0100, Clément Bœsch wrote: --- libavcodec/ass.c | 5 +++-- libavcodec/ass.h | 3 +++ 2 files changed, 6 insertions(+), 2 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Freedom in capitalist society

Re: [FFmpeg-devel] [PATCH] avfilter/f_sendcmd: consider it an error if there are no commands

2015-02-08 Thread Michael Niedermayer
On Fri, Jan 16, 2015 at 12:23:32AM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavfilter/f_sendcmd.c |5 + 1 file changed, 5 insertions(+) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Good

Re: [FFmpeg-devel] [PATCH] Docs for Blackmagic high bit depth video/audio

2015-02-08 Thread Michael Niedermayer
On Sun, Jan 18, 2015 at 08:50:15PM +, Timothy Gu wrote: On Sun Jan 18 2015 at 11:46:03 AM Georg Lippitsch georg.lippit...@gmx.at wrote: [...] @@ -1023,6 +1023,16 @@ Defaults to @option{false}. If set to @option{true}, print a list of supported formats and exit. Defaults to

Re: [FFmpeg-devel] [PATCH] ffprobe: Force string_validation to int type, its accessed via AVOption as int

2015-02-08 Thread Clément Bœsch
On Fri, Feb 06, 2015 at 10:10:22PM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffprobe.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ffprobe.c b/ffprobe.c index d352bb6b..8617c96 100644 --- a/ffprobe.c +++

Re: [FFmpeg-devel] Enable stream copy of data

2015-02-08 Thread Anshul
On February 8, 2015 11:07:12 PM IST, Kieran Kunhya kier...@obe.tv wrote: It has timestamp inside the data stream, and for making a simple use case scte need -copyts and -vsync 0 param. Otherwise user can ignore the data as they always did. -Anshul Is FFmpeg aware of this PTS and how to place

Re: [FFmpeg-devel] [PATCH] ffprobe: Force string_validation to int type, its accessed via AVOption as int

2015-02-08 Thread Michael Niedermayer
On Fri, Feb 06, 2015 at 10:10:22PM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffprobe.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) i intend to apply this and similar fixes to other parts of ffmpeg soon ... diff --git

Re: [FFmpeg-devel] [PATCH] ffprobe: Force string_validation to int type, its accessed via AVOption as int

2015-02-08 Thread Michael Niedermayer
On Mon, Feb 09, 2015 at 01:44:43AM +0100, Clément Bœsch wrote: On Fri, Feb 06, 2015 at 10:10:22PM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffprobe.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ffprobe.c