Was mir noch bei den Monstern aufgefallen ist:

In *get_money_for_dragon:

        if (u2 != u && is_guard(u2, GUARD_TAX) && u->faction!=u2->faction) {
            /*In E3 + E4 etwas problematisch, da der Regionsbesitzer immer bewacht. Der Drache greift also immer die Burg an!*/

Esetzen durch

        if (u2 != u && is_guard(u2, GUARD_TAX) && u->faction!=u2->faction && !inside_building(u2)) {

um endlich das Regionsbesitzer Problem zu lösen. Loot geht bei Fliegern auch wenn nicht Flieger bewachen, ein Angriff auf Bewacher in einer Burg ist also nicht nötig. Könnte auch ganz raus.

Der check ob das Monster bewacht sollte vieleicht nach *monster_attack

Sowie

    keyword_t kwd = keyword_disabled(K_TAX) ? K_LOOT : K_TAX;
    return create_order(kwd, default_locale, NULL);

ersetzen durch

return create_order(K_LOOT, default_locale, NULL);

wie auch schon weiter oben im code. Es gibt keinen Grund das je nach Laune der Drachen in E2 mal Tax and mal Loot benutzt wird.

monster_is_waiting

bool monster_is_waiting(const unit * u) { if (fval(u, UFL_ISNEW | UFL_MOVED)) return true; return false; }

Sollte eher auf ISNEW und auf Turn < 30 oder so Checken. Da Monster die nicht bewachen nicht angreifen, ist so schon eine Runde Vorwarnzeit eingebaut. Im moment sind es 2 da waiting Monster auch kein Bewache setzten. Auch sorgt dies im Moment dafür das sich Drachen immer ein Feld bewegen, dann eine Runde stehen bleiben und nix machen, dann wieder ein Feld bewegen (auch auf dem Ozean) usw.

In eaten_by_monster wird ein Multiplikator für die Weltgröße benutzt:

multi = RESOURCE_QUANTITY * newterrain(T_PLAIN)->size / 10000.0;

in

absorbed_by_monster

und

scared_by_monster

nicht. Ebendso nicht in

int income(const unit * u) (in der economy.c (Da das glaube ich nur für Monster genutzt wird, warum ist das nicht bei den Monstern) Das sollte auch noch geändert werden.