Re: [FFmpeg-devel] [PATCH 9/9] hevc: reorder loops

2014-07-10 Thread Michael Niedermayer
On Thu, Jul 10, 2014 at 06:47:56AM +, Christophe Gisquet wrote: --- libavcodec/hevc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Awnsering whenever a program halts

Re: [FFmpeg-devel] [PATCH] configure: disable MMX only if runtime cpu detect was disabled

2014-07-10 Thread Michael Niedermayer
On Thu, Jul 10, 2014 at 07:53:25PM +0900, KO Myung-Hun wrote: --- configure |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 658efb2..c5e854d 100755 --- a/configure +++ b/configure @@ -3784,7 +3784,7 @@ elif enabled x86; then

Re: [FFmpeg-devel] [PATCH 5/9] hevc: derive partial merge list

2014-07-10 Thread Michael Niedermayer
On Thu, Jul 10, 2014 at 06:47:52AM +, Christophe Gisquet wrote: The merge list only needs to be derived up to the merge index. --- libavcodec/hevc_mvs.c | 35 ++- 1 file changed, 26 insertions(+), 9 deletions(-) applied thanks [...] -- Michael GnuPG

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 timothyg...@gmail.com 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

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

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 Leppkes

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

2014-07-14 Thread Michael Niedermayer
On Tue, Jul 15, 2014 at 01:05:22AM +0530, Anshul Maheshwari wrote: On 7/13/14, Michael Niedermayer michae...@gmx.at wrote: 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

Re: [FFmpeg-devel] [PATCH] lavd/x11grab: add an option to disable MIT-SHM.

2014-07-15 Thread Michael Niedermayer
On Tue, Jul 15, 2014 at 11:40:16AM +0200, Nicolas George wrote: With remote displays supporting the MIT-SHM extension, the extension is detected and used, but attaching fails asynchronously. Signed-off-by: Nicolas George geo...@nsup.org --- doc/indevs.texi | 4

[FFmpeg-devel] [PATCH] avcodec/mjpegenc_common: always store JFIF, idependent of the aspect ratio

2014-07-15 Thread Michael Niedermayer
Found-by: kriegero1 Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/mjpegenc_common.c | 20 tests/ref/lavf/jpg |4 ++-- tests/ref/lavf/smjpeg |4 ++-- tests/ref/vsynth/vsynth1-ljpeg |4 ++-- tests/ref

Re: [FFmpeg-devel] [PATCH] lavfi: check refcount before merging.

2014-07-15 Thread Michael Niedermayer
On Tue, Jul 15, 2014 at 07:30:35PM +0200, Nicolas George wrote: When merging the formats around the automatically inserted convert filters, the refcount of the format lists can not be 0. Coverity does not detect it, and suspects a memory leak, because if refcount is 0 the newly allocated lists

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

2014-07-15 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 Leppkes

Re: [FFmpeg-devel] [PATCH v2] Blackframe video filter now sets metadata accordingly.

2014-07-15 Thread Michael Niedermayer
On Fri, Jul 11, 2014 at 11:40:07PM +0200, Stepan Bujnak wrote: the libavfilter/vf_blackframe.c filter now not only logs detected values, but also sets frame metadata. Currently, only `pblack` value is set but `SET_META` macro has been introduced to ease development in the future.

Re: [FFmpeg-devel] [PATCH v3] drawtext: Add basic text shaping using libfribidi - Fixes ticket #3758

2014-07-15 Thread Michael Niedermayer
On Sat, Jun 21, 2014 at 05:41:45AM +0100, Marc Jeffreys wrote: Regarding positioning of the '|' operator: Unless anyone objects, I think it's fine being at the end of the line (if anything, that's the style used for '||' in vf_drawtext.c). ---8--- --- configure |3 +

Re: [FFmpeg-devel] [PATCH]Write valid free atom with -moov_size

2014-07-16 Thread Michael Niedermayer
On Wed, Jul 16, 2014 at 01:45:47PM +0200, Carl Eugen Hoyos wrote: Hi! Attached patch fixes ticket #3769. Please review, Carl Eugen movenc.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) c040bdb810885856a278402be659e75e3afb8d1a patchmoovfree.diff diff --git

Re: [FFmpeg-devel] [PATCH] avcodec/mjpegenc_common: always store JFIF, idependent of the aspect ratio

