(svn r13155) -Codechange: make a window class of the company league and graph legenda windows.

src/graph_gui.cpp

@@ -266,26 +266,26 @@ static void DrawGraph(const GraphDrawer *gw)
 /* GRAPH LEGEND */
 /****************/
 
-static void GraphLegendWndProc(Window *w, WindowEvent *e)
+struct GraphLegendWindow : Window {
+	GraphLegendWindow(const WindowDesc *desc, WindowNumber window_number) : Window(desc, window_number)
 	{
-	switch (e->event) {
+		for (uint i = 3; i < this->widget_count; i++) {
-		case WE_CREATE:
+			if (!HasBit(_legend_excluded_players, i - 3)) this->LowerWidget(i);
-			for (uint i = 3; i < w->widget_count; i++) {
+		}
-				if (!HasBit(_legend_excluded_players, i - 3)) w->LowerWidget(i);
 	}
-			break;
 
-		case WE_PAINT: {
+	virtual void OnPaint()
+	{
 		const Player *p;
 
 		FOR_ALL_PLAYERS(p) {
 			if (p->is_active) continue;
 
 			SetBit(_legend_excluded_players, p->index);
-				w->RaiseWidget(p->index + 3);
+			this->RaiseWidget(p->index + 3);
 		}
 
-			w->DrawWidgets();
+		this->DrawWidgets();
 
 		FOR_ALL_PLAYERS(p) {
 			if (!p->is_active) continue;
@@ -296,23 +296,22 @@ static void GraphLegendWndProc(Window *w, WindowEvent *e)
 			SetDParam(1, p->index);
 			DrawString(21, 17 + p->index * 12, STR_7021, HasBit(_legend_excluded_players, p->index) ? TC_BLACK : TC_WHITE);
 		}
-			break;
 	}
 
-		case WE_CLICK:
+	virtual void OnClick(Point pt, int widget)
-			if (!IsInsideMM(e->we.click.widget, 3, 11)) return;
+	{
+		if (!IsInsideMM(widget, 3, 11)) return;
 
-			ToggleBit(_legend_excluded_players, e->we.click.widget - 3);
+		ToggleBit(_legend_excluded_players, widget - 3);
-			w->ToggleWidgetLoweredState(e->we.click.widget);
+		this->ToggleWidgetLoweredState(widget);
-			w->SetDirty();
+		this->SetDirty();
 		InvalidateWindow(WC_INCOME_GRAPH, 0);
 		InvalidateWindow(WC_OPERATING_PROFIT, 0);
 		InvalidateWindow(WC_DELIVERED_CARGO, 0);
 		InvalidateWindow(WC_PERFORMANCE_HISTORY, 0);
 		InvalidateWindow(WC_COMPANY_VALUE, 0);
-			break;
-	}
 	}
+};
 
 static const Widget _graph_legend_widgets[] = {
 {   WWT_CLOSEBOX,   RESIZE_NONE,    14,     0,    10,     0,    13, STR_00C5,                       STR_018B_CLOSE_WINDOW},
@@ -334,12 +333,12 @@ static const WindowDesc _graph_legend_desc = {
 	WC_GRAPH_LEGEND, WC_NONE,
 	WDF_STD_TOOLTIPS | WDF_STD_BTN | WDF_DEF_WIDGET,
 	_graph_legend_widgets,
-	GraphLegendWndProc
+	NULL
 };
 
 static void ShowGraphLegend()
 {
-	AllocateWindowDescFront<Window>(&_graph_legend_desc, 0);
+	AllocateWindowDescFront<GraphLegendWindow>(&_graph_legend_desc, 0);
 }
 
 /********************/
@@ -879,14 +878,17 @@ static int CDECL PerfHistComp(const void* elem1, const void* elem2)
 	return p2->old_economy[1].performance_history - p1->old_economy[1].performance_history;
 }
 
-static void CompanyLeagueWndProc(Window *w, WindowEvent *e)
+struct CompanyLeagueWindow : Window {
+	CompanyLeagueWindow(const WindowDesc *desc, WindowNumber window_number) : Window(desc, window_number)
+	{
+	}
+
+	virtual void OnPaint()
 	{
-	switch (e->event) {
-		case WE_PAINT: {
 		const Player *plist[MAX_PLAYERS];
 		const Player *p;
 
-			w->DrawWidgets();
+		this->DrawWidgets();
 
 		uint pl_num = 0;
 		FOR_ALL_PLAYERS(p) if (p->is_active) plist[pl_num++] = p;
@@ -903,11 +905,8 @@ static void CompanyLeagueWndProc(Window *w, WindowEvent *e)
 			DrawString(2, 15 + i * 10, i == 0 ? STR_7054 : STR_7055, TC_FROMSTRING);
 			DrawPlayerIcon(p->index, 27, 16 + i * 10);
 		}
-
-			break;
-		}
-	}
 	}
+};
 
 
 static const Widget _company_league_widgets[] = {
@@ -923,12 +922,12 @@ static const WindowDesc _company_league_desc = {
 	WC_COMPANY_LEAGUE, WC_NONE,
 	WDF_STD_TOOLTIPS | WDF_STD_BTN | WDF_DEF_WIDGET | WDF_STICKY_BUTTON,
 	_company_league_widgets,
-	CompanyLeagueWndProc
+	NULL
 };
 
 void ShowCompanyLeagueTable()
 {
-	AllocateWindowDescFront<Window>(&_company_league_desc, 0);
+	AllocateWindowDescFront<CompanyLeagueWindow>(&_company_league_desc, 0);
 }
 
 /*****************************/