Re: [FFmpeg-devel] [PATCH 2/2] dxva2: get the slice number directly from the surface in D3D11VA

2016-12-01 Thread Steve Lhomme
On Thu, Dec 1, 2016 at 3:43 PM, Hendrik Leppkes wrote: > On Thu, Dec 1, 2016 at 3:13 PM, Stève Lhomme wrote: >> I think I should update the minor version number as I need to rely on >> this behaviour change in VLC, the surface array will be empty. >> >> Should I also change the headers so that th

Re: [FFmpeg-devel] [PATCH 2/2] dxva2: get the slice number directly from the surface in D3D11VA

2016-12-01 Thread Hendrik Leppkes
On Thu, Dec 1, 2016 at 3:13 PM, Stève Lhomme wrote: > I think I should update the minor version number as I need to rely on > this behaviour change in VLC, the surface array will be empty. > > Should I also change the headers so that they don't require the output > surface to be pre-allocated ? H

[FFmpeg-devel] [PATCH 2/2] dxva2: get the slice number directly from the surface in D3D11VA

2016-12-01 Thread Steve Lhomme
No need to loop through the known surfaces, we'll use it anyway. The loop is only done for DXVA2 --- libavcodec/dxva2.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/dxva2.c b/libavcodec/dxva2.c index a64b1b5..459e267 100644 --- a/libavcodec/dxva2

Re: [FFmpeg-devel] [PATCH 2/2] dxva2: get the slice number directly from the surface in D3D11VA

2016-12-01 Thread Stève Lhomme
I think I should update the minor version number as I need to rely on this behaviour change in VLC, the surface array will be empty. Should I also change the headers so that they don't require the output surface to be pre-allocated ? On Thu, Dec 1, 2016 at 3:10 PM, Steve Lhomme wrote: > No need

Re: [FFmpeg-devel] [PATCH 2/2] dxva2: get the slice number directly from the surface in D3D11VA

2016-12-01 Thread Stève Lhomme
I think I should update the minor version number as I need to rely on this behaviour change in VLC, the surface array will be empty. Should I also change the headers so that they don't require the output surface to be pre-allocated ? On Thu, Dec 1, 2016 at 3:10 PM, Steve Lhomme wrote: > No need