Re: [FFmpeg-devel] [PATCH 2/5] avcodec/vc1: fix mquant calculation for interlace field pictures

2018-05-20 Thread Carl Eugen Hoyos
2018-05-20 0:01 GMT+02:00, Michael Niedermayer : > On Fri, May 18, 2018 at 05:06:23PM +0200, Jerome Borsboom wrote: >> For interlace field pictures s->mb_height indicates the height of the >> full >> picture in MBs, i.e. the two fields combined. A single field is half this >> size. When calculating

Re: [FFmpeg-devel] [PATCH] avutil/error: document av_err2str

2018-05-20 Thread Tomas Härdin
tor 2018-05-17 klockan 12:19 +0200 skrev Tobias Rapp: > On 17.05.2018 11:24, Zhao Zhili wrote: > > --- > >   libavutil/error.h | 3 +++ > >   1 file changed, 3 insertions(+) > > > > diff --git a/libavutil/error.h b/libavutil/error.h > > index 71df4da..b357bfa 100644 > > --- a/libavutil/error.h > >

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/vc1: store zero MVs for all blocks in a MB

2018-05-20 Thread Carl Eugen Hoyos
2018-05-20 3:45 GMT+02:00, James Almer : > I see no emms in either ff_clear_blocks_mmx() or anywhere in > vc1_block.c, so either nobody tried this decoder on machines > without sse I do have access to an SSE-only system, I doubt FFmpeg was used or tested on MMX-only systems in a very, very long t

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/vc1: FIELDTX is only coded raw in interlaced frame I pictures

2018-05-20 Thread Jerome Borsboom
>> libavcodec/vc1_block.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/vc1_block.c b/libavcodec/vc1_block.c >> index f59c440943..daf30fdbfe 100644 >> --- a/libavcodec/vc1_block.c >> +++ b/libavcodec/vc1_block.c >> @@ -2680,7 +2680,7 @@ static void vc1_dec

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/vc1: fix mquant calculation for interlace field pictures

2018-05-20 Thread Jerome Borsboom
>> On Fri, May 18, 2018 at 05:06:23PM +0200, Jerome Borsboom wrote: >>> For interlace field pictures s->mb_height indicates the height of the >>> full >>> picture in MBs, i.e. the two fields combined. A single field is half this >>> size. When calculating mquant for interlace field pictures, the bo

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/vc1: store zero MVs for all blocks in a MB

2018-05-20 Thread Jerome Borsboom
>> Direct prediction for interlace frame B pictures references the mv in the >> second block in an MB in the backward reference frame for the twomv case. >> When the backward reference frame is an I frame, this value may be unset. >> >> Signed-off-by: Jerome Borsboom >> --- >> libavcodec/vc1_blo

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/vc1: fix mquant calculation for interlace field pictures

2018-05-20 Thread Carl Eugen Hoyos
2018-05-20 11:00 GMT+02:00, Jerome Borsboom : >>> for patch 1 you list a file that it improves. >> >> The file - afaict - is related to the patchset as a whole. >> The current version of the patchset does not fix the >> file though, so I believe we should wait for a new >> version of the patchset.

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Carl Eugen Hoyos
2018-05-19 23:48 GMT+02:00, James Almer : > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: >> On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: >>> From: Aman Gupta >>> >>> These fields will allow the mpegts demuxer to expose details about >>> the PMT/program which created the AVProgra

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/vc1: FIELDTX is only coded raw in interlaced frame I pictures

2018-05-20 Thread Michael Niedermayer
On Sun, May 20, 2018 at 10:56:19AM +0200, Jerome Borsboom wrote: > >> libavcodec/vc1_block.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/libavcodec/vc1_block.c b/libavcodec/vc1_block.c > >> index f59c440943..daf30fdbfe 100644 > >> --- a/libavcodec/vc1_block.c

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Jan Ekström
On Sun, May 20, 2018 at 1:54 PM, Carl Eugen Hoyos wrote: > > Reading the original submission above, isn't the whole point of the > patch to add public symbols to help downstream with real-world > issues? > > Carl Eugen Yes, currently the API user has no visibility over how the programs are being

Re: [FFmpeg-devel] [PATCH v2 1/5] avcodec/vc1: FIELDTX is only present in interlaced frame I/BI pictures

