Re: [FFmpeg-devel] [PATCH 0/5] RGB48/64 with bits_per_raw_sample

2014-08-23 Thread Carl Eugen Hoyos
Christophe Gisquet gmail.com> writes: > 2014-08-20 20:26 GMT+02:00 Carl Eugen Hoyos: > > Christophe Gisquet writes: > > > >> Depending on the input and/or filters, you sometime > >> have an input or output pixel format like "rgb48le(12 > >> bpc)". Unfortunately, most often, the 12 bpc > >> inform

Re: [FFmpeg-devel] [PATCH] lavc/tiff: add support for LZMA compression

2014-08-23 Thread Michael Niedermayer
On Sat, Aug 23, 2014 at 11:57:04AM -0300, James Almer wrote: > On 23/08/14 8:37 AM, Hendrik Leppkes wrote: > > On Sat, Aug 23, 2014 at 1:31 PM, Michael Niedermayer > > wrote: > >> On Fri, Aug 22, 2014 at 11:39:57PM -0300, James Almer wrote: > >>> Derived from deflate code. > >>> Requires liblzma.

Re: [FFmpeg-devel] [PATCH]Do not show an unreadable error code if mov header parsing fails

2014-08-23 Thread Michael Niedermayer
On Sun, Aug 24, 2014 at 01:16:16AM +0200, Carl Eugen Hoyos wrote: > On Sunday 24 August 2014 12:50:14 am Michael Niedermayer wrote: > > On Sat, Aug 23, 2014 at 02:02:49PM +0200, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > I don't think showing "-1094995529" or similar makes much sense. > > > The

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-08-23 Thread Michael Niedermayer
On Sun, Jul 20, 2014 at 02:23:07PM -0700, Timothy Gu wrote: > texi2html is deprecated by upstream in favor of makeinfo/texi2any. See: > > - https://www.gnu.org/software/texinfo/manual/texinfo/html_node/texi2html.html > - https://wiki.debian.org/Texi2htmlTransition > - https://lists.debian.org/debi

Re: [FFmpeg-devel] [PATCH]Do not show an unreadable error code if mov header parsing fails

2014-08-23 Thread Carl Eugen Hoyos
On Sunday 24 August 2014 12:50:14 am Michael Niedermayer wrote: > On Sat, Aug 23, 2014 at 02:02:49PM +0200, Carl Eugen Hoyos wrote: > > Hi! > > > > I don't think showing "-1094995529" or similar makes much sense. > > The calling application can still decide to show the error > > string (as ffmpeg d

Re: [FFmpeg-devel] Filters

2014-08-23 Thread JULIAN GARDNER
> > From: JULIAN GARDNER >To: FFmpeg development discussions and patches >Sent: Monday, 18 August 2014, 19:51 >Subject: Re: [FFmpeg-devel] Filters > > > >> >> From: Clément Bœsch >>To: FFmpeg development discussions and pat

Re: [FFmpeg-devel] [PATCH]Do not show an unreadable error code if mov header parsing fails

2014-08-23 Thread Michael Niedermayer
On Sat, Aug 23, 2014 at 02:02:49PM +0200, Carl Eugen Hoyos wrote: > Hi! > > I don't think showing "-1094995529" or similar makes much sense. > The calling application can still decide to show the error > string (as ffmpeg does). > > Please comment, Carl Eugen > mov.c |2 +- > 1 file change

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

2014-08-23 Thread Clément Bœsch
On Sun, Aug 24, 2014 at 12:27:09AM +0200, Clément Bœsch 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

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

2014-08-23 Thread Clément Bœsch
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 patches that will make API evolutions. So the projects can continue to drop

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

2014-08-23 Thread Clément Bœsch
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 patches that will make API evolutions. So the projects can continue to drop

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

