(svn r7522) -Codechange: Have CopyGRFConfigList clear the destination list before assigning
values to it. This cuts down on memleaks which could exist when the function was
not used carefully.
--- a/newgrf_config.c Thu Dec 21 00:13:56 2006 +0000
+++ b/newgrf_config.c Thu Dec 21 10:09:43 2006 +0000
@@ -115,6 +115,8 @@
{
GRFConfig *c;
+ /* Clear destination as it will be overwritten */
+ ClearGRFConfigList(dst);
for (; src != NULL; src = src->next) {
c = calloc(1, sizeof(*c));
*c = *src;
@@ -135,8 +137,6 @@
{
GRFConfig **c = &_grfconfig;
- ClearGRFConfigList(c);
-
if (defaults) c = CopyGRFConfigList(c, _grfconfig_newgame);
CopyGRFConfigList(c, _grfconfig_static);
}