From: Siggi L. <lan...@fa...> - 2003-01-03 04:48:10
|
Hi Miguel, Attached patch shows how to get plugin config values static for the a52dec plugin. This is what I had in mind as a solution to the vanishing codec settings. advantages: - change to the plugin loader is minimal - pre_init function may do more than just register config values (any useful suggestions?) - plugins still don't have to be loaded (saves space on handhelds ;) - 100% compatible with what we currently have disadvantages: - after unloading the plugin, config value descriptions are deallocated, so you currently just see garbage there This can be easily fixed by using strdup, either in the config system or in the plugin pre_init code. - breaks abstraction layers to some degree: The pre_init function has to pre-initialize a complete plugin library, not just one plugin. IMHO, this still makes sense, as you would have to call all pre_init functions in the plugin loader, anyway. Well, I'd really like to compare that with Miguel's solution, or any other suggestions, but I need some sleep first... Good night, Siggi |