Re: [FFmpeg-devel] [PATCH] lavf/mov: fix stream extradata_size allocation

2016-07-14 Thread Matthieu Bouron
On Thu, Jul 14, 2016 at 12:48:28AM +0200, Michael Niedermayer wrote: > On Wed, Jul 13, 2016 at 10:51:23PM +0200, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > Fixes CID 1363963. > > --- > > libavformat/mov.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM > > thx

[FFmpeg-devel] [PATCH] Add bitrate and framerate option for openH264

2016-07-14 Thread Steven Liu
Signed-off-by: ErLiu <88486...@qq.com> --- libavcodec/libopenh264enc.c | 38 ++ 1 file changed, 38 insertions(+) diff --git a/libavcodec/libopenh264enc.c b/libavcodec/libopenh264enc.c index 24bc228..f7f11f9 100644 --- a/libavcodec/libopenh264enc.c +++ b/libavco

Re: [FFmpeg-devel] [PATCH] Add bitrate and framerate option for openH264

2016-07-14 Thread Carl Eugen Hoyos
Steven Liu gmail.com> writes: > +{ "bitrate", "Set bitrate", OFFSET(bitrate), > AV_OPT_TYPE_INT, { 0 }, 0, 70, VE }, > +{ "framerate", "Set framerate", OFFSET(framerate), > AV_OPT_TYPE_INT, { 0 }, 0, 30, VE }, Both options already exist in FFmpeg, it may be a good idea to map them

Re: [FFmpeg-devel] [PATCH] Add bitrate and framerate option for openH264

2016-07-14 Thread Steven Liu
2016-07-14 18:22 GMT+08:00 Carl Eugen Hoyos : > Steven Liu gmail.com> writes: > > > +{ "bitrate", "Set bitrate", OFFSET(bitrate), > > AV_OPT_TYPE_INT, { 0 }, 0, 70, VE }, > > +{ "framerate", "Set framerate", OFFSET(framerate), > > AV_OPT_TYPE_INT, { 0 }, 0, 30, VE }, > > Both options

Re: [FFmpeg-devel] [PATCH] Add bitrate and framerate option for openH264

2016-07-14 Thread Carl Eugen Hoyos
Steven Liu gmail.com> writes: > This patch can modified the bitrate and frame at > the encoder runtime, for example: Simply compare iBitrate and s->bitrate for every frame and reconfigure if necessary: No new option needed. Carl Eugen ___ ffmpeg-d

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mux: Apply auto bsfs in av_write_frame() too

2016-07-14 Thread Michael Niedermayer
On Thu, Jul 14, 2016 at 06:22:40AM +0800, Steven Liu wrote: > 2016-07-14 1:08 GMT+08:00 Michael Niedermayer : > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mux.c |4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/libavformat/mux.c b/libavformat/mux.c > > ind

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mux: Factor do_packet_auto_bsf() out

2016-07-14 Thread Michael Niedermayer
On Thu, Jul 14, 2016 at 06:23:13AM +0800, Steven Liu wrote: > 2016-07-14 1:08 GMT+08:00 Michael Niedermayer : > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mux.c | 102 > > ++--- > > 1 file changed, 57 insertions(+), 45 deletion

Re: [FFmpeg-devel] [PATCH] lavf: add textdata virtual demuxer and demuxer