2014-07-16 Thread Michael Niedermayer
On Wed, Jul 16, 2014 at 08:18:58AM +0200, Reimar Döffinger wrote: On 15.07.2014, at 16:31, Michael Niedermayer michae...@gmx.at wrote: Found-by: kriegero1 Signed-off-by: Michael Niedermayer michae...@gmx.at Nit: iNdependent in commit message. fixed +/* JFIF header

Re: [FFmpeg-devel] [PATCH 1/3] get_bits: remove unused assignment

2014-07-16 Thread Michael Niedermayer
On Tue, Jul 15, 2014 at 08:56:28PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavcodec/get_bits.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The

Re: [FFmpeg-devel] Issue 3052

2014-07-16 Thread Michael Niedermayer
On Wed, Jul 16, 2014 at 05:40:35PM +0200, Benoit Fouet wrote: - Mail original - Hi, I had a look at https://trac.ffmpeg.org/ticket/3052. The issue is that the last frame duration is lost. Is there any reason why the packet duration is not transmitted from the demuxer to

Re: [FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

2014-07-16 Thread Michael Niedermayer
On Wed, Jul 16, 2014 at 05:31:47PM +0200, Clément Bœsch wrote: On Wed, Jul 16, 2014 at 05:06:27PM +0200, Clément Bœsch wrote: Also add an example exporting the MB information as a CSV stream. --- A bunch of questions stand: - Ideally the source for a given macroblock should be a

Re: [FFmpeg-devel] [PATCH 3/3] aaccoder: remove unused assignment

2014-07-16 Thread Michael Niedermayer
On Tue, Jul 15, 2014 at 08:56:30PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavcodec/aaccoder.c | 2 -- 1 file changed, 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The educated differ

Re: [FFmpeg-devel] [PATCH 3/4] armv6: Accelerate vector_fmul_window

2014-07-16 Thread Michael Niedermayer
On Fri, Jul 11, 2014 at 12:14:30AM +0100, Ben Avison wrote: I benchmarked the result by measuring the number of gperftools samples that hit anywhere in the AAC decoder (starting from aac_decode_frame()) or specifically in vector_fmul_window_c() / ff_vector_fmul_window_vfp() for the same sample

Re: [FFmpeg-devel] [PATCH 4/4] armv6: Accelerate butterflies_float

2014-07-16 Thread Michael Niedermayer
On Fri, Jul 11, 2014 at 12:14:31AM +0100, Ben Avison wrote: I benchmarked the result by measuring the number of gperftools samples that hit anywhere in the AAC decoder (starting from aac_decode_frame()) or specifically in butterflies_float_c() / ff_butterflies_float_vfp() for the same sample

[FFmpeg-devel] [PATCH] avformat/dvenc: Implement 32khz 44.1khz for 25/50fps

2014-07-16 Thread Michael Niedermayer
3/1001 fps isnt implemented as i dont know which are the correct interleaving parameters, pointers to the appropriate spec and page are welcome Fixes Ticket2597 Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/dvenc.c | 54

[FFmpeg-devel] [PATCH] avformat/dv: implement fallback in dv_extract_pack()

2014-07-16 Thread Michael Niedermayer
Fixes Ticket2340 Fixes Ticket2341 Based-on mail from Dave Rice d...@dericed.com Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/dv.c | 43 +++ 1 file changed, 23 insertions(+), 20 deletions(-) diff --git a/libavformat/dv.c b

Re: [FFmpeg-devel] [PATCH] avformat/dv: implement fallback in dv_extract_pack()

2014-07-17 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 07:37:58AM -0400, Dave Rice wrote: On Jul 16, 2014, at 10:57 PM, Michael Niedermayer michae...@gmx.at wrote: Fixes Ticket2340 Fixes Ticket2341 I tested with the samples from tickets 2340 and 2341. In both cases the patch fixes the originally reported issue

Re: [FFmpeg-devel] [Updated PATCH 2/4] armv6: Accelerate ff_fft_calc for general case (nbits != 4)

2014-07-17 Thread Michael Niedermayer
On Fri, Jul 11, 2014 at 11:32:08AM +0100, Ben Avison wrote: The previous implementation targeted DTS Coherent Acoustics, which only requires nbits == 4 (fft16()). This case was (and still is) linked directly rather than being indirected through ff_fft_calc_vfp(), but now the full range from

Re: [FFmpeg-devel] [PATCH 1/2] proresenc_anatoliy: use tables in proresdata.c instead of redefining them

2014-07-17 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 06:04:06PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavcodec/Makefile | 4 ++-- libavcodec/proresenc_anatoliy.c | 43 ++--- 2 files changed, 16 insertions(+), 31 deletions(-) it

Re: [FFmpeg-devel] [PATCH 2/2] proresenc: move profile handling to global avcodec options

2014-07-17 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 06:04:07PM -0700, Timothy Gu wrote: This allows users to use the two encoders with the same syntax. Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavcodec/avcodec.h| 5 + libavcodec/options_table.h | 5 +

Re: [FFmpeg-devel] [PATCH 2/4] kerndeint: remove dead assignments

2014-07-17 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 07:25:41PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavfilter/vf_kerndeint.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 4/4] a64multienc: remove dead assignment

2014-07-17 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 07:25:43PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavcodec/a64multienc.c | 1 - 1 file changed, 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Freedom in

Re: [FFmpeg-devel] [PATCH 2/2] alpha/me_cmp: move code out of dsputil

2014-07-17 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 11:45:35PM -0300, James Almer wrote: Signed-off-by: James Almer jamr...@gmail.com --- Untested. libavcodec/alpha/Makefile | 5 ++- libavcodec/alpha/dsputil_alpha.c | 40 --

Re: [FFmpeg-devel] [PATCH 3/4] smacker: remove dead code

2014-07-18 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 10:04:05AM +0200, Paul B Mahol wrote: On Fri, Jul 18, 2014 at 4:25 AM, Timothy Gu timothyg...@gmail.com wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavcodec/smacker.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/smacker.c

Re: [FFmpeg-devel] [PATCH] ffmpeg_filter: refuse to configure input without a decoder.

2014-07-18 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 10:43:22AM +0200, Nicolas George wrote: The decoder is necessary in order to filter frames. This makes the error message clearer in this case: currently, it will usually fail because the pixel or sample format is not defined and is converted into (null) (non-portable).

Re: [FFmpeg-devel] [PATCH] lavfi/drawtext: add alias expr_int_format to expansion function eif

2014-07-18 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 07:53:48AM +0200, Stefano Sabatini wrote: On date Friday 2014-07-18 01:00:40 +0300, Andrey Utkin encoded: --- doc/filters.texi | 2 +- libavfilter/vf_drawtext.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/doc/filters.texi

Re: [FFmpeg-devel] [PATCH 1/7] avformat/hlsenc: correctly compute target duration

2014-07-18 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 10:57:41AM +0200, Nicolas Martyanoff wrote: With HLS, the duration of all segments must be lower or equal to the target duration. Therefore floor(duration + 0.5) yields incorrect results. For example, for duration = 1.35, floor(duration + 0.5) yields 1.0, but the

Re: [FFmpeg-devel] [PATCH] lavf/segment: do not allow to create segments with no key-frames

2014-07-18 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 08:39:16PM +0200, Stefano Sabatini wrote: Fix trac ticket #3749. --- libavformat/segment.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) probably ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who

Re: [FFmpeg-devel] [PATCH] oss_audio: use a macro to simplify ioctl() error checking

2014-07-18 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 02:56:16PM -0700, Timothy Gu wrote: Also add a note about SNDCTL_DSP_GETFMTS which may fail even if OSS is available. Signed-off-by: Timothy Gu timothyg...@gmail.com --- libavdevice/oss_audio.c | 27 ++- 1 file changed, 14 insertions(+), 13

Re: [FFmpeg-devel] [PATCH] avformat/dvenc: Implement 32khz 44.1khz for 25/50fps

2014-07-19 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 02:02:30AM +0200, Michael Niedermayer wrote: 3/1001 fps isnt implemented as i dont know which are the correct interleaving parameters, pointers to the appropriate spec and page are welcome Fixes Ticket2597 Signed-off-by: Michael Niedermayer michae...@gmx.at

Re: [FFmpeg-devel] new webpage design patches

2014-07-20 Thread Michael Niedermayer
On Sun, Jul 20, 2014 at 10:15:03AM +0200, Clément Bœsch wrote: On Sun, Jul 20, 2014 at 10:11:39AM +0200, Christophe Gisquet wrote: Hi, 2014-07-15 10:10 GMT+02:00 Clément Bœsch u...@pkh.me: The new website can be previewed @ http://db0.galo.pe/ffmpeg-web/ Some comments: - The

Re: [FFmpeg-devel] [PATCH] libavu: add pkt_timebase to AVFrame.

2014-07-20 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 01:12:39PM +0200, Michael Niedermayer wrote: On Fri, Jul 18, 2014 at 12:47:06PM +0200, Hendrik Leppkes wrote: Am 18.07.2014 12:04 schrieb Benoit Fouet benoit.fo...@free.fr: In order to easily correlate pkt_duration to its real duration, add the packet time base

Re: [FFmpeg-devel] [PATCH] libavu: add pkt_timebase to AVFrame.

2014-07-20 Thread Michael Niedermayer
On Sun, Jul 20, 2014 at 04:26:01PM +0200, wm4 wrote: On Fri, 18 Jul 2014 13:12:39 +0200 Michael Niedermayer michae...@gmx.at wrote: On Fri, Jul 18, 2014 at 12:47:06PM +0200, Hendrik Leppkes wrote: Am 18.07.2014 12:04 schrieb Benoit Fouet benoit.fo...@free.fr: In order to easily

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add db0 for the website

2014-07-20 Thread Michael Niedermayer
On Sun, Jul 20, 2014 at 04:38:49PM +0200, db0company wrote: --- MAINTAINERS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have never wished to cater to the crowd; for what I

Re: [FFmpeg-devel] [PATCH] libavu: add pkt_timebase to AVFrame.

2014-07-20 Thread Michael Niedermayer
On Sun, Jul 20, 2014 at 05:27:16PM +0200, wm4 wrote: On Sun, 20 Jul 2014 17:01:42 +0200 Michael Niedermayer michae...@gmx.at wrote: [...] also some codecs allocate multiple AVFrames with different dimensions hevc is one. and hypothetical future support of things like spatial scalability

Re: [FFmpeg-devel] [PATCH] avfilter/biquads: avoid clipping when using floating type

2014-07-20 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 03:19:25PM +0700, Muhammad Faiz wrote: I think clipping isn't needed when using floating type. Thanks From 3ffce13b5b118d84b724f2aec342baa6536271da Mon Sep 17 00:00:00 2001 From: Muhammad Faiz mfc...@gmail.com Date: Thu, 17 Jul 2014 10:49:04 +0700 Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH] avutil/opt.c: fix bug when parsing option

2014-07-21 Thread Michael Niedermayer
On Mon, Jul 21, 2014 at 02:31:41AM -0700, Muhammad Faiz wrote: fix these strange behavior: -af flanger=delay=10 [OK] -af flanger=delay=1e1 [OK] -af flanger=delay=1e+1 [Error] -af flanger=delay=0.1 [OK] -af flanger=delay=1e-1 [Error] thank's --- libavutil/opt.c | 2 +- 1 file

Re: [FFmpeg-devel] [PATCH] avformat: Add image3 demuxers with format autodetection

2014-07-21 Thread Michael Niedermayer
On Tue, Jul 22, 2014 at 03:34:45AM +0200, Andreas Cadhalpun wrote: On 22.07.2014 02:12, Andreas Cadhalpun wrote: On 21.07.2014 23:18, Carl Eugen Hoyos wrote: Andreas Cadhalpun andreas.cadhalpun at googlemail.com writes: This patch has a strange side effect. With it gst-libav1.0 (version

Re: [FFmpeg-devel] [PATCH] x86: hevcdsp: align

2014-07-23 Thread Michael Niedermayer
On Wed, Jul 23, 2014 at 10:08:37PM +0200, Mickaël Raulet wrote: Le 23 juil. 2014 à 19:33, Christophe Gisquet christophe.gisq...@gmail.com a écrit : This should be mostly cosmetics. Okay. This would also need to clean all transform_dc_ since they are not use anymore with

[FFmpeg-devel] [PATCH] avformat/rtpenc_chain: Set timebase

2014-07-23 Thread Michael Niedermayer
Avoid warning that its not set Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/rtpenc_chain.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/rtpenc_chain.c b/libavformat/rtpenc_chain.c index dea1f70..a023450 100644 --- a/libavformat/rtpenc_chain.c +++ b

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

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 08:06:43AM +0200, db0company wrote: --- README |2 +- src/css_credit |6 ++ 2 files changed, 7 insertions(+), 1 deletions(-) create mode 100644 src/css_credit applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] Test of mail system pllease ignore

2014-07-24 Thread Michael Niedermayer
Hi Tim On Thu, Jul 24, 2014 at 12:48:11PM +0100, tim nicholson wrote: Should munge the from headers for the record, works great also, thanks alot for working on / fixing yahoo/DMARC/DKIM on ffmpeg/mplayer MLs [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is?wrong

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 12:50:55PM +0200, Carl Eugen Hoyos wrote: On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote: Carl Eugen Hoyos cehoyos at ag.or.at writes: Hi! Attached patch fixes ticket #3777 for me, analyzed by Oliver Fromme. What about streaming growing files? It's

Re: [FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is wrong

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 03:15:56PM +0200, Carl Eugen Hoyos wrote: On Thursday 24 July 2014 03:00:02 pm Michael Niedermayer wrote: On Thu, Jul 24, 2014 at 12:50:55PM +0200, Carl Eugen Hoyos wrote: On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote: Carl Eugen Hoyos cehoyos

Re: [FFmpeg-devel] [PATCH] ffplay: remove dead code

2014-07-24 Thread Michael Niedermayer
Hi On Thu, Jul 17, 2014 at 06:02:02PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- ffplay.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index af0e199..8abbeae 100644 --- a/ffplay.c +++ b/ffplay.c @@ -775,8 +775,6 @@ static

Re: [FFmpeg-devel] [PATCH 1/4] lavfi: add filter messages infrastructure.

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 03:39:42PM +0200, Nicolas George wrote: Messaages are passed as frames with a negative format code, only to filter that declare supporting them. The only message for now is EOF; this mechanism augments the current mechanism based on request_frame() returning

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

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 01:17:23PM -0700, Timothy Gu wrote: [...] The cons: - Anchor names are changed. are there any linkes that use these anchors ? also applying this patchset will not affect how our server generates the html, as it does not execute files from git. If its

Re: [FFmpeg-devel] [PATCH] x86: hevc_mc: replace simple leas by adds

2014-07-26 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 08:23:21PM +0200, Christophe Gisquet wrote: Hi, the impact is relatively important (3 to 25 cycles, ie up to 2%) so I also include bench.patch if anybody wants to confirm the timings. Although I thought openhevc's MC code was faster, the benchmarked functions are

Re: [FFmpeg-devel] [PATCH] configure: replace pulse-simple with pulse

2014-07-26 Thread Michael Niedermayer
On Sat, Jul 26, 2014 at 03:38:27AM +0200, Lukasz Marek wrote: pulse-simple is additional library built on top of PulseAudio. It causes ffmpeg to link to library which is not needed anymore. This commit removes that redundant dependency. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com

[FFmpeg-devel] [PATCH 3/3] avcodec: drop dsp context init functions from public ABI

2014-07-26 Thread Michael Niedermayer
This can be applied when we bump soname the next time Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/libavcodec.v |4 1 file changed, 4 deletions(-) diff --git a/libavcodec/libavcodec.v b/libavcodec/libavcodec.v index b4bd2c8..b753222 100644 --- a/libavcodec

[FFmpeg-devel] [PATCH 1/3] avcodec: add avdsp

2014-07-26 Thread Michael Niedermayer
This provides a public sustainable API/ABI for DSP functions. Only externally used dsp functions are included. The structure is extensible without ABI issues compared to the existing dsp contexts. Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/Makefile |1 + libavcodec

[FFmpeg-devel] [PATCH 2/3] avfilter: use AVDSP

2014-07-26 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavfilter/deshake.h |5 ++--- libavfilter/f_select.c | 20 libavfilter/vf_deshake.c| 16 +++- libavfilter/vf_mpdecimate.c | 26 +- libavfilter/vf_spp.c

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: add avdsp

2014-07-26 Thread Michael Niedermayer
On Sat, Jul 26, 2014 at 09:58:16AM -0400, Ronald S. Bultje wrote: Hi, On Sat, Jul 26, 2014 at 9:34 AM, Michael Niedermayer michae...@gmx.at wrote: This provides a public sustainable API/ABI for DSP functions. Only externally used dsp functions are included. The structure

Re: [FFmpeg-devel] [PATCH] x86/hevc_idct: replace old and unused idct functions

2014-07-26 Thread Michael Niedermayer
On Sat, Jul 26, 2014 at 11:50:05AM +0200, Mickaël Raulet wrote: Hi, Only 8-bit and 10-bit idct_dc() functions are included (adding others should be trivial). Benchmarks on an Intel Core i5-4200U: idct8x8_dc SSE2 MMXEXT C cycles 22 26 57 idct16x16_dc

Re: [FFmpeg-devel] [PATCH] Add support for Opus in MPEG-TS

2014-07-26 Thread Michael Niedermayer
On Sat, Jul 26, 2014 at 03:06:14PM +0100, Kieran Kunhya wrote: --- libavcodec/opus.c| 41 +++ libavcodec/opus.h| 11 + libavcodec/opus_parser.c | 101 -- libavcodec/opusdec.c |4 +-

Re: [FFmpeg-devel] [PATCH] x86/hevc_idct: add 12bit idct_dc

2014-07-26 Thread Michael Niedermayer
On Sat, Jul 26, 2014 at 09:17:26PM +0200, Mickaël Raulet wrote: Hi, Signed-off-by: James Almer jamr...@gmail.com --- Didn't realize that the newest HEVC fate tests were 12 bit, so i didn't bother adding what i thought i couldn't feasibly test. And as i mentioned in the previous

Re: [FFmpeg-devel] [PATCH] avcodec: add avdct

2014-07-27 Thread Michael Niedermayer
On Sun, Jul 27, 2014 at 03:41:55PM +0200, Michael Niedermayer wrote: This provides a public sustainable API/ABI for DCT functions. Only externally used dct functions are included. The structure is extensible without ABI issues compared to the existing dct contexts. See Mailing list and IRC

Re: [FFmpeg-devel] [PATCH 2/3] avfilter: use AVDSP

2014-07-27 Thread Michael Niedermayer
On Sat, Jul 26, 2014 at 06:59:49PM +0200, Paul B Mahol wrote: On Sat, Jul 26, 2014 at 3:34 PM, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavfilter/deshake.h |5 ++--- libavfilter/f_select.c | 20

Re: [FFmpeg-devel] [PATCH] lavfi/setpts: allow to parse a metadata tag as a timestamp.

2014-07-27 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 08:01:24PM +0200, Nicolas George wrote: Signed-off-by: Nicolas George geo...@nsup.org --- doc/filters.texi | 22 ++ libavfilter/setpts.c | 64 ++- libavfilter/version.h | 2 +- 3 files changed,

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote: This works around ABI issues with applications which depend on libavutil internal values like sizeof(AVFrame) One such application is VLC 2.1.4 as well

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote: [...] in absence of further comments i might apply this patch to master and release/2.3

Re: [FFmpeg-devel] [PATCH 2/2] x86/hevc_deblock: load less data in hevc_h_loop_filter_luma_8

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 04:17:35PM -0300, James Almer wrote: Reading 8 bytes is enough. Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 16 1 file changed, 8 insertions(+), 8 deletions(-) applied thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 4/4] lavfi/vf_fps: accept EOF timestamp.

2014-07-28 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 03:39:45PM +0200, Nicolas George wrote: This makes the FPS filter duplicate the last frame to take its duration into account, exactly like the other ones. TODO find corresponding trac ticket(s). Signed-off-by: Nicolas George geo...@nsup.org ---

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 10:54:47PM +0200, Andreas Cadhalpun wrote: On 28.07.2014 20:51, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200

Re: [FFmpeg-devel] [PATCH] hevc_filter: run vertical and horizontal together

2014-07-28 Thread Michael Niedermayer
On Sun, Jul 27, 2014 at 10:27:35AM +0200, Christophe Gisquet wrote: One may interleave even more vertical and horizontal passes, but this is less straightforward (more code changes than just moves). -- Christophe hevc_filter.c | 90

Re: [FFmpeg-devel] [PATCH 1/4] lavfi: add filter messages infrastructure.

2014-07-28 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 01:29:16AM +0200, Nicolas George wrote: Le decadi 10 thermidor, an CCXXII, Michael Niedermayer a écrit : Breaks ABI, AVFrame size is depending on libavutil and may be non constant for a libavfilter build That was intentional (that makes the code simpler with less

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-29 Thread Michael Niedermayer
On Sun, Jul 27, 2014 at 10:52:26PM +0200, Carl Eugen Hoyos wrote: Hi! Some DVD dumps contain subtitles that start after more than 2G, attached patch should allow to specify sufficiently large values for -probesize. Please review, Carl Eugen doc/APIchanges |4

Re: [FFmpeg-devel] [PATCH 1/3] x86/hevc_deblock: remove some unnecessary instructions

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 04:30:11AM -0300, James Almer wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 2/3] x86/hevc_deblock: improve luma functions register allocation

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 04:30:12AM -0300, James Almer wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 88 - 1 file changed, 44 insertions(+), 44 deletions(-) applied thanks [...] -- Michael GnuPG

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

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 04:30:13AM -0300, James Almer wrote: ~5% faster than SSSE3 Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 10 +- libavcodec/x86/hevcdsp_init.c | 30 ++ 2 files changed, 39 insertions(+), 1

Re: [FFmpeg-devel] [PATCH 4/7] avformat/hlsenc: do not regenerate the playlist for each segment

2014-07-29 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 10:57:44AM +0200, Nicolas Martyanoff wrote: Since we need all segments to find out the target duration, we can only generate the playlist after writing all segments. There is no need to rewrite the segment list every time we create a new segment file. doesnt this

Re: [FFmpeg-devel] [PATCH 3/7] avformat/hlsenc: fix segmentation issues

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 03:40:54PM +0200, Michael Niedermayer wrote: [...] @@ -289,7 +321,7 @@ static int hls_write_header(AVFormatContext *ctx) ret = avformat_write_header(hls-ctx, NULL); if (ret 0) -return ret; +goto fail; return 0

Re: [FFmpeg-devel] [PATCH] movdec: remove nonsensical snprintf.

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 09:25:46PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavformat/mov.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

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

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 09:26:48PM +0200, Reimar Döffinger wrote: Unfortunately this was not explicitly documented and thus might be very risky. But basically all uses I saw in FFmpeg had a memleak in these cases. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de ---

Re: [FFmpeg-devel] [PATCH] web: remove projects page and associated dingleberries

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 12:58:04PM -0800, Lou Logan wrote: Alexander Strasser added a redirect on the web server, so this page is no longer needed. The current projects page is located on the wiki: https://trac.ffmpeg.org/wiki/Projects Signed-off-by: Lou Logan l...@lrcd.com ---

[FFmpeg-devel] [PATCH 1/2] avutil/frame: add av_frame_side_data_name()

2014-07-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavutil/frame.c | 14 ++ libavutil/frame.h |5 + 2 files changed, 19 insertions(+) diff --git a/libavutil/frame.c b/libavutil/frame.c index fdfbc46..2c62922 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c

[FFmpeg-devel] [PATCH 2/2] ffprobe: print some basic information about avframe side data

2014-07-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffprobe.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/ffprobe.c b/ffprobe.c index 1329466..114da50 100644 --- a/ffprobe.c +++ b/ffprobe.c @@ -123,6 +123,8 @@ typedef enum

Re: [FFmpeg-devel] [PATCH] ffprobe: print some basic information about avframe side data

2014-07-29 Thread Michael Niedermayer
On Tue, Jul 01, 2014 at 07:20:13PM +0200, Clément Bœsch wrote: On Tue, Jul 01, 2014 at 05:50:11PM +0200, Michael Niedermayer wrote: On Mon, Jun 30, 2014 at 06:37:45PM +0200, Clément Bœsch wrote: On Mon, Jun 30, 2014 at 04:16:17AM +0200, Michael Niedermayer wrote: Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH] add m4b to list of ipod playable files

2014-07-29 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 02:20:30AM +0200, Gerion Entrup wrote: m4b is the extension used by iDevices to detect audiobooks. --- libavformat/movenc.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] ffprobe: print some basic information about avframe side data

2014-07-30 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- doc/ffprobe.xsd | 11 +++ ffprobe.c | 21 - 2 files changed, 31 insertions(+), 1 deletion(-) diff --git a/doc/ffprobe.xsd b/doc/ffprobe.xsd index 9ef9ecb..e0e9179 100644 --- a/doc/ffprobe.xsd +++ b/doc

Re: [FFmpeg-devel] [PATCH 1/2] avutil/frame: add av_frame_side_data_name()

2014-07-30 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 02:18:47AM +0200, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavutil/frame.c | 14 ++ libavutil/frame.h |5 + 2 files changed, 19 insertions(+) patch applied with APIChages version bump

Re: [FFmpeg-devel] [PATCH] ffprobe: print some basic information about avframe side data

2014-07-30 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 05:21:15PM +0200, Stefano Sabatini wrote: On date Wednesday 2014-07-30 15:21:06 +0200, Michael Niedermayer encoded: Signed-off-by: Michael Niedermayer michae...@gmx.at --- doc/ffprobe.xsd | 11 +++ ffprobe.c | 21 - 2 files

Re: [FFmpeg-devel] [PATCH 1/4] dict.c: minor simplification.

2014-07-30 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 08:38:05PM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavutil/dict.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [FATESERVER/PATCH] Update to new website style

2014-07-31 Thread Michael Niedermayer
On Thu, Jul 31, 2014 at 11:55:59PM +0200, Michael Niedermayer wrote: On Thu, Jul 31, 2014 at 02:11:25PM -0700, Timothy Gu wrote: On Thu, Jul 31, 2014 at 12:54 PM, Michael Niedermayer michae...@gmx.at wrote: also light gray text on yellow/green is rather hard to read where

Re: [FFmpeg-devel] [PATCH] doc/indevs: mention required configure options

2014-07-31 Thread Michael Niedermayer
On Thu, Jul 31, 2014 at 03:04:18PM -0800, Lou Logan wrote: For x11grab, libcdio, and libdc1394 input devices. Signed-off-by: Lou Logan l...@lrcd.com --- doc/indevs.texi | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) LGTM [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] h264_mp4toannexb_bsf: always set idr_sps_pps_seen when SPS/PPS is seen.

2014-07-31 Thread Michael Niedermayer
On Thu, Jul 31, 2014 at 03:40:51PM +0200, Benoit Fouet wrote: In order not to break a sequence like SPS IDR SPS IDR, the boolean telling that the SPS/PPS has been seen should always be set. --- libavcodec/h264_mp4toannexb_bsf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM

[FFmpeg-devel] FFmpeg and OPW

2014-07-31 Thread Michael Niedermayer
Hi all OPW (Outreach Program for Women) is twice per year (compared to google summer of code which is just once a year) FFmpeg can participate in the next round but we need to fund at least 1 intern/student (6250 USD) for that. OPW is not run by a large corporation with deep pockets. Thus my

Re: [FFmpeg-devel] [PATCH] h264_mp4toannexb_bsf: always set idr_sps_pps_seen when SPS/PPS is seen.

2014-08-01 Thread Michael Niedermayer
On Fri, Aug 01, 2014 at 01:54:14AM +0200, Michael Niedermayer wrote: On Thu, Jul 31, 2014 at 03:40:51PM +0200, Benoit Fouet wrote: In order not to break a sequence like SPS IDR SPS IDR, the boolean telling that the SPS/PPS has been seen should always be set. --- libavcodec

Re: [FFmpeg-devel] [PATCH] h264: remove useless assignment.

2014-08-01 Thread Michael Niedermayer
On Thu, Jul 31, 2014 at 02:32:02PM +0200, Benoit Fouet wrote: source index, as well as dest one, is unconditionnaly set afterwards, before being effectively used. --- libavcodec/h264.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index

[FFmpeg-devel] [PATCH] avformat/util: change av_find_default_stream_index() to use a score based system

2014-08-01 Thread Michael Niedermayer
Disfavor video streams with unknown resolution and no packets Fixes seeking in audio-only-speex.flv Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/utils.c | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/libavformat/utils.c b

Re: [FFmpeg-devel] [PATCH] fix tls/tcp protocol after a 302 move in https

2014-08-01 Thread Michael Niedermayer
On Fri, Aug 01, 2014 at 08:37:27PM -0400, compn wrote: patch from https://trac.ffmpeg.org/ticket/3824 -compn diff --git a/libavformat/http.c b/libavformat/http.c index 33585b0..3dffaee 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -229,6 +229,7 @@ redo:

Re: [FFmpeg-devel] [PATCH 3/4] ffmpeg: use av_buffersrc_close().

2014-08-02 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 11:44:48PM +0200, Nicolas George wrote: Signed-off-by: Nicolas George geo...@nsup.org --- ffmpeg.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

2014-08-02 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 11:44:47PM +0200, Nicolas George wrote: Also deprecate adding a NULL frame to mark EOF. TODO APIchanges entry, version bump. Signed-off-by: Nicolas George geo...@nsup.org --- libavfilter/buffersrc.c | 40 ++--

  1   2   3   4   5   6   7   8   9   10   >