[vlc-devel] [PATCH] include: add variables to video_format_t for 360 videos

Adrien Maglo magsoft at videolan.org
Mon May 23 16:47:29 CEST 2016


Le 23/05/2016 16:19, Derek Buitenhuis a écrit :
> On 5/23/2016 3:15 PM, Adrien Maglo wrote:
>> +    int i_cubemap_padding;      /**< padding in pixels of the cube map faces */
>
> "`padding` is a 32-bit unsigned integer measuring the number of pixels to pad from the edge of each cube face."

int -> unsigned int, uint32 ?
There is not enough documentation ?

unsigned int may have indeed more sense to store a padding. I am not 
sure about uint32_t. The idea here is not necessarily to match perfectly 
the Google spec. It is more to find the representation that makes more 
sense inside VLC.

Best regards,


-- 
MagSoft


More information about the vlc-devel mailing list