From 27d4a640201bfb4a085bddf4d223acfda24709c5 Mon Sep 17 00:00:00 2001
From: alberth <alberth@openttd.org>
Date: Mon, 20 Dec 2010 19:20:38 +0000
Subject: [PATCH] (svn r21549) -Codechange: Unfold the _build_road_button_proc
 array into a switch.

---
 src/road_gui.cpp | 68 ++++++++++++++++++++++++++++++++++--------------
 1 file changed, 49 insertions(+), 19 deletions(-)

diff --git a/src/road_gui.cpp b/src/road_gui.cpp
index 653e416756..1b92c78139 100644
--- a/src/road_gui.cpp
+++ b/src/road_gui.cpp
@@ -433,21 +433,7 @@ static void BuildRoadClick_Remove(Window *w)
 	SndPlayFx(SND_15_BEEP);
 }
 
-/** Array with the handlers of the button-clicks for the road-toolbar */
-static OnButtonClick * const _build_road_button_proc[] = {
-	BuildRoadClick_X_Dir,
-	BuildRoadClick_Y_Dir,
-	BuildRoadClick_AutoRoad,
-	BuildRoadClick_Demolish,
-	BuildRoadClick_Depot,
-	BuildRoadClick_BusStation,
-	BuildRoadClick_TruckStation,
-	BuildRoadClick_OneWay,
-	BuildRoadClick_Bridge,
-	BuildRoadClick_Tunnel,
-	BuildRoadClick_Remove
-};
-
+/** Road toolbar window handler. */
 struct BuildRoadToolbarWindow : Window {
 	BuildRoadToolbarWindow(const WindowDesc *desc, WindowNumber window_number) : Window()
 	{
@@ -529,10 +515,54 @@ struct BuildRoadToolbarWindow : Window {
 
 	virtual void OnClick(Point pt, int widget, int click_count)
 	{
-		if (widget >= RTW_ROAD_X) {
-			_remove_button_clicked = false;
-			_one_way_button_clicked = false;
-			_build_road_button_proc[widget - RTW_ROAD_X](this);
+		_remove_button_clicked = false;
+		_one_way_button_clicked = false;
+		switch (widget) {
+			case RTW_ROAD_X:
+				BuildRoadClick_X_Dir(this);
+				break;
+
+			case RTW_ROAD_Y:
+				BuildRoadClick_Y_Dir(this);
+				break;
+
+			case RTW_AUTOROAD:
+				BuildRoadClick_AutoRoad(this);
+				break;
+
+			case RTW_DEMOLISH:
+				BuildRoadClick_Demolish(this);
+				break;
+
+			case RTW_DEPOT:
+				BuildRoadClick_Depot(this);
+				break;
+
+			case RTW_BUS_STATION:
+				BuildRoadClick_BusStation(this);
+				break;
+
+			case RTW_TRUCK_STATION:
+				BuildRoadClick_TruckStation(this);
+				break;
+
+			case RTW_ONE_WAY:
+				BuildRoadClick_OneWay(this);
+				break;
+
+			case RTW_BUILD_BRIDGE:
+				BuildRoadClick_Bridge(this);
+				break;
+
+			case RTW_BUILD_TUNNEL:
+				BuildRoadClick_Tunnel(this);
+				break;
+
+			case RTW_REMOVE:
+				BuildRoadClick_Remove(this);
+				break;
+
+			default: NOT_REACHED();
 		}
 		this->UpdateOptionWidgetStatus((RoadToolbarWidgets)widget);
 		if (_ctrl_pressed) RoadToolbar_CtrlChanged(this);