capability.c function headers to say 'TRUE' instead of 'true'

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

詳細

When referring to value of type bool, capability.c function headers should say 'TRUE' instead of 'true'

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

2021-05-09 20:54 更新者: cazfi

• 新しいチケット "capability.c function headers to say 'TRUE' instead of 'true'" が作成されました

2022-07-19 05:52 更新者: dark-ether

• 添付ファイル 0001-change-function-explaining-comments-to-contain-TRUE-.patch (File ID: 9848) が付加されました

2022-07-19 05:55 更新者: dark-ether

コメント

i was looking on old tickets to see if any interested me and this seemed too easy to just leave undone.

I assume you did this ticket to generate discussion, in my opinion we should be consistent with style and leave all function headers in all caps when they refer to the macros

2022-07-19 15:44 更新者: cazfi

コメント

Reply To dark-ether

in my opinion we should be consistent with style and leave all function headers in all caps when they refer to the macros

It's not only a style issue, but the fact that 'TRUE' (the macro) is not the same thing as boolean concept (or C++ / newer C boolean value) 'true'.

Typically I open this kind of tickets when I'm working on something else that I don't want to interrupt (maybe also having modifications for that work in my source tree) and don't want to forget what I've noticed.

Likely there's similar issues in other headers too.

2022-07-25 09:30 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が (未割り当て) から 3.1.0 (完了済み) に更新されました

コメント

The patch does not apply to S3_0. Just targeting the ticket to S3_1 instead of waiting for separate S3_0 patch.

2022-07-27 07:24 更新者: cazfi

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