diff options
author | yexo <yexo@openttd.org> | 2010-01-29 21:06:35 +0000 |
---|---|---|
committer | yexo <yexo@openttd.org> | 2010-01-29 21:06:35 +0000 |
commit | dc714960e3699144fcd39a13460b0a8f9fd0ca4d (patch) | |
tree | 0ee3365a9a70d00ac7ece7d5d12933f8daadc8c1 | |
parent | b0349704b0ae7ffc3f62a38b9e93e0bb65a7b5a4 (diff) | |
download | openttd-dc714960e3699144fcd39a13460b0a8f9fd0ca4d.tar.xz |
(svn r18952) -Fix (r18950): loading recent (=newer then title game) savegames failed
-rw-r--r-- | src/saveload/saveload.h | 10 | ||||
-rw-r--r-- | src/settings.cpp | 2 |
2 files changed, 11 insertions, 1 deletions
diff --git a/src/saveload/saveload.h b/src/saveload/saveload.h index b5218456c..c3864c04f 100644 --- a/src/saveload/saveload.h +++ b/src/saveload/saveload.h @@ -295,6 +295,16 @@ static inline VarType GetVarFileType(VarType type) return type & 0xF; // GB(type, 0, 4); } +/** + * Check if the given saveload type is a numeric type. + * @param conv the type to check + * @return True if it's a numeric type. + */ +static bool IsNumericType(VarType conv) +{ + return GetVarMemType(conv) <= SLE_VAR_U64; +} + /** Get the address of the variable. Which one to pick depends on the object * pointer. If it is NULL we are dealing with global variables so the address * is taken. If non-null only the offset is stored in the union and we need diff --git a/src/settings.cpp b/src/settings.cpp index 3150ebc17..9af46b4be 100644 --- a/src/settings.cpp +++ b/src/settings.cpp @@ -1784,7 +1784,7 @@ static void LoadSettings(const SettingDesc *osd, void *object) void *ptr = GetVariableAddress(object, sld); if (!SlObjectMember(ptr, sld)) continue; - Write_ValidateSetting(ptr, osd, ReadValue(ptr, sld->conv)); + if (IsNumericType(sld->conv)) Write_ValidateSetting(ptr, osd, ReadValue(ptr, sld->conv)); } } |