[vlc-devel] [PATCH 4/4] ytdl: use redirect for single media

Alexandre Janniaux ajanni at videolabs.io
Tue Sep 22 20:44:09 CEST 2020


Hi,

On Tue, Sep 22, 2020 at 07:28:09PM +0200, Pierre Ynard via vlc-devel wrote:
> > Can't we wait to merge this before designing new methods to fix the
> > playlist demux design itself? it seems to me that the discussion would
> > be better focused on a dedicated thread, especially given that Rémi
> > did express his desire to not extend the patchset to the core.
>
> I'm not talking about changing demuxes or the core. I'm saying that it
> looks possible to do it now with the current core and within the current
> access module framework.
>
> > The discussion are not even on the quality of code which is what
> > modules reviews are supposed to be for, especially at the module
> > creation.
>
> The discussion is about the quality of the design approach. The
> discussion is still going on, about and exploring several aspects. Are
> you in a hurry to merge this? I think it's a great feature, but there's
> no hurry.

Well, I'm not in a hurry, but it does take contributor time even if
Rémi didn't seem to be interested in adding hacks or changing the
input item handling. If the discussion goes on and you don't find
a consensus approach for a new feature in this module, when do we
merge, and more importantly when do we tackle the real issues with
the module integration and what we do with it?

Regards,
--
Alexandre Janniaux


More information about the vlc-devel mailing list