# HG changeset patch
# User alberth <alberth@openttd.org>
# Date 2010-08-08 11:02:57
# Node ID f28325f49ee0f213ba9abe6a65268bc8e7b8be96
# Parent  ae8e2e185874ab73c653c2069900c73cb4dcddbe

(svn r20412) -Codechange: Replace an if by a switch in IndustryCargoesWindow::OnClick.

diff --git a/src/industry_gui.cpp b/src/industry_gui.cpp
--- a/src/industry_gui.cpp
+++ b/src/industry_gui.cpp
@@ -2346,33 +2346,36 @@ struct IndustryCargoesWindow : public Wi
 
 	virtual void OnClick(Point pt, int widget, int click_count)
 	{
-		if (widget != ICW_PANEL) return;
+		switch (widget) {
+			case ICW_PANEL: {
+				Point fieldxy, xy;
+				if (!CalculatePositionInWidget(pt, &fieldxy, &xy)) return;
 
-		Point fieldxy, xy;
-		if (!CalculatePositionInWidget(pt, &fieldxy, &xy)) return;
+				const CargoesField *fld = this->fields[fieldxy.y].columns + fieldxy.x;
+				switch (fld->type) {
+					case CFT_INDUSTRY:
+						if (fld->u.industry.ind_type < NUM_INDUSTRYTYPES) this->ComputeIndustryDisplay(fld->u.industry.ind_type);
+						break;
 
-		const CargoesField *fld = this->fields[fieldxy.y].columns + fieldxy.x;
-		switch (fld->type) {
-			case CFT_INDUSTRY:
-				if (fld->u.industry.ind_type < NUM_INDUSTRYTYPES) this->ComputeIndustryDisplay(fld->u.industry.ind_type);
-				break;
+					case CFT_CARGO: {
+						CargoesField *lft = (fieldxy.x > 0) ? this->fields[fieldxy.y].columns + fieldxy.x - 1 : NULL;
+						CargoesField *rgt = (fieldxy.x < 4) ? this->fields[fieldxy.y].columns + fieldxy.x + 1 : NULL;
+						CargoID cid = fld->CargoClickedAt(lft, rgt, xy);
+						if (cid != INVALID_CARGO) this->ComputeCargoDisplay(cid);
+						break;
+					}
 
-			case CFT_CARGO: {
-				CargoesField *lft = (fieldxy.x > 0) ? this->fields[fieldxy.y].columns + fieldxy.x - 1 : NULL;
-				CargoesField *rgt = (fieldxy.x < 4) ? this->fields[fieldxy.y].columns + fieldxy.x + 1 : NULL;
-				CargoID cid = fld->CargoClickedAt(lft, rgt, xy);
-				if (cid != INVALID_CARGO) this->ComputeCargoDisplay(cid);
+					case CFT_CARGO_LABEL: {
+						CargoID cid = fld->CargoLabelClickedAt(xy);
+						if (cid != INVALID_CARGO) this->ComputeCargoDisplay(cid);
+						break;
+					}
+
+					default:
+						break;
+				}
 				break;
 			}
-
-			case CFT_CARGO_LABEL: {
-				CargoID cid = fld->CargoLabelClickedAt(xy);
-				if (cid != INVALID_CARGO) this->ComputeCargoDisplay(cid);
-				break;
-			}
-
-			default:
-				break;
 		}
 	}