summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorfrosch <frosch@openttd.org>2013-02-24 16:42:30 +0000
committerfrosch <frosch@openttd.org>2013-02-24 16:42:30 +0000
commitb9aeb050e1372c8bc42c59f53e1a760bcae097af (patch)
tree43b01ed276767322b0756f031771d0dd88af8b85
parent22bb015f3d1f172227a74cb6d9d0bc96378f452a (diff)
downloadopenttd-b9aeb050e1372c8bc42c59f53e1a760bcae097af.tar.xz
(svn r25042) -Codechange/Fix: Simplify accumulation of refit options; also don't compare GRF local IDs from different GRFs.
-rw-r--r--src/vehicle_gui.cpp40
1 files changed, 12 insertions, 28 deletions
diff --git a/src/vehicle_gui.cpp b/src/vehicle_gui.cpp
index 1a60c6edd..fd70a1aa2 100644
--- a/src/vehicle_gui.cpp
+++ b/src/vehicle_gui.cpp
@@ -251,15 +251,11 @@ byte GetBestFittingSubType(Vehicle *v_from, Vehicle *v_for, CargoID dest_cargo_t
/* 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;
+ StringID subtype = GetCargoSubtypeText(v);
+ if (subtype == STR_EMPTY) break;
- if (!subtypes.Contains(GetCargoSubtypeText(v))) continue;
+ if (!subtypes.Contains(subtype)) continue;
/* We found something matching. */
ret_refit_cyc = refit_cyc;
@@ -286,8 +282,7 @@ byte GetBestFittingSubType(Vehicle *v_from, Vehicle *v_for, CargoID dest_cargo_t
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 SubtypeList list[NUM_CARGO], int sel, u
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++;