Re: [FFmpeg-devel] [PATCH] vaapi_h265: general_level_idc should times 3.

2017-11-28 Thread Li, Zhong
> libavcodec/vaapi_encode_h265.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vaapi_encode_h265.c > b/libavcodec/vaapi_encode_h265.c index 3ae92a7..32b8bc6 100644 > --- a/libavcodec/vaapi_encode_h265.c > +++ b/libavcodec/vaapi_encode_h265.c > @@ -219,7 +219,

Re: [FFmpeg-devel] Accurately describing ffmpeg-cvslog list [was: Re: [PATCH] Refactor Developer Docs, update dev list section (v2)]

2017-11-28 Thread Jim DeLaHunt
On 2017-11-27 15:03, Carl Eugen Hoyos wrote: 2017-11-26 22:57 GMT+01:00 Jim DeLaHunt : On 2017-11-26 03:42, Carl Eugen Hoyos wrote: 2017-11-26 9:31 GMT+01:00 Jim DeLaHunt : [...] + @subheading Subscribe to the ffmpeg-cvslog mailing list. -It is important to do this as the diffs of all com

Re: [FFmpeg-devel] Accurately describing ffmpeg-cvslog list [was: Re: [PATCH] Refactor Developer Docs, update dev list section (v2)]

2017-11-28 Thread Clément Bœsch
On Mon, Nov 27, 2017 at 11:19:01PM +, Rostislav Pehlivanov wrote: > On 27 November 2017 at 23:05, Ronald S. Bultje wrote: > > > Hi, > > > > On Mon, Nov 27, 2017 at 6:03 PM, Carl Eugen Hoyos > > wrote: > > > > > 2017-11-26 22:57 GMT+01:00 Jim DeLaHunt : > > > > On 2017-11-26 03:42, Carl Eugen

[FFmpeg-devel] [PATCH] avdevice/gdigrab: Fix screen size and mouse position calculations on hi-DPI screens

2017-11-28 Thread Harald Gaechter
Signed-off-by: Harald Gaechter --- libavdevice/gdigrab.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavdevice/gdigrab.c b/libavdevice/gdigrab.c index 87f5012034..ff2ef3b162 100644 --- a/libavdevice/gdigrab.c +++ b/libavdevice/gdigrab.c @@ -235,7 +235,9 @@ gdi

Re: [FFmpeg-devel] AMD external header

2017-11-28 Thread Timo Rothenpieler
Your use-case looks like an argument for moving the external headers into some separate repository. Not that I personally care much about bundling or not bundling. To me the more important question seems to be whether to auto-enable the encoders/decoders that depend on the external headers and

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mxfenc: write reel_name if metadata key is present

2017-11-28 Thread Tomas Härdin
On 2017-11-28 05:35, Mark Reid wrote: On Mon, Nov 27, 2017 at 2:40 AM, Tomas Härdin wrote: On Sun, 2017-11-26 at 21:42 -0800, Mark Reid wrote: @@ -1396,13 +1410,17 @@ static int mxf_write_package(AVFormatContext *s, MXFPackage *package) } // write multiple descriptor reference -

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fixed initial setting for end_pts

2017-11-28 Thread Steven Liu
2017-11-27 13:16 GMT+08:00 Steven Liu : > 2017-11-27 13:03 GMT+08:00 Karthick J : >> This patch fixes Bug #6868 >> Sometimes end_pts is getting initialized to audio stream's >> first pts, while the duration is calculated based on video stream's pts. >> In this patch the end_pts is initialized with

Re: [FFmpeg-devel] AMD external header

2017-11-28 Thread Nicolas George
Pavel Koshevoy (2017-11-27): > That is unnecessarily un-diplomatic and will likely offend the > contributors from nvidia and amd. Well, I find offensive that they benefit from my work yet make extra efforts to make sure I cannot benefit from theirs. Maybe I should start putting my contributions un

Re: [FFmpeg-devel] [PATCH] Fix for ticket 6658 (Dash demuxer segfault)

2017-11-28 Thread Carl Eugen Hoyos
2017-11-28 1:46 GMT+01:00 Colin NG : > +char tmp_str[MAX_URL_SIZE]; > +char tmp_str_2[MAX_URL_SIZE]; Assuming this is not speed-critical code, please also allocate these two with av_malloc(), FFmpeg can be compiled for systems with tiny stack. Carl Eugen _

Re: [FFmpeg-devel] [PATCH] Add android_capture indev

