Color roles, coming this Monday to kdelibs?
Matthew Woehlke
mw_triad at users.sourceforge.net
Thu Jun 7 15:45:57 BST 2007
Tobias Koenig wrote:
> On Wed, Jun 06, 2007 at 05:58:46PM -0500, Matthew Woehlke wrote:
>> + /**
>> + * This enum describes the color being selected from the given set. All
>> + * roles are valid for all sets.
>> + */
>> + enum ColorRole {
>> + /**
>> + * Normal background.
>> + */
>> + bgNormal = 32,
> Could you change these enums to a form like
> BackgroundNormal = 32
> to stay consistent with other enum definitions, please?
Sure, not a problem. Just to make sure, the first letter /is/
capitalized for an enum?
Do please point out any other stylistic changes that need to be made :-).
--
Matthew
Any sufficiently advanced bug is indistinguishable from a feature.
(...with apologies to Arthur C. Clarke)
More information about the kde-core-devel
mailing list