Es stimmt, es werden jetzt alle sichtbaren Einheiten im CR, aber nicht im NR, angezeigt. Ich kann ausschließen, dass getarnte Einheiten angezeigt werden.

Das bisherige Verhalten war was genau? Ich glaube, es wurden nur Schiffe, Gebäude und deren Insassen angezeigt, nicht EInheiten, die einfach so in den überwachten Regionen waren.

Ich finde das neue Verhalten logischer als das alte. Wieso sollten nur Einheiten in Gebäuden sichtbar sein? Das bisherige Verhalten kommt mir eher wie ein Bug vor: Man wollte Schiffe mit Besatzung sichtbar machen, hat aber bei der Gelegenheit aus Versehen auch solche in Gebäuden sichtbar gemacht.

Ich finde nicht, dass man es auf Monstereinheiten begrenzen sollte. Das ergibt keinen Sinn.

Ich sehe drei Möglichkeiten:

• Bisheriges Verhalten wiederherstellen
• Bisheriges Verhalten korrigieren: Nur Schiffe und deren Einheiten werden gesehen, aber keine Gebäude
• Jetziges Verhalten beibehalten.

Mir gefällt das neue Verhalten am besten. Ich finde, die Leuchttürme können ein bisschen Aufwertung vertragen. Und es spart Einheiten. Man kann jetzt Regionen aus der Ferne überwachen. Man sollte allerdings die Spieler darauf hinweisen, falls das Verhalten so bleibt.

Ich glaube nicht, dass hier enorme Eile geboten ist. Niemand wird das Spiel verlieren, oder aussteigen, weil so ein Fehler existiert.

Zur Lösung: Ich finde, Drachen und Seeschlangen im Ozean sollte man von einem LT sehen können, die sind ja fast so groß wie Schiffe. Auch spielerisch macht das Sinn. Das man mit einem Leuchtturm an Land Burgen oder Einheiten erspähen kann, finde ich allerdings auch verkehrt, da bauen sonst irgendwann Spieler ihre Beobachtungstürme irgendwo, wo es nicht einmal ein Meer gibt. An Land kann man sich immer in einem Haus oder Wäldchen verstecken, oder anderweitig von der Landschaft verborgen sein.

Ich schaue mal, was hier verkehrt ist, und stelle evtl. einfach das alte Verhalten wieder her.

Es wäre mir sehr geholfen worden, wenn ich die Nummer einer fälschlich gesehenen Einheit gesagt bekommen hätte.

Beispiel, Partei ufo, Report 1055, E2: Die Einheit 4xkv der Partei yido in Mystland bewacht die Region, steht in keiner Burg, und wird vom Leuchtturm in Asghard gesehen, wo meine Einheit 75se im Leuchtturm 3887 steht. Ich habe keine eigenen Einheiten in Mystland. Im NR erscheint die Einheit nicht.

Hier wird cansee_ex aufgerufen, was den seen_mode der Region gar nicht benutzt?

Die Einheit ist laut cansee sichtbar, weil sie bewacht. Es wäre immer noch prima, wenn es ein Beispiel gäbe, wo die Einheit nicht bewacht...

Der Kommentar widerspricht, und sagt, man braucht für Bewacher eine Einheit in der Region:

/* simple visibility, just gotta have a viewer in the region to see 'em */
if (leftship(u) || is_guard(u) || usiege(u) || u->building || u->ship) {
    return true;
}

Aber der seen_mode wird wie gesagt gar nicht beachtet. Warum ist das im NR überhaupt anders? Das sollte ich mir mal ansehen für die Region.

Im NR steht:

        if (stealthmod > INT_MIN && r->seen.mode >= seen_unit) {
            if (u->faction == f || cansee_ex(f, r, u, stealthmod, r->seen.mode)) {
                nr_unit(out, f, u, 4, r->seen.mode);
            }
        }

Da ist die Logik also ganz anders. Es ist zum kotzen, dass hier für jeden Report-Typ so viel Code dupliziert wird.

Offensichtlich ist diese Schachtel-Bedingung zu kompliziert, um mitten im Code zu stehen (und doppelt ist sie auch noch), und gehört in eine eigene Funktion in reports.c, die dann an beiden Stellen benutzt wird. Das ist zumindest ein Schritt in Richtung Besserung, und die Funktion kann dann auch in Isolation getestet werden.

Ich hab's. Tests habe ich noch keine geschrieben, aber am Beispiel von meinem Report aus Runde 1055 scheint es jetzt richtig zu sein.