[vlc-devel] [PATCH] libvlc: add language and frame rate to libvlc_media_track_info_t

Luca Barbato lu_zero at gentoo.org
Wed Jan 30 17:33:30 CET 2013


On 09/10/12 16:49, Rémi Denis-Courmont wrote:
> On Tue,  9 Oct 2012 16:34:03 +0200, Rafaël Carré <funman at videolan.org>
> wrote:
>> From: Sébastien Toque <xilasz at gmail.com>
> 
> Changing the size of libvlc_media_track_info_t is not allowed, since it
> breaks binary compatibility.
> 

You should be safe if you mark the struct only allocable by the function
providing the array of infos, only accessible by pointer, and you append
fields in the end.

If you feel somebody would do something stupid like making a

struct mine{
	libvlc_media_track_info_t info;
	int mine;
}

(or it is used)

Then would be better come up with an additional info struct, marked
accordingly and mark the old one for deprecation.

lu



More information about the vlc-devel mailing list