チケット #43357

Configure check for proper size_t printf() format

登録: 2021-12-09 10:48 最終更新: 2021-12-24 17:48

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

詳細

For better support of using size_t variables in the printf() family of functions, add a configure check for determining which kind of specifier should be used for them. This ticket is about autotools implementation, though care should be taken not to completely break meson build.

As part of #43200, this has also been posted to http://forum.freeciv.org/f/viewtopic.php?f=14&t=92093

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

2021-12-09 10:48 更新者: cazfi
  • 新しいチケット "Confgure check for proper size_t printf() format" が作成されました
2021-12-09 10:52 更新者: cazfi
  • 解決法なし から 受領 に更新されました
2021-12-09 11:00 更新者: cazfi
  • 概要が更新されました
2021-12-09 12:01 更新者: cazfi
  • 詳細が更新されました
2021-12-13 00:38 更新者: cazfi
  • 解決法受領 から なし に更新されました
コメント

That "%ldd" should be "%lld"

2021-12-13 06:22 更新者: cazfi
  • 解決法なし から 受領 に更新されました
コメント

- New version corrects "%lld"

2021-12-13 13:12 更新者: cazfi
  • 解決法受領 から なし に更新されました
コメント

This was a great pain to debug in msys2, but it turns out that %zu is valid format specifier for printf() (which we used in configure test), but not for our function declared with attribute((format(printf, ...)))

I have other fixes to this as well. Will submit a new version later.

2021-12-16 12:55 更新者: cazfi
  • 解決法なし から 受領 に更新されました
コメント

New patch version
- Make sure that the build test has -Wall and -Werror set so that it fails for specifiers for which it should
- Added "checking..." outputs
- In the compilation test, call function defined with attribute((format(printf, ...))) instead of printf()

2021-12-20 14:00 更新者: cazfi
  • 解決法受領 から なし に更新されました
コメント

Crosser based build fails with the current patch version. It would expect "%I64d"

2021-12-20 21:20 更新者: cazfi
  • 解決法なし から 受領 に更新されました
コメント

Support for "%I64d" added in latest version

2021-12-24 17:48 更新者: cazfi
  • 状況オープン から 完了 に更新されました
  • 担当者(未割り当て) から cazfi に更新されました
  • 解決法受領 から 修正済み に更新されました

編集

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