Abdelrazak Younes <[EMAIL PROTECTED]> writes:
>> Looks OK. The names of the enum values could be shorter, maybe
>> (ASTR_NONE...).
> Then they are not meaningful. I'll change that to AS_STR as a compromise.

OK.

>
>>  And didn't we agree they should be lowercase?
>>   
> Not really. We agreed that the enum type should be as all other types 
> but that enum value should stay uppercase (see FontEnums.h). We should 
> probably uniformize all this...

In Rules, I see:

* Enumerators
        enum {
                one = 1,
                two = 2,
                three = 3
        };
        -NOT-
        enum { one = 1, two = 2, three 3 }; // wrong
        -NOT-
        enum {
                ONE = 1,
                TWO = 2,
                THREE = 3
        };

I do not say I care much about it, but we should be consistent with
new enums.

JMarc

Reply via email to