refactor donations into a seaprate code module, add a basic test

src/CMakeLists.txt

@@ -86,6 +86,7 @@ set (ERESSEA_SRC
   names.c
   lighthouse.c
   reports.c
+  donations.c
   seen.c
   eressea.c
   callback.c
@@ -175,6 +176,7 @@ target_link_libraries(eressea
 )
 
 set(TESTS_SRC
+  donations.test.c
   wormhole.test.c
   alchemy.test.c
   test_eressea.c

src/donations.c

@@ -0,0 +1,60 @@
+#include <kernel/types.h>
+#include "donations.h"
+
+#include <kernel/faction.h>
+#include <kernel/region.h>
+#include <kernel/messages.h>
+
+#include <stdlib.h>
+
+void add_donation(faction * f1, faction * f2, int amount, region * r)
+{
+    donation *sp;
+
+    sp = r->donations_;
+
+    while (sp) {
+        if (sp->f1 == f1 && sp->f2 == f2) {
+            sp->amount += amount;
+            return;
+        }
+        sp = sp->next;
+    }
+
+    sp = calloc(1, sizeof(donation));
+    sp->f1 = f1;
+    sp->f2 = f2;
+    sp->amount = amount;
+    sp->next = r->donations_;
+    r->donations_ = sp;
+}
+
+void free_donations(void) {
+    region *r;
+    for (r = regions; r; r = r->next) {
+        while (r->donations_) {
+            donation *don = r->donations_;
+            r->donations_ = don->next;
+            free(don);
+        }
+    }
+}
+
+void report_donations(void)
+{
+    region *r;
+    for (r = regions; r; r = r->next) {
+        while (r->donations_) {
+            donation *sp = r->donations_;
+            if (sp->amount > 0) {
+                struct message *msg = msg_message("donation",
+                    "from to amount", sp->f1, sp->f2, sp->amount);
+                r_addmessage(r, sp->f1, msg);
+                r_addmessage(r, sp->f2, msg);
+                msg_release(msg);
+            }
+            r->donations_ = sp->next;
+            free(sp);
+        }
+    }
+}

src/donations.h

@@ -0,0 +1,20 @@
+#pragma once
+
+#ifndef H_DONATIONS
+#define H_DONATIONS
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+    struct faction;
+    struct region;
+
+    void add_donation(struct faction * f1, struct faction * f2, int amount, struct region * r);
+    void free_donations(void);
+    void report_donations(void);
+
+#ifdef __cplusplus
+}
+#endif
+#endif

src/donations.test.c

@@ -0,0 +1,29 @@
+#include "donations.h"
+
+#include <kernel/faction.h>
+#include <kernel/region.h>
+#include <tests.h>
+
+#include <CuTest.h>
+
+static void test_add_donation(CuTest *tc) {
+    faction *f1, *f2;
+    region *r;
+
+    test_cleanup();
+    r = test_create_region(0, 0, 0);
+    f1 = test_create_faction(0);
+    f2 = test_create_faction(0);
+    add_donation(f1, f2, 100, r);
+    report_donations();
+    CuAssertPtrNotNull(tc, test_find_messagetype(r->individual_messages->msgs, "donation"));
+    free_donations();
+    test_cleanup();
+}
+
+CuSuite *get_donations_suite(void)
+{
+    CuSuite *suite = CuSuiteNew();
+    SUITE_ADD_TEST(suite, test_add_donation);
+    return suite;
+}

src/economy.c

@@ -23,6 +23,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 #include "alchemy.h"
 #include "direction.h"
+#include "donations.h"
 #include "give.h"
 #include "laws.h"
 #include "randenc.h"
@@ -705,28 +706,6 @@ static int forget_cmd(unit * u, order * ord)
     return 0;
 }
 
-void add_spende(faction * f1, faction * f2, int amount, region * r)
-{
-    donation *sp;
-
-    sp = r->donations;
-
-    while (sp) {
-        if (sp->f1 == f1 && sp->f2 == f2) {
-            sp->amount += amount;
-            return;
-        }
-        sp = sp->next;
-    }
-
-    sp = calloc(1, sizeof(donation));
-    sp->f1 = f1;
-    sp->f2 = f2;
-    sp->amount = amount;
-    sp->next = r->donations;
-    r->donations = sp;
-}
-
 static bool maintain(building * b, bool first)
 /* first==false -> take money from wherever you can */
 {
@@ -886,7 +865,7 @@ static bool maintain(building * b, bool first)
                         cost -= give;
                         fset(ua->faction, FFL_SELECT);
                         if (m->rtype == rsilver)
-                            add_spende(ua->faction, u->faction, give, r);
+                            add_donation(ua->faction, u->faction, give, r);
                         if (cost <= 0)
                             break;
                     }