2017-11-28 Thread Carl Eugen Hoyos
2017-11-28 8:50 GMT+01:00 Felix Matouschek : > One last note if you try to build it yourself: > Your NDK needs to have the patch in commit be9b8bc of this > issue https://github.com/android-ndk/ndk/issues/559 applied. Sorry if I misunderstand: The patch breaks default compilation for Android? Ca

Re: [FFmpeg-devel] [PATCH] lavc/vp8: Fix HWAccel VP8 decoder can't support resolution change.

2017-11-28 Thread Carl Eugen Hoyos
2017-11-28 6:20 GMT+01:00 Jun Zhao : Could be split. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Add android_capture indev

2017-11-28 Thread Felix Matouschek
Am 28.11.2017 14:01, schrieb Carl Eugen Hoyos: The patch breaks default compilation for Android? Actually... you are right, it does. It's a bit unfortunate as the header file in the NDK is broken. We can't work around that in ffmpeg, the NDK needs to be fixed. As soon as you include NdkCamera

Re: [FFmpeg-devel] [PATCH] Add android_capture indev

2017-11-28 Thread Felix Matouschek
Sorry... tested it again, it does not break the default compilation. The library check in the configure script just fails and the indev is not included when using the broken NDK. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/

Re: [FFmpeg-devel] [PATCH 1/2] libavdevice/decklink: Add support for EIA-708 output over SDI

