Re: [FFmpeg-devel] [PATCH] lavc/vc1dec: add multi-slice decoding support for hwaccel.

2016-11-11 Thread Hendrik Leppkes
On Fri, Nov 11, 2016 at 9:09 AM, Jun Zhao wrote: > Do you have a sample file for this case? AFAIK all vc1 files I ever saw worked with the DXVA2 hwaccel before, just want to make sure they are not getting broken. - Hendrik ___

Re: [FFmpeg-devel] [PATCH] increment initial_prog_date_time when removing old segments

2016-11-11 Thread Steven Liu
2016-11-11 17:13 GMT+08:00 Robert Nagy : > Fixes https://trac.ffmpeg.org/ticket/5940#ticket > > --- > libavformat/hlsenc.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > index 9ca2df7..80a598a 100644 > ---

[FFmpeg-devel] [PATCH] increment initial_prog_date_time when removing old segments

2016-11-11 Thread Robert Nagy
Fixes https://trac.ffmpeg.org/ticket/5940#ticket --- libavformat/hlsenc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 9ca2df7..80a598a 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -183,6 +183,7 @@ static int

Re: [FFmpeg-devel] [Patch] hwaccel transocode broken

2016-11-11 Thread Hendrik Leppkes
On Fri, Nov 11, 2016 at 6:34 AM, Yogender Gupta wrote: >>>I was trying libnpp on Big Bunny video by below command: > > > >>>./ffmpeg -loglevel debug -y -hwaccel cuvid -c:v h264_cuvid -vsync 0 -i > >>>/root/root/bunny.mp4 -vf scale_npp=1920:1072 -vcodec h264_nvenc >

[FFmpeg-devel] [PATCH] lavc/vc1dec: add multi-slice decoding support for hwaccel.

2016-11-11 Thread Jun Zhao
From 95eebc4d94a2f2db9f03e569b660d94ae083d26c Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Fri, 11 Nov 2016 16:05:57 +0800 Subject: [PATCH] lavc/vc1dec: add multi-slice decoding support for hwaccel. add multi-slice decoding support for hwaccel, now only test with vaapi as

Re: [FFmpeg-devel] [Patch] hwaccel transocode broken

2016-11-11 Thread Yogender Gupta
>> ffmpeg.c is in the middle of some changes being merged, and once that is >> done ffmpeg_cuvid.c will need to be adapted to a new system. Until then, it >> would make everything easier if the code actively >> being modified by these >> merges is not modified by hackish fixes. When do we

Re: [FFmpeg-devel] [Patch] hwaccel transocode broken

2016-11-11 Thread Carl Eugen Hoyos
2016-11-11 9:03 GMT+01:00 Hendrik Leppkes : > On Fri, Nov 11, 2016 at 6:34 AM, Yogender Gupta wrote: I was trying libnpp on Big Bunny video by below command: >> >> >> ./ffmpeg -loglevel debug -y -hwaccel cuvid -c:v h264_cuvid -vsync 0 -i >>

Re: [FFmpeg-devel] [PATCH] feature: xcbgrab single window

