can_unit_do_activity_targeted_at(): ACTIVITY_PILLAGE case may fall through

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

詳細

With -Wimplicit-fallthrough in master:

../../../src/common/unit.c: In function ‘can_unit_do_activity_targeted_at’:
../../../src/common/unit.c:913:5: error: this statement may fall through -Werror=implicit-fallthrough=

913 | switch (action_get_target_kind(paction)) { \

| ^~

../../../src/common/unit.c:913:5: note: in definition of macro ‘RETURN_IS_ACTIVITY_ENABLED_UNIT_ON’

913 | switch (action_get_target_kind(paction)) { \

| ^~

../../../src/common/unit.c:1019:3: note: here

1019 | case ACTIVITY_EXPLORE:

| ^~

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

2021-09-19 14:41 更新者: cazfi

• 新しいチケット "can_unit_do_activity_targeted_at(): ACTIVITY_EXPLORE case may fall through" が作成されました

2021-09-23 11:23 更新者: cazfi

• 概要が更新されました

コメント

It's the ACTIVITY_PILLAGE case above that is falling through *to* ACTIVITY_EXPLORE.

The problem is in macro RETURN_IS_ACTIVITY_ENABLED_UNIT_ON() that has no return for invalid value.

2021-09-23 11:29 更新者: cazfi

• 添付ファイル 0031-RETURN_IS_ACTIVITY_ENABLED_UNIT_ON-Return-even-when-.patch (File ID: 7519) が付加されました

2021-09-23 11:29 更新者: cazfi

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

2021-10-01 13:27 更新者: cazfi

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