Re: [FFmpeg-devel] [PATCH 1/2] libavutil/thread.h: Support OS/2 threads

2014-11-29 Thread Michael Niedermayer
On Fri, Nov 28, 2014 at 11:48:01PM -0800, Dave Yeo wrote: Support OS/2 threading and correct an include that shouldn't be system. Dave both patches applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have often repented speaking, but never of

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Nicolas George
Le septidi 7 frimaire, an CCXXIII, Timo Rothenpieler a écrit : Done that primarily to keep things cleaned up and easier to read. Can as well put it all in one huge file. IMHO, your choice in the end. Propably, will split that out when i get to it. Thanks. Most of this code is ported from a

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Michael Niedermayer
On Sat, Nov 29, 2014 at 12:13:18PM +0100, Timo Rothenpieler wrote: I've spent a lot more time looking at the nvidia patch, but from a quick look through Timo's version, I'd say the following: * Timo's is more concise but not as feature complete. * nvidia one has windows support * The

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Philip Langdale
On 29 Nov 2014 03:13, Timo Rothenpieler t...@rothenpieler.org wrote: I've spent a lot more time looking at the nvidia patch, but from a quick look through Timo's version, I'd say the following: * Timo's is more concise but not as feature complete. * nvidia one has windows support

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Timo Rothenpieler
BTW, can you verify that interlaced encoding works for you? I get initialization errors back from the hardware. Doesn't work for me either. It looks like it's not (yet?) supported and just available in the API. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Timo Rothenpieler
does supporting these additional features need the extra complexity that the nvidia version has ? or could these features be added into your version while keeping its simplicity ? (note do not copy any code from the nvidia one as its not redistriutable nor *GPL compatible with the current

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Philip Langdale
On Sat, 29 Nov 2014 19:00:02 +0100 Timo Rothenpieler t...@rothenpieler.org wrote: does supporting these additional features need the extra complexity that the nvidia version has ? or could these features be added into your version while keeping its simplicity ? (note do not copy any code

[FFmpeg-devel] [PATCH] avfilter/vf_yadif: fix extra leading dup frame when deint=1

2014-11-29 Thread Neil Birkbeck
Logic for handling single frame in yadif (0f9f24c9), caused deint=1 (e.g., yadif=0:-1:1) to output extra duplicate leading frame: ffmpeg -i fate-suite/ffmpeg-synthetic/vsynth1/%02d.pgm -vf yadif=0:-1:1,showinfo -f null -y /dev/null [Parsed_showinfo_1 @ 0x1d967d0] n:0 pts:0 pts_time:0 pos:-1

Re: [FFmpeg-devel] [PATCH 0/2][RFC] xbr, hqx: minor simplification

2014-11-29 Thread Clément Bœsch
On Sat, Nov 29, 2014 at 03:26:08AM +0100, Alexander Strasser wrote: Hi, while reading the recent evolution of xbr I came up with the idea for below 2 patches. Maybe I am missing something, but AFAICT both filters use a single shared filter function for n in [2, 4] using n as a param

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Timo Rothenpieler
Did some refactoring, now using a dynamic ring-buffer for both the surface lists as well as the timestamp list. There should be no thread safety problem anymore, as there are no non-constant static global variables anymore. I think i addressed most of the issues now, new patch is attached.

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Philip Langdale
On Sun, 30 Nov 2014 00:04:37 +0100 Timo Rothenpieler t...@rothenpieler.org wrote: Did some refactoring, now using a dynamic ring-buffer for both the surface lists as well as the timestamp list. There should be no thread safety problem anymore, as there are no non-constant static global

[FFmpeg-devel] [PATCH 0/6] Toward a SPS filter for H.264

2014-11-29 Thread Christophe Gisquet
Most of the code is actually not mine but originated from Direct264: http://forum.doom9.org/showthread.php?t=152419 https://svn.code.sf.net/p/direct264/code/Patches/ Therefore I've tried to split as best as possible the code I have added. There are 3 controversial parts in this patch set: - the

[FFmpeg-devel] [PATCH 2/6] h264_ps: move and export aspect_ratio

2014-11-29 Thread Christophe Gisquet
Needed for filters that may need it, like an SPS bsf. --- libavcodec/h264_ps.c | 26 +++--- libavcodec/h264data.h | 19 +++ 2 files changed, 22 insertions(+), 23 deletions(-) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index 2013670..4070490

[FFmpeg-devel] [PATCH 4/6] h264_changesps_bsf: fix compilation

2014-11-29 Thread Christophe Gisquet
--- libavcodec/h264_changesps_bsf.c | 24 +++- 1 file changed, 3 insertions(+), 21 deletions(-) diff --git a/libavcodec/h264_changesps_bsf.c b/libavcodec/h264_changesps_bsf.c index 5daea89..a166748 100644 --- a/libavcodec/h264_changesps_bsf.c +++

[FFmpeg-devel] [PATCH 6/6] h264_changesps_bsf: allow specifying full cropping

2014-11-29 Thread Christophe Gisquet
The decoder support is unfortunately dismal. --- libavcodec/h264_changesps_bsf.c | 103 +--- 1 file changed, 64 insertions(+), 39 deletions(-) diff --git a/libavcodec/h264_changesps_bsf.c b/libavcodec/h264_changesps_bsf.c index 43e0d62..39896fa 100644 ---

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-29 Thread Philip Langdale
On Sat, 29 Nov 2014 15:52:00 -0800 Philip Langdale phil...@overt.org wrote: On Sun, 30 Nov 2014 00:04:37 +0100 Timo Rothenpieler t...@rothenpieler.org wrote: Did some refactoring, now using a dynamic ring-buffer for both the surface lists as well as the timestamp list. There should

Re: [FFmpeg-devel] [PATCH 4/6] h264_changesps_bsf: fix compilation

2014-11-29 Thread Carl Eugen Hoyos
Christophe Gisquet christophe.gisquet at gmail.com writes: libavcodec/h264_changesps_bsf.c | 24 +++- 1 file changed, 3 insertions(+), 21 deletions(-) Please merge this into the earlier patch, as-is regression tests are unnecessarily hard. Carl Eugen

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: initial support for EssenceGroups

2014-11-29 Thread Mark Reid
On Fri, Nov 28, 2014 at 9:26 AM, Tomas Härdin tomas.har...@codemill.se wrote: On Tue, 2014-11-25 at 15:14 -0800, Mark Reid wrote: --- libavformat/mxf.c | 1 + libavformat/mxf.h | 1 + libavformat/mxfdec.c | 148 -

Re: [FFmpeg-devel] [PATCH 1/6] bitstream_filter: add an init function

2014-11-29 Thread Michael Niedermayer
On Sun, Nov 30, 2014 at 12:45:51AM +, Christophe Gisquet wrote: From: Zongyi Zhou zho...@os.pku.edu.cn Original patch was adding an arg field in the context structure. As it is most often only useful once, allow passing the arguments on init. The proper solution would be using