Re: [FFmpeg-devel] patch for x32 for libpostproc

2014-09-05 Thread Reimar Döffinger
On 05.09.2014, at 03:46, Reinhard Tartler siret...@gmail.com wrote: On Thu, Sep 4, 2014 at 9:32 PM, Michael Niedermayer michae...@gmx.at wrote: At the end of the day, I need a source tarball that contains maintained sources of a stand-alone libpostproc. I don't care too much how it is created,

Re: [FFmpeg-devel] [PATCH v3] ffprobe: Add bits_per_component to stream output

2014-09-05 Thread Tobias Rapp
On 04.09.2014 17:39, wm4 wrote: diff --git a/libavutil/pixdesc.h b/libavutil/pixdesc.h index 41e81db..f8af18f 100644 --- a/libavutil/pixdesc.h +++ b/libavutil/pixdesc.h @@ -251,6 +251,13 @@ int av_get_bits_per_pixel(const AVPixFmtDescriptor *pixdesc); int av_get_padded_bits_per_pixel(const

Re: [FFmpeg-devel] [PATCH] vfi/x86/vf_idet: fix incorrect use of paddq

2014-09-05 Thread Pascal Massimino
James, On Thu, Sep 4, 2014 at 3:22 PM, James Darnley james.darn...@gmail.com wrote: paddq is an SSE2 instruction so it cannot be used for MMX. This was probably just a typo because the sums are dwords anyway. --- We should have picked this up in review but we didn't...

Re: [FFmpeg-devel] [PATCH 1/6] x264asm: warn when inappropriate instruction used in function with specified cpuflags

2014-09-05 Thread Christophe Gisquet
Hi, Le 5 sept. 2014 00:14, James Darnley james.darn...@gmail.com a écrit : I think I tested the same patch and for some reason it incorrectly flagged packusdw in a macro in this tree: https://github.com/kurosu/ffmpeg/tree/hevc_mt I haven't yet had time to investigate where the problem is.

[FFmpeg-devel] [PATCH] tiff: fix {2,4}bpp grayscale palettes.

2014-09-05 Thread Benoit Fouet
Create a default grayscale palette for 2 or 4 bpp grayscale tiff, if there is no palette defined. Fixes ticket #3915 --- libavcodec/tiff.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index 1e2d235..0352639 100644 ---

Re: [FFmpeg-devel] [PATCH] doc/filters.texi: fix time duration references

2014-09-05 Thread Stefano Sabatini
On date Thursday 2014-09-04 13:19:44 -0700, Mark Harris encoded: Make time duration references consistent, using @ref links, and eliminate incorrect syntax [-]HH[:MM[:SS[.m...]]]. --- doc/filters.texi | 68 1 file changed, 29

[FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Pascal Massimino
Hi, should have been LGPL. Kudos to Carl for spotting that. /skal From cd067fd2f7ae77b6be950fe2e841b9c285208612 Mon Sep 17 00:00:00 2001 From: skal pascal.massim...@gmail.com Date: Fri, 5 Sep 2014 11:43:32 +0200 Subject: [PATCH] fix license header: should have been LGPL. ---

Re: [FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Clément Bœsch
On Fri, Sep 05, 2014 at 11:47:42AM +0200, Pascal Massimino wrote: Hi, should have been LGPL. Kudos to Carl for spotting that. /skal From cd067fd2f7ae77b6be950fe2e841b9c285208612 Mon Sep 17 00:00:00 2001 From: skal pascal.massim...@gmail.com While at it, can you fix your username in git

Re: [FFmpeg-devel] [PATCH v3] ffprobe: Add bits_per_component to stream output

2014-09-05 Thread Stefano Sabatini
On date Friday 2014-09-05 08:29:45 +0200, Tobias Rapp encoded: On 04.09.2014 17:39, wm4 wrote: diff --git a/libavutil/pixdesc.h b/libavutil/pixdesc.h index 41e81db..f8af18f 100644 --- a/libavutil/pixdesc.h +++ b/libavutil/pixdesc.h @@ -251,6 +251,13 @@ int av_get_bits_per_pixel(const

Re: [FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Stefano Sabatini
On date Friday 2014-09-05 11:50:35 +0200, Clément Bœsch encoded: On Fri, Sep 05, 2014 at 11:47:42AM +0200, Pascal Massimino wrote: Hi, should have been LGPL. Kudos to Carl for spotting that. /skal From cd067fd2f7ae77b6be950fe2e841b9c285208612 Mon Sep 17 00:00:00 2001 From: skal

Re: [FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Pascal Massimino
On Fri, Sep 5, 2014 at 11:57 AM, Stefano Sabatini stefa...@gmail.com wrote: On date Friday 2014-09-05 11:50:35 +0200, Clément Bœsch encoded: On Fri, Sep 05, 2014 at 11:47:42AM +0200, Pascal Massimino wrote: Hi, should have been LGPL. Kudos to Carl for spotting that. /skal

Re: [FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Clément Bœsch
On Fri, Sep 05, 2014 at 12:01:20PM +0200, Pascal Massimino wrote: On Fri, Sep 5, 2014 at 11:57 AM, Stefano Sabatini stefa...@gmail.com wrote: On date Friday 2014-09-05 11:50:35 +0200, Clément Bœsch encoded: On Fri, Sep 05, 2014 at 11:47:42AM +0200, Pascal Massimino wrote: Hi,

Re: [FFmpeg-devel] [PATCH] Add input support for DeckLink devices.

2014-09-05 Thread Deti Fliegl
On 04.09.14 16:44, Deron wrote: What I did to test capture capabilities was use two decklink devices and connect them together. To test output, I happen to have an HD-SDI capable monitor. Good luck, I've not had any luck with the official ffmpeg decklink driver. Could you please explain what you

[FFmpeg-devel] [PATCH] avfilter/x86/idet: fix license header (GPL - LGPL)

2014-09-05 Thread Pascal Massimino
Clément, On Fri, Sep 5, 2014 at 12:03 PM, Clément Bœsch u...@pkh.me wrote: On Fri, Sep 05, 2014 at 12:01:20PM +0200, Pascal Massimino wrote: On Fri, Sep 5, 2014 at 11:57 AM, Stefano Sabatini stefa...@gmail.com wrote: On date Friday 2014-09-05 11:50:35 +0200, Clément Bœsch encoded:

Re: [FFmpeg-devel] [PATCH] avfilter/x86/idet: fix license header (GPL - LGPL)

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 12:10:12PM +0200, Pascal Massimino wrote: Clément, On Fri, Sep 5, 2014 at 12:03 PM, Clément Bœsch u...@pkh.me wrote: On Fri, Sep 05, 2014 at 12:01:20PM +0200, Pascal Massimino wrote: On Fri, Sep 5, 2014 at 11:57 AM, Stefano Sabatini stefa...@gmail.com wrote:

Re: [FFmpeg-devel] [PATCH] vfi/x86/vf_idet: fix incorrect use of paddq

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 12:08:56AM -0700, Pascal Massimino wrote: James, On Thu, Sep 4, 2014 at 3:22 PM, James Darnley james.darn...@gmail.com wrote: paddq is an SSE2 instruction so it cannot be used for MMX. This was probably just a typo because the sums are dwords anyway. ---

Re: [FFmpeg-devel] [PATCH] tiff: fix {2,4}bpp grayscale palettes.

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 11:15:13AM +0200, Benoit Fouet wrote: Create a default grayscale palette for 2 or 4 bpp grayscale tiff, if there is no palette defined. Fixes ticket #3915 --- libavcodec/tiff.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) applied thanks

Re: [FFmpeg-devel] [PATCH v3] ffprobe: Add bits_per_component to stream output

2014-09-05 Thread Tobias Rapp
On 05.09.2014 11:56, Stefano Sabatini wrote: On date Friday 2014-09-05 08:29:45 +0200, Tobias Rapp encoded: On 04.09.2014 17:39, wm4 wrote: diff --git a/libavutil/pixdesc.h b/libavutil/pixdesc.h index 41e81db..f8af18f 100644 --- a/libavutil/pixdesc.h +++ b/libavutil/pixdesc.h @@ -251,6 +251,13

Re: [FFmpeg-devel] [PATCH]Set H264 bitrate

2014-09-05 Thread Michael Niedermayer
On Tue, Aug 19, 2014 at 08:35:29AM +0200, Reimar Döffinger wrote: On 19.08.2014, at 02:24, Kieran Kunhya kier...@obe.tv wrote: On 19 August 2014 00:32, Carl Eugen Hoyos ceho...@ag.or.at wrote: Hi! Attached patch from MIKEH / Elemental is apparently meant to implement setting h264

Re: [FFmpeg-devel] [PATCH v3] ffprobe: Add bits_per_component to stream output

2014-09-05 Thread Stefano Sabatini
On date Friday 2014-09-05 13:40:17 +0200, Tobias Rapp encoded: On 05.09.2014 11:56, Stefano Sabatini wrote: [...] Does something such highly specialized really warrant its own API function? There are several ways how to handle the case if the component bit depths are not uniform, or even how

Re: [FFmpeg-devel] [PATCH] Add input support for DeckLink devices.

2014-09-05 Thread Roger Pack
So you tested this in linux and windows? And it works? What is the advantage of this over dshow, does it work better somehow [I know dshow is deficient in multiple ways, FWIW]. Or was it just to add linux support? Cheers! -roger- On Mon, Aug 18, 2014 at 12:40 PM, Deti Fliegl d...@fliegl.de

[FFmpeg-devel] [PATCH 1/2] avcodec/mpegvideo: Factor ff_mpv_decode_init() out

2014-09-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/h261dec.c |7 ++- libavcodec/h263dec.c |6 ++ libavcodec/mpeg12dec.c |3 +-- libavcodec/mpegvideo.c | 12 libavcodec/mpegvideo.h |1 + libavcodec/rv10.c |3 +-- libavcodec/rv34.c

Re: [FFmpeg-devel] [PATCH] Add input support for DeckLink devices.

2014-09-05 Thread Deti Fliegl
Using the native DeckLink api is more efficient and allows you to do more things. On top you get Linux and MacOS X support. Deti On 05.09.14 16:37, Roger Pack wrote: So you tested this in linux and windows? And it works? What is the advantage of this over dshow, does it work better somehow [I

Re: [FFmpeg-devel] [PATCH 3/5] lavd/v4l2: remove unneeded variable in device_init()

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 05:57:12PM +0200, Giorgio Vazzana wrote: v4l2.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) 845057fe112aa2d74fc05a4575fabe2910dee8ac 0003-lavd-v4l2-remove-unneeded-variable-in-device_init.patch From fab5a8ac8d2151dce5384c6ff70d5dee25e72066 Mon Sep

Re: [FFmpeg-devel] [PATCH 4/5] lavd/v4l2: simplify list_framesizes()

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 05:57:18PM +0200, Giorgio Vazzana wrote: v4l2.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) 83c2d56d72c9f31afb52514eb07d4c55037d2be3 0004-lavd-v4l2-simplify-list_framesizes.patch From 7b62611a54861e5d972e61c94a00a843aca44b19 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] patch for x32 for libpostproc

2014-09-05 Thread Nicolas George
L'octidi 18 fructidor, an CCXXII, wm4 a écrit : And in general, it's a good idea to move optional libraries out of ffmpeg.git. (In fact, ffmpeg.git should be split into several git repos, one for each lib.) That is a terrible idea. We already have too many difficulties maintaining binaries

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mpegvideo: Factor ff_mpv_decode_init() out

2014-09-05 Thread Benoit Fouet
Hi, Le 05/09/2014 17:41, Michael Niedermayer a écrit : Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/h261dec.c |7 ++- libavcodec/h263dec.c |6 ++ libavcodec/mpeg12dec.c |3 +-- libavcodec/mpegvideo.c | 12 libavcodec/mpegvideo.h

[FFmpeg-devel] [PATCH v3] avformat/srtdec: UTF-16 support

2014-09-05 Thread wm4
--- Fixes issues with ff_subtitles_read_line() EOF handling pointed out by ubitux on IRC. --- libavformat/srtdec.c| 25 +-- libavformat/subtitles.c | 53 +++-- libavformat/subtitles.h | 34 --- 3 files

Re: [FFmpeg-devel] [PATCH 5/5] lavd/v4l2: simplify list_formats()

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 05:57:25PM +0200, Giorgio Vazzana wrote: v4l2.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) 58aab8a76d42bf75a7f8e46ec217b08b5e186878 0005-lavd-v4l2-simplify-list_formats.patch From 5ab8d5700c69033fd5dc27d49525df84863a38f9 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH] Add a license to the LESS file

2014-09-05 Thread Timothy Gu
On Fri, Sep 5, 2014 at 12:10 PM, db0 company db0comp...@gmail.com wrote: On Wed, Aug 27, 2014 at 12:55 AM, Timothy Gu timothyg...@gmail.com wrote: You should make it clear that only style.less is covered in this license, not anything else. I added the full license in the header. The first

Re: [FFmpeg-devel] [PATCH] Add a license to the LESS file

2014-09-05 Thread Timothy Gu
On Fri, Sep 5, 2014 at 12:19 PM, Timothy Gu timothyg...@gmail.com wrote: On Fri, Sep 5, 2014 at 12:10 PM, db0 company db0comp...@gmail.com wrote: On Wed, Aug 27, 2014 at 12:55 AM, Timothy Gu timothyg...@gmail.com wrote: You should make it clear that only style.less is covered in this license,

Re: [FFmpeg-devel] [PATCH v2 1/6] avformat/assdec: UTF-16 support

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:22PM +0200, wm4 wrote: Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around AVIOContext. It also can work on a static buffer, needed for format probing. The

Re: [FFmpeg-devel] [PATCH v2 3/6] avformat/samidec: UTF-16 support

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:24PM +0200, wm4 wrote: ff_smil_extract_next_chunk() is still used by RealText. --- No changes. --- libavformat/samidec.c | 17 ++--- libavformat/subtitles.c | 15 --- libavformat/subtitles.h | 5 + 3 files changed, 27

Re: [FFmpeg-devel] [PATCH v2 4/6] avformat/realtextdec: UTF-16 support

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:25PM +0200, wm4 wrote: Also remove ff_smil_extract_next_chunk - this was the last user of it. --- No changes. --- libavformat/realtextdec.c | 17 ++--- libavformat/subtitles.c | 9 - libavformat/subtitles.h | 5 - 3 files

Re: [FFmpeg-devel] [PATCH v2 5/6] avformat/srtdec: speed up probing

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:26PM +0200, wm4 wrote: --- As suggested by ubitux. Note that the change to ff_subtitles_read_line() already cuts down the probing time a lot, so maybe this is not needed. --- libavformat/srtdec.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [FFmpeg-devel] [PATCH v2 6/6] avformat/subtitles: guess UTF-16 if no BOM present

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:27PM +0200, wm4 wrote: --- Follows reimar's suggestion for detecting UTF-16. If the detection goes wrong, probing the format won't succeed, so this should not break anything. I didn't use ffio_ensure_seekback), because this apparently reallocates the buffer -

Re: [FFmpeg-devel] [PATCH v2 6/6] avformat/subtitles: guess UTF-16 if no BOM present

2014-09-05 Thread Michael Niedermayer
On Thu, Sep 04, 2014 at 10:40:27PM +0200, wm4 wrote: --- Follows reimar's suggestion for detecting UTF-16. If the detection goes wrong, probing the format won't succeed, so this should not break anything. I didn't use ffio_ensure_seekback), because this apparently reallocates the buffer -

Re: [FFmpeg-devel] [PATCH] libavdevice/v4l2: fix of crash caused by assert

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 08:58:30PM +0200, Giorgio Vazzana wrote: 2014-09-04 11:29 GMT+02:00 Dmitry Volyntsev xeioexcept...@gmail.com: Yes, I could confirm that patch is LGTM. Successfully checked it on one day long run. Thanks for taking the time to test it, Dmitry. Michael, I suppose

Re: [FFmpeg-devel] [PATCH v2 6/6] avformat/subtitles: guess UTF-16 if no BOM present

2014-09-05 Thread wm4
On Sat, 6 Sep 2014 00:00:10 +0200 Michael Niedermayer michae...@gmx.at wrote: On Thu, Sep 04, 2014 at 10:40:27PM +0200, wm4 wrote: --- Follows reimar's suggestion for detecting UTF-16. If the detection goes wrong, probing the format won't succeed, so this should not break anything.

Re: [FFmpeg-devel] [PATCH] Add a license to the LESS file

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 12:20:52PM -0700, Timothy Gu wrote: On Fri, Sep 5, 2014 at 12:19 PM, Timothy Gu timothyg...@gmail.com wrote: On Fri, Sep 5, 2014 at 12:10 PM, db0 company db0comp...@gmail.com wrote: On Wed, Aug 27, 2014 at 12:55 AM, Timothy Gu timothyg...@gmail.com wrote: You should

[FFmpeg-devel] [PATCH] ffmpeg: Copy extradata if it has been initialized later from the encoder

2014-09-05 Thread Michael Niedermayer
Fixes Ticket3909 Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffmpeg.c |8 1 file changed, 8 insertions(+) diff --git a/ffmpeg.c b/ffmpeg.c index 220816b..0807eec 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -590,6 +590,14 @@ static void write_frame(AVFormatContext *s,