# HG changeset patch
# User smatz <smatz@openttd.org>
# Date 2009-05-30 20:44:34
# Node ID 08018908cfaf8956c0596c3bfdd564e0ad782eb5
# Parent  f99d2e8d78fc41ea959986bbef36618cb8d91a68

(svn r16473) -Fix [FS#2934](r16442): show all non-effect vehicles in the minimap, not only effect vehicles

diff --git a/src/smallmap_gui.cpp b/src/smallmap_gui.cpp
--- a/src/smallmap_gui.cpp
+++ b/src/smallmap_gui.cpp
@@ -20,7 +20,6 @@
 #include "vehicle_base.h"
 #include "sound_func.h"
 #include "window_func.h"
-#include "effectvehicle_base.h"
 
 #include "table/strings.h"
 #include "table/sprites.h"
@@ -740,9 +739,13 @@ public:
 
 		/* draw vehicles? */
 		if (this->map_type == SMT_CONTOUR || this->map_type == SMT_VEHICLES) {
-			EffectVehicle *v;
-			FOR_ALL_EFFECTVEHICLES(v) {
-				if ((v->vehstatus & (VS_HIDDEN | VS_UNCLICKABLE)) == 0) {
+			Vehicle *v;
+			bool skip;
+			byte colour;
+
+			FOR_ALL_VEHICLES(v) {
+				if (v->type != VEH_EFFECT &&
+						(v->vehstatus & (VS_HIDDEN | VS_UNCLICKABLE)) == 0) {
 					/* Remap into flat coordinates. */
 					Point pt = RemapCoords(
 						v->x_pos / TILE_SIZE - this->scroll_x / TILE_SIZE, // divide each one separately because (a-b)/c != a/c-b/c in integer world
@@ -756,7 +759,7 @@ public:
 					if (!IsInsideMM(y, 0, dpi->height)) continue;
 
 					/* Default is to draw both pixels. */
-					bool skip = false;
+					skip = false;
 
 					/* Offset X coordinate */
 					x -= this->subscroll + 3 + dpi->left;
@@ -773,7 +776,7 @@ public:
 					}
 
 					/* Calculate pointer to pixel and the colour */
-					byte colour = (this->map_type == SMT_VEHICLES) ? _vehicle_type_colours[v->type] : 0xF;
+					colour = (this->map_type == SMT_VEHICLES) ? _vehicle_type_colours[v->type] : 0xF;
 
 					/* And draw either one or two pixels depending on clipping */
 					blitter->SetPixel(dpi->dst_ptr, x, y, colour);