diff options
author | rubidium <rubidium@openttd.org> | 2009-05-26 15:46:24 +0000 |
---|---|---|
committer | rubidium <rubidium@openttd.org> | 2009-05-26 15:46:24 +0000 |
commit | e3c5bc8d2ebb9f8ab8f4cb64002bc3b8d0e10e37 (patch) | |
tree | 65f84aef7bfb97e14166b4b973f182e4c7a6c0f7 /src/vehicle_gui.cpp | |
parent | 2a5a8f395c0c71146259433a1a4abe7807d2f8ac (diff) | |
download | openttd-e3c5bc8d2ebb9f8ab8f4cb64002bc3b8d0e10e37.tar.xz |
(svn r16434) -Cleanup: remove some dead code; primarily stuff that can't be reached like break after returns or break after functions that never return (i.e. NOT_REACHED)
Diffstat (limited to 'src/vehicle_gui.cpp')
-rw-r--r-- | src/vehicle_gui.cpp | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/src/vehicle_gui.cpp b/src/vehicle_gui.cpp index 57bb893b1..455aed43a 100644 --- a/src/vehicle_gui.cpp +++ b/src/vehicle_gui.cpp @@ -906,7 +906,7 @@ struct VehicleListWindow : public BaseVehicleListWindow { case VEH_ROAD: this->widget[VLW_WIDGET_CAPTION].data = STR_VEHICLE_LIST_ROAD_CAPTION; break; case VEH_SHIP: this->widget[VLW_WIDGET_CAPTION].data = STR_VEHICLE_LIST_SHIP_CAPTION; break; case VEH_AIRCRAFT: this->widget[VLW_WIDGET_CAPTION].data = STR_VEHICLE_LIST_AIRCRAFT_CAPTION; break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } break; @@ -920,7 +920,7 @@ struct VehicleListWindow : public BaseVehicleListWindow { case VEH_ROAD: this->widget[VLW_WIDGET_CAPTION].data = STR_SCHEDULED_ROAD_VEHICLES; break; case VEH_SHIP: this->widget[VLW_WIDGET_CAPTION].data = STR_SCHEDULED_SHIPS; break; case VEH_AIRCRAFT: this->widget[VLW_WIDGET_CAPTION].data = STR_SCHEDULED_AIRCRAFT; break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } break; @@ -930,10 +930,10 @@ struct VehicleListWindow : public BaseVehicleListWindow { case VEH_ROAD: this->widget[VLW_WIDGET_CAPTION].data = STR_VEHICLE_LIST_ROADVEH_DEPOT; break; case VEH_SHIP: this->widget[VLW_WIDGET_CAPTION].data = STR_VEHICLE_LIST_SHIP_DEPOT; break; case VEH_AIRCRAFT: this->widget[VLW_WIDGET_CAPTION].data = STR_VEHICLE_LIST_AIRCRAFT_DEPOT; break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } switch (this->vehicle_type) { @@ -963,7 +963,7 @@ struct VehicleListWindow : public BaseVehicleListWindow { case VEH_ROAD: this->sorting = &_sorting.roadveh; break; case VEH_SHIP: this->sorting = &_sorting.ship; break; case VEH_AIRCRAFT: this->sorting = &_sorting.aircraft; break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } this->vehicles.SetListing(*this->sorting); @@ -1024,7 +1024,7 @@ struct VehicleListWindow : public BaseVehicleListWindow { case VEH_ROAD: SetDParam(0, STR_DEPOT_ROAD_CAPTION); break; case VEH_SHIP: SetDParam(0, STR_DEPOT_SHIP_CAPTION); break; case VEH_AIRCRAFT: SetDParam(0, STR_DEPOT_AIRCRAFT_CAPTION); break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } if (this->vehicle_type == VEH_AIRCRAFT) { SetDParam(1, index); // Airport name @@ -1033,7 +1033,7 @@ struct VehicleListWindow : public BaseVehicleListWindow { } SetDParam(2, this->vscroll.count); break; - default: NOT_REACHED(); break; + default: NOT_REACHED(); } /* Hide the widgets that we will not use in this window @@ -1393,12 +1393,11 @@ struct VehicleDetailsWindow : Window { { switch (vehicle_type) { default: NOT_REACHED(); - case VEH_TRAIN: return _settings_game.vehicle.servint_trains != 0; break; - case VEH_ROAD: return _settings_game.vehicle.servint_roadveh != 0; break; - case VEH_SHIP: return _settings_game.vehicle.servint_ships != 0; break; - case VEH_AIRCRAFT: return _settings_game.vehicle.servint_aircraft != 0; break; + case VEH_TRAIN: return _settings_game.vehicle.servint_trains != 0; + case VEH_ROAD: return _settings_game.vehicle.servint_roadveh != 0; + case VEH_SHIP: return _settings_game.vehicle.servint_ships != 0; + case VEH_AIRCRAFT: return _settings_game.vehicle.servint_aircraft != 0; } - return false; // kill a compiler warning } /** |