src/economy.h

@@ -54,7 +54,6 @@ extern "C" {
 
     enum { IC_WORK, IC_ENTERTAIN, IC_TAX, IC_TRADE, IC_TRADETAX, IC_STEAL, IC_MAGIC, IC_LOOT };
     void maintain_buildings(struct region *r, bool crash);
-    void add_spende(struct faction *f1, struct faction *f2, int betrag, struct region *r);
     int make_cmd(struct unit *u, struct order *ord);
     void split_allocations(struct region *r);
     int recruit_archetypes(void);

src/kernel/config.c

@@ -73,6 +73,8 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include <util/umlaut.h>
 #include <util/xml.h>
 
+#include "donations.h"
+
 #ifdef USE_LIBXML2
 /* libxml includes */
 #include <libxml/tree.h>
@@ -1761,6 +1763,7 @@ int markets_module(void)
 void free_gamedata(void)
 {
     int i;
+    free_donations();
     free_units();
     free_regions();
     free_borders();

src/kernel/region.c

@@ -839,12 +839,6 @@ void free_region(region * r)
         free(res);
     }
 
-    while (r->donations) {
-        donation *don = r->donations;
-        r->donations = don->next;
-        free(don);
-    }
-
     while (r->units) {
         unit *u = r->units;
         r->units = u->next;

src/kernel/region.h

@@ -133,7 +133,7 @@ extern "C" {
             struct message_list *msgs;
         } *individual_messages;
         struct attrib *attribs;
-        struct donation *donations;
+        struct donation *donations_;
         const struct terrain_type *terrain;
         struct rawmaterial *resources;
 #ifdef FAST_CONNECT

src/reports.c

@@ -23,6 +23,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include "seen.h"
 #include "travelthru.h"
 #include "lighthouse.h"
+#include "donations.h"
 
 /* kernel includes */
 #include <kernel/alliance.h>
@@ -1602,25 +1603,6 @@ int write_reports(faction * f, time_t ltime)
     return 0;
 }
 
-static void report_donations(void)
-{
-    region *r;
-    for (r = regions; r; r = r->next) {
-        while (r->donations) {
-            donation *sp = r->donations;
-            if (sp->amount > 0) {
-                struct message *msg = msg_message("donation",
-                    "from to amount", sp->f1, sp->f2, sp->amount);
-                r_addmessage(r, sp->f1, msg);
-                r_addmessage(r, sp->f2, msg);
-                msg_release(msg);
-            }
-            r->donations = sp->next;
-            free(sp);
-        }
-    }
-}
-
 static void write_script(FILE * F, const faction * f)
 {
     report_type *rtype;

src/skill.h

@@ -1,6 +1,9 @@
+#pragma once
+
 #ifndef H_SKILL_H
 #define H_SKILL_H
 
+#include <platform.h>
 struct locale;
 
 typedef enum {

src/test_eressea.c

@@ -81,6 +81,7 @@ int RunAllTests(void)
   RUN_TESTS(suite, messages);
   /* gamecode */
   RUN_TESTS(suite, battle);
+  RUN_TESTS(suite, donations);
   RUN_TESTS(suite, travelthru);
   RUN_TESTS(suite, economy);
   RUN_TESTS(suite, give);

src/upkeep.c

@@ -16,6 +16,7 @@
 #include "alchemy.h"
 #include "economy.h"
 #include "monster.h"
+#include "donations.h"
 
 #include <assert.h>
 
@@ -55,7 +56,7 @@ static void help_feed(unit * donor, unit * u, int *need_p)
         change_money(donor, -give);
         change_money(u, give);
         need -= give;
-        add_spende(donor->faction, u->faction, give, donor->region);
+        add_donation(donor->faction, u->faction, give, donor->region);
     }
     *need_p = need;
 }