[vlc-devel] [PATCH] avcommon: refactor usage of `quiet' variable
ajanni at videolabs.io
Mon Jan 20 08:14:49 CET 2020
Any news on this ? Where should we move it?
On Thu, Dec 12, 2019 at 11:10:08AM +0100, Alexandre Janniaux wrote:
> On Thu, Dec 12, 2019 at 11:04:14AM +0100, Marvin Scholz wrote:
> > On 12 Dec 2019, at 10:01, Alexandre Janniaux wrote:
> > > The `quiet' VLC variable is used although it is exposed only by other
> > > non-mandatory modules. Rename it to avcodec/avformat-quiet so as to
> > > prevent assertion on platform that won't ship with a module exposing a
> > > quiet variable.
> > Hi, thanks for the patch!
> > While this approach works to fix the issue and seems certainly like a
> > possible
> > solution to me, I think it would be more convenient to have just one `quiet`
> > variable and not a per-module one. So maybe the variable should be „owned“
> > by
> > the core instead?
> > Although it seems we already have several module-specific quiet options
> > so maybe we want to keep it that way even though to me it seems weird…
> That's a good question, about moving it to libvlc-modules.c
> instead. I don't know, because it's quite alike the
> fullscreen option or volume option. I've never used this and
> don't really need it, but it was preventing vlc-ios 4.0 from
> running videos with avcodec/avformat.
> I'm fine with either what I did or moving it to
> Alexandre Janniaux
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel