Re: [FFmpeg-devel] [PATCH] kCVPixelFormatType_OneComponent8 only exists from 10.8 onward

2014-07-13 Thread Reimar Döffinger
On 14.07.2014, at 04:55, Hanspeter Niederstrasser wrote: > The AVFoundation indev uses kCVPixelFormatType_OneComponent8, but that > only exists on 10.8 and up. This puts it behind a #if. It's not ideal. Maybe not worth the effort, but it would be better if a version compiled for older OSX coul

[FFmpeg-devel] [PATCH] kCVPixelFormatType_OneComponent8 only exists from 10.8 onward

2014-07-13 Thread Hanspeter Niederstrasser
The AVFoundation indev uses kCVPixelFormatType_OneComponent8, but that only exists on 10.8 and up. This puts it behind a #if. Hanspeter -- Disclaimer: By sending an email to ANY of my addresses you are agreeing that: 1. I am by definition, "the intended recipient" 2. All information in the emai

Re: [FFmpeg-devel] new webpage design patches

2014-07-13 Thread compn
On Mon, 14 Jul 2014 02:20:09 +0200 Michael Niedermayer wrote: > Hi all > > Theres a pull request that modernises our webpage > https://github.com/FFmpeg/web/pull/1 > https://github.com/FFmpeg/FFmpeg/pull/77 > i like the new design, and give it the OK. i did not review the code itself. maybe i

[FFmpeg-devel] [PATCH] Add a release note for 2.3

2014-07-13 Thread Timothy Gu
Based on a patch by Clément Bœsch . Signed-off-by: Timothy Gu --- Moved API info up per IRC --- RELEASE_NOTES | 172 ++ 1 file changed, 172 insertions(+) create mode 100644 RELEASE_NOTES diff --git a/RELEASE_NOTES b/RELEASE_NOTES new f

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 04:01:05PM +0200, Michael Niedermayer wrote: > MSVC does not allow passing file pointers between libs > This API can thus not work with MSVC and as it was very recently added > and its it was in no release its removial should not cause any problems > > Found-by: Hendrik Lep

[FFmpeg-devel] new webpage design patches

2014-07-13 Thread Michael Niedermayer
Hi all Theres a pull request that modernises our webpage https://github.com/FFmpeg/web/pull/1 https://github.com/FFmpeg/FFmpeg/pull/77 -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB You can kill me, but you cannot change the truth. signature.asc Description: Digita

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 07:32:56PM +0100, Derek Buitenhuis wrote: > mplayer-specifc hacks should not be in our codebase. mplayer should fix > it's own code. It is not our responsibility to work around their broken > code. > > This reverts commit e8e575633faf19711910cf9caf59f7db300a9ccd. > > Signe

Re: [FFmpeg-devel] [PATCH] AVFoundation: Add -pixel_format option

