チケット #44902

Cast int to double, compare to 0.01

登録: 2022-06-22 06:33 最終更新: 2023-02-22 12:40

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

詳細

I'm not sure what this code is supposed to do, but it looks wrong:

https://github.com/freeciv/freeciv/blob/master/client/text.c#L1555

get_seconds_to_new_turn() returns int, not double. Even then it would be a pretty stringent test (less than one frame worth of time).

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

2022-06-22 06:33 更新者: None
  • 新しいチケット "Cast int to double, compare to 0.01" が作成されました
2022-06-22 06:53 更新者: cazfi
コメント

Reply To (Anonymous)

get_seconds_to_new_turn() returns int, not double. Even then it would be a pretty stringent test (less than one frame worth of time).

Looking around where the get_seconds_to_new_turn() value comes, there seems to be more double/integer confusion (there's 'ceil() + 0.1' assigned to that integer...)

I think the point of that test is to avoid displaying negative values if the turn change takes longer than estimated (the estimate is just the lenght of the previous turn change)

2023-02-04 22:22 更新者: cazfi
  • マイルストーン(未割り当て) から 3.0.7 (完了済み) に更新されました
  • コンポーネント(未割り当て) から Client common に更新されました
2023-02-18 03:57 更新者: cazfi
  • 担当者(未割り当て) から cazfi に更新されました
  • 解決法なし から 受領 に更新されました
2023-02-22 12:40 更新者: cazfi
  • 状況オープン から 完了 に更新されました
  • 解決法受領 から 修正済み に更新されました

編集

このチケットにコメントを追加するには、ログインが必要です » ログインする