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