Re: [FFmpeg-devel] [PATCH 1/1] avformat/http: flushing tcp receive buffer when it is write only mode

2018-03-27 Thread Steven Liu
> On 28 Mar 2018, at 12:52, vdi...@akamai.com wrote: > > From: Vishwanath Dixit > > In write only mode, the TCP receive buffer keeps growing and eventually > becomes full. This results in zero tcp window size, which in turn causes > unwanted issues, like, terminated tcp

[FFmpeg-devel] [PATCH 1/1] avformat/http: flushing tcp receive buffer when it is write only mode

2018-03-27 Thread vdixit
From: Vishwanath Dixit In write only mode, the TCP receive buffer keeps growing and eventually becomes full. This results in zero tcp window size, which in turn causes unwanted issues, like, terminated tcp connection. The issue is apparent when http persistent connection is

Re: [FFmpeg-devel] [PATCH] avcodec/get_bits: Make sure the input bitstream with padding can be addressed

2018-03-27 Thread James Almer
On 3/25/2018 9:13 PM, Michael Niedermayer wrote: > On Sat, Mar 24, 2018 at 01:56:26AM +0100, Michael Niedermayer wrote: >> Signed-off-by: Michael Niedermayer >> --- >> libavcodec/get_bits.h | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) > > will apply HOSTCC

[FFmpeg-devel] [PATCH] movtextenc: fix handling of utf-8 subtitles

2018-03-27 Thread Philip Langdale
See the earlier fix for movtextdec for details. The equivalent bug is present on the encoder side as well. We need to track the text length in 'characters' (which seems to really mean codepoints) to ensure that styles are applied across the correct ranges. Signed-off-by: Philip Langdale

Re: [FFmpeg-devel] [PATCH v2] avfilter: add OpenCL scale filter

2018-03-27 Thread Rostislav Pehlivanov
On 27 March 2018 at 15:54, Gabriel Machado wrote: > On 3/27/18 10:18 AM Rostislav Pehlivanov wrote: > > > No license, can't use it. Shadertoy has no explicit license. > > > > > > Moreover the whole filter is incorrectly designed. Take a look at what > mpv > > does and

[FFmpeg-devel] [PATCH] avutil/buffer: add a dynamic size buffer pool API

2018-03-27 Thread James Almer
Signed-off-by: James Almer --- Implemented as a completely separate API as suggested. Missing Changelog, APIChanges and version bump as usual. libavutil/buffer.c | 159 libavutil/buffer.h | 53 +++

Re: [FFmpeg-devel] [PATCH][RFC] avcodec/avutil: Add timeline side data

2018-03-27 Thread Derek Buitenhuis
On 3/27/2018 11:46 PM, Alexander Strasser wrote: >> + * of the stream's rate, for example: 1.2 means play back this entry at >> 1.2x speed. >> + * If this value is 0, then the first sample (located at 'start') must >> be displayed >> + * for the duration of the entry. >> + */ >>

Re: [FFmpeg-devel] [PATCH] hwcontext_d3d11: Fix crash with valid adapter but no device

2018-03-27 Thread Mark Thompson
On 27/03/18 16:10, wm4 wrote: > On Tue, 27 Mar 2018 01:22:34 +0100 > Mark Thompson wrote: > >> This crash was introduced by 8bbf2dacbfb4ead1535dea411035994f507f517d, >> which could incorrectly overwrite the failure result from creating the >> device. >> >> Fixes ticket #7108. >>

Re: [FFmpeg-devel] [PATCH 2/5] h264_metadata: Add support for A/53 closed captions

2018-03-27 Thread Mark Thompson
On 27/03/18 21:38, Michael Niedermayer wrote: > On Tue, Mar 27, 2018 at 01:31:43AM +0100, Mark Thompson wrote: >> On 27/03/18 01:20, Michael Niedermayer wrote: >>> On Sun, Mar 25, 2018 at 06:41:34PM +0100, Mark Thompson wrote: Allows insertion (from side data), extraction (to side data), and

Re: [FFmpeg-devel] [PATCH] [RFC][WIP] avutil/buffer: add add a dynamnic size buffer pool API

2018-03-27 Thread James Almer
On 3/27/2018 2:58 PM, wm4 wrote: > On Tue, 27 Mar 2018 14:16:15 -0300 > James Almer wrote: > >> On 3/20/2018 7:44 PM, James Almer wrote: >>> On 3/16/2018 3:21 PM, James Almer wrote: Signed-off-by: James Almer --- This is a proof of concept

