[vlc-devel] [patch] add range for --verbose

[jnqnfe at gmail.com]

v2.

- Extended range (to arbritrary larger value of 8), leaving a little
extra room for extra plugin verbosity.
- Adjusted descriptions.

On Thu, 2018-02-22 at 17:05 +0100, Thomas Guillem wrote:
> On Tue, Feb 20, 2018, at 06:14, jnqnfe at gmail.com wrote:
> > --verbose, per its description (VERBOSE_TEXT and VERBOSE_LONGTEXT)
> > has
> > a range of 0-2, but this range is not enforced. This patch adds
> > that
> > missing enforcement.
> > 
> > This does break some plugin debug output, for instance h264 (if not
> > exclusively this), which outputs even more info at greater
> > values... Of
> > course we could just simply expand the range a little to give a
> > little
> > extra room, if not wanting to force plugins into line or to use
> > their
> > own option.
> 
> Hum, not sure about this one.
> 
> The OpenGL vout display shaders with a verbose value of 4 (or 5)
> The ffmepg modules display ffmpeg logs with a verbose value of 3
> 
> We don't want to break that.
> 
> > 
> > ----
> > 
> > I notice that plugins all add +1 (+VLC_MSG_ERR) to verbosity, thus
> > setting default to info+error. We could simplify this by removing
> > the
> > +1, setting the default vebosity to 1, and this new range to 1-3...
> > _______________________________________________
> > vlc-devel mailing list
> > To unsubscribe or modify your subscription options:
> > https://mailman.videolan.org/listinfo/vlc-devel
> > Email had 1 attachment:
> > + verbose_range.patch
> >   1k (text/x-patch)
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: verbose_range_v2.patch
Type: text/x-patch
Size: 1320 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20180302/a0488eef/attachment.bin>