2017-11-28 Thread Devin Heitmueller
Hello Marton, Thanks for taking the time to review. Most of the comments you’ve raised will be fixed and I’ll resubmit an updated patch. Comments on other issues inline below. >> >>/* Options */ >>int list_devices; >> @@ -88,6 +93,7 @@ struct decklink_ctx { >>DecklinkPtsSource aud

Re: [FFmpeg-devel] lavd/alsa: Double maximum alsa buffer size.

2017-11-28 Thread Nicolas George
Carl Eugen Hoyos (2017-11-27): > The patch that introduced the regression was not discussed and > it used a guessed limit afair. Two wrong don't make a right. You should not have pushed without discussion. > The patch that fixed ticket #373 simply doubled the value to fix > a particular use-case.

Re: [FFmpeg-devel] [PATCH 2/2] decklink: Add support for output of Active Format Description (AFD)

2017-11-28 Thread Devin Heitmueller
Hi Marton, Comments inline. >> +data = av_packet_get_side_data(pkt, AV_PKT_DATA_AFD, &size); >> +if (data) { >> +struct klvanc_packet_afd_s *pkt; >> +uint16_t *afd; >> +uint16_t len; >> + >> +ret = klvanc_create_AFD(&pkt); >> +if (ret != 0) >> +

Re: [FFmpeg-devel] AMD external header

2017-11-28 Thread Mironov, Mikhail
> > Pavel Koshevoy (2017-11-27): > > That is unnecessarily un-diplomatic and will likely offend the > > contributors from nvidia and amd. > > Well, I find offensive that they benefit from my work yet make extra efforts > to make sure I cannot benefit from theirs. Maybe I should start putting my >

[FFmpeg-devel] [PATCH 2/2] vf_zscale: Add more supported input properties

2017-11-28 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- This version should be more complete. Regarding configure changes, this library is not packaged by any distribution that I could find, so users will just need to use zimg HEAD or any stable snapshot (2.6.2). Vittorio libavfilter/vf_zscale.c | 51 ++

[FFmpeg-devel] [PATCH 1/2] vf_zscale: Relax color properties maximum bounds

2017-11-28 Thread Vittorio Giovara
This simplifies adding new values, which are already validated elsewhere. Signed-off-by: Vittorio Giovara --- libavfilter/vf_zscale.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavfilter/vf_zscale.c b/libavfilter/vf_zscale.c index 09fd842fe5.

Re: [FFmpeg-devel] [PATCH]ffmpeg_opt: Silence a const warning

2017-11-28 Thread Michael Niedermayer
On Tue, Nov 28, 2017 at 12:07:33AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes a warning: > fftools/ffmpeg_opt.c: In function ‘add_input_streams’: > fftools/ffmpeg_opt.c:804:29: warning: assignment discards ‘const’ > qualifier from pointer target type > > Please comment, Carl Eu

Re: [FFmpeg-devel] [PATCH] mov: Support mdcv and clli boxes for mastering display an color light level

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 03:20:10PM -0500, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > libavformat/mov.c | 71 > +++ > 1 file changed, 71 insertions(+) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index 79023ef

Re: [FFmpeg-devel] [PATCH] avcodec: set correct return value in ff_mpeg_ref_picture

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 08:41:10PM +0800, Pan Bian wrote: > In function ff_mpeg_ref_picture(), it returns 0 on the error path that > the return value of av_buffer_ref() is NULL. 0 indicates success, which > seems to deviate from the fact. Set ret to AVERROR(ENOMEM) to propagate > the error status t

Re: [FFmpeg-devel] [PATCH 2/2] vf_zscale: Add more supported input properties

2017-11-28 Thread Paul B Mahol
On 11/28/17, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > This version should be more complete. > Regarding configure changes, this library is not packaged by any > distribution > that I could find, so users will just need to use zimg HEAD or any stable > snapshot (2.6.2). >

Re: [FFmpeg-devel] [PATCH 2/2] vf_zscale: Add more supported input properties

2017-11-28 Thread James Almer
On 11/28/2017 3:57 PM, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > This version should be more complete. > Regarding configure changes, this library is not packaged by any distribution > that I could find, so users will just need to use zimg HEAD or any stable > snapshot (2.6

[FFmpeg-devel] [PATCH 2/2] avcodec/mlpenc: fix undefined shift

2017-11-28 Thread Paul B Mahol
Decreases artifacts. Signed-off-by: Paul B Mahol --- libavcodec/mlpenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mlpenc.c b/libavcodec/mlpenc.c index c588f5b904..eceb0ddbb2 100644 --- a/libavcodec/mlpenc.c +++ b/libavcodec/mlpenc.c @@ -466,7 +466,7 @@ stati

Re: [FFmpeg-devel] avutil/x86util : add macro for 128 bits constant load

2017-11-28 Thread Henrik Gramner
On Mon, Nov 27, 2017 at 11:37 PM, James Almer wrote: > On 11/27/2017 7:33 PM, James Darnley wrote: >> If the condition was made "mmsize > 16" would this work correctly for >> zmm registers? (Assume I finally push my AVX-512 patches). > > No, there's no EVEX variant of vbroadcasti128. For that you

[FFmpeg-devel] [PATCH 1/2] avcodec/mlpenc: encode last frame too

2017-11-28 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/mlpenc.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/libavcodec/mlpenc.c b/libavcodec/mlpenc.c index 7536d3b2f5..c588f5b904 100644 --- a/libavcodec/mlpenc.c +++ b/libavcodec/mlpenc.c @@ -2225,29 +2225,27 @

Re: [FFmpeg-devel] [PATCH 1/2] libavdevice/decklink: Add support for EIA-708 output over SDI

2017-11-28 Thread Marton Balint
On Tue, 28 Nov 2017, Devin Heitmueller wrote: Hello Marton, Thanks for taking the time to review. Most of the comments you’ve raised will be fixed and I’ll resubmit an updated patch. Comments on other issues inline below. /* Options */ int list_devices; @@ -88,6 +93,7 @@ struct dec

Re: [FFmpeg-devel] [PATCH 1/4] avformat/mxfenc: pass MXFPackage around instead of type

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 11:00:51AM +0100, Tomas Härdin wrote: > On Sun, 2017-11-26 at 21:42 -0800, Mark Reid wrote: > > --- > > libavformat/mxfenc.c | 99 +- > > -- > > 1 file changed, 55 insertions(+), 44 deletions(-) > > > > diff --git a/libavform

Re: [FFmpeg-devel] [PATCH 1/2] libavdevice/decklink: Add support for EIA-708 output over SDI

2017-11-28 Thread Devin Heitmueller
Hello Marton, > SDK says: > > When capturing ancillary data with a 4K DeckLink device, the ancillary > data will always be in the 10-bit YUV pixel format. > > This also applies to 8 bit YUV captures according to my experience. > Thanks for refreshing my memory. I remember reading that text, b

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/avfilter: simplify filter registration

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 04:30:19AM +, Rostislav Pehlivanov wrote: > Atomics were entirely pointless and did nothing but slow and complicate > the process down. This could be improved further still but the main > objective of this commit is to simplify. > > Signed-off-by: Rostislav Pehlivanov

Re: [FFmpeg-devel] [PATCH]lavf/mov: Do not blindly allocate stts entries

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 05:24:14AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch avoids allocations >1GB for (short and) invalid mov > files with only reasonable speed impact. > > Please review, Carl Eugen > mov.c | 16 +--- > 1 file changed, 13 insertions(+), 3 deletion

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/avfilter: simplify filter registration

2017-11-28 Thread Paul B Mahol
On 11/28/17, Michael Niedermayer wrote: > On Mon, Nov 27, 2017 at 04:30:19AM +, Rostislav Pehlivanov wrote: >> Atomics were entirely pointless and did nothing but slow and complicate >> the process down. This could be improved further still but the main >> objective of this commit is to simpli

Re: [FFmpeg-devel] [FFmpeg-devel 2/2] avformat/mov: return correct value in mov_read_cmov

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 11:13:05AM +0800, Pan Bian wrote: > On some failure paths, the error code is not correctly set. > > Signed-off-by: Pan Bian > --- > libavformat/mov.c | 1 + > 1 file changed, 1 insertion(+) will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BAD

Re: [FFmpeg-devel] [FFmpeg-devel 1/2] avformat/mov: set correct error code in mov_read_custom

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 11:12:56AM +0800, Pan Bian wrote: > In function mov_read_custom(), it returns 0 on the path that av_malloc() > returns a NULL pointer. 0 indicates success. An error code should be > assigned to ret. > > Signed-off-by: Pan Bian > --- > libavformat/mov.c | 4 +++- > 1 file

Re: [FFmpeg-devel] [PATCH] avfilter/drawbox: rename variable for maximum thickness

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 10:10:08AM +0530, Gyan Doshi wrote: > > On 11/23/2017 7:12 PM, Gyan Doshi wrote: > > > >On 11/20/2017 3:59 PM, Gyan Doshi wrote: > >>At present, the value name 'max' for maximum thickness in > >>drawbox (and drawgrid) filter leads to a parse error if the > >>thickness expre

Re: [FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 06:43:32PM -0800, Steven Robertson wrote: > It's... less wrong. > > The VC-3 DNxHR update added a 'clv' field to an existing byte in the frame > header to allow for the carriage of a limited set of color info (0 = > 709/709, 1=2020/2020NCL, 2=2020/2020CL, 3=container specif

[FFmpeg-devel] [ogg] Respect AVERROR codes returned by ogg header parsing.

2017-11-28 Thread Dale Curtis
Fixes ticket #6804. All of the ogg header parsers may return standard AVERROR codes; these return values should not be treated as success. Signed-off-by: Dale Curtis From 34d68a511a2d933b111964f84cfd2d0a289dec97 Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Tue, 28 Nov 2017 13:40:20 -0800 Sub

[FFmpeg-devel] [ogg] Free opus extradata before reallocating.

2017-11-28 Thread Dale Curtis
Otherwise ff_alloc_extradata() just leaks any existing allocated memory. Similar to the patch for ogm extradata leaks. Signed-off-by: Dale Curtis From ac7f2deaa48ac0578be19b178b7c0bc8040bc278 Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Tue, 28 Nov 2017 13:44:49 -0800 Subject: [PATCH] [ogg]

Re: [FFmpeg-devel] [ogg] Respect AVERROR codes returned by ogg header parsing.

2017-11-28 Thread Dale Curtis
Actually packet() was broken too, updated the patch to fix this case too. - dale On Tue, Nov 28, 2017 at 1:41 PM, Dale Curtis wrote: > Fixes ticket #6804. All of the ogg header parsers may return > standard AVERROR codes; these return values should not be > treated as success. > > Signed-off-by

Re: [FFmpeg-devel] [ogg] Respect AVERROR codes returned by ogg header parsing.

2017-11-28 Thread Dale Curtis
Fixes bad parens in the above patch. - dale On Tue, Nov 28, 2017 at 2:03 PM, Dale Curtis wrote: > Actually packet() was broken too, updated the patch to fix this case too. > > - dale > > On Tue, Nov 28, 2017 at 1:41 PM, Dale Curtis > wrote: > >> Fixes ticket #6804. All of the ogg header parser

Re: [FFmpeg-devel] [PATCH V3] examples/vaapi_encode: Add a VA-API encode example.

2017-11-28 Thread Mark Thompson
On 20/11/17 01:38, Jun Zhao wrote: > V3: Use av_hwframe_transfer_date load YUV to HW surface > > From c0edef97f4f123e74e62396b45db01236ae6477d Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Mon, 6 Nov 2017 14:45:27 +0800 > Subject: [PATCH V3] examples/vaapi_encode: Add a VA-API encode example.

Re: [FFmpeg-devel] [PATCH] Add HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-28 Thread Mark Thompson
On 28/11/17 02:23, Mironov, Mikhail wrote: > > >> Ok; done, plus some trivial testing to make sure it works. If you're happy >> with this and noone else says anything then I'll push it tomorrow. > > OK. > >> Relatedly: your name on patches does not match your full name - I've >> preserved th

Re: [FFmpeg-devel] [PATCH] vaapi_h265: general_level_idc should times 3.

2017-11-28 Thread Mark Thompson
On 28/11/17 07:46, Ruiling Song wrote: > Signed-off-by: Ruiling Song > --- > libavcodec/vaapi_encode_h265.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c > index 3ae92a7..32b8bc6 100644 > --- a/libavcodec/

[FFmpeg-devel] [oggvp8] Don't manipulate duration when it's AV_NOPTS_VALUE.

2017-11-28 Thread Dale Curtis
This leads to signed integer overflow. Signed-off-by: Dale Curtis From c1fa869b79aeb314f30746a561903fcfa8f305fb Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Tue, 28 Nov 2017 14:26:55 -0800 Subject: [PATCH] [oggvp8] Don't manipulate duration when it's AV_NOPTS_VALUE. This leads to signed int

Re: [FFmpeg-devel] [PATCH 1/4] avformat/mxfenc: pass MXFPackage around instead of type

2017-11-28 Thread Tomas Härdin
On Tue, 2017-11-28 at 21:09 +0100, Michael Niedermayer wrote: > On Mon, Nov 27, 2017 at 11:00:51AM +0100, Tomas Härdin wrote: > > On Sun, 2017-11-26 at 21:42 -0800, Mark Reid wrote: > > > --- > > > libavformat/mxfenc.c | 99 +- > > > > > > -- > > >

Re: [FFmpeg-devel] [PATCH] lavc/vp8: Fix HWAccel VP8 decoder can't support resolution change.

2017-11-28 Thread Michael Niedermayer
On Tue, Nov 28, 2017 at 01:20:59PM +0800, Jun Zhao wrote: > > vp8.c | 46 ++ > 1 file changed, 34 insertions(+), 12 deletions(-) > d548af3b82a9d77f67fe9621fc7aeee0392fcc89 > 0001-lavc-vp8-Fix-HWAccel-VP8-decoder-can-t-support-resol.patch > From 94d

Re: [FFmpeg-devel] [PATCH] libvpxenc, vp9: add corpus-complexity option

2017-11-28 Thread James Zern
On Mon, Nov 27, 2017 at 10:55 AM, James Zern wrote: > On Mon, Nov 20, 2017 at 5:36 PM, James Zern wrote: >> Corpus VBR mode is a variant of standard VBR where the complexity >> distribution midpoint is passed in rather than calculated for a specific >> clip or chunk. >> >> The valid range is [0,

Re: [FFmpeg-devel] [ogg] Respect AVERROR codes returned by ogg header parsing.

2017-11-28 Thread Michael Niedermayer
On Tue, Nov 28, 2017 at 02:17:21PM -0800, Dale Curtis wrote: > Fixes bad parens in the above patch. > > - dale > > On Tue, Nov 28, 2017 at 2:03 PM, Dale Curtis > wrote: > > > Actually packet() was broken too, updated the patch to fix this case too. > > > > - dale > > > > On Tue, Nov 28, 2017 at

[FFmpeg-devel] [PATCH] examples/hw_decode: Use hw-config information to find pixfmt

2017-11-28 Thread Mark Thompson
This removes all remaining device-type specificity. --- doc/examples/hw_decode.c | 55 1 file changed, 23 insertions(+), 32 deletions(-) diff --git a/doc/examples/hw_decode.c b/doc/examples/hw_decode.c index 9c7adbf51a..8f48f3b706 100644 --- a/doc/

Re: [FFmpeg-devel] [PATCH] examples/hw_decode: Use hw-config information to find pixfmt

2017-11-28 Thread Jun Zhao
On 2017/11/29 8:34, Mark Thompson wrote: > This removes all remaining device-type specificity. > --- > doc/examples/hw_decode.c | 55 > > 1 file changed, 23 insertions(+), 32 deletions(-) > > diff --git a/doc/examples/hw_decode.c b/doc/examples/h

Re: [FFmpeg-devel] [PATCH] lavc/vp8: Fix HWAccel VP8 decoder can't support resolution change.

2017-11-28 Thread Jun Zhao
On 2017/11/29 7:09, Michael Niedermayer wrote: > On Tue, Nov 28, 2017 at 01:20:59PM +0800, Jun Zhao wrote: >> vp8.c | 46 ++ >> 1 file changed, 34 insertions(+), 12 deletions(-) >> d548af3b82a9d77f67fe9621fc7aeee0392fcc89 >> 0001-lavc-vp8-Fix-HWAcc

[FFmpeg-devel] [PATCH] avcodec/dnxhddec: Do not overwrite colorspace if the container has set it.

2017-11-28 Thread Steven Robertson
The existing logic overrides container metadata even in cases where the container metadata must be trusted (e.g. HDR). The original spec had no provision for specifying color volume, so many files rely on the assumption of Rec. 709. An update to the spec included a 'clv' field for explicitly signa

[FFmpeg-devel] [PATCH] swscale/utils: Remove bpc==8 gating init_range_convert

2017-11-28 Thread Neil Birkbeck
On higher bit depth YUV inputs with range metadata signaled as PC/full levels, this change makes the results of scaling without explicitly setting the input range on vf_scale as if it were explicitly set. For example, the results with implicit color settings from the frame: -vf scale=-1:-1:out_ran

[FFmpeg-devel] [PATCH 1/5] avformat/avc: return an error in ff_isom_write_avcc if the buffer lenght is too small

2017-11-28 Thread James Almer
Signed-off-by: James Almer --- libavformat/avc.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavformat/avc.c b/libavformat/avc.c index 094a95821f..7b32590778 100644 --- a/libavformat/avc.c +++ b/libavformat/avc.c @@ -105,7 +105,9 @@ int ff_avc_parse_nal_units_buf(co

[FFmpeg-devel] [PATCH 2/5] avformat/avc: refactor ff_isom_write_avcc

2017-11-28 Thread James Almer
This lets us remove one indentation level. Signed-off-by: James Almer --- libavformat/avc.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/libavformat/avc.c b/libavformat/avc.c index 7b32590778..6ec2f19577 100644 --- a/libavformat/avc.c +++ b/libavforma

[FFmpeg-devel] [PATCH 4/5] avformat/avc: free buffer in ff_isom_write_avcc on failure

2017-11-28 Thread James Almer
Signed-off-by: James Almer --- libavformat/avc.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavformat/avc.c b/libavformat/avc.c index 7542db684b..6ef6e08778 100644 --- a/libavformat/avc.c +++ b/libavformat/avc.c @@ -145,8 +145,10 @@ int ff_isom_write_avcc(AVI

[FFmpeg-devel] [PATCH 3/5] avformat/avc: reindent after the last commit

2017-11-28 Thread James Almer
Signed-off-by: James Almer --- libavformat/avc.c | 91 --- 1 file changed, 46 insertions(+), 45 deletions(-) diff --git a/libavformat/avc.c b/libavformat/avc.c index 6ec2f19577..7542db684b 100644 --- a/libavformat/avc.c +++ b/libavformat/avc.c

[FFmpeg-devel] [PATCH 5/5] avformat/avc: support writting more than one sps/pps in ff_isom_write_avcc

2017-11-28 Thread James Almer
Addresses ticket #6864 Signed-off-by: James Almer --- I don't have the h264 in isobmff spec at hand, so i just looked at what the h264_mp4toannexb bsf does to handle more than one sps and pps and worked with that. If there's a pps limit it's not currently enforced, but adding it is trivial. The s

Re: [FFmpeg-devel] [PATCH 1/4] avformat/mxfenc: pass MXFPackage around instead of type

2017-11-28 Thread Mark Reid
On Tue, Nov 28, 2017 at 3:05 PM, Tomas Härdin wrote: > On Tue, 2017-11-28 at 21:09 +0100, Michael Niedermayer wrote: > > On Mon, Nov 27, 2017 at 11:00:51AM +0100, Tomas Härdin wrote: > > > On Sun, 2017-11-26 at 21:42 -0800, Mark Reid wrote: > > > > --- > > > > libavformat/mxfenc.c | 99 +

Re: [FFmpeg-devel] [PATCH] swscale/utils: Remove bpc==8 gating init_range_convert

2017-11-28 Thread Neil Birkbeck
On Tue, Nov 28, 2017 at 5:14 PM, Neil Birkbeck wrote: > -//The srcBpc check is possibly wrong but we seem to lack a definitive > reference to test this > -//and what we have in ticket 2939 looks better with this check > -if (need_reinit && (c->srcBpc == 8 || !isYUV(c->srcFormat))) > +

Re: [FFmpeg-devel] [PATCH] lavc/vp8: Fix HWAccel VP8 decoder can't support resolution change.

2017-11-28 Thread Jun Zhao
On 2017/11/28 21:23, Carl Eugen Hoyos wrote: > 2017-11-28 6:20 GMT+01:00 Jun Zhao : > > Could be split. Do you means split with 2 patches? 1) for issue fix 2) for coding reconstruct ? > > Carl Eugen > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpe

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mlpenc: encode last frame too

2017-11-28 Thread James Almer
On 11/28/2017 4:55 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/mlpenc.c | 22 +++--- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/libavcodec/mlpenc.c b/libavcodec/mlpenc.c > index 7536d3b2f5..c588f5b904 100644 > --- a/libavcode

Re: [FFmpeg-devel] [PATCH] Add android_capture indev

2017-11-28 Thread Michael Niedermayer
On Fri, Nov 10, 2017 at 09:41:04PM +0100, Felix Matouschek wrote: > Hello, > > here is a new version of the patch with further fixes. [...] > +static const char *camera_status_string(camera_status_t val) > +{ > +switch(val) { > +case ACAMERA_OK: > +return "ACAMERA_OK"; if

Re: [FFmpeg-devel] [ogg] Free opus extradata before reallocating.

2017-11-28 Thread Michael Niedermayer
On Tue, Nov 28, 2017 at 01:47:11PM -0800, Dale Curtis wrote: > Otherwise ff_alloc_extradata() just leaks any existing allocated > memory. Similar to the patch for ogm extradata leaks. > > Signed-off-by: Dale Curtis > oggparseopus.c |1 + > 1 file changed, 1 insertion(+) > bf0b03b22a4253

Re: [FFmpeg-devel] [PATCH] vaapi_h265: general_level_idc should times 3.

2017-11-28 Thread Li, Zhong
> On 28/11/17 07:46, Ruiling Song wrote: > > Signed-off-by: Ruiling Song > > --- > > libavcodec/vaapi_encode_h265.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/vaapi_encode_h265.c > > b/libavcodec/vaapi_encode_h265.c index 3ae92a7..32b8bc6 100644 > >

Re: [FFmpeg-devel] [oggvp8] Don't manipulate duration when it's AV_NOPTS_VALUE.

2017-11-28 Thread James Almer
On 11/28/2017 7:28 PM, Dale Curtis wrote: > This leads to signed integer overflow. > > Signed-off-by: Dale Curtis > > > ogg_vp8_v1.patch > > > From c1fa869b79aeb314f30746a561903fcfa8f305fb Mon Sep 17 00:00:00 2001 > From: Dale Curtis > Date: Tue, 28 Nov 2017 14:26:55 -0800 > Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/avfilter: simplify filter registration

2017-11-28 Thread Rostislav Pehlivanov
On 28 November 2017 at 20:23, Michael Niedermayer wrote: > On Mon, Nov 27, 2017 at 04:30:19AM +, Rostislav Pehlivanov wrote: > > Atomics were entirely pointless and did nothing but slow and complicate > > the process down. This could be improved further still but the main > > objective of thi

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mlpenc: fix undefined shift

2017-11-28 Thread Rostislav Pehlivanov
On 28 November 2017 at 19:55, Paul B Mahol wrote: > Decreases artifacts. > > Signed-off-by: Paul B Mahol > --- > libavcodec/mlpenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/mlpenc.c b/libavcodec/mlpenc.c > index c588f5b904..eceb0ddbb2 100644 > --- a/l

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mlpenc: fix undefined shift

2017-11-28 Thread James Almer
On 11/29/2017 1:05 AM, Rostislav Pehlivanov wrote: > On 28 November 2017 at 19:55, Paul B Mahol wrote: > >> Decreases artifacts. >> >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/mlpenc.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/mlpenc.c b/liba

[FFmpeg-devel] [PATCH v2 2/3] avformat/mxfenc: write reel_name if metadata key is present

2017-11-28 Thread Mark Reid
--- libavformat/mxf.h| 1 + libavformat/mxfenc.c | 42 +++--- 2 files changed, 36 insertions(+), 7 deletions(-) diff --git a/libavformat/mxf.h b/libavformat/mxf.h index 2d5b44943b..ffcc429a8b 100644 --- a/libavformat/mxf.h +++ b/libavformat/mxf.h @@ -47,6

[FFmpeg-devel] [PATCH v2 1/3] avformat/mxfenc: use track count to generate component instance uuid

2017-11-28 Thread Mark Reid
--- libavformat/mxf.h | 1 - libavformat/mxfenc.c| 30 +- tests/ref/fate/copy-trac4914| 2 +- tests/ref/fate/time_base| 2 +- tests/ref/lavf/mxf | 6 +++--- tests/ref/lavf/mxf_d10 | 2 +- tests/ref/lavf/mx

[FFmpeg-devel] [PATCH v2 0/3] avformat/mxfenc add reel_name write support

2017-11-28 Thread Mark Reid
changes since v1 * simplified uuid_offset * added SourcePackage type checks * only support reel_name at the format level Mark Reid (3): avformat/mxfenc: use track count to generate component instance uuid avformat/mxfenc: write reel_name if metadata key is present fate/mxf: add reel name tes

[FFmpeg-devel] [PATCH v2 3/3] fate/mxf: add reel name test

2017-11-28 Thread Mark Reid
--- tests/fate/mxf.mak | 8 ++-- tests/ref/fate/mxf-reel_name | 1 + 2 files changed, 7 insertions(+), 2 deletions(-) create mode 100644 tests/ref/fate/mxf-reel_name diff --git a/tests/fate/mxf.mak b/tests/fate/mxf.mak index 7714b61569..dce23d522e 100644 --- a/tests/fate/mxf.mak ++

Re: [FFmpeg-devel] [PATCH v2 3/3] libavformat/hlsenc: Persistent HTTP connections supported as an option

2017-11-28 Thread Jeyapal, Karthick
On 11/24/17, 3:49 PM, "Jeyapal, Karthick" wrote: > >On 11/24/17, 3:40 PM, "刘歧" wrote: > >>> 在 2017年11月24日,18:02,Jeyapal, Karthick 写道: >>> >>> On 11/22/17, 1:27 PM, "Jeyapal, Karthick" wrote: >>> > On 11/22/17, 9:32 AM, "刘歧" wrote: > > This patch is ok, but i see it need an API:

Re: [FFmpeg-devel] [PATCH v2 3/3] libavformat/hlsenc: Persistent HTTP connections supported as an option

2017-11-28 Thread 刘歧
> 在 2017年11月29日,12:05,Jeyapal, Karthick 写道: > > > On 11/24/17, 3:49 PM, "Jeyapal, Karthick" wrote: >> >> On 11/24/17, 3:40 PM, "刘歧" wrote: >> 在 2017年11月24日,18:02,Jeyapal, Karthick 写道: On 11/22/17, 1:27 PM, "Jeyapal, Karthick" wrote: >> On 11/22/17, 9:32 AM, "刘歧"

[FFmpeg-devel] [PATCH v3 1/3] libavformat/avio: Utility function to return URLContext

2017-11-28 Thread Karthick J
--- libavformat/avio_internal.h | 8 libavformat/aviobuf.c | 13 + 2 files changed, 21 insertions(+) diff --git a/libavformat/avio_internal.h b/libavformat/avio_internal.h index c01835d..04c1ad5 100644 --- a/libavformat/avio_internal.h +++ b/libavformat/avio_internal.h

[FFmpeg-devel] [PATCH v3 2/3] libavformat/http: Handled multiple_requests option during write

2017-11-28 Thread Karthick J
--- libavformat/http.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index 056d5f6..cf86adc 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -171,6 +171,7 @@ static int http_connect(URLContext *h, const char *path, const char *local_p

[FFmpeg-devel] [PATCH v3 3/3] libavformat/hlsenc: Persistent HTTP connections supported as an option

2017-11-28 Thread Karthick J
--- doc/muxers.texi | 3 +++ libavformat/hlsenc.c | 48 +--- 2 files changed, 44 insertions(+), 7 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 9d9ca31..8ec48c2 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -854,6 +854,

Re: [FFmpeg-devel] [PATCH v2 3/3] libavformat/hlsenc: Persistent HTTP connections supported as an option

2017-11-28 Thread Jeyapal, Karthick
On 11/29/17, 11:41 AM, "刘歧" wrote: >> 在 2017年11月29日,12:05,Jeyapal, Karthick 写道: >> >> A gentle reminder. >> 7 days have gone by since the last version of this patchset was submitted. >> And no objections so far. >> Could this patchset be pushed, please? Thanks. >Can you resubmit a new version

Re: [FFmpeg-devel] [PATCH v3 3/3] libavformat/hlsenc: Persistent HTTP connections supported as an option

2017-11-28 Thread 刘歧
> 在 2017年11月29日,14:19,Karthick J 写道: > > --- > doc/muxers.texi | 3 +++ > libavformat/hlsenc.c | 48 +--- > 2 files changed, 44 insertions(+), 7 deletions(-) > > diff --git a/doc/muxers.texi b/doc/muxers.texi > index 9d9ca31..8ec48c2 100644 > ---

Re: [FFmpeg-devel] [PATCH v3 3/3] libavformat/hlsenc: Persistent HTTP connections supported as an option

2017-11-28 Thread Jeyapal, Karthick
On 11/29/17, 12:04 PM, "刘歧" wrote: >Patchset pushed! Thanks a lot! Regards, Karthick > > >Thanks ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel