Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-04 Thread Timo Rothenpieler
Am 04.10.2017 um 11:05 schrieb wm4: On Tue, 3 Oct 2017 07:17:25 -0700 Philip Langdale wrote: I'd propose to use this as a chance to get in line with nvidias new naming, and call the new cuvid decoder/hwaccel nvdec. This is quite a deviation from libav, but we need to rename

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-04 Thread wm4
On Tue, 3 Oct 2017 07:17:25 -0700 Philip Langdale wrote: > > I'd propose to use this as a chance to get in line with nvidias new > > naming, and call the new cuvid decoder/hwaccel nvdec. This is quite a > > deviation from libav, but we need to rename it anyways, so might as

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread Philip Langdale
On Tue, 3 Oct 2017 15:15:18 +0200 wm4 wrote: > From: Anton Khirnov > > Some parts of the code are based on a patch by > Timo Rothenpieler > > Merges Libav commit b9129ec4668c511e0a79e25c6f25d748cee172c9. > > As a

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread Philip Langdale
On Tue, 3 Oct 2017 16:08:32 +0200 Timo Rothenpieler wrote: > Am 03.10.2017 um 15:15 schrieb wm4: > > From: Anton Khirnov > > > > Some parts of the code are based on a patch by > > Timo Rothenpieler > > > > Merges Libav commit

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread wm4
On Tue, 3 Oct 2017 16:08:32 +0200 Timo Rothenpieler wrote: > I'm not a fan of there being cuvid and cuvid_hwaccel now, meaning > potentially multiple things. It seems super confusing to me. Yes, that's a pretty annoying situation. > I'd propose to use this as a chance

Re: [FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread Timo Rothenpieler
Am 03.10.2017 um 15:15 schrieb wm4: From: Anton Khirnov Some parts of the code are based on a patch by Timo Rothenpieler Merges Libav commit b9129ec4668c511e0a79e25c6f25d748cee172c9. As a complication, all the names conflict. Add a _hwaccel suffix

[FFmpeg-devel] [PATCH 7/7] h264dec: add a CUVID hwaccel

2017-10-03 Thread wm4
From: Anton Khirnov Some parts of the code are based on a patch by Timo Rothenpieler Merges Libav commit b9129ec4668c511e0a79e25c6f25d748cee172c9. As a complication, all the names conflict. Add a _hwaccel suffix to the merged code where needed. This