switch case文におけるfall through関連の整理

• 报告人: habu
• 属主: habu
• 类型: リファクタリング
• 状态: 关闭
• 组件: (无)
• 里程碑: 変愚蛮怒3.0.0Alpha1リリース (关闭)
• 优先: 6
• 严重性: 6
• 处理结果: Fixed
• 文件: 无

Details

switch case文でbreak;のつけ忘れによる意図しないfall throughを防ぐため、gccでコンパイルするときは-Wextraも付加する。（全体としては、"-Wall -Wextra -Wno-format-overflow"になる）

しかし、変愚蛮怒では歴史的にfall throughによるテクニックが多用されているため、意図した箇所でも警告が多量に発生してしまう。コメントに /* Fall through */ と追加すると、この警告を抑制できるし、また意図したfall throughである事も明確となるので、該当箇所にはすべて /* Fall through */ コメントをつける。

さらにその過程で本当に意図しないfall throughが見つかれば修正する。

任务单历史 (3/4 Histories)

2020-04-19 22:30 Updated by: habu

• New Ticket "switch case文におけるfall through関連の整理" created

2021-01-02 20:16 Updated by: deskull

• 状态 Update from 开启 to 关闭
• 里程碑 Update from (无) to 変愚蛮怒3.0.0alphaリリース (关闭)
• 处理结果 Update from 无 to Fixed

评论

一通り施してあることを確認しましたので、一度チケット締め切ります。コーディング規約にこの件追加しておきます。

2021-01-15 23:04 Updated by: habu

• 里程碑 Update from 変愚蛮怒3.0.0alphaリリース (关闭) to 変愚蛮怒3.0.0Alpha1リリース (关闭)
• 状态 Update from 关闭 to 开启

评论

前回の修正以降にいくつかやらかしが発生したようなので、修正しました。マージお願いします。

https://osdn.net/projects/hengband/scm/git/hengband/commits/30724d8cf4ee3d49b48c95aba70ec6c8eaec0110

2021-01-15 23:17 Updated by: deskull

• 状态 Update from 开启 to 关闭

评论

https://osdn.net/projects/hengband/scm/git/hengband/commits/b91de96569cbff3e759c909540fdd7a74496e925 マージしました。お疲れ様です。