[mythtv] [mythtv-commits] mythtv commit: r15381 by nigel

[Nigel Pearson]

>> Instead of commenting out the VERBOSE lines that are too chatty  
>> for normal use,
>> wrap them in #ifdef EXTRA_VERBOSITY
>
> I've often thought about adding some kind of way to specify that
> you want extra verbosity.  It should be easy to do now that we can
> OR together the VB flags.

True, and worth doing, but I have recently
started being careful about reducing bloat.
Having a lot of "programmer debug" verbosity
will just slow MythTV down for most users.


> We could create a new VB_EXTRA and remove this bit from the VB_MOST
> mask.  It would work just like VB_TIMESTAMP does now.
>
> VERBOSE(VB_PLAYBACK, "This is a normal playback verbose entry");
> VERBOSE(VB_PLAYBACK|VB_EXTRA, "This is a really verbose playback  
> verbose entry");

Just committed, in [16814].


Note that VB_BLAH+VB_EXTRA is probably clearer, meaning-wise?

--
Nigel Pearson, nigel at ind.tansu.com.au|"I haven't tested it yet,
Telstra Net. Eng., Sydney, Australia | but it should be perfectly safe."
Office: 9202 3900    Fax:  9261 3912 |"Just a bit or harmless brain-
Mobile: 0408 664435  Home: 9792 6998 | manipulation, that's all" -  
Wallace