2018-05-20 Thread Jerome Borsboom
If v->fieldtx_is_raw is not reset to zero, it may spill over from a previous interlaced frame I/BI picture. Signed-off-by: Jerome Borsboom --- This may address the concerns. Will make a mental note to clean up the parser at a later time. Thank you for the review. libavcodec/vc1.c | 3 ++- 1 fi

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Jan Ekström
On Sun, May 20, 2018 at 2:06 PM, Jan Ekström wrote: > On Sun, May 20, 2018 at 1:54 PM, Carl Eugen Hoyos wrote: >> >> Reading the original submission above, isn't the whole point of the >> patch to add public symbols to help downstream with real-world >> issues? >> >> Carl Eugen > > Yes, currently

Re: [FFmpeg-devel] HLS Questions

2018-05-20 Thread Ronak Patel
No I’ll give it a try and let you know. Since this is just audio (no keyframes) I wouldn’t have thought this would have mattered. Sent from my iPhone > On May 7, 2018, at 3:52 PM, Aman Gupta wrote: > > On Wed, May 2, 2018 at 7:59 AM, Ronak Patel < > ronak2121-at-yahoo@ffmpeg.org> wrote:

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Aman Gupta
On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: > > On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: > >> From: Aman Gupta > >> > >> These fields will allow the mpegts demuxer to expose details about > >> the PMT/program which crea

[FFmpeg-devel] [GSoC] FFserver config file support

2018-05-20 Thread Stephan Holljes
This patch set builds upon the previous set that implemented ffserver with a more flexible httpd interface. It includes a fix for timestamp handling, which sadly still does not properly fix the issue. The majority of this patch set adds the ability to read lua-config files. After reading up on how

