View Issue Details

IDProjectCategoryView StatusLast Update
0000704LDMud 3.5Otherpublic2010-03-14 17:07
ReporterCoogan Assigned Tozesstra  
PrioritynormalSeveritytweakReproducibilityalways
Status resolvedResolutionfixed 
Target Version3.5.0Fixed in Version3.5.0 
Summary0000704: commandline options result in divergent named macros
DescriptionIn my opinion the commandline options should match the corresponding macros (and vice versa).

Good examples:
__ALARM_TIME__ and --alarm-time
__MAX_COMMAND_LENGTH__ and --max-command-length
__MAX_MALLOC__ and --max-malloc

But:
__HEART_BEAT_INTERVAL__ and --heart-interval
__SYNCHRONOUS_HEART_BEAT__ and --sync-heart

Because the macros are named descriptively correct, their divergent commandline options should renamed.
TagsNo tags attached.

Activities

zesstra

2010-02-15 17:56

administrator   ~0001730

I am inclined to change this in 3.5.x, then we don't break launch scripts from one minor revision to the next.
A argument against would be that people are lazy on the command-line, but we have the --args flag which should take care of most of the problem...

Are there more than these two? ;-)

zesstra

2010-02-16 15:52

administrator   ~0001736

I changed in r2865:
* --heart-interval -> --heart-beat-interval
* --sync-heart -> --synchronous-heart-beat
* --async-heart -> --asynchronous-heart-beat
* --no-heart -> --no-heart-beat

zesstra

2010-03-14 17:07

administrator   ~0001800

Closing for now, please ask me to re-open if there are others.

Issue History

Date Modified Username Field Change
2009-11-12 03:23 Coogan New Issue
2010-02-15 17:56 zesstra Note Added: 0001730
2010-02-15 17:56 zesstra Assigned To => zesstra
2010-02-15 17:56 zesstra Status new => assigned
2010-02-15 17:56 zesstra Project LDMud => LDMud 3.5
2010-02-15 17:56 zesstra Product Version 3.3 =>
2010-02-15 17:56 zesstra Target Version => 3.5.0
2010-02-16 15:52 zesstra Note Added: 0001736
2010-03-02 13:39 zesstra Status assigned => feedback
2010-03-14 17:07 zesstra Note Added: 0001800
2010-03-14 17:07 zesstra Status feedback => resolved
2010-03-14 17:07 zesstra Fixed in Version => 3.5.0
2010-03-14 17:07 zesstra Resolution open => fixed