[vlc-devel] [PATCH] core: Move preparser out of the playlist

Rémi Denis-Courmont remi at remlab.net
Tue Jun 26 13:07:29 CEST 2018


Hi,

Exported symbols must be namespaced because of how libdl works. Types, macros and inlines in public headers should be namespaced to preclude compile-time conflicts, but it's not as grave.

Le 26 juin 2018 10:12:39 GMT+01:00, Romain Vimont <rom1v at videolabs.io> a écrit :
>On Tue, Jun 26, 2018 at 11:30:29AM +0300, Rémi Denis-Courmont wrote:
>> Le lundi 25 juin 2018, 18:45:25 EEST Romain Vimont a écrit :
>> > The preparser was implemented in playlist code, but it was
>(already)
>> > independant of the playlist.
>> > 
>> > Therefore, move the implementation to a separate folder and rename
>the
>> > functions.
>> 
>> No new clobbers of the global symbols namespace please.
>
>What would you suggest instead?
>
>Prefixing everything with "vlc_" (including symbols that are not
>exported)?
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20180626/62ef5e96/attachment.html>


More information about the vlc-devel mailing list