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

2014-09-05 Thread Reimar Döffinger
On 06.09.2014, at 00:23, wm4 wrote: > On Sat, 6 Sep 2014 00:00:10 +0200 > Michael Niedermayer 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

[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 --- 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, AVPacket *pkt, OutputS

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 wrote: > > On Fri, Sep 5, 2014 at 12:10 PM, db0 company wrote: > >> On Wed, Aug 27, 2014 at 12:55 AM, Timothy Gu wrote: > >>> You should make it clear that only style.less is covered in this

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 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. > > > > > I

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 : > > 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 it's safe to app

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 bu

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 buf

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 a/liba

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 ins

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

2014-09-05 Thread Clément Bœsch
On Fri, Sep 05, 2014 at 08:37:01PM +0200, wm4 wrote: > --- > Fixes issues with ff_subtitles_read_line() EOF handling pointed out by > ubitux on IRC. > --- > libavformat/srtdec.c| 25 +-- > libavformat/subtitles.c | 53 > +++-- >

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 FFTex

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 wrote: > On Fri, Sep 5, 2014 at 12:10 PM, db0 company wrote: >> On Wed, Aug 27, 2014 at 12:55 AM, Timothy Gu wrote: >>> You should make it clear that only style.less is covered in this >>> license, not anything else. >> >> I added the full license in t

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 wrote: > On Wed, Aug 27, 2014 at 12:55 AM, Timothy Gu 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 patch is for ffmpeg-web, the second

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

2014-09-05 Thread Giorgio Vazzana
2014-09-04 11:29 GMT+02:00 Dmitry Volyntsev : > 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 it's safe to apply the patch now, so that this fix will be included in FFmpeg 2.4. Regards, Gi

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

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 08:34:05PM +0200, Benoit Fouet wrote: > Hi, > > Le 05/09/2014 17:41, Michael Niedermayer a écrit : > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/h261dec.c |7 ++- > > libavcodec/h263dec.c |6 ++ > > libavcodec/mpeg12dec.c |3 +-- > >

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mpegvideo: set codec tags in ff_mpv_decode_init()

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 08:34:54PM +0200, Benoit Fouet wrote: > Hi, > > Le 05/09/2014 17:41, Michael Niedermayer a écrit : > > Fixes Ticket3912 > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/mpegvideo.c |8 > > 1 file changed, 4 insertions(+), 4 deletions(-) > > >

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:

[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 2/2] avcodec/mpegvideo: set codec tags in ff_mpv_decode_init()

2014-09-05 Thread Benoit Fouet
Hi, Le 05/09/2014 17:41, Michael Niedermayer a écrit : > Fixes Ticket3912 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/mpegvideo.c |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > LGTM as well -- Ben ___ ffmpeg-deve

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 > --- > libavcodec/h261dec.c |7 ++- > libavcodec/h263dec.c |6 ++ > libavcodec/mpeg12dec.c |3 +-- > libavcodec/mpegvideo.c | 12 > libavcodec/mpegvideo.h |1 +

Re: [FFmpeg-devel] [PATCH v2 1/4] libavutil/error: fix build with musl toolchain

2014-09-05 Thread Reimar Döffinger
On Thu, Sep 04, 2014 at 11:27:49PM +0200, Jörg Krause wrote: > > On 09/04/2014 05:27 PM, Hendrik Leppkes wrote: > >On Wed, Sep 3, 2014 at 11:53 AM, Jörg Krause wrote: > >>Add the feature test macro which is required for building with the > >>musl toolchain. > >> > >>The feature test macro _XOPEN_

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 c

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:

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 Mo

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

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 05:57:04PM +0200, Giorgio Vazzana wrote: > > v4l2.c | 12 +--- > 1 file changed, 5 insertions(+), 7 deletions(-) > 25b21ceef271d79d1b5eae22ef1b7186d836e5ab > 0002-lavd-v4l2-simplify-first_field.patch > From 5642dbb81dadff02c63b454726b1ca4c99ee1c2c Mon Sep 17 0

Re: [FFmpeg-devel] [PATCH 1/5] lavd/v4l2: Replace s1 with ctx for consistency.

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 05:56:51PM +0200, Giorgio Vazzana wrote: > > v4l2.c | 114 > - > 1 file changed, 57 insertions(+), 57 deletions(-) > 07d7c60c6a3413b1b2f487839b75259997fd6a0e > 0001-lavd-v4l2-Replace-s1-with-ctx-for-consi

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

2014-09-05 Thread Giorgio Vazzana
From 5ab8d5700c69033fd5dc27d49525df84863a38f9 Mon Sep 17 00:00:00 2001 From: Giorgio Vazzana Date: Fri, 5 Sep 2014 17:43:59 +0200 Subject: [PATCH 5/5] lavd/v4l2: simplify list_formats() We can avoid passing file descriptor fd explicitely. --- libavdevice/v4l2.c |6 +++--- 1 file changed, 3

[FFmpeg-devel] [PATCH 2/5] lavd/v4l2: simplify first_field()

2014-09-05 Thread Giorgio Vazzana
From 5642dbb81dadff02c63b454726b1ca4c99ee1c2c Mon Sep 17 00:00:00 2001 From: Giorgio Vazzana Date: Fri, 5 Sep 2014 17:24:43 +0200 Subject: [PATCH 2/5] lavd/v4l2: simplify first_field() There is no need to pass fd as a second parameter. Additionally remove unneeded curly braces. --- libavdevice/

[FFmpeg-devel] [PATCH 1/5] lavd/v4l2: Replace s1 with ctx for consistency.

2014-09-05 Thread Giorgio Vazzana
From b85da403cb0856887d4a657aa706b6cdfbb619e0 Mon Sep 17 00:00:00 2001 From: Giorgio Vazzana Date: Fri, 5 Sep 2014 17:12:39 +0200 Subject: [PATCH 1/5] lavd/v4l2: Replace s1 with ctx for consistency. No functional change in the code. --- libavdevice/v4l2.c | 114 ++--

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

2014-09-05 Thread Giorgio Vazzana
From fab5a8ac8d2151dce5384c6ff70d5dee25e72066 Mon Sep 17 00:00:00 2001 From: Giorgio Vazzana Date: Fri, 5 Sep 2014 17:34:51 +0200 Subject: [PATCH 3/5] lavd/v4l2: remove unneeded variable in device_init() --- libavdevice/v4l2.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --

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

2014-09-05 Thread Giorgio Vazzana
From 7b62611a54861e5d972e61c94a00a843aca44b19 Mon Sep 17 00:00:00 2001 From: Giorgio Vazzana Date: Fri, 5 Sep 2014 17:39:50 +0200 Subject: [PATCH 4/5] lavd/v4l2: simplify list_framesizes() We can avoid passing file descriptor fd explicitely. --- libavdevice/v4l2.c |6 +++--- 1 file changed,

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add myself as lavd/v4l2 maintainer

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 04:30:31PM +0200, Giorgio Vazzana wrote: > Hello, > > here's a patch to add myself as a maintainer for lavd/v4l2. I asked > Luca about this a while back and he was fine with this change. > Regards, > > Giorgio Vazzana patch applied thanks to all [...] -- Michael G

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 [

[FFmpeg-devel] [PATCH 2/2] avcodec/mpegvideo: set codec tags in ff_mpv_decode_init()

2014-09-05 Thread Michael Niedermayer
Fixes Ticket3912 Signed-off-by: Michael Niedermayer --- libavcodec/mpegvideo.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index f5306cb..41a8799 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@

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

2014-09-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- 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 |7 +---

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 wrote: > Rebased

[FFmpeg-devel] [PATCH] MAINTAINERS: add myself as lavd/v4l2 maintainer

2014-09-05 Thread Giorgio Vazzana
Hello, here's a patch to add myself as a maintainer for lavd/v4l2. I asked Luca about this a while back and he was fine with this change. Regards, Giorgio Vazzana From 17faef15e981fe6797455084c0212927a2a99357 Mon Sep 17 00:00:00 2001 From: Giorgio Vazzana Date: Fri, 5 Sep 2014 16:26:11 +0200 Sub

Re: [FFmpeg-devel] [PATCH 6/6] x86inc: Make INIT_CPUFLAGS support an arbitrary number of cpuflags

2014-09-05 Thread Michael Niedermayer
On Fri, Sep 05, 2014 at 12:13:38AM +0200, James Darnley wrote: > From: Henrik Gramner > > Previously there was a limit of two cpuflags. > --- > libavutil/x86/x86inc.asm | 41 ++--- > 1 files changed, 22 insertions(+), 19 deletions(-) applied thanks [...]

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, o

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 wrote: > > On 19 August 2014 00:32, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch from MIKEH / Elemental is apparently meant to implement > >> setting h264 bitrate. It makes no

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

2014-09-05 Thread Carl Eugen Hoyos
Clément Bœsch pkh.me> writes: > While at it, can you fix your username in git config? > > "skal" doesn't have any legal value and can cause > problems in the future. I don't know French law but for Austria, this is simply wrong. (Is it on the contrary not allowed here to change the "name" in

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] 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 than

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 > 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 s

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 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 > > > wrote: > > > > > > > On da

[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 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 > > wrote: > > > > > On date Friday 2014-09-05 11:50:35 +0200, Clément Bœsch encoded: > > > > On Fri, Sep 05,

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 yo

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 > 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] 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 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 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

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_pixe

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 While at it, can you fix your username in git config? "skal" does

[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 Date: Fri, 5 Sep 2014 11:43:32 +0200 Subject: [PATCH] fix license header: should have been LGPL. --- libavfilter/vf_idet.h | 12 ++---

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 in

[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 --- a/

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" 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. Probably PEBKAC as it had n

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 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... > > libavfilter/x86/vf_idet.asm |