diff --git a/src/cargopacket.cpp b/src/cargopacket.cpp
index 1342389901..79c31d9d85 100644
--- a/src/cargopacket.cpp
+++ b/src/cargopacket.cpp
@@ -78,7 +78,7 @@ CargoList::~CargoList()
 
 void CargoList::AgeCargo()
 {
-	if (this->empty) return;
+	if (this->Empty()) return;
 
 	uint dit = 0;
 	for (List::const_iterator it = this->packets.begin(); it != this->packets.end(); it++) {
@@ -214,12 +214,11 @@ bool CargoList::MoveTo(CargoList *dest, uint count, CargoList::MoveToAction mta,
 
 void CargoList::InvalidateCache()
 {
-	this->empty = this->packets.empty();
 	this->count = 0;
 	this->feeder_share = 0;
 	this->days_in_transit = 0;
 
-	if (this->empty) return;
+	if (this->packets.empty()) return;
 
 	uint dit = 0;
 	for (List::const_iterator it = this->packets.begin(); it != this->packets.end(); it++) {
diff --git a/src/cargopacket.h b/src/cargopacket.h
index c5d0bf822b..7f8114c2dd 100644
--- a/src/cargopacket.h
+++ b/src/cargopacket.h
@@ -163,7 +163,6 @@ public:
 private:
 	List packets;         ///< The cargo packets in this list
 
-	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
 	uint days_in_transit; ///< Cache for the number of days in transit
@@ -197,7 +196,7 @@ public:
 	 */
 	FORCEINLINE bool Empty() const
 	{
-		return this->empty;
+		return this->count == 0;
 	}
 
 	/**
@@ -224,7 +223,7 @@ public:
 	 */
 	FORCEINLINE StationID Source() const
 	{
-		return this->Empty() ? INVALID_STATION : this->packets.front()->source;;
+		return this->Empty() ? INVALID_STATION : this->packets.front()->source;
 	}
 
 	/**