Re: [FFmpeg-devel] [PATCH 13/26] vaapi_encode: Add quality level to common options

2018-04-25 Thread Xiang, Haihao
On Sun, 2018-04-22 at 16:29 +0100, Mark Thompson wrote: > This was previously accessible only via AVCodecContext.compression_level > for all encoders except H.264 where it was also a private option. This > makes it an explicit option everywhere. > --- > libavcodec/vaapi_encode.c | 87

Re: [FFmpeg-devel] [PATCH] fix memory leak of parsing dash MPD

2018-04-25 Thread Steven Liu
> On 20 Apr 2018, at 11:33, Steven Liu wrote: > > > >> On 19 Apr 2018, at 20:28, guikunzhi wrote: >> >> Signed-off-by: guikunzhi >> --- >> libavformat/dashdec.c | 14 +++--- >> 1 file changed, 11 insertions(+), 3

[FFmpeg-devel] [PATCH 1/2] ffmpeg: handling copyts use case in forced key frames functionality

2018-04-25 Thread vdixit
From: Vishwanath Dixit Forced key frames creation functionality was assuming the first PTS value to be zero, but, when 'copyts' is enalbed, the first PTS can be any big number. This was eventually forcing all the frames as key frames. To overcome this issue, the actual first

[FFmpeg-devel] [PATCH 2/2] ffmpeg: handling copyts use case in progress report computation

2018-04-25 Thread vdixit
From: Vishwanath Dixit Progress report computation functionality was assuming the first PTS value to be zero, but, when 'copyts' is enalbed, the first PTS can be any big number. This was eventually causing display of weird statistics during execution of an ffmpeg command. To

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Aman Gupta
On Tue, Apr 24, 2018 at 7:08 PM Aman Gupta wrote: > From: Aman Gupta > > Based largely on VLC's modules/codec/telx.c. > > Processes only teletext pages marked as subtitles, so depending > on the stream it might not produce any output. > > Subtitles are rendered

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Aman Gupta
On Wed, Apr 25, 2018 at 6:15 PM Marton Balint wrote: > > > On Thu, 26 Apr 2018, Carl Eugen Hoyos wrote: > > > 2018-04-25 4:07 GMT+02:00, Aman Gupta : > > > >> Processes only teletext pages marked as subtitles, so > >> depending on the stream it might not produce

Re: [FFmpeg-devel] ? Inserting an image in the middle of a video stream

2018-04-25 Thread John P Poet
On Wed, Apr 25, 2018 at 4:25 PM Devin Heitmueller < dheitmuel...@kernellabs.com> wrote: > Hi John, > > On Wed, Apr 25, 2018 at 5:11 PM, John P Poet wrote: > > I am working on a project where I need to insert a repeating image (10 > > seconds) in the middle of a video stream

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Aman Gupta
On Wed, Apr 25, 2018 at 5:30 PM Carl Eugen Hoyos wrote: > 2018-04-25 4:07 GMT+02:00, Aman Gupta : > > > Processes only teletext pages marked as subtitles, so > > depending on the stream it might not produce any output. > > Shouldn't there be at least an

Re: [FFmpeg-devel] [PATCH 01/14] avcodec/vc1: re-implement and expand VC-1 overlap smooting

2018-04-25 Thread James Almer
On 4/23/2018 3:58 PM, Jerome Borsboom wrote: > The existing implementation did overlap smoothing for progressive > frames only. This rewritten version implements overlap smoothing > for all applicable frame types for both progessive and > frame/field-interlace. > > Signed-off-by: Jerome Borsboom

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Carl Eugen Hoyos
2018-04-26 3:15 GMT+02:00, Marton Balint : > > > On Thu, 26 Apr 2018, Carl Eugen Hoyos wrote: > >> 2018-04-25 4:07 GMT+02:00, Aman Gupta : >> >>> Processes only teletext pages marked as subtitles, so >>> depending on the stream it might not produce any output. >>

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Marton Balint
On Thu, 26 Apr 2018, Carl Eugen Hoyos wrote: 2018-04-25 4:07 GMT+02:00, Aman Gupta : Processes only teletext pages marked as subtitles, so depending on the stream it might not produce any output. Shouldn't there be at least an option to output whatever page was requested?

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Carl Eugen Hoyos
2018-04-25 4:07 GMT+02:00, Aman Gupta : > Processes only teletext pages marked as subtitles, so > depending on the stream it might not produce any output. Shouldn't there be at least an option to output whatever page was requested? +1 for not needing an external dependency.