2016-11-11 Thread Nicolas George
Thanks for the patch. See comments below. > Subject: Re: [FFmpeg-devel] [PATCH] feature: xcbgrab single window The recommended syntax would be "lavd/xcbgrab: add single window mode". > Allows to grab only a single window by using syntax: > ffmpeg -f x11grab -r 15 -i ":0/0x0580001b/parent2" ...

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread Clément Bœsch
(sorry for initially replying in private by mistake) On Thu, Nov 03, 2016 at 07:36:29PM +0100, Nicolas George wrote: > Le duodi 12 brumaire, an CCXXV, Clement Boesch a écrit : > > --- > > So this is just a prototype that is starting to work. It needs a split > > (and completion of the long TODO

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread Nicolas George
Le septidi 17 brumaire, an CCXXV, Clement Boesch a écrit : > I didn't. Duration is somehow broken currently for some reason. I did > nothing for sparseness: the reason I added basic support in lavfi is > because it was much simpler to handle at ffmpeg.c level, so it's currently > just a

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread Nicolas George
Le primidi 21 brumaire, an CCXXV, wm4 a écrit : > Sounds like a bad idea. Do you have a better one? signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread wm4
On Fri, 11 Nov 2016 12:49:31 +0100 Nicolas George wrote: > Le primidi 21 brumaire, an CCXXV, wm4 a écrit : > > Sounds like a bad idea. > > Do you have a better one? No, but I'm confident that you can find one after having thought about the problem for so long.

Re: [FFmpeg-devel] [Patch] hwaccel transocode broken

2016-11-11 Thread wm4
On Fri, 11 Nov 2016 10:19:08 +0100 Carl Eugen Hoyos wrote: > 2016-11-11 9:03 GMT+01:00 Hendrik Leppkes : > > On Fri, Nov 11, 2016 at 6:34 AM, Yogender Gupta wrote: > I was trying libnpp on Big Bunny video by below command: > >>

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread wm4
On Fri, 11 Nov 2016 12:45:03 +0100 Nicolas George wrote: > Le septidi 17 brumaire, an CCXXV, Clement Boesch a écrit : > > I didn't. Duration is somehow broken currently for some reason. I did > > nothing for sparseness: the reason I added basic support in lavfi is > > because it

Re: [FFmpeg-devel] [Patch] hwaccel transocode broken

2016-11-11 Thread Yogender Gupta
>>These are merges. Unless you volunteer to do the merges yourself (and >>properly please) you'll have to live with this. So were there merges that got left and broke the functionality. When do we get these merges in ? Please let me know if I can help get this up and fixed. There have been

Re: [FFmpeg-devel] [PATCH] lavc/vaapi: Add VP8 decode hwaccel

2016-11-11 Thread Timo Rothenpieler
This patch seems incomplete. Did you forget to add the vaapi_vp8.c file before commiting? Also, adding a new field to the API which is deprecated from the start seems odd. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread Nicolas George
Le primidi 21 brumaire, an CCXXV, wm4 a écrit : > No, but I'm confident that you can find one after having thought about > the problem for so long. Well, I have thought about it for long, and not only is this the solution I have found, but I do not think it is a "bad idea" at all. And I would

Re: [FFmpeg-devel] [PATCH]lavc/dpx: Support gray12

2016-11-11 Thread Michael Niedermayer
On Thu, Nov 10, 2016 at 11:08:23PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Not sure how useful this is but we also support gprp12. > > Please comment, Carl Eugen > dpx.c | 12 > 1 file changed, 8 insertions(+), 4 deletions(-) > ce5928daf60172d4d2d16a39ad12ef1b2a2e7a0a >

Re: [FFmpeg-devel] [PATCH] increment initial_prog_date_time when removing old segments

2016-11-11 Thread Michael Niedermayer
On Fri, Nov 11, 2016 at 10:13:43AM +0100, Robert Nagy wrote: > Fixes https://trac.ffmpeg.org/ticket/5940#ticket > > --- > libavformat/hlsenc.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > index 9ca2df7..80a598a 100644 > ---

Re: [FFmpeg-devel] [PATCH] Make the process of uuid-xmp atom faster.

2016-11-11 Thread Michael Niedermayer
On Fri, Nov 11, 2016 at 11:42:26AM +0800, Chen Meng wrote: > Ya. It’s really annoying everyone using patchwork. (So lng text.) I’m > trying to fix it. > Please use my name, Chen Meng, if nothing changed in a short time. > > 2016-11-11 10:36 GMT+08:00 Chen Meng :

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread wm4
On Fri, 11 Nov 2016 15:36:04 +0100 Nicolas George wrote: > > It seems to me that not libavfilter should handle this, but the one > > which feeds libavfilter. > > Its collaboration is mandatory. But the bulk of the work should be done > by the library, not duplicated in all

Re: [FFmpeg-devel] [PATCH] increment initial_prog_date_time when removing old segments

2016-11-11 Thread Steven Liu
2016-11-11 21:40 GMT+08:00 Michael Niedermayer : > On Fri, Nov 11, 2016 at 10:13:43AM +0100, Robert Nagy wrote: > > Fixes https://trac.ffmpeg.org/ticket/5940#ticket > > > > --- > > libavformat/hlsenc.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread wm4
On Fri, 11 Nov 2016 13:45:54 +0100 Nicolas George wrote: > Le primidi 21 brumaire, an CCXXV, wm4 a écrit : > > No, but I'm confident that you can find one after having thought about > > the problem for so long. > > Well, I have thought about it for long, and not only is this

Re: [FFmpeg-devel] [PATCH]lavc/dpx: Support gray12

2016-11-11 Thread Kieran O Leary
Hi Carl, On Fri, Nov 11, 2016 at 1:31 PM, Michael Niedermayer wrote: > On Thu, Nov 10, 2016 at 11:08:23PM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Not sure how useful this is but we also support gprp12. > > > > Please comment, Carl Eugen > I'm sure that this

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: fix forced_idr logic

2016-11-11 Thread Michael Niedermayer
On Thu, Oct 13, 2016 at 05:32:54PM +0200, Michael Niedermayer wrote: > On Wed, Oct 12, 2016 at 09:57:16PM +0200, Timo Rothenpieler wrote: > > Currently, it forces IDR frames for both true and false. > > Not entirely sure what the original idea behind the tri-state bool > > option is. > > --- > >

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread Nicolas George
Le primidi 21 brumaire, an CCXXV, wm4 a écrit : > OK, let's think about alternative approaches. How about not trying to > let libavfilter do synchronizing subtitles and other streams in all > cases? Why not just send nothing to the subtitle buffer sources if the > subtitles are sparse and there is

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread Nicolas George
Le primidi 21 brumaire, an CCXXV, wm4 a écrit : > I'm saying that a good solution is application-specific. It depends on > the exact data flow depending on what kind of filter graph is used. The library should still try to make things as simple as possible. > libavfilter still doesn't know

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: fix forced_idr logic

2016-11-11 Thread Derek Buitenhuis
On 11/11/2016 2:30 PM, Michael Niedermayer wrote: > if noone adds suport fo the 3rd constant then the patch should > likely be applied, unless i miss something This patch LGTM. When I added this option in c981b1145a857c8f962c93b8eecb1c613b20ffe9, the type was INT, and the default was -1, and it

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: fix forced_idr logic

2016-11-11 Thread Derek Buitenhuis
On 11/11/2016 3:26 PM, Derek Buitenhuis wrote: > This patch LGTM. > > When I added this option in c981b1145a857c8f962c93b8eecb1c613b20ffe9, the > type was > INT, and the default was -1, and it was correct. It was later broken in > fb99ef0bd39a1859d0e65c6c16caa8e17dd3cfbe. I suggest looking at

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

2016-11-11 Thread wm4
On Wed, 2 Nov 2016 23:09:34 +0100 Clément Bœsch wrote: > --- > So this is just a prototype that is starting to work. It needs a split > (and completion of the long TODO list) before review but I wanted to > share it for early feedbacks. > > So long story short: AVSubtitle →

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-11 Thread wm4
On Fri, 5 Aug 2016 17:18:39 -0700 Sasi Inguva wrote: > In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit Found a file that does not show video at all with edit lists enabled: https://0x0.st/2Ep.mp4

Re: [FFmpeg-devel] [PATCH] increment initial_prog_date_time when removing old segments

2016-11-11 Thread Michael Niedermayer
On Fri, Nov 11, 2016 at 09:59:16PM +0800, Steven Liu wrote: > 2016-11-11 21:58 GMT+08:00 Steven Liu : > > > > > > > 2016-11-11 21:40 GMT+08:00 Michael Niedermayer : > > > >> On Fri, Nov 11, 2016 at 10:13:43AM +0100, Robert Nagy wrote: > >> > Fixes

[FFmpeg-devel] [PATCH 1/3] lavu: Add AVSphericalMapping type and frame side data

2016-11-11 Thread Vittorio Giovara
While no decoder currently exports spherical information, this type represents a frame property that has to be passed through from container to frames. Signed-off-by: Vittorio Giovara --- Missing version bump (do so when pushing). Please CC. Vittorio

Re: [FFmpeg-devel] [PATCH] smvjpegdec: make sure cur_frame is not negative

2016-11-11 Thread Andreas Cadhalpun
On 11.11.2016 01:58, Michael Niedermayer wrote: > On Thu, Nov 10, 2016 at 10:14:04PM +0100, Andreas Cadhalpun wrote: >> This fixes a heap-buffer-overflow detected by AddressSanitizer. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/smvjpegdec.c | 4

Re: [FFmpeg-devel] [PATCH] pnm: limit maxval to UINT16_MAX

2016-11-11 Thread Andreas Cadhalpun
On 11.11.2016 01:17, Michael Niedermayer wrote: > On Thu, Nov 10, 2016 at 08:54:37PM +0100, Andreas Cadhalpun wrote: >> From 'man ppm': The maximum color value (Maxval), again in ASCII decimal. >> Must be less than 65536. >> >> Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] pnmdec: make sure v is capped by maxval

