cpp/openttd-patchpack/source Changeset - r5077:d84dfecd5f73

Changeset - r5077:d84dfecd5f73

Parent rev.

Child rev.

[Not reviewed]

master

0 5 0

bjarni - 18 years ago 2006-11-13 20:33:51
bjarni@openttd.org

(svn r7138) -Fix: [vehicle list windows] fixed a rare crash where having some (not all) vehicle list windows open for a player, that goes bankrupt would crash the game
-Codechange: closing all windows for a player will now loop all windows and close those, which got the player as caption instead of having a list of windows to close

5 files changed with 42 insertions and 21 deletions:

economy.c

player.h

players.c

window.c

window.h

0 comments (0 inline, 0 general)

economy.c

➞

Show inline comments

@@ @@ -333,13 +333,8 @@ void ChangeOwnershipOfPlayerItems(Player @@
 		} while (++tile != MapSize());
+	}
 	// Change color of existing windows
 	if (new_player != PLAYER_SPECTATOR) {
 		Window *w;
 		for (w = _windows; w != _last_window; w++) {
 			if (w->caption_color == old_player) w->caption_color = new_player;
+		}
+	}
 	/* Change color of existing windows */
 	if (new_player != PLAYER_SPECTATOR) ChangeWindowOwner(old_player, new_player);
+	{
 		Player *p;

player.h

➞

Show inline comments

@@ @@ -252,7 +252,6 @@ static inline bool IsValidPlayer(PlayerI @@
 	return pi < MAX_PLAYERS;
+}
 void DeletePlayerWindows(PlayerID pi);
 byte GetPlayerRailtypes(PlayerID p);
 /** Finds out if a Player has a certain railtype available */

players.c

➞

Show inline comments

@@ @@ -26,6 +26,7 @@ @@
 #include "engine.h"
 #include "ai/ai.h"
 #include "date.h"
 #include "window.h"
 uint16 GetDrawStringPlayerColor(PlayerID player)
@@ @@ -616,19 +617,6 @@ void PlayersYearlyLoop(void) @@
+	}
+}
 void DeletePlayerWindows(PlayerID pi)
+{
 	DeleteWindowById(WC_COMPANY, pi);
 	DeleteWindowById(WC_PLAYER_COLOR, pi);
 	DeleteWindowById(WC_FINANCES, pi);
 	DeleteWindowById(WC_STATION_LIST, pi);
 	DeleteWindowById(WC_TRAINS_LIST,   (INVALID_STATION << 16) | pi);
 	DeleteWindowById(WC_ROADVEH_LIST,  (INVALID_STATION << 16) | pi);
 	DeleteWindowById(WC_SHIPS_LIST,    (INVALID_STATION << 16) | pi);
 	DeleteWindowById(WC_AIRCRAFT_LIST, (INVALID_STATION << 16) | pi);
 	DeleteWindowById(WC_BUY_COMPANY, pi);
+}
 byte GetPlayerRailtypes(PlayerID p)
+{
 	byte rt = 0;

window.c

➞

Show inline comments

@@ @@ -348,6 +348,43 @@ void DeleteWindowByClass(WindowClass cls @@
+	}
+}
 void DeletePlayerWindows(PlayerID pi)
+{
 	Window *w;
 	for (w = _windows; w != _last_window;) {
 		if (w->caption_color == pi) {
 			DeleteWindow(w);
 			w = _windows;
 		} else {
 			w++;
+		}
+	}
 	/* Also delete the player specific windows, that haven't got the caption set */
 	DeleteWindowById(WC_BUY_COMPANY, pi);
+}
 /* Change the owner of all the windows one player can take over from another player (like vehicle view windows) */
 void ChangeWindowOwner(PlayerID old_player, PlayerID new_player)
+{
 	Window *w;
 	for (w = _windows; w != _last_window; w++) {
 		if (w->caption_color != old_player)      continue;
 		if (w->window_class == WC_PLAYER_COLOR)  continue;
 		if (w->window_class == WC_FINANCES)      continue;
 		if (w->window_class == WC_STATION_LIST)  continue;
 		if (w->window_class == WC_TRAINS_LIST)   continue;
 		if (w->window_class == WC_ROADVEH_LIST)  continue;
 		if (w->window_class == WC_SHIPS_LIST)    continue;
 		if (w->window_class == WC_AIRCRAFT_LIST) continue;
 		if (w->window_class == WC_BUY_COMPANY)   continue;
 		if (w->window_class == WC_COMPANY)       continue;
 		w->caption_color = new_player;
+	}
+}
 static Window *BringWindowToFront(Window *w);

window.h

➞

Show inline comments

@@ @@ -613,6 +613,8 @@ void SendWindowMessageClass(WindowClass @@
 Window *FindWindowById(WindowClass cls, WindowNumber number);
 void DeleteWindow(Window *w);
 void DeletePlayerWindows(PlayerID pi);
 void ChangeWindowOwner(PlayerID old_player, PlayerID new_player);
 Window *BringWindowToFrontById(WindowClass cls, WindowNumber number);
 Window *FindWindowFromPt(int x, int y);

0 comments (0 inline, 0 general)