[vlc-devel] [patch] purge old and obsolete logger plugin

jnqnfe at gmail.com jnqnfe at gmail.com
Fri Mar 16 21:50:23 CET 2018


On Fri, 2018-03-16 at 22:13 +0200, Rémi Denis-Courmont wrote:
> 	Hello,
> 
> Le perjantaina 16. maaliskuuta 2018, 21.48.49 EET jnqnfe at gmail.com a
> écrit :
> > On Sat, 2018-03-10 at 00:00 +0200, Rémi Denis-Courmont wrote:
> > > Le vendredi 9 mars 2018, 20:22:24 EET jnqnfe at gmail.com a écrit :
> > > > attached
> > > 
> > > Looks incomplete, does this not break the testsuite?
> > 
> > I can;t be certain, since as I've mentioned before, I'm not in a
> > good
> > position to compile. However I don't see why it would. I've had a
> > quick
> > look through the test code and not seen anything relating to it,
> > and
> > also searched it for "log" and not found any link to this old
> > module.
> > The only logger referenced is the console logger.
> 
> I take that as a "yes it does" answer and ignore the patch.

Well as I said, I looked at things in ./test and saw nothing that made
me suspect there was any interaction at all with this plugin, so I
don't see any reason to suspect that it would cause a problem... Is
there any part of the test suite code that you have in mind that I
should take a closer look at?


More information about the vlc-devel mailing list