openttd-ottdgrfs: comparison src/settings.cpp

equal deleted inserted replaced

-:96b4b92b86ae
+:9c3129cb019b
 #include "newgrf.h"
 #include "newgrf_config.h"
 #include "genworld.h"
 #include "date.h"
 #include "rail.h"
+#include "helpers.hpp"
 #ifdef WITH_FREETYPE
 #include "gfx.h"
 #include "fontcache.h"
 #endif
 static SettingsMemoryPool *pool_new(uint minsize)
 {
 	SettingsMemoryPool *p;
 	if (minsize < 4096 - 12) minsize = 4096 - 12;
-	p = malloc(sizeof(SettingsMemoryPool) - 1 + minsize);
+	p = (SettingsMemoryPool*)malloc(sizeof(SettingsMemoryPool) - 1 + minsize);
 	p->pos = 0;
 	p->size = minsize;
 	p->next = NULL;
 	return p;
 }
 	return p->mem + pos;
 }
 static void *pool_strdup(SettingsMemoryPool **pool, const char *mem, uint size)
 {
-	byte *p = pool_alloc(pool, size + 1);
+	byte *p = (byte*)pool_alloc(pool, size + 1);
 	p[size] = 0;
 	memcpy(p, mem, size);
 	return p;
 }
 }
 // allocate an ini group object
 static IniGroup *ini_group_alloc(IniFile *ini, const char *grpt, int len)
 {
-	IniGroup *grp = pool_alloc(&ini->pool, sizeof(IniGroup));
+	IniGroup *grp = (IniGroup*)pool_alloc(&ini->pool, sizeof(IniGroup));
 	grp->ini = ini;
-	grp->name = pool_strdup(&ini->pool, grpt, len);
+	grp->name = (char*)pool_strdup(&ini->pool, grpt, len);
 	if (!strcmp(grp->name, "newgrf") || !strcmp(grp->name, "servers") || !strcmp(grp->name, "bans")) {
 		grp->type = IGT_LIST;
 	} else {
 		grp->type = IGT_VARIABLES;
 	}
 	return grp;
 }
 static IniItem *ini_item_alloc(IniGroup *group, const char *name, int len)
 {
-	IniItem *item = pool_alloc(&group->ini->pool, sizeof(IniItem));
+	IniItem *item = (IniItem*)pool_alloc(&group->ini->pool, sizeof(IniItem));
-	item->name = pool_strdup(&group->ini->pool, name, len);
+	item->name = (char*)pool_strdup(&group->ini->pool, name, len);
 	item->next = NULL;
 	item->comment = NULL;
 	item->value = NULL;
 	*group->last_item = item;
 	group->last_item = &item->next;
 			uint pos;
 			// add to comment
 			if (ns > a) {
 				a = max(a, 128);
 				do a*=2; while (a < ns);
-				comment = realloc(comment, comment_alloc = a);
+				ReallocT(&comment, comment_alloc = a);
 			}
 			pos = comment_size;
 			comment_size += (e - s + 1);
 			comment[pos + e - s] = '\n'; // comment newline
 			memcpy(comment + pos, s, e - s); // copy comment contents
 				e--;
 			}
 			s++; // skip [
 			group = ini_group_alloc(ini, s, e - s);
 			if (comment_size) {
-				group->comment = pool_strdup(&ini->pool, comment, comment_size);
+				group->comment = (char*)pool_strdup(&ini->pool, comment, comment_size);
 				comment_size = 0;
 			}
 		} else if (group) {
 			// find end of keyname
 			for (t = s; *t != '\0' && *t != '=' && *t != '\t' && *t != ' '; t++);
 			// it's an item in an existing group
 			item = ini_item_alloc(group, s, t-s);
 			if (comment_size) {
-				item->comment = pool_strdup(&ini->pool, comment, comment_size);
+				item->comment = (char*)pool_strdup(&ini->pool, comment, comment_size);
 				comment_size = 0;
 			}
 			// find start of parameter
 			while (*t == '=' || *t == ' ' || *t == '\t') t++;
 			// remove ending quotation marks
 			e = t + strlen(t);
 			if (e > t && e[-1] == '\"') e--;
 			*e = '\0';
-			item->value = pool_strdup(&ini->pool, t, e - t);
+			item->value = (char*)pool_strdup(&ini->pool, t, e - t);
 		} else {
 			// it's an orphan item
 			ShowInfoF("ini: '%s' outside of group", buffer);
 		}
 	}
 	if (comment_size > 0) {
-		ini->comment = pool_strdup(&ini->pool, comment, comment_size);
+		ini->comment = (char*)pool_strdup(&ini->pool, comment, comment_size);
 		comment_size = 0;
 	}
 	free(comment);
 	fclose(in);
 		if (!memcmp(group->name, name, len) && group->name[len] == 0)
 			return group;
 	// otherwise make a new one
 	group = ini_group_alloc(ini, name, len);
