[mythtv-firehose] mythtv/master commit: 9d9ae7d32 by Gavin Hurlbut (Beirdo)

MythTV noreply at mythtv.org
Thu Jun 23 09:31:42 UTC 2011


      Author:  Gavin Hurlbut <ghurlbut at mythtv.org>
 Change Date:  2011-06-23T02:31:04-07:00
   Push Date:  2011/06/23 02:31:31 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  9d9ae7d3276c53bb3c90c4e6033a03192692bffb
   Changeset:  https://github.com/MythTV/mythtv/commit/9d9ae7d32

Log:

Reimplemented the VB_* definitions

The old macros made for confusing code that was rather hard to maintain.  I
changed it to store the verbose mask options in a QMap, which also has the
advantage of far simpler searching for the options while parsing.

Additionally, I have attempted to expand the size of the mask to 64 bits,
although we will have to see if it likes extending enum past 32 bits.  All I've
read online seems to indicate that enum is designed to auto-size and that you
shouldn't depend on it always being 32-bit, so hopefully when assigned a value
> 32bit in size, gcc will do the right thing.  The internal variables have been
changed to uint64_t in any case, so if there's borkage, I expect it will be in
creating the enum itself (which is in verbosedefs.h now)

Added:

   mythtv/libs/libmythbase/verbosedefs.h

Modified:

   mythtv/libs/libmythbase/libmythbase.pro
   mythtv/libs/libmythbase/mythverbose.cpp
   mythtv/libs/libmythbase/mythverbose.h



More information about the mythtv-firehose mailing list