2014-08-23 Thread 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 [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He who knows, do

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-23 Thread Kieran Kunhya
> but either way, id like to suggest again, we move forward and > rather discuss how we can improve the situation, do something about > the split and move toward un-doing it! We look forward to seeing you in Dublin then. ___ ffmpeg-devel mailing list ffm

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

2014-08-23 Thread Deti Fliegl
On 19.08.14 15:43, Carl Eugen Hoyos wrote: Deti Fliegl fliegl.de> writes: Minor update to propagate field dominance. At least a Changelog entry and a libavdevice version bump are still missing, but consider waiting for a real review. Well in the meantime I updated my patch once again to add

Re: [FFmpeg-devel] [PATCH 2/2] Correct few "ffmpeg" typos

2014-08-23 Thread Clément Bœsch
On Sat, Aug 23, 2014 at 07:40:22PM +, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/aasc.c| 2 +- > libavcodec/cinepak.c | 2 +- > libavcodec/cyuv.c| 2 +- > libavcodec/flicvideo.c | 2 +- > libavcodec/idcinvideo.c | 2 +- > li

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/xan: fix style issue

2014-08-23 Thread Clément Bœsch
On Sat, Aug 23, 2014 at 07:40:21PM +, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/xan.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/xan.c b/libavcodec/xan.c > index 7489113..bb28916 100644 > --- a/libavcodec/xan.c > +++ b/libav

[FFmpeg-devel] [PATCH 1/2] avcodec/xan: fix style issue

2014-08-23 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/xan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/xan.c b/libavcodec/xan.c index 7489113..bb28916 100644 --- a/libavcodec/xan.c +++ b/libavcodec/xan.c @@ -556,7 +556,7 @@ static int xan_decode_frame(AVCodecContext *avct

[FFmpeg-devel] [PATCH 2/2] Correct few "ffmpeg" typos

2014-08-23 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/aasc.c| 2 +- libavcodec/cinepak.c | 2 +- libavcodec/cyuv.c| 2 +- libavcodec/flicvideo.c | 2 +- libavcodec/idcinvideo.c | 2 +- libavcodec/interplayvideo.c | 2 +- libavcodec/libopencore-amr.c | 2 +- libavc

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

2014-08-23 Thread Clément Bœsch
--- This is 100% untested and probably doesn't even compile. Can anyone with PPC/Altivec HW test or provide such access? After the altivec optims are ported we can drop the duplicated version in libavcodec entirely. The fate-pixelutils tests should cover the alignment checks; there might be some

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-23 Thread Michael Niedermayer
Hi On Thu, Aug 21, 2014 at 03:16:50AM +0200, Attila Kinali wrote: > Servus, > > On Wed, 20 Aug 2014 18:43:18 +0900 > Norbert Preining wrote: > > > By continuing old fights, inspite of the very clearly friendly and > > open offers and suggestions byu Michael, you and others from AV continue > >

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-08-23 Thread Timothy Gu
Hi, On Aug 23, 2014 9:52 AM, "Michael Niedermayer" wrote: > > On Sat, Aug 23, 2014 at 03:13:50PM +0200, Andreas Cadhalpun wrote: > > Is there a reason, why this patch hasn't been applied? > > the makeinfo available on our webserver is too old (4.13) > a newer version (provided by timothy), did n

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

2014-08-23 Thread Hii
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 parameters functional. Also make b_qfactor changeable by x26

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-08-23 Thread Michael Niedermayer
On Sat, Aug 23, 2014 at 03:13:50PM +0200, Andreas Cadhalpun wrote: > Hi, > > On 20.07.2014 23:23, Timothy Gu wrote: > >texi2html is deprecated by upstream in favor of makeinfo/texi2any. See: > > > >- > >https://www.gnu.org/software/texinfo/manual/texinfo/html_node/texi2html.html > >- https://wiki

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread Christophe Gisquet
Hi, 2014-08-23 17:48 GMT+02:00 Mickaël Raulet : > For avx2 I have some to push to the trunk, I did merge it yesterday with > all recent changes. But I don t remember what those tables looks like. Well, my point was hypothetical, but I guess this means some conflicts are to be expected when either

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread Mickaël Raulet
For avx2 I have some to push to the trunk, I did merge it yesterday with all recent changes. But I don t remember what those tables looks like. For 10 and 12bits, ssse3 should slow down the decoding since it uses 4 more instructions in the loop. Le samedi 23 août 2014, Christophe Gisquet a écrit

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread James Almer
On 23/08/14 12:20 PM, Christophe Gisquet wrote: > Hi, > > 2014-08-23 17:16 GMT+02:00 James Almer : >>> What do you mean by duplicated? That tables for 10 and 12 are? > [...] >> I was talking about the opt suffix since both the ssse3 and sse4 tables will >> be the same. > > Oh ok, in case we have

Re: [FFmpeg-devel] [PATCH] lavc/tiff: add support for LZMA compression

2014-08-23 Thread James Almer
On 23/08/14 8:31 AM, Michael Niedermayer wrote: > On Fri, Aug 22, 2014 at 11:39:57PM -0300, James Almer wrote: >> Derived from deflate code. >> Requires liblzma. >> >> Signed-off-by: James Almer > > how can this be tested ? > do you have a testcase / sample ? libtiff can create lzma compressed f

Re: [FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread James Almer
On 23/08/14 12:15 PM, Christophe Gisquet wrote: > Hi, > > 2014-08-23 17:01 GMT+02:00 James Almer : There's a PACK macro in lavfi/x86/yasm-16.asm that does this without >>> intrinsics. >>> >>> You meant yadif-16, right? >>> >>> Timothy >> >> Oops, yes i meant that :P > > I expect it to be nee

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread Christophe Gisquet
Hi, 2014-08-23 17:16 GMT+02:00 James Almer : >> What do you mean by duplicated? That tables for 10 and 12 are? [...] > I was talking about the opt suffix since both the ssse3 and sse4 tables will > be the same. Oh ok, in case we have to instantiate sse4 versions. Because at the moment there are o

Re: [FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread Christophe Gisquet
Hi, 2014-08-23 17:01 GMT+02:00 James Almer : >>> There's a PACK macro in lavfi/x86/yasm-16.asm that does this without >> intrinsics. >> >> You meant yadif-16, right? >> >> Timothy > > Oops, yes i meant that :P I expect it to be needed for the weighted pred functions, so I'll split it from yadif-1

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread James Almer
On 23/08/14 12:11 PM, Christophe Gisquet wrote: > Hi, > > 2014-08-23 16:52 GMT+02:00 James Almer : >>> -QPEL_TABLE 8, 8, b, sse4 >>> -QPEL_TABLE 10, 4, w, sse4 >>> -QPEL_TABLE 12, 4, w, sse4 >>> +QPEL_TABLE 8, 8, b, ssse3 >>> +QPEL_TABLE 10, 4, w, ssse3 >>> +QPEL_TABLE 12, 4, w, ssse3 >> >> Do t

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread Christophe Gisquet
Hi, 2014-08-23 16:52 GMT+02:00 James Almer : >> -QPEL_TABLE 8, 8, b, sse4 >> -QPEL_TABLE 10, 4, w, sse4 >> -QPEL_TABLE 12, 4, w, sse4 >> +QPEL_TABLE 8, 8, b, ssse3 >> +QPEL_TABLE 10, 4, w, ssse3 >> +QPEL_TABLE 12, 4, w, ssse3 > > Do these need to be duplicated? You could just remove the suffix a

Re: [FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread James Almer
On 23/08/14 11:55 AM, Timothy Gu wrote: > On Aug 23, 2014 7:47 AM, "James Almer" wrote: >> >> On 23/08/14 11:07 AM, Mickaël Raulet wrote: >>> For 10bits and 12bits, they should stay sse4 as well because of > packusdw. You need some instructions to convert it to ssse3 see below >>> >>> >>> static a

Re: [FFmpeg-devel] [PATCH] lavc/tiff: add support for LZMA compression

2014-08-23 Thread James Almer
On 23/08/14 8:37 AM, Hendrik Leppkes wrote: > On Sat, Aug 23, 2014 at 1:31 PM, Michael Niedermayer wrote: >> On Fri, Aug 22, 2014 at 11:39:57PM -0300, James Almer wrote: >>> Derived from deflate code. >>> Requires liblzma. >>> >>> Signed-off-by: James Almer >> >> how can this be tested ? >> do yo

Re: [FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread Timothy Gu
On Aug 23, 2014 7:47 AM, "James Almer" wrote: > > On 23/08/14 11:07 AM, Mickaël Raulet wrote: > > For 10bits and 12bits, they should stay sse4 as well because of packusdw. You need some instructions to convert it to ssse3 see below > > > > > > static av_always_inline __m128i _MM_PACKUS_EPI32( __m1

Re: [FFmpeg-devel] [PATCH 2/2] x86: hevc_mc: convert to ssse3

2014-08-23 Thread James Almer
On 23/08/14 10:22 AM, Christophe Gisquet wrote: > 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,

Re: [FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread James Almer
On 23/08/14 11:07 AM, Mickaël Raulet wrote: > For 10bits and 12bits, they should stay sse4 as well because of packusdw. You > need some instructions to convert it to ssse3 see below > > > static av_always_inline __m128i _MM_PACKUS_EPI32( __m128i a, __m128i b ) > { > a = _mm_slli_epi32 (a, 1

Re: [FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread Mickaël Raulet
For 10bits and 12bits, they should stay sse4 as well because of packusdw. You need some instructions to convert it to ssse3 see below static av_always_inline __m128i _MM_PACKUS_EPI32( __m128i a, __m128i b ) { a = _mm_slli_epi32 (a, 16); a = _mm_srai_epi32 (a, 16); b = _mm_slli_epi

Re: [FFmpeg-devel] [PATCH 3/4] dict.c: Free non-strduped av_dict_set arguments on error.

2014-08-23 Thread Reimar Döffinger
On Fri, Aug 22, 2014 at 07:47:09AM +0200, Reimar Döffinger wrote: > On 22.08.2014, at 07:36, Reimar Döffinger wrote: > > On 22.08.2014, at 04:06, Michael Niedermayer wrote: > >> On Sat, Aug 16, 2014 at 02:43:46PM +0200, Reimar Döffinger wrote: > >>> On Wed, Aug 13, 2014 at 01:57:56PM +0200, Micha

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

2014-08-23 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 Af

[FFmpeg-devel] [PATCH 0/2] x86: hevc_mc: port to SSSE3

2014-08-23 Thread Christophe Gisquet
As far as I can see, the only reason those functions are SSE4 is because of the pextrw needed for the following block widths: - 2, used only by chroma; - 6, used by chroma and indirectly by luma; - 12, used by both. The better solution would be to convert all chroma handling to NV12, but it is vas

[FFmpeg-devel] [PATCH 1/2] x86: hevc_mc: split differently calls

2014-08-23 Thread Christophe Gisquet
In some cases, 2 or 3 calls are performed to functions for unusual widths. Instead, perform 2 calls for different widths to split the workload. The 8+16 and 4+8 widths for respectively 8 and more than 8 bits can't be processed that way without modifications: some calls use unaligned buffers, and h

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-08-23 Thread Andreas Cadhalpun
Hi, On 20.07.2014 23:23, Timothy Gu wrote: texi2html is deprecated by upstream in favor of makeinfo/texi2any. See: - https://www.gnu.org/software/texinfo/manual/texinfo/html_node/texi2html.html - https://wiki.debian.org/Texi2htmlTransition - https://lists.debian.org/debian-devel/2013/05/msg0151

Re: [FFmpeg-devel] [PATCH] web/style: Add a comment in the generated css file for the doc in the main repo

2014-08-23 Thread Andreas Cadhalpun
Hi, On 25.07.2014 11:25, db0 company wrote: Since the style in the release is the one generated from the website, it has to be the same license. If we add a license to the css/less, then it's better to have the same on the whole website. I'll add one. Did you choose a license already? Best re

[FFmpeg-devel] [PATCH]Do not show an unreadable error code if mov header parsing fails

2014-08-23 Thread Carl Eugen Hoyos
Hi! I don't think showing "-1094995529" or similar makes much sense. The calling application can still decide to show the error string (as ffmpeg does). Please comment, Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index b3eb287..ae48c02 100644 --- a/libavformat/mov.c +++ b/libav

Re: [FFmpeg-devel] [PATCH] lavc/tiff: add support for LZMA compression

2014-08-23 Thread Hendrik Leppkes
On Sat, Aug 23, 2014 at 1:31 PM, Michael Niedermayer wrote: > On Fri, Aug 22, 2014 at 11:39:57PM -0300, James Almer wrote: >> Derived from deflate code. >> Requires liblzma. >> >> Signed-off-by: James Almer > > how can this be tested ? > do you have a testcase / sample ? > This image should be l

Re: [FFmpeg-devel] [PATCH] lavc/tiff: add support for LZMA compression

2014-08-23 Thread Michael Niedermayer
On Fri, Aug 22, 2014 at 11:39:57PM -0300, James Almer wrote: > Derived from deflate code. > Requires liblzma. > > Signed-off-by: James Almer how can this be tested ? do you have a testcase / sample ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst fo

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-23 Thread Andreas Cadhalpun
Hi, On 17.08.2014 00:49, Andreas Cadhalpun wrote: I have now sent the pkg-config patches to the BTS [1]. I have found a simpler way to make it possible to link packages not using pkg-config against FFmpeg in Debian: The lib*-ffmpeg-dev packages now install symbolic links from the standard li

Re: [FFmpeg-devel] [PATCH] avfilter/showcqt: add fontcolor option

2014-08-23 Thread Muhammad Faiz
On Sat, Aug 23, 2014 at 3:09 PM, Clément Bœsch wrote: > Next time please send a git-format-patch (git commit ... && git > format-patch -1, or use git send-email) instead, so it saves the committer > the time to copy/paste the commit message and author. > > Thank's for the suggestion Muhammad Fai

Re: [FFmpeg-devel] [PATCH 3/5] avutil/pixelutils: faster pixelutils_sad_[au]_16x16

2014-08-23 Thread Clément Bœsch
On Sun, Aug 17, 2014 at 01:51:13PM +0200, Michael Niedermayer wrote: > On Thu, Aug 14, 2014 at 11:05:13PM +0200, Clément Bœsch wrote: > > ~560 → ~500 decicycles > > > > This is following the comments from Michael in > > https://ffmpeg.org/pipermail/ffmpeg-devel/2014-August/160599.html > > > > Usi

Re: [FFmpeg-devel] [PATCH] avfilter/showcqt: add fontcolor option

2014-08-23 Thread Clément Bœsch
On Sat, Aug 23, 2014 at 05:51:31AM +0700, Muhammad Faiz wrote: [...] > Update patch to the latest master branch. > > Thank's. > > Muhammad Faiz > > --- > doc/filters.texi | 26 > libavfilter/avf_showcqt.c | 102 > -- > libavfi