Ticket #40340

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

Open Date: 2020-04-19 22:30 Last Update: 2021-01-15 23:17

Reporter:
Owner:
Status:
Closed
Component:
(None)
Priority:
6
Severity:
6
Resolution:
Fixed
File:
None

Details

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

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

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

Ticket History (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
Comment

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

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

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

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

2021-01-15 23:17 Updated by: deskull
  • Status Update from Open to Closed

Attachment File List

No attachments

Edit

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Login