[vlc-devel] [PATCH] modules: add module_shortcut_exists
robux4 at gmail.com
Fri Feb 12 14:43:39 CET 2016
On Fri, Feb 12, 2016 at 2:28 PM, Thomas Guillem <thomas at gllm.fr> wrote:
> On Fri, Feb 12, 2016, at 13:47, Rémi Denis-Courmont wrote:
>> Le 2016-02-12 13:27, Thomas Guillem a écrit :
>> > And remove module_find_by_shortcut that was unused.
>> > This new function will be used by service discoveries in order to
>> > know if a
>> > protocol is handled by the current vlc instance.
>> Seems as broken as the previous function. As it says, this was but a
>> temporary hack. If it is no longer needed, it should be removed and
> There are services discoverers that can scan for several protocols.
> What do we do if it discovers an item that is not handled by the current
> instance of VLC because a module is missing ? Should we display it
> anyway ? Should we hide it ?
>From the user point of view, it wouldn't make much sense to display
items we cannot use at all. On the other hand the current
module_exists() is limited to the name of the modules. Hardcoding
modules names for each type of renderer (and each possible module that
claim to handle it) is not a good long term solution. Isn't there a
way to look deeper in the module description the protocols it can
>> Rémi Denis-Courmont
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel