[vlc-devel] [PATCH] vout: opengl: add NV12/NV12 support

[Denis Charmet]

On 2016-12-03 13:10, Rémi Denis-Courmont wrote:
> I have not had time to read my OpenGL paper book, so I can't comment
> on the OpenGL aspects. But I think that the file is getting much too
> big, but I don't like meaningless splits with tight dependencies
> between files either :/
> 
> I would though think that chroma conversion belongs in a separate
> conversion plugin. It might not be feasible without an opaque OpenGL
> texture chroma type however.
> 
> Also, NV12 specifically is a hardware decoder output format. For
> sure, it needs to be supported eventually... But does it really make
> sense at this point, that the OpenGL output does not support
> pass-through, so the NV12 will be copied anyway. If copied, then
> conversion to I420 can be done on the fly for free in SIMD. IIRC,
> Laurent even implemented that for x86 already.

I concur that we will need to standardize in vlc_pictures.h how to 
store OpenGL/DX/... textures in picture_t. It will eventually be helpful 
for hardware encoders/filters.

Regards,
-- 
Denis Charmet - TypX
Le mauvais esprit est un art de vivre