Also, erst einmal möchte ich klagen, weil der Titel dieses Bugs nichtssagend ist. Zweitens: Du hast natürlich total Recht, die Doppelung mit (land,sail) ist ungeschickt. Die ist noch aus einer Zeit, als es quasi zwei Klassen von Schiffen gab: Boote, und Nicht-Boote.

Spricht den etwas dagegen das Boote in Gängen anlanden können? Ist ja sonst eine Kleinigkeit. Die Flags an den Regionen sollte man dann auch gleich löschen.

Aktuelle Implementation in move.c ist:

1. Das Terrain muss das SAIL_INTO flag haben, und
2. Das Terrain muss in der coasts Liste des Schiffs sein.

Ich habe allerdings das dunkle Gefühl, dass Piraterie das flag benutzt, ohne sich um den Schiffstypen zu scheren?

Piraterie ruft einfach nur move_cmd auf, und das reduziert das ganze auf move_cmd. Damit geht das am Ende alles durch den selben Codepath, in check_ship_allowed.

Ich glaube, dass das korrekt ist. In Hallen und Gänge soll man nicht segeln können. Zum Code: Das sieht mir nach einer Optimierung aus. Es ist weitaus schneller, auf ein Flag wie SAIL_INTO zu checken, als eine Liste von Terrains zu durchsuchen. Wenn es also ein Terrain gibt, in dem überhaupt kein Schiff landen darf, dann kann man das so markieren. Wenn ich das Flag von dem Terrain entferne, ändert das am Resultat nichts.

Natürlich ist das irgendwo doppelt gemoppelt, und kann zu Verwirrung führen. Schöner wäre, wenn das nur über die ship->coast Liste läuft. Das SAIL_INTO Flag ist, das sieht man schon am Namen, noch aus Atlantis, als für alle Schiffe die gleichen Regeln galten.

Dann habe ich mich noch gefragt, warum "ocean" in ship_type.coasts aufgelistet wird. Jedes Schiff darf ja wohl in einen Ozean (SEA_REGION) fahren? Aber ich glaube, es gab da mal einen Plan, verschiedene Typen von Ozean zu implementieren, stillere Gewässer, in denen alle Schiffstypen erlaubt sind, und stürmischen Ozean, wo man ein spezielles Schiff braucht. Daraus ist nichts geworden, und dieses Feature ist damit tot und irreführend.

Was ich hier tun möchte:

1. Das Flag aus dem Code entfernen
2. "ocean" aus ship_type.coasts entfernen, durch einen Test auf SEA_REGION ersetzen.

Kannst du das in einem separaten Bug machen? Das hat mit Spezialterrain ja eher nichts mehr zu tun.

Für Betreten ist SEA_REGION übrigens nicht gedacht, das sind WALK_INTO bzw. FLY_INTO. Es gibt da eine Menge Flags, die leider alle nicht dokumentiert sind. Also, noch ein Punkt für die Liste:

1. Das Flag aus dem Code entfernen
2. "ocean" aus ship_type.coasts entfernen, durch einen Test auf SEA_REGION ersetzen.
3. Die Terrain-Flags dokumentieren

PR https://github.com/eressea/server/pull/515

Der PR ist schon vor einer Weile akzeptiert worden. Die Probleme mit Piraterie existieren wahrscheinich noch immer, sollten aber einen eigenen Bug bekommen.