Re: [FFmpeg-devel] [PATCH 2/2] avformat/apng: set max_fps to no limit by default

2017-03-22 Thread Benoit Fouet
Hi, On 21/03/2017 14:03, James Almer wrote: > On 3/21/2017 9:52 AM, Michael Niedermayer wrote: >> On Mon, Mar 20, 2017 at 11:03:23PM -0300, James Almer wrote: >>> Should fix ticket #6252 >>> >>> Signed-off-by: James Almer >>> --- >>> libavformat/apngdec.c | 2 +- >>> 1 file changed, 1 insertion

Re: [FFmpeg-devel] [PATCH 1/3] avutil/spherical: add av_spherical_projection_name()

2017-03-30 Thread Benoit Fouet
Hi, On 29/03/2017 04:55, James Almer wrote: > Signed-off-by: James Almer > --- > doc/APIchanges| 3 +++ > libavutil/spherical.c | 15 +++ > libavutil/spherical.h | 9 + > 3 files changed, 27 insertions(+) > [...] > diff --git a/libavutil/spherical.c b/libavutil/s

Re: [FFmpeg-devel] [PATCH] [UPDATE] HLS, add option to skip down streams,

2017-04-21 Thread Benoit Fouet
Hi, On 20/04/2017 18:18, Amine kabab wrote: > From 5079f9b7114589626a4c9fff0fbb8f6e0d2f4fd9 Mon Sep 17 00:00:00 2001 > From: amine kabab > Date: Thu, 20 Apr 2017 15:59:42 + > Subject: [PATCH] HLS skip down streams > > --- > libavformat/hls.c | 14 +- > 1 file changed, 13 inserti

