[FFmpeg-devel] [PATCH] lavc/ccaption_dec: remove usage of avctx->time_base

2016-03-28 Thread Clément Bœsch
lavc/utils already rescales avpkt->pts to sub->pts in AV_TIME_BASE_Q before calling the decode callback. This prevents from rescaling again into the decoder, and avoid the use of avctx->time_base which will disappear in the incoming codecpar merge. This commit also replaces the use of "20

[FFmpeg-devel] [PATCH] lavc/utils: use pkt_timebase for legacy subtitles timing code

2016-03-28 Thread Clément Bœsch
This is consistent with other AVSubtitle timing adjustments. --- libavcodec/utils.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index c625bbc..2c8fc9c 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2582,8

Re: [FFmpeg-devel] [PATCH] doc/developer.texi: Add a code of conduct

2016-03-26 Thread Clément Bœsch
On Sat, Mar 26, 2016 at 02:28:46PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 19 +++ > 1 file changed, 19 insertions(+) > > diff --git a/doc/developer.texi b/doc/developer.texi > index

Re: [FFmpeg-devel] [PATCH 0/4] fftw exploration (WIP)

2016-03-25 Thread Clément Bœsch
On Thu, Mar 24, 2016 at 05:50:48PM -0700, Ganesh Ajjanagadde wrote: > Ganesh Ajjanagadde (4): > configure: add fftw3 detection > lavc/fftw: add initial fftw wrapper > lavc/fft-test: add FFTW3 tests > lavc/fft-test: update benchmark code > Why? Using an external library for such an

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add ff_hscale_8_to_15_neon

2016-03-24 Thread Clément Bœsch
On Thu, Mar 24, 2016 at 09:35:01AM -0400, Ronald S. Bultje wrote: > Hi, > > On Mar 24, 2016 8:28 AM, "Clément Bœsch" <u...@pkh.me> wrote: > > > > From: Clément Bœsch <clem...@stupeflix.com> > > > > ./ffmpeg -nostats -f lavfi -i testsrc2=4k:d

[FFmpeg-devel] [PATCH] sws/aarch64: add ff_hscale_8_to_15_neon

2016-03-24 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> ./ffmpeg -nostats -f lavfi -i testsrc2=4k:d=2 -vf bench=start,scale=1024x1024,bench=stop -f null - before: t:0.489726 avg:0.489883 max:0.491852 min:0.489482 after: t:0.256515 avg:0.256458 max:0.256999 min:0.253755 --- Changes:

[FFmpeg-devel] [PATCH] sws/aarch64: add ff_hscale_8_to_15_neon

2016-03-23 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> ./ffmpeg -nostats -f lavfi -i testsrc2=4k:d=2 -vf bench=start,scale=1024x1024,bench=stop -f null - before: t:0.489726 avg:0.489883 max:0.491852 min:0.489482 after: t:0.259438 avg:0.257707 max:0.260125 min:0.255893 --- I don't reall

Re: [FFmpeg-devel] size=0, but av_malloc(1)

2016-03-23 Thread Clément Bœsch
On Wed, Mar 23, 2016 at 12:25:07PM +, Carl Eugen Hoyos wrote: > Chris Cunningham chromium.org> writes: > > > I'm seeing an interesting pattern [0][1] where we allocate 1 byte in places > > where I would expect no allocation to be necessary. Why is this being done? > > > > [0]

Re: [FFmpeg-devel] GP License question

2016-03-22 Thread Clément Bœsch
On Tue, Mar 22, 2016 at 11:29:52AM -0400, Aaron Boxer wrote: [...] > Does your codec take advantage of multiple cores? Or is it only single > threaded? ffmpeg -h decoder=jpeg2000 will answer your question [...] -- Clément B. signature.asc Description: PGP signature

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-21 Thread Clément Bœsch
On Mon, Mar 21, 2016 at 12:27:13AM +0100, Gerion Entrup wrote: > On Sonntag, 20. März 2016 16:52:36 CET Clément Bœsch wrote: > > On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > > > Good day, > > > > > > I attached the patch. The MPEG7 vide

Re: [FFmpeg-devel] [PATCH] lavf/dv: use c->sys->frame_size in dv_frame_offset()

2016-03-20 Thread Clément Bœsch
On Sun, Mar 20, 2016 at 07:18:43PM +0100, Michael Niedermayer wrote: > On Sun, Mar 20, 2016 at 04:20:32PM +0100, Clément Bœsch wrote: > > dv_frame_offset() is static and called only from dv_read_seek(), where > > c->sys->frame_size is already used. > > > > This

Re: [FFmpeg-devel] [PATCH] lavf/dv: do not check for c->sys

2016-03-20 Thread Clément Bœsch
On Sun, Mar 20, 2016 at 03:09:51PM +0100, Michael Niedermayer wrote: > On Sun, Mar 20, 2016 at 02:28:05PM +0100, Clément Bœsch wrote: > > dv_extract_video_info() is a static function called only from > > avpriv_dv_produce_packet(), where c->sys is made sure to be set. > > -

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Clément Bœsch
On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > Good day, > > I attached the patch. The MPEG7 video signature is a method to calculate a > fingerprint of a video and to compare two or more of this fingerprints. Most > of > the standard is implemented. I've done this work as

