[vlc-devel] [PATCH 2/2] modules: add 'vlc_' prefix to module_exists()

Lyndon Brown jnqnfe at gmail.com
Tue Sep 29 00:53:11 CEST 2020


On Mon, 2020-09-28 at 09:15 +0200, Steve Lhomme wrote:
> On 2020-09-27 1:36, Lyndon Brown wrote:
> > Subject: modules: add 'vlc_' prefix to module_exists()
> 
> It seems odd that it's the only function of that header with the
> proper 
> prefix. I think it's better to do all or none at once.

There are three others in that header with the prefix
(vlc_module_match, vlc_module_map and vlc_module_load), granted two
only just recently added.

My impression has been that adding the prefix to existing functions is
something only really being done opportunistically.

If there's consensus to just go ahead and add it across the board, I'm
not opposed to crafting the patches to do so at some point, but there's
a bunch of work to be submitted later for which it would be best to
delay doing so since in many cases they do so anyway as part of
changing the API.



More information about the vlc-devel mailing list