Re: [FFmpeg-devel] libavcodec/psd : add support for rgb/gray float

2017-03-01 Thread wm4
On Wed, 1 Mar 2017 23:42:28 +0100 Martin Vignali wrote: > Hello, > > In attach patch for decoding float data (rgb/gray only) inside psd > Like in the openexr decoder, float data is convert to uint16 > A gamma conversion can be apply before uint16 conversion (with an

Re: [FFmpeg-devel] [PATCH v3 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

2017-03-01 Thread Damien Riegel
On Wed, Mar 01, 2017 at 12:12:42AM +0100, Thomas Volkert wrote: > > On 28.02.2017 19:39, Damien Riegel wrote: > > This adds partial support for the RFC 4175 (raw video over RTP). The > > only supported formats are the YCbCr-4:2:2 8 bit because it's natively > > supported by FFmpeg with pixel

[FFmpeg-devel] Convert audio to DBFS

2017-03-01 Thread Chau Pham
Hi there I am doing audio converting and use ffmpeg. I looked into a ffmpeg sample program demuxing_decoding.c and inside decode_packet function ... size_t unpadded_linesize = frame->nb_samples * av_get_bytes_per_sample((AVSampleFormat)frame->format);

Re: [FFmpeg-devel] [PATCH v2 1/3] ffmpeg_opt: pass output framerate as a hint to the encoder

2017-03-01 Thread Michael Niedermayer
On Mon, Feb 27, 2017 at 09:50:31AM +0100, Tobias Rapp wrote: > On 26.02.2017 16:09, Michael Niedermayer wrote: > >On Mon, Feb 20, 2017 at 04:05:00PM +0100, Tobias Rapp wrote: > >>On 20.02.2017 15:09, Mark Thompson wrote: > >>>On 06/02/17 12:33, Tobias Rapp wrote: > Sets framerate field in

Re: [FFmpeg-devel] [PATCH] avcodec/vp56: Clear dimensions in case of failure in the middle of a resolution change

2017-03-01 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 04:28:23AM +0100, Michael Niedermayer wrote: > Similar code is used elsewhere in vp56 to force a more complete reinit in the > future. > Fixes null pointer dereference > Fixes: 707/clusterfuzz-testcase-4717453097566208 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/h264idct_template: Fix several runtime error: signed integer overflow

2017-03-01 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 10:07:36PM +0100, Michael Niedermayer wrote: > Fixes: 689/clusterfuzz-testcase-6029352737177600 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer > --- >

Re: [FFmpeg-devel] [PATCH]lavf/flacdec: Return maximum score if a valid streaminfo header was found

2017-03-01 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 09:38:58AM +0100, Carl Eugen Hoyos wrote: > Hi! > > I implemented this patch to fix ticket #6208 but it turned out the flac probe > function wasn't the issue. Still looks like a good idea to me. > > Please comment, Carl Eugen > flacdec.c |9 - > 1 file

[FFmpeg-devel] [PATCH 2/2] avcodec/mpeg4videodec: Improve the overflow checks in mpeg4_decode_sprite_trajectory()

2017-03-01 Thread Michael Niedermayer
Also clear the state on errors Fixes integer overflows in 701/clusterfuzz-testcase-6594719951880192 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/mpeg4videodec.c | 16

[FFmpeg-devel] [PATCH 1/2] avcodec/vp56: Require a correctly decoded frame before using vp56_conceal_mb()

2017-03-01 Thread Michael Niedermayer
Fixes timeout with 700/clusterfuzz-testcase-5660909504561152 Fixes timeout with 702/clusterfuzz-testcase-4553541576294400 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer ---

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread James Almer
On 3/1/2017 7:18 PM, Carl Eugen Hoyos wrote: > 2017-03-01 17:56 GMT+01:00 James Almer : >> On 2/28/2017 4:45 PM, Carl Eugen Hoyos wrote: >>> Hi! >>> >>> Attached patch fixes ticket #6206 here. >>> >>> Please comment, Carl Eugen >> >> This doesn't consider the CodecPrivate

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-03-01 Thread Ganapathy Raman Kasi
Thanks for review. If qmin and qmax are not set, nvenc will assume default of qmin 15 and qmax 51 which is good enough for most content. Setting qmin 5 would be similar to qmin of 15 at lower bitrates as ratecontrol may not reach below QP 15. Ganapathy -Original Message- From:

Re: [FFmpeg-devel] libavcodec/psd : add support for rgb/gray float

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 23:42 GMT+01:00 Martin Vignali : > Hello, > > In attach patch for decoding float data (rgb/gray only) inside psd > Like in the openexr decoder, float data is convert to uint16 > A gamma conversion can be apply before uint16 conversion (with an option) > > the

Re: [FFmpeg-devel] [PATCH] lavf/dashenc: update bitrates on dash_write_trailer

2017-03-01 Thread Przemysław Sobala
23 lut 2017 21:39 "Przemysław Sobala" napisał(a): On Wednesday, February 22, 2017, Przemysław Sobala < przemyslaw.sob...@gmail.com> wrote: > From: Przemysław Sobala > > Provides a way to change bandwidth (bitrate) parameter after CRF

[FFmpeg-devel] [PATCH]lavc/sheervideo: Fix Y prediction for interlaced frames with alpha

2017-03-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6210 visually. Please review, Carl Eugen From 305b1916d99465896abbd1fbf401c6e4fba062a5 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed, 1 Mar 2017 23:45:10 +0100 Subject: [PATCH] lavc/sheervideo: Fix Y prediction for interlaced frames

[FFmpeg-devel] libavcodec/psd : add support for rgb/gray float

2017-03-01 Thread Martin Vignali
Hello, In attach patch for decoding float data (rgb/gray only) inside psd Like in the openexr decoder, float data is convert to uint16 A gamma conversion can be apply before uint16 conversion (with an option) the function : static inline uint16_t psd_flt2uint(uint32_t v) is a copy/paste of

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 13:07 GMT+01:00 Michael Niedermayer : > On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote: >> 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : >> > On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote: >> >> Hi!

[FFmpeg-devel] [PATCH]lavc/bitstream_filter: Make a cast explicit

2017-03-01 Thread Carl Eugen Hoyos
Hi! Attached patch silences one of three warnings when compiling bitstream_filter.o, I suspect this cast is necessary. Please comment, Carl Eugen From 15963817149fd4249f7812b42566eaf23afac6ce Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed, 1 Mar 2017 23:35:14 +0100

[FFmpeg-devel] [PATCH]lavf/matroska: Support QDMC

2017-03-01 Thread Carl Eugen Hoyos
Hi! Attached patch allows muxing and demuxing of QDMC in mkv. Please comment, Carl Eugen From 8d383886e19dbfd362497144fc9610ecdcedc144 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed, 1 Mar 2017 23:22:40 +0100 Subject: [PATCH] lavf/matroska: Support QDMC. ---

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 17:56 GMT+01:00 James Almer : > On 2/28/2017 4:45 PM, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes ticket #6206 here. >> >> Please comment, Carl Eugen > > This doesn't consider the CodecPrivate contents for FFV1 v3 > and above. It's only valid for v1 and

Re: [FFmpeg-devel] [PATCH]lavf/cavsvideodec: Return a lower score if only one frame was found

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 19:01 GMT+01:00 Michael Niedermayer : > On Wed, Mar 01, 2017 at 09:40:08AM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes ticket #6208. >> >> Please comment, Carl Eugen > > breaks tickets/3400/akiyo_cif.avs I feared so. Not sure I can fix

Re: [FFmpeg-devel] [PATCH]lavc/internal: Constify AVPacket* in AVCodecInternal.

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 13:04 GMT+01:00 Michael Niedermayer : > On Sun, Feb 26, 2017 at 12:38:14PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes another warning when compiling libavcodec/utils.o. >> >> Please comment, Carl Eugen > > if it doesnt produce any new

[FFmpeg-devel] [PATCH] omx: Add support for specifying H.264 profile [v5]

2017-03-01 Thread Takayuki 'January June' Suwa
From: Takayuki 'January June' Suwa This adds "-profile[:v] profile_name"-style option. --- libavcodec/omx.c | 33 + 1 file changed, 33 insertions(+) diff --git a/libavcodec/omx.c b/libavcodec/omx.c index 16df50e..c0d64e2 100644

Re: [FFmpeg-devel] [PATCH]lavf/cavsvideodec: Return a lower score if only one frame was found

2017-03-01 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 09:40:08AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6208. > > Please comment, Carl Eugen breaks tickets/3400/akiyo_cif.avs [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are

Re: [FFmpeg-devel] [PATCH 1/3] avformat/oggdec: remove unused parameter of ogg_restore()

2017-03-01 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 03:31:10PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/oggdec.c | 13 + > 1 file changed, 5 insertions(+), 8 deletions(-) patchset applied [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH 2/3] avcodec/mpegaudiodec_template: Fix runtime error: signed integer overflow: 2053224902 + 2053224902 cannot be represented in type 'int'

2017-03-01 Thread Michael Niedermayer
Fixes: 696/clusterfuzz-testcase-5853632270434304 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/mpegaudiodec_template.c | 2 +- 1 file changed, 1 insertion(+), 1

[FFmpeg-devel] [PATCH 1/3] avcodec/mjpegdec: Fix runtime error: left shift of negative value -511

2017-03-01 Thread Michael Niedermayer
Fixes: 693/clusterfuzz-testcase-6109776066904064 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/mjpegdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[FFmpeg-devel] [PATCH 3/3] avcodec/interplayvideo: Fix timeout from lack of bitstream end check

2017-03-01 Thread Michael Niedermayer
Fixes: 697/clusterfuzz-testcase-6111250582863872 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/interplayvideo.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread James Almer
On 2/28/2017 4:45 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6206 here. > > Please comment, Carl Eugen This doesn't consider the CodecPrivate contents for FFV1 v3 and above. It's only valid for v1 and v2. > > >

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Jerome Martinez
Le 01/03/2017 à 15:51, wm4 a écrit : On Wed, 1 Mar 2017 15:00:27 +0100 Jerome Martinez wrote: [...] then in 12 (or 24, or 36 months), the time that new FFmpeg versions propagate everywhere (we also open tickets for other demuxers in order to get the support of such

Re: [FFmpeg-devel] [PATCH 2/2] fate: update libswresample fate

2017-03-01 Thread Muhammad Faiz
On Wed, Mar 1, 2017 at 10:15 PM, James Almer wrote: > On 3/1/2017 11:25 AM, Muhammad Faiz wrote: >> on 'swresample/resample: use uniform normalization' >> also update other fates that depend on resampling >> >> Signed-off-by: Muhammad Faiz >> --- >>

[FFmpeg-devel] [PATCH] swresample/resample: use uniform normalization

2017-03-01 Thread Muhammad Faiz
this gives better frequency response update swresample fate and other fates that depend on resampling Signed-off-by: Muhammad Faiz --- libswresample/resample.c | 14 +-- tests/fate/libswresample.mak | 276 +--

Re: [FFmpeg-devel] [PATCH 2/2] fate: update libswresample fate

2017-03-01 Thread James Almer
On 3/1/2017 11:25 AM, Muhammad Faiz wrote: > on 'swresample/resample: use uniform normalization' > also update other fates that depend on resampling > > Signed-off-by: Muhammad Faiz > --- > tests/fate/libswresample.mak | 276 > +-- >

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread wm4
On Wed, 1 Mar 2017 15:00:27 +0100 Jerome Martinez wrote: > Le 01/03/2017 à 09:43, Carl Eugen Hoyos a écrit : > > 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : > >> On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote: > >>> Hi! >

Re: [FFmpeg-devel] [PATCH 1/2] swresample/resample: use uniform normalization

2017-03-01 Thread Muhammad Faiz
On Wed, Mar 1, 2017 at 9:24 PM, Muhammad Faiz wrote: > this gives better frequency response > > Signed-off-by: Muhammad Faiz > --- > libswresample/resample.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > Test case, from 44100 to

[FFmpeg-devel] [PATCH 2/2] fate: update libswresample fate

2017-03-01 Thread Muhammad Faiz
on 'swresample/resample: use uniform normalization' also update other fates that depend on resampling Signed-off-by: Muhammad Faiz --- tests/fate/libswresample.mak | 276 +-- tests/ref/acodec/roqaudio| 2 +- tests/ref/acodec/s302m

[FFmpeg-devel] [PATCH 1/2] swresample/resample: use uniform normalization

2017-03-01 Thread Muhammad Faiz
this gives better frequency response Signed-off-by: Muhammad Faiz --- libswresample/resample.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libswresample/resample.c b/libswresample/resample.c index 8e2f769..b0d14d1 100644 ---

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Jerome Martinez
Le 01/03/2017 à 09:43, Carl Eugen Hoyos a écrit : 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote: Hi! Attached patch fixes ticket #6206 here. Please comment, Carl Eugen previous ffmpeg versions dont

Re: [FFmpeg-devel] Hardware Encode H264

2017-03-01 Thread Ricardo Constantino
Wrong mailing list. You're looking for http://ffmpeg.org/mailman/listinfo/ffmpeg-user. Also, your e-mail client sending HTML e-mails as plain text for some reason which makes them quite unreadable. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread wm4
On Wed, 1 Mar 2017 14:26:34 +0100 Michael Niedermayer wrote: > On Wed, Mar 01, 2017 at 01:36:26PM +0100, wm4 wrote: > > On Wed, 1 Mar 2017 13:07:12 +0100 > > Michael Niedermayer wrote: > > > > > On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 01:36:26PM +0100, wm4 wrote: > On Wed, 1 Mar 2017 13:07:12 +0100 > Michael Niedermayer wrote: > > > On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote: > > > 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : >

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread wm4
On Wed, 1 Mar 2017 13:06:29 +0100 Carl Eugen Hoyos wrote: > 2017-03-01 12:36 GMT+01:00 wm4 : > > On Wed, 1 Mar 2017 12:20:10 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-02-25 15:59 GMT+01:00 wm4 : > >>

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread wm4
On Wed, 1 Mar 2017 13:07:12 +0100 Michael Niedermayer wrote: > On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote: > > 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : > > > On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Reto Kromer
Michael Niedermayer wrote: >> FFV1 didn't have any mapping in matroska before this, so >> what exactly is there to support? > >mkv supports all avi identifers, i assume that is what was >used before > >> Also, format support is technically a feature, so back- >> porting seems ill-advised. > >I

[FFmpeg-devel] [PATCH] configure: convert MSVC ident to utf-8 if possible

2017-03-01 Thread Timo Rothenpieler
--- configure | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 0199fec..398d530 100755 --- a/configure +++ b/configure @@ -4095,7 +4095,11 @@ probe_cc(){ disable stripping elif $_cc -nologo- 2>&1 | grep -q Microsoft; then

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 01:18:09PM +0100, Hendrik Leppkes wrote: > On Wed, Mar 1, 2017 at 1:07 PM, Michael Niedermayer > wrote: > > On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote: > >> 2017-03-01 4:40 GMT+01:00 Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/vp8: Check for bitsteam end in decode_mb_row_no_filter()

2017-03-01 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 10:19:05PM -0500, Ronald S. Bultje wrote: > Hi Michael, > > On Tue, Feb 28, 2017 at 11:29 AM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > > > On Tue, Feb 28, 2017 at 07:44:12AM -0500, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Mon, Feb 27, 2017 at 10:28

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 13:18 GMT+01:00 Hendrik Leppkes : > FFV1 didn't have any mapping in matroska before this I believe it is safe to say that many people used ffv1 in mkv and they would be surprised if their new files suddenly didn't play in vlc. Carl Eugen

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Hendrik Leppkes
On Wed, Mar 1, 2017 at 1:07 PM, Michael Niedermayer wrote: > On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote: >> 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : >> > On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote:

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/h264_direct: Fix runtime error: left shift of negative value -14

2017-03-01 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 04:28:29AM +0100, Michael Niedermayer wrote: > Fixes: 682/clusterfuzz-testcase-4799120021651456 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer > --- >

Re: [FFmpeg-devel] [PATCH] avcodec/avpacket: Check metadata key in av_packet_unpack_dictionary()

2017-03-01 Thread Michael Niedermayer
On Mon, Feb 27, 2017 at 04:12:09AM +0100, Michael Niedermayer wrote: > Fixes timeout > Fixes: 501/clusterfuzz-testcase-5672752870588416 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote: > 2017-03-01 4:40 GMT+01:00 Michael Niedermayer : > > On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch fixes ticket #6206 here. > >> > >> Please comment,

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 12:36 GMT+01:00 wm4 : > On Wed, 1 Mar 2017 12:20:10 +0100 > Carl Eugen Hoyos wrote: > >> 2017-02-25 15:59 GMT+01:00 wm4 : >> > I'm documenting existing practice. >> >> > -@subheading Always wait long enough before

Re: [FFmpeg-devel] [PATCH]lavc/internal: Constify AVPacket* in AVCodecInternal.

2017-03-01 Thread Michael Niedermayer
On Sun, Feb 26, 2017 at 12:38:14PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes another warning when compiling libavcodec/utils.o. > > Please comment, Carl Eugen if it doesnt produce any new warnings elsewere then its probably ok [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] Revert "avutil/frame: Disallow zero sized frame side data"

2017-03-01 Thread Michael Niedermayer
Found a case where we use size==0, the other related commits remain needed, and should be sufficient to fix the original issue This reverts commit 7e4f32f4e4b93c95dcc872cb844c5548e69f352e. --- libavutil/frame.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavutil/frame.c

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread wm4
On Wed, 1 Mar 2017 12:20:10 +0100 Carl Eugen Hoyos wrote: > 2017-02-25 15:59 GMT+01:00 wm4 : > > I'm documenting existing practice. > > > -@subheading Always wait long enough before pushing changes > > +@subheading Always wait long enough before

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread Carl Eugen Hoyos
2017-02-25 21:23 GMT+01:00 Marton Balint : > Of course, I could just subscribe to csvlog as well I was under the impression that if you want commit rights, you are required to be subscribed to -cvslog. Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread Carl Eugen Hoyos
2017-02-25 19:35 GMT+01:00 Marton Balint : > Instead of this, I'd prefer all patches on the ML. This would mean we changed from a policy that we successful applied for over a decade to a policy of which we - to the best of my knowledge - know that it does not work well. >

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread Carl Eugen Hoyos
2017-02-25 16:03 GMT+01:00 Ivan Kalvachev : > I do prefer all the code to go though the maillist. > Even trivial changes. This is not how we have done it for more than a decade and I am against changing our (extremely successful) practice. Carl Eugen

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread Carl Eugen Hoyos
2017-02-25 15:59 GMT+01:00 wm4 : > I'm documenting existing practice. > -@subheading Always wait long enough before pushing changes > +@subheading Always wait long enough before pushing changes to code actively > maintained by others I suspect this is missing an exception

Re: [FFmpeg-devel] [PATCH] doc: clarify development rules

2017-03-01 Thread wm4
On Sat, 25 Feb 2017 21:23:27 +0100 (CET) Marton Balint wrote: > On Sat, 25 Feb 2017, Rostislav Pehlivanov wrote: > > > On 25 February 2017 at 18:35, Marton Balint wrote: > > > >> > >> On Sat, 25 Feb 2017, wm4 wrote: > >> > >> I'm documenting existing practice.

Re: [FFmpeg-devel] [PATCH] (for discussion): cuvid: allow to crop and resize in decoder

2017-03-01 Thread Timo Rothenpieler
>> We recently just had all sorts of discussions what decoders should and >> should not do, I don't think scaling in a decoder is a good thing to >> start doing here. > > scaling in some decoders is mandated by some specs > some standards support reduced resolution which can switch from frame >

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 4:40 GMT+01:00 Michael Niedermayer : > On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes ticket #6206 here. >> >> Please comment, Carl Eugen > > previous ffmpeg versions dont play these files What do you suggest?

Re: [FFmpeg-devel] [PATCH]Makefile: Clean compat subdirectory

2017-03-01 Thread Carl Eugen Hoyos
2017-03-01 2:14 GMT+01:00 Michael Niedermayer : > On Tue, Feb 21, 2017 at 02:38:22PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes ticket #5546 here. >> >> Please comment, Carl Eugen > >> Makefile |2 ++ >> 1 file changed, 2 insertions(+) >>

[FFmpeg-devel] [PATCH]lavf/cavsvideodec: Return a lower score if only one frame was found

2017-03-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6208. Please comment, Carl Eugen From 901443bd61d33f62a4fd904aad5f270d0a018a8a Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed, 1 Mar 2017 09:35:16 +0100 Subject: [PATCH 2/2] lavf/cavsvideodec: Return a lower score if only one frame

[FFmpeg-devel] [PATCH]lavf/flacdec: Return maximum score if a valid streaminfo header was found

2017-03-01 Thread Carl Eugen Hoyos
Hi! I implemented this patch to fix ticket #6208 but it turned out the flac probe function wasn't the issue. Still looks like a good idea to me. Please comment, Carl Eugen From acc7558a3585d99776523ed670747597c4de99fb Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed,