changeset 13215:5065fbe3e545 draft

(svn r17722) -Codechange: simplify the cargopacket constructor; unduplicate if and don't set a value to 0 if it already is 0.
author rubidium <rubidium@openttd.org>
date Tue, 06 Oct 2009 17:33:01 +0000
parents 835409aa5da2
children b2e6da395b1b
files src/cargopacket.cpp
diffstat 1 files changed, 10 insertions(+), 12 deletions(-) [+]
line wrap: on
line diff
--- a/src/cargopacket.cpp
+++ b/src/cargopacket.cpp
@@ -23,20 +23,18 @@
 	_cargopacket_pool.CleanPool();
 }
 
-CargoPacket::CargoPacket(StationID source, uint16 count, SourceType source_type, SourceID source_id)
+CargoPacket::CargoPacket(StationID source, uint16 count, SourceType source_type, SourceID source_id) :
+	count(count),
+	source(source),
+	source_id(source_id)
 {
-	if (source != INVALID_STATION) assert(count != 0);
+	this->source_type     = source_type;
 
-//	this->feeder_share    = 0; // no need to zero already zeroed data (by operator new)
-	this->source_xy       = (source != INVALID_STATION) ? Station::Get(source)->xy : 0;
-	this->loaded_at_xy    = this->source_xy;
-	this->source          = source;
-
-	this->count           = count;
-//	this->days_in_transit = 0;
-
-	this->source_type     = source_type;
-	this->source_id       = source_id;
+	if (source != INVALID_STATION) {
+		assert(count != 0);
+		this->source_xy    = Station::Get(source)->xy;
+		this->loaded_at_xy = this->source_xy;
+	}
 }
 
 CargoPacket::CargoPacket(uint16 count, byte days_in_transit, Money feeder_share, SourceType source_type, SourceID source_id) :