2014-07-13 Thread Carl Eugen Hoyos
Hanspeter Niederstrasser gmail.com> writes: > +#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1080 This was my next suggestion, sorry for the first one. > I'll submit a real patch if it seems correct. Please send a patch made with git format-patch if it fixes the problem for you. > If 2.3 is going t

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 11:02 PM, Reimar Döffinger wrote: > As far as I can tell MPlayer is now no longer affected. > Unfortunately older versions might be. > I guess it's up to you to decide the > "getting rid of awful hack vs. keeping compatibility" importance. I'd expect people using ffmpeg git HEAD are n

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread wm4
On Sun, 13 Jul 2014 22:17:44 +0200 Reimar Döffinger wrote: > On Sun, Jul 13, 2014 at 08:38:44PM +0200, wm4 wrote: > > On Sun, 13 Jul 2014 19:32:56 +0100 > > Derek Buitenhuis wrote: > > > > > mplayer-specifc hacks should not be in our codebase. mplayer should fix > > > it's own code. It is not o

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Reimar Döffinger
Hi, On Sun, Jul 13, 2014 at 07:32:56PM +0100, Derek Buitenhuis wrote: > mplayer-specifc hacks should not be in our codebase. mplayer should fix > it's own code. It is not our responsibility to work around their broken > code. As far as I can tell MPlayer is now no longer affected. Unfortunately o

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Clément Bœsch
On Sun, Jul 13, 2014 at 09:57:39PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 9:56 PM, Clément Bœsch wrote: > > In ffplay? > > O U. > > At least we don't move libSDL to avdevice or something... :P > Well, actually... -- Clément B. pgpcRmGxDToMl.pgp Description: PGP signature

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 9:56 PM, Clément Bœsch wrote: > In ffplay? O U. At least we don't move libSDL to avdevice or something... :P - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Clément Bœsch
On Sun, Jul 13, 2014 at 09:36:48PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 9:27 PM, compn wrote: > > googletv does not have easy shell and cannot mount smb. at least mine > > cant > > See what I said about implementing support in the layer above (the player) > instead of ffmpeg. > In ffpla

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 9:27 PM, compn wrote: > googletv does not have easy shell and cannot mount smb. at least mine > cant See what I said about implementing support in the layer above (the player) instead of ffmpeg. - Derek ___ ffmpeg-devel mailing list ffmpeg-

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 9:17 PM, Reimar Döffinger wrote: > Well, IMHO if this is a requirement that was added, I would say > it was an API change, and MPlayer might just be the first where > the API change was detected. Indeed there should have been a version bump in 419ade4b6193c6eb626cda01b21e7091f42b2cc2

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread compn
On Sun, 13 Jul 2014 18:04:54 +0100 Derek Buitenhuis wrote: > On 7/13/2014 5:54 PM, Lukasz Marek wrote: > > You also ignored the case where user might have no access to shell > > at all to mount remote fs. > > That's a pretty damn specialized case. googletv does not have easy shell and cannot mo

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Reimar Döffinger
On Sun, Jul 13, 2014 at 08:38:44PM +0200, wm4 wrote: > On Sun, 13 Jul 2014 19:32:56 +0100 > Derek Buitenhuis wrote: > > > mplayer-specifc hacks should not be in our codebase. mplayer should fix > > it's own code. It is not our responsibility to work around their broken > > code. > > > > This rev

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 06:44:21PM +0200, Lukasz Marek wrote: > On 13.07.2014 18:04, Derek Buitenhuis wrote: > >On 7/12/2014 5:20 PM, Lukasz Marek wrote: > >>+libsmbclient_protocol_deps="libsmbclient gpl" > > > >Samba is actually GPLv3, so you need to add dependency on that. > > Fixed. > Changel

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread wm4
On Sun, 13 Jul 2014 19:32:56 +0100 Derek Buitenhuis wrote: > mplayer-specifc hacks should not be in our codebase. mplayer should fix > it's own code. It is not our responsibility to work around their broken > code. > > This reverts commit e8e575633faf19711910cf9caf59f7db300a9ccd. > > Signed-off

