(svn r16766) -Codechange: remove station->MarkDirty. It is in all cases meant to only update the sign and not invalidate the widget; the widget would be invalidated by a call before/after the call to MarkDirty or it wouldn't make sense because e.g. the window doesn't exist anymore/yet.

src/station.cpp

@@ -75,7 +75,7 @@ Station::~Station()
 		}
 	}
 
-	this->MarkDirty();
+	this->sign.MarkDirty();
 	InvalidateWindowData(WC_STATION_LIST, this->owner, 0);
 
 	DeleteWindowById(WC_STATION_VIEW, index);
@@ -151,14 +151,6 @@ void Station::AddFacility(StationFacility new_facility_bit, TileIndex facil_xy)
 	this->build_date = _date;
 }
 
-void Station::MarkDirty() const
-{
-	if (this->sign.width_normal != 0) {
-		InvalidateWindowWidget(WC_STATION_VIEW, index, SVW_CAPTION);
-		this->sign.MarkDirty();
-	}
-}
-
 void Station::MarkTilesDirty(bool cargo_change) const
 {
 	TileIndex tile = this->train_tile;

src/station_base.h

@@ -147,13 +147,6 @@ public:
 
 	void AddFacility(StationFacility new_facility_bit, TileIndex facil_xy);
 
-	/**
-	 * Mark the sign of a station dirty for repaint.
-	 *
-	 * @ingroup dirty
-	 */
-	void MarkDirty() const;
-
 	/**
 	 * Marks the tiles of the station as dirty.
 	 *

src/station_cmd.cpp

@@ -403,9 +403,9 @@ void UpdateAllStationVirtCoord()
  */
 static void UpdateStationVirtCoordDirty(Station *st)
 {
-	st->MarkDirty();
+	st->sign.MarkDirty();
 	st->UpdateVirtCoord();
-	st->MarkDirty();
+	st->sign.MarkDirty();
 }
 
 /** Get a mask of the cargo types that the station accepts.