cpp/openttd-patchpack/source Changeset - r20079:6dd928cf041b

Changeset - r20079:6dd928cf041b

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

frosch - 11 years ago 2013-02-24 16:42:30
frosch@openttd.org

(svn r25042) -Codechange/Fix: Simplify accumulation of refit options; also don't compare GRF local IDs from different GRFs.

1 file changed with 15 insertions and 31 deletions:

src/vehicle_gui.cpp

0 comments (0 inline, 0 general)

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -251,15 +251,11 @@ byte GetBestFittingSubType(Vehicle *v_fr @@
 				/* Make sure we don't pick up anything cached. */
 				v->First()->InvalidateNewGRFCache();
 				v->InvalidateNewGRFCache();
 				uint16 callback = GetVehicleCallback(CBID_VEHICLE_CARGO_SUFFIX, 0, 0, v->engine_type, v);
 				if (callback != CALLBACK_FAILED) {
 					if (callback > 0x400) ErrorUnknownCallbackResult(v->GetGRFID(), CBID_VEHICLE_CARGO_SUFFIX, callback);
 					if (callback >= 0x400 || (v->GetGRF()->grf_version < 8 && callback == 0xFF)) callback = CALLBACK_FAILED;
+				}
 				if (callback == CALLBACK_FAILED) break;
 				if (!subtypes.Contains(GetCargoSubtypeText(v))) continue;
 				StringID subtype = GetCargoSubtypeText(v);
 				if (subtype == STR_EMPTY) break;
 				if (!subtypes.Contains(subtype)) continue;
 				/* We found something matching. */
 				ret_refit_cyc = refit_cyc;
@@ @@ -286,8 +282,7 @@ byte GetBestFittingSubType(Vehicle *v_fr @@
 struct RefitOption {
 	CargoID cargo;    ///< Cargo to refit to
 	byte subtype;     ///< Subcargo to use
 	uint16 value;     ///< GRF-local String to display for the cargo
 	const Engine *engine;  ///< Engine for which to resolve #value
 	StringID string;  ///< GRF-local String to display for the cargo
 	/**
 	 * Inequality operator for #RefitOption.
@@ @@ -296,7 +291,7 @@ struct RefitOption { @@
 	 */
 	inline bool operator != (const RefitOption &other) const
+	{
-		return other.cargo != this->cargo || other.value != this->value;
+		return other.cargo != this->cargo || other.string != this->string;
+	}
 	/**
@@ @@ -306,7 +301,7 @@ struct RefitOption { @@
 	 */
 	inline bool operator == (const RefitOption &other) const
+	{
-		return other.cargo == this->cargo && other.value == this->value;
+		return other.cargo == this->cargo && other.string == this->string;
+	}
 };
@@ @@ -339,13 +334,8 @@ static void DrawVehicleRefitWindow(const @@
 			const RefitOption refit = list[i][j];
 			/* Get the cargo name. */
 			SetDParam(0, CargoSpec::Get(refit.cargo)->name);
 			/* If the callback succeeded, draw the cargo suffix. */
 			if (refit.value != CALLBACK_FAILED && refit.value < 0x400) {
 				SetDParam(1, GetGRFStringID(refit.engine->GetGRFID(), 0xD000 + refit.value));
 				DrawString(r.left + WD_MATRIX_LEFT, r.right - WD_MATRIX_RIGHT, y, STR_JUST_STRING_STRING, colour);
 			} else {
 				DrawString(r.left + WD_MATRIX_LEFT, r.right - WD_MATRIX_RIGHT, y, STR_JUST_STRING, colour);
+			}
 			SetDParam(1, refit.string);
 			DrawString(r.left + WD_MATRIX_LEFT, r.right - WD_MATRIX_RIGHT, y, STR_JUST_STRING_STRING, colour);
 			y += delta;
 			current++;
@@ @@ -423,19 +413,14 @@ struct RefitWindow : public Window { @@
 						/* Make sure we don't pick up anything cached. */
 						v->First()->InvalidateNewGRFCache();
 						v->InvalidateNewGRFCache();
 						uint16 callback = GetVehicleCallback(CBID_VEHICLE_CARGO_SUFFIX, 0, 0, v->engine_type, v);
 						if (callback != CALLBACK_FAILED) {
 							if (callback > 0x400) ErrorUnknownCallbackResult(v->GetGRFID(), CBID_VEHICLE_CARGO_SUFFIX, callback);
 							if (callback >= 0x400 || (v->GetGRF()->grf_version < 8 && callback == 0xFF)) callback = CALLBACK_FAILED;
+						}
 						if (refit_cyc != 0 && callback == CALLBACK_FAILED) break;
 						StringID subtype = GetCargoSubtypeText(v);
 						if (refit_cyc != 0 && subtype == STR_EMPTY) break;
 						RefitOption option;
 						option.cargo   = cid;
 						option.subtype = refit_cyc;
 						option.value   = callback;
 						option.engine  = v->GetEngine();
 						option.string  = subtype;
 						this->list[current_index].Include(option);
+					}
@@ @@ -451,8 +436,7 @@ struct RefitWindow : public Window { @@
 					RefitOption option;
 					option.cargo   = cid;
 					option.subtype = 0;
 					option.value   = CALLBACK_FAILED;
 					option.engine  = NULL;
 					option.string  = STR_EMPTY;
 					this->list[current_index].Include(option);
+				}
 				current_index++;

0 comments (0 inline, 0 general)