# HG changeset patch # User frosch # Date 2012-01-15 17:33:35 # Node ID 3e25b87ea626b0150ec924df0e56b016c1d038d0 # Parent 9f835e6d0ca203a99f0f2039308de00a90637b63 (svn r23807) -Codechange: GRFError::num_params is not needed, remove it. diff --git a/src/newgrf.cpp b/src/newgrf.cpp --- a/src/newgrf.cpp +++ b/src/newgrf.cpp @@ -6195,12 +6195,10 @@ static void GRFLoadError(ByteReader *buf } /* Only two parameter numbers can be used in the string. */ - uint i = 0; - for (; i < 2 && buf->HasData(); i++) { + for (uint i = 0; i < lengthof(error->param_value) && buf->HasData(); i++) { uint param_number = buf->ReadByte(); error->param_value[i] = _cur.grffile->GetParam(param_number); } - error->num_params = i; _cur.grfconfig->error = error; } diff --git a/src/newgrf_config.cpp b/src/newgrf_config.cpp --- a/src/newgrf_config.cpp +++ b/src/newgrf_config.cpp @@ -188,8 +188,7 @@ GRFError::GRFError(const GRFError &error custom_message(error.custom_message), data(error.data), message(error.message), - severity(error.severity), - num_params(error.num_params) + severity(error.severity) { if (error.custom_message != NULL) this->custom_message = strdup(error.custom_message); if (error.data != NULL) this->data = strdup(error.data); diff --git a/src/newgrf_config.h b/src/newgrf_config.h --- a/src/newgrf_config.h +++ b/src/newgrf_config.h @@ -107,7 +107,6 @@ struct GRFError : ZeroedMemoryAllocator char *data; ///< Additional data for message and custom_message StringID message; ///< Default message StringID severity; ///< Info / Warning / Error / Fatal - uint8 num_params; ///< Number of additinal parameters for message and custom_message (0, 1 or 2) uint32 param_value[2]; ///< Values of GRF parameters to show for message and custom_message }; diff --git a/src/newgrf_gui.cpp b/src/newgrf_gui.cpp --- a/src/newgrf_gui.cpp +++ b/src/newgrf_gui.cpp @@ -50,7 +50,7 @@ void ShowNewGRFError() SetDParamStr(3, c->filename); SetDParam (4, STR_JUST_RAW_STRING); SetDParamStr(5, c->error->data); - for (uint i = 0; i < c->error->num_params; i++) { + for (uint i = 0; i < lengthof(c->error->param_value); i++) { SetDParam(6 + i, c->error->param_value[i]); } ShowErrorMessage(STR_NEWGRF_ERROR_FATAL_POPUP, INVALID_STRING_ID, WL_CRITICAL); @@ -67,7 +67,7 @@ static void ShowNewGRFInfo(const GRFConf SetDParamStr(2, c->filename); SetDParam (3, STR_JUST_RAW_STRING); SetDParamStr(4, c->error->data); - for (uint i = 0; i < c->error->num_params; i++) { + for (uint i = 0; i < lengthof(c->error->param_value); i++) { SetDParam(5 + i, c->error->param_value[i]); } GetString(message, c->error->custom_message == NULL ? c->error->message : STR_JUST_RAW_STRING, lastof(message));