give_shared_vision() tripled map_get_own_seen() call

• 報告者: cazfi
• 担当者: cazfi
• チケットの種類: バグ
• 状況: 完了
• コンポーネント: Server
• マイルストーン: 3.1.0-beta4 (完了済み)
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: 2

詳細 返信

give_shared_vision() has

              V_RADIUS(map_get_own_seen(pplayer, ptile, V_MAIN),
                       map_get_own_seen(pplayer, ptile, V_INVIS),
                       map_get_own_seen(pplayer, ptile, V_SUBSURFACE));

That is, exactly same vision base is calculated three times. We recently fixed similar case in unittools.c, so functions supporting optimization already exist.

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

2023-06-10 12:06 更新者: cazfi

• 新しいチケット "give_shared_vision() tripled map_get_own_seen() call" が作成されました

2023-08-29 19:47 更新者: cazfi

• マイルストーン が 3.1.0-beta3 (完了済み) から 3.1.0-beta4 (完了済み) に更新されました

2023-09-25 06:26 更新者: cazfi

• 添付ファイル 0049-Avoid-tripled-map_get_player_tile-calls-on-shared-vi.patch (File ID: 13315) が付加されました

2023-09-25 06:26 更新者: cazfi

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

2023-09-25 07:37 更新者: cazfi

• 解決法 が 受領 から なし に更新されました

コメント 返信

clang:

../../../src/server/maphand.c:1137:19: error: unused function 'map_get_own_seen' [-Werror,-Wunused-function]
static inline int map_get_own_seen(const struct player *pplayer,

2023-09-25 07:49 更新者: cazfi

• 添付ファイル 0049-Avoid-tripled-map_get_player_tile-calls-on-shared-vi.patch (File ID: 13316) が付加されました

2023-09-25 08:01 更新者: cazfi

• 解決法 が なし から 受領 に更新されました

コメント 返信

Reply To cazfi

clang: {{{ ../../../src/server/maphand.c:1137:19: error: unused function 'map_get_own_seen' -Werror,-Wunused-function static inline int map_get_own_seen(const struct player *pplayer, }}}

Fixed in attached version

2023-10-01 05:49 更新者: cazfi

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