[mythtv] [PATCH] Re: policy on using 'VERBOSE' in myth

Isaac Richards ijr at po.cwru.edu
Sun Aug 24 22:05:59 EDT 2003


On Sunday 24 August 2003 08:06 pm, Geoffrey Hausheer wrote:
> Okay, well, here is a patch that actually implements a useful 'VERBOSE'
> command.
> '--verbose' in mythbackend now takes an argument to enable different
> sections.  the default is 'general', which is less verbose than it use to
> be without '-v' (with some exceptions).  To enable specific options, a
> comma seperated argument after  '-v' will work.  '-v all' is equivalent
> to what '-v' was before the patch.
>
> The availiable flags for 'VERBOSE' are:
> VB_IMPORTANT : I haven't actually used this one yet, but it should be all
> mesages that are very important but non fatal.
> VB_GENERAL   : This is anything the 'normal' user might want to know
> about
> VB_RECORD    : Debug messages related to recording
> VB_PLAYBACK  : Debug messages related to playback
> VB_OSD       : Debug messages related to the OSD
> VB_CHANNEL   : Debug messages related to changing channels
> VB_FILE      : Debug messages related to reading/writing files
>
> (the defulat is VB_IMPORTANT | VB_GENERAL)
>
> I have done most of libmythtv, but haven't done much with mythbackend yet
> (besides the mandatory changes).
>
> Isaac, if you like the idea, but think this is too much to do all at
> once, only the changes to libmyth, tv_play.cpp, and mythbackend are
> needed.  Everything else could be added later.

This looks fine to me, but you've only got:

enum VerboseMask {
    VB_GENERAL  = 0x01,
    VB_RECORD   = 0x02,
    VB_PLAYBACK = 0x04,
    VB_CHANNEL  = 0x08,
    VB_NONE     = 0x00,
    VB_ALL      = 0xff
};

in the patch, not what you described?

Isaac


More information about the mythtv-dev mailing list