Re: [FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: separate macro arguments with commas

2017-05-02 Thread Benoit Fouet
Hi, On 28/04/2017 21:58, Matthieu Bouron wrote: > Untested: fixes ticket #6324. > --- > libavcodec/aarch64/simple_idct_neon.S | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/aarch64/simple_idct_neon.S > b/libavcodec/aarch64/simple_idct_neon.S > i

Re: [FFmpeg-devel] [PATCH]Two patches from github

2017-05-16 Thread Benoit Fouet
Hi, On 15/05/2017 12:40, Carl Eugen Hoyos wrote: > > 0002-avutil-Use-_SC_NPROCESSORS_CONF.patch > > > From 42766f345dbf398716c6fd9072f072f5fa91c940 Mon Sep 17 00:00:00 2001 > From: Steve Kondik > Date: Tue, 16 Dec 2014 01:37:57 -0800 > Subject: [PATCH 2/2] avutil: Use _SC_NPROCESSORS_CONF > > *

Re: [FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference

2016-10-17 Thread Benoit Fouet
Hi, On 17/10/2016 06:49, James Almer wrote: On 10/16/2016 9:30 PM, James Almer wrote: On 10/16/2016 5:11 PM, Andreas Cadhalpun wrote: The problem was introduced in commit 1273bc6. Signed-off-by: Andreas Cadhalpun --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 dele

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: zero initialize codec_name in mov_parse_stsd_video()

2016-10-17 Thread Benoit Fouet
Hi, On 17/10/2016 02:34, James Almer wrote: > Fixes valgrind warning about "Conditional jump or move depends on > uninitialised value(s)" > > Signed-off-by: James Almer > --- > libavformat/mov.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mov.c b/libavf

Re: [FFmpeg-devel] [PATCHv3] mov: Evaluate the movie display matrix

2016-10-18 Thread Benoit Fouet
Hi, On 14/10/2016 00:50, Vittorio Giovara wrote: > diff --git a/libavformat/mov.c b/libavformat/mov.c > index a15c8d1..e8da77f 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c [...] > @@ -3798,16 +3804,33 @@ static int mov_read_meta(MOVContext *c, AVIOContext > *pb, MOVAtom atom) >

Re: [FFmpeg-devel] [PATCH] lavc/ffjni: replace ff_jni_{attach, detach} with ff_jni_get_env

2016-07-25 Thread Benoit Fouet
Hi, On 24/07/2016 23:05, Matthieu Bouron wrote: From: Matthieu Bouron If a JNI environment is not already attached to the thread where the MediaCodec calls are made the current implementation will attach / detach an environment for each MediaCodec call wasting some CPU time. ff_jni_get_env rep

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: fix SODB escaping

2016-09-07 Thread Benoit Fouet
Hi, On 06/09/2016 16:53, Matthieu Bouron wrote: From: Matthieu Bouron Fixes escaping of consecutive 0x00, 0x00, 0x0{0-3} sequences. --- libavcodec/mediacodecdec_h264.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/mediacodecdec_h264.c b/libavcodec/med

Re: [FFmpeg-devel] [PATCH] avcodec/h264: Use ptrdiff_t for (bi)width functions

2016-09-21 Thread Benoit Fouet
Hi, On 21/09/2016 03:34, Michael Niedermayer wrote: Might fix some mysterious asm related issues like Ticket5579 Signed-off-by: Michael Niedermayer --- libavcodec/h264dsp.h | 5 +++-- libavcodec/h264dsp_template.c | 4 ++-- libavcodec/mips/h264dsp_mips.h | 26 +---

Re: [FFmpeg-devel] [PATCH] avfilter: add nlmeans filter

2016-09-21 Thread Benoit Fouet
Hi, On 20/09/2016 21:52, Clément Bœsch wrote: Fixes Ticket #4910 --- I actually tried to implement the better defaults suggestion from ipol (see @todo) but it wasn't convincing; probably because of different scales, so I need to investigate. Also, integral is still inplace in the filter for no

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: retain error codes in matroska_resync() and matroska_read_packet()

2016-09-23 Thread Benoit Fouet
Hi, On 22/09/2016 23:03, Sophia Wang wrote: Signed-off-by: Sophia Wang --- libavformat/matroskadec.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 77b8a5d..936690d 100644 --- a/libavformat/matros

Re: [FFmpeg-devel] [PATCH] lavc/psymodel: check for av_malloc failure

2016-03-10 Thread Benoit Fouet
Hi, Le 04/03/2016 04:06, Ganesh Ajjanagadde a écrit : No idea why in commit 01ecb7172b684f1c4b3e748f95c5a9a494ca36ec the checks were removed; this can lead to NULL pointer dereferences. This effectively reverts that portion of the commit. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/psym

Re: [FFmpeg-devel] [PATCH 02/10] swscale/arm/yuv2rgb: fix comments and factorize lsl in load_args_yuv422p

2016-03-30 Thread Benoit Fouet
Hi, Le 25/03/2016 23:45, Matthieu Bouron a écrit : From: Matthieu Bouron --- libswscale/arm/yuv2rgb_neon.S | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/libswscale/arm/yuv2rgb_neon.S b/libswscale/arm/yuv2rgb_neon.S index f40327b..aac0773 100644 --- a/libswscale

Re: [FFmpeg-devel] [PATCH 06/10] swscale/arm/yuv2rgb: only process one line at a time for the yuv420p and nv{12, 21} formats

2016-03-30 Thread Benoit Fouet
Hi, Le 26/03/2016 13:05, Matthieu Bouron a écrit : On Sat, Mar 26, 2016 at 2:09 AM, Michael Niedermayer >wrote: >On Fri, Mar 25, 2016 at 11:46:01PM +0100, Matthieu Bouron wrote: > >From: Matthieu Bouron > > > >--- > > libswscale/arm/yuv2rgb_neon.S | 89 >--

Re: [FFmpeg-devel] [PATCH v2 6/9] swscale/arm/yuv2rgb: macro-ify

2016-03-31 Thread Benoit Fouet
On 28/03/2016 21:19, Matthieu Bouron wrote: --- libswscale/arm/yuv2rgb_neon.S | 137 ++ 1 file changed, 60 insertions(+), 77 deletions(-) diff --git a/libswscale/arm/yuv2rgb_neon.S b/libswscale/arm/yuv2rgb_neon.S index ef7b0a6..e1b68c1 100644 --- a/li

Re: [FFmpeg-devel] [PATCH v2 6/9] swscale/arm/yuv2rgb: macro-ify

2016-03-31 Thread Benoit Fouet
Hi, (sorry for the first mail, fuzzy fingers...) On 28/03/2016 21:19, Matthieu Bouron wrote: --- libswscale/arm/yuv2rgb_neon.S | 137 ++ 1 file changed, 60 insertions(+), 77 deletions(-) diff --git a/libswscale/arm/yuv2rgb_neon.S b/libswscale/arm/yuv2

Re: [FFmpeg-devel] [PATCH v2 8/9] swscale/arm/yuv2rgb: save a few instructions by processing the luma line interleaved

2016-03-31 Thread Benoit Fouet
Hi, On 28/03/2016 21:19, Matthieu Bouron wrote: --- libswscale/arm/yuv2rgb_neon.S | 88 +-- 1 file changed, 34 insertions(+), 54 deletions(-) diff --git a/libswscale/arm/yuv2rgb_neon.S b/libswscale/arm/yuv2rgb_neon.S index 124d7d3..6b911c8 100644 --- a

Re: [FFmpeg-devel] [PATCH v2 6/9] swscale/arm/yuv2rgb: macro-ify

2016-03-31 Thread Benoit Fouet
On 28/03/2016 21:19, Matthieu Bouron wrote: --- libswscale/arm/yuv2rgb_neon.S | 137 ++ 1 file changed, 60 insertions(+), 77 deletions(-) diff --git a/libswscale/arm/yuv2rgb_neon.S b/libswscale/arm/yuv2rgb_neon.S index ef7b0a6..e1b68c1 100644 --- a/li

Re: [FFmpeg-devel] [PATCH] swscale/arm: add yuv2planeX_8_neon

2016-04-11 Thread Benoit Fouet
Hi, (again, thanks to both of you for documenting all this assembly /NEON code) On 09/04/2016 10:22, Matthieu Bouron wrote: From: Matthieu Bouron --- Hello, The following patch add yuv2planeX_8_neon function for the arm platform. It is currently restricted to 8-bit per component sources un

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-12 Thread Benoit Fouet
Hi, I mostly have nits remarks. On 11/05/2016 18:39, Matthieu Bouron wrote: From: Matthieu Bouron [...] diff --git a/libswresample/arm/resample.S b/libswresample/arm/resample.S new file mode 100644 index 000..13462e3 --- /dev/null +++ b/libswresample/arm/resample.S @@ -0,0 +1,77 @@ [

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-12 Thread Benoit Fouet
Hi, On 12/05/2016 15:22, Matthieu Bouron wrote: On Thu, May 12, 2016 at 10:01 AM, Benoit Fouet wrote: Hi, I mostly have nits remarks. On 11/05/2016 18:39, Matthieu Bouron wrote: From: Matthieu Bouron [...] diff --git a/libswresample/arm/resample.S b/libswresample/arm/resample.S new

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

2016-05-19 Thread Benoit Fouet
Hi, Le 18 mai 2016 20:40:08 GMT+02:00, Michael Niedermayer a écrit : >Signed-off-by: Michael Niedermayer >--- > doc/developer.texi | 29 + > 1 file changed, 29 insertions(+) > >diff --git a/doc/developer.texi b/doc/developer.texi >index 6db93ce..4d3a7ae 100644 >---

Re: [FFmpeg-devel] [PATCH] lavf/mpegtsenc: move putstr8 definition up

2016-05-21 Thread Benoit Fouet
Hi, Le 19/05/2016 18:45, Stefano Sabatini a écrit : This allows to use the function in a future commit. --- libavformat/mpegtsenc.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index

[FFmpeg-devel] [PATCH] hls muxer doc: clarify segment splitting option

2016-06-08 Thread Benoit Fouet
Hi, find attached a patch to $subj This would have been useful at least to me :-) -- Ben From aac7f87102337b7b1a59f88fc69766a20dfa0790 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Wed, 8 Jun 2016 11:42:51 +0200 Subject: [PATCH] hls muxer doc: clarify segment splitting option --- doc

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Benoit Fouet
Hi, On 12/06/2016 22:58, Paul B Mahol wrote: Hi, As requested in the IRC meeting I hereby request for the voting committee to begin voting on whatever to ban Carl Eugen Hoyos from mailing list, trac and IRC for 4 months, starting after the voting has finished. Voting will last 7 days from now

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

2016-06-13 Thread Benoit Fouet
Hi, On 13/06/2016 10:21, Clément Bœsch wrote: On Mon, Jun 13, 2016 at 05:50:18AM +0200, Matt Oliver wrote: ffmpeg | branch: master | Matt Oliver | Mon Jun 6 17:04:39 2016 +1000| [37787f261639c53998487400e874741c17e85fc6] | committer: Matt Oliver lavf/os_support.h: Fix for unicode filename

Re: [FFmpeg-devel] [PATCH] hls muxer doc: clarify segment splitting option

2016-06-13 Thread Benoit Fouet
Hi, Le 08/06/2016 11:46, Benoit Fouet a écrit : Hi, find attached a patch to $subj This would have been useful at least to me :-) Anyone against this patch? If not, can someone please apply it? Thanks, -- Ben ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: allow devices not implementing VIDIOC_G_PARM

2016-06-16 Thread Benoit Fouet
Hi, On 15/06/2016 17:21, Niklas Söderlund wrote: Not all v4l2 devices implement the VIDIOC_G_PARM ioctl. This patch allow ffmpeg to open such device and treat it the same as devices that do implement the ioctl but returns that it do not implement the V4L2_CAP_TIMEPERFRAME capability. Signed-of

[FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, Unless I totally missed something, the FIXME in H264ParamSets structure should be fixed by attached patch. -- Ben From 28ae10498f81070539bdb8f40236326743350101 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Tue, 21 Jun 2016 14:17:13 +0200 Subject: [PATCH] h264: make H264ParamSets sps

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally missed something, the FIXME in H264ParamSets structure should be fixed by attached patch. -- Ben From

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-23 Thread Benoit Fouet
Hi, On 21/06/2016 16:42, Benoit Fouet wrote: Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-24 Thread Benoit Fouet
Hi, On 23/06/2016 22:37, Michael Niedermayer wrote: On Thu, Jun 23, 2016 at 03:28:10PM +0200, Benoit Fouet wrote: Hi, On 21/06/2016 16:42, Benoit Fouet wrote: Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: drop people who do not appear in git history > 2013

2016-06-24 Thread Benoit Fouet
thieu Bouron + aiffdec.c Matthieu Bouron + aiffenc.c Matthieu Bouron apngdec.c Benoit Fouet - ass* Aurelien Jacobs astdec.c Paul B

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-27 Thread Benoit Fouet
Hi, On 25/06/2016 14:15, Clément Bœsch wrote: On Fri, Jun 24, 2016 at 09:20:35AM +0200, Benoit Fouet wrote: [...] Any objection to this patch now? iam ok with the patch, maybe give others a bit of time to reply before applying though Yeah, I'm in no hurry, I just saw this FIXME i

[FFmpeg-devel] [PATCH] H264ParamSets: make sps const

2016-06-27 Thread Benoit Fouet
Hi, First patch change decode_scaling_matrices function, so that it does not affect the SPS structure it gets, and marks SPS and PPS const in its arguments. Second patch straightens the check for macroblock sizes in ff_h264_decode_seq_parameter_set and removes the unneeded check in H.264 sl

[FFmpeg-devel] [PATCH 1/3] h264_ps: change decode_scaling_matrices so that it takes

2016-06-27 Thread Benoit Fouet
From c2606da98ecd04762305734f4f45ca8eaf266459 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Mon, 27 Jun 2016 12:00:39 +0200 Subject: [PATCH 1/3] h264_ps: change decode_scaling_matrices so that it takes const {s,p}ps In order to be able to make SPS const in H264ParamSets, modify

[FFmpeg-devel] [PATCH 3/3] h264: make H264ParamSets sps const

2016-06-27 Thread Benoit Fouet
From 735362df589eb5f8b05063c56862ff18589475ad Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Tue, 21 Jun 2016 14:17:13 +0200 Subject: [PATCH 3/3] h264: make H264ParamSets sps const --- libavcodec/h264.h| 3 +-- libavcodec/h264_parser.c | 2 +- libavcodec/h264_ps.c | 4

[FFmpeg-devel] [PATCH 2/3] h264: straighten dimensions check

2016-06-27 Thread Benoit Fouet
From 91b000bf2e0b01695803c5ef98cfb06590f5f409 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Mon, 27 Jun 2016 13:31:21 +0200 Subject: [PATCH 2/3] h264: straighten dimensions check ff_h264_decode_seq_parameter_set The MBS only flag was not taken into account when checking macroblock

Re: [FFmpeg-devel] [PATCH 1/3] h264_ps: change decode_scaling_matrices so that it takes

2016-06-28 Thread Benoit Fouet
Hi, On 27/06/2016 18:31, Michael Niedermayer wrote: On Mon, Jun 27, 2016 at 02:38:50PM +0200, Benoit Fouet wrote: h264_ps.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) ea8cc471972e1dbaa4f4f03cd7a5fe92a3b848e9 0001-h264_ps-change-decode_scaling_matrices-so-that-it

Re: [FFmpeg-devel] [PATCH] H264ParamSets: make sps const

2016-06-28 Thread Benoit Fouet
Hi, On 27/06/2016 14:37, Benoit Fouet wrote: Hi, First patch change decode_scaling_matrices function, so that it does not affect the SPS structure it gets, and marks SPS and PPS const in its arguments. Second patch straightens the check for macroblock sizes in

[FFmpeg-devel] [PATCH] git: ignore test binaries

2016-06-29 Thread Benoit Fouet
From e514644033781cb431641ae088482f5a8aa2de42 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Wed, 29 Jun 2016 17:53:50 +0200 Subject: [PATCH] git: ignore test binaries --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 524fb73..670d1d2 100644

Re: [FFmpeg-devel] [PATCH] git: ignore test binaries

2016-06-29 Thread Benoit Fouet
Hi, On 29/06/2016 17:57, Clément Bœsch wrote: On Wed, Jun 29, 2016 at 05:54:37PM +0200, Benoit Fouet wrote: From e514644033781cb431641ae088482f5a8aa2de42 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Wed, 29 Jun 2016 17:53:50 +0200 Subject: [PATCH] git: ignore test binaries

Re: [FFmpeg-devel] [PATCH] H264ParamSets: make sps const

2016-06-30 Thread Benoit Fouet
Hi, On 27/06/2016 14:37, Benoit Fouet wrote: Hi, First patch change decode_scaling_matrices function, so that it does not affect the SPS structure it gets, and marks SPS and PPS const in its arguments. Second patch straightens the check for macroblock sizes in

Re: [FFmpeg-devel] [PATCH] mediacodecdec_h264: properly convert extradata to annex-b

2016-07-05 Thread Benoit Fouet
Hi, On 04/07/2016 10:12, Matthieu Bouron wrote: From: Matthieu Bouron H264ParamSets has its SPS/PPS stored raw (SODB) and needs to be converted to NAL units before sending them to MediaCodec. This patch adds the missing convertion of the SPS/PPS from SOBP to RBSP which makes the resulting NAL

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: add hwaccel support

2016-07-08 Thread Benoit Fouet
Hi, On 07/07/2016 17:43, Matthieu Bouron wrote: [...] 0001-lavc-add-mediacodec-hwaccel-support.patch From 9bb86990f0f7a26d25878a771f5977ae83d14769 Mon Sep 17 00:00:00 2001 From: Matthieu Bouron Date: Fri, 11 Mar 2016 17:21:04 +0100 Subject: [PATCH] lavc: add mediacodec hwaccel support ---

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: add hwaccel support

2016-07-08 Thread Benoit Fouet
On 08/07/2016 15:40, Matthieu Bouron wrote: On Fri, Jul 08, 2016 at 01:21:48PM +0200, Benoit Fouet wrote: Hi, On 07/07/2016 17:43, Matthieu Bouron wrote: [...] 0001-lavc-add-mediacodec-hwaccel-support.patch From 9bb86990f0f7a26d25878a771f5977ae83d14769 Mon Sep 17 00:00:00 2001 From

Re: [FFmpeg-devel] [PATCH] SSE2 version of vf_idet's filter_line()

2014-09-03 Thread Benoit Fouet
Hi, - Mail original - > Hi, > > > updated patch, sorry for the broken format in the previous one. Hope > it's > ok now. > This is just missing the new header file vf_idet.h -- Ben ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http:/

Re: [FFmpeg-devel] [PATCH 1/1] libavdevice/v4l2: fix descriptors leak on error paths

2014-09-04 Thread Benoit Fouet
Hi, - Mail original - > From: Dmitry Volyntsev > > Signed-off-by: Dmitry Volytnsev > --- > libavdevice/v4l2.c | 37 - > 1 file changed, 20 insertions(+), 17 deletions(-) > > diff --git a/libavdevice/v4l2.c b/libavdevice/v4l2.c > index 9f9f944..7b9

Re: [FFmpeg-devel] [PATCH] avdevice/x11grab: fix error handling in pixel formats

2014-09-04 Thread Benoit Fouet
Hi, - Mail original - > Signed-off-by: Michael Niedermayer > --- > libavdevice/x11grab.c |5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavdevice/x11grab.c b/libavdevice/x11grab.c > index 120f754..e0d1dfa 100644 > --- a/libavdevice/x11grab.c > +++ b/li

[FFmpeg-devel] [PATCH] tiff: fix {2,4}bpp grayscale palettes.

2014-09-05 Thread Benoit Fouet
Create a default grayscale palette for 2 or 4 bpp grayscale tiff, if there is no palette defined. Fixes ticket #3915 --- libavcodec/tiff.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index 1e2d235..0352639 100644 --- a/

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mpegvideo: Factor ff_mpv_decode_init() out

2014-09-05 Thread Benoit Fouet
Hi, Le 05/09/2014 17:41, Michael Niedermayer a écrit : > Signed-off-by: Michael Niedermayer > --- > libavcodec/h261dec.c |7 ++- > libavcodec/h263dec.c |6 ++ > libavcodec/mpeg12dec.c |3 +-- > libavcodec/mpegvideo.c | 12 > libavcodec/mpegvideo.h |1 +

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mpegvideo: set codec tags in ff_mpv_decode_init()

2014-09-05 Thread Benoit Fouet
Hi, Le 05/09/2014 17:41, Michael Niedermayer a écrit : > Fixes Ticket3912 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/mpegvideo.c |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > LGTM as well -- Ben ___ ffmpeg-deve

Re: [FFmpeg-devel] [PATCH] ffplay: Dont autoexeit by default on single images

2014-09-07 Thread Benoit Fouet
Hi, Le 07/09/2014 02:16, Michael Niedermayer a écrit : > Signed-off-by: Michael Niedermayer > --- > ffplay.c |7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > I'm fine with it in principle (though the commit message should be fixed), but this -1 value should be somewhat docume

Re: [FFmpeg-devel] [PATCH 1/1] configure: fix check_cmd in check_host_cpp

2014-09-11 Thread Benoit Fouet
Hi, - Mail original - > Use correct cpp and c flags variables for the host libc. > > Signed-off-by: Jörg Krause > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index e66519d..7a711a2 100755 > --- a/configure > +++ b/con

[FFmpeg-devel] [PATCH] img2dec: make probing score zero for known file extensions.

2014-09-11 Thread Benoit Fouet
When a known extension has its own probing function, make img_read_probe return 0, so that the probing functions are actually called instead of relying only on the file extensions. Fixes issue #3901 --- libavformat/img2dec.c | 24 +++- 1 file changed, 23 insertions(+), 1 delet

Re: [FFmpeg-devel] [PATCH 3/4] avformat/format: Run image2 probe again when file content data is available

2014-09-12 Thread Benoit Fouet
Hi, - Mail original - > Signed-off-by: Michael Niedermayer > --- > libavformat/format.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/format.c b/libavformat/format.c > index 828ab52..006cc87 100644 > --- a/libavformat/format.c > +++ b/libavforma

Re: [FFmpeg-devel] [PATCH 2/4] avformat/img2dec: reduce bmppipe probe score

2014-09-12 Thread Benoit Fouet
Hi, - Mail original - > bmp pipe needs the bmp parser which is not bug free and should thus > not be favored > over the bmp image2 demuxer > > Signed-off-by: Michael Niedermayer > --- > libavformat/img2dec.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/li

Re: [FFmpeg-devel] [PATCH] img2dec: make probing score zero for known file extensions.

2014-09-12 Thread Benoit Fouet
Hi, - Mail original - > On Thu, Sep 11, 2014 at 03:48:21PM +0200, Benoit Fouet wrote: > > When a known extension has its own probing function, make > > img_read_probe > > return 0, so that the probing functions are actually called instead > > of > > re

Re: [FFmpeg-devel] [PATCH 4/4] avformat/img2dec: Fail probing when no data is yet available and the filename contains no number/glob patterns either.

2014-09-12 Thread Benoit Fouet
Hi, - Mail original - > Fixes Ticket3901 > > Signed-off-by: Michael Niedermayer > --- > libavformat/img2dec.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c > index 9f48c5d..a21429f 100644 > --- a/libavformat/img2dec.c > +++

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add AVFMT_FILE to allow demuxers to be used with and without files

2014-09-12 Thread Benoit Fouet
Hi, - Mail original - > On Fri, 12 Sep 2014 12:29:36 +0200 > Michael Niedermayer wrote: > > > Suggested-by: Benoit Fouet > > Signed-off-by: Michael Niedermayer > > --- > > doc/APIchanges |3 +++ > > libavformat/avformat.h |8

Re: [FFmpeg-devel] [PATCH 3/4] avformat/format: Run image2 probe again when file content data is available

2014-09-12 Thread Benoit Fouet
Hi, - Mail original - > On Fri, Sep 12, 2014 at 10:13:20AM +0200, Benoit Fouet wrote: > > Hi, > > > > - Mail original - > > > Signed-off-by: Michael Niedermayer > > > --- > > > libavformat/format.c |2 +- > > > 1 file

[FFmpeg-devel] [PATCH] avformat/img2: remove useless 'pix' duplicated entry.

2014-09-12 Thread Benoit Fouet
The second entry will never be selected, so let's save some bytes in the library. --- libavformat/img2.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/img2.c b/libavformat/img2.c index 8002054..d6f1244 100644 --- a/libavformat/img2.c +++ b/libavformat/img2.c @@ -57,7 +57,6 @@ stat

[FFmpeg-devel] [PATCH] avformat/avidec: assumes that extradata begins with palette when bpp <= 8.

2014-09-16 Thread Benoit Fouet
Some other information is stored in extradata, such that the BottomUp field. But it seems that extradata always starts with the palette. Fixes ticket #1304 --- libavformat/avidec.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavformat/avidec.c b/libavformat/avidec.c

[FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Benoit Fouet
Considering the palette is located at the end of extradata may be flawed when the extradata contains the palette followed by the BottomUp field. When the BottomUp field is present, exclude it from the palette. Fixes ticket #1304 --- libavformat/avidec.c | 4 1 file changed, 4 insertions(+) d

Re: [FFmpeg-devel] [PATCH] avformat/img2: remove useless 'pix' duplicated entry.

2014-09-22 Thread Benoit Fouet
ping? - Mail original - > The second entry will never be selected, so let's save some bytes in > the > library. > --- > libavformat/img2.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavformat/img2.c b/libavformat/img2.c > index 8002054..d6f1244 100644 > --- a/libavformat/i

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Benoit Fouet
Hi, - Mail original - > On Mon, Sep 22, 2014 at 09:57:37AM +0200, Benoit Fouet wrote: > > Considering the palette is located at the end of extradata may be > > flawed > > when the extradata contains the palette followed by the BottomUp > > field. > > W

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Benoit Fouet
Hi, - Mail original - > On Mon, Sep 22, 2014 at 11:28:31AM +0200, Benoit Fouet wrote: > > Hi, > > > > - Mail original - > > > On Mon, Sep 22, 2014 at 09:57:37AM +0200, Benoit Fouet wrote: > > > > Considering the palette is loca

[FFmpeg-devel] [PATCH] avformat/img2dec: fix glob pattern detection.

2014-09-22 Thread Benoit Fouet
The is_glob() function was not working with unescaped glob patterns, which is the way only glob_sequence (which is deprecated) works. Fixes ticket #3948 --- libavformat/img2dec.c | 14 +- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/libavformat/img2dec.c b/libavformat/

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-23 Thread Benoit Fouet
Hi, - Mail original - [...] > > > > avienc.c |7 +++ > > > > 1 file changed, 7 insertions(+) > > > > 8ae96276e07eb00474ab2115f9ff9c3d0f690723 avienc.diff > > > > diff --git a/libavformat/avienc.c b/libavformat/avienc.c > > > > index c9d8b7f..2855293 100644 > > > > --- a/libavfo

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-23 Thread Benoit Fouet
he BITMAPINFOHEADER is wrong. > > > So the image is still flipped. > > > > Well then you need a local variable/array or value in the avi muxer > > context. values in AVCodecContext shouldnt really be randomly > > be overwritten by the muxer > > > >

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-23 Thread Benoit Fouet
Hi, - Mail original - > On Tue, Sep 23, 2014 at 10:19:17AM +0200, Benoit Fouet wrote: > > Here is a patch to illustrate this... > > this looks better but: > > > [...] > > -ff_put_bmp_header(pb, enc, ff_codec_bmp_tags, 0, > > 0);

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-23 Thread Benoit Fouet
Hi, - Mail original - > Hi, > > - Mail original - > > On Tue, Sep 23, 2014 at 10:19:17AM +0200, Benoit Fouet wrote: > > > > Here is a patch to illustrate this... > > > > this looks better but: > > > > > &g

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-23 Thread Benoit Fouet
Hi, - Mail original - > On Tue, Sep 23, 2014 at 03:11:58PM +0200, Michael Niedermayer wrote: > > On Tue, Sep 23, 2014 at 01:33:30PM +0200, Benoit Fouet wrote: > > > Hi, > > > > > > - Mail original - > > > > Hi, > > > >

Re: [FFmpeg-devel] [PATCH] ffplay: use av_codec_get_pkt_timebase()

2014-09-24 Thread Benoit Fouet
Hi, - Mail original - > Signed-off-by: Michael Niedermayer > --- > ffplay.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ffplay.c b/ffplay.c > index 6eb5aef..7728452 100644 > --- a/ffplay.c > +++ b/ffplay.c > @@ -598,7 +598,7 @@ static int decoder_decode_f

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

2014-09-26 Thread Benoit Fouet
Hi, - Mail original - > Michael Niedermayer gmx.at> writes: > > > > > 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

Re: [FFmpeg-devel] xtory codec support

2014-09-26 Thread Benoit Fouet
Hi, - Mail original - > On Wed, Sep 24, 2014 at 10:21:01AM +0400, I.C. Wiener wrote: > > > > Hello, i have just upload the sample of dxtory 2.0 output file with > > ffmpeg stder dump(in corresponding txt file) to > > ftp://upload.ffmpeg.org/incoming . Please, have a look. Recent > > versi

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

2014-09-26 Thread Benoit Fouet
Hi, Le 26/09/2014 18:38, Michael Niedermayer a écrit : > On Fri, Sep 26, 2014 at 09:27:01AM +0200, Benoit Fouet wrote: >> Hi, >> >> - Mail original - >>> Michael Niedermayer gmx.at> writes: >>> >>>> >>>> On Fri, Aug 01,

Re: [FFmpeg-devel] [PATCH]Do not add palette size to the bitmapheader size

2014-09-29 Thread Benoit Fouet
Hi, - Mail original - > On Monday 29 September 2014 01:22:38 am Carl Eugen Hoyos wrote: > > Attached patch is the first part of the fix for ticket #1304. > > New, tested patch attached. > > Please comment, Carl Eugen > LGTM, -- Ben ___ ffmp

Re: [FFmpeg-devel] [PATCH]Set the rawvideo image size to 0 in the bitmapinfoheader

2014-09-29 Thread Benoit Fouet
Hi, - Mail original - > On Monday 29 September 2014 01:30:44 am Carl Eugen Hoyos wrote: > > Attached patch fixes ticket #1304: biSizeImage may be 0 for images > > with > > codec tag 0, if the size is set too small, WMP refuses to play the > > file. > > Patch with changes to fate attached.

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/utils: Fix off by 1 error causing unneeded allocation in ff_fast_malloc()

2014-09-29 Thread Benoit Fouet
Hi, - Mail original - > Signed-off-by: Michael Niedermayer > --- > libavcodec/utils.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index b27f918..9eb2b5b 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/utils.c

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/put_bits: Add rebase_put_bits()

2014-09-29 Thread Benoit Fouet
Hi, - Mail original - > Signed-off-by: Michael Niedermayer > --- > libavcodec/put_bits.h | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/libavcodec/put_bits.h b/libavcodec/put_bits.h > index 8081fb9..f16d04a 100644 > --- a/libavcodec/put_bits.h > +++ b/liba

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

2014-09-29 Thread Benoit Fouet
Hi, - Mail original - > Hi, > > Le 26/09/2014 18:38, Michael Niedermayer a écrit : > > On Fri, Sep 26, 2014 at 09:27:01AM +0200, Benoit Fouet wrote: > >> Hi, > >> > >> - Mail original - > >>> Michael Niedermayer gmx.at>

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

2014-09-30 Thread Benoit Fouet
Hi, - Mail original - > On Mon, Sep 29, 2014 at 03:31:36PM +0200, Benoit Fouet wrote: > [...] > does this ensure that the sps is before the pps ? > if not that might be the reason for the warnings > It does not. I can update this so that when the pps is seen and

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

2014-09-30 Thread Benoit Fouet
Hi, - Mail original - > On Tue, Sep 30, 2014 at 09:45:47AM +0200, Benoit Fouet wrote: > > Hi, > > > > - Mail original - > > > On Mon, Sep 29, 2014 at 03:31:36PM +0200, Benoit Fouet wrote: > > > > > > > [...] > > > &g

Re: [FFmpeg-devel] [PATCH] avformat/img2dec: fix glob pattern detection.

2014-09-30 Thread Benoit Fouet
Hi, - Mail original - > The is_glob() function was not working with unescaped glob patterns, > which is the way only glob_sequence (which is deprecated) works. > Fixes ticket #3948 > --- > libavformat/img2dec.c | 14 +- > 1 file changed, 1 insertion(+), 13 deletions(-) > > di

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

2014-09-30 Thread Benoit Fouet
Hi, - Mail original - > On Tue, Sep 30, 2014 at 11:15:06AM +0200, Benoit Fouet wrote: > > Hi, > > > > - Mail original - > > > On Tue, Sep 30, 2014 at 09:45:47AM +0200, Benoit Fouet wrote: > > > > Hi, > > > > > > > &

[FFmpeg-devel] [PATCH 1/2] avcodec/h264_mp4toannexb_bsf: use the given padding in h264_extradata_to_annexb().

2014-09-30 Thread Benoit Fouet
--- libavcodec/h264_mp4toannexb_bsf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h264_mp4toannexb_bsf.c b/libavcodec/h264_mp4toannexb_bsf.c index 3ec4170..be42304 100644 --- a/libavcodec/h264_mp4toannexb_bsf.c +++ b/libavcodec/h264_mp4toannexb_bsf.c @@ -117,7 +

[FFmpeg-devel] [PATCH 2/2] avcodec/h264_mp4toannexb_bsf: reset the new IDR flag when SPS/PPS is seen.

2014-09-30 Thread Benoit Fouet
--- libavcodec/h264_mp4toannexb_bsf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/h264_mp4toannexb_bsf.c b/libavcodec/h264_mp4toannexb_bsf.c index be42304..ae96ee9 100644 --- a/libavcodec/h264_mp4toannexb_bsf.c +++ b/libavcodec/h264_mp4toannexb_bsf.c @@ -186

[FFmpeg-devel] [PATCH] avformat/mxfenc: add jpeg2000 support

2014-09-30 Thread Benoit Fouet
448810ec5b39e7b95d7a43a76ed7994e7fca3b27 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Tue, 30 Sep 2014 14:16:52 +0200 Subject: [PATCH] avformat/mxfenc: add jpeg2000 support. Fixes ticket #1542 --- libavformat/mxfenc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/mxfenc.c b

[FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-02 Thread Benoit Fouet
Fixes ticket #3074 --- libavformat/isom.c | 1 + libavformat/movenc.c | 44 +++- 2 files changed, 44 insertions(+), 1 deletion(-) diff --git a/libavformat/isom.c b/libavformat/isom.c index d768c32..1509021 100644 --- a/libavformat/isom.c +++ b/libavforma

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: add jpeg2000 support

2014-10-02 Thread Benoit Fouet
Hi, - Mail original - > On Tue, 2014-09-30 at 15:37 +0200, Benoit Fouet wrote: > > Hi, > > > > this patch adds support for j2k muxing in MXF. > > tested with: > > $ ffmpeg -t 5 -f lavfi -i testsrc -y -c:v libopenjpeg -y out.mxf > > > > Played

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-03 Thread Benoit Fouet
Hi, - Mail original - > 2014-10-02 19:39 GMT+09:00 Benoit Fouet : > [...] > > diff --git a/libavformat/movenc.c b/libavformat/movenc.c > > index bfee866..62b2d4b 100644 > > --- a/libavformat/movenc.c > > +++ b/libavformat/movenc.c > > @@

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-03 Thread Benoit Fouet
Hi, - Mail original - > Benoit Fouet free.fr> writes: > > > Do you have a sample you can share? > > Please look at the 7.1 samples in > http://samples.ffmpeg.org//A-codecs/AC3/eac3/ > I was about to send an email stating I found t

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-03 Thread Benoit Fouet
Hi, - Mail original - > Benoit Fouet free.fr> writes: > > > > Please look at the 7.1 samples in > > > http://samples.ffmpeg.org//A-codecs/AC3/eac3/ > > > > I was about to send an email stating I found those ones. > > Since all samples we h

[FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-07 Thread Benoit Fouet
Support only one independent substream right now, and only syncframes containing 6 blocks. Fixes part of ticket #3074 --- Right now, this produces the same output as the previous patch for supported streams, and rejects the unsupported ones. Support for syncframes concatenation will come afterwar

[FFmpeg-devel] [PATCH] avformat/mov: E-AC-3 streams need parsing too.

2014-10-09 Thread Benoit Fouet
syncframes in E-AC-3 can be combined to provide 6 audio blocks per sample, thus requiring parsing for proper decoding. --- libavformat/mov.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index fdd0671..40c2fc4 100644 --- a/libavformat/mov.c +++ b/libavfo

[FFmpeg-devel] [PATCH] avformat/movenc: add support for syncframes concatenation for E-AC-3.

2014-10-09 Thread Benoit Fouet
E-AC-3 samples should contain 6 audio blocks, so concatenate syncframes in order to achieve this. --- libavformat/movenc.c | 41 +++-- 1 file changed, 35 insertions(+), 6 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 18c5955..4f4f1

  1   2   3   >