All,

Another thing I have noticed is that enum values are not capitalized.  General 
coding convention is that enum values are declared in all caps using an 
underscore to separate words.  I notice that our coding conventions are silent 
on enumerations.  Any opposition to adding this rule to our coding conventions?

Thanks,
-John

On Jul 17, 2013, at 12:24 PM, Alex Huang <alex.hu...@citrix.com> wrote:

> That's because the first rule of auto-formatting is do no harm.
> 
> The formatter is set not to screw with lines that are already wrapped 
> assuming the previous developer intended it that way.
> 
> --Alex
> 
>> -----Original Message-----
>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>> Sent: Wednesday, July 17, 2013 8:23 AM
>> To: dev
>> Subject: Re: code formatting for enums
>> 
>> thanks,
>> it doesn't correct back to the one per line format, but at least it doesn't
>> garble the enum when right anymore.
>> 
>> 
>> 
>> On Wed, Jul 17, 2013 at 4:24 PM, Alex Huang <alex.hu...@citrix.com> wrote:
>> 
>>> Windows->Preferences
>>> Java->Formatter
>>> Click on Edit in Active Profiles
>>> Line Wrapping tab
>>> Look for 'enum' declaration->Constants Select Wrap all elements, every
>>> element on a new line in the "Line Wrapping policy:" drop down
>>> 
>>> --Alex
>>> 
>>> 
>>>> -----Original Message-----
>>>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>>>> Sent: Wednesday, July 17, 2013 6:22 AM
>>>> To: dev
>>>> Subject: code formatting for enums
>>>> 
>>>> H,
>>>> 
>>>> I am working on Networks with the eclipse.epf file loaded. Now the
>>>> enum BroadcastDomainType gets saved as
>>>>        Native(null, null), Vlan("vlan", Integer.class), Vswitch("vs",
>>>>                String.class), LinkLocal(null, null), Vnet("vnet",
>>> Long.class), Storage(
>>>>                "storage", Integer.class), Lswitch("lswitch",
>>> String.class), Mido(
>>>>                "mido", String.class), Pvlan("pvlan", String.class),
>>> UnDecided(
>>>>                null, null);
>>>> instead of
>>>>        Native(null, null),
>>>>        Vlan("vlan", Integer.class),
>>>>        Vswitch("vs", String.class),
>>>>        LinkLocal(null, null),
>>>>        Vnet("vnet", Long.class),
>>>>        Storage("storage", Integer.class),
>>>>        Lswitch("lswitch", String.class),
>>>>        Mido("mido", String.class),
>>>>        Pvlan("pvlan", String.class),
>>>>        UnDecided(null, null);
>>>> anybody know how to fix this?
>>>> 
>>>> thanks,
>>>> Daan
>>> 

Reply via email to