Re: [FFmpeg-devel] avformat/mpegts: set AV_DISPOSITION_DESCRIPTIONS for OIPF cases

2018-04-25 Thread Michael Niedermayer
On Wed, Apr 25, 2018 at 08:17:38AM +0200, Łukasz Krzciuk wrote: > Yes, I have checked it and I implemented it according to OIPF spec: 8.4.2 > AVComponent, audioDescription case (as I wrote in my 1st email). This > implementation is tested by official org.hbbtv_HTML50420 testcase. ok, ill apply it

Re: [FFmpeg-devel] [GSoC] FFserver further development direction

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 11:33:53PM +0200, Stephan Holljes wrote: > Hi all, > > I've discussed this on IRC a bit, but I don't want to exclude those > views that are not present there. > > The consensus seems to be that there are more disadvantages in using > the http server of libavformat than

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Michael Niedermayer
On Wed, Apr 25, 2018 at 09:32:43PM +0200, Marton Balint wrote: > > > On Tue, 24 Apr 2018, Aman Gupta wrote: > > >From: Aman Gupta > > > >Based largely on VLC's modules/codec/telx.c. > > > >Processes only teletext pages marked as subtitles, so depending > >on the stream it might

Re: [FFmpeg-devel] [GSoC] FFserver further development direction

2018-04-25 Thread Kieran Kunhya
On Wed, 25 Apr 2018 at 23:18 Nicolas George wrote: > Josh de Kock (2018-04-25): > > If anything, this should have never been added and a suitable > > external library should have been picked. > > This opinion should have been expressed three years ago. It was decided

[FFmpeg-devel] [PATCH 1/2] avcodec/cbs: create reference counted fragments in ff_cbs_read_extradata()

2018-04-25 Thread James Almer
This way, every CodedBitstreamType->split_fragment() function can safely assume the fragment passed to them will be reference counted, potentially simplifying code. Signed-off-by: James Almer --- libavcodec/cbs.c | 41 - 1 file changed,

[FFmpeg-devel] [PATCH 2/2] avcodec/cbs_mpeg2: use existing buffer reference when splitting fragments

2018-04-25 Thread James Almer
Signed-off-by: James Almer --- libavcodec/cbs_mpeg2.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 94b9591b21..30d2bb6fbf 100644 --- a/libavcodec/cbs_mpeg2.c +++ b/libavcodec/cbs_mpeg2.c @@

Re: [FFmpeg-devel] ? Inserting an image in the middle of a video stream

2018-04-25 Thread Devin Heitmueller
Hi John, On Wed, Apr 25, 2018 at 5:11 PM, John P Poet wrote: > I am working on a project where I need to insert a repeating image (10 > seconds) in the middle of a video stream while also transcoding from one > video codec to another. The images are going to be either png or

Re: [FFmpeg-devel] [PATCH 23/26] lavc/cbs: Add JPEG support

