チケット #44882

citizenshand.c doing useless work when FREECIV_DEBUG not set

登録: 2022-06-20 23:00 最終更新: 2022-06-25 17:12

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

詳細

../../../src/server/citizenshand.c:159:18: warning: variable 'pplayer' set but not used [-Wunused-but-set-variable!]

That's basically because pplayer would only be used in log_citizens(), which currently is defined as log_debug(), which expands to nothing when FREECIV_DEBUG is not set.

Ideally the fix would support changing log_citizens() to be something else than log_debug()

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

2022-06-20 23:00 更新者: cazfi
  • 新しいチケット "citizenshand.c doing useless work when FREECIV_DEBUG not set" が作成されました
2022-06-23 01:29 更新者: cazfi
  • 優先度5 - 中 から 9 - 最高 に更新されました
コメント

Reply To cazfi

../../../src/server/citizenshand.c:159:18: warning: variable 'pplayer' set but not used [-Wunused-but-set-variable!]

That's a build failure on CI now. To be resolved as an emergency fix.

2022-06-23 02:03 更新者: cazfi
  • 担当者(未割り当て) から cazfi に更新されました
  • 解決法なし から 受領 に更新されました
  • マイルストーン(未割り当て) から 3.1.0 に更新されました
コメント

Fix to immediate build issues (as quickly reproduced - can't promise there isn't failing corner cases still) attached. Will push immediately.

2022-06-23 02:31 更新者: cazfi
コメント

Further adjustment, going to push that one too (to S3_1 and master).

2022-06-23 03:00 更新者: cazfi
コメント

S3_0 patch attached. That one is not for an emergency, but has the usual review period.

All branches pass all eight combinations of debug/not, ndebug/not, LOG_CITIZENS_DEBUG/not

2022-06-23 08:03 更新者: cazfi
  • 優先度9 - 最高 から 5 - 中 に更新されました
2022-06-25 17:12 更新者: cazfi
  • 状況オープン から 完了 に更新されました
  • 解決法受領 から 修正済み に更新されました

編集

ログインしていません。ログインしていない状態では、コメントに記載者の記録が残りません。 » ログインする