[FFmpeg-devel] [PATCH 1/5 v2] avformat/utils: make AVPacketList helper functions shared

2018-03-27 Thread James Almer
Based on a patch by Luca Barbato. Signed-off-by: James Almer --- libavformat/internal.h | 38 + libavformat/utils.c| 57 +- 2 files changed, 71 insertions(+), 24 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH][RFC] avcodec/avutil: Add timeline side data

2018-03-27 Thread Alexander Strasser
Hi Derek, I am happy to see someone trying to extend FFmpeg to support these kind of features in a deeper way. Good luck on this journey! Below I am mostly commenting on typos and rather minor things. So you should for now mostly ignore my comments, but it is easier to write them now that I

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/eac3: add support for dependent stream

2018-03-27 Thread Michael Niedermayer
On Tue, Mar 27, 2018 at 10:55:05PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/aac_ac3_parser.c | 9 ++- > libavcodec/ac3_parser.c | 2 +- > libavcodec/ac3dec.c | 177 > +++- >

Re: [FFmpeg-devel] [PATCH 1/5] avformat/utils: make AVPacketList helper functions shared

2018-03-27 Thread Michael Niedermayer
On Tue, Mar 27, 2018 at 06:17:09PM -0300, James Almer wrote: > On 3/27/2018 5:22 PM, Michael Niedermayer wrote: > > On Mon, Mar 26, 2018 at 03:02:35PM -0300, James Almer wrote: > >> Based on a patch by Luca Barbato. > >> > >> Signed-off-by: James Almer > >> --- > >>

Re: [FFmpeg-devel] [PATCH 1/5] avformat/utils: make AVPacketList helper functions shared

2018-03-27 Thread James Almer
On 3/27/2018 5:22 PM, Michael Niedermayer wrote: > On Mon, Mar 26, 2018 at 03:02:35PM -0300, James Almer wrote: >> Based on a patch by Luca Barbato. >> >> Signed-off-by: James Almer >> --- >> libavformat/internal.h | 35 ++ >>

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread wm4
On Tue, 27 Mar 2018 16:45:23 -0400 Dave Rice wrote: > > On Mar 27, 2018, at 4:33 PM, wm4 wrote: > > > > On Tue, 27 Mar 2018 16:11:11 -0400 > > Dave Rice wrote: > > > >>> On Mar 27, 2018, at 4:01 PM, Derek Buitenhuis > >>>

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread wm4
On Tue, 27 Mar 2018 16:45:23 -0400 Dave Rice wrote: > > On Mar 27, 2018, at 4:33 PM, wm4 wrote: > > > > On Tue, 27 Mar 2018 16:11:11 -0400 > > Dave Rice wrote: > > > >>> On Mar 27, 2018, at 4:01 PM, Derek Buitenhuis > >>>

[FFmpeg-devel] [PATCH 1/2] avcodec/eac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/aac_ac3_parser.c | 9 ++- libavcodec/ac3_parser.c | 2 +- libavcodec/ac3dec.c | 177 +++- libavcodec/ac3dec.h | 10 ++- libavcodec/eac3dec.c| 11 +-- 5 files

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Dave Rice
> On Mar 27, 2018, at 4:33 PM, wm4 wrote: > > On Tue, 27 Mar 2018 16:11:11 -0400 > Dave Rice wrote: > >>> On Mar 27, 2018, at 4:01 PM, Derek Buitenhuis >>> wrote: >>> >>> On 3/27/2018 8:52 PM, Rostislav Pehlivanov wrote:

Re: [FFmpeg-devel] [PATCH 2/5] h264_metadata: Add support for A/53 closed captions

2018-03-27 Thread Michael Niedermayer
On Tue, Mar 27, 2018 at 01:31:43AM +0100, Mark Thompson wrote: > On 27/03/18 01:20, Michael Niedermayer wrote: > > On Sun, Mar 25, 2018 at 06:41:34PM +0100, Mark Thompson wrote: > >> Allows insertion (from side data), extraction (to side data), and removal > >> of closed captions in SEI messages.

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread wm4
On Tue, 27 Mar 2018 16:11:11 -0400 Dave Rice wrote: > > On Mar 27, 2018, at 4:01 PM, Derek Buitenhuis > > wrote: > > > > On 3/27/2018 8:52 PM, Rostislav Pehlivanov wrote: > >> I think we should drop the internal crap if the tools and the API