[FFmpeg-devel] [PATCH 5/8] publisher.h: Add stream_name to PublisherContext

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- publisher.c | 3 ++- publisher.h | 4 +++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/publisher.c b/publisher.c index 1123056..2e96f2f 100644 --- a/publisher.c +++ b/publisher.c @@ -93,11 +93,12 @@ void client_push_segment(struct Client *c, s

[FFmpeg-devel] [PATCH 6/8] ffserver.c: Add config file reading

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- ffserver.c | 248 ++--- 1 file changed, 172 insertions(+), 76 deletions(-) diff --git a/ffserver.c b/ffserver.c index 44fc263..3d842d8 100644 --- a/ffserver.c +++ b/ffserver.c @@ -38,6 +38,7 @@ #include "

[FFmpeg-devel] [PATCH 1/8] ffserver.c: Different timestamp scaling, still does not properly work with mp4 files?

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- ffserver.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/ffserver.c b/ffserver.c index 39e1c32..bd7c694 100644 --- a/ffserver.c +++ b/ffserver.c @@ -127,13 +127,13 @@ void *read_thread(void *arg) pkt.dts =

[FFmpeg-devel] [PATCH 7/8] Makefile: Update Makefile

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- Makefile | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index fbecdeb..83bc4e0 100644 --- a/Makefile +++ b/Makefile @@ -1,10 +1,11 @@ all: ffserver LAV_FLAGS = $(shell pkg-config --libs --cflags libavformat libav

[FFmpeg-devel] [PATCH 3/8] ffserver.c: rename ReadInfo.in_filename to ReadInfo.in_uri

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- ffserver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffserver.c b/ffserver.c index bd7c694..44fc263 100644 --- a/ffserver.c +++ b/ffserver.c @@ -45,7 +45,7 @@ struct ReadInfo { struct PublisherContext *pub; AVFormatContext *i

[FFmpeg-devel] [PATCH 4/8] httpd.h: Adapt structs to config file

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- httpd.h | 17 + 1 file changed, 17 insertions(+) diff --git a/httpd.h b/httpd.h index 6fb91bd..25cbe11 100644 --- a/httpd.h +++ b/httpd.h @@ -26,11 +26,28 @@ #include "publisher.h" +/* Supported stream formats, for now only matroska */ +enu

[FFmpeg-devel] [PATCH 8/8] doc: Update Documentation.txt and add sample config as supplement

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- Documentation.txt | 17 - sample_config.lua | 28 2 files changed, 36 insertions(+), 9 deletions(-) create mode 100644 sample_config.lua diff --git a/Documentation.txt b/Documentation.txt index 9a7f0bf..c8fef11 1006

[FFmpeg-devel] [PATCH 2/8] ffserver: Implement lua config file reader

2018-05-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- configreader.c | 211 + configreader.h | 46 + 2 files changed, 257 insertions(+) create mode 100644 configreader.c create mode 100644 configreader.h diff --git a/configreader.c b/configread

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread James Almer
On 5/20/2018 3:37 PM, Aman Gupta wrote: > On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > >> On 5/19/2018 6:31 PM, Michael Niedermayer wrote: >>> On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: From: Aman Gupta These fields will allow the mpegts demuxer to expose

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Michael Niedermayer
On Sun, May 20, 2018 at 11:37:22AM -0700, Aman Gupta wrote: > On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > > > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: > > > On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: > > >> From: Aman Gupta > > >> > > >> These fields will allow

Re: [FFmpeg-devel] [PATCH 2/8] ffserver: Implement lua config file reader

2018-05-20 Thread Michael Niedermayer
On Sun, May 20, 2018 at 08:53:58PM +0200, Stephan Holljes wrote: > Signed-off-by: Stephan Holljes > --- > configreader.c | 211 > + > configreader.h | 46 + > 2 files changed, 257 insertions(+) > create mode 100644 configreade

Re: [FFmpeg-devel] [PATCH 6/8] ffserver.c: Add config file reading

2018-05-20 Thread Michael Niedermayer
On Sun, May 20, 2018 at 08:54:02PM +0200, Stephan Holljes wrote: > Signed-off-by: Stephan Holljes > --- > ffserver.c | 248 > ++--- > 1 file changed, 172 insertions(+), 76 deletions(-) > > diff --git a/ffserver.c b/ffserver.c > index 44fc2

Re: [FFmpeg-devel] [PATCH v2 1/5] avcodec/vc1: FIELDTX is only present in interlaced frame I/BI pictures

2018-05-20 Thread Michael Niedermayer
On Sun, May 20, 2018 at 01:45:56PM +0200, Jerome Borsboom wrote: > If v->fieldtx_is_raw is not reset to zero, it may spill over from a previous > interlaced frame I/BI picture. > > Signed-off-by: Jerome Borsboom > --- > This may address the concerns. Will make a mental note to clean up the parser

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mov: Only fail for STCO/STSC contradictions if both exist

2018-05-20 Thread Michael Niedermayer
On Tue, May 15, 2018 at 06:53:49PM +0200, Carl Eugen Hoyos wrote: > 2018-05-15 17:07 GMT+02:00, Michael Niedermayer : > > Fixes regression with playback of > > GF9720Repeal20the20Eighth20with20Helen20Linehan.m4a > > See: crbug 822666 > > Please mention ticket #7165 if related. its unrelated, ill

[FFmpeg-devel] [PATCH] avformat/mov: replace a value error by clipping into valid range in mov_read_stsc()

2018-05-20 Thread Michael Niedermayer
Fixes: #7165 Signed-off-by: Michael Niedermayer --- libavformat/mov.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index a078bf4712..f2a540ad50 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2642,14 +2642,22

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

2018-05-20 Thread Łukasz Krzciuk
Any updates on this issue? Regards, *Łukasz Krzciuk* Developer Vewd ul. Grabarska 2, Pegaz 2A, 50-079 Wrocław, Polska On Tue, May 15, 2018 at 12:51 PM, Łukasz Krzciuk wrote: > I have prepared a testcase. PMT data comes from org.hbbtv_HTML50420 - I > have squashed them and there is one file a

[FFmpeg-devel] [PATCH V2] lavc/qsvenc: add an option to disable MFE mode

2018-05-20 Thread Zhong Li
Not convenient if using numerals to set MFE mode. It is ambiguous and misleading (e.g: user may misunderstand setting mfmode to 1 is to enable MFE but actually it is to disable MFE, and set it to be 5 or above is meaningless). V2: remove the manual option since it is not supported now. Signed-of

[FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-20 Thread Ruiling Song
This filter does HDR(HDR10/HLG) to SDR conversion with tone-mapping. An example command to use this filter with vaapi codecs: FFMPEG -init_hw_device vaapi=va:/dev/dri/renderD128 -init_hw_device \ opencl=ocl@va -hwaccel vaapi -hwaccel_device va -hwaccel_output_format \ vaapi -i INPUT -filter_hw_dev

[FFmpeg-devel] [PATCH] lavfi: a minor fix to tonemap peak detection.

2018-05-20 Thread Ruiling Song
If the transfer was SMPTE2084, use the peak of 1 even if not tagged. Otherwise, we would assume it is HLG with a peak of 1200. Based on suggestion by Niklas Haas. Signed-off-by: Ruiling Song --- libavfilter/vf_tonemap.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a