diff options
author | tron <tron@openttd.org> | 2006-02-06 09:18:04 +0000 |
---|---|---|
committer | tron <tron@openttd.org> | 2006-02-06 09:18:04 +0000 |
commit | 087fe86ed28fb6ac5885cac7f64d34ccd5a9c140 (patch) | |
tree | 3a4002304d8204d11c37ba706d6ffbd2e5dfac1b /train_cmd.c | |
parent | 75a70e1def8c677d2340a49e9f12e4c2a5d65aee (diff) | |
download | openttd-087fe86ed28fb6ac5885cac7f64d34ccd5a9c140.tar.xz |
(svn r3564) Several smaller changes:
- Don't treat non-booleans as booleans
- Reduce variable scope
- Bracing
- Use DeMorgan's law to make conditionals easier to read
- if cascade -> switch
- Replace some magic numbers by symbolic names
- Avoid assignments within other statements
Diffstat (limited to 'train_cmd.c')
-rw-r--r-- | train_cmd.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/train_cmd.c b/train_cmd.c index 3b0347a7a..045b7d478 100644 --- a/train_cmd.c +++ b/train_cmd.c @@ -44,9 +44,7 @@ static void TrainCargoChanged(Vehicle* v) for (u = v; u != NULL; u = u->next) { const RailVehicleInfo *rvi = RailVehInfo(u->engine_type); - uint16 vweight = 0; - - vweight += (_cargoc.weights[u->cargo_type] * u->cargo_count) / 16; + uint16 vweight = (_cargoc.weights[u->cargo_type] * u->cargo_count) / 16; // Vehicle weight is not added for articulated parts. if (!IsArticulatedPart(u)) { |