Re: Find dParamSuspensionERP in ODE.NET

Hi Remi.

An enum that depends on the order of the declarations is not robust. If you
*must* have JointParam.LoStop < JointParam.HiStop, then you should specify
values. Doing so makes it clear that the values matter. In general, the
values behind enums shouldn't matter.

Dave

----- Original Message ----- 
From: "Remi Ricard" <[email protected]>
To: "Dave Grundgeiger" <[email protected]>; "ODE Mailing List 
(E-mail)" <[email protected]>
Sent: Friday, September 14, 2007 12:44 PM
Subject: Re: [ODE] Find dParamSuspensionERP in ODE.NET


> Hi Dave,
>
>
>> Just a personal design preference: I assume from this definition that 
>> each enumerated constant *must* have a specific underlying value (LoStop 
>> == 0, HiStop == 1, ..., LoStop2 = 256, HiStop2 == 257, etc.) and that the 
>> code will break if they don't have these specific values. If that is the 
>> case, then you should specify all values explicitly.
>>
> What you really need is to keep the ordering for
> LoStopX
> HiStopX
> .....
>
> to be the same( where X is nothing, 2, 3), in C this is achieve with 
> macro.
>
> Remi
>
>> ----- Original Message ----- 
>> From: "Jason Perkins" <[email protected]>
>> To: "Cardinal4" <[email protected]>
>> Cc: <[email protected]>
>> Sent: Friday, September 14, 2007 7:03 AM
>> Subject: Re: [ODE] Find dParamSuspensionERP in ODE.NET
>>
>>
>>
>>> On 9/13/07, Cardinal4 <[email protected]> wrote:
>>>
>>>> In ODE.NET, I couldn't locate anything called d.ParamSuspensionERP, or
>>>> anything similar.
>>>>
>>> I've added these as a new JointParam enumeration:
>>>
>>> public enum JointParam : int
>>> {
>>> LoStop,
>>> HiStop,
>>> Vel,
>>> FMax,
>>> FudgeFactor,
>>> Bounce,
>>> CFM,
>>> StopERP,
>>> StopCFM,
>>> SuspensionERP,
>>> SuspensionCFM,
>>> LoStop2 = 256,
>>> HiStop2,
>>> Vel2,
>>> FMax2,
>>> FudgeFactor2,
>>> Bounce2,
>>> CFM2,
>>> StopERP2,
>>> StopCFM2,
>>> SuspensionERP2,
>>> SuspensionCFM2,
>>> LoStop3 = 512,
>>> HiStop3,
>>> Vel3,
>>> FMax3,
>>> FudgeFactor3,
>>> Bounce3,
>>> CFM3,
>>> StopERP3,
>>> StopCFM3,
>>> SuspensionERP3,
>>> SuspensionCFM3
>>> }
>>>
>>> Comments and suggestions welcome.
>>>
>>> Jason
>>> _______________________________________________
>>> ODE mailing list
>>> [email protected]
>>> http://ode.org/mailman/listinfo/ode
>>>
>>>
>>
>> _______________________________________________
>> ODE mailing list
>> [email protected]
>> http://ode.org/mailman/listinfo/ode
>>
>>
>
>