changeset 5802:fd85af2792ae draft

(svn r8364) -Codechange (r8362): added an assert to catch conditions where a request wants to draw more vehicles than the list contains
author bjarni <bjarni@openttd.org>
date Tue, 23 Jan 2007 01:00:56 +0000
parents 0bfadf996941
children 56f8199a57ab
files src/build_vehicle_gui.cpp
diffstat 1 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/build_vehicle_gui.cpp
+++ b/src/build_vehicle_gui.cpp
@@ -739,7 +739,7 @@
  * @param x,y Where should the list start
  * @param eng_list What engines to draw
  * @param min where to start in the list
- * @param max where in the list to end. MAKE SURE THAT THIS IS NOT LONGER THAN THE ENGINE LIST ITSELF!
+ * @param max where in the list to end
  * @param selected_id what engine to highlight as selected, if any
  */
 static void DrawEngineList(byte type, int x, int y, const EngineList eng_list, uint16 min, uint16 max, EngineID selected_id)
@@ -748,6 +748,8 @@
 	byte x_offset = 0;
 	byte y_offset = 0;
 
+	assert(max <= EngList_Count(&eng_list));
+
 	switch (type) {
 		case VEH_Train:
 			x++; // train and road vehicles use the same offset, except trains are one more pixel to the right