-	group->comment = pool_strdup(&ini->pool, "\n", 1);
+	group->comment = (char*)pool_strdup(&ini->pool, "\n", 1);
 	return group;
 }
 // lookup an item or make a new one
 static IniItem *ini_getitem(IniGroup *group, const char *name, bool create)
 		ShowInfoF("ini: invalid setting value '%s' for '%s'", str, desc->name);
 		break;
 	case SDT_STRING:
 	case SDT_INTLIST: return str;
+	default: break;
 	}
 	return NULL;
 }
 		case SDT_STRING:
 			switch (GetVarMemType(sld->conv)) {
 				case SLE_VAR_STRB:
 				case SLE_VAR_STRBQ:
-					if (p != NULL) ttd_strlcpy((char*)ptr, p, sld->length);
+					if (p != NULL) ttd_strlcpy((char*)ptr, (const char*)p, sld->length);
 					break;
 				case SLE_VAR_STR:
 				case SLE_VAR_STRQ:
 					if (p != NULL) {
 						free(*(char**)ptr);
 				default: NOT_REACHED(); break;
 			}
 			break;
 		case SDT_INTLIST: {
-			if (!load_intlist(p, ptr, sld->length, GetVarMemType(sld->conv)))
+			if (!load_intlist((const char*)p, ptr, sld->length, GetVarMemType(sld->conv)))
 				ShowInfoF("ini: error in array '%s'", sdb->name);
 			break;
 		}
 		default: NOT_REACHED(); break;
 		}
 			case SDT_NUMX:
 			case SDT_ONEOFMANY:
 			case SDT_MANYOFMANY:
 				switch (GetVarMemType(sld->conv)) {
 				case SLE_VAR_BL:
-					if (*(bool*)ptr == (bool)(unsigned long)p) continue;
+					if (*(bool*)ptr == (p != NULL)) continue;
 					break;
 				case SLE_VAR_I8:
 				case SLE_VAR_U8:
 					if (*(byte*)ptr == (byte)(unsigned long)p) continue;
 					break;
 			break;
 		default: NOT_REACHED();
 		}
 		/* The value is different, that means we have to write it to the ini */
-		item->value = pool_strdup(&ini->pool, buf, strlen(buf));
+		item->value = (char*)pool_strdup(&ini->pool, buf, strlen(buf));
 	}
 }
 /** Loads all items from a 'grpname' section into a list
 * The list parameter can be a NULL pointer, in this case nothing will be
 * internal structure somewhat so it needs a little looking. There are _NULL()
 * macros as well, these fill up space so you can add more patches there (in
 * place) and you DON'T have to increase the savegame version. */
 #define NSD_GENERAL(name, def, cmd, guiflags, min, max, interval, many, str, proc)\
-	{name, (const void*)(def), cmd, guiflags, min, max, interval, many, str, proc}
+	{name, (const void*)(def), {cmd}, {guiflags}, min, max, interval, many, str, proc}
 /* Macros for various objects to go in the configuration file.
 * This section is for global variables */
 #define SDTG_GENERAL(name, sdt_cmd, sle_cmd, type, flags, guiflags, var, length, def, min, max, interval, full, str, proc, from, to)\
-{NSD_GENERAL(name, def, sdt_cmd, guiflags, min, max, interval, full, str, proc), SLEG_GENERAL(sle_cmd, var, type | flags, length, from, to)}
+	{NSD_GENERAL(name, def, sdt_cmd, guiflags, min, max, interval, full, str, proc), SLEG_GENERAL(sle_cmd, var, type | flags, length, from, to)}
 #define SDTG_CONDVAR(name, type, flags, guiflags, var, def, min, max, interval, str, proc, from, to)\
 	SDTG_GENERAL(name, SDT_NUMX, SL_VAR, type, flags, guiflags, var, 0, def, min, max, interval, NULL, str, proc, from, to)
 #define SDTG_VAR(name, type, flags, guiflags, var, def, min, max, interval, str, proc)\
 	SDTG_CONDVAR(name, type, flags, guiflags, var, def, min, max, interval, str, proc, 0, SL_MAX_VERSION)
 	SDTG_GENERAL(name, SDT_MANYOFMANY, SL_VAR, type, flags, guiflags, var, 0, def, 0, 0, 0, full, str, proc, from, to)
 #define SDTG_MMANY(name, type, flags, guiflags, var, def, full, str, proc)\
 	SDTG_CONDMMANY(name, type, flags, guiflags, var, def, full, str, proc, 0, SL_MAX_VERSION)
 #define SDTG_CONDNULL(length, from, to)\
