Re: [FFmpeg-devel] [PATCH] concatdec: Fix handling of H.264 in MP4 in case of "-auto_convert 0"

2016-04-26 Thread Nicolas George
Le septidi 7 floréal, an CCXXIV, Andrey Utkin a écrit : > Your patch breaks a particular usecase, however my patch handles it well. > [h264 @ 0x1aa1000] no frame! > Last message repeated 8 times > [mpegts @ 0x1aa4180] H.264 bitstream malformed, no startcode found, use the > video bitstream f

Re: [FFmpeg-devel] [PATCH 2/2] configure: build fix for P5600 with mips code restructuring

2016-04-26 Thread Michael Niedermayer
On Tue, Apr 26, 2016 at 12:17:15PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Note:- backporting commit 15ef98afd10b3696d29fb6d19606ba03a9dd47ad from head > > Signed-off-by: Shivraj Patil > --- > configure | 252 > ++---

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

2016-04-26 Thread KO Myung-Hun
Dave Yeo wrote: > On 04/24/16 05:53 PM, Michael Niedermayer wrote: >> On Sun, Apr 24, 2016 at 11:55:06AM -0700, Dave Yeo wrote: >>> On 04/23/16 12:53 PM, Michael Niedermayer wrote: On Wed, Apr 20, 2016 at 11:15:39PM -0700, Dave Yeo wrote: >> On 04/20/16 03:48 PM, Dmitriy Kuminov wrote: >

Re: [FFmpeg-devel] [PATCH v2 03/12] lavc/videotoolboxenc: Require hardware encoding

2016-04-26 Thread Carl Eugen Hoyos
Richard Kern gmail.com> writes: > >> +#if !TARGET_OS_IPHONE > >> +{ "allow_sw", "Allow software encoding", OFFSET(allow_sw), > > > > I have no strong opinion, but shouldn't the option > > always exist (but only work in some cases)? > iOS doesn’t support software encoding, and the dictionar

Re: [FFmpeg-devel] [PATCH v2 01/12] lavc/videotoolboxenc: Use shared pixel buffer pool