2016-07-14 Thread Stefano Sabatini
On date Tuesday 2016-06-28 11:42:42 +0200, Nicolas George encoded: > Le decadi 10 messidor, an CCXXIV, Stefano Sabatini a écrit : > > +The fftextdata bytestream consists of a sequence of packets. Each > > +packet starts with a timestamps expressed in a format recognized by > > +FFmpeg (see > > +@re

Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-07-14 Thread Carl Eugen Hoyos
Andrey Utkin fastmail.com> writes: > It is impossible to pass "aspect" parameter to encoder from ffmpeg > CLI because option from lavc/options_table.h is eclipsed by option > with same name in ffmpeg_opt.c, which has different meaning (DAR, > not SAR). Wasn't the old behaviour intended becaus

[FFmpeg-devel] [PATCHv6] lavf: add libopenmpt demuxer

2016-07-14 Thread Josh de Kock
Fixes ticket #5623 TODO: bump lavf minor --- Changelog| 1 + configure| 4 ++ libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/libopenmpt.c | 176 +++ 5 files changed, 183 insertions(+)

Re: [FFmpeg-devel] [PATCH]lavf/vplayerdec: Stricter probing

2016-07-14 Thread Clément Bœsch
On Tue, Jul 12, 2016 at 12:01:25PM +, Carl Eugen Hoyos wrote: > > potentially simpler alternative: if you replace the original formats with > > "%*3d:%*2d:%*2d.%*2d%c" and "%*3d:%*2d:%*2d%c", does it work? > > Please commit this, it looks simpler to me. > You're in a better position to test.

[FFmpeg-devel] [PATCH] fate: Add HDCD filter tests for false positive and error detection

2016-07-14 Thread Burt P
Signed-off-by: Burt P --- tests/fate-run.sh | 3 ++- tests/fate/filter-audio.mak | 12 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/tests/fate-run.sh b/tests/fate-run.sh index c898695..5841b0c 100755 --- a/tests/fate-run.sh +++ b/tests/fate-run.sh @@ -328

Re: [FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread Vignesh Venkatasubramanian
On Tue, Jul 12, 2016 at 12:22 PM, James Almer wrote: > On 7/12/2016 3:48 PM, Vignesh Venkatasubramanian wrote: >> VPx (VP8/VP9) alpha encoding has been part of FFmpeg. Now, add the >> ability to decode such files with alpha channel. >> >> Signed-off-by: Vignesh Venkatasubramanian >> --- >> libav

[FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread Vignesh Venkatasubramanian
VPx (VP8/VP9) alpha encoding has been part of FFmpeg. Now, add the ability to decode such files with alpha channel. Signed-off-by: Vignesh Venkatasubramanian --- libavcodec/libvpxdec.c | 102 +++-- 1 file changed, 82 insertions(+), 20 deletions(-) dif

Re: [FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread Vignesh Venkatasubramanian
On Tue, Jul 12, 2016 at 1:47 PM, James Almer wrote: > On 7/12/2016 3:48 PM, Vignesh Venkatasubramanian wrote: >> VPx (VP8/VP9) alpha encoding has been part of FFmpeg. Now, add the >> ability to decode such files with alpha channel. >> >> Signed-off-by: Vignesh Venkatasubramanian >> --- >> libavc

Re: [FFmpeg-devel] [PATCH] remove the check_bitstream from segment

2016-07-14 Thread Michael Niedermayer
On Thu, Jul 14, 2016 at 06:37:02AM +0800, Steven Liu wrote: > because Michael Niedermayer has re-factor BSF logic into a shareable > function and > make both av_write_frame and av_interleaved_write_frame use it. > > Signed-off-by: LiuQi > --- > libavformat/segment.c | 21 - >

Re: [FFmpeg-devel] [FFmpeg-cvslog] mov: Support prores with multiple stsd

2016-07-14 Thread compn
On Wed, 13 Jul 2016 17:06:43 +0200 (CEST) g...@videolan.org (Vittorio Giovara) wrote: > ffmpeg | branch: master | Vittorio Giovara > | Thu Jun 9 18:55:19 2016 -0400| > [846a3e78a535f05ee61bb23a160f3378f041f751] | committer: Vittorio > Giovara > > mov: Support prores with multiple stsd > > This

Re: [FFmpeg-devel] [FFmpeg-cvslog] mov: Support prores with multiple stsd

2016-07-14 Thread compn
On Thu, 14 Jul 2016 16:26:09 -0400 compn wrote: > shouldnt these other 4 tags also be added? also i think adding a comment in isom.c about adding any new prores tags to this hack in mov.c would be useful. i am assuming prores will have more tags in the future. -compn __

Re: [FFmpeg-devel] [PATCH]lavf/img2: Remove ffv1-img

2016-07-14 Thread Michael Niedermayer
On Wed, Jul 13, 2016 at 11:09:48AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch removes a line from img2.c, afaict the ffv1 format > cannot be used without a container, at least it doesn't work with > current (and very old) FFmpeg. > > Carl Eugen > img2.c |1 - > 1 file changed

Re: [FFmpeg-devel] [PATCH 2/5] af_hdcd: fewer false positives by ignoring code_counterC in HDCD detection

2016-07-14 Thread Michael Niedermayer
On Tue, Jul 12, 2016 at 12:54:07PM -0500, Burt P wrote: > Signed-off-by: Burt P > --- > libavfilter/af_hdcd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB There will always be a questio

Re: [FFmpeg-devel] [PATCH 3/5] af_hdcd: only hdcd_update_info() when something changes

2016-07-14 Thread Michael Niedermayer
On Tue, Jul 12, 2016 at 12:54:08PM -0500, Burt P wrote: > Only call hdcd_update_info() when the control code changes > instead of every frame, so the counters are more meaningful. > > Signed-off-by: Burt P > --- > libavfilter/af_hdcd.c | 34 +- > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: detect and report encoding errors and oddities

2016-07-14 Thread Michael Niedermayer
On Tue, Jul 12, 2016 at 12:54:10PM -0500, Burt P wrote: > Count and report when a code is signaled but fails to match a known pattern. > For example try Līve - Secret Samadhi. > > Signed-off-by: Burt P > --- > libavfilter/af_hdcd.c | 73 > --- > 1

Re: [FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread James Zern
On Tue, Jul 12, 2016 at 11:48 AM, Vignesh Venkatasubramanian wrote: > On Mon, Jul 11, 2016 at 5:55 PM, James Zern > wrote: >> On Thu, Jul 7, 2016 at 11:43 AM, Vignesh Venkatasubramanian >> wrote: >> >>> [...] >>> -av_image_copy(picture->data, picture->linesize, (const uint8_t >>> **)img

Re: [FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread James Zern
On Thu, Jul 14, 2016 at 12:15 PM, Vignesh Venkatasubramanian wrote: > +ret = vpx_init( > +avctx, > +(avctx->codec_id == AV_CODEC_ID_VP8) ? > +&vpx_codec_vp8_dx_algo : &vpx_codec_vp9_dx_algo, > +1);

Re: [FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread James Zern
On Tue, Jul 12, 2016 at 1:47 PM, James Almer wrote: > What's the first libvpx version that supports vp8a and vp9a? > Configure currently checks for 0.9.1 as the oldest supported > version for vp8 decoding, 0.9.7 for vp8 encoding, and 1.3.0 for > both vp9 components. We then use a bunch of ifdeffer

Re: [FFmpeg-devel] [PATCH] libavcodec/libvpx: Add VPx alpha decode support

2016-07-14 Thread James Almer
On 7/15/2016 12:31 AM, James Zern wrote: > On Tue, Jul 12, 2016 at 1:47 PM, James Almer wrote: >> What's the first libvpx version that supports vp8a and vp9a? >> Configure currently checks for 0.9.1 as the oldest supported >> version for vp8 decoding, 0.9.7 for vp8 encoding, and 1.3.0 for >> both

Re: [FFmpeg-devel] [PATCH] Restore compatibility with powerpc-apple-darwin9-gcc-4.2.1

2016-07-14 Thread Pavel Koshevoy
I've re-sent this patch via git send-email to make it easier to view. It has been tested with gcc 4.2.1 on osx 10.5 by me, and with clang and gcc 4.9 by Jing Yu on ubuntu -- http://ffmpeg.org/pipermail/ffmpeg-devel/2016-July/196547.html Is there any reason it shouldn't be applied? Pavel. On

[FFmpeg-devel] [PATCH] Restore compatibility with powerpc-apple-darwin9-gcc-4.2.1

2016-07-14 Thread pkoshevoy
From: Pavel Koshevoy ... and attempt to preserve compatibility with clang that was introduced in 311a953c76081fca99b872629d248f9d69ebc0c3 (untested) --- libavcodec/ppc/asm.S | 8 +++- libavcodec/ppc/fft_altivec.S | 26 +++--- 2 files changed, 22 insertions(+), 12

Re: [FFmpeg-devel] [PATCH] libavcodec/qsvdec.c: Restoring decoding functionality after unsuccessful merge from libav.

2016-07-14 Thread Chao Liu
Ivan Uskov nablet.com> writes: > > Hello All, > > After commit d30cf57a7b2097b565db02ecfffbdc9c16423d0e qsv-based decoding > aborts with crash, there are many incorrect places appeared. The attached > patch fixes the issues but keeps new method of the 'sync' variable allocation > in