Update terrains.json - #2
Conversation
Feuerwand zu Landregion gemacht, damit Schiffe in der Nähe nicht mehr abtreiben können.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c0d1f612b0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| }, | ||
| "firewall": { | ||
| "flags": [ "forbidden" ] | ||
| "flags": [ "forbidden", "land" ] |
There was a problem hiding this comment.
When a ship is adjacent to a firewall terrain, this flag change still does not prevent storm drift: the drift check skips every FORBIDDEN_REGION neighbor before testing SEA_REGION (src/move.c:2006-2009), so a forbidden+land firewall is ignored rather than treated like coastline. In an otherwise all-sea neighborhood, storm remains true and rnext can still be selected, so ships can continue to drift near firewalls unless the storm neighbor test is changed to make forbidden land break the open-sea condition.
Useful? React with 👍 / 👎.
1 participant
Feuerwand zu Landregion gemacht, damit Schiffe in der Nähe nicht mehr abtreiben können.