[vlc-devel] [PATCH 00/14] Move VLM implementation to a module

[Denis Charmet]

Hi,

On 2020-09-15 19:37, Romain Vimont wrote:
> This removes a lot of VLM code (including parsing) from libvlccore.
> 
> This patchset is a pure technical refactor (moving code around), there 
> is no
> change in features or design, and it does not impact callers (Qt and 
> lua).

To be fair, I have mixed feelings regarding this patchset.

While I'm all for refactoring and removing unmaintained stuff, I'm not 
sure, creating a new module type is the path to go.

I'm afraid it could encourage thirdparties to write their own 
implementations and therefore making it more used and keeping it even 
more "alive".
My personal experience of multimedia is that if you give a chance of 
people to do stupid things and they will do it.

IMO I think the first step should be to evaluate what is actually used 
of VLM (in VLC's code and user base) to determine which features are 
used and for what and then triage which ones should be in the core and 
which ones could benefit to be in a module.


Regards,
-- 
Denis Charmet - TypX
Le mauvais esprit est un art de vivre