# HG changeset patch
# User skidd13
# Date 1213973687 0
# Node ID ba4f22e6e3d95d841ba2df3e886d37d781214baa
# Parent  071b2187a1767c729fd87ae53341531fc64b6402
(svn r13592) -Fix: window size remembering was broken

diff -r 071b2187a176 -r ba4f22e6e3d9 src/bridge_gui.cpp
--- a/src/bridge_gui.cpp	Fri Jun 20 07:23:00 2008 +0000
+++ b/src/bridge_gui.cpp	Fri Jun 20 14:54:47 2008 +0000
@@ -60,7 +60,7 @@
 class BuildBridgeWindow : public Window {
 private:
 	/* Runtime saved values */
-	static uint last_size;
+	static uint16 last_size;
 	static Listing last_sorting;
 
 	/* Constants for sorting the bridges */
@@ -132,9 +132,8 @@
 			this->vscroll.cap = 4;
 		} else {
 			/* Resize the bridge selection window if we used a bigger one the last time */
-			this->vscroll.cap = (this->vscroll.count > this->last_size) ? this->last_size : this->vscroll.count;
+			this->vscroll.cap = min(this->last_size, this->vscroll.count);
 			ResizeWindow(this, 0, (this->vscroll.cap - 4) * this->resize.step_height);
-			this->widget[BBSW_BRIDGE_LIST].data = (this->vscroll.cap << 8) + 1;
 		}
 
 		this->FindWindowPlacementAndResize(desc);
@@ -222,12 +221,12 @@
 		this->widget[BBSW_BRIDGE_LIST].data = (this->vscroll.cap << 8) + 1;
 		SetVScrollCount(this, this->bridges->Length());
 
-		this->last_size = this->vscroll.cap;
+		this->last_size = max(this->vscroll.cap, this->last_size);
 	}
 };
 
 /* Set the default size of the Build Bridge Window */
-uint BuildBridgeWindow::last_size = 4;
+uint16 BuildBridgeWindow::last_size = 4;
 /* Set the default sorting for the bridges */
 Listing BuildBridgeWindow::last_sorting = {false, 0};