cpp/openttd-patchpack/source Changeset - r12794:fd691da0efc6

Changeset - r12794:fd691da0efc6

Parent rev.

Child rev.

[Not reviewed]

master

0 9 0

smatz - 15 years ago 2009-08-27 13:31:26
smatz@openttd.org

(svn r17292) -Codechange: use unified ToPercent() function to convert fract numbers to percents

9 files changed with 58 insertions and 36 deletions:

bin/ai/regression/regression.txt

src/ai/api/ai_engine.cpp

src/ai/api/ai_vehicle.cpp

src/build_vehicle_gui.cpp

src/core/math_func.hpp

src/industry_gui.cpp

src/order_cmd.cpp

src/station_gui.cpp

src/vehicle_gui.cpp

0 comments (0 inline, 0 general)

bin/ai/regression/regression.txt

➞

Show inline comments

@@ @@ -1252,7 +1252,7 @@ @@
     GetCargoType():     255
     CanRefitCargo():    false
     GetCapacity():      -1
-    GetReliability():   79
+    GetReliability():   80
     GetMaxSpeed():      112
     GetPrice():         41
     GetMaxAge():        7686
@@ @@ -1270,7 +1270,7 @@ @@
     GetCargoType():     255
     CanRefitCargo():    false
     GetCapacity():      -1
-    GetReliability():   83
+    GetReliability():   84
     GetMaxSpeed():      128
     GetPrice():         61
     GetMaxAge():        7320
@@ @@ -3322,7 +3322,7 @@ @@
     GetCargoType():     1
     CanRefitCargo():    true
     GetCapacity():      20
-    GetReliability():   76
+    GetReliability():   77
     GetMaxSpeed():      48
     GetPrice():         347
     GetMaxAge():        5490
@@ @@ -3376,7 +3376,7 @@ @@
     GetCargoType():     2
     CanRefitCargo():    false
     GetCapacity():      22
-    GetReliability():   91
+    GetReliability():   92
     GetMaxSpeed():      48
     GetPrice():         370
     GetMaxAge():        5490
@@ @@ -3484,7 +3484,7 @@ @@
     GetCargoType():     3
     CanRefitCargo():    false
     GetCapacity():      21
-    GetReliability():   97
+    GetReliability():   98
     GetMaxSpeed():      48
     GetPrice():         354
     GetMaxAge():        5490
@@ @@ -3538,7 +3538,7 @@ @@
     GetCargoType():     4
     CanRefitCargo():    false
     GetCapacity():      14
-    GetReliability():   96
+    GetReliability():   97
     GetMaxSpeed():      48
     GetPrice():         337
     GetMaxAge():        5490
@@ @@ -3592,7 +3592,7 @@ @@
     GetCargoType():     5
     CanRefitCargo():    false
     GetCapacity():      14
-    GetReliability():   86
+    GetReliability():   87
     GetMaxSpeed():      48
     GetPrice():         344
     GetMaxAge():        5490
@@ @@ -3646,7 +3646,7 @@ @@
     GetCargoType():     6
     CanRefitCargo():    false
     GetCapacity():      20
-    GetReliability():   96
+    GetReliability():   97
     GetMaxSpeed():      48
     GetPrice():         366
     GetMaxAge():        5490
@@ @@ -3700,7 +3700,7 @@ @@
     GetCargoType():     7
     CanRefitCargo():    false
     GetCapacity():      20
-    GetReliability():   97
+    GetReliability():   98
     GetMaxSpeed():      48
     GetPrice():         379
     GetMaxAge():        5490
@@ @@ -3754,7 +3754,7 @@ @@
     GetCargoType():     8
     CanRefitCargo():    false
     GetCapacity():      22
-    GetReliability():   96
+    GetReliability():   97
     GetMaxSpeed():      48
     GetPrice():         389
     GetMaxAge():        5490
@@ @@ -3808,7 +3808,7 @@ @@
     GetCargoType():     9
     CanRefitCargo():    false
     GetCapacity():      15
-    GetReliability():   81
+    GetReliability():   82
     GetMaxSpeed():      48
     GetPrice():         360
     GetMaxAge():        5490