2016-04-26 Thread Carl Eugen Hoyos
Richard Kern gmail.com> writes: > >> static const enum AVPixelFormat pix_fmts[] = { > >> AV_PIX_FMT_NV12, > >> -#if !TARGET_OS_IPHONE > >> AV_PIX_FMT_YUV420P, > >> -#endif > >> AV_PIX_FMT_NONE > > > > Sorry: How is this related? > Using a shared pixel buffer pool also sets up an inte

Re: [FFmpeg-devel] [PATCH v2 2/2] avutil/log: added test case for av_log_format_line2

2016-04-26 Thread Michael Niedermayer
On Tue, Apr 26, 2016 at 08:31:06AM +0200, Andreas Weis wrote: > From: Andreas Weis > > --- > libavutil/log.c | 30 ++ > 1 file changed, 30 insertions(+) > > diff --git a/libavutil/log.c b/libavutil/log.c > index 0efba7a..6d192b6 100644 > --- a/libavutil/log.c > +++ b

[FFmpeg-devel] [PATCH] Bug fixed: wrong ticks_per_frame. For H.264 stream ticks_per_frame should be 2

2016-04-26 Thread Ivan Uskov
Hello All, Monday, April 25, 2016, 6:50:18 PM, you wrote: HL> On Mon, Apr 25, 2016 at 5:44 PM, Ivan Uskov wrote: >> Hello Derek, >> >> Monday, April 25, 2016, 4:50:28 PM, you wrote: >> >> DB> On 4/25/2016 2:14 PM, Ivan Uskov wrote: The attached patch does fixes the issue of frames

Re: [FFmpeg-devel] [PATCH] concatdec: Fix handling of H.264 in MP4 in case of "-auto_convert 0"

2016-04-26 Thread Andrey Utkin
On Tue, Apr 26, 2016 at 10:44:06AM +0200, Nicolas George wrote: > I can not reproduce this with my patches. Or more precisely, I can reproduce > it with only the second patch applied. Are you sure you tested with both > patches applied? Sorry, I didn't notice that both of your patches must be appl

Re: [FFmpeg-devel] [PATCH] lavf/concatdec: remove unrelated change during codecpar merge.

2016-04-26 Thread Nicolas George
Le septidi 7 floréal, an CCXXIV, Derek Buitenhuis a écrit : > I'm not opposed to it, as long as the API conversion is done in a reasonable > amount of time. Ok, both patches pushed after Andrey confirmed they work. And I start lookint more closely at the new API. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

2016-04-26 Thread Dmitriy Kuminov
On 2016-04-26 11:51:28 +, KO Myung-Hun said: However, ln_s is always set to 'cp -f' on OS/2. Please remove ln_s overriding codes as well. Yes, but my initial patch already removes this override (check the first hunk) so just applying it + Dave's should get it all done. -- Kind regards,

Re: [FFmpeg-devel] [PATCH v2 01/12] lavc/videotoolboxenc: Use shared pixel buffer pool

2016-04-26 Thread Richard Kern
> On Apr 26, 2016, at 8:14 AM, Carl Eugen Hoyos wrote: > > Richard Kern gmail.com> writes: > static const enum AVPixelFormat pix_fmts[] = { AV_PIX_FMT_NV12, -#if !TARGET_OS_IPHONE AV_PIX_FMT_YUV420P, -#endif AV_PIX_FMT_NONE >>> >>> Sorry: How is this rel

Re: [FFmpeg-devel] [PATCH] Bug fixed: wrong ticks_per_frame. For H.264 stream ticks_per_frame should be 2

2016-04-26 Thread wm4
On Tue, 26 Apr 2016 17:21:10 +0300 Ivan Uskov wrote: > From 75d3a992e3c11edd39607cdc0109928db7905638 Mon Sep 17 00:00:00 2001 > From: Ivan Uskov > Date: Tue, 26 Apr 2016 10:13:04 -0400 > Subject: [PATCH] Bug fixed: wrong ticks_per_frame. For H.264 stream > ticks_per_frame should be 2 > > --- >

Re: [FFmpeg-devel] [PATCH] Bug fixed: wrong ticks_per_frame. For H.264 stream ticks_per_frame should be 2

2016-04-26 Thread Derek Buitenhuis
On 4/26/2016 4:45 PM, wm4 wrote: > I can see that this code is run only for h264, and I can see that you > set the field to 2. The added comment adds no new information and is > useless. The ticks_per_frame docu literally says "should be 2 for h.264" or something. I only thought it should be in t

[FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-04-26 Thread Ronald S. Bultje
--- doc/filters.texi | 13 libavfilter/colorspacedsp.c | 9 +++ libavfilter/colorspacedsp.h | 6 ++ libavfilter/colorspacedsp_template.c | 128 +++ libavfilter/vf_colorspace.c | 53 ++- 5 files ch

[FFmpeg-devel] [RFC]lavc/adpcm: Support baby monitor adpcm_ms

2016-04-26 Thread Carl Eugen Hoyos
Hi! Attached patch fixes decoding a user-provided file here: https://dropfile.to/x7hQfJt I don't think there is a demuxer issue. Please comment, Carl Eugen diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c index 5d8aa41..5a11faa 100644 --- a/libavcodec/adpcm.c +++ b/libavcodec/adpcm.c @@ -840,

Re: [FFmpeg-devel] [PATCH] configure: Don't require nonfree for nvenc

2016-04-26 Thread Carl Eugen Hoyos
Timo Rothenpieler rothenpieler.org> writes: > As the nvEncodeApi.h header is now MIT licensed, this can be dropped. > The loaded CUDA and NVENC libraries are part of the nvidia driver, and > thus count as system libraries. If the explanation is correct, the patch is ok imo. Thank you, Carl Euge

Re: [FFmpeg-devel] [RFC]lavc/adpcm: Support baby monitor adpcm_ms

2016-04-26 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > Attached patch fixes decoding a user-provided file here: > https://dropfile.to/x7hQfJt I added the following comment locally: // Levana Ovia Baby Monitor prepends each audio frame with these two bytes Carl Eugen _

Re: [FFmpeg-devel] Refund request for FFmpeg at CLT 2016

2016-04-26 Thread Thomas Volkert
Hi, On 21.03.2016 21:49, Thilo Borgmann wrote: Am 21.03.16 um 20:42 schrieb Michael Niedermayer: On Mon, Mar 21, 2016 at 12:14:40PM +0100, Thilo Borgmann wrote: Hi, last weekend, the Chemnitzer Linux Tage in Germany took place and we had quite a good experience and contacts to our end-users.

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-04-26 Thread Derek Buitenhuis
On 4/26/2016 5:41 PM, Ronald S. Bultje wrote: > --- > doc/filters.texi | 13 > libavfilter/colorspacedsp.c | 9 +++ > libavfilter/colorspacedsp.h | 6 ++ > libavfilter/colorspacedsp_template.c | 128 > +++ > libavfilt

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-04-26 Thread Ronald S. Bultje
Hi, On Tue, Apr 26, 2016 at 2:17 PM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 4/26/2016 5:41 PM, Ronald S. Bultje wrote: > > --- > > doc/filters.texi | 13 > > libavfilter/colorspacedsp.c | 9 +++ > > libavfilter/colorspacedsp.h |

Re: [FFmpeg-devel] [PATCH 3/3] configure: Remove -Wredundant-decls on OS/2 to suppress noise.

2016-04-26 Thread Dmitriy Kuminov
On 2016-04-21 06:46:38 +, Reimar Döffinger said: Have something that results in adding -I/usr/include to the compile options or to C_INCLUDE_PATH? Probably your compiler is misconfigured to not consider that a system include path. Adding -isystem /usr/include should fix it. I finally had

Re: [FFmpeg-devel] [RFC]lavc/adpcm: Support baby monitor adpcm_ms

2016-04-26 Thread Paul B Mahol
On 4/26/16, Carl Eugen Hoyos wrote: > Carl Eugen Hoyos ag.or.at> writes: > >> Attached patch fixes decoding a user-provided file here: >> https://dropfile.to/x7hQfJt > > I added the following comment locally: > // Levana Ovia Baby Monitor prepends each audio frame with these two bytes Sorry, lif

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-04-26 Thread Michael Niedermayer
On Tue, Apr 26, 2016 at 12:41:02PM -0400, Ronald S. Bultje wrote: > --- > doc/filters.texi | 13 > libavfilter/colorspacedsp.c | 9 +++ > libavfilter/colorspacedsp.h | 6 ++ > libavfilter/colorspacedsp_template.c | 128 >

[FFmpeg-devel] [PATCH] avcodec/alsdec: Fix bitstream reading

2016-04-26 Thread Umair Khan
According to the latest ISO specification pdf, the byte align should be according to the patch. Tested the change with all the sample ALS files. This doesn't produce any bug. From 7af5076eabc28a441ed90f920bf1a6a1e0855c3f Mon Sep 17 00:00:00 2001 From: Umair Khan Date: Wed, 27 Apr 2016 09:04:18 +05

[FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-04-26 Thread Александр Слободенюк
Signed-off-by: Aleksandr.Slobodeniuk --- libavformat/riff.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/riff.c b/libavformat/riff.c index cf5a2ff..7a1f3bc 100644 --- a/libavformat/riff.c +++ b/libavformat/riff.c @@ -168,6 +168,7 @@ const AVCodecTag ff_codec_bmp_tags[] = {

[FFmpeg-devel] [PATCH v3 2/2] avutil/log: added test case for av_log_format_line2

2016-04-26 Thread Andreas Weis
From: Andreas Weis Signed-off-by: Andreas Weis --- libavutil/log.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/libavutil/log.c b/libavutil/log.c index 0efba7a..7e279ad 100644 --- a/libavutil/log.c +++ b/libavutil/log.c @@ -444,6 +444,17 @@ void avpriv_rep

[FFmpeg-devel] [PATCH v3 1/2] avutil/log: added av_log_format_line2 which returns buffer length

2016-04-26 Thread Andreas Weis
From: Andreas Weis The new function behaves the same as av_log_format_line, but also forwards the return value from the underlying snprintf call. This will allow callers to accurately determine the size requirements for the line buffer. Signed-off-by: Andreas Weis --- doc/APIchanges | 4