2016-11-11 Thread Andreas Cadhalpun
On 11.11.2016 02:03, Michael Niedermayer wrote: > On Thu, Nov 10, 2016 at 08:52:29PM +0100, Andreas Cadhalpun wrote: >> pnmdec.c | 10 +++--- >> 1 file changed, 7 insertions(+), 3 deletions(-) >> 0dd61abddc422dd2ac37356f8271822d7e801b8e >> 0001-pnmdec-make-sure-v-is-capped-by-maxval.patch

[FFmpeg-devel] [PATCH] configure: properly add dxva2 link dependencies

2016-11-11 Thread Hendrik Leppkes
Fixes building with --disable-everything --enable-shared --enable-dxva2 The hwcontext DXVA2 implementation in avutil needs this library now, instead of just the ffmpeg program. --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index

[FFmpeg-devel] [PATCH 2/2] lavf/fifo: fix undefined behaviour in deinit when destroying mutex

2016-11-11 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/fifo.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/fifo.c b/libavformat/fifo.c index 15435fe..8f525e5 100644 --- a/libavformat/fifo.c +++ b/libavformat/fifo.c @@ -73,6 +73,7 @@ typedef struct

[FFmpeg-devel] [PATCH 1/2] ffmpeg: add newline to avformat_write_header error message