2018-04-25 Thread James Almer
On 4/25/2018 6:31 PM, Mark Thompson wrote: > +static int cbs_jpeg_split_fragment(CodedBitstreamContext *ctx, > + CodedBitstreamFragment *frag, > + int header) > +{ > +AVBufferRef *data_ref; > +uint8_t *data; > +size_t

Re: [FFmpeg-devel] [PATCH 10/26] vaapi_encode: Choose profiles dynamically

2018-04-25 Thread Mark Thompson
On 25/04/18 09:43, Xiang, Haihao wrote:>> Previously there was one fixed choice for each codec (e.g. H.265 -> Main >> profile), and using anything else then required an explicit option from >> the user. This changes to selecting the profile based on the input format >> and the set of profiles

Re: [FFmpeg-devel] [GSoC] FFserver further development direction

2018-04-25 Thread Nicolas George
Josh de Kock (2018-04-25): > If anything, this should have never been added and a suitable > external library should have been picked. This opinion should have been expressed three years ago. It was decided then that lavf deserved a HTTP server. It is done. Regards, -- Nicolas

Re: [FFmpeg-devel] [PATCH 01/26] vaapi_encode_h264: Move options and common structures into context

2018-04-25 Thread Mark Thompson
On 25/04/18 08:02, Xiang, Haihao wrote:>> This will make it easier to support options in common between different >> encoders. It also cleans up some of the field naming. >> --- >> libavcodec/vaapi_encode_h264.c | 228 >> +--- >> - >> 1 file changed, 115

Re: [FFmpeg-devel] [PATCH 02/26] vaapi_encode_h265: Move options and common structures into context

2018-04-25 Thread Mark Thompson
On 25/04/18 08:35, Xiang, Haihao wrote:>> Matching previous commit for H.264. >> --- >> libavcodec/vaapi_encode_h265.c | 114 >> +++- >> - >> 1 file changed, 54 insertions(+), 60 deletions(-) >> >> diff --git a/libavcodec/vaapi_encode_h265.c

Re: [FFmpeg-devel] fate clients

2018-04-25 Thread Michael Niedermayer
On Wed, Apr 25, 2018 at 05:52:38PM -0300, James Almer wrote: > On 4/25/2018 1:32 PM, Michael Niedermayer wrote: > > On Wed, Apr 25, 2018 at 12:07:53AM +0200, Michael Niedermayer wrote: > >> Hi all > >> > >> before anyone notices and asks ... > >> my box with my virtual fate clients on it died

Re: [FFmpeg-devel] [PATCH 11/26] vaapi_encode: Add common options between all encoders

2018-04-25 Thread Mark Thompson
On 25/04/18 02:21, Zhao Jun wrote: > 2018-04-22 23:29 GMT+08:00 Mark Thompson : >> The only common option here is low_power - it was previously supported >> for H.264 only, that specific option is removed. >> --- >> doc/encoders.texi | 14 -- >>

Re: [FFmpeg-devel] [PATCH v4 1/3] avcodec/bitpacked: move ff_get_buffer

2018-04-25 Thread Rostislav Pehlivanov
On 25 April 2018 at 22:22, Patrick Keroulas < patrick.kerou...@savoirfairelinux.com> wrote: > From: Damien Riegel > > ff_get_buffer is used to allocate a buffer to hold frame's content. This > function was called in the function in charge of decoding an

Re: [FFmpeg-devel] [PATCH v4 2/3] avcodec/bitpacked: add interlace support

2018-04-25 Thread Rostislav Pehlivanov
On 25 April 2018 at 22:22, Patrick Keroulas < patrick.kerou...@savoirfairelinux.com> wrote: > From: Damien Riegel > > This codec is already capable of depacking some combinations of pixel > formats and depth as defined in the RFC4175. The only difference

Re: [FFmpeg-devel] [PATCH] lavc/amfenc: DXVA2 textures support implementation by AMF encoder

2018-04-25 Thread Mark Thompson
On 24/04/18 11:51, Alexander Kravchenko wrote: >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of >> Mark Thompson >> Sent: Sunday, April 22, 2018 6:49 PM >> To: FFmpeg development discussions and patches >> Subject:

Re: [FFmpeg-devel] [PATCH] avcodec/cbs_mpeg2: use existing buffer reference if available when splitting fragments

2018-04-25 Thread Mark Thompson
On 24/04/18 23:22, James Almer wrote: > On 4/23/2018 11:17 PM, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavcodec/cbs_mpeg2.c | 15 +-- >> 1 file changed, 9 insertions(+), 6 deletions(-) >> >> diff --git a/libavcodec/cbs_mpeg2.c

[FFmpeg-devel] [PATCH 23/26] lavc/cbs: Add JPEG support

2018-04-25 Thread Mark Thompson
--- On 24/04/18 03:16, James Almer wrote: > On 4/22/2018 12:29 PM, Mark Thompson wrote: >> +static int cbs_jpeg_split_fragment(CodedBitstreamContext *ctx, >> + CodedBitstreamFragment *frag, >> + int header) >> +{ >> +uint8_t

Re: [FFmpeg-devel] [GSoC] FFserver further development direction

2018-04-25 Thread Josh de Kock
On 2018/04/24 22:46, Nicolas George wrote: Stephan Holljes (2018-04-24): The consensus seems to be that there are more disadvantages in using the http server of libavformat than there are advantages. I completely disagree. There is no point in having the HTTP server in libavformat if it

[FFmpeg-devel] [PATCH v4 3/3] avformat/rtpdec_rfc4175: handle interlace format

2018-04-25 Thread Patrick Keroulas
From: Damien Riegel In order to handle the interlaced formats, the demuxer has only a few things to do: - parse the SDP correctly and propagate the information - check the field bit in the RFC4175 header, and pass that information to the decoder In

[FFmpeg-devel] [PATCH v4 2/3] avcodec/bitpacked: add interlace support

2018-04-25 Thread Patrick Keroulas
From: Damien Riegel This codec is already capable of depacking some combinations of pixel formats and depth as defined in the RFC4175. The only difference between progressive and interlace is that either a packet will contain the whole frame, or only a field

[FFmpeg-devel] [PATCH v4 1/3] avcodec/bitpacked: move ff_get_buffer

2018-04-25 Thread Patrick Keroulas
From: Damien Riegel ff_get_buffer is used to allocate a buffer to hold frame's content. This function was called in the function in charge of decoding an AVPacket containing raw video with the yuv422 pixel format and a depth of 10-bit. RFC4175 supports both

[FFmpeg-devel] ? Inserting an image in the middle of a video stream

2018-04-25 Thread John P Poet
Hi all, I am working on a project where I need to insert a repeating image (10 seconds) in the middle of a video stream while also transcoding from one video codec to another. The images are going to be either png or jpg images. I have a frame number within the video stream which tells me where

Re: [FFmpeg-devel] fate clients

2018-04-25 Thread James Almer
On 4/25/2018 1:32 PM, Michael Niedermayer wrote: > On Wed, Apr 25, 2018 at 12:07:53AM +0200, Michael Niedermayer wrote: >> Hi all >> >> before anyone notices and asks ... >> my box with my virtual fate clients on it died today >> my arm clients should still be working though >> >> Ill try to get

Re: [FFmpeg-devel] [PATCH] avcodec/h2645_parse: Replace RNXYA by RNXY in ff_h2645_extract_rbsp()

2018-04-25 Thread Michael Niedermayer
On Sat, Mar 10, 2018 at 09:51:09PM +0100, Michael Niedermayer wrote: > Fixes misaligned accesses > Found-by: Matt Wolenetz > Signed-off-by: Michael Niedermayer > --- > libavcodec/h2645_parse.c | 8 > 1 file changed, 4 insertions(+), 4

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/h263dec: Check slice_ret in mspeg4 slice loop

2018-04-25 Thread Michael Niedermayer
On Sun, Apr 22, 2018 at 10:20:31PM +0200, Michael Niedermayer wrote: > Fixes infinite loop > Fixes: > 6858/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_MSMPEG4V3_fuzzer-4681563766784000 > Fixes: > 6890/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_WMV1_fuzzer-4756103142309888 > > Found-by: continuous

Re: [FFmpeg-devel] [PATCH] avcodec/elsdec: Fix memleaks

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 10:12:30PM -0300, James Almer wrote: > On 4/24/2018 9:58 PM, Michael Niedermayer wrote: > > Fixes: > > 6798/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_G2M_fuzzer-5135899701542912 > > > > Found-by: continuous fuzzing process > >

Re: [FFmpeg-devel] [PATCH] avcodec/cbs_mpeg2: use memcpy when assembling fragments

2018-04-25 Thread James Almer
On 4/25/2018 4:35 PM, Michael Niedermayer wrote: > On Mon, Apr 23, 2018 at 11:39:54PM -0300, James Almer wrote: >> before: >> 419022 decicycles in assemble_fragment,2047 runs, 1 skips >> >> after: >> 104621 decicycles in assemble_fragment,2045 runs, 3 skips >> >> Benched with a 2

Re: [FFmpeg-devel] [PATCH] avcodec/mpeg4_unpack_bframes: cache input packets directly

2018-04-25 Thread James Almer
On 3/21/2018 11:21 AM, James Almer wrote: > Avoids unnecessary allocs+copies and makes the code slightly simpler. > > Signed-off-by: James Almer > --- > libavcodec/mpeg4_unpack_bframes_bsf.c | 55 > +-- > 1 file changed, 20 insertions(+), 35

Re: [FFmpeg-devel] [PATCH] avcodec/cbs_mpeg2: use memcpy when assembling fragments

2018-04-25 Thread Michael Niedermayer
On Mon, Apr 23, 2018 at 11:39:54PM -0300, James Almer wrote: > before: > 419022 decicycles in assemble_fragment,2047 runs, 1 skips > > after: > 104621 decicycles in assemble_fragment,2045 runs, 3 skips > > Benched with a 2 minutes long 720x480 DVD mpeg2 sample. > >

Re: [FFmpeg-devel] [PATCH 1/3] avformat/utils: function to get the formatted ntp time

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 02:46:56PM +0530, vdi...@akamai.com wrote: > From: Vishwanath Dixit > > This utility function creates 64-bit NTP time format as per the RFC > 5905. > A simple explaination of 64-bit NTP time format is here > http://www.beaglesoft.com/Manual/page53.htm >

Re: [FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

2018-04-25 Thread Marton Balint
On Tue, 24 Apr 2018, Aman Gupta wrote: From: Aman Gupta Based largely on VLC's modules/codec/telx.c. Processes only teletext pages marked as subtitles, so depending on the stream it might not produce any output. Subtitles are rendered directly to ASS, with support for

Re: [FFmpeg-devel] av_read_frame() question about blocking

2018-04-25 Thread Nicolas George
Philip Prindeville (2018-04-24): > It looks like it’s internally going to keep making multiple calls to > the read_packet function until it’s acquired enough. > > Is that correct? Yes. There is AVFMT_FLAG_NONBLOCK that would cause demuxers to return AVERROR(EAGAIN) if not enough data is

[FFmpeg-devel] [PATCH] avcodec/vc1: more corrections for AC inverse quantization scaling

2018-04-25 Thread Jerome Borsboom
HALFQP should only be added to the inverse quantizer when the block is coded with PQUANT. When PQUANT is equal to ALTPQUANT, the original test for the addition of HALFQP fails. A negative value for mquant indicates that the value was derived from VOPDQUANT. Fixes #4372 Signed-off-by: Jerome

Re: [FFmpeg-devel] [PATCH 4/5] avutil/pixdesc: add AV_PIX_FMT_FLAG_ALPHA to AV_PIX_FMT_PAL8

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 09:05:00PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/APIchanges| 3 +++ > libavcodec/tests/imgconvert.c | 4 > libavutil/pixdesc.c | 3 +-- > libavutil/pixdesc.h | 8 ++-- >

Re: [FFmpeg-devel] [PATCH 5/5] Use AV_PIX_FMT_FLAG_ALPHA for detecting transparency where nb_components was used

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 09:05:01PM +0200, Marton Balint wrote: > Temporarily keep the old method for ffmpeg_filters.c choose_pix_fmt and > avfiltergraph.c pick_format() until a paletted pixel format without alpha is > introduced. > > Signed-off-by: Marton Balint > --- >

Re: [FFmpeg-devel] [PATCH 3/5] avcodec/hnm4video: fix palette alpha

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 09:04:59PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavcodec/hnm4video.c | 1 + > 1 file changed, 1 insertion(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Observe your

Re: [FFmpeg-devel] [PATCH 1/5] avformat/qtpalette: parse color table according to the QuickTime file format specs

2018-04-25 Thread Michael Niedermayer
On Tue, Apr 24, 2018 at 09:04:57PM +0200, Marton Balint wrote: > The specs says that the the first color component in the color array is > not alpha, but simply 0. > > Fixes 0 alpha of fate-suite/cvid/catfight-cvid-pal8-partial.mov > > Signed-off-by: Marton Balint > --- >

Re: [FFmpeg-devel] fate clients

2018-04-25 Thread Michael Niedermayer
On Wed, Apr 25, 2018 at 12:07:53AM +0200, Michael Niedermayer wrote: > Hi all > > before anyone notices and asks ... > my box with my virtual fate clients on it died today > my arm clients should still be working though > > Ill try to get it up and working ASAP. > > Symptoms for the curious >

Re: [FFmpeg-devel] [PATCH] libavformat: add mbedTLS based TLS

2018-04-25 Thread Rostislav Pehlivanov
On 24 April 2018 at 07:18, Thomas Volkert wrote: > On 23.04.2018 21:33, Rostislav Pehlivanov wrote: > > On 23 April 2018 at 20:16, Thomas Volkert wrote: > > > >> On 23.04.2018 11:27, Thomas Volkert wrote: > >>> On 22.04.2018 20:03, Carl Eugen Hoyos wrote: >

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-04-25 Thread Thomas Mundt
Hi, 2018-04-25 16:24 GMT+02:00 Vasile Toncu : > Hello, > > Here is patch 3. I renamed reinterlace to tinterlace and replaced the > functionality of tinterlace with the one from reinterlace. Please review. > > Thank you, > Vasile > Same as I wrote for your previous

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/mediacodecdec: clarify delay_flush specific code

2018-04-25 Thread Matthieu Bouron
On Tue, Apr 24, 2018 at 01:59:23PM -0700, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/mediacodecdec.c| 24 +++- > libavcodec/mediacodecdec_common.c | 12 > 2 files changed, 15 insertions(+), 21 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/mediacodecdec: refactor pts handling

2018-04-25 Thread Matthieu Bouron
On Tue, Apr 24, 2018 at 01:59:25PM -0700, Aman Gupta wrote: > From: Aman Gupta > > Also fixes a bug where EOS buffer was sent with incorrect > pts when not using surface generation. > --- > libavcodec/mediacodecdec_common.c | 17 ++--- > 1 file changed, 6

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/mediacodecdec: use AV_TIME_BASE_Q

2018-04-25 Thread Matthieu Bouron
On Tue, Apr 24, 2018 at 01:59:24PM -0700, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/mediacodecdec_common.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/mediacodecdec_common.c > b/libavcodec/mediacodecdec_common.c >

Re: [FFmpeg-devel] [PATCH v2] mov: Properly abide by the track's media duration

2018-04-25 Thread Derek Buitenhuis
On 4/23/2018 4:46 PM, Derek Buitenhuis wrote: > The track's media duration from the mdhd atom takes precedence > over both the stts and elst atom for calculating and setting > the track's total duraion. > > Technically, we shouldn't be using the stts atom at all for > calculating stream

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-04-25 Thread Gyan Doshi
On 4/25/2018 7:54 PM, Vasile Toncu wrote: Hello, Here is patch 3. I renamed reinterlace to tinterlace and replaced the functionality of tinterlace with the one from reinterlace. Please review. If any options are new or different from the original tinterlace filter, please add doc entries

Re: [FFmpeg-devel] [PATCH 3/5] reitnerlace - tinterlace-like filter under LGPL

2018-04-25 Thread Thomas Mundt
Hi, 2018-04-23 18:22 GMT+02:00 Vasile Toncu : > Hello, > > Here is patch 3. Please review. > > Thank you, > Vasile > Some fate tests fail when both patches are applied. I also get lots of compiler warnings. Please fix them and, as already said, run fate. All tests must

Re: [FFmpeg-devel] [PATCH 2/5] Added reinterlace filter

2018-04-25 Thread Vasile Toncu
Hello, Here is patch 2. In this patch I added the reinterlace filter to ffmpeg. Please review. Thank you, Vasile >From 71458a6cba548c9d5a559cdadae7a85a7825e98d Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: Tue, 24 Apr 2018 17:46:45 +0300 Subject: [PATCH] Patch2

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-04-25 Thread Vasile Toncu
Hello, Here is patch 3. I renamed reinterlace to tinterlace and replaced the functionality of tinterlace with the one from reinterlace. Please review. Thank you, Vasile >From ee33ed769207a99adb193eece32a5043ff0b6d94 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date:

Re: [FFmpeg-devel] [GSoC] FFserver further development direction

2018-04-25 Thread wm4
On Tue, 24 Apr 2018 23:46:09 +0200 Nicolas George wrote: > Stephan Holljes (2018-04-24): > > The consensus seems to be that there are more disadvantages in using > > the http server of libavformat than there are advantages. > > I completely disagree. There is no point in

Re: [FFmpeg-devel] [PATCH] avcodec/h264_slice: use the new SAR early when setting the decoder

2018-04-25 Thread wm4
On Wed, 25 Apr 2018 09:55:10 +0200 Steve Lhomme wrote: > Le 24/04/2018 à 08:28, Hendrik Leppkes a écrit : > > On Fri, Jan 19, 2018 at 1:01 PM, Steve Lhomme wrote: > >> If we don't do that get_format might not be called for a while and the > >> proper > >>

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-25 Thread Nicolas George
Paul B Mahol (2018-04-25): > Looks like interest is very low... Obviously you think it is interesting enough to post several mails about it. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-25 Thread Paul B Mahol
On 4/25/18, Tomas Haerdin wrote: > ons 2018-04-25 klockan 09:55 +0100 skrev Josh de Kock: >> On 2018/04/25 9:35, Paul B Mahol wrote: >> > On 4/25/18, Tomas Haerdin wrote: >> > > [...] >> > > >> > > I'll push this this some time later this week if I don't

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-25 Thread Tomas Härdin
ons 2018-04-25 klockan 09:55 +0100 skrev Josh de Kock: > On 2018/04/25 9:35, Paul B Mahol wrote: > > On 4/25/18, Tomas Haerdin wrote: > > > [...] > > > > > > I'll push this this some time later this week if I don't hear any > > > objections > > > > What is point of it if

Re: [FFmpeg-devel] [PATCH] avcodec/h264_slice: use the new SAR early when setting the decoder

2018-04-25 Thread Hendrik Leppkes
On Wed, Apr 25, 2018 at 9:55 AM, Steve Lhomme wrote: > Le 24/04/2018 à 08:28, Hendrik Leppkes a écrit : >> >> On Fri, Jan 19, 2018 at 1:01 PM, Steve Lhomme wrote: >>> >>> If we don't do that get_format might not be called for a while and the >>> proper >>> SAR

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-25 Thread Josh de Kock
On 2018/04/25 9:35, Paul B Mahol wrote: On 4/25/18, Tomas Haerdin wrote: [...] I'll push this this some time later this week if I don't hear any objections What is point of it if there is only one nickname? There are two nicknames. -- Josh

Re: [FFmpeg-devel] [PATCH 10/26] vaapi_encode: Choose profiles dynamically

2018-04-25 Thread Xiang, Haihao
> Previously there was one fixed choice for each codec (e.g. H.265 -> Main > profile), and using anything else then required an explicit option from > the user. This changes to selecting the profile based on the input format > and the set of profiles actually supported by the driver (e.g. P010

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-25 Thread Paul B Mahol
On 4/25/18, Tomas Haerdin wrote: > tor 2018-04-12 klockan 16:54 +0200 skrev Tomas Haerdin: >> tor 2018-04-12 klockan 16:47 +0200 skrev wm4: >> > On Thu, 12 Apr 2018 16:43:48 +0200 >> > Tomas Haerdin wrote: >> > >> > > soen 2018-04-08 klockan 09:20 -0800

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-25 Thread Tomas Härdin
tor 2018-04-12 klockan 16:54 +0200 skrev Tomas Härdin: > tor 2018-04-12 klockan 16:47 +0200 skrev wm4: > > On Thu, 12 Apr 2018 16:43:48 +0200 > > Tomas Härdin wrote: > > > > > sön 2018-04-08 klockan 09:20 -0800 skrev Lou Logan: > > > > On Sun, Apr 8, 2018, at 3:05 AM, Tomas

Re: [FFmpeg-devel] [PATCH] make swresample optional for ffmpeg

2018-04-25 Thread Paul B Mahol
On 4/16/18, Paul B Mahol wrote: > On 4/12/18, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> configure| 2 +- >> fftools/ffmpeg_opt.c | 2 ++ >> 2 files changed, 3 insertions(+), 1 deletion(-) >> > > ping > Will

Re: [FFmpeg-devel] [PATCH] avcodec/h264_slice: use the new SAR early when setting the decoder

2018-04-25 Thread Steve Lhomme
Le 24/04/2018 à 08:28, Hendrik Leppkes a écrit : On Fri, Jan 19, 2018 at 1:01 PM, Steve Lhomme wrote: If we don't do that get_format might not be called for a while and the proper SAR not used. See the sample mentioned here: https://trac.videolan.org/vlc/ticket/19435 ---

Re: [FFmpeg-devel] [PATCH 04/26] vaapi_encode_mpeg2: Move common structure into context

2018-04-25 Thread Xiang, Haihao
> --- > libavcodec/vaapi_encode_mpeg2.c | 50 -- > --- > 1 file changed, 24 insertions(+), 26 deletions(-) > > diff --git a/libavcodec/vaapi_encode_mpeg2.c b/libavcodec/vaapi_encode_mpeg2.c > index 42df77ea49..5dd1f39cbc 100644 > ---

Re: [FFmpeg-devel] [PATCH 02/26] vaapi_encode_h265: Move options and common structures into context

2018-04-25 Thread Xiang, Haihao
> Matching previous commit for H.264. > --- > libavcodec/vaapi_encode_h265.c | 114 +++- > - > 1 file changed, 54 insertions(+), 60 deletions(-) > > diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c > index 5203c6871d..ba6b426eed

Re: [FFmpeg-devel] [PATCH 5/5] amfenc: Remove spurious initialisations

2018-04-25 Thread Alexander Kravchenko
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Rostislav Pehlivanov > Sent: Tuesday, April 24, 2018 4:04 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 5/5] amfenc:

Re: [FFmpeg-devel] [PATCH 01/26] vaapi_encode_h264: Move options and common structures into context

2018-04-25 Thread Xiang, Haihao
> This will make it easier to support options in common between different > encoders. It also cleans up some of the field naming. > --- > libavcodec/vaapi_encode_h264.c | 228 +--- > - > 1 file changed, 115 insertions(+), 113 deletions(-) > > diff --git

Re: [FFmpeg-devel] avformat/mpegts: set AV_DISPOSITION_DESCRIPTIONS for OIPF cases

2018-04-25 Thread Łukasz Krzciuk
Yes, I have checked it and I implemented it according to OIPF spec: 8.4.2 AVComponent, audioDescription case (as I wrote in my 1st email). This implementation is tested by official org.hbbtv_HTML50420 testcase. Regards, *Łukasz Krzciuk* Developer Vewd ul. Grabarska 2, Pegaz 2A, 50-079 Wrocław,