(svn r17730) -Codechange: do not cache the source of a packet in the cargo list. It's only used for (some) GUI/NewGRF purposes so precalculating it costs more than calculating when it's actually used.

src/cargopacket.h

@@ -166,7 +166,6 @@ private:
 	bool empty;           ///< Cache for whether this list is empty or not
 	uint count;           ///< Cache for the number of cargo entities
 	Money feeder_share;   ///< Cache for the feeder share
-	StationID source;     ///< Cache for the source of the packet
 	uint days_in_transit; ///< Cache for the number of days in transit
 
 public:
@@ -225,7 +224,7 @@ public:
 	 */
 	FORCEINLINE StationID Source() const
 	{
-		return this->source;
+		return this->Empty() ? INVALID_STATION : this->packets.front()->source;;
 	}
 
 	/**