Archives/OpenTTD-patches
2
0
Fork 0
mirror of https://github.com/JGRennison/OpenTTD-patches.git synced 2024-11-19 15:25:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

(svn r6228) -Fix: redrawing an empty list of shared vehicles will now close the window

Browse Source 
Since the order is no more, the player will not need that particular window anymore
	this could happen if the player opened the window and then sold all the vehicles (or crashed them)
	A crash could happen if a new order gets the same (now freed) OrderID and the vehicle using it is not of the same type as the window expect
This commit is contained in:
bjarni 2006-08-29 22:47:38 +00:00
parent c820704f11
commit 1305b2365f
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
vehicle_gui.c
View File

@ -1244,6 +1244,18 @@ void PlayerVehWndProc(Window *w, WindowEvent *e)
switch (window_type) {
case VLW_SHARED_ORDERS:
/* Shared Orders */
if (vl->list_length == 0) {
/* The list is empty, so the last vehicle is sold or crashed */
/* Delete the window because the order is now not in use anymore */
switch (vehicle_type) {
case VEH_Train: DeleteWindowById(WC_TRAINS_LIST, w->window_number); break;
case VEH_Road: DeleteWindowById(WC_ROADVEH_LIST, w->window_number); break;
case VEH_Ship: DeleteWindowById(WC_SHIPS_LIST, w->window_number); break;
case VEH_Aircraft: DeleteWindowById(WC_AIRCRAFT_LIST, w->window_number); break;
default: NOT_REACHED(); break;
}
return;
}
SetDParam(0, w->vscroll.count);
w->widget[1].unkA = STR_VEH_WITH_SHARED_ORDERS_LIST;
w->widget[9].unkA = STR_EMPTY;