[vlc-devel] [PATCH] lib: media: fix header include from MSVC

Rémi Denis-Courmont remi at remlab.net
Thu Oct 22 17:03:47 CEST 2020

Le torstaina 22. lokakuuta 2020, 13.29.36 EEST Thomas Guillem a écrit :
> On Thu, Oct 22, 2020, at 09:17, Thomas Guillem wrote:
> > I don't understand Rémi's comment about namespace violation.
> > 
> > Could you elaborate?
> OK, "Exposing identifiers that you're not supposed to own is a namespace
> violation, potentially leading to potential name conflicts inside the final
> executable that includes it."
> What do you think of
> typedef long (*libvlc_media_read_cb)(void *opaque, unsigned char *buf,
> unsigned long len);

At this point, this would cause a lot of regression with invalid callback 
types and with mismatched integer types, as long and ssize_t are not 
universally identical.


Rémi Denis-Courmont

More information about the vlc-devel mailing list