[FFmpeg-devel] [PATCH] lavf/dv: use c->sys->frame_size in dv_frame_offset()

2016-03-20 Thread Clément Bœsch
dv_frame_offset() is static and called only from dv_read_seek(), where c->sys->frame_size is already used. This simplifies the incoming codecpar merge where avctx->{coded_width,coded_height,time_base} are not accessible anymore. --- libavformat/dv.c | 7 +++ 1 file changed, 3 insertions(+),

[FFmpeg-devel] [PATCH] lavf/dv: do not check for c->sys

2016-03-20 Thread Clément Bœsch
dv_extract_video_info() is a static function called only from avpriv_dv_produce_packet(), where c->sys is made sure to be set. --- libavformat/dv.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavformat/dv.c b/libavformat/dv.c index 84c3061..b1f8759 100644 ---

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-20 Thread Clément Bœsch
On Wed, Mar 16, 2016 at 10:24:09PM +0100, Thilo Borgmann wrote: [...] Not sure if I'm commenting on the last version... > From b6b889c06edc7872f0a31fd0482793a199ddd28e Mon Sep 17 00:00:00 2001 > From: Thilo Borgmann > Date: Wed, 16 Mar 2016 22:18:20 +0100 > Subject:

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/dxva2_h264: Fix incorrect assert statement.

2016-03-19 Thread Clément Bœsch
On Wed, Mar 16, 2016 at 12:31:35PM +0100, Matt Oliver wrote: > ffmpeg | branch: master | Matt Oliver | Wed Mar 16 > 22:28:29 2016 +1100| [109dfed7fc265f3e071854d5e6de5dd7f82ff9fb] | committer: > Matt Oliver > > lavc/dxva2_h264: Fix incorrect assert statement. > >

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: unroll abs_pow34_v loop