@@ @@ -3862,7 +3862,7 @@ @@
     GetCargoType():     10
     CanRefitCargo():    false
     GetCapacity():      12
-    GetReliability():   75
+    GetReliability():   76
     GetMaxSpeed():      48
     GetPrice():         466
     GetMaxAge():        5490
@@ @@ -4780,7 +4780,7 @@ @@
     GetCargoType():     3
     CanRefitCargo():    false
     GetCapacity():      220
-    GetReliability():   98
+    GetReliability():   99
     GetMaxSpeed():      24
     GetPrice():         515
     GetMaxAge():        10980
@@ @@ -4816,7 +4816,7 @@ @@
     GetCargoType():     0
     CanRefitCargo():    false
     GetCapacity():      100
-    GetReliability():   87
+    GetReliability():   88
     GetMaxSpeed():      32
     GetPrice():         309
     GetMaxAge():        10980
@@ @@ -5050,7 +5050,7 @@ @@
     GetCargoType():     0
     CanRefitCargo():    false
     GetCapacity():      30
-    GetReliability():   76
+    GetReliability():   77
     GetMaxSpeed():      238
     GetPrice():         48
     GetMaxAge():        10980
@@ @@ -5757,17 +5757,17 @@ @@
 => 14
 => 12
   Reliability ListDump:
 => 97
 => 97
 => 96
 => 96
 => 96
 => 91
 => 86
 => 81
 => 98
 => 98
 => 97
 => 97
 => 97
 => 92
 => 87
 => 82
 => 78
 => 76
 => 75
 => 77
 => 76
   MaxSpeed ListDump:
 => 56
 => 48
@@ @@ -7090,9 +7090,9 @@ @@
   GetBankBalance():     805063
 --AIBuoyList--
-  Count():              1
   Count():             1
   Location ListDump:
   RemoveWaterDepot():   true
   RemoveDock():         true

src/ai/api/ai_engine.cpp

➞

Show inline comments

@@ @@ -99,7 +99,7 @@ @@
 	if (!IsValidEngine(engine_id)) return -1;
 	if (GetVehicleType(engine_id) == AIVehicle::VT_RAIL && IsWagon(engine_id)) return -1;
-	return (::Engine::Get(engine_id)->reliability * 100 >> 16);
+	return ::ToPercent16(::Engine::Get(engine_id)->reliability);
+}
 /* static */ int32 AIEngine::GetMaxSpeed(EngineID engine_id)

src/ai/api/ai_vehicle.cpp

➞

Show inline comments

@@ @@ -438,5 +438,5 @@ @@
 	if (!IsValidVehicle(vehicle_id)) return -1;
 	const Vehicle *v = ::Vehicle::Get(vehicle_id);
-	return v->reliability * 100 >> 16;
+	return ::ToPercent16(v->reliability);
+}

src/build_vehicle_gui.cpp

➞

Show inline comments

@@ @@ -691,7 +691,7 @@ int DrawVehiclePurchaseInfo(int left, in @@
 		y += FONT_HEIGHT_NORMAL;
 		/* Reliability */
-		SetDParam(0, e->reliability * 100 >> 16);
+		SetDParam(0, ToPercent16(e->reliability));
 		DrawString(left, right, y, STR_PURCHASE_INFO_RELIABILITY);
 		y += FONT_HEIGHT_NORMAL;
+	}

src/core/math_func.hpp

➞

Show inline comments

@@ @@ -269,6 +269,28 @@ static FORCEINLINE void Swap(T &a, T &b) @@
 	b = t;
+}
 /**
  * Converts a "fract" value 0..255 to "percent" value 0..100
  * @param i value to convert, range 0..255
  * @return value in range 0..100
  */
 static FORCEINLINE uint ToPercent8(uint i)
+{
 	assert(i < 256);
 	return i * 101 >> 8;
+}
 /**
  * Converts a "fract" value 0..65535 to "percent" value 0..100
  * @param i value to convert, range 0..65535
  * @return value in range 0..100
  */
 static FORCEINLINE uint ToPercent16(uint i)