Re: [FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 7:32 PM, Derek Buitenhuis wrote: > mplayer-specifc hacks should not be in our codebase. mplayer should fix > it's own code. It is not our responsibility to work around their broken > code. it's -> its - Derek ___ ffmpeg-devel mailing list f

[FFmpeg-devel] [PATCH] Revert "lavc/utils: Do not require dimensions for PNG."

2014-07-13 Thread Derek Buitenhuis
mplayer-specifc hacks should not be in our codebase. mplayer should fix it's own code. It is not our responsibility to work around their broken code. This reverts commit e8e575633faf19711910cf9caf59f7db300a9ccd. Signed-off-by: Derek Buitenhuis --- libavcodec/utils.c |4 +--- 1 files changed

[FFmpeg-devel] [PATCH] Add a release note for 2.3

2014-07-13 Thread Timothy Gu
Based on a patch by Clément Bœsch . Signed-off-by: Timothy Gu --- RELEASE_NOTES | 170 ++ 1 file changed, 170 insertions(+) create mode 100644 RELEASE_NOTES diff --git a/RELEASE_NOTES b/RELEASE_NOTES new file mode 100644 index 000..1a

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Lukasz Marek a écrit : > I am not sure about that. I agree it is a bit strange libavformat > has both (de)muxers and protocols. Some protocols are tied tightly to the corresponding muxer or demuxer, or the other way around. And anyway, what good would it do sep

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread wm4
On Sun, 13 Jul 2014 18:54:16 +0100 Derek Buitenhuis wrote: > Personally I've always thought it would be preferable to integrate something > like libcurl instead of writing our own implementations of every little > protocol, and the terrible mess that is our TLS support. +1 __

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 6:51 PM, Nicolas George wrote: > - You can not dig a hole without a shovel. > - Yes, you can, if someone already dug it. Your analogy is flawed, of course. If the hole is "accessing the file on a CIFS share", then it's more like: - You need to dig a hole, but you need a permit. - Pe

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 6:38 PM, Lukasz Marek wrote: > I am not sure about that. I agree it is a bit strange libavformat has > both (de)muxers and protocols. I would expect it to be separated, but > overall I don't don't agree it is bad ffmpeg has them at all. Such a library exists. It's called libcurl. >

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Subtitle maintainer

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 02:05:46PM +0200, Clément Bœsch wrote: > On Sun, Jul 13, 2014 at 03:32:19PM +0530, Anshul Maheshwari wrote: > > From: Anshul Maheshwari > > > > --- > > MAINTAINERS | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index d06030

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > You claimed its not possible, and it indeed is, with proper setup. - You can not dig a hole without a shovel. - Yes, you can, if someone already dug it. This is getting silly and boring, count me out. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 6:31 PM, Nicolas George wrote: > Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : >> You skillfully ignored the words "with proper setup", good job. > > I did not think you would insist on your "no true Scotsman" fallacy. > > Fact is that mounting is often not possible,

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 19:04, Derek Buitenhuis wrote: On 7/13/2014 5:54 PM, Lukasz Marek wrote: You also ignored the case where user might have no access to shell at all to mount remote fs. That's a pretty damn specialized case. Also, I wouldn't like a player (or anything else) mount anything under c

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > You skillfully ignored the words "with proper setup", good job. I did not think you would insist on your "no true Scotsman" fallacy. Fact is that mounting is often not possible, and running ffmpeg in that case is still legitimate, i

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Michael Niedermayer
On Mon, Jul 14, 2014 at 12:18:09AM +0800, Star Brilliant wrote: > The updated patch along with sample LRC file. > > Moved to SCNu64 during reading timestamp as Derek suggests. > > I have no idea on how to use FATE. Please help me, if possible, to add > this lrc file to the FATE dataset. Thanks.

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 06:09:09PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 5:36 PM, Star Brilliant wrote: > > The updated patch and sample file. > > Seems fairly reasonable to me. sample uploaded patch applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Star Brilliant
On 2014-07-14 01:09, Derek Buitenhuis wrote: On 7/13/2014 5:36 PM, Star Brilliant wrote: The updated patch and sample file. Seems fairly reasonable to me. So is there any problems now? (I'm just too impatient) Thanks. ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Star Brilliant
On 2014-07-14 01:09, Derek Buitenhuis wrote: On 7/13/2014 5:36 PM, Star Brilliant wrote: The updated patch and sample file. Seems fairly reasonable to me. So is there any problems now? (Sorry but I'm just too impatient) Thanks. ___ ffmpeg-devel ma

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:36 PM, Star Brilliant wrote: > The updated patch and sample file. Seems fairly reasonable to me. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:54 PM, Lukasz Marek wrote: > You also ignored the case where user might have no access to shell at > all to mount remote fs. That's a pretty damn specialized case. > Also, I wouldn't like a player (or anything else) mount anything under > carpet on my workstation. Case when I'm f

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 18:17, Derek Buitenhuis wrote: On 7/13/2014 5:10 PM, Nicolas George wrote: Technically, wrong. Technically right. Do you want an unprivileged account somewhere so I can dare you to mount something with FUSE (without using a local root exploit of course, that would be cheating)?

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 18:04, Derek Buitenhuis wrote: On 7/12/2014 5:20 PM, Lukasz Marek wrote: +libsmbclient_protocol_deps="libsmbclient gpl" Samba is actually GPLv3, so you need to add dependency on that. Fixed. >From 2218dd824524265e563c9df113c07179d74c982c Mon Sep 17 00:00:00 2001 From: Lukasz Ma

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Star Brilliant
On 2014-07-14 00:18, Star Brilliant wrote: The updated patch along with sample LRC file. Moved to SCNu64 during reading timestamp as Derek suggests. I have no idea on how to use FATE. Please help me, if possible, to add this lrc file to the FATE dataset. Thanks. Wish to see it in FFmpeg 2.4 re

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:21 PM, Michael Niedermayer wrote: > but id like to remove thi file descriptor based API before the relase > so noone has to hit the same msvc issue I am OK with it, but others may want to comment, particularly its author. - Derek ___ ffmp

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 04:54:00PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 3:01 PM, Michael Niedermayer wrote: > > MSVC does not allow passing file pointers between libs > > This API can thus not work with MSVC and as it was very recently added > > and its it was in no release its removial sh

Re: [FFmpeg-devel] [PATCH] AVFormat: LRC demuxer and muxer

2014-07-13 Thread Star Brilliant
The updated patch along with sample LRC file. Moved to SCNu64 during reading timestamp as Derek suggests. I have no idea on how to use FATE. Please help me, if possible, to add this lrc file to the FATE dataset. Thanks. Wish to see it in FFmpeg 2.4 release. >From 5ba3c51eaa8a46f813acdea344e987c

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:10 PM, Nicolas George wrote: >> Technically, wrong. > > Technically right. Do you want an unprivileged account somewhere so I can > dare you to mount something with FUSE (without using a local root exploit of > course, that would be cheating)? You skillfully ignored the words "with

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > Technically, wrong. Technically right. Do you want an unprivileged account somewhere so I can dare you to mount something with FUSE (without using a local root exploit of course, that would be cheating)? > You're going to mount a re

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/12/2014 5:20 PM, Lukasz Marek wrote: > +libsmbclient_protocol_deps="libsmbclient gpl" Samba is actually GPLv3, so you need to add dependency on that. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/f

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 4:58 PM, Nicolas George wrote: > Not wrong, merely too concise for the inattentive reader. Technically, wrong. Since with proper setup you *can* achieve the goal. > And indeed, I know of several hosting services where FUSE is not enabled. > Low-end VPS services frequently fall in tha

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > Wrong. FUSE lets you mount samba shares as non-root. Not wrong, merely too concise for the inattentive reader. You need root privileges to install FUSE on a system and enable it for any or all users: FUSE does not allow to mount wit

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 3:01 PM, Michael Niedermayer wrote: > MSVC does not allow passing file pointers between libs > This API can thus not work with MSVC and as it was very recently added > and its it was in no release its removial should not cause any problems > > Found-by: Hendrik Leppkes > Signed-off-b

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 2:39 PM, Lukasz Marek wrote: > - it is right you say for desktops where you have access to shell (and > root?), but how about cases where you create some embedded system > without shell access or have no root? FUSE allows non-rot mounts. > - you should still consider users that are

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 2:36 PM, Nicolas George wrote: > and mounting requires root privileges. Wrong. FUSE lets you mount samba shares as non-root. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] Support for metadata in libavutil/eval.c

2014-07-13 Thread Stepan Bujnak
Hi, I was thinking about introducing support for meta() function in the expression evaluation language. As the ffmpeg filters gradually add support for metadata, it would be convenient to do something like this: > ffmpeg -i input.webm -vf > cropdetect,crop=x=meta('lavfi.cropdetect.x'):y=meta('la

Re: [FFmpeg-devel] [PATCH 1/4] armv6: Accelerate ff_imdct_half for general case (mdct_bits != 6)

2014-07-13 Thread Michael Niedermayer
On Fri, Jul 11, 2014 at 12:14:28AM +0100, Ben Avison wrote: > The previous implementation targeted DTS Coherent Acoustics, which only > requires mdct_bits == 6. This relatively small size lent itself to > unrolling the loops a small number of times, and encoding offsets > calculated at assembly tim

Re: [FFmpeg-devel] [PATCH] AVFoundation: Add -pixel_format option

2014-07-13 Thread Hanspeter Niederstrasser
This lets the build work on 10.7 (and seems to fail properly if I change the version to >= 1070) --- 8< --- 8< --- 8< --- 8< --- 8< --- diff --git a/libavdevice/avfoundation.m b/libavdevice/avfoundation.m index df322c6..74d7811 100644 --- a/libavdevice/avfoundation.m +++ b/libavdevice/avfoundation

[FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Michael Niedermayer
MSVC does not allow passing file pointers between libs This API can thus not work with MSVC and as it was very recently added and its it was in no release its removial should not cause any problems Found-by: Hendrik Leppkes Signed-off-by: Michael Niedermayer --- libavdevice/lavfi.c | 39 +

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 15:22, wm4 wrote: On Sat, 12 Jul 2014 18:20:46 +0200 Lukasz Marek wrote: --- Changelog | 1 + configure | 5 ++ doc/general.texi | 1 + doc/protocols.texi | 29 +++ libavformat/Makefile | 1 + libavform

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, wm4 a écrit : > You can mount samba shares directly in most (all?) desktop OSes. FFmpeg is not used on desktop, and mounting requires root privileges. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 13:47, Michael Niedermayer wrote: On Sun, Jul 13, 2014 at 12:55:47AM +0200, Lukasz Marek wrote: On 12.07.2014 22:31, Michael Niedermayer wrote: On Sat, Jul 12, 2014 at 06:20:46PM +0200, Lukasz Marek wrote: die_license_disabled gpl frei0r die_license_disabled gpl libcdio +die_

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread wm4
On Sat, 12 Jul 2014 18:20:46 +0200 Lukasz Marek wrote: > --- > Changelog | 1 + > configure | 5 ++ > doc/general.texi | 1 + > doc/protocols.texi | 29 +++ > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + >

Re: [FFmpeg-devel] [PATCH] avutil: actually install hash.h

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 09:54:42AM +0200, Reimar Döffinger wrote: > On 06.07.2014, at 06:09, Timothy Gu wrote: > > This header is designed as a public header (with APIchanges entry and > > everything), but it is forgotten to put into the headers to be installed > > list. > > Note I think that ori

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Subtitle maintainer

2014-07-13 Thread Clément Bœsch
On Sun, Jul 13, 2014 at 03:32:19PM +0530, Anshul Maheshwari wrote: > From: Anshul Maheshwari > > --- > MAINTAINERS | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index d06030b..65c5ce0 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -132,6 +132,8 @@ G

Re: [FFmpeg-devel] [PATCH] x86/hevc_deblock: add ff_hevc_[hv]_loop_filter_luma_{8, 10}_sse2

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 12:39:31PM +0100, Kieran Kunhya wrote: > On 13 July 2014 07:00, James Almer wrote: > > Signed-off-by: James Almer > > LGTM. There's the odd use of 3-operand in some of the macros. Have you applied thx > benched AVX? > > Kieran > __

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 12:55:47AM +0200, Lukasz Marek wrote: > On 12.07.2014 22:31, Michael Niedermayer wrote: > >On Sat, Jul 12, 2014 at 06:20:46PM +0200, Lukasz Marek wrote: > >>--- > >> Changelog | 1 + > >> configure | 5 ++ > >> doc/general.texi

Re: [FFmpeg-devel] [PATCH] x86/hevc_deblock: add ff_hevc_[hv]_loop_filter_luma_{8, 10}_sse2

2014-07-13 Thread Kieran Kunhya
On 13 July 2014 07:00, James Almer wrote: > Signed-off-by: James Almer LGTM. There's the odd use of 3-operand in some of the macros. Have you benched AVX? Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/

Re: [FFmpeg-devel] [PATCH] h264: fix direct temporal mvs for bottom-field-first poc order.

2014-07-13 Thread Michael Niedermayer
On Sat, Jul 12, 2014 at 06:38:13PM -0400, Ronald S. Bultje wrote: > Fixes http://forum.doom9.org/showthread.php?t=170867. > --- > libavcodec/h264_direct.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF1336

[FFmpeg-devel] [PATCH] MAINTAINERS: add Subtitle maintainer

2014-07-13 Thread Anshul Maheshwari
From: Anshul Maheshwari --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index d06030b..65c5ce0 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -132,6 +132,8 @@ Generic Parts: tableprint.c, tableprint.h Reimar Doeffinger fixed point F

Re: [FFmpeg-devel] [PATCH] avutil: actually install hash.h

2014-07-13 Thread Reimar Döffinger
On 06.07.2014, at 06:09, Timothy Gu wrote: > This header is designed as a public header (with APIchanges entry and > everything), but it is forgotten to put into the headers to be installed > list. Note I think that originally the API was considered bad and subject to change, thus it wasn't inst