[vlc-devel] [RFC PATCH 01/12] modules: add module_need_next and vlc_module_load_next

Rémi Denis-Courmont remi at remlab.net
Tue Jul 19 20:03:38 CEST 2016


Le tiistaina 19. heinäkuuta 2016, 19.36.23 EEST Thomas Guillem a écrit :
> These functions save the previous state of a successful vlc_module_load(). A
> second call to this function with the saved context allows to load a module
> that come after the previous one (i.e. with a lower priority or after the
> list given in command line).

This is very ugly and I still completely fail to see the point. VLC module 
loading is already more than complicated enough.

Besides I still fail to see the point of the most of the patch series.

> TODO: document vlc_modules.h

If you can´t document your API, 99.9% chance that your API is wrong.

-- 
Rémi Denis-Courmont
http://www.remlab.net/



More information about the vlc-devel mailing list