free configuration data (valgrind calls it still-reachable leaks).

src/eressea.c

@@ -14,23 +14,27 @@
 #include <triggers/triggers.h>
 #include <util/language.h>
 #include <util/functions.h>
-#include <kernel/xmlreader.h>
-#include <kernel/item.h>
-#include <kernel/curse.h>
 #include <kernel/building.h>
+#include <kernel/curse.h>
+#include <kernel/equipment.h>
+#include <kernel/item.h>
+#include <kernel/xmlreader.h>
 #include <modules/gmcmd.h>
 #include <modules/xmas.h>
 #include <items/itemtypes.h>
 #include <attributes/attributes.h>
+#include <util/message.h>
 #include <races/races.h>
+
 #include "chaos.h"
-#include "report.h"
-#include "items.h"
 #include "creport.h"
+#include "items.h"
 #include "jsreport.h"
 #include "names.h"
-#include "wormhole.h"
+#include "report.h"
+#include "reports.h"
 #include "spells.h"
+#include "wormhole.h"
 
 void game_done(void)
 {
@@ -51,6 +55,9 @@ void game_done(void)
     free_functions();
     free_config();
     free_locales();
+    message_done();
+    equipment_done();
+    reports_done();
     curses_done();
     kernel_done();
 }

src/kernel/config.c

@@ -1080,7 +1080,6 @@ void free_gamedata(void)
 {
     int i;
     free_donations();
-    free_equipment();
 
     for (i = 0; i != MAXLOCALES; ++i) {
         if (defaults[i]) {
@@ -1094,15 +1093,15 @@ void free_gamedata(void)
     free_borders();
     free_alliances();
 
+    while (global.attribs) {
+        a_remove(&global.attribs, global.attribs);
+    }
+
     while (planes) {
         plane *pl = planes;
         planes = planes->next;
         free_plane(pl);
     }
-
-    while (global.attribs) {
-        a_remove(&global.attribs, global.attribs);
-    }
 }
 
 const char * game_name(void) {

src/kernel/equipment.c

@@ -224,7 +224,7 @@ void equip_items(struct item **items, const struct equipment *eq)
     }
 }
 
-void free_equipment(void) {
+void equipment_done(void) {
     equipment **eqp = &equipment_sets;
     while (*eqp) {
         equipment *eq = *eqp;

src/kernel/equipment.h

@@ -54,6 +54,8 @@ extern "C" {
         void(*callback) (const struct equipment *, struct unit *);
     } equipment;
 
+    void equipment_done(void);
+
     struct equipment *create_equipment(const char *eqname);
     struct equipment *get_equipment(const char *eqname);
 
@@ -75,7 +77,6 @@ extern "C" {
         int mask);
     void equip_items(struct item **items, const struct equipment *eq);
 
-    void free_equipment(void);
 #ifdef __cplusplus
 }
 #endif

src/report.c

@@ -2547,13 +2547,6 @@ static void update_find(void)
     initial = false;
 }
 
-bool kann_finden(faction * f1, faction * f2)
-{
-    update_find();
-    return (bool)(can_find(f1, f2) != NULL);
-}
-
-/******* end summary ******/
 
 void register_nr(void)
 {

src/reports.c

@@ -1094,6 +1094,15 @@ void register_reporttype(const char *extension, report_fun write, int flag)
     report_types = type;
 }
 
+void reports_done(void) {
+    report_type **rtp = &report_types;
+    while (*rtp) {
+        report_type *rt = *rtp;
+        *rtp = rt->next;
+        free(rt);
+    }
+}
+
 static quicklist *get_regions_distance(region * root, int radius)
 {
     quicklist *ql, *rlist = NULL;

src/reports.h

@@ -44,8 +44,8 @@ extern "C" {
     extern bool noreports;
     extern const char *visibility[];
 
-    /* kann_finden speedups */
+    void reports_done(void);
-    bool kann_finden(struct faction *f1, struct faction *f2);
+
     struct unit *can_find(struct faction *, struct faction *);
 
     /* funktionen zum schreiben eines reports */

src/util/message.c

@@ -32,7 +32,7 @@ const char *mt_name(const message_type * mtype)
     return mtype->name;
 }
 
-arg_type *argtypes = NULL;
+static arg_type *argtypes = NULL;
 
 void
 register_argtype(const char *name, void(*free_arg) (variant),
@@ -246,3 +246,12 @@ struct message *msg_addref(struct message *msg)
     ++msg->refcount;
     return msg;
 }
+
+void message_done(void) {
+    arg_type **atp = &argtypes;
+    while (*atp) {
+        arg_type *at = *atp;
+        *atp = at->next;
+        free(at);
+    }
+}

src/util/message.h

@@ -40,8 +40,10 @@ extern "C" {
         int refcount;
     } message;
 
+    void message_done(void);
+
     void mt_clear(void);
-    struct message_type *mt_new(const char *name, const char **args);
+    struct message_type *mt_new(const char *name, const char *args[]);
     struct message_type *mt_new_va(const char *name, ...);
     /* mt_new("simple_sentence", "subject:string", "predicate:string",
      *        "object:string", "lang:locale", NULL); */