View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000704 | LDMud 3.5 | Other | public | 2009-11-12 04:23 | 2010-03-14 18:07 |
| Reporter | Coogan | Assigned To | zesstra | ||
| Priority | normal | Severity | tweak | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Target Version | 3.5.0 | Fixed in Version | 3.5.0 | ||
| Summary | 0000704: commandline options result in divergent named macros | ||||
| Description | In 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. | ||||
| Tags | No tags attached. | ||||
|
|
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? ;-) |
|
|
I changed in r2865: * --heart-interval -> --heart-beat-interval * --sync-heart -> --synchronous-heart-beat * --async-heart -> --asynchronous-heart-beat * --no-heart -> --no-heart-beat |
|
|
Closing for now, please ask me to re-open if there are others. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-11-12 04:23 | Coogan | New Issue | |
| 2010-02-15 18:56 | zesstra | Note Added: 0001730 | |
| 2010-02-15 18:56 | zesstra | Assigned To | => zesstra |
| 2010-02-15 18:56 | zesstra | Status | new => assigned |
| 2010-02-15 18:56 | zesstra | Project | LDMud => LDMud 3.5 |
| 2010-02-15 18:56 | zesstra | Product Version | 3.3 => |
| 2010-02-15 18:56 | zesstra | Target Version | => 3.5.0 |
| 2010-02-16 16:52 | zesstra | Note Added: 0001736 | |
| 2010-03-02 14:39 | zesstra | Status | assigned => feedback |
| 2010-03-14 18:07 | zesstra | Note Added: 0001800 | |
| 2010-03-14 18:07 | zesstra | Status | feedback => resolved |
| 2010-03-14 18:07 | zesstra | Fixed in Version | => 3.5.0 |
| 2010-03-14 18:07 | zesstra | Resolution | open => fixed |
