cpp/openttd-patchpack/source Changeset - r25602:6ef361b0a726

Changeset - r25602:6ef361b0a726

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

Patric Stout - 3 years ago 2021-06-07 21:03:12
truebrain@openttd.org

Codechange: make [Save|Load]Settings() behave more like other Save/Load code (#9335)

Prepare the full description and send it to SlObject. This does
require some code to be able to read to a SLE_VAR_NULL, like strings
etc, as there is no way to know their length beforehand.

3 files changed with 48 insertions and 30 deletions:

src/saveload/saveload.cpp

src/saveload/saveload.h

src/settings.cpp

0 comments (0 inline, 0 general)

src/saveload/saveload.cpp

➞

Show inline comments

@@ @@ -584,7 +584,7 @@ static inline uint SlGetArrayLength(size @@
  * @param conv VarType type of variable that is used for calculating the size
  * @return Return the size of this type in bytes
  */
 uint SlCalcConvMemLen(VarType conv)
+static inline uint SlCalcConvMemLen(VarType conv)
+{
 	static const byte conv_mem_size[] = {1, 1, 1, 2, 2, 4, 4, 8, 8, 0};
 	byte length = GB(conv, 4, 4);
@@ @@ -943,6 +943,9 @@ static void SlString(void *ptr, size_t l @@
 			switch (GetVarMemType(conv)) {
 				default: NOT_REACHED();
 				case SLE_VAR_NULL:
 					SlSkipBytes(len);
 					return;
 				case SLE_VAR_STRB:
 					if (len >= length) {
 						DEBUG(sl, 1, "String length in savegame is bigger than buffer, truncating");
@@ @@ -1007,8 +1010,12 @@ static void SlStdString(void *ptr, VarTy @@
 		case SLA_LOAD_CHECK:
 		case SLA_LOAD: {
 			size_t len = SlReadArrayLength();
 			if (GetVarMemType(conv) == SLE_VAR_NULL) {
 				SlSkipBytes(len);
 				return;
+			}
 			char *buf = AllocaM(char, len + 1);
 			SlCopyBytes(buf, len);
 			buf[len] = '\0'; // properly terminate the string
@@ @@ -1469,6 +1476,8 @@ size_t SlCalcObjMemberLength(const void @@
  */
 [[maybe_unused]] static bool IsVariableSizeRight(const SaveLoad &sld)
+{
 	if (GetVarMemType(sld.conv) == SLE_VAR_NULL) return true;
 	switch (sld.cmd) {
 		case SL_VAR:
 			switch (GetVarMemType(sld.conv)) {

src/saveload/saveload.h

➞

Show inline comments

@@ @@ -893,7 +893,6 @@ void WriteValue(void *ptr, VarType conv, @@
 void SlSetArrayIndex(uint index);
 int SlIterateArray();
 uint SlCalcConvMemLen(VarType conv);
 void SlAutolength(AutolengthProc *proc, void *arg);
 size_t SlGetFieldLength();
 void SlSetLength(size_t length);

src/settings.cpp

➞

Show inline comments

@@ @@ -2015,6 +2015,31 @@ void IConsoleListSettings(const char *pr @@
+}
 /**
  * Get the SaveLoad description for the SettingTable.
  * @param settings SettingDesc struct containing all information.
  * @param is_loading True iff the SaveLoad table is for loading.
  * @return Vector with SaveLoad entries for the SettingTable.
  */
 static std::vector<SaveLoad> GetSettingsDesc(const SettingTable &settings, bool is_loading)
+{
 	std::vector<SaveLoad> saveloads;
 	for (auto &sd : settings) {
 		if (sd->flags & SF_NOT_IN_SAVE) continue;
 		if (is_loading && (sd->flags & SF_NO_NETWORK_SYNC) && _networking && !_network_server) {
 			/* We don't want to read this setting, so we do need to skip over it. */
 			saveloads.push_back({sd->save.cmd, GetVarFileType(sd->save.conv) | SLE_VAR_NULL, sd->save.length, sd->save.version_from, sd->save.version_to, 0, nullptr, 0});
 			continue;
+		}
 		saveloads.push_back(sd->save);
+	}
 	return saveloads;
+}
 /**
  * Save and load handler for settings
  * @param settings SettingDesc struct containing all information
  * @param object can be either nullptr in which case we load global variables or
@@ @@ -2022,20 +2047,18 @@ void IConsoleListSettings(const char *pr @@
  */
 static void LoadSettings(const SettingTable &settings, void *object)
+{
 	for (auto &osd : settings) {
 		if (osd->flags & SF_NOT_IN_SAVE) continue;
 	const std::vector<SaveLoad> slt = GetSettingsDesc(settings, true);
 	SlObject(object, slt);
 		SaveLoad sl = osd->save;
 		if ((osd->flags & SF_NO_NETWORK_SYNC) && _networking && !_network_server) {
 			/* We don't want to read this setting, so we do need to skip over it. */
 			sl = SLE_NULL(static_cast<uint16>(SlCalcConvMemLen(osd->save.conv) * osd->save.length));
+		}
 	/* Ensure all IntSettings are valid (min/max could have changed between versions etc). */
 	for (auto &sd : settings) {
 		if (sd->flags & SF_NOT_IN_SAVE) continue;
 		if ((sd->flags & SF_NO_NETWORK_SYNC) && _networking && !_network_server) continue;
 		if (!SlIsObjectCurrentlyValid(sd->save.version_from, sd->save.version_to)) continue;
 		void *ptr = GetVariableAddress(object, sl);
 		if (!SlObjectMember(ptr, sl)) continue;
 		if (osd->IsIntSetting()) {
 			const IntSettingDesc *int_setting = osd->AsIntSetting();
 		if (sd->IsIntSetting()) {
 			const IntSettingDesc *int_setting = sd->AsIntSetting();
 			int_setting->MakeValueValidAndWrite(object, int_setting->Read(object));
+		}
+	}
@@ @@ -2049,22 +2072,9 @@ static void LoadSettings(const SettingTa @@
  */
 static void SaveSettings(const SettingTable &settings, void *object)
+{
 	/* We need to write the CH_RIFF header, but unfortunately can't call
 	 * SlCalcLength() because we have a different format. So do this manually */
 	size_t length = 0;
 	for (auto &sd : settings) {
 		if (sd->flags & SF_NOT_IN_SAVE) continue;
 	const std::vector<SaveLoad> slt = GetSettingsDesc(settings, false);
 		length += SlCalcObjMemberLength(object, sd->save);
+	}
 	SlSetLength(length);
 	for (auto &sd : settings) {
 		if (sd->flags & SF_NOT_IN_SAVE) continue;
 		void *ptr = GetVariableAddress(object, sd->save);
 		SlObjectMember(ptr, sd->save);
+	}
 	SlObject(object, slt);
+}
 static void Load_OPTS()

0 comments (0 inline, 0 general)