2016-03-19 Thread Clément Bœsch
On Fri, Mar 18, 2016 at 10:12:14PM -0700, Ganesh Ajjanagadde wrote: > It seems like in all usages, size is a multiple of 4. This is documented > as an assert. > > Yields speedup in this function, and small speedup for aac encoding overall. > > Sample benchmark (Haswell, -march=native + GCC): >

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-14 Thread Clément Bœsch
On Mon, Mar 14, 2016 at 03:38:52PM +0100, Thilo Borgmann wrote: > Am 14.03.16 um 13:56 schrieb Clément Bœsch: > > On Mon, Mar 14, 2016 at 01:46:52PM +0100, Thilo Borgmann wrote: > >> Am 14.03.16 um 11:22 schrieb Clément Bœsch: > >>> [...] > >>>> +.

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-14 Thread Clément Bœsch
On Mon, Mar 14, 2016 at 01:46:52PM +0100, Thilo Borgmann wrote: > Am 14.03.16 um 11:22 schrieb Clément Bœsch: > > [...] > >> +./ffmpeg -f lavfi -i > >> nullsrc=s=100x100,coreimage=filter=CIQRCodeGenerator@@inputMessage=https\://FFmpeg.org/@@inputCorrectionLevel=H &

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-14 Thread Clément Bœsch
On Mon, Mar 14, 2016 at 10:19:59AM +0100, Thilo Borgmann wrote: > Am 13.03.16 um 22:01 schrieb Clément Bœsch: > > On Sun, Mar 13, 2016 at 09:09:39PM +0100, Thilo Borgmann wrote: > >> Am 13.03.16 um 20:55 schrieb Nicolas George: > >>> Le quartidi 24 ventôse, an

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-14 Thread Clément Bœsch
On Sun, Mar 13, 2016 at 09:09:39PM +0100, Thilo Borgmann wrote: > Am 13.03.16 um 20:55 schrieb Nicolas George: > > Le quartidi 24 ventôse, an CCXXIV, Thilo Borgmann a écrit : > >> +{ "list_filters", "list available filters", OFFSET(list_filters), > >> AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1,

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-13 Thread Clément Bœsch
On Sun, Mar 13, 2016 at 09:09:39PM +0100, Thilo Borgmann wrote: > Am 13.03.16 um 20:55 schrieb Nicolas George: > > Le quartidi 24 ventôse, an CCXXIV, Thilo Borgmann a écrit : > >> +{ "list_filters", "list available filters", OFFSET(list_filters), > >> AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1,

[FFmpeg-devel] [PATCH] sws/aarch64/yuv2rgb: use altmacro for the rgb combinations

2016-03-11 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- I just wanted to see if it was possible to do something less dumb than the current code. I'm not sure I will push this patch (.altmacro is known to cause issues and it's currently not used for aarch64), but that was just as an exercise. It

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-10 Thread Clément Bœsch
On Thu, Mar 10, 2016 at 06:12:57PM +0100, Gerion Entrup wrote: [...] > > - an USF demuxer which extracts the timing and text (with its markup) of > > every event, and put them into an AVPacket > > > > - introduce an USF codec and write a decoder that will transform the > > xml-like markup

Re: [FFmpeg-devel] [PATCH] avutil/frame: Assert that width/height/channels is 0 for the destination of av_frame*_ref()

2016-03-09 Thread Clément Bœsch
On Tue, Mar 08, 2016 at 09:14:58PM +0100, Michael Niedermayer wrote: > This should detect caes where these functions are called in unclean > destinations > --- > libavutil/frame.c |6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/libavutil/frame.c b/libavutil/frame.c > index

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-08 Thread Clément Bœsch
On Tue, Mar 08, 2016 at 10:33:44PM +0100, Gerion Entrup wrote: [...] > > For text subtitles, the ass field contains the text in ASS markup: indeed, > > we consider the ASS markup to be the best/least worst superset supporting > > almost every style of every other subtitles formats have, so it's

Re: [FFmpeg-devel] Subtitles for GSoC

2016-03-08 Thread Clément Bœsch
On Tue, Mar 08, 2016 at 06:21:12PM +0100, Gerion Entrup wrote: > Hello, > Hi, > my own ideas seems not to be suitable for GSoC, so I looked again on the > ideas page, > because I have high interest to do something for FFmpeg this summer. > > The project, that I find most interesting,

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: allow subtitle text format to be ASS without timing

2016-03-04 Thread Clément Bœsch
On Fri, Mar 04, 2016 at 04:22:44PM +0100, Michael Niedermayer wrote: > On Fri, Feb 26, 2016 at 10:01:51PM +0100, Clément Bœsch wrote: > > ffmpeg | branch: master | Clément Bœsch <u...@pkh.me> | Wed Jan 6 13:43:23 > > 2016 +0100| [29412821241050c846dbceaad4b9752857659977]

Re: [FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread Clément Bœsch
On Tue, Mar 01, 2016 at 10:41:32AM -0800, NagaChaitanya Vellanki wrote: > --- > libavutil/Makefile | 1 + > libavutil/color_utils.c | 52 > + > 2 files changed, 53 insertions(+) > I'm afraid testing the result of a switch with an array has

Re: [FFmpeg-devel] [PATCH v2] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-03-01 Thread Clément Bœsch
On Tue, Mar 01, 2016 at 05:18:36PM +0100, Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 11:11:36AM +0100, Clément Bœsch wrote: > > On Mon, Feb 29, 2016 at 10:55:49AM +0100, Clément Bœsch wrote: > > > From: Clément Bœsch <clem...@stupeflix.com> > > > >

Re: [FFmpeg-devel] [PATCH v2] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-03-01 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 10:55:49AM +0100, Clément Bœsch wrote: > From: Clément Bœsch <clem...@stupeflix.com> > > --- > Changes since latest version: > - remove unused 32-bit path > - make 16-bit path more accurate by mirroring the MMX code (still not > bitexact) > -

Re: [FFmpeg-devel] [PATCH] lavfi: add bench and abench filters

2016-03-01 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 07:37:35PM +0100, Paul B Mahol wrote: > On 2/29/16, Clement Boesch wrote: > > From: Clement Boesch > > > > --- > > TODO: doc, bump, Changelog > > > > example: -vf bench=start,gradfun,format=rgba,hqx,bench=stop > > Nice, LGTM with

Re: [FFmpeg-devel] [PATCH] submit_thread should also lock the mutex that guards the source avctx (prev_thread->avctx) when calling update_context_from_thread.

2016-02-29 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 11:47:14AM -0800, Wan-Teh Chang wrote: > On Fri, Feb 26, 2016 at 5:04 PM, Ronald S. Bultje wrote: > > > > To be a little bit more explicit, it should be relatively easy to > > accomplish this by changing the position of qscale in the relevant struct, >

Re: [FFmpeg-devel] [PATCH v8] VideoToolbox H.264 Encoder

2016-02-29 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 08:14:10PM +0100, Nicolas George wrote: > Le primidi 11 ventôse, an CCXXIV, Clement Boesch a écrit : > > > > +case H264_PROF_BASELINE: > > > > +switch (vtctx->level) { > > > > +case 0: *profile_level_val = > > > >

Re: [FFmpeg-devel] [PATCH v8] VideoToolbox H.264 Encoder

2016-02-29 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 06:57:15PM +0100, wm4 wrote: [...] > > +static bool get_vt_profile_level(AVCodecContext *avctx, > > + CFStringRef*profile_level_val) > > +{ > > +VTEncContext *vtctx = avctx->priv_data; > > +int64_t profile = vtctx->profile; > > +

Re: [FFmpeg-devel] [PATCH v7] VideoToolbox H.264 Encoder

2016-02-29 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 11:14:17PM +0800, Rick Kern wrote: [...] > +if (!profile) { > +//VideoToolbox auto-selects profile and level. > +return true; > +} else if (!av_strcasecmp("baseline", profile)) { > +if (!strcmp("auto", level)) *profile_level_val = >

[FFmpeg-devel] [PATCH] lavfi: add bench and abench filters

2016-02-29 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- TODO: doc, bump, Changelog example: -vf bench=start,gradfun,format=rgba,hqx,bench=stop --- libavfilter/Makefile | 2 + libavfilter/allfilters.c | 2 + libavfilter/f_bench.c| 132 +++ 3

Re: [FFmpeg-devel] [PATCH] build: add install_name_dir option

2016-02-29 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 02:56:52PM +0100, Clément Bœsch wrote: > On Thu, Feb 25, 2016 at 02:47:45PM +0100, Hendrik Leppkes wrote: > > On Thu, Feb 25, 2016 at 2:35 PM, Clément Bœsch <u...@pkh.me> wrote: > > > From: Clément Bœsch <clem...@stupeflix.com> > > >

[FFmpeg-devel] [PATCH v2] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-02-29 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- Changes since latest version: - remove unused 32-bit path - make 16-bit path more accurate by mirroring the MMX code (still not bitexact) - the code as originally trying to process 2 lines at a time to save chroma pre mult computations and

Re: [FFmpeg-devel] [PATCH 2/2] sws/yuv2rgb: clarify precision of coeff and offset for mmx code

2016-02-27 Thread Clément Bœsch
On Sat, Feb 27, 2016 at 01:06:08AM +0100, Michael Niedermayer wrote: > On Fri, Feb 26, 2016 at 10:34:46PM +0100, Clément Bœsch wrote: > > It makes easier looking at the difference with the generic code just > > below. > > --- > > libswscale/yuv2rgb.c | 12 ++--

Re: [FFmpeg-devel] [PATCH 1/2] sws/yuv2rgb: avoid a few ub on signed left shifts

2016-02-27 Thread Clément Bœsch
On Sat, Feb 27, 2016 at 01:05:35AM +0100, Michael Niedermayer wrote: > On Fri, Feb 26, 2016 at 10:34:45PM +0100, Clément Bœsch wrote: > > --- > > libswscale/yuv2rgb.c | 20 ++-- > > 1 file changed, 10 insertions(+), 10 deletions(-) > > LGTM > >

Re: [FFmpeg-devel] [PATCH] avutil/md5: fix unaligned loads

2016-02-26 Thread Clément Bœsch
On Wed, Feb 24, 2016 at 09:20:11AM +0100, Clément Bœsch wrote: > On Tue, Feb 23, 2016 at 10:40:08PM -0300, James Almer wrote: > [...] > > That aside, note that these runtime erros happen with every test using md5 > > and > > don't make ubsan register them as failed. In

[FFmpeg-devel] [PATCH 1/2] sws/yuv2rgb: avoid a few ub on signed left shifts

2016-02-26 Thread Clément Bœsch
--- libswscale/yuv2rgb.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libswscale/yuv2rgb.c b/libswscale/yuv2rgb.c index 3671fe3..9d79d79 100644 --- a/libswscale/yuv2rgb.c +++ b/libswscale/yuv2rgb.c @@ -824,18 +824,18 @@ av_cold int

[FFmpeg-devel] [PATCH 2/2] sws/yuv2rgb: clarify precision of coeff and offset for mmx code

2016-02-26 Thread Clément Bœsch
It makes easier looking at the difference with the generic code just below. --- libswscale/yuv2rgb.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libswscale/yuv2rgb.c b/libswscale/yuv2rgb.c index 9d79d79..62abb7d 100644 --- a/libswscale/yuv2rgb.c +++

Re: [FFmpeg-devel] [PATCH 6/6] lavc: deprecate decoded ass subtitles with timings

2016-02-26 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 01:08:33PM +0100, Clément Bœsch wrote: > --- > libavcodec/assenc.c| 2 ++ > libavcodec/avcodec.h | 2 ++ > libavcodec/movtextenc.c| 7 ++- > libavcodec/options_table.h | 6 ++ > libavcodec/srtenc.c| 7 ++- >

Re: [FFmpeg-devel] [PATCH 5/6] lavfi/ass: use ass_process_chunk() instead of ass_process_data()

2016-02-26 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 01:08:32PM +0100, Clément Bœsch wrote: > --- > libavfilter/vf_subtitles.c | 10 +- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c > index 63b22c3..9e55002 100644 &

Re: [FFmpeg-devel] [PATCH 4/6] ffmpeg: set sub_text_format to ass (without timing) by default

2016-02-26 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 01:08:31PM +0100, Clément Bœsch wrote: > --- > ffmpeg.c | 2 ++ > tests/ref/fate/sub-charenc | 2 +- > tests/ref/fate/sub-textenc | 66 > ++-- > tests/ref/fate/

Re: [FFmpeg-devel] [PATCH 3/6] lavc/options: add ass_ro_flush_noop to flags2

2016-02-26 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 01:08:30PM +0100, Clément Bœsch wrote: > --- > libavcodec/ass.c | 3 ++- > libavcodec/avcodec.h | 4 > libavcodec/ccaption_dec.c| 3 ++- > libavcodec/libzvbi-teletextdec.c | 3 ++- > libavcodec/movtext

Re: [FFmpeg-devel] [PATCH 2/6] lavc: allow subtitle text format to be ASS without timing

2016-02-26 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 02:10:15AM +0100, Michael Niedermayer wrote: > On Wed, Feb 24, 2016 at 03:37:53PM +0100, Clément Bœsch wrote: > > On Sun, Feb 21, 2016 at 06:34:43PM +0100, Michael Niedermayer wrote: > > > On Sun, Feb 21, 2016 at 01:08:29PM +0100, C

Re: [FFmpeg-devel] [PATCH 1/6] Kill timed SSA

2016-02-26 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 01:08:28PM +0100, Clément Bœsch wrote: > --- > libavcodec/Makefile | 4 ++-- > libavcodec/assdec.c | 60 > +++- > libavcodec/assenc.c | 14 ++-- > libavformat/assenc.c | 14 &g

Re: [FFmpeg-devel] [PATCH] mjpegdec: Do not assume unused plane pointer are NULL.

2016-02-26 Thread Clément Bœsch
On Fri, Feb 26, 2016 at 11:29:05AM +0100, wm4 wrote: > On Fri, 26 Feb 2016 02:38:13 +0100 > Michael Niedermayer wrote: > > > On Fri, Feb 26, 2016 at 12:15:19AM +0100, Reimar Döffinger wrote: > > > We do neither document nor check such a requirement > > > and for

Re: [FFmpeg-devel] [PATCH 2/2] img2dec: Add mime_type to image formats

2016-02-25 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 10:37:40PM +0100, Clément Bœsch wrote: > On Thu, Feb 25, 2016 at 10:35:39PM +0100, Clément Bœsch wrote: > > On Thu, Feb 25, 2016 at 04:30:13PM +, Carl Eugen Hoyos wrote: > > > Justin Ruggles writes: > > > > > > > In terms of how t

Re: [FFmpeg-devel] [PATCH 2/2] img2dec: Add mime_type to image formats

2016-02-25 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 10:35:39PM +0100, Clément Bœsch wrote: > On Thu, Feb 25, 2016 at 04:30:13PM +, Carl Eugen Hoyos wrote: > > Justin Ruggles writes: > > > > > In terms of how the score for a MIME type match compares with > > > those of the individual

Re: [FFmpeg-devel] [PATCH 2/2] img2dec: Add mime_type to image formats

2016-02-25 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 04:30:13PM +, Carl Eugen Hoyos wrote: > Justin Ruggles writes: > > > In terms of how the score for a MIME type match compares with > > those of the individual content matching probe functions, I'd > > say it makes sense. The stronger probing functions have a > >

Re: [FFmpeg-devel] [PATCH] configure: only check dispatch header on darwin

2016-02-25 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 03:34:37PM +, Josh de Kock wrote: > On 25/02/2016 14:00, Matthieu Bouron wrote: > >From: Matthieu Bouron > > > >Fixes build of lavd/jack on linux if dispatch happens to be available on > >this platform. dispatch, as well as its

Re: [FFmpeg-devel] [PATCH] build: add install_name_dir option

2016-02-25 Thread Clément Bœsch
On Thu, Feb 25, 2016 at 02:47:45PM +0100, Hendrik Leppkes wrote: > On Thu, Feb 25, 2016 at 2:35 PM, Clément Bœsch <u...@pkh.me> wrote: > > From: Clément Bœsch <clem...@stupeflix.com> > > > > This option is typically useful when cross-compiling dynamic libr

[FFmpeg-devel] [PATCH] build: add install_name_dir option

2016-02-25 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> This option is typically useful when cross-compiling dynamic libraries for iOS, with something such as --install_name_dir=@rpath --- configure | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 8

Re: [FFmpeg-devel] [PATCH 2/6] lavc: allow subtitle text format to be ASS without timing

2016-02-24 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 06:34:43PM +0100, Michael Niedermayer wrote: > On Sun, Feb 21, 2016 at 01:08:29PM +0100, Clément Bœsch wrote: > [...] > > diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h > > index aa8bfac..b493dd1 100644 > > --- a/libavcodec/o

Re: [FFmpeg-devel] Killing ASS timings in AVSubtitle ass text field

2016-02-24 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 01:08:27PM +0100, Clément Bœsch wrote: > [PATCH 1/6] Kill timed SSA > > This one is an old patch, which kills the usage of the old and unused > AV_CODEC_ID_SSA > > [PATCH 2/6] lavc: allow subtitle text format to be ASS without timing > > Th

Re: [FFmpeg-devel] [PATCH] avutil/md5: fix unaligned loads

2016-02-24 Thread Clément Bœsch
On Tue, Feb 23, 2016 at 10:40:08PM -0300, James Almer wrote: [...] > That aside, note that these runtime erros happen with every test using md5 and > don't make ubsan register them as failed. In the link above something else > made > it fail. It makes me wonder how many other ubsan errors are

Re: [FFmpeg-devel] [PATCH] web: "news" about FFmpeg 3.0

2016-02-22 Thread Clément Bœsch
On Mon, Feb 22, 2016 at 11:45:43PM +0100, Michael Niedermayer wrote: > Found-by: Ganesh Ajjanagadde > --- > src/index |9 + > 1 file changed, 9 insertions(+) > > diff --git a/src/index b/src/index > index 827d803..840ed10 100644 > --- a/src/index > +++ b/src/index > @@ -37,6 +37,15

Re: [FFmpeg-devel] Killing ASS timings in AVSubtitle ass text field

2016-02-21 Thread Clément Bœsch
On Sun, Feb 21, 2016 at 06:29:42PM +0100, Michael Niedermayer wrote: > On Sun, Feb 21, 2016 at 01:08:27PM +0100, Clément Bœsch wrote: > > [PATCH 1/6] Kill timed SSA > > > > This one is an old patch, which kills the usage of the old and unused > > AV_CODEC_ID_SSA >

Re: [FFmpeg-devel] [PATCH 2/2] lavc/srtdec: fix rounding errors in end times

2016-02-21 Thread Clément Bœsch
On Mon, Jan 11, 2016 at 03:58:18AM -0600, Rodger Combs wrote: > --- > libavcodec/srtdec.c | 4 +++ > tests/ref/fate/sub-textenc | 66 > ++-- > tests/ref/fate/sub-webvttenc | 66 > ++-- > 3 files changed,

[FFmpeg-devel] [PATCH 4/6] ffmpeg: set sub_text_format to ass (without timing) by default

2016-02-21 Thread Clément Bœsch
--- ffmpeg.c | 2 ++ tests/ref/fate/sub-charenc | 2 +- tests/ref/fate/sub-textenc | 66 ++-- tests/ref/fate/sub-webvttenc | 66 ++-- 4 files changed, 69 insertions(+), 67 deletions(-) diff

[FFmpeg-devel] [PATCH 6/6] lavc: deprecate decoded ass subtitles with timings

2016-02-21 Thread Clément Bœsch
--- libavcodec/assenc.c| 2 ++ libavcodec/avcodec.h | 2 ++ libavcodec/movtextenc.c| 7 ++- libavcodec/options_table.h | 6 ++ libavcodec/srtenc.c| 7 ++- libavcodec/utils.c | 4 libavcodec/version.h | 3 +++ libavcodec/webvttenc.c | 7

[FFmpeg-devel] [PATCH 3/6] lavc/options: add ass_ro_flush_noop to flags2

2016-02-21 Thread Clément Bœsch
--- libavcodec/ass.c | 3 ++- libavcodec/avcodec.h | 4 libavcodec/ccaption_dec.c| 3 ++- libavcodec/libzvbi-teletextdec.c | 3 ++- libavcodec/movtextdec.c | 3 ++- libavcodec/options_table.h | 1 + libavcodec/samidec.c | 3 ++-

[FFmpeg-devel] [PATCH 5/6] lavfi/ass: use ass_process_chunk() instead of ass_process_data()

2016-02-21 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 63b22c3..9e55002 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -393,6 +393,8 @@ static av_cold

[FFmpeg-devel] [PATCH 2/6] lavc: allow subtitle text format to be ASS without timing

2016-02-21 Thread Clément Bœsch
--- libavcodec/ass.c | 96 +++- libavcodec/ass.h | 56 ++--- libavcodec/ass_split.c | 49 ++ libavcodec/ass_split.h | 15 ++ libavcodec/assdec.c

[FFmpeg-devel] [PATCH 1/6] Kill timed SSA

2016-02-21 Thread Clément Bœsch
--- libavcodec/Makefile | 4 ++-- libavcodec/assdec.c | 60 +++- libavcodec/assenc.c | 14 ++-- libavformat/assenc.c | 14 libavformat/nut.c| 1 - 5 files changed, 24 insertions(+), 69 deletions(-) diff --git

[FFmpeg-devel] Killing ASS timings in AVSubtitle ass text field

2016-02-21 Thread Clément Bœsch
[PATCH 1/6] Kill timed SSA This one is an old patch, which kills the usage of the old and unused AV_CODEC_ID_SSA [PATCH 2/6] lavc: allow subtitle text format to be ASS without timing The big commit. Text based decoders do not have to rescale and insert the timing in the ASS text anymore. A

Re: [FFmpeg-devel] pngdsp add bytes padding

2016-02-18 Thread Clément Bœsch
On Thu, Feb 18, 2016 at 12:42:59PM +0100, Michael Niedermayer wrote: > On Thu, Feb 18, 2016 at 11:36:18AM +0100, Clément Bœsch wrote: > > Hi, > > > > While working on PNG DSP, I realized the filter up was doing extra > > overflow checks on the bytes non mul

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-02-18 Thread Clément Bœsch
On Thu, Feb 18, 2016 at 11:12:19AM -0300, James Almer wrote: > On 2/18/2016 10:42 AM, Clément Bœsch wrote: > > On Thu, Feb 18, 2016 at 02:40:06PM +0100, Clément Bœsch wrote: > >> On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: > >>>

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-02-18 Thread Clément Bœsch
On Thu, Feb 18, 2016 at 02:40:06PM +0100, Clément Bœsch wrote: > On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: > > Clément Bœsch pkh.me> writes: > > > > > It's not tested on real hardware (qemu only), so I have > > > no idea ab

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-02-18 Thread Clément Bœsch
On Thu, Feb 18, 2016 at 01:00:10PM +, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > It's not tested on real hardware (qemu only), so I have > > no idea about performance. > > I can test this on real hardware if you can provide > (exact) comma

[FFmpeg-devel] pngdsp add bytes padding

2016-02-18 Thread Clément Bœsch
Hi, While working on PNG DSP, I realized the filter up was doing extra overflow checks on the bytes non multiple of the long word size. This logic is present in both C and x86 ASM. Here is the current C code for reference: -- 8< // 0x7f7f7f7f or 0x7f7f7f7f7f7f7f7f

Re: [FFmpeg-devel] [PATCH 3/3] configure: fix --enable-x11grab

2016-02-17 Thread Clément Bœsch
On Wed, Feb 17, 2016 at 10:02:49PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > configure |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index d7cda01..b94830b 100755 > --- a/configure

Re: [FFmpeg-devel] [PATCH 1/3] configure: Use set_all to force the dependency refresh

2016-02-17 Thread Clément Bœsch
On Wed, Feb 17, 2016 at 10:02:47PM +0100, Michael Niedermayer wrote: > From: Luca Barbato > strange mail. > Allow to issue twice --enable options. > > Signed-off-by: Michael Niedermayer > --- > configure |2 +- > 1

Re: [FFmpeg-devel] [PATCH v3 2/2] lavc/ccaption_dec: implement special and extended character sets

2016-02-17 Thread Clément Bœsch
On Sun, Feb 14, 2016 at 06:11:54PM -0800, Aman Gupta wrote: > From: Aman Gupta > > character sets implemented as defined in > https://en.wikipedia.org/wiki/EIA-608#Characters > --- > libavcodec/ccaption_dec.c | 152 > +- > 1 file

[FFmpeg-devel] [PATCH] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-02-17 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- This code is mirroring the ARM version. It's not tested on real hardware (qemu only), so I have no idea about performance. This also means I didn't put any prefetch mechanism (which had a noticeable impact on performance on ARM).

Re: [FFmpeg-devel] [PATCH v2 2/2] lavc/ccaption_dec: implement special and extended character sets

2016-02-13 Thread Clément Bœsch
On Sat, Feb 13, 2016 at 12:35:18PM -0800, Aman Gupta wrote: [...] > +static const char *charset_overrides[4][128] = > +{ > +[CCSET_BASIC_AMERICAN] = { > +[0x27] = "’", > +[0x2a] = "á", > +[0x5c] = "é", > +[0x5e] = "í", > +[0x5f] = "ó", > +[0x60]

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-06 Thread Clément Bœsch
On Sat, Feb 06, 2016 at 04:26:43PM +0100, Andreas Cadhalpun wrote: > On 03.02.2016 00:41, Andreas Cadhalpun wrote: > > I see, it passes '-b $(SRC_PATH)' to lcov, which is now unnecessary/harmful > > for out-of-tree builds using the src link. > > The first attached patch changes it to simply use

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-06 Thread Clément Bœsch
On Wed, Feb 03, 2016 at 12:41:01AM +0100, Andreas Cadhalpun wrote: [...] > The second patch was necessary to fix the following error: > lcov: ERROR: cannot write to coverage.info! > > Does it work for you without that patch? > I confirm it fails the same way as you without the second patch. --

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Clément Bœsch
On Sat, Feb 06, 2016 at 06:37:56PM +0100, Mats Peterson wrote: > On 02/06/2016 06:25 PM, wm4 wrote: > >A regression in VLC. Make them fix it. > > > > > VLC = Very Lame Crap. Just kidding. > Please refrain from such comment, even for "kidding". Regards, -- Clément B. signature.asc

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-02 Thread Clément Bœsch
On Mon, Jan 25, 2016 at 08:51:32PM +0100, Andreas Cadhalpun wrote: > ffmpeg | branch: master | Andreas Cadhalpun > | Mon Jan 25 01:42:23 2016 +0100| > [b46aae093634271931395d65f422f4b2a23112d3] | committer: Andreas Cadhalpun > > build: use a link instead of

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-30 Thread Clément Bœsch
On Sat, Jan 30, 2016 at 05:28:28PM +0100, Michael Niedermayer wrote: > On Sat, Jan 30, 2016 at 02:23:10PM +0100, Clément Bœsch wrote: > > On Sun, Jan 24, 2016 at 07:00:30PM -0300, James Almer wrote: > > > On 1/24/2016 6:22 PM, Clément Bœsch wrote: > > > > On Sun, Ja

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-30 Thread Clément Bœsch
On Sun, Jan 24, 2016 at 07:00:30PM -0300, James Almer wrote: > On 1/24/2016 6:22 PM, Clément Bœsch wrote: > > On Sun, Jan 24, 2016 at 10:09:49PM +0100, Michael Niedermayer wrote: > > [...] > >> fails to build: > >> make distclean ; ../configure --enable-pthreads

Re: [FFmpeg-devel] [PATCH] avutil: Rename FF_CEIL_COMPAT to AV_CEIL_COMPAT

2016-01-27 Thread Clément Bœsch
On Wed, Jan 27, 2016 at 03:21:34PM +, Derek Buitenhuis wrote: > Libav, for some reason, merged this as a public API function. This will > aid in future merges. > > A define is left for backwards compat, just in case some person > used it, since it is in a public header. > > Signed-off-by:

Re: [FFmpeg-devel] [PATCH 2/2] avformat: add protocol_whitelist

2016-01-24 Thread Clément Bœsch
On Sun, Jan 24, 2016 at 12:47:59PM +0100, Nicolas George wrote: > Le quintidi 5 pluviôse, an CCXXIV, Michael Niedermayer a écrit : > > --- a/libavformat/avio.h > > +++ b/libavformat/avio.h > > @@ -595,6 +595,10 @@ int avio_open(AVIOContext **s, const char *url, int > > flags); > > int

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-24 Thread Clément Bœsch
On Wed, Jan 20, 2016 at 09:15:15PM -0300, James Almer wrote: > On 1/20/2016 7:46 PM, Clément Bœsch wrote: > > diff --git a/libavutil/thread.h b/libavutil/thread.h > > index 32ddf40..42430fb 100644 > > --- a/libavutil/thread.h > > +++ b/libavutil/t

Re: [FFmpeg-devel] [PATCH 2/2] avformat: add protocol_whitelist

2016-01-24 Thread Clément Bœsch
On Sun, Jan 24, 2016 at 01:57:05PM +0100, wm4 wrote: > On Sun, 24 Jan 2016 13:43:40 +0100 > Clément Bœsch <u...@pkh.me> wrote: > > > On Sun, Jan 24, 2016 at 12:47:59PM +0100, Nicolas George wrote: > > > Le quintidi 5 pluviôse, an CCXXIV, Michael Niedermayer a écrit

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-24 Thread Clément Bœsch
On Sun, Jan 24, 2016 at 10:09:49PM +0100, Michael Niedermayer wrote: [...] > fails to build: > make distclean ; ../configure --enable-pthreads --arch=x86_32 > --target-os=linux --extra-cflags=-m32 --extra-ldflags=-m32 > --enable-cross-compile && make -j12 > In file included from

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
On Wed, Jan 20, 2016 at 12:48:52PM -0800, Timothy Gu wrote: > On Wed, Jan 20, 2016 at 09:44:32PM +0100, Clément Bœsch wrote: > > diff --git a/libavutil/thread.h b/libavutil/thread.h > > index 32ddf40..a7b5763 100644 > > --- a/libavutil/thread.h > > +++ b/libavutil/thr

Re: [FFmpeg-devel] [PATCH] lavu: add pthread asserts if ASSERT_LEVEL>1

2016-01-20 Thread Clément Bœsch
On Tue, Dec 22, 2015 at 09:05:24PM +0100, Clément Bœsch wrote: > --- > libavutil/thread.h | 96 > ++ > 1 file changed, 96 insertions(+) > > diff --git a/libavutil/thread.h b/libavutil/thread.h > index 3d15737..0bb745e 100

[FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
--- libavcodec/frame_thread_encoder.c | 2 ++ libavcodec/pthread_slice.c| 2 ++ libavfilter/pthread.c | 2 ++ libavformat/async.c | 2 ++ libavformat/udp.c | 3 +++ libavutil/thread.h| 9 + 6 files changed, 20 insertions(+)

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
On Wed, Jan 20, 2016 at 09:50:40PM +0100, Nicolas George wrote: > Le primidi 1er pluviôse, an CCXXIV, Clement Boesch a écrit : > > --- > > libavcodec/frame_thread_encoder.c | 2 ++ > > libavcodec/pthread_slice.c| 2 ++ > > libavfilter/pthread.c | 2 ++ > > libavformat/async.c

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
On Wed, Jan 20, 2016 at 11:11:35PM +0100, Clément Bœsch wrote: [...] > +#elif defined(__linux__) locally changed to defined(__linux__) && defined(__GLIBC__) > +pthread_setname_np(pthread_self(), name); > +#endif > +} > + [...] -- Clément B. signature.asc Des

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
On Wed, Jan 20, 2016 at 07:29:02PM -0300, James Almer wrote: [...] > CC libavformat/async.o > F:/msys/ffmpeg/src/libavformat/async.c: In function 'async_buffer_task': > F:/msys/ffmpeg/src/libavformat/async.c:184:5: error: implicit declaration of > function 'ff_thread_setname'

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
On Wed, Jan 20, 2016 at 10:55:54PM +0100, wm4 wrote: [...] > > +static inline void ff_pthread_setname(const char *name) > > +{ > > +#if defined(__APPLE__) > > +pthread_setname_np(name); > > +#elif defined(__linux__) > > +pthread_setname_np(pthread_self(), name); > > Just because you're on

[FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2016-01-20 Thread Clément Bœsch
--- Changes: - < 16 char length - no spaces - ff_pthread_setname → ff_thread_setname - changed ifdefery protection bonus: http://ubitux.fr/pub/pics/ffthreads.png --- libavcodec/frame_thread_encoder.c | 2 ++ libavcodec/pthread_frame.c| 2 ++ libavcodec/pthread_slice.c| 2 ++

<    2   3   4   5   6   7   8   9   10   11   >