Re: [FFmpeg-devel] [PATCH 1/5] avformat/utils: make AVPacketList helper functions shared

2018-03-27 Thread Michael Niedermayer
On Mon, Mar 26, 2018 at 03:02:35PM -0300, James Almer wrote: > Based on a patch by Luca Barbato. > > Signed-off-by: James Almer > --- > libavformat/internal.h | 35 ++ > libavformat/utils.c| 51 >

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Rostislav Pehlivanov
On 27 March 2018 at 21:01, Derek Buitenhuis wrote: > On 3/27/2018 8:52 PM, Rostislav Pehlivanov wrote: > > I think we should drop the internal crap if the tools and the API support > > it. Would also solve a lot of issues like ffmpeg.c not trimming the start > > frame

Re: [FFmpeg-devel] [PATCH 5/5] avformat/mp3enc: use AVPacketList helper functions to queue packets

2018-03-27 Thread Michael Niedermayer
On Mon, Mar 26, 2018 at 03:02:39PM -0300, James Almer wrote: > Simplifies code. > > Signed-off-by: James Almer > --- > libavformat/mp3enc.c | 23 +++ > 1 file changed, 7 insertions(+), 16 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Dave Rice
> On Mar 27, 2018, at 4:01 PM, Derek Buitenhuis > wrote: > > On 3/27/2018 8:52 PM, Rostislav Pehlivanov wrote: >> I think we should drop the internal crap if the tools and the API support >> it. Would also solve a lot of issues like ffmpeg.c not trimming the start

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Paul B Mahol
On 3/27/18, Derek Buitenhuis wrote: > On 3/27/2018 8:52 PM, Rostislav Pehlivanov wrote: >> I think we should drop the internal crap if the tools and the API support >> it. Would also solve a lot of issues like ffmpeg.c not trimming the start >> frame (so people

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Derek Buitenhuis
On 3/27/2018 8:52 PM, Rostislav Pehlivanov wrote: > I think we should drop the internal crap if the tools and the API support > it. Would also solve a lot of issues like ffmpeg.c not trimming the start > frame (so people complain all the time about longer files). I personally agree, but I thought

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Rostislav Pehlivanov
On 27 March 2018 at 20:44, Derek Buitenhuis wrote: > So, I know we have edit list "support" in mov.c, but I am steadfast in my > belief that it is incorrect to implement it at the demuxing layer. By the > ISOBMFF spec, it is supposed to be applied at the presenattion

[FFmpeg-devel] [PATCH v3] avfilter: add OpenCL scale filter

2018-03-27 Thread Gabriel Machado
> > +__kernel void bilinear(__write_only image2d_t dst, > > + __read_only image2d_t src) > > +{ > > +const sampler_t sampler = (CLK_NORMALIZED_COORDS_TRUE | > > + CLK_ADDRESS_CLAMP_TO_EDGE | > > +

[FFmpeg-devel] [PATCH 1/2] avcodec/eac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/ac3_parser.c | 2 +- libavcodec/ac3dec.c | 167 +++- libavcodec/ac3dec.h | 9 ++- libavcodec/eac3dec.c| 11 +--- 4 files changed, 147 insertions(+), 42 deletions(-) diff

[FFmpeg-devel] [PATCH][RFC] avcodec/avutil: Add timeline side data

2018-03-27 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- libavcodec/avcodec.h | 8 +++ libavutil/timeline.h | 160 +++ 2 files changed, 168 insertions(+) create mode 100644 libavutil/timeline.h diff --git a/libavcodec/avcodec.h

[FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Derek Buitenhuis
So, I know we have edit list "support" in mov.c, but I am steadfast in my belief that it is incorrect to implement it at the demuxing layer. By the ISOBMFF spec, it is supposed to be applied at the presenattion layer. I'm aware we probably cannot remove the current implementation very easily (or

Re: [FFmpeg-devel] State of decklink_ctx vs. decklink_cctx

2018-03-27 Thread Devin Heitmueller
Hi Grady, > On Mar 27, 2018, at 3:05 PM, grady player wrote: > > So I haven't looked in great detail so this may all be info that you know, > and maybe not helpful... > > 1. You can easily link C objects to C++ code by marking it `extern "C"` > 2. You can not easily link

Re: [FFmpeg-devel] State of decklink_ctx vs. decklink_cctx

2018-03-27 Thread grady player
So I haven't looked in great detail so this may all be info that you know, and maybe not helpful... 1. You can easily link C objects to C++ code by marking it `extern "C"` 2. You can not easily link C++ objects to C code, because there are classes, vtables, name mangling etc. 3. The solution

[FFmpeg-devel] State of decklink_ctx vs. decklink_cctx

2018-03-27 Thread Devin Heitmueller
Hello, I’m continuing to do some cleanup work on the decklink libavdevice input/output, and I’m trying to understand how the way state is managed has evolved over time. Specifically, there are two key state structures - decklink_ctx and decklink_cctx. It would appear the motivation behind

Re: [FFmpeg-devel] [PATCH 4/5] mpeg2_metadata: Add support for A/53 closed captions

2018-03-27 Thread Aman Gupta
On Sun, Mar 25, 2018 at 10:41 AM, Mark Thompson wrote: > Allows extraction (to side data) and removal of closed captions in > user data blocks. > --- > doc/bitstream_filters.texi | 12 ++ > libavcodec/Makefile | 2 +- > libavcodec/mpeg2_metadata_bsf.c | 81

Re: [FFmpeg-devel] [PATCH] [RFC][WIP] avutil/buffer: add add a dynamnic size buffer pool API

2018-03-27 Thread wm4
On Tue, 27 Mar 2018 14:16:15 -0300 James Almer wrote: > On 3/20/2018 7:44 PM, James Almer wrote: > > On 3/16/2018 3:21 PM, James Almer wrote: > >> Signed-off-by: James Almer > >> --- > >> This is a proof of concept for a dynamic size buffer pool API. > >>

Re: [FFmpeg-devel] [GSOC] [PATCH] SRCNN filter

2018-03-27 Thread Michael Niedermayer
On Sun, Mar 25, 2018 at 08:04:17PM +0300, Sergey Lavrushkin wrote: > Hello, > > This is my implementation of qualification task for GSOC Super Resolution > Filter project. > For default x2 upsampling with provided srcnn filter following command can > be used: > ffmpeg -i input -vf

Re: [FFmpeg-devel] [PATCH] [RFC][WIP] avutil/buffer: add add a dynamnic size buffer pool API

2018-03-27 Thread James Almer
On 3/20/2018 7:44 PM, James Almer wrote: > On 3/16/2018 3:21 PM, James Almer wrote: >> Signed-off-by: James Almer >> --- >> This is a proof of concept for a dynamic size buffer pool API. >> >> For the purpose of easy testing and reviewing I replaced the current >> linked list

Re: [FFmpeg-devel] [PATCH v2 RESEND] avfilter: add OpenCL scale filter

2018-03-27 Thread Rostislav Pehlivanov
On 27 March 2018 at 16:09, Gabriel Machado wrote: > Sorry, my mail client corrupted the patch. > > --- > configure | 1 + > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/opencl/scale.cl | 111

[FFmpeg-devel] [PATCH 2/2] avcodec: add eac3_core bitstream filter

2018-03-27 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/bitstream_filters.texi | 4 libavcodec/Makefile| 1 + libavcodec/bitstream_filters.c | 1 + libavcodec/eac3_core_bsf.c | 52 ++ 4 files changed, 58 insertions(+) create mode

[FFmpeg-devel] [PATCH 1/2] avcodec/eac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/ac3_parser.c | 2 +- libavcodec/ac3dec.c | 160 +++- libavcodec/ac3dec.h | 9 ++- libavcodec/eac3dec.c| 11 +--- 4 files changed, 140 insertions(+), 42 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] avfilter/vf_scale: Do not set YUV color range for RGB formats

2018-03-27 Thread wm4
On Tue, 27 Mar 2018 12:14:55 +0200 Michael Niedermayer wrote: > On Mon, Mar 26, 2018 at 10:56:47PM +0100, Mark Thompson wrote: > > On 26/03/18 22:44, Michael Niedermayer wrote: > > > On Mon, Mar 26, 2018 at 08:34:06AM +0200, Paul B Mahol wrote: > > >> On 3/26/18,

Re: [FFmpeg-devel] [PATCH] hwcontext_d3d11: Fix crash with valid adapter but no device

2018-03-27 Thread wm4
On Tue, 27 Mar 2018 01:22:34 +0100 Mark Thompson wrote: > This crash was introduced by 8bbf2dacbfb4ead1535dea411035994f507f517d, > which could incorrectly overwrite the failure result from creating the > device. > > Fixes ticket #7108. > --- > libavutil/hwcontext_d3d11va.c | 8

[FFmpeg-devel] [PATCH v2 RESEND] avfilter: add OpenCL scale filter

2018-03-27 Thread Gabriel Machado
Sorry, my mail client corrupted the patch. --- configure | 1 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/opencl/scale.cl | 111 + libavfilter/opencl_source.h | 1 + libavfilter/vf_scale_opencl.c | 284

Re: [FFmpeg-devel] Moving enum AVFieldOrder to libavutil?

2018-03-27 Thread wm4
On Mon, 26 Mar 2018 11:48:01 -0400 Devin Heitmueller wrote: > Hello all, > > > On Mar 24, 2018, at 6:37 AM, Michael Niedermayer > > wrote: > > > > On Sat, Mar 24, 2018 at 01:07:48AM +0100, Marton Balint wrote: > >> > >> > >> On Fri, 23

[FFmpeg-devel] [PATCH v2] avfilter: add OpenCL scale filter

2018-03-27 Thread Gabriel Machado
On 3/27/18 10:18 AM Rostislav Pehlivanov wrote: > No license, can't use it. Shadertoy has no explicit license. > > > Moreover the whole filter is incorrectly designed. Take a look at what mpv > does and how it has no explicit per-algorithm scaling functions. Thanks for the feedback! I removed

Re: [FFmpeg-devel] [PATCH] avcodec/ac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
On 3/27/18, Hendrik Leppkes wrote: > On Tue, Mar 27, 2018 at 1:57 PM, Paul B Mahol wrote: >> /* keep last block for error concealment in next frame */ >> for (ch = 0; ch < s->out_channels; ch++) >> -memcpy(s->output[ch], output[ch], >>

Re: [FFmpeg-devel] [PATCH] avcodec/ac3: add support for dependent stream

2018-03-27 Thread Hendrik Leppkes
On Tue, Mar 27, 2018 at 1:57 PM, Paul B Mahol wrote: > /* keep last block for error concealment in next frame */ > for (ch = 0; ch < s->out_channels; ch++) > -memcpy(s->output[ch], output[ch], AC3_BLOCK_SIZE*sizeof(SHORTFLOAT)); > +memcpy(s->output[ch +

Re: [FFmpeg-devel] [PATCH v3] libavformat/dashdec: Support signaling of last segment number

2018-03-27 Thread Steven Liu
> On 27 Mar 2018, at 9:28 PM, sanilraut wrote: > > Last segment indicated by mpd is not parsed. > Example stream: > http://dash.akamaized.net/dash264/TestCasesIOP41/LastSegmentNumber/1/manifest_last_segment_num.mpd > > This patch supports parsing of Supplemental

Re: [FFmpeg-devel] [PATCH] avcodec/avpacket: remove unnecessary check in av_packet_make_writable()

2018-03-27 Thread James Almer
On 3/25/2018 9:25 PM, James Almer wrote: > Zero sized packets are already handled below in the function. > This is more in line with av_packet_ref(). > > Signed-off-by: James Almer > --- > libavcodec/avpacket.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] avcodec/ac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
On 3/27/18, James Almer wrote: > On 3/27/2018 8:57 AM, Paul B Mahol wrote: >> @@ -1511,7 +1534,7 @@ static int ac3_decode_frame(AVCodecContext * avctx, >> void *data, >> break; >> case AAC_AC3_PARSE_ERROR_FRAME_TYPE: >> /* skip frame if CRC is

Re: [FFmpeg-devel] [PATCH] avcodec/ac3: add support for dependent stream

2018-03-27 Thread James Almer
On 3/27/2018 8:57 AM, Paul B Mahol wrote: > @@ -1511,7 +1534,7 @@ static int ac3_decode_frame(AVCodecContext * avctx, > void *data, > break; > case AAC_AC3_PARSE_ERROR_FRAME_TYPE: > /* skip frame if CRC is ok. otherwise use error concealment. */ > -

[FFmpeg-devel] [PATCH v3] libavformat/dashdec: Support signaling of last segment number

2018-03-27 Thread sanilraut
Last segment indicated by mpd is not parsed. Example stream: http://dash.akamaized.net/dash264/TestCasesIOP41/LastSegmentNumber/1/manifest_last_segment_num.mpd This patch supports parsing of Supplemental Descriptor with @schemeIdUri set to http://dashif.org/guide- lines/last-segment-number with

Re: [FFmpeg-devel] [PATCH v2] avformat/dashdec: Support signaling of last segment number

2018-03-27 Thread sanilraut
Thanks. Have re-submitted the patch. Sanil On Mon, Mar 26, 2018 at 7:10 PM, Steven Liu wrote: > > > > On 26 Mar 2018, at 04:01, sanilraut wrote: > > > > Last segment indicated by mpd is not parsed. > > Example stream:

Re: [FFmpeg-devel] [PATCH] avfilter: add OpenCL scale filter

2018-03-27 Thread Rostislav Pehlivanov
On 27 March 2018 at 05:48, Gabriel Machado wrote: > From: Gabriel Machado > > Some scaling filters implemented as OpenCL kernels. Can be used as: > > scale_opencl=::flags= > where can be `neighbor', `bilinear', `bicubic' or `fast_bicubic' > >

[FFmpeg-devel] [PATCH v3] avcodec/arm/hevcdsp_sao : add NEON optimization for sao

2018-03-27 Thread Yingming Fan
From: Meng Wang Signed-off-by: Meng Wang --- This v3 patch removed unused codes 'stride_dst /= sizeof(uint8_t);' compared to v1. V1 have this codes because we referred to hevc dsp template codes. Also removed type cast like 'uint8_t

Re: [FFmpeg-devel] [PATCH] avcodec/ac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
On 3/27/18, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/ac3_parser.c | 2 +- > libavcodec/ac3dec.c | 152 > +++- > libavcodec/ac3dec.h | 9 ++- > libavcodec/eac3dec.c| 11

[FFmpeg-devel] [PATCH] avcodec/ac3: add support for dependent stream

2018-03-27 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/ac3_parser.c | 2 +- libavcodec/ac3dec.c | 152 +++- libavcodec/ac3dec.h | 9 ++- libavcodec/eac3dec.c| 11 +--- 4 files changed, 134 insertions(+), 40 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] avfilter/vf_scale: Do not set YUV color range for RGB formats

2018-03-27 Thread Michael Niedermayer
On Mon, Mar 26, 2018 at 10:56:47PM +0100, Mark Thompson wrote: > On 26/03/18 22:44, Michael Niedermayer wrote: > > On Mon, Mar 26, 2018 at 08:34:06AM +0200, Paul B Mahol wrote: > >> On 3/26/18, Michael Niedermayer wrote: > >>> On Wed, Mar 21, 2018 at 09:18:21AM +0100, Paul

Re: [FFmpeg-devel] [PATCH] lavc/hevc: Don't parse NAL unit for a dummy buffer

2018-03-27 Thread Steven Liu
> On 27 Mar 2018, at 15:57, Haihao Xiang wrote: > > hevc parser mistakenly reports the following message if a dummy buffer > is padded for EOF > > [hevc @ 0x559b63848610] missing picture in access unit > > Signed-off-by: Haihao Xiang > --- >

[FFmpeg-devel] [PATCH] lavc/hevc: Don't parse NAL unit for a dummy buffer

2018-03-27 Thread Haihao Xiang
hevc parser mistakenly reports the following message if a dummy buffer is padded for EOF [hevc @ 0x559b63848610] missing picture in access unit Signed-off-by: Haihao Xiang --- libavcodec/hevc_parser.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff

Re: [FFmpeg-devel] [PATCH 1/2] lavc/cfhd: error due to improper allocation of height in buffers

2018-03-27 Thread Gagandeep Singh
On Tue, 27 Mar 2018, 12:41 Gagandeep Singh, wrote: > ticket #6675 the distortion in the bottom 8 pixels fixed > --- > libavcodec/cfhd.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c > index

[FFmpeg-devel] [PATCH] lavc/cfhd: debugged padding issue for last 8 pixels distortion

2018-03-27 Thread Gagandeep Singh
fate reference has also been updated and can be tested that the patched output is better for the sample cfhd_odd.mov in fate reference. fixes ticket #6675. --- libavcodec/cfhd.c | 5 +++-- tests/ref/fate/cfhd-3 | 20 ++-- 2 files changed, 13 insertions(+), 12 deletions(-)

[FFmpeg-devel] [PATCH 1/2] lavc/cfhd: error due to improper allocation of height in buffers

2018-03-27 Thread Gagandeep Singh
ticket #6675 the distortion in the bottom 8 pixels fixed --- libavcodec/cfhd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index e35732df45..f10742f4fa 100644 --- a/libavcodec/cfhd.c +++ b/libavcodec/cfhd.c @@ -213,13 +213,14 @@