Mercurial > hg > openttd
changeset 4278:a8f33594ffbc draft
(svn r5908) - Codechange (r5903): Move the retrieval of the new value of a patch to console_cmds.c, so there is no need for that function in settings.c
author | Darkvater <Darkvater@openttd.org> |
---|---|
date | Tue, 15 Aug 2006 07:37:01 +0000 |
parents | c440e1d4f31a |
children | 41f710dfab62 |
files | console_cmds.c settings.c settings.h |
diffstat | 3 files changed, 7 insertions(+), 12 deletions(-) [+] |
line wrap: on
line diff
--- a/console_cmds.c +++ b/console_cmds.c @@ -1342,7 +1342,8 @@ if (argc == 2) { IConsoleGetPatchSetting(argv[1]); } else { - IConsoleSetPatchSetting(argv[1], argv[2]); + int32 val; + if (GetArgumentInteger(&val, argv[2])) IConsoleSetPatchSetting(argv[1], val); } return true;
--- a/settings.c +++ b/settings.c @@ -1597,13 +1597,9 @@ } /* Those 2 functions need to be here, else we have to make some stuff non-static - * and besides, it is also better to keep stuff like this at the same place - * XXX - Perhaps back to console[_cmds].c? They are console functions after all */ -extern bool GetArgumentInteger(uint32 *value, const char *arg); - -void IConsoleSetPatchSetting(const char *name, const char *value) + * and besides, it is also better to keep stuff like this at the same place */ +void IConsoleSetPatchSetting(const char *name, int32 value) { - int32 val; uint index; const SettingDesc *sd = GetPatchFromName(name, &index); const Patches *patches_ptr; @@ -1614,13 +1610,11 @@ return; } - if (!GetArgumentInteger(&val, value)) return; - patches_ptr = (_game_mode == GM_MENU) ? &_patches_newgame : &_patches; ptr = ini_get_variable(&sd->save, patches_ptr); - SetPatchValue(index, patches_ptr, val); - if (sd->desc.proc != NULL) sd->desc.proc(val); + SetPatchValue(index, patches_ptr, value); + if (sd->desc.proc != NULL) sd->desc.proc(value); } void IConsoleGetPatchSetting(const char *name)
--- a/settings.h +++ b/settings.h @@ -77,7 +77,7 @@ /** The patch values that are used for new games and/or modified in config file */ extern Patches _patches_newgame; -void IConsoleSetPatchSetting(const char *name, const char *value); +void IConsoleSetPatchSetting(const char *name, int32 value); void IConsoleGetPatchSetting(const char *name); const SettingDesc *GetPatchFromName(const char *name, uint *i); void SetPatchValue(uint index, const Patches *object, int32 value);