+{
 	assert(i < 65536);
 	return i * 101 >> 16;
+}
 int LeastCommonMultiple(int a, int b);
 int GreatestCommonDivisor(int a, int b);

src/industry_gui.cpp

➞

Show inline comments

@@ @@ -554,7 +554,7 @@ public: @@
 			SetDParam(1, i->last_month_production[j]);
 			SetDParam(2, GetCargoSuffix(j + 3, CST_VIEW, i, i->type, ind));
-			SetDParam(3, i->last_month_pct_transported[j] * 100 >> 8);
+			SetDParam(3, ToPercent8(i->last_month_pct_transported[j]));
 			uint x = 4 + (IsProductionAlterable(i) ? 30 : 0);
 			DrawString(x, this->widget[IVW_INFO].right, y, STR_INDUSTRY_VIEW_TRANSPORTED);
 			/* Let's put out those buttons.. */
@@ @@ -833,7 +833,7 @@ protected: @@
 		assert(id < lengthof(i->produced_cargo));
 		if (i->produced_cargo[id] == CT_INVALID) return 101;
-		return i->last_month_pct_transported[id] * 100 >> 8;
+		return ToPercent8(i->last_month_pct_transported[id]);
+	}
 	/**
@@ @@ -968,7 +968,7 @@ public: @@
 			/* Transported productions */
 			for (byte j = 0; j < lengthof(i->produced_cargo); j++) {
 				if (i->produced_cargo[j] == CT_INVALID) continue;
-				SetDParam(p++, i->last_month_pct_transported[j] * 100 >> 8);
+				SetDParam(p++, ToPercent8(i->last_month_pct_transported[j]));
+			}
 			/* Drawing the right string */

src/order_cmd.cpp

➞

Show inline comments

@@ @@ -1630,7 +1630,7 @@ VehicleOrderID ProcessConditionalOrder(c @@
 	switch (order->GetConditionVariable()) {
 		case OCV_LOAD_PERCENTAGE:  skip_order = OrderConditionCompare(occ, CalcPercentVehicleFilled(v, NULL), value); break;
-		case OCV_RELIABILITY:      skip_order = OrderConditionCompare(occ, v->reliability * 100 >> 16,        value); break;
+		case OCV_RELIABILITY:      skip_order = OrderConditionCompare(occ, ToPercent16(v->reliability),       value); break;
 		case OCV_MAX_SPEED:        skip_order = OrderConditionCompare(occ, v->GetDisplayMaxSpeed() * 10 / 16, value); break;
 		case OCV_AGE:              skip_order = OrderConditionCompare(occ, v->age / DAYS_IN_LEAP_YEAR,        value); break;
 		case OCV_REQUIRES_SERVICE: skip_order = OrderConditionCompare(occ, v->NeedsServicing(),               value); break;

src/station_gui.cpp

➞

Show inline comments

@@ @@ -954,7 +954,7 @@ struct StationViewWindow : public Window @@
 				if (!HasBit(ge->acceptance_pickup, GoodsEntry::PICKUP)) continue;
 				SetDParam(0, cs->name);
-				SetDParam(2, ge->rating * 101 >> 8);
+				SetDParam(2, ToPercent8(ge->rating));
 				SetDParam(1, STR_CARGO_RATING_APPALLING + (ge->rating >> 5));
 				DrawString(this->widget[SVW_ACCEPTLIST].left + 8, this->widget[SVW_ACCEPTLIST].right - 2, y, STR_STATION_VIEW_CARGO_RATING);
 				y += 10;

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -1424,7 +1424,7 @@ struct VehicleDetailsWindow : Window { @@
 		DrawString(2, this->width - 2, 35, STR_VEHICLE_INFO_PROFIT_THIS_YEAR_LAST_YEAR);
 		/* Draw breakdown & reliability */
-		SetDParam(0, v->reliability * 100 >> 16);
+		SetDParam(0, ToPercent16(v->reliability));
 		SetDParam(1, v->breakdowns_since_last_service);
 		DrawString(2, this->width - 2, 45, STR_VEHICLE_INFO_RELIABILITY_BREAKDOWNS);

0 comments (0 inline, 0 general)