2016-11-11 Thread Marton Balint
Signed-off-by: Marton Balint --- ffmpeg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index 067ef1d..37f4fec 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -2800,7 +2800,7 @@ static int check_init_output_file(OutputFile *of, int file_index)

Re: [FFmpeg-devel] [PATCHv2 1/3] tools: add loudnorm script example to use loudnorm

2016-11-11 Thread Marton Balint
On Sun, 23 Oct 2016, Marton Balint wrote: Based on a patch by Kyle Swanson . Signed-off-by: Marton Balint --- tools/loudnorm.rb | 61 +++ 1 file changed, 61 insertions(+) create mode 100755 tools/loudnorm.rb

Re: [FFmpeg-devel] [PATCH 2/3] lavfi/loudnorm: add an internal libebur128 library

2016-11-11 Thread Marton Balint
On Thu, 10 Nov 2016, Kyle Swanson wrote: On Thu, Nov 10, 2016 at 4:16 PM, Marton Balint wrote: On Thu, 10 Nov 2016, Kyle Swanson wrote: On Tue, Nov 8, 2016 at 9:39 AM, Kyle Swanson wrote: These patches look good to me. If we're going to do this, we really

[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata. Also updates fate ref for rgb24-mkv and fate-mov-zombie as the output will contain the new metadata field.

2016-11-11 Thread Zhenni Huang
--- libavformat/mov.c | 19 +-- tests/ref/fate/mov-zombie | 2 +- tests/ref/fate/rgb24-mkv | 4 ++-- 3 files changed, 20 insertions(+), 5 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9ec7d03..b547832 100644 --- a/libavformat/mov.c +++

[FFmpeg-devel] [PATCH 2/3] lavc: Add spherical packet side data API

2016-11-11 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- Missing version bump (do so when pushing). Please CC. Vittorio ffprobe.c | 18 ++ libavcodec/avcodec.h | 8 +++- libavcodec/avpacket.c | 1 + libavcodec/utils.c| 1 + libavformat/dump.c|

[FFmpeg-devel] [PATCH 3/3] mov: Export spherical information

2016-11-11 Thread Vittorio Giovara
This implements Spherical Video V1 and V2, as described in the spatial-media collection by Google. Signed-off-by: Vittorio Giovara --- Please CC. Vittorio libavformat/isom.h | 7 ++ libavformat/mov.c | 281 - 2

Re: [FFmpeg-devel] [PATCH 3/3] mov: Export spherical information

2016-11-11 Thread James Almer
On 11/11/2016 7:49 PM, Vittorio Giovara wrote: > This implements Spherical Video V1 and V2, as described in the > spatial-media collection by Google. > > Signed-off-by: Vittorio Giovara > --- > Please CC. > Vittorio > > libavformat/isom.h | 7 ++ >

[FFmpeg-devel] [PATCH] configure: use check_lib2 for cuda and cuvid

2016-11-11 Thread Hendrik Leppkes
Fixes building on Windows with MSVC using the link libraries distributed with the CUDA SDK. check_lib2 is required here because it includes the header to get the full signature of the function, including the stdcall calling convention and all of its arguments, which enabled the linker to

Re: [FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata. Also updates fate ref for rgb24-mkv and fate-mov-zombie as the output will contain the new metadata field.

2016-11-11 Thread Michael Niedermayer
On Fri, Nov 11, 2016 at 04:14:37PM -0800, Zhenni Huang wrote: > --- > libavformat/mov.c | 19 +-- > tests/ref/fate/mov-zombie | 2 +- > tests/ref/fate/rgb24-mkv | 4 ++-- > 3 files changed, 20 insertions(+), 5 deletions(-) > > diff --git a/libavformat/mov.c

Re: [FFmpeg-devel] [PATCH 1/3] lavu: Add AVSphericalMapping type and frame side data

2016-11-11 Thread Michael Niedermayer
On Fri, Nov 11, 2016 at 05:49:00PM -0500, Vittorio Giovara wrote: [...] > +/** > + * This structure describes how to handle spherical videos, outlining > + * information about projection, initial layout, and any other view modifier. > + * > + * @note The struct must be allocated with

[FFmpeg-devel] Problem with vstats file and frame number

2016-11-11 Thread Thierry Foucu
Hi, Before sending you some patches to fix it, I would like to know how you want this issue to be fixed. Let me explain it: When we enable vstats file to be generated, each line of the file contains the frame number of the encoded video frames. The frame number is the one from AVStream.nb_frames

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-11 Thread Sasi Inguva
The above file has an edit list with duration zero . ​ Even quick time player doesn't play the video. Enabling the old code to populate skip_samples by looking at first edit list duration, when -ignore_editlist is true is a good fallback. I'll send a commit soon. On Fri, Nov 11, 2016 at 11:08

Re: [FFmpeg-devel] [PATCH 2/3] lavfi/loudnorm: add an internal libebur128 library

2016-11-11 Thread Kyle Swanson
On Fri, Nov 11, 2016 at 6:12 PM, Marton Balint wrote: > > On Thu, 10 Nov 2016, Kyle Swanson wrote: > >> On Thu, Nov 10, 2016 at 4:16 PM, Marton Balint wrote: >>> >>> >>> On Thu, 10 Nov 2016, Kyle Swanson wrote: >>> On Tue, Nov 8, 2016 at 9:39 AM, Kyle Swanson

[FFmpeg-devel] [PATCH] lavf/mov.c: Set skip_samples according to first edit list, when -ignore_editlist is set.

2016-11-11 Thread Sasi Inguva
Signed-off-by: Sasi Inguva --- libavformat/isom.h | 2 ++ libavformat/mov.c | 22 +- 2 files changed, 23 insertions(+), 1 deletion(-) diff --git a/libavformat/isom.h b/libavformat/isom.h index d684502..6a8a4e3 100644 --- a/libavformat/isom.h +++