(svn r5665) - Codechange: check pointers against NULL, coding style, tabulation; nothing serious.
--- a/train_cmd.c Sun Jul 30 22:43:50 2006 +0000
+++ b/train_cmd.c Sun Jul 30 22:55:17 2006 +0000
@@ -958,8 +958,7 @@
// if nothing is selected as destination, try and find a matching vehicle to drag to.
if (d == INVALID_VEHICLE) {
- dst = NULL;
- if (!IsTrainEngine(src)) dst = FindGoodVehiclePos(src);
+ dst = IsTrainEngine(src) ? NULL : FindGoodVehiclePos(src);
} else {
dst = GetVehicle(d);
}
@@ -1009,14 +1008,13 @@
if (IsTrainEngine(u) && IsMultiheaded(u) && u->u.rail.other_multiheaded_part != NULL) {
engine = u;
}
- if (engine != NULL && engine->u.rail.other_multiheaded_part == u) {
- engine = NULL;
- }
- if (u == dst) {
- if (engine != NULL) dst = engine->u.rail.other_multiheaded_part;
- break;
- }
-
+ if (engine != NULL && engine->u.rail.other_multiheaded_part == u) {
+ engine = NULL;
+ }
+ if (u == dst) {
+ if (engine != NULL) dst = engine->u.rail.other_multiheaded_part;
+ break;
+ }
}
}
@@ -1111,8 +1109,7 @@
}
} else {
// if moving within the same chain, dont use dst_head as it may get invalidated
- if (src_head == dst_head)
- dst_head = NULL;
+ if (src_head == dst_head) dst_head = NULL;
// unlink single wagon from linked list
src_head = UnlinkWagon(src, src_head);
GetLastEnginePart(src)->next = NULL;
@@ -1188,7 +1185,7 @@
src_head = NULL; // don't do anything more to this train since the new call will do it
}
- if (src_head) {
+ if (src_head != NULL) {
NormaliseTrainConsist(src_head);
TrainConsistChanged(src_head);
if (IsFrontEngine(src_head)) {
@@ -1202,7 +1199,7 @@
InvalidateWindow(WC_VEHICLE_DEPOT, src_head->tile);
};
- if (dst_head) {
+ if (dst_head != NULL) {
NormaliseTrainConsist(dst_head);
TrainConsistChanged(dst_head);
if (IsFrontEngine(dst_head)) {