Drop checks whether unit should be in game

• 報告者: cazfi
• 担当者: cazfi
• チケットの種類: パッチ
• 状況: 完了
• コンポーネント: General
• マイルストーン: 3.2.0
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: 1

詳細

Freeciv code often checks if unit tech req is A_NEVER, indicating that the unit isn't in the game at all.

That's relic from the time (obsolete at least 20 years) units were in a static array with fixed order, and those in the middle had to be marked as removed that way. With the current dynamically filled array we don't insert such units at all, and those checks are obsolete.

Get rid of them. Such a use of unit required_advance is also one of the bigger remaining blockers against #44927

チケットの履歴 (5 件中 3 件表示)

2022-07-05 11:21 更新者: cazfi

• 新しいチケット "Drop checks whether unit should be in game" が作成されました

2022-07-16 20:49 更新者: cazfi

コメント

Concern regarding savegame compatibility came up for similar "Never" support of techs: https://osdn.net/projects/freeciv/ticket/45115#comment:12505:45115:1657972075

2022-07-19 02:25 更新者: cazfi

• 添付ファイル 0057-Drop-checks-if-a-valid-unit-isn-t-in-game.patch (File ID: 9844) が付加されました

2022-07-19 02:25 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました

コメント

Reply To cazfi

Concern regarding savegame compatibility came up for similar "Never" support of techs: https://osdn.net/projects/freeciv/ticket/45115#comment:12505:45115:1657972075

In case of units, it was already impossible to get such units to the game (prevented ruleset load time), and there are flags that can be used to mark unit as unbuildable etc.

2022-07-28 11:34 更新者: cazfi

• 状況 が オープン から 完了 に更新されました
• 解決法 が 受領 から 修正済み に更新されました