-	{{"", NULL, 0, 0, 0, 0, 0, NULL, STR_NULL, NULL}, SLEG_CONDNULL(length, from, to)}
+	{{"", NULL, {0}, {0}, 0, 0, 0, NULL, STR_NULL, NULL}, SLEG_CONDNULL(length, from, to)}
-#define SDTG_END() {{NULL, NULL, 0, 0, 0, 0, 0, NULL, STR_NULL, NULL}, SLEG_END()}
+#define SDTG_END() {{NULL, NULL, {0}, {0}, 0, 0, 0, NULL, STR_NULL, NULL}, SLEG_END()}
 /* Macros for various objects to go in the configuration file.
 * This section is for structures where their various members are saved */
 #define SDT_GENERAL(name, sdt_cmd, sle_cmd, type, flags, guiflags, base, var, length, def, min, max, interval, full, str, proc, from, to)\
 	{NSD_GENERAL(name, def, sdt_cmd, guiflags, min, max, interval, full, str, proc), SLE_GENERAL(sle_cmd, base, var, type | flags, length, from, to)}
 	SDT_GENERAL(#var, SDT_MANYOFMANY, SL_VAR, type, flags, guiflags, base, var, 1, def, 0, 0, 0, full, str, proc, from, to)
 #define SDT_MMANY(base, var, type, flags, guiflags, def, full, str, proc)\
 	SDT_CONDMMANY(base, var, type, 0, SL_MAX_VERSION, flags, guiflags, def, full, str, proc)
 #define SDT_CONDNULL(length, from, to)\
-	{{"", NULL, 0, 0, 0, 0, 0, NULL, STR_NULL, NULL}, SLE_CONDNULL(length, from, to)}
+	{{"", NULL, {0}, {0}, 0, 0, 0, NULL, STR_NULL, NULL}, SLE_CONDNULL(length, from, to)}
-#define SDT_END() {{NULL, NULL, 0, 0, 0, 0, 0, NULL, STR_NULL, NULL}, SLE_END()}
+#define SDT_END() {{NULL, NULL, {0}, {0}, 0, 0, 0, NULL, STR_NULL, NULL}, SLE_END()}
 /* Shortcuts for macros below. Logically if we don't save the value
 * we also don't sync it in a network game */
 #define S SLF_SAVE_NO | SLF_NETWORK_NO
 #define NS SLF_SAVE_NO
 	GRFConfig **curr = &first;
 	if (group == NULL) return NULL;
 	for (item = group->item; item != NULL; item = item->next) {
-		GRFConfig *c = calloc(1, sizeof(*c));
+		GRFConfig *c;
+		CallocT(&c, 1);
 		c->filename = strdup(item->name);
 		/* Parse parameters */
 		if (*item->value != '\0') {
 			c->num_params = parse_intlist(item->value, (int*)c->param, lengthof(c->param));
 	for (c = list; c != NULL; c = c->next) {
 		char params[512];
 		GRFBuildParamList(params, c, lastof(params));
 		*item = ini_item_alloc(group, c->filename, strlen(c->filename));
-		(*item)->value = pool_strdup(&ini->pool, params, strlen(params));
+		(*item)->value = (char*)pool_strdup(&ini->pool, params, strlen(params));
 		item = &(*item)->next;
 	}
 }
 /* Common handler for saving/loading variables to the configuration file */
 	 * them with the default values just as it was in the old days.
 	 * Also new games need this copying-over */
 	_patches = _patches_newgame; /* backwards compatibility */
 }
-const ChunkHandler _setting_chunk_handlers[] = {
+extern const ChunkHandler _setting_chunk_handlers[] = {
 	{ 'OPTS', Save_OPTS, Load_OPTS, CH_RIFF},
 	{ 'PATS', Save_PATS, Load_PATS, CH_RIFF | CH_LAST},
 };
 static bool IsSignedVarMemType(VarType vt)

changeset 5838	9c3129cb019b
parent 5835	e0ff603ae0b7